@nbakka/mcp-appium 2.0.85 → 2.0.87

package/lib/server.js

@@ -799,9 +799,9 @@ let approvalSessions = new Map();
 
 tool(
     "review_testcases",
-    "Open test cases in browser for manual approval. Accepts test cases in format: [description, type, id?]",
+    "Open test cases in browser for manual approval.",
     {
-        testCases: zod_1.z.array(zod_1.z.array(zod_1.z.string())).describe("Array of test case arrays: [description, type, id?] where type is 'New', 'Modify', or 'Remove'")
+        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 {
@@ -817,22 +817,27 @@ 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') {
-                    parsedTestCases.remove.push({
-                        id: id || 'N/A',
-                        description: description
-                    });
+                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 === 4) {
+                    const type = tc[2]?.toLowerCase();
+                    if (type === 'modify') {
+                        // 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]
+                        parsedTestCases.remove.push({
+                            id: tc[3] || 'N/A',
+                            description: tc[0]
+                        });
+                    }
                 }
             });
 
@@ -871,7 +876,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!");
@@ -942,8 +947,8 @@ tool(
         sessionId: zod_1.z.string().describe("The session ID returned from review_testcases tool")
     },
     async ({ sessionId }) => {
+        await new Promise(resolve => setTimeout(resolve, 25000));
         const session = approvalSessions.get(sessionId);
-
         if (!session) {
             return JSON.stringify({
                 status: "error",

package/package.json

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