@nbakka/mcp-appium 2.0.90 → 2.0.91

package/lib/review-ui/index.html

@@ -67,7 +67,7 @@
             color: #666;
             font-size: 0.9em;
         }
-        .edit-btn, .save-btn, .cancel-btn-inline {
+        .edit-btn, .save-btn, .cancel-btn-inline, .delete-btn {
             padding: 6px 12px;
             border: none;
             border-radius: 4px;
@@ -87,6 +87,13 @@
             background-color: #757575;
             color: white;
         }
+        .delete-btn {
+            background-color: #f44336;
+            color: white;
+        }
+        .delete-btn:hover {
+            background-color: #d32f2f;
+        }
         .original-text {
             color: #999;
             text-decoration: line-through;
@@ -232,6 +239,7 @@
                     <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>
                     <div id="new-${index}-display" style="display: block;">
                         <strong>Description:</strong> ${tc.description}
@@ -259,6 +267,7 @@
                     <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>
                     <div id="modify-${index}-display" style="display: block;">
                         <div class="original-text"><strong>Original:</strong> ${tc.original}</div>
@@ -397,6 +406,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 })
+            });
+
+            if (!response.ok) {
+                throw new Error(`HTTP ${response.status}: ${response.statusText}`);
+            }
+
+            // 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);
+            }
+
+            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;
 </script>

package/lib/server.js

@@ -896,6 +896,30 @@ tool(
                 res.json({ status: 'cancelled', message: 'Review cancelled' });
             });
 
+            app.post(`/delete/${sessionId}`, (req, res) => {
+                const session = approvalSessions.get(sessionId);
+                if (!session) {
+                    return res.status(404).json({ error: 'Session not found' });
+                }
+
+                const { type, index } = req.body;
+
+                try {
+                    if (type === 'new' && session.testCases.new && index < session.testCases.new.length) {
+                        session.testCases.new.splice(index, 1);
+                    } else if (type === 'modify' && session.testCases.modify && index < session.testCases.modify.length) {
+                        session.testCases.modify.splice(index, 1);
+                    } else {
+                        return res.status(400).json({ error: 'Invalid type or index' });
+                    }
+
+                    approvalSessions.set(sessionId, session);
+                    res.json({ status: 'deleted', message: 'Test case deleted successfully' });
+                } catch (error) {
+                    res.status(500).json({ error: 'Failed to delete test case' });
+                }
+            });
+
             const server = app.listen(port, async () => {
                 // Remove console.log to prevent JSON parsing errors
                 try {
@@ -951,10 +975,10 @@ tool(
 
         const session = approvalSessions.get(sessionId);
         if (!session) {
-            return JSON.stringify({
+            return {
                 status: "error",
                 message: `Session not found. Invalid session ID: ${sessionId}`
-            });
+            };
         }
 
         const currentTime = Date.now();
@@ -968,41 +992,41 @@ tool(
             }
             approvalSessions.delete(sessionId);
 
-            return JSON.stringify({
+            return {
                 status: "approved",
                 message: "Test cases approved successfully!",
                 elapsedTime: elapsedTime,
                 approvedTestCases: approvedTestCases
-            });
+            };
         } else if (session.status === 'cancelled') {
             if (session.server) {
                 session.server.close();
             }
             approvalSessions.delete(sessionId);
 
-            return JSON.stringify({
+            return {
                 status: "cancelled",
                 message: "Review was cancelled by the user",
                 elapsedTime: elapsedTime
-            });
+            };
         } else if (session.status === 'timeout' || elapsedTime > 300) {
             if (session.server) {
                 session.server.close();
             }
             approvalSessions.delete(sessionId);
 
-            return JSON.stringify({
+            return {
                 status: "timeout",
                 message: "Review session timed out",
                 elapsedTime: elapsedTime
-            });
+            };
         } else {
-            return JSON.stringify({
+            return {
                 status: "pending",
                 message: "Test cases are still pending approval",
                 elapsedTime: elapsedTime,
                 remainingTime: Math.max(0, 300 - elapsedTime)
-            });
+            };
         }
     }
 );

package/package.json

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