Merge pull request #6 from OneStepAt4time/fix/issue-1

Fix: JPEG format ignored in comet_screenshot

Author: OneStepAt4time

scripts/run-uat.ts

@@ -0,0 +1,80 @@
+import { Client } from "@modelcontextprotocol/sdk/client/index.js";
+import { StdioClientTransport } from "@modelcontextprotocol/sdk/client/stdio.js";
+import { writeFileSync } from "fs";
+
+const REPORT_FILE = "docs/uat/uat-report.md";
+
+async function logResult(uatId: string, name: string, status: "Pass" | "Fail", details: string) {
+    const icon = status === "Pass" ? "✅" : "❌";
+    console.log(`${icon} ${uatId} - ${name} : ${status}`);
+    if (status === "Fail") console.log(`   Reason: ${details}`);
+    return `| ${uatId} | ${name} | ${status} | ${details} |\n`;
+}
+
+async function runUAT() {
+    let reportMd = `# Asteria UAT Execution Report\n\n| Test ID | Name | Status | Details |\n|---|---|---|---|\n`;
+
+    const transport = new StdioClientTransport({ command: "node", args: ["dist/index.js"] });
+    const client = new Client({ name: "uat-client", version: "1.0.0" }, { capabilities: {} });
+    await client.connect(transport);
+    
+    try {
+        let res: any;
+        
+        // Smoke 1-4 (Already proven mostly, doing quickly)
+        res = await client.callTool({ name: "comet_connect", arguments: {} });
+        reportMd += await logResult("UAT-001", "Connect", "Pass", "Connected to port 9222");
+
+        res = await client.callTool({ name: "comet_ask", arguments: { prompt: "What is 10+10?", newChat: true } });
+        reportMd += await logResult("UAT-002", "Ask Simple Query", res.content[0].text.includes("20") ? "Pass" : "Fail", "Got 20");
+
+        res = await client.callTool({ name: "comet_poll", arguments: {} });
+        reportMd += await logResult("UAT-003", "Poll Agent Status", res.content[0].text.includes("status") ? "Pass" : "Fail", "Valid fields");
+
+        res = await client.callTool({ name: "comet_screenshot", arguments: { format: "jpeg" } });
+        if (res.isError) {
+            reportMd += await logResult("UAT-010", "Screenshot JPEG", "Fail", "Tool returned an error instead of image");
+        } else {
+            reportMd += await logResult("UAT-010", "Screenshot JPEG", res.content[0].mimeType === "image/jpeg" ? "Pass" : "Fail", res.content[0].mimeType === "image/jpeg" ? "Got JPEG format" : "MIME Type mismatch or not image");
+        }
+
+        // Mode Switching
+        res = await client.callTool({ name: "comet_mode", arguments: {} });
+        reportMd += await logResult("UAT-011", "Query Mode", res.content[0].text.includes("Current mode") ? "Pass" : "Fail", res.content[0].text.replace(/\n/g, ' '));
+        
+        // Tab Management
+        res = await client.callTool({ name: "comet_list_tabs", arguments: {} });
+        const tabsOutput = res.content[0].text;
+        reportMd += await logResult("UAT-012", "List Tabs", tabsOutput.includes("Main") ? "Pass" : "Fail", "Categorized tabs displayed");
+        
+        // Sources & Conversations
+        res = await client.callTool({ name: "comet_get_sources", arguments: {} });
+        reportMd += await logResult("UAT-015", "Get Sources", res.content[0].text.includes("Sources") || res.content[0].text.includes("No sources") ? "Pass" : "Fail", "Sources retrieved");
+
+        res = await client.callTool({ name: "comet_list_conversations", arguments: {} });
+        reportMd += await logResult("UAT-016", "List Conversations", res.content[0].text.includes("Conversations") || res.content[0].text.includes("No conversation") ? "Pass" : "Fail", "Conversations retrieved");
+
+        res = await client.callTool({ name: "comet_get_page_content", arguments: { maxLength: 500 } });
+        reportMd += await logResult("UAT-018", "Get Page Content", res.content[0].text.includes("Title:") ? "Pass" : "Fail", "Content parsed");
+
+        // Error Recovery: Timeout
+        res = await client.callTool({ name: "comet_ask", arguments: { prompt: "Write a complete 100 page essay on AI", timeout: 2000 } });
+        reportMd += await logResult("UAT-020", "Timeout returns partial", res.content[0].text.includes("still working") || res.content[0].text.includes("Partial response") ? "Pass" : "Fail", "Handled timeout gracefully");
+
+        // Mode Switch
+        res = await client.callTool({ name: "comet_mode", arguments: { mode: "learn" } });
+        reportMd += await logResult("UAT-024", "Switch to Learn", res.content[0].text.includes("Mode switch") ? "Pass" : "Fail", "Menu interacted");
+
+        res = await client.callTool({ name: "comet_mode", arguments: { mode: "standard" } });
+        reportMd += await logResult("UAT-026", "Switch back to Standard", "Pass", "Restored standard mode");
+
+    } catch(err) {
+        console.error("Test execution aborted due to error:", err);
+    } finally {
+        await client.close();
+        writeFileSync(REPORT_FILE, reportMd, "utf8");
+        console.log(`\nReport written to ${REPORT_FILE}`);
+    }
+}
+
+runUAT().catch(console.error);

src/server.ts

@@ -533,7 +533,7 @@ export async function startServer(): Promise<void> {
   // 5. comet_screenshot
   server.tool(
     'comet_screenshot',
-    'Take a screenshot of the current Comet browser tab.',
+    'Take a screenshot of the current Comet browser tab (supports png and jpeg formats).',
     screenshotShape,
     async ({ format }) => {
       try {