@nbakka/mcp-appium 2.0.87 → 2.0.88

package/lib/server.js

@@ -822,9 +822,9 @@ tool(
                 if (tc.length === 2 && tc[1].toLowerCase() === 'new') {
                     // Format: [description, "New"]
                     parsedTestCases.new.push(tc[0]);
-                } else if (tc.length === 4) {
+                } else if (tc.length >= 3) {
                     const type = tc[2]?.toLowerCase();
-                    if (type === 'modify') {
+                    if (type === 'modify' && tc.length >= 4) {
                         // Format: [originalDescription, newDescription, "Modify", testCaseId]
                         parsedTestCases.modify.push({
                             id: tc[3] || 'N/A',
@@ -832,12 +832,18 @@ tool(
                             modified: tc[1]  // New/modified description
                         });
                     } else if (type === 'remove') {
-                        // Format: [description, "", "Remove", testCaseId]
+                        // Format: [description, "Remove", testCaseId] or [description, "", "Remove", testCaseId]
                         parsedTestCases.remove.push({
-                            id: tc[3] || 'N/A',
+                            id: tc.length >= 4 ? tc[3] : tc[1], // testCaseId could be in position 1 or 3
                             description: tc[0]
                         });
                     }
+                } else if (tc.length === 3 && tc[1].toLowerCase() === 'remove') {
+                    // Handle format: [description, "Remove", testCaseId]
+                    parsedTestCases.remove.push({
+                        id: tc[2] || 'N/A',
+                        description: tc[0]
+                    });
                 }
             });
 
@@ -922,20 +928,14 @@ tool(
                 }
             }, 300000); // 5 minutes
 
-            return JSON.stringify({
-                status: "review_started",
-                sessionId: sessionId,
-                message: "Test case review interface opened in browser. Use check_approval_status tool to poll for approval.",
-                testCasesCount: testCases.length,
-                browserUrl: `http://localhost:${port}`,
-                instructions: "Poll every 25 seconds using check_approval_status tool until approved or timeout (5 minutes)"
-            });
+            return `✅ Test case review interface opened in browser.
+Session ID: ${sessionId}
+Test Cases Count: ${testCases.length}
+Browser URL: http://localhost:${port}
+Instructions: Use check_approval_status tool to poll for approval every 25 seconds.`;
 
         } catch (err) {
-            return JSON.stringify({
-                status: "error",
-                message: `Error setting up review interface: ${err.message}`
-            });
+            return `❌ Error setting up review interface: ${err.message}`;
         }
     }
 );
@@ -950,10 +950,7 @@ tool(
         await new Promise(resolve => setTimeout(resolve, 25000));
         const session = approvalSessions.get(sessionId);
         if (!session) {
-            return JSON.stringify({
-                status: "error",
-                message: "Session not found. Invalid session ID."
-            });
+            return `❌ Session not found. Invalid session ID: ${sessionId}`;
         }
 
         const currentTime = Date.now();
@@ -968,12 +965,9 @@ tool(
             }
             approvalSessions.delete(sessionId);
 
-            return JSON.stringify({
-                status: "approved",
-                message: "Test cases have been approved by the user.",
-                approvedTestCases: approvedTestCases,
-                elapsedTime: elapsedTime
-            });
+            return `✅ Test cases approved successfully!
+Elapsed time: ${elapsedTime} seconds
+Approved test cases: ${JSON.stringify(approvedTestCases, null, 2)}`;
         } else if (session.status === 'cancelled') {
             // Clean up session and close server
             if (session.server) {
@@ -981,11 +975,7 @@ tool(
             }
             approvalSessions.delete(sessionId);
 
-            return JSON.stringify({
-                status: "cancelled",
-                message: "Review was cancelled by the user.",
-                elapsedTime: elapsedTime
-            });
+            return `❌ Review was cancelled by the user. Elapsed time: ${elapsedTime} seconds`;
         } else if (session.status === 'timeout' || elapsedTime > 300) { // 5 minutes
             // Clean up session and close server
             if (session.server) {
@@ -993,18 +983,12 @@ tool(
             }
             approvalSessions.delete(sessionId);
 
-            return JSON.stringify({
-                status: "timeout",
-                message: `Review session timed out. Elapsed time: ${elapsedTime} seconds`,
-                elapsedTime: elapsedTime
-            });
+            return `⏰ Review session timed out. Elapsed time: ${elapsedTime} seconds`;
         } else {
-            return JSON.stringify({
-                status: "pending",
-                message: "Test cases are still pending approval. Continue polling.",
-                elapsedTime: elapsedTime,
-                remainingTime: Math.max(0, 300 - elapsedTime)
-            });
+            return `⏳ Test cases are still pending approval.
+Elapsed time: ${elapsedTime} seconds
+Remaining time: ${Math.max(0, 300 - elapsedTime)} seconds
+Continue polling with check_approval_status tool.`;
         }
     }
 );

package/package.json

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