@nbakka/mcp-appium 1.0.27 → 1.0.30

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nbakka/mcp-appium",
-  "version": "1.0.27",
+  "version": "1.0.30",
   "description": "Selenium WebDriver MCP Server",
   "type": "module",
   "main": "src/lib/server.js",
@@ -21,4 +21,4 @@
     "yargs": "^17.7.2",
 "xml2js": "0.6.2"
   }
-}
+}

package/src/lib/server.js

@@ -147,40 +147,49 @@ server.tool(
 // Get visible text elements including duplicates
 server.tool(
   "get_visible_text_elements",
-  "Get all visible texts from screen JSON source including duplicates",
+  "Get all visible texts from screen XML source including duplicates",
   {},
   async () => {
     if (!state.sessionId) return { content: [{ type: "text", text: "No active session" }] };
     try {
       const response = await axios.get(`${APPIUM_URL}/session/${state.sessionId}/source`);
-      const data = response.data;
-	console.log(response.data);
-      function collectTextsFromJson(node, texts = []) {
+
+const xml = response.data.value;
+
+if (!xml || typeof xml !== "string") {
+  throw new Error("No XML source found in response");
+}
+	console.log(response.data.value);
+
+      const parsed = await parseStringPromise(xml, { explicitArray: false, mergeAttrs: true });
+
+      function collectTextsFromXml(node, texts = []) {
         if (!node) return texts;
+
+        // If array, recurse each element
         if (Array.isArray(node)) {
-          node.forEach(child => collectTextsFromJson(child, texts));
+          node.forEach(child => collectTextsFromXml(child, texts));
           return texts;
         }
-        if (typeof node === "object") {
-          if (node.text && node.text.trim() !== "") {
-            texts.push(node.text.trim());
-          }
-          if (node.children) {
-            collectTextsFromJson(node.children, texts);
-          }
-          for (const key in node) {
-            if (key !== "text" && key !== "children" && typeof node[key] === "object") {
-              collectTextsFromJson(node[key], texts);
-            }
+
+        // If node has 'text' attribute with non-empty value, collect it
+        if (node.text && node.text.trim() !== "") {
+          texts.push(node.text.trim());
+        }
+
+        // Recurse into all children nodes
+        for (const key in node) {
+          if (key !== "text" && typeof node[key] === "object") {
+            collectTextsFromXml(node[key], texts);
           }
         }
         return texts;
       }
 
-      const allTexts = collectTextsFromJson(data);
+      const allTexts = collectTextsFromXml(parsed);
       return { content: [{ type: "json", json: allTexts }] };
     } catch (e) {
-      return { content: [{ type: "text", text: `Error fetching or parsing source: ${e.message}` }] };
+      return { content: [{ type: "text", text: `Error fetching or parsing source XML: ${e.message}` }] };
     }
   }
 );