mcp-feedback-enhanced 0.1.46 → 0.1.64

package/dist/index.js

@@ -173,13 +173,29 @@ async function findExtensionForProjectAsync(projectPath) {
     if (servers.length === 0) {
         return null;
     }
-    // Strategy 1: Exact workspace match
-    for (const server of servers) {
-        if (server.workspaces?.includes(projectPath)) {
-            debug(`✓ Exact workspace match: port=${server.port}`);
-            return server;
+    // Get MCP server's CURSOR_TRACE_ID (if running in Cursor)
+    const myTraceId = process.env['CURSOR_TRACE_ID'] || '';
+    debug(`My CURSOR_TRACE_ID: ${myTraceId || '(not set)'}`);
+    // Strategy 0: CURSOR_TRACE_ID match (HIGHEST PRIORITY - same Cursor window)
+    if (myTraceId) {
+        const traceMatch = servers.find(s => s.cursorTraceId === myTraceId);
+        if (traceMatch) {
+            debug(`✓ CURSOR_TRACE_ID match: port=${traceMatch.port}, traceId=${myTraceId}`);
+            return traceMatch;
         }
     }
+    // Strategy 1: Exact workspace match + traceId filter if available
+    const workspaceMatches = servers.filter(s => s.workspaces?.includes(projectPath));
+    if (workspaceMatches.length === 1) {
+        debug(`✓ Exact workspace match (single): port=${workspaceMatches[0].port}`);
+        return workspaceMatches[0];
+    }
+    if (workspaceMatches.length > 1) {
+        // Multiple windows have this workspace - take most recent
+        const sorted = workspaceMatches.sort((a, b) => b.timestamp - a.timestamp);
+        debug(`✓ Exact workspace match (multiple, taking most recent): port=${sorted[0].port}`);
+        return sorted[0];
+    }
     // Strategy 2: Prefix match (project is inside a workspace)
     for (const server of servers) {
         for (const ws of server.workspaces || []) {
@@ -206,7 +222,7 @@ async function findExtensionForProjectAsync(projectPath) {
     debug(`✓ Using most recent server: port=${sorted[0].port}`);
     debug(`  Available servers:`);
     sorted.forEach(s => {
-        debug(`    - pid=${s.pid}, port=${s.port}, workspaces=${s.workspaces?.join(', ')}`);
+        debug(`    - pid=${s.pid}, port=${s.port}, traceId=${s.cursorTraceId}, workspaces=${s.workspaces?.join(', ')}`);
     });
     return sorted[0];
 }
@@ -217,11 +233,19 @@ function findExtensionForProject(projectPath) {
     const servers = getLiveServers();
     if (servers.length === 0)
         return null;
-    // Same matching logic but sync
-    for (const server of servers) {
-        if (server.workspaces?.includes(projectPath))
-            return server;
+    // Strategy 0: CURSOR_TRACE_ID match (highest priority)
+    const myTraceId = process.env['CURSOR_TRACE_ID'] || '';
+    if (myTraceId) {
+        const traceMatch = servers.find(s => s.cursorTraceId === myTraceId);
+        if (traceMatch)
+            return traceMatch;
+    }
+    // Strategy 1: Exact workspace match
+    const workspaceMatches = servers.filter(s => s.workspaces?.includes(projectPath));
+    if (workspaceMatches.length >= 1) {
+        return workspaceMatches.sort((a, b) => b.timestamp - a.timestamp)[0];
     }
+    // Strategy 2: Prefix match
     for (const server of servers) {
         for (const ws of server.workspaces || []) {
             if (projectPath.startsWith(ws + path.sep) || ws.startsWith(projectPath + path.sep)) {
@@ -229,9 +253,11 @@ function findExtensionForProject(projectPath) {
             }
         }
     }
+    // Strategy 3: parentPid match
     const parentMatch = servers.find(s => s.parentPid === process.ppid);
     if (parentMatch)
         return parentMatch;
+    // Fallbacks
     if (servers.length === 1)
         return servers[0];
     return servers.sort((a, b) => b.timestamp - a.timestamp)[0];
@@ -428,19 +454,29 @@ server.tool('interactive_feedback', 'Collect feedback from user through VSCode e
     project_directory: z.string().describe('The project directory path for context'),
     summary: z.string().describe('Summary of AI work completed for user review'),
     timeout: z.number().optional().default(600).describe('Timeout in seconds (default: 600)'),
-    agent_name: z.string().optional().describe('Display name for this agent/chat (e.g. "Chat 1", "Refactor Task")')
+    agent_name: z.string().optional().describe('Unique identifier for this agent/chat session. Use a stable name like the chat title or task description (e.g. "Chat 1", "Pre-credit Implementation", "Bug Fix #123"). If not provided, defaults to "Agent" and all messages will appear in the same tab. Providing a unique name allows multiple agents to have separate conversation tabs.')
 }, async ({ project_directory, summary, timeout, agent_name }) => {
     debug(`interactive_feedback called: project=${project_directory}, agent=${agent_name || 'default'}`);
     try {
         const result = await requestFeedback(project_directory, summary, timeout || 600, agent_name);
-        // Format response
-        let response = `User Feedback:\n${result.feedback}`;
+        // Build content array with text and images
+        const content = [{ type: 'text', text: `User Feedback:\n${result.feedback}` }];
+        // Add images as MCP image content items
         if (result.images && result.images.length > 0) {
-            response += `\n\n[${result.images.length} image(s) attached]`;
+            debug(`Processing ${result.images.length} image(s)`);
+            for (const img of result.images) {
+                // img is { name: string, data: string (base64) }
+                if (img && img.data) {
+                    content.push({
+                        type: 'image',
+                        data: img.data,
+                        mimeType: 'image/png' // Default to PNG; could infer from name
+                    });
+                    debug(`Added image: ${img.name || 'unnamed'}`);
+                }
+            }
         }
-        return {
-            content: [{ type: 'text', text: response }]
-        };
+        return { content };
     }
     catch (error) {
         debug(`Feedback error: ${error.message}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mcp-feedback-enhanced",
-  "version": "0.1.46",
+  "version": "0.1.64",
   "description": "MCP Feedback Enhanced Server - Interactive feedback collection for AI assistants",
   "type": "module",
   "main": "dist/index.js",
@@ -17,11 +17,11 @@
     "vscode",
     "claude"
   ],
-  "author": "Minidoracat",
+  "author": "yuanming.chen",
   "license": "MIT",
   "repository": {
     "type": "git",
-    "url": "https://github.com/Minidoracat/mcp-feedback-enhanced"
+    "url": "https://github.com/xxx/mcp-feedback-enhanced"
   },
   "scripts": {
     "build": "tsc",