@nbakka/mcp-appium 2.0.94 → 2.0.96

package/lib/server.js

@@ -801,7 +801,9 @@ tool(
   "review_testcases",
   "Open test cases in browser for manual approval.",
   {
-    testCases: z.string().array().array().describe("test cases array generated by tool generate_testcases_from_ticket_data")
+    testCases: zod_1.z
+      .array(zod_1.z.array(zod_1.z.string()))
+      .describe("test cases array generated by tool generate_testcases_from_ticket_data")
   },
   async ({ testCases }) => {
     try {
@@ -870,7 +872,6 @@ tool(
       app.post(`/approve/${sessionId}`, (req, res) => {
         const session = approvalSessions.get(sessionId);
         if (session) {
-          // overwrite authoritative copy with the submitted version
           session.testCases = req.body;
           session.finalTestCases = req.body;
           session.status = 'approved';
@@ -909,16 +910,17 @@ tool(
         }
       }, 300000);
 
-      return {
-        status: "review_started",
-        sessionId,
-        message: "Test case review interface opened.",
-        testCasesCount: testCases.length,
-        browserUrl: `http://localhost:${port}`,
-        instructions: "Use check_approval_status tool to poll."
-      };
+      // Return a simple string instead of an object
+      return `✅ Test case review interface opened successfully!
+Session ID: ${sessionId}
+Test Cases Count: ${testCases.length}
+Browser URL: http://localhost:${port}
+Status: review_started
+
+Instructions: Use check_approval_status tool with session ID "${sessionId}" to poll for approval status.`;
+
     } catch (err) {
-      return { status: "error", message: `Error setting up review interface: ${err.message}` };
+      return `❌ Error setting up review interface: ${err.message}`;
     }
   }
 );
@@ -927,7 +929,7 @@ tool(
 tool(
   "check_approval_status",
   "Check the approval status of test cases review session (short-circuits if already approved)",
-  { sessionId: z.string().describe("Session ID from review_testcases") },
+  { sessionId: zod_1.z.string().describe("Session ID from review_testcases") },
   async ({ sessionId }) => {
     const session = approvalSessions.get(sessionId);
     if (!session) return `❌ Session not found. Invalid session ID: ${sessionId}`;
@@ -942,7 +944,6 @@ tool(
       return `✅ Test cases approved. Elapsed: ${elapsed}s\n${JSON.stringify(approvedTestCases, null, 2)}`;
     }
 
-    // only wait if still pending
     await new Promise(r => setTimeout(r, 5000));
 
     const refreshed = approvalSessions.get(sessionId);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nbakka/mcp-appium",
-  "version": "2.0.94",
+  "version": "2.0.96",
   "description": "Appium MCP",
   "engines": {
     "node": ">=18"