super-feedback-mcp 0.2.5 → 0.2.6

package/dist/index.js

@@ -39,7 +39,7 @@ async function callConvexAPI(endpoint, options) {
 // Create MCP server
 const server = new McpServer({
     name: "super-feedback",
-    version: "0.2.5",
+    version: "0.2.6",
 });
 // Tool 1: Get feedback summary (lightweight overview)
 server.registerTool("get_feedback_summary", {
@@ -137,8 +137,10 @@ Returns everything needed to work on the feedback:
 - All replies/thread history
 - Design suggestions (if any)
 - Cross-element references (if any)
+- Image attachment count (if any)
 
-Use this after get_feedback_summary to dive into a specific comment.`,
+Use this after get_feedback_summary to dive into a specific comment.
+If the comment has attached images, use get_comment_images to fetch and view them.`,
     inputSchema: {
         commentId: z.string().describe("The comment ID to get full details for"),
     },
@@ -207,46 +209,100 @@ Use this after get_feedback_summary to dive into a specific comment.`,
                 text += `\n  ${change.property}: ${change.oldValue} → ${change.newValue}`;
             });
         }
-        // Note if there are images attached
+        // Note if there are images attached (use get_comment_images to fetch them)
         if (c.imageUrls && c.imageUrls.length > 0) {
-            text += `\n\n🖼️ Attached Images: ${c.imageUrls.length} image(s)`;
+            text += `\n\n🖼️ Attached Images: ${c.imageUrls.length} image(s) - use get_comment_images("${c.id}") to view`;
+        }
+        return {
+            content: [{ type: "text", text }],
+            structuredContent: { found: true, comment: c },
+        };
+    }
+    catch (error) {
+        const message = error instanceof Error ? error.message : "Unknown error";
+        return {
+            content: [{ type: "text", text: `Error fetching comment details: ${message}` }],
+            structuredContent: { found: false, comment: null },
+            isError: true,
+        };
+    }
+});
+// Tool 3: Get images attached to a comment
+server.registerTool("get_comment_images", {
+    title: "Get Comment Images",
+    description: `Fetches and returns the images attached to a feedback comment.
+
+Use this when:
+- get_comment_details indicates a comment has attached images
+- You need to see visual context for the feedback (screenshots, mockups, etc.)
+- The user's feedback references something visual
+
+Returns the images as viewable content. Call this after get_comment_details 
+if you need to see the attached images.`,
+    inputSchema: {
+        commentId: z.string().describe("The comment ID to fetch images for"),
+    },
+    outputSchema: {
+        found: z.boolean(),
+        imageCount: z.number(),
+    },
+}, async ({ commentId }) => {
+    try {
+        const params = getAuthParams();
+        params.set("commentId", commentId);
+        const response = await callConvexAPI(`/api/feedback/ai/comment?${params.toString()}`);
+        if (!response.found || !response.comment) {
+            return {
+                content: [{ type: "text", text: `Comment ${commentId} not found` }],
+                structuredContent: { found: false, imageCount: 0 },
+            };
+        }
+        const c = response.comment;
+        if (!c.imageUrls || c.imageUrls.length === 0) {
+            return {
+                content: [{ type: "text", text: `No images attached to comment ${commentId}` }],
+                structuredContent: { found: true, imageCount: 0 },
+            };
         }
         // Fetch images and convert to base64 for MCP ImageContent
         const imageContents = [];
-        if (c.imageUrls && c.imageUrls.length > 0) {
-            for (const url of c.imageUrls) {
-                try {
-                    const imgResponse = await fetch(url);
-                    if (imgResponse.ok) {
-                        const buffer = await imgResponse.arrayBuffer();
-                        const base64 = Buffer.from(buffer).toString("base64");
-                        const mimeType = imgResponse.headers.get("content-type") || "image/png";
-                        imageContents.push({ type: "image", data: base64, mimeType });
-                    }
+        const failedUrls = [];
+        for (const url of c.imageUrls) {
+            try {
+                const imgResponse = await fetch(url);
+                if (imgResponse.ok) {
+                    const buffer = await imgResponse.arrayBuffer();
+                    const base64 = Buffer.from(buffer).toString("base64");
+                    const mimeType = imgResponse.headers.get("content-type") || "image/png";
+                    imageContents.push({ type: "image", data: base64, mimeType });
                 }
-                catch {
-                    // Skip images that fail to fetch
+                else {
+                    failedUrls.push(url);
                 }
             }
+            catch {
+                failedUrls.push(url);
+            }
         }
+        const summary = `🖼️ Images for comment ${commentId}: ${imageContents.length} of ${c.imageUrls.length} loaded successfully${failedUrls.length > 0 ? ` (${failedUrls.length} failed)` : ""}`;
         return {
             content: [
-                { type: "text", text },
+                { type: "text", text: summary },
                 ...imageContents,
             ],
-            structuredContent: { found: true, comment: c },
+            structuredContent: { found: true, imageCount: imageContents.length },
         };
     }
     catch (error) {
         const message = error instanceof Error ? error.message : "Unknown error";
         return {
-            content: [{ type: "text", text: `Error fetching comment details: ${message}` }],
-            structuredContent: { found: false, comment: null },
+            content: [{ type: "text", text: `Error fetching images: ${message}` }],
+            structuredContent: { found: false, imageCount: 0 },
             isError: true,
         };
     }
 });
-// Tool 3: Mark feedback as resolved
+// Tool 4: Mark feedback as resolved
 server.registerTool("mark_feedback_resolved", {
     title: "Mark Feedback Resolved",
     description: `Marks a feedback comment as resolved after you've fixed the issue.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "super-feedback-mcp",
-  "version": "0.2.5",
+  "version": "0.2.6",
   "description": "MCP server for SuperFeedback - enables AI agents to query and resolve client feedback",
   "type": "module",
   "main": "dist/index.js",

package/src/index.ts

@@ -116,7 +116,7 @@ async function callConvexAPI<T>(endpoint: string, options?: RequestInit): Promis
 // Create MCP server
 const server = new McpServer({
   name: "super-feedback",
-  version: "0.2.5",
+  version: "0.2.6",
 });
 
 // Tool 1: Get feedback summary (lightweight overview)
@@ -226,8 +226,10 @@ Returns everything needed to work on the feedback:
 - All replies/thread history
 - Design suggestions (if any)
 - Cross-element references (if any)
+- Image attachment count (if any)
 
-Use this after get_feedback_summary to dive into a specific comment.`,
+Use this after get_feedback_summary to dive into a specific comment.
+If the comment has attached images, use get_comment_images to fetch and view them.`,
     inputSchema: {
       commentId: z.string().describe("The comment ID to get full details for"),
     },
@@ -310,48 +312,114 @@ Use this after get_feedback_summary to dive into a specific comment.`,
         });
       }
 
-      // Note if there are images attached
+      // Note if there are images attached (use get_comment_images to fetch them)
       if (c.imageUrls && c.imageUrls.length > 0) {
-        text += `\n\n🖼️ Attached Images: ${c.imageUrls.length} image(s)`;
+        text += `\n\n🖼️ Attached Images: ${c.imageUrls.length} image(s) - use get_comment_images("${c.id}") to view`;
+      }
+
+      return {
+        content: [{ type: "text", text }],
+        structuredContent: { found: true, comment: c },
+      };
+    } catch (error) {
+      const message = error instanceof Error ? error.message : "Unknown error";
+      return {
+        content: [{ type: "text", text: `Error fetching comment details: ${message}` }],
+        structuredContent: { found: false, comment: null },
+        isError: true,
+      };
+    }
+  }
+);
+
+// Tool 3: Get images attached to a comment
+server.registerTool(
+  "get_comment_images",
+  {
+    title: "Get Comment Images",
+    description: `Fetches and returns the images attached to a feedback comment.
+
+Use this when:
+- get_comment_details indicates a comment has attached images
+- You need to see visual context for the feedback (screenshots, mockups, etc.)
+- The user's feedback references something visual
+
+Returns the images as viewable content. Call this after get_comment_details 
+if you need to see the attached images.`,
+    inputSchema: {
+      commentId: z.string().describe("The comment ID to fetch images for"),
+    },
+    outputSchema: {
+      found: z.boolean(),
+      imageCount: z.number(),
+    },
+  },
+  async ({ commentId }) => {
+    try {
+      const params = getAuthParams();
+      params.set("commentId", commentId);
+      
+      const response = await callConvexAPI<{ found: boolean; comment: FullCommentDetails | null }>(
+        `/api/feedback/ai/comment?${params.toString()}`
+      );
+      
+      if (!response.found || !response.comment) {
+        return {
+          content: [{ type: "text", text: `Comment ${commentId} not found` }],
+          structuredContent: { found: false, imageCount: 0 },
+        };
+      }
+      
+      const c = response.comment;
+      
+      if (!c.imageUrls || c.imageUrls.length === 0) {
+        return {
+          content: [{ type: "text", text: `No images attached to comment ${commentId}` }],
+          structuredContent: { found: true, imageCount: 0 },
+        };
       }
 
       // Fetch images and convert to base64 for MCP ImageContent
       const imageContents: Array<{ type: "image"; data: string; mimeType: string }> = [];
-      if (c.imageUrls && c.imageUrls.length > 0) {
-        for (const url of c.imageUrls) {
-          try {
-            const imgResponse = await fetch(url);
-            if (imgResponse.ok) {
-              const buffer = await imgResponse.arrayBuffer();
-              const base64 = Buffer.from(buffer).toString("base64");
-              const mimeType = imgResponse.headers.get("content-type") || "image/png";
-              imageContents.push({ type: "image", data: base64, mimeType });
-            }
-          } catch {
-            // Skip images that fail to fetch
+      const failedUrls: string[] = [];
+      
+      for (const url of c.imageUrls) {
+        try {
+          const imgResponse = await fetch(url);
+          if (imgResponse.ok) {
+            const buffer = await imgResponse.arrayBuffer();
+            const base64 = Buffer.from(buffer).toString("base64");
+            const mimeType = imgResponse.headers.get("content-type") || "image/png";
+            imageContents.push({ type: "image", data: base64, mimeType });
+          } else {
+            failedUrls.push(url);
           }
+        } catch {
+          failedUrls.push(url);
         }
       }
 
+      const summary = `🖼️ Images for comment ${commentId}: ${imageContents.length} of ${c.imageUrls.length} loaded successfully${failedUrls.length > 0 ? ` (${failedUrls.length} failed)` : ""}`;
+
       return {
         content: [
-          { type: "text", text },
+          { type: "text", text: summary },
           ...imageContents,
         ],
-        structuredContent: { found: true, comment: c },
+        structuredContent: { found: true, imageCount: imageContents.length },
       };
     } catch (error) {
       const message = error instanceof Error ? error.message : "Unknown error";
       return {
-        content: [{ type: "text", text: `Error fetching comment details: ${message}` }],
-        structuredContent: { found: false, comment: null },
+        content: [{ type: "text", text: `Error fetching images: ${message}` }],
+        structuredContent: { found: false, imageCount: 0 },
         isError: true,
       };
     }
   }
 );
 
-// Tool 3: Mark feedback as resolved
+// Tool 4: Mark feedback as resolved
 server.registerTool(
   "mark_feedback_resolved",
   {