@nbakka/mcp-appium 2.0.91 → 2.0.93

package/lib/review-ui/index.html

@@ -235,11 +235,13 @@
         const container = document.getElementById('new-cases');
         if (testCases.new && testCases.new.length > 0) {
             container.innerHTML = testCases.new.map((tc, index) => `
-                <div class="test-case">
+                <div class="test-case" id="new-case-${index}">
                     <div class="test-case-header">
                         <span class="test-case-id">ID: ${tc.id}</span>
-                        <button class="edit-btn" onclick="toggleEdit('new', ${index})">Edit</button>
-                        <button class="delete-btn" onclick="deleteTestCase('new', ${index})">Delete</button>
+                        <div>
+                            <button class="edit-btn" onclick="toggleEdit('new', ${index})">Edit</button>
+                            <button class="delete-btn" onclick="deleteTestCase('new', ${index})">Delete</button>
+                        </div>
                     </div>
                     <div id="new-${index}-display" style="display: block;">
                         <strong>Description:</strong> ${tc.description}
@@ -263,11 +265,13 @@
         const container = document.getElementById('modify-cases');
         if (testCases.modify && testCases.modify.length > 0) {
             container.innerHTML = testCases.modify.map((tc, index) => `
-                <div class="test-case">
+                <div class="test-case" id="modify-case-${index}">
                     <div class="test-case-header">
                         <span class="test-case-id">ID: ${tc.id}</span>
-                        <button class="edit-btn" onclick="toggleEdit('modify', ${index})">Edit</button>
-                        <button class="delete-btn" onclick="deleteTestCase('modify', ${index})">Delete</button>
+                        <div>
+                            <button class="edit-btn" onclick="toggleEdit('modify', ${index})">Edit</button>
+                            <button class="delete-btn" onclick="deleteTestCase('modify', ${index})">Delete</button>
+                        </div>
                     </div>
                     <div id="modify-${index}-display" style="display: block;">
                         <div class="original-text"><strong>Original:</strong> ${tc.original}</div>
@@ -406,35 +410,36 @@
         }
     }
 
-    async function deleteTestCase(type, index) {
-        const confirmation = confirm("Are you sure you want to delete this test case?");
-        if (!confirmation) return;
-
-        try {
-            showStatus('Deleting test case...', 'info');
-            const response = await fetch(`/delete/${sessionId}`, {
-                method: 'POST',
-                headers: { 'Content-Type': 'application/json' },
-                body: JSON.stringify({ type, index })
-            });
+async function deleteTestCase(type, index) {
+    const confirmation = confirm("Are you sure you want to delete this test case?");
+    if (!confirmation) return;
 
-            if (!response.ok) {
-                throw new Error(`HTTP ${response.status}: ${response.statusText}`);
-            }
+    try {
+        showStatus('Deleting test case...', 'info');
+        const response = await fetch(`/delete/${sessionId}`, {
+            method: 'POST',
+            headers: { 'Content-Type': 'application/json' },
+            body: JSON.stringify({ type, index })
+        });
 
-            // Remove the deleted test case from the local state
-            if (type === 'new') {
-                testCases.new.splice(index, 1);
-            } else if (type === 'modify') {
-                testCases.modify.splice(index, 1);
-            }
+        if (!response.ok) {
+            throw new Error(`HTTP ${response.status}: ${response.statusText}`);
+        }
 
-            renderTestCases();
-            showStatus('Test case deleted successfully!', 'success');
-        } catch (error) {
-            showStatus(`Error deleting test case: ${error.message}`, 'error');
+        // Remove the deleted test case from the local state first
+        if (type === 'new') {
+            testCases.new.splice(index, 1);
+        } else if (type === 'modify') {
+            testCases.modify.splice(index, 1);
         }
+
+        // Re-render the UI to update indexes and remove the element
+        renderTestCases();
+        showStatus('Test case deleted successfully!', 'success');
+    } catch (error) {
+        showStatus(`Error deleting test case: ${error.message}`, 'error');
     }
+}
 
     // Load test cases when page loads
     window.onload = loadTestCases;

package/lib/server.js

@@ -903,7 +903,7 @@ tool(
                 }
 
                 const { type, index } = req.body;
-
+                
                 try {
                     if (type === 'new' && session.testCases.new && index < session.testCases.new.length) {
                         session.testCases.new.splice(index, 1);
@@ -952,7 +952,7 @@ tool(
                 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)"
+                instructions: "Poll using check_approval_status tool continuously for  max 10 times"
             });
 
         } catch (err) {
@@ -975,10 +975,7 @@ tool(
 
         const session = approvalSessions.get(sessionId);
         if (!session) {
-            return {
-                status: "error",
-                message: `Session not found. Invalid session ID: ${sessionId}`
-            };
+            return `❌ Session not found. Invalid session ID: ${sessionId}`;
         }
 
         const currentTime = Date.now();
@@ -992,41 +989,24 @@ tool(
             }
             approvalSessions.delete(sessionId);
 
-            return {
-                status: "approved",
-                message: "Test cases approved successfully!",
-                elapsedTime: elapsedTime,
-                approvedTestCases: approvedTestCases
-            };
+            return `✅ Test cases approved successfully! Elapsed time: ${elapsedTime}s\n\nApproved test cases:\n${JSON.stringify(approvedTestCases, null, 2)}`;
         } else if (session.status === 'cancelled') {
             if (session.server) {
                 session.server.close();
             }
             approvalSessions.delete(sessionId);
 
-            return {
-                status: "cancelled",
-                message: "Review was cancelled by the user",
-                elapsedTime: elapsedTime
-            };
+            return `❌ Review was cancelled by the user. Elapsed time: ${elapsedTime}s`;
         } else if (session.status === 'timeout' || elapsedTime > 300) {
             if (session.server) {
                 session.server.close();
             }
             approvalSessions.delete(sessionId);
 
-            return {
-                status: "timeout",
-                message: "Review session timed out",
-                elapsedTime: elapsedTime
-            };
+            return `⏰ Review session timed out. Elapsed time: ${elapsedTime}s`;
         } else {
-            return {
-                status: "pending",
-                message: "Test cases are still pending approval",
-                elapsedTime: elapsedTime,
-                remainingTime: Math.max(0, 300 - elapsedTime)
-            };
+            const remainingTime = Math.max(0, 300 - elapsedTime);
+            return `⏳ Test cases are still pending approval. Elapsed: ${elapsedTime}s, Remaining: ${remainingTime}s`;
         }
     }
 );

package/lib/testcases-generation-context.txt

@@ -19,4 +19,6 @@ if any new test case is to be created then don't mention any id against it
 for review_testcases test cases tool data should be in this format, strictly follow this format espcially for Modify test cases
 {"testCases":[["Verify that brochure section is visible under overview tab","Verify brochure section is removed from property tour tab","Modify","SCRUM-TC-1"],
 ["Verify similar properties section is visible under over tab in pdp section","Remove","SCRUM-TC-2"],
-["Verify Project Brochure section appears above the 'Explore on map' section in Overview tab","New"]]}
+["Verify Project Brochure section appears above the 'Explore on map' section in Overview tab","New"]]}
+
+NOTE: keep polling continously for 10 times max till the test cases are approved by user

package/package.json

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