npm - @nbakka/mcp-appium - Versions diffs - 2.0.67 → 2.0.69 - Mend

@nbakka/mcp-appium 2.0.67 → 2.0.69

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 (2) hide show

package/lib/server.js +242 -62
package/package.json +1 -1

package/lib/server.js CHANGED Viewed

@@ -651,6 +651,9 @@ tool(
   }
 );
+// Global state for tracking approval sessions
+let approvalSessions = new Map();
 tool(
     "review_testcases",
     "Open JSON test cases in browser for manual approval",
@@ -664,87 +667,264 @@ tool(
     async ({ testCases }) => {
         try {
             const express = require('express');
-            const open = require('open');
-            return new Promise((resolve, reject) => {
-                const app = express();
-                const port = 3001;
-                let server;
-                // Serve a simple HTML page with test cases
-                app.get("/", (req, res) => {
-                    let html = `
-                        <html>
-                        <head>
-                            <title>Test Case Approval</title>
-                            <style>
-                                body { font-family: Arial, sans-serif; margin: 20px; }
-                                h1 { color: #333; }
-                                li { margin: 10px 0; padding: 10px; border: 1px solid #ddd; border-radius: 5px; }
-                                button { padding: 10px 20px; background: #007cba; color: white; border: none; border-radius: 5px; cursor: pointer; }
-                                button:hover { background: #005a87; }
-                            </style>
-                        </head>
-                        <body>
-                        <h1>Review Test Cases</h1>
-                        <ul>
-                    `;
+            const sessionId = Date.now().toString();
+            // Initialize session state
+            approvalSessions.set(sessionId, {
+                status: 'pending',
+                testCases: testCases,
+                server: null,
+                startTime: Date.now()
+            });
-                    testCases.forEach(tc => {
-                        html += `<li><strong>${tc.id} - ${tc.title}</strong><p>${tc.description}</p></li>`;
-                    });
+            const app = express();
+            const port = 3001;
+            // Serve a simple HTML page with test cases
+            app.get("/", (req, res) => {
+                let html = `
+                    <html>
+                    <head>
+                        <title>Test Case Approval</title>
+                        <style>
+                            body {
+                                font-family: Arial, sans-serif;
+                                margin: 20px;
+                                background-color: #f5f5f5;
+                            }
+                            .container {
+                                max-width: 1200px;
+                                margin: 0 auto;
+                                background: white;
+                                padding: 20px;
+                                border-radius: 8px;
+                                box-shadow: 0 2px 10px rgba(0,0,0,0.1);
+                            }
+                            h1 {
+                                color: #333;
+                                text-align: center;
+                                border-bottom: 2px solid #007cba;
+                                padding-bottom: 10px;
+                            }
+                            .test-case {
+                                margin: 15px 0;
+                                padding: 15px;
+                                border: 1px solid #ddd;
+                                border-radius: 8px;
+                                background: #fafafa;
+                            }
+                            .test-id {
+                                font-weight: bold;
+                                color: #007cba;
+                                font-size: 16px;
+                            }
+                            .test-title {
+                                font-weight: bold;
+                                margin: 5px 0;
+                                color: #333;
+                            }
+                            .test-description {
+                                color: #666;
+                                line-height: 1.4;
+                            }
+                            .button-container {
+                                text-align: center;
+                                margin-top: 30px;
+                                padding-top: 20px;
+                                border-top: 1px solid #ddd;
+                            }
+                            button {
+                                padding: 15px 30px;
+                                background: #28a745;
+                                color: white;
+                                border: none;
+                                border-radius: 5px;
+                                cursor: pointer;
+                                font-size: 16px;
+                                font-weight: bold;
+                            }
+                            button:hover {
+                                background: #218838;
+                            }
+                            .status {
+                                text-align: center;
+                                margin-top: 20px;
+                                font-weight: bold;
+                                display: none;
+                            }
+                        </style>
+                    </head>
+                    <body>
+                    <div class="container">
+                        <h1>Review Test Cases</h1>
+                        <div id="test-cases">
+                `;
+                testCases.forEach(tc => {
                     html += `
-                        </ul>
-                        <button onclick="fetch('/approve').then(()=>window.close())">Approve</button>
-                        </body>
-                        </html>
+                        <div class="test-case">
+                            <div class="test-id">${tc.id}</div>
+                            <div class="test-title">${tc.title}</div>
+                            <div class="test-description">${tc.description}</div>
+                        </div>
                     `;
-                    res.send(html);
                 });
-                // Approval endpoint
-                app.get("/approve", (req, res) => {
-                    res.send("Approved!");
-                    if (server) {
-                        server.close();
-                    }
-                    resolve("Test cases approved by user");
-                });
+                html += `
+                        </div>
+                        <div class="button-container">
+                            <button onclick="approveTestCases()">✓ Approve Test Cases</button>
+                            <div id="status" class="status"></div>
+                        </div>
+                    </div>
+                    <script>
+                        function approveTestCases() {
+                            document.querySelector('button').disabled = true;
+                            document.querySelector('button').style.background = '#6c757d';
+                            document.querySelector('button').innerHTML = 'Processing...';
+                            const status = document.getElementById('status');
+                            status.style.display = 'block';
+                            status.style.color = '#28a745';
+                            status.innerHTML = 'Test cases approved! You can close this window.';
+                            fetch('/approve/${sessionId}')
+                                .then(response => response.text())
+                                .then(data => {
+                                    status.innerHTML = data + ' You can close this window.';
+                                    setTimeout(() => {
+                                        window.close();
+                                    }, 2000);
+                                })
+                                .catch(error => {
+                                    status.style.color = '#dc3545';
+                                    status.innerHTML = 'Error: ' + error.message;
+                                });
+                        }
+                    </script>
+                    </body>
+                    </html>
+                `;
+                res.send(html);
+            });
-                // Start server and open browser
-                server = app.listen(port, async () => {
-                    try {
-                        await open(`http://localhost:${port}`);
-                    } catch (openError) {
-                        console.error('Failed to open browser:', openError);
-                        // Fallback: just provide the URL
-                        resolve(`Test cases review page available at: http://localhost:${port}`);
-                    }
-                });
+            // Approval endpoint
+            app.get(`/approve/${sessionId}`, (req, res) => {
+                const session = approvalSessions.get(sessionId);
+                if (session) {
+                    session.status = 'approved';
+                    approvalSessions.set(sessionId, session);
+                }
+                res.send("✓ Test cases approved successfully!");
+            });
-                // Handle server errors
-                server.on('error', (err) => {
-                    reject(`Server error: ${err.message}`);
-                });
+            // Start server
+            const server = app.listen(port, async () => {
+                console.log(`Test case review server started on http://localhost:${port}`);
+                try {
+                    // Use dynamic import for the open package (ES module)
+                    const { default: open } = await import('open');
+                    await open(`http://localhost:${port}`);
+                    console.log('Browser opened successfully');
+                } catch (openError) {
+                    console.log('Failed to open browser automatically:', openError.message);
+                    console.log(`Please manually open: http://localhost:${port}`);
+                }
+            });
-                // Timeout after 5 minutes
-                setTimeout(() => {
-                    if (server) {
-                        server.close();
+            // Store server reference in session
+            const session = approvalSessions.get(sessionId);
+            session.server = server;
+            approvalSessions.set(sessionId, session);
+            // Auto cleanup after 5 minutes
+            setTimeout(() => {
+                const session = approvalSessions.get(sessionId);
+                if (session && session.status === 'pending') {
+                    session.status = 'timeout';
+                    if (session.server) {
+                        session.server.close();
                     }
-                    reject("Review session timed out after 5 minutes");
-                }, 300000);
+                    approvalSessions.set(sessionId, session);
+                }
+            }, 300000); // 5 minutes
+            return JSON.stringify({
+                status: "review_started",
+                sessionId: sessionId,
+                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 10 seconds using check_approval_status tool until approved or timeout (5 minutes)"
             });
         } catch (err) {
-            return `Error setting up review interface: ${err.message}`;
+            return JSON.stringify({
+                status: "error",
+                message: `Error setting up review interface: ${err.message}`
+            });
         }
     }
 );
+tool(
+    "check_approval_status",
+    "Check the approval status of test cases review session",
+    {
+        sessionId: zod_1.z.string().describe("The session ID returned from review_testcases tool")
+    },
+    async ({ sessionId }) => {
+        const session = approvalSessions.get(sessionId);
+        if (!session) {
+            return JSON.stringify({
+                status: "error",
+                message: "Session not found. Invalid session ID."
+            });
+        }
+        const currentTime = Date.now();
+        const elapsedTime = Math.floor((currentTime - session.startTime) / 1000);
+        if (session.status === 'approved') {
+            // Clean up session and close server
+            if (session.server) {
+                session.server.close();
+            }
+            approvalSessions.delete(sessionId);
+            return JSON.stringify({
+                status: "approved",
+                message: "Test cases have been approved by the user. Continuing with next operations...",
+                testCasesCount: session.testCases.length,
+                elapsedTime: elapsedTime
+            });
+        } else if (session.status === 'timeout' || elapsedTime > 300) { // 5 minutes
+            // Clean up session and close server
+            if (session.server) {
+                session.server.close();
+            }
+            approvalSessions.delete(sessionId);
+            return JSON.stringify({
+                status: "timeout",
+                message: "Review session timed out after 5 minutes. Test cases were not approved.",
+                testCasesCount: session.testCases.length,
+                elapsedTime: elapsedTime
+            });
+        } else {
+            return JSON.stringify({
+                status: "pending",
+                message: "Test cases are still pending approval. Continue polling.",
+                testCasesCount: session.testCases.length,
+                elapsedTime: elapsedTime,
+                remainingTime: Math.max(0, 300 - elapsedTime)
+            });
+        }
+    }
+);
     return server;
 };

package/package.json CHANGED Viewed

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