@nbakka/mcp-appium 2.0.90 → 2.0.92

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;
@@ -228,10 +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>
+                        <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}
@@ -255,10 +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>
+                        <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>
@@ -397,6 +410,37 @@
         }
     }
 
+    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);
+            }
+
+            // 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;
 </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,7 @@ tool(
 
         const session = approvalSessions.get(sessionId);
         if (!session) {
-            return JSON.stringify({
-                status: "error",
-                message: `Session not found. Invalid session ID: ${sessionId}`
-            });
+            return `❌ Session not found. Invalid session ID: ${sessionId}`;
         }
 
         const currentTime = Date.now();
@@ -968,41 +989,24 @@ tool(
             }
             approvalSessions.delete(sessionId);
 
-            return JSON.stringify({
-                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 JSON.stringify({
-                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 JSON.stringify({
-                status: "timeout",
-                message: "Review session timed out",
-                elapsedTime: elapsedTime
-            });
+            return `⏰ Review session timed out. Elapsed time: ${elapsedTime}s`;
         } else {
-            return JSON.stringify({
-                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.90",
+  "version": "2.0.92",
   "description": "Appium MCP",
   "engines": {
     "node": ">=18"