@nbakka/mcp-appium 3.0.9 → 3.0.11

package/lib/server.js

@@ -837,22 +837,46 @@ tool(
 
       // Process test cases - handle the specific format properly
       const processedTestCases = testCases.map((testCase, index) => {
-        // Each testCase is an array like ["title", "description", "status", "originalCase"]
         if (Array.isArray(testCase)) {
-          return {
-            title: testCase[0] || `Test Case ${index + 1}`,
-            description: testCase[1] || '',
-            status: testCase[2] || 'New',
-            originalCase: testCase[3] || '',
-            index: index
-          };
+          const arrayLength = testCase.length;
+
+          if (arrayLength === 4) {
+            // Modify case: ["original title", "new description", "Modify", "SCRUM-TC-1"]
+            return {
+              originalTitle: testCase[0] || `Test Case ${index + 1}`,
+              newDescription: testCase[1] || '',
+              status: testCase[2] || 'Modify',
+              testId: testCase[3] || '',
+              index: index
+            };
+          } else if (arrayLength === 3) {
+            // Remove case: ["title", "Remove", "SCRUM-TC-2"]
+            return {
+              title: testCase[0] || `Test Case ${index + 1}`,
+              status: testCase[1] || 'Remove',
+              testId: testCase[2] || '',
+              index: index
+            };
+          } else if (arrayLength === 2) {
+            // New case: ["title", "New"]
+            return {
+              title: testCase[0] || `Test Case ${index + 1}`,
+              status: testCase[1] || 'New',
+              index: index
+            };
+          } else {
+            // Fallback for unexpected format
+            return {
+              title: testCase[0] || `Test Case ${index + 1}`,
+              status: 'New',
+              index: index
+            };
+          }
         } else {
-          // Fallback for unexpected format
+          // Fallback for non-array format
           return {
             title: String(testCase) || `Test Case ${index + 1}`,
-            description: '',
             status: 'New',
-            originalCase: '',
             index: index
           };
         }
@@ -864,12 +888,12 @@ tool(
 
         if (status === 'modify') {
           // For modify cases, show original → changed format
-          return `${testCase.title}\n\nOriginal: ${testCase.originalCase || 'Not specified'}\nChanged to: ${testCase.description || 'Not specified'}`;
+          return `Original: ${testCase.originalTitle}\nChanged to: ${testCase.newDescription}`;
         } else if (status === 'remove') {
-          // For remove cases, just show the title
+          // For remove cases, show the title
           return testCase.title;
         } else {
-          // For new cases, just show the title
+          // For new cases, show the title
           return testCase.title;
         }
       };
@@ -1176,10 +1200,10 @@ tool(
 
               // Create proper label and test ID display for modify/remove cases
               let labelAndIdDisplay = '';
-              if (testCase.status.toLowerCase() === 'modify' && testCase.originalCase) {
-                labelAndIdDisplay = `<div style="margin-bottom: 8px; font-weight: 600; color: #856404; font-size: 13px;">Modify - ${testCase.originalCase}</div>`;
-              } else if (testCase.status.toLowerCase() === 'remove' && testCase.originalCase) {
-                labelAndIdDisplay = `<div style="margin-bottom: 8px; font-weight: 600; color: #721c24; font-size: 13px;">Remove - ${testCase.originalCase}</div>`;
+              if (testCase.status.toLowerCase() === 'modify' && testCase.testId) {
+                labelAndIdDisplay = `<div style="margin-bottom: 8px; font-weight: 600; color: #856404; font-size: 13px;">Modify - ${testCase.testId}</div>`;
+              } else if (testCase.status.toLowerCase() === 'remove' && testCase.testId) {
+                labelAndIdDisplay = `<div style="margin-bottom: 8px; font-weight: 600; color: #721c24; font-size: 13px;">Remove - ${testCase.testId}</div>`;
               }
 
               return `
@@ -1245,7 +1269,9 @@ tool(
                 const status = originalTestCase.status.toLowerCase();
 
                 if (status === 'modify') {
-                    resetValue = originalTestCase.title + '\\n\\nOriginal: ' + (originalTestCase.originalCase || 'Not specified') + '\\nChanged to: ' + (originalTestCase.description || 'Not specified');
+                    resetValue = 'Original: ' + originalTestCase.originalTitle + '\\nChanged to: ' + originalTestCase.newDescription;
+                } else if (status === 'remove') {
+                    resetValue = originalTestCase.title;
                 } else {
                     resetValue = originalTestCase.title;
                 }
@@ -1275,13 +1301,32 @@ tool(
                         const textarea = el.querySelector('textarea');
                         const originalTestCase = testCases[index];
 
-                        // Create the updated test case array in the original format
-                        const updatedCase = [
-                            textarea.value.trim(), // Updated title/content
-                            originalTestCase.description, // Keep original description
-                            originalTestCase.status, // Keep original status
-                            originalTestCase.originalCase // Keep original case reference
-                        ];
+                        // Create the updated test case array based on the original format
+                        let updatedCase;
+                        const status = originalTestCase.status.toLowerCase();
+
+                        if (status === 'modify') {
+                            // For modify: [updatedContent, newDescription, "Modify", testId]
+                            updatedCase = [
+                                textarea.value.trim(),
+                                originalTestCase.newDescription,
+                                originalTestCase.status,
+                                originalTestCase.testId
+                            ];
+                        } else if (status === 'remove') {
+                            // For remove: [updatedContent, "Remove", testId]
+                            updatedCase = [
+                                textarea.value.trim(),
+                                originalTestCase.status,
+                                originalTestCase.testId
+                            ];
+                        } else {
+                            // For new: [updatedContent, "New"]
+                            updatedCase = [
+                                textarea.value.trim(),
+                                originalTestCase.status
+                            ];
+                        }
 
                         updatedTestCases.push(updatedCase);
                     }
@@ -1464,10 +1509,35 @@ tool(
             return `${index + 1}. ${String(tc)} (New)`;
           }
 
-          const title = tc[0] || `Test Case ${index + 1}`;
-          const description = tc[1] || '';
-          const status = tc[2] || 'New';
-          const originalCase = tc[3] || '';
+          // Handle different array structures based on length and content
+          let title, description, status, originalCase;
+
+          if (tc.length === 4) {
+            // Standard format: [title, description, status, originalCase]
+            title = tc[0] || `Test Case ${index + 1}`;
+            description = tc[1] || '';
+            status = tc[2] || 'New';
+            originalCase = tc[3] || '';
+          } else if (tc.length === 3) {
+            // Could be [title, status, originalCase] for remove cases
+            title = tc[0] || `Test Case ${index + 1}`;
+            if (tc[1] && tc[1].toLowerCase() === 'remove') {
+              status = tc[1];
+              originalCase = tc[2] || '';
+              description = '';
+            } else {
+              // [title, description, status]
+              description = tc[1] || '';
+              status = tc[2] || 'New';
+              originalCase = '';
+            }
+          } else {
+            // Fallback
+            title = tc[0] || `Test Case ${index + 1}`;
+            description = tc[1] || '';
+            status = tc[2] || 'New';
+            originalCase = tc[3] || '';
+          }
 
           const statusLower = status.toLowerCase();
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nbakka/mcp-appium",
-  "version": "3.0.9",
+  "version": "3.0.11",
   "description": "Appium MCP",
   "engines": {
     "node": ">=18"
@@ -54,4 +54,4 @@
     "lib": "lib"
   },
   "author": "nbakka"
-}
+}