fix: Update timestamps in MCP API Adapter test entry and improve note handling in WorkspaceDevlogManager

Author: Marvin Zhang

.devlog/entries/271-fix-mcp-api-adapter-test-failures-11-failing-tests.json

@@ -7,7 +7,7 @@
   "status": "done",
   "priority": "high",
   "createdAt": "2025-07-24T09:37:37.634Z",
-  "updatedAt": "2025-07-24T09:45:48.068Z",
+  "updatedAt": "2025-07-24T09:54:53.090Z",
   "notes": [
     {
       "id": "1017bbc0-e67e-4482-83dc-b6d23d1738a9",
@@ -47,5 +47,5 @@
     "lastAIUpdate": "2025-07-24T09:37:37.634Z",
     "contextVersion": 1
   },
-  "closedAt": "2025-07-24T09:45:48.068Z"
+  "closedAt": "2025-07-24T09:54:53.090Z"
 }

packages/core/src/managers/devlog/workspace-devlog-manager.ts

@@ -629,18 +629,25 @@ export class WorkspaceDevlogManager {
       throw new Error(`Devlog ${id} not found`);
     }
 
+    // Add completion note first if summary is provided
+    if (summary) {
+      await this.addNote(id, `Completed: ${summary}`, 'progress');
+    }
+
+    // Get the updated entry (with note if added) and mark as completed
+    const entryWithNote = await this.getDevlog(id);
+    if (!entryWithNote) {
+      throw new Error(`Devlog ${id} not found after adding note`);
+    }
+
     const now = new Date().toISOString();
     const updated: DevlogEntry = {
-      ...existing,
+      ...entryWithNote,
       status: 'done',
       updatedAt: now,
       closedAt: now,
     };
 
-    if (summary) {
-      await this.addNote(id, `Completed: ${summary}`, 'progress');
-    }
-
     const provider = await this.getCurrentStorageProvider();
     await provider.save(updated);
 

packages/mcp/src/index.ts

@@ -232,7 +232,7 @@ async function main() {
     adapterInstance.setCurrentWorkspaceId(defaultWorkspace);
   }
 
-  // Assign the adapter instance directly (not Object.assign which doesn't copy methods)
+  // Assign the adapter instance directly
   adapter = adapterInstance;
 
   const transport = new StdioServerTransport();