@nbakka/mcp-appium 2.0.92 → 2.0.93

package/lib/review-ui/index.html

@@ -410,36 +410,36 @@
         }
     }
 
-    async function deleteTestCase(type, index) {
-        const confirmation = confirm("Are you sure you want to delete this test case?");
-        if (!confirmation) return;
+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 })
-            });
-
-            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}`);
+        }
 
-            // 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');
+        // 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

@@ -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) {

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.92",
+  "version": "2.0.93",
   "description": "Appium MCP",
   "engines": {
     "node": ">=18"