npm - @nbakka/mcp-appium - Versions diffs - 2.0.99 → 3.0.1 - Mend

@nbakka/mcp-appium 2.0.99 → 3.0.1

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (5) hide show

package/lib/server.js +493 -215
package/package.json +1 -1
package/lib/review-ui/index.html +0 -774
package/lib/review-ui/script.js +0 -176
package/lib/review-ui/styles.css +0 -244

package/lib/server.js CHANGED Viewed

@@ -794,275 +794,553 @@ tool(
   }
 );
-// Global state for tracking approval sessions
-let approvalSessions = new Map();
+// Tool 1: Review Test Cases - Launch HTML UI for approval
 tool(
   "review_testcases",
   "Open test cases in browser for manual approval.",
   {
-    testCases: zod_1.z
-      .array(zod_1.z.array(zod_1.z.string()))
-      .describe("test cases array generated by tool generate_testcases_from_ticket_data")
+    testCases: zod_1.z.array(zod_1.z.array(zod_1.z.string())).describe("test cases array generated by tool generate_testcases_from_ticket_data")
   },
   async ({ testCases }) => {
     try {
-      const sessionId = Date.now().toString();
-      function parseTestCases(testCasesArray) {
-        const parsed = { new: [], modify: [], remove: [] };
-        testCasesArray.forEach(tc => {
-          if (tc.length === 4 && tc[2] === 'Modify') {
-            parsed.modify.push({ id: tc[3], original: tc[0], modified: tc[1] });
-          } else if (tc.length === 3 && tc[1] === 'Remove') {
-            parsed.remove.push({ id: tc[2], description: tc[0] });
-          } else if (tc.length === 2 && tc[1] === 'New') {
-            parsed.new.push({
-              description: tc[0],
-              id: `NEW-${Date.now()}-${Math.random().toString(36).slice(2)}`
-            });
-          }
-        });
-        return parsed;
+      // These should already be imported at the top of your server file
+      // const express = require('express');
+      // const open = require('open');
+      const app = express();
+      const port = 3001;
+      // Generate unique session ID
+      const sessionId = `session_${Date.now()}_${Math.random().toString(36).substr(2, 9)}`;
+      // Store approval status
+      let approvalStatus = 'pending';
+      let finalTestCases = JSON.parse(JSON.stringify(testCases)); // Deep copy
+      // Serve static files from review-ui folder if it exists
+      const reviewUiPath = path.join(__dirname, 'review-ui');
+      try {
+        await fs.access(reviewUiPath);
+        app.use('/static', express.static(reviewUiPath));
+      } catch (err) {
+        // review-ui folder doesn't exist, continue without it
       }
-      const parsedTestCases = parseTestCases(testCases);
+      app.use(express.json());
+      app.use(express.static('public'));
+      // Main review page
+      app.get('/', (req, res) => {
+        res.send(`
+<!DOCTYPE html>
+<html lang="en">
+<head>
+    <meta charset="UTF-8">
+    <meta name="viewport" content="width=device-width, initial-scale=1.0">
+    <title>Test Cases Review & Approval</title>
+    <style>
+        * {
+            margin: 0;
+            padding: 0;
+            box-sizing: border-box;
+        }
-      // Initialize session with proper locking mechanism
-      approvalSessions.set(sessionId, {
-        status: 'pending',
-        testCases: parsedTestCases,
-        originalTestCases: JSON.parse(JSON.stringify(parsedTestCases)),
-        server: null,
-        startTime: Date.now(),
-        locked: false
-      });
+        body {
+            font-family: 'Segoe UI', Tahoma, Geneva, Verdana, sans-serif;
+            background: linear-gradient(135deg, #667eea 0%, #764ba2 100%);
+            min-height: 100vh;
+            padding: 20px;
+        }
-      const app = express();
+        .container {
+            max-width: 1200px;
+            margin: 0 auto;
+            background: white;
+            border-radius: 15px;
+            box-shadow: 0 20px 60px rgba(0, 0, 0, 0.1);
+            overflow: hidden;
+        }
-      // Try multiple ports if 3001 is busy
-      let port = 3001;
-      let server = null;
-      const tryStartServer = (portToTry) => {
-        return new Promise((resolve, reject) => {
-          const testServer = app.listen(portToTry, () => {
-            resolve({ server: testServer, port: portToTry });
-          }).on('error', (err) => {
-            if (err.code === 'EADDRINUSE') {
-              reject(err);
-            } else {
-              reject(err);
-            }
-          });
-        });
-      };
+        .header {
+            background: linear-gradient(135deg, #4facfe 0%, #00f2fe 100%);
+            color: white;
+            padding: 30px;
+            text-align: center;
+        }
-      // Try ports 3001-3010
-      for (let i = 0; i < 10; i++) {
-        try {
-          const result = await tryStartServer(port + i);
-          server = result.server;
-          port = result.port;
-          break;
-        } catch (err) {
-          if (i === 9) throw new Error(`No available ports found (tried ${port}-${port + 9})`);
+        .header h1 {
+            font-size: 2.5rem;
+            margin-bottom: 10px;
+            font-weight: 700;
         }
-      }
-      app.use(express.json());
-      app.use(express.static(path.join(__dirname, 'review-ui')));
+        .header p {
+            font-size: 1.1rem;
+            opacity: 0.9;
+        }
-      app.get("/", (_req, res) =>
-        res.sendFile(path.join(__dirname, 'review-ui', 'index.html'))
-      );
+        .stats {
+            display: flex;
+            justify-content: space-around;
+            background: #f8f9fa;
+            padding: 20px;
+            border-bottom: 1px solid #e9ecef;
+        }
-      app.get("/api/testcases", (_req, res) => {
-        const session = approvalSessions.get(sessionId);
-        if (!session) return res.status(404).json({ error: 'Session not found' });
-        res.json({ sessionId, testCases: session.testCases });
-      });
+        .stat-item {
+            text-align: center;
+        }
-      app.post(`/delete/${sessionId}`, (req, res) => {
-        const session = approvalSessions.get(sessionId);
-        if (!session) return res.status(404).json({ error: 'Session not found' });
+        .stat-number {
+            font-size: 2rem;
+            font-weight: bold;
+            color: #495057;
+        }
-        const { type, id } = req.body;
-        if (!['new', 'modify', 'remove'].includes(type) || !id) {
-            return res.status(400).json({ error: 'Invalid delete request' });
+        .stat-label {
+            color: #6c757d;
+            margin-top: 5px;
         }
-        try {
-          if (session.testCases[type] && Array.isArray(session.testCases[type])) {
-            session.testCases[type] = session.testCases[type].filter(tc => tc.id !== id);
-            approvalSessions.set(sessionId, session);
-            return res.json({ status: 'deleted', testCases: session.testCases });
-          } else {
-            return res.status(400).json({ error: `Invalid type: ${type}` });
-          }
-        } catch (error) {
-          console.error('Delete error:', error);
-          return res.status(500).json({ error: 'Failed to delete test case' });
+        .content {
+            padding: 30px;
         }
-      });
-      app.post(`/update/${sessionId}`, (req, res) => {
-        const session = approvalSessions.get(sessionId);
-        if (!session) return res.status(404).json({ error: 'Session not found' });
+        .test-case {
+            background: #fff;
+            border: 2px solid #e9ecef;
+            border-radius: 12px;
+            margin-bottom: 20px;
+            overflow: hidden;
+            transition: all 0.3s ease;
+            position: relative;
+        }
-        try {
-          const { type, id, data } = req.body;
-          if (!['new', 'modify'].includes(type) || !id || !data) {
-            return res.status(400).json({ error: 'Invalid update request' });
-          }
+        .test-case:hover {
+            border-color: #4facfe;
+            box-shadow: 0 8px 25px rgba(79, 172, 254, 0.15);
+        }
-          const items = session.testCases[type];
-          const index = items.findIndex(tc => tc.id === id);
-          if (index === -1) {
-            return res.status(404).json({ error: 'Test case not found' });
-          }
+        .test-case.deleted {
+            opacity: 0.5;
+            background: #f8f9fa;
+            border-color: #dc3545;
+        }
-          // Update the test case
-          if (type === 'new') {
-            items[index].description = data.description;
-          } else if (type === 'modify') {
-            items[index].modified = data.modified;
-          }
+        .test-case-header {
+            background: #f8f9fa;
+            padding: 15px 20px;
+            border-bottom: 1px solid #e9ecef;
+            display: flex;
+            justify-content: between;
+            align-items: center;
+        }
-          approvalSessions.set(sessionId, session);
-          return res.json({ status: 'updated', testCases: session.testCases });
-        } catch (error) {
-          console.error('Update error:', error);
-          return res.status(500).json({ error: 'Failed to update test case' });
+        .test-case-id {
+            font-weight: 600;
+            color: #495057;
         }
-      });
-      app.post(`/approve/${sessionId}`, (req, res) => {
-        const session = approvalSessions.get(sessionId);
-        if (!session) return res.status(404).json({ error: 'Session not found' });
+        .test-case-status {
+            padding: 5px 12px;
+            border-radius: 20px;
+            font-size: 0.85rem;
+            font-weight: 500;
+            text-transform: uppercase;
+        }
-        try {
-          // Use atomic update with locking
-          session.locked = true;
-          session.testCases = req.body;
-          session.finalTestCases = JSON.parse(JSON.stringify(req.body)); // Deep copy
-          session.status = 'approved';
-          session.approvedAt = Date.now();
-          approvalSessions.set(sessionId, session);
-          console.log(`Session ${sessionId} approved at ${new Date().toISOString()}`);
-          res.json({ status: 'approved', message: 'Test cases approved successfully!' });
-        } catch (error) {
-          console.error('Approval error:', error);
-          res.status(500).json({ error: 'Failed to approve test cases' });
+        .status-new {
+            background: #d4edda;
+            color: #155724;
+        }
+        .status-modify {
+            background: #fff3cd;
+            color: #856404;
+        }
+        .status-remove {
+            background: #f8d7da;
+            color: #721c24;
+        }
+        .test-case-content {
+            padding: 20px;
+        }
+        .test-case-title {
+            font-size: 1.1rem;
+            font-weight: 500;
+            color: #212529;
+            margin-bottom: 15px;
+            line-height: 1.5;
+        }
+        .test-case-title input {
+            width: 100%;
+            border: 1px solid #ced4da;
+            border-radius: 6px;
+            padding: 10px;
+            font-size: 1rem;
+            transition: border-color 0.3s ease;
+        }
+        .test-case-title input:focus {
+            outline: none;
+            border-color: #4facfe;
+            box-shadow: 0 0 0 3px rgba(79, 172, 254, 0.1);
+        }
+        .test-case-actions {
+            display: flex;
+            gap: 10px;
+        }
+        .btn {
+            padding: 8px 16px;
+            border: none;
+            border-radius: 6px;
+            cursor: pointer;
+            font-size: 0.9rem;
+            transition: all 0.3s ease;
+            font-weight: 500;
+        }
+        .btn-delete {
+            background: #dc3545;
+            color: white;
         }
+        .btn-delete:hover {
+            background: #c82333;
+        }
+        .btn-restore {
+            background: #28a745;
+            color: white;
+        }
+        .btn-restore:hover {
+            background: #218838;
+        }
+        .footer {
+            background: #f8f9fa;
+            padding: 30px;
+            text-align: center;
+            border-top: 1px solid #e9ecef;
+        }
+        .btn-primary {
+            background: linear-gradient(135deg, #4facfe 0%, #00f2fe 100%);
+            color: white;
+            padding: 15px 40px;
+            font-size: 1.1rem;
+            border: none;
+            border-radius: 50px;
+            cursor: pointer;
+            transition: all 0.3s ease;
+            margin: 0 10px;
+        }
+        .btn-primary:hover {
+            transform: translateY(-2px);
+            box-shadow: 0 10px 30px rgba(79, 172, 254, 0.4);
+        }
+        .btn-secondary {
+            background: #6c757d;
+            color: white;
+            padding: 15px 40px;
+            font-size: 1.1rem;
+            border: none;
+            border-radius: 50px;
+            cursor: pointer;
+            transition: all 0.3s ease;
+            margin: 0 10px;
+        }
+        .btn-secondary:hover {
+            background: #5a6268;
+            transform: translateY(-2px);
+        }
+        .notification {
+            position: fixed;
+            top: 20px;
+            right: 20px;
+            padding: 15px 25px;
+            background: #28a745;
+            color: white;
+            border-radius: 8px;
+            box-shadow: 0 5px 15px rgba(0, 0, 0, 0.2);
+            display: none;
+            z-index: 1000;
+        }
+    </style>
+</head>
+<body>
+    <div class="container">
+        <div class="header">
+            <h1>Test Cases Review</h1>
+            <p>Review, edit, and approve test cases for JIRA ticket</p>
+        </div>
+        <div class="stats">
+            <div class="stat-item">
+                <div class="stat-number" id="totalCount">${testCases.length}</div>
+                <div class="stat-label">Total Test Cases</div>
+            </div>
+            <div class="stat-item">
+                <div class="stat-number" id="activeCount">${testCases.length}</div>
+                <div class="stat-label">Active</div>
+            </div>
+            <div class="stat-item">
+                <div class="stat-number" id="deletedCount">0</div>
+                <div class="stat-label">Deleted</div>
+            </div>
+        </div>
+        <div class="content">
+            <div id="testCases">
+                ${testCases.map((testCase, index) => `
+                <div class="test-case" data-index="${index}">
+                    <div class="test-case-header">
+                        <span class="test-case-id">Test Case #${index + 1}</span>
+                        <span class="test-case-status status-${testCase[2] ? testCase[2].toLowerCase() : 'new'}">${testCase[2] || 'New'}</span>
+                    </div>
+                    <div class="test-case-content">
+                        <div class="test-case-title">
+                            <input type="text" value="${testCase[0].replace(/"/g, '&quot;')}" data-index="${index}">
+                        </div>
+                        ${testCase[2] === 'Modify' ? `
+                        <div class="original-case">
+                            <small><strong>Original:</strong> ${testCase[3] || 'N/A'}</small>
+                        </div>
+                        ` : ''}
+                        ${testCase[2] === 'Remove' ? `
+                        <div class="remove-case">
+                            <small><strong>TCMS ID:</strong> ${testCase[2] || 'N/A'}</small>
+                        </div>
+                        ` : ''}
+                        <div class="test-case-actions">
+                            <button class="btn btn-delete" onclick="toggleDelete(${index})">Delete</button>
+                        </div>
+                    </div>
+                </div>
+                `).join('')}
+            </div>
+        </div>
+        <div class="footer">
+            <button class="btn-primary" onclick="approveTestCases()">Approve Test Cases</button>
+            <button class="btn-secondary" onclick="resetAll()">Reset All</button>
+        </div>
+    </div>
+    <div class="notification" id="notification"></div>
+    <script>
+        let testCases = ${JSON.stringify(testCases)};
+        let deletedIndices = new Set();
+        function updateStats() {
+            document.getElementById('activeCount').textContent = testCases.length - deletedIndices.size;
+            document.getElementById('deletedCount').textContent = deletedIndices.size;
+        }
+        function toggleDelete(index) {
+            const testCaseEl = document.querySelector(\`[data-index="\${index}"]\`);
+            const btn = testCaseEl.querySelector('.btn-delete');
+            if (deletedIndices.has(index)) {
+                // Restore
+                deletedIndices.delete(index);
+                testCaseEl.classList.remove('deleted');
+                btn.textContent = 'Delete';
+                btn.className = 'btn btn-delete';
+            } else {
+                // Delete
+                deletedIndices.add(index);
+                testCaseEl.classList.add('deleted');
+                btn.textContent = 'Restore';
+                btn.className = 'btn btn-restore';
+            }
+            updateStats();
+        }
+        function resetAll() {
+            deletedIndices.clear();
+            document.querySelectorAll('.test-case').forEach((el, index) => {
+                el.classList.remove('deleted');
+                const btn = el.querySelector('.btn-delete, .btn-restore');
+                btn.textContent = 'Delete';
+                btn.className = 'btn btn-delete';
+                // Reset input value
+                const input = el.querySelector('input');
+                input.value = testCases[index][0];
+            });
+            updateStats();
+        }
+        function showNotification(message, type = 'success') {
+            const notification = document.getElementById('notification');
+            notification.textContent = message;
+            notification.style.background = type === 'success' ? '#28a745' : '#dc3545';
+            notification.style.display = 'block';
+            setTimeout(() => {
+                notification.style.display = 'none';
+            }, 3000);
+        }
+        function approveTestCases() {
+            // Collect updated test cases
+            const updatedTestCases = [];
+            document.querySelectorAll('.test-case').forEach((el, index) => {
+                if (!deletedIndices.has(index)) {
+                    const input = el.querySelector('input');
+                    const originalTestCase = [...testCases[index]];
+                    originalTestCase[0] = input.value;
+                    updatedTestCases.push(originalTestCase);
+                }
+            });
+            // Send approval to server
+            fetch('/approve', {
+                method: 'POST',
+                headers: {
+                    'Content-Type': 'application/json',
+                },
+                body: JSON.stringify({
+                    sessionId: '${sessionId}',
+                    testCases: updatedTestCases
+                })
+            })
+            .then(response => response.json())
+            .then(data => {
+                if (data.success) {
+                    showNotification('Test cases approved successfully!');
+                    setTimeout(() => {
+                        window.close();
+                    }, 2000);
+                } else {
+                    showNotification('Error approving test cases', 'error');
+                }
+            })
+            .catch(error => {
+                showNotification('Error approving test cases', 'error');
+                console.error('Error:', error);
+            });
+        }
+        // Update test cases when input changes
+        document.addEventListener('input', function(e) {
+            if (e.target.tagName === 'INPUT') {
+                const index = parseInt(e.target.getAttribute('data-index'));
+                testCases[index][0] = e.target.value;
+            }
+        });
+    </script>
+</body>
+</html>
+        `);
       });
-      app.post(`/cancel/${sessionId}`, (_req, res) => {
-        const session = approvalSessions.get(sessionId);
-        if (session) {
-          session.status = 'cancelled';
-          session.cancelledAt = Date.now();
-          approvalSessions.set(sessionId, session);
-          console.log(`Session ${sessionId} cancelled at ${new Date().toISOString()}`);
+      // Approval endpoint
+      app.post('/approve', (req, res) => {
+        try {
+          const { testCases: approvedTestCases } = req.body;
+          finalTestCases = approvedTestCases;
+          approvalStatus = 'approved';
+          // Save to global state for the check tool
+          global.approvalSessions = global.approvalSessions || {};
+          global.approvalSessions[sessionId] = {
+            status: 'approved',
+            testCases: finalTestCases,
+            timestamp: Date.now()
+          };
+          res.json({ success: true, message: 'Test cases approved successfully' });
+          // Close server after approval
+          setTimeout(() => {
+            server.close();
+          }, 3000);
+        } catch (error) {
+          res.status(500).json({ success: false, message: error.message });
         }
-        res.json({ status: 'cancelled', message: 'Review cancelled' });
       });
-      // Update session with server reference
-      const stored = approvalSessions.get(sessionId);
-      stored.server = server;
-      approvalSessions.set(sessionId, stored);
+      // Start server
+      const server = app.listen(port, () => {
+        console.log(`Test case review server running at http://localhost:${port}`);
+      });
       // Open browser
-      try {
-        const { default: open } = await import('open');
-        await open(`http://localhost:${port}`);
-      } catch { /* silent */ }
-      // Timeout safeguard - but don't override approved status
-      setTimeout(() => {
-        const s = approvalSessions.get(sessionId);
-        if (s && s.status === 'pending') {
-          s.status = 'timeout';
-          s.server?.close();
-          approvalSessions.set(sessionId, s);
-        }
-      }, 300000);
+      await open(`http://localhost:${port}`);
-      return `✅ Test case review interface opened successfully!
-Session ID: ${sessionId}
-Test Cases Count: ${testCases.length}
-Browser URL: http://localhost:${port}
-Status: review_started
+      // Store session globally for status checking
+      global.approvalSessions = global.approvalSessions || {};
+      global.approvalSessions[sessionId] = {
+        status: 'pending',
+        testCases: testCases,
+        timestamp: Date.now()
+      };
-Instructions: Use check_approval_status tool with session ID "${sessionId}" to poll for approval status.`;
+      return `✅ Test case review session started. Session ID: ${sessionId}. Browser opened for manual review and approval.`;
     } catch (err) {
-      return `❌ Error setting up review interface: ${err.message}`;
+      return `❌ Error starting test case review: ${err.message}`;
     }
   }
 );
+// Tool 2: Check Approval Status
 tool(
   "check_approval_status",
-  "Check the approval status of test cases review session (waits 20 seconds before checking)",
-  { sessionId: zod_1.z.string().describe("Session ID from review_testcases") },
+  "Check the approval status of test cases review session (waits 25 seconds before checking)",
+  {
+    sessionId: zod_1.z.string().describe("Session ID from review_testcases")
+  },
   async ({ sessionId }) => {
-    const session = approvalSessions.get(sessionId);
-    if (!session) return `❌ Session not found. Invalid session ID: ${sessionId}`;
-    const now = Date.now();
-    const elapsed = Math.floor((now - session.startTime) / 1000);
-    // Check if already approved BEFORE waiting
-    if (session.status === 'approved') {
-      const approvedTestCases = session.finalTestCases || session.testCases;
-      session.server?.close();
-      approvalSessions.delete(sessionId);
-      return `✅ Test cases approved. Elapsed: ${elapsed}s\n${JSON.stringify(approvedTestCases, null, 2)}`;
-    }
+    try {
+      // Wait for 25 seconds
+      await new Promise(resolve => setTimeout(resolve, 25000));
-    if (session.status === 'cancelled') {
-      session.server?.close();
-      approvalSessions.delete(sessionId);
-      return `❌ Review cancelled. Elapsed: ${elapsed}s`;
-    }
+      // Check global approval sessions
+      if (!global.approvalSessions || !global.approvalSessions[sessionId]) {
+        return "❌ Session not found. Please ensure the review session is still active.";
+      }
-    if (session.status === 'timeout' || elapsed > 300) {
-      session.server?.close();
-      approvalSessions.delete(sessionId);
-      return `⏰ Review session timed out. Elapsed: ${elapsed}s`;
-    }
+      const session = global.approvalSessions[sessionId];
-    // Wait 20 seconds before checking status again
-    await new Promise(r => setTimeout(r, 20000));
-    // Re-fetch session after wait to check for status changes
-    const refreshed = approvalSessions.get(sessionId);
-    if (!refreshed) return `❌ Session expired during wait`;
-    const newElapsed = Math.floor((Date.now() - refreshed.startTime) / 1000);
-    if (refreshed.status === 'approved') {
-      const approvedTestCases = refreshed.finalTestCases || refreshed.testCases;
-      refreshed.server?.close();
-      approvalSessions.delete(sessionId);
-      return `✅ Test cases approved. Elapsed: ${newElapsed}s\n${JSON.stringify(approvedTestCases, null, 2)}`;
-    } else if (refreshed.status === 'cancelled') {
-      refreshed.server?.close();
-      approvalSessions.delete(sessionId);
-      return `❌ Review cancelled. Elapsed: ${newElapsed}s`;
-    } else if (refreshed.status === 'timeout' || newElapsed > 300) {
-      refreshed.server?.close();
-      approvalSessions.delete(sessionId);
-      return `⏰ Review session timed out. Elapsed: ${newElapsed}s`;
-    }
+      if (session.status === 'approved') {
+        const result = {
+          status: 'approved',
+          testCases: session.testCases,
+          approvedCount: session.testCases.length,
+          sessionId: sessionId
+        };
-    return `⏳ Still pending. Elapsed: ${newElapsed}s Remaining: ${Math.max(0, 300 - newElapsed)}s`;
-  }
-);
+        // Clean up session after returning result
+        delete global.approvalSessions[sessionId];
+        return `✅ Test cases approved successfully!\n\nApproved ${result.approvedCount} test cases:\n\n${
+          result.testCases.map((tc, index) =>
+            `${index + 1}. ${tc[0]}${tc[2] ? ` (${tc[2]})` : ''}`
+          ).join('\n')
+        }\n\nSession ID: ${sessionId}`;
+      } else {
+        return "⏳ Still waiting for approval. The review session is active but not yet approved. Please complete the review in the browser.";
+      }
+    } catch (err) {
+      return `❌ Error checking approval status: ${err.message}`;
+    }
+  }
+);
     return server;
 };