more

cleanup

.gitignore

@@ -21,4 +21,4 @@ lerna-debug.log*
 *.sln
 *.sw?
 
-backend/.secrets.js
+backend/.env

README.md

@@ -24,12 +24,12 @@ unzip kongruity.zip
 cd kongruity
 ```
 
-### 2. Create a secrets file
+### 2. Create an environment file
 
-The backend expects a `.secrets.js` file containing an Anthropic API key in the root `backend/` directory. This file is git-ignored and must be created manually:
+The backend expects a `.env` file containing an Anthropic API key in the root `backend/` directory. This file is git-ignored and must be created manually:
 
 ```bash
-echo 'export const anthropicApiKey = "<your Anthropic API key>"' > backend/.secrets.js
+echo 'ANTHROPIC_API_KEY=<your Anthropic API key>' > backend/.env
 ```
 
 Replace `<your Anthropic API key>` with your actual key.

backend/app.js

@@ -11,6 +11,6 @@ app.use('/v1/notes', router);
 
 app.use((req, res) => {
    res.status(404).json({ error: `Requested path is invalid or does not exist: ${req.method} ${req.originalUrl}` });
- });
+});
 
 export default app;

backend/routes/notes.routes.js

@@ -1,14 +1,9 @@
 import { Router } from 'express';
 import { readFile } from 'fs/promises';
-import { fileURLToPath } from 'url';
-import { dirname, join } from 'path';
 import { clusterNotes } from '../services/clustering.service.js';
 
 const router = Router();
-
+const DATA_PATH = '../data/notes.json'
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = dirname(__filename);
-const DATA_PATH = join(__dirname, '..', 'data', 'notes.json');
 
 const loadNotes = async () => {
   const raw = await readFile(DATA_PATH, 'utf-8');

backend/server.js

@@ -1,3 +1,4 @@
+import 'dotenv/config';
 import app from './app.js';
 
 const PORT = process.env.PORT || 3001;

backend/services/clustering.service.js

@@ -1,8 +1,7 @@
 import Anthropic from "@anthropic-ai/sdk";
-import { anthropicApiKey } from "../.secrets.js";
 
 const client = new Anthropic({
-  apiKey: anthropicApiKey,
+  apiKey: process.env.ANTHROPIC_API_KEY,
 });
 
 const buildPrompt = (notes) => {
@@ -41,6 +40,15 @@ export const clusterNotes = async (notes) => {
    ],
   });
 
-  const raw = response.content[0].text;
+  const textBlock = response?.content?.[0];
-  return JSON.parse(raw);
+  
+  if (!textBlock || textBlock.type !== 'text' || typeof textBlock.text !== 'string') {
+    throw new Error('Unexpected response from LLM API: no text content returned');
+  }
+
+  try {
+    return JSON.parse(textBlock.text);
+  } catch {
+    throw new Error('LLM API returned non-JSON response');
+  }
 };

backend/tests/Clustering.service.test.js

@@ -14,10 +14,6 @@ vi.mock('@anthropic-ai/sdk', () => {
   };
 });
 
-vi.mock('../.secrets.js', () => ({
-  anthropicApiKey: 'test-key-not-real',
-}));
-
 import { clusterNotes } from '../services/clustering.service.js';
 
 const MOCK_NOTES = [
@@ -39,7 +35,7 @@ describe('clusterNotes service', () => {
 
   it('should call Anthropic messages.create with the correct model', async () => {
     createMock.mockResolvedValue({
-      content: [{ text: JSON.stringify(MOCK_CLUSTERS) }],
+      content: [{ type: 'text', text: JSON.stringify(MOCK_CLUSTERS) }],
     });
 
     await clusterNotes(MOCK_NOTES);
@@ -52,7 +48,7 @@ describe('clusterNotes service', () => {
 
   it('should include all note texts in prompt sent to the LLM API', async () => {
     createMock.mockResolvedValue({
-      content: [{ text: JSON.stringify(MOCK_CLUSTERS) }],
+      content: [{ type: 'text', text: JSON.stringify(MOCK_CLUSTERS) }],
     });
 
     await clusterNotes(MOCK_NOTES);
@@ -66,7 +62,7 @@ describe('clusterNotes service', () => {
 
   it('should parse and return the clustered JSON from the API response', async () => {
     createMock.mockResolvedValue({
-      content: [{ text: JSON.stringify(MOCK_CLUSTERS) }],
+      content: [{ type: 'text', text: JSON.stringify(MOCK_CLUSTERS) }],
     });
 
     const result = await clusterNotes(MOCK_NOTES);
@@ -76,10 +72,16 @@ describe('clusterNotes service', () => {
 
   it('should throw error when the API returns non-JSON', async () => {
     createMock.mockResolvedValue({
-      content: [{ text: 'An unknown error occured when generting structured response.' }],
+      content: [{ type: 'text', text: 'An unknown error occured when generting structured response.' }],
     });
 
-    await expect(clusterNotes(MOCK_NOTES)).rejects.toThrow();
+    await expect(clusterNotes(MOCK_NOTES)).rejects.toThrow('non-JSON response');
+  });
+
+  it('should throw error when the API response has no text content', async () => {
+    createMock.mockResolvedValue({ content: [] });
+
+    await expect(clusterNotes(MOCK_NOTES)).rejects.toThrow('no text content returned');
   });
 
   it('should throw error when Anthropic API authentication fails', async () => {