@nbakka/mcp-appium 2.0.86 → 2.0.88

package/lib/server.js

@@ -817,21 +817,32 @@ tool(
             };
 
             testCases.forEach(tc => {
-                const [description, type, id] = tc;
-                const typeKey = type.toLowerCase();
-
-                if (typeKey === 'new') {
-                    parsedTestCases.new.push(description);
-                } else if (typeKey === 'modify') {
-                    parsedTestCases.modify.push({
-                        id: id || 'N/A',
-                        original: description,
-                        modified: description // Default to original, user can edit
-                    });
-                } else if (typeKey === 'remove') {
+                if (!tc || tc.length < 2) return; // Skip invalid entries
+
+                if (tc.length === 2 && tc[1].toLowerCase() === 'new') {
+                    // Format: [description, "New"]
+                    parsedTestCases.new.push(tc[0]);
+                } else if (tc.length >= 3) {
+                    const type = tc[2]?.toLowerCase();
+                    if (type === 'modify' && tc.length >= 4) {
+                        // Format: [originalDescription, newDescription, "Modify", testCaseId]
+                        parsedTestCases.modify.push({
+                            id: tc[3] || 'N/A',
+                            original: tc[0], // Original description
+                            modified: tc[1]  // New/modified description
+                        });
+                    } else if (type === 'remove') {
+                        // Format: [description, "Remove", testCaseId] or [description, "", "Remove", testCaseId]
+                        parsedTestCases.remove.push({
+                            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: id || 'N/A',
-                        description: description
+                        id: tc[2] || 'N/A',
+                        description: tc[0]
                     });
                 }
             });
@@ -871,7 +882,7 @@ tool(
                 const session = approvalSessions.get(sessionId);
                 if (session) {
                     session.status = 'approved';
-                    session.finalTestCases = req.body; // Store the final approved test cases
+                    session.finalTestCases = req.body;
                     approvalSessions.set(sessionId, session);
                 }
                 res.send("✓ Test cases approved successfully!");
@@ -917,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}`;
         }
     }
 );
@@ -945,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();
@@ -963,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) {
@@ -976,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) {
@@ -988,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.86",
+  "version": "2.0.88",
   "description": "Appium MCP",
   "engines": {
     "node": ">=18"