@nbakka/mcp-appium 2.0.91 → 2.0.92

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>
@@ -429,6 +433,7 @@
                 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) {

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);
@@ -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/package.json

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