refactor: update note handling in MCPAdapter to use note category

Author: tikazyq

packages/ai/src/__tests__/parsers/copilot.test.ts

@@ -76,7 +76,7 @@ describe('CopilotParser Integration Tests', () => {
   });
 
   describe('getWorkspaces', () => {
-    it.skipIf(!hasVSCodeData)('should return workspaces for valid application', async () => {
+    it.runIf(() => hasVSCodeData)('should return workspaces for valid application', async () => {
       const applications = await parser.getApplications();
 
       if (applications.length === 0) {
@@ -115,7 +115,7 @@ describe('CopilotParser Integration Tests', () => {
   });
 
   describe('getChatSessions', () => {
-    it.skipIf(!hasVSCodeData)('should return chat sessions for valid workspace', async () => {
+    it.runIf(() => hasVSCodeData)('should return chat sessions for valid workspace', async () => {
       const applications = await parser.getApplications();
 
       if (applications.length === 0) {
@@ -168,7 +168,7 @@ describe('CopilotParser Integration Tests', () => {
   });
 
   describe('parseChatSession', () => {
-    it.skipIf(!hasVSCodeData)('should parse individual chat session correctly', async () => {
+    it.runIf(() => hasVSCodeData)('should parse individual chat session correctly', async () => {
       const applications = await parser.getApplications();
 
       if (applications.length === 0) {
@@ -248,7 +248,7 @@ describe('CopilotParser Integration Tests', () => {
   });
 
   describe('Chat message extraction', () => {
-    it.skipIf(!hasVSCodeData)('should extract messages with correct structure', async () => {
+    it.runIf(() => hasVSCodeData)('should extract messages with correct structure', async () => {
       const applications = await parser.getApplications();
 
       if (applications.length === 0) {
@@ -353,7 +353,7 @@ describe('CopilotParser Integration Tests', () => {
   });
 
   describe('Performance and scalability', () => {
-    it.skipIf(!hasVSCodeData)('should handle large numbers of sessions efficiently', async () => {
+    it.runIf(() => hasVSCodeData)('should handle large numbers of sessions efficiently', async () => {
       const startTime = Date.now();
 
       const applications = await parser.getApplications();
@@ -391,7 +391,7 @@ describe('CopilotParser Integration Tests', () => {
   });
 
   describe('Data integrity and consistency', () => {
-    it.skipIf(!hasVSCodeData)('should maintain consistent data relationships', async () => {
+    it.runIf(() => hasVSCodeData)('should maintain consistent data relationships', async () => {
       const applications = await parser.getApplications();
 
       if (applications.length === 0) {
@@ -421,7 +421,7 @@ describe('CopilotParser Integration Tests', () => {
       }
     });
 
-    it.skipIf(!hasVSCodeData)('should have consistent timestamps', async () => {
+    it.runIf(() => hasVSCodeData)('should have consistent timestamps', async () => {
       const applications = await parser.getApplications();
 
       if (applications.length === 0) {

packages/ai/src/parsers/copilot.ts

@@ -51,7 +51,6 @@ export class CopilotParser extends BaseParser {
         const workspaceCount = Object.keys(workspaceMapping).length;
 
         if (workspaceCount > 0) {
-          console.log(chalk.green(`✓ Found VS Code ${appName} with ${workspaceCount} workspaces`));
           applications.push({
             id: appId,
             name: appName,
@@ -62,10 +61,15 @@ export class CopilotParser extends BaseParser {
           });
         }
       } catch (error) {
-        console.warn(`VS Code path not found: ${basePath}`);
+        // Skip inaccessible paths silently
       }
     }
 
+    if (applications.length > 0) {
+      const totalWorkspaces = applications.reduce((sum, app) => sum + (app.workspaceCount || 0), 0);
+      console.log(chalk.green(`✓ Found ${applications.length} VS Code installation(s) with ${totalWorkspaces} total workspaces`));
+    }
+
     return applications;
   }
 
@@ -115,7 +119,7 @@ export class CopilotParser extends BaseParser {
         }
       }
     } catch (error) {
-      console.error(chalk.red(`Failed to get workspaces for application ${applicationId}:`), error);
+      // Return empty array on error
     }
 
     return workspaces;
@@ -140,7 +144,6 @@ export class CopilotParser extends BaseParser {
       for (const sessionFile of sessionFiles) {
         const sessionId = sessionFile.split('/').pop()?.replace('.json', '');
         if (!sessionId) {
-          console.warn(chalk.yellow(`Session file without ID found: ${sessionFile}`));
           continue;
         }
         const session = await this.parseChatSession(applicationId, workspaceId, sessionId);
@@ -165,7 +168,6 @@ export class CopilotParser extends BaseParser {
   ): Promise<ChatSession | null> {
     const targetBasePath = this.getTargetBasePath(applicationId);
     if (!targetBasePath) {
-      console.error(chalk.red(`Application not found: ${applicationId}`));
       return null;
     }
     const filePath = resolve(
@@ -223,7 +225,6 @@ export class CopilotParser extends BaseParser {
         turns: this.extractChatTurns(sessionId, sessionData),
       };
 
-      console.info(chalk.blue(`✓ Parsed chat session ${actualSessionId} with ${(sessionData.requests || []).length} turns`));
       return session;
     } catch (error) {
       console.error(chalk.red(`Error parsing chat session ${sessionId}:`), error instanceof Error ? error.message : String(error));
@@ -278,7 +279,7 @@ export class CopilotParser extends BaseParser {
           stat(basePath);
           return basePath;
         } catch (error) {
-          console.warn(chalk.yellow(`VS Code path not found: ${basePath}`));
+          // Path not accessible, continue
         }
       }
     }

packages/mcp/src/adapters/mcp-adapter.ts

@@ -198,7 +198,7 @@ export class MCPAdapter {
       // Handle update with optional note
       if (args.note) {
         // Then add the note
-        await this.apiClient.addDevlogNote(args.id, args.note.content, 'progress');
+        await this.apiClient.addDevlogNote(args.id, args.note.content, args.note.category);
 
         return this.toStandardResponse(true, { id: args.id }, `Updated entry ${args.id} with note`);
       } else {