appium-mcp 1.8.1 → 1.8.3

package/src/locators/element-filter.ts

@@ -39,9 +39,11 @@ function matchesAttributeFilters(
 ): boolean {
   if (requireAttributes.length > 0) {
     const hasRequiredAttr = requireAttributes.some(
-      attr => element.attributes && element.attributes[attr]
+      (attr) => element.attributes && element.attributes[attr]
     );
-    if (!hasRequiredAttr) return false;
+    if (!hasRequiredAttr) {
+      return false;
+    }
   }
 
   if (
@@ -87,7 +89,7 @@ function isInteractableElement(
         ];
 
   return (
-    interactableTags.some(tag => element.tagName.includes(tag)) ||
+    interactableTags.some((tag) => element.tagName.includes(tag)) ||
     element.attributes?.clickable === 'true' ||
     element.attributes?.focusable === 'true'
   );

package/src/locators/generate-all-locators.ts

@@ -104,7 +104,7 @@ function traverseAndProcessElements(
 
   // Recursively process children (even if parent was filtered out)
   if (element.children && element.children.length > 0) {
-    element.children.forEach(child =>
+    element.children.forEach((child) =>
       traverseAndProcessElements(
         child,
         sourceXML,

package/src/locators/locator-generation.ts

@@ -82,7 +82,7 @@ export function getSimpleSuggestedLocators(
   isNative: boolean = true
 ): Record<string, string> {
   const res: Record<string, string> = {};
-  for (let [strategyAlias, strategy] of SIMPLE_STRATEGY_MAPPINGS) {
+  for (const [strategyAlias, strategy] of SIMPLE_STRATEGY_MAPPINGS) {
     // accessibility id is only supported in native context
     if (!(strategy === 'accessibility id' && !isNative)) {
       const value = attributes[strategyAlias];
@@ -103,7 +103,7 @@ export function getComplexSuggestedLocators(
   isNative: boolean,
   automationName: string
 ): Record<string, string> {
-  let complexLocators: Record<string, string | null> = {};
+  const complexLocators: Record<string, string | null> = {};
   const domNode = findDOMNodeByPath(path, sourceDoc);
   if (isNative) {
     switch (automationName) {
@@ -342,7 +342,7 @@ export function getOptimalXPath(
     ];
     const attrPairsPermutations: [string, string][] = attrsForPairs.flatMap(
       (v1, i) =>
-        attrsForPairs.slice(i + 1).map(v2 => [v1, v2] as [string, string])
+        attrsForPairs.slice(i + 1).map((v2) => [v1, v2] as [string, string])
     );
 
     const cases = [
@@ -405,7 +405,7 @@ export function getOptimalXPath(
 
       // If there's more than one sibling, append the index
       if (childNodes.length > 1) {
-        let index = childNodes.indexOf(domNode);
+        const index = childNodes.indexOf(domNode);
         xpath += `[${index + 1}]`;
       }
     }
@@ -440,7 +440,7 @@ export function getOptimalClassChain(
     // BASE CASE #2: If this node has a unique class chain based on attributes, return it
     let classChain: string, othersWithAttr: XMLNode[];
 
-    for (let attrName of CHECKED_CLASS_CHAIN_ATTRIBUTES) {
+    for (const attrName of CHECKED_CLASS_CHAIN_ATTRIBUTES) {
       const attrValue = (domNode as XMLElement).getAttribute?.(attrName);
       if (_.isEmpty(attrValue)) {
         continue;
@@ -462,7 +462,7 @@ export function getOptimalClassChain(
 
       // If the attribute isn't actually unique, get its index too
       if (othersWithAttr.length > 1) {
-        let index = othersWithAttr.indexOf(domNode);
+        const index = othersWithAttr.indexOf(domNode);
         classChain = `${classChain}[${index + 1}]`;
       }
       return classChain;
@@ -484,7 +484,7 @@ export function getOptimalClassChain(
 
       // If there's more than one sibling, append the index
       if (childNodes.length > 1) {
-        let index = childNodes.indexOf(domNode);
+        const index = childNodes.indexOf(domNode);
         classChain += `[${index + 1}]`;
       }
     }
@@ -513,11 +513,11 @@ export function getOptimalPredicateString(
     }
 
     // BASE CASE #2: Check all attributes and try to find the best way
-    let xpathAttributes: string[] = [];
-    let predicateString: string[] = [];
+    const xpathAttributes: string[] = [];
+    const predicateString: string[] = [];
     let othersWithAttr: XMLNode[];
 
-    for (let attrName of CHECKED_PREDICATE_ATTRIBUTES) {
+    for (const attrName of CHECKED_PREDICATE_ATTRIBUTES) {
       const attrValue = (domNode as XMLElement).getAttribute?.(attrName);
       if (_.isEmpty(attrValue)) {
         continue;
@@ -579,7 +579,7 @@ export function getOptimalUiAutomatorSelector(
     // BASE CASE #3: If looking for an element that is not inside
     // the last direct child of the hierarchy, return null
     const lastHierarchyChildIndex = (hierarchyChildren.length - 1).toString();
-    let pathArray = path.split('.');
+    const pathArray = path.split('.');
     const requestedHierarchyChildIndex = pathArray[0];
     if (requestedHierarchyChildIndex !== lastHierarchyChildIndex) {
       return null;

package/src/server.ts

@@ -15,11 +15,11 @@ registerResources(server);
 registerTools(server);
 
 // Handle client connection and disconnection events
-server.on('connect', event => {
+server.on('connect', (event) => {
   log.info('Client connected:', event.session);
 });
 
-server.on('disconnect', async event => {
+server.on('disconnect', async (event) => {
   log.info('Client disconnected:', event.session);
   // Only try to clean up if there's an active session
   if (hasActiveSession()) {

package/src/session-store.ts

@@ -97,8 +97,12 @@ export async function safeDeleteSession(): Promise<boolean> {
 }
 
 export const getPlatformName = (driver: any): string => {
-  if (driver instanceof AndroidUiautomator2Driver) return PLATFORM.android;
-  if (driver instanceof XCUITestDriver) return PLATFORM.ios;
+  if (driver instanceof AndroidUiautomator2Driver) {
+    return PLATFORM.android;
+  }
+  if (driver instanceof XCUITestDriver) {
+    return PLATFORM.ios;
+  }
 
   if ((driver as Client).isAndroid) {
     return PLATFORM.android;

package/src/tests/__mocks__/@appium/support.ts

@@ -2,10 +2,10 @@
 // This avoids the ESM/CommonJS mismatch with uuid dependency
 
 export const logger = {
-  getLogger: (name: string) => {
+  getLogger: (name: string) =>
     // Simple logger implementation for tests
     // No-op functions that match the logger interface
-    return {
+    ({
       debug: (message: string, ...args: any[]) => {
         // Silent in tests by default
       },
@@ -21,8 +21,7 @@ export const logger = {
       trace: (message: string, ...args: any[]) => {
         // Silent in tests by default
       },
-    };
-  },
+    }),
 };
 
 // Export other commonly used utilities from @appium/support if needed

package/src/tests/generate-all-locators.test.ts

@@ -75,9 +75,9 @@ describe('generateAllElementLocators', () => {
     // If the filter works, either the result will be empty (if no buttons found)
     // or all elements will be buttons
     if (result.length > 0) {
-      expect(result.every(element => element.tagName.includes('Button'))).toBe(
-        true
-      );
+      expect(
+        result.every((element) => element.tagName.includes('Button'))
+      ).toBe(true);
     }
   });
 
@@ -88,7 +88,7 @@ describe('generateAllElementLocators', () => {
     });
 
     // Verify no Button elements are included
-    expect(result.every(element => !element.tagName.includes('Button'))).toBe(
+    expect(result.every((element) => !element.tagName.includes('Button'))).toBe(
       true
     );
   });
@@ -129,12 +129,11 @@ describe('generateAllElementLocators', () => {
       ];
 
       expect(
-        result.every(element => {
-          return (
-            interactableTags.some(tag => element.tagName.includes(tag)) ||
+        result.every(
+          (element) =>
+            interactableTags.some((tag) => element.tagName.includes(tag)) ||
             element.clickable === true
-          );
-        })
+        )
       ).toBe(true);
     }
   });
@@ -160,9 +159,9 @@ describe('generateAllElementLocators', () => {
       ];
 
       expect(
-        result.every(element => {
-          return interactableTags.some(tag => element.tagName.includes(tag));
-        })
+        result.every((element) =>
+          interactableTags.some((tag) => element.tagName.includes(tag))
+        )
       ).toBe(true);
     }
   });
@@ -174,6 +173,6 @@ describe('generateAllElementLocators', () => {
     });
 
     // Verify all elements are clickable
-    expect(result.every(element => element.clickable === true)).toBe(true);
+    expect(result.every((element) => element.clickable === true)).toBe(true);
   });
 });

package/src/tests/test-setup-wda.ts

@@ -31,14 +31,18 @@ async function downloadFile(url: string, destPath: string): Promise<void> {
     const file = fs.createWriteStream(destPath);
 
     https
-      .get(url, response => {
+      .get(url, async (response) => {
         // Handle redirects
         if (response.statusCode === 302 || response.statusCode === 301) {
           file.close();
           fs.unlinkSync(destPath);
-          return downloadFile(response.headers.location!, destPath)
-            .then(resolve)
-            .catch(reject);
+          try {
+            await downloadFile(response.headers.location!, destPath);
+            resolve();
+          } catch (err) {
+            reject(err);
+          }
+          return;
         }
 
         if (response.statusCode !== 200) {
@@ -55,7 +59,7 @@ async function downloadFile(url: string, destPath: string): Promise<void> {
         );
         let downloadedSize = 0;
 
-        response.on('data', chunk => {
+        response.on('data', (chunk) => {
           downloadedSize += chunk.length;
           const percent = ((downloadedSize / totalSize) * 100).toFixed(1);
           process.stdout.write(
@@ -71,7 +75,7 @@ async function downloadFile(url: string, destPath: string): Promise<void> {
           resolve();
         });
       })
-      .on('error', err => {
+      .on('error', (err) => {
         file.close();
         fs.unlinkSync(destPath);
         reject(err);
@@ -102,10 +106,10 @@ async function getLatestWDAVersion(): Promise<string> {
     };
 
     https
-      .get(options, response => {
+      .get(options, (response) => {
         let data = '';
 
-        response.on('data', chunk => {
+        response.on('data', (chunk) => {
           data += chunk;
         });
 
@@ -124,7 +128,7 @@ async function getLatestWDAVersion(): Promise<string> {
           }
         });
       })
-      .on('error', err => {
+      .on('error', (err) => {
         reject(err);
       });
   });
@@ -163,7 +167,7 @@ async function main() {
       // Show app contents
       const appContents = fs.readdirSync(appPath);
       console.log('\n📋 Cached App bundle contents:');
-      appContents.forEach(item => {
+      appContents.forEach((item) => {
         console.log(`      - ${item}`);
       });
 
@@ -204,7 +208,7 @@ async function main() {
     // List contents
     console.log('\n📋 Extracted contents:');
     const contents = fs.readdirSync(extractDir);
-    contents.forEach(item => {
+    contents.forEach((item) => {
       const itemPath = path.join(extractDir, item);
       const isDir = fs.statSync(itemPath).isDirectory();
       console.log(`   ${isDir ? '📁' : '📄'} ${item}`);
@@ -218,7 +222,7 @@ async function main() {
       // Show app contents
       const appContents = fs.readdirSync(appPath);
       console.log('\n   App bundle contents:');
-      appContents.forEach(item => {
+      appContents.forEach((item) => {
         console.log(`      - ${item}`);
       });
     } else {
@@ -241,7 +245,11 @@ async function main() {
 }
 
 // Run main function
-main().catch(error => {
-  console.error('\n💥 Unexpected error:', error);
-  process.exit(1);
-});
+(async () => {
+  try {
+    await main();
+  } catch (error: any) {
+    console.error('\n💥 Unexpected error:', error);
+    process.exit(1);
+  }
+})();

package/src/tools/documentation/reasoning-rag.ts

@@ -220,8 +220,8 @@ export class ReasoningRAG {
     for (let i = 0; i < chunks.length; i += batchSize) {
       const batch = chunks.slice(i, i + batchSize);
 
-      const batchPromises = batch.flatMap(chunk =>
-        configs.map(config =>
+      const batchPromises = batch.flatMap((chunk) =>
+        configs.map((config) =>
           this.performReasoning(chunk.pageContent, config, query)
         )
       );
@@ -249,16 +249,16 @@ export class ReasoningRAG {
   ): Promise<string> {
     // Extract all reasoning outputs
     const summaries = reasoningResults
-      .filter(result => result.metadata.task === 'summarization')
-      .map(result => result.reasoningOutput);
+      .filter((result) => result.metadata.task === 'summarization')
+      .map((result) => result.reasoningOutput);
 
     const analyses = reasoningResults
-      .filter(result => result.metadata.task === 'analysis')
-      .map(result => result.reasoningOutput);
+      .filter((result) => result.metadata.task === 'analysis')
+      .map((result) => result.reasoningOutput);
 
     const qaResults = reasoningResults
-      .filter(result => result.metadata.task === 'question-answering')
-      .map(result => result.reasoningOutput);
+      .filter((result) => result.metadata.task === 'question-answering')
+      .map((result) => result.reasoningOutput);
 
     // Combine all insights
     let comprehensiveSummary = `## Query: ${query}\n\n`;
@@ -332,7 +332,7 @@ export class ReasoningRAG {
       ];
 
       // Filter configs based on requested tasks
-      const filteredConfigs = configs.filter(config =>
+      const filteredConfigs = configs.filter((config) =>
         reasoningTasks.includes(config.task)
       );
 
@@ -355,7 +355,7 @@ export class ReasoningRAG {
 
       // Step 5: Extract best answer from reasoning results
       const qaResults = reasoningResults.filter(
-        result =>
+        (result) =>
           result.metadata.task === 'question-answering' &&
           !result.metadata.error
       );

package/src/tools/documentation/simple-pdf-indexer.ts

@@ -72,7 +72,7 @@ async function saveDocuments(
     }
 
     // Serialize the new documents
-    const serializedNew = documents.map(doc => ({
+    const serializedNew = documents.map((doc) => ({
       pageContent: doc.pageContent,
       metadata: doc.metadata,
     }));
@@ -357,12 +357,12 @@ export async function indexAllMarkdownFiles(
         // Add file metadata to each document
         const filename = path.basename(markdownFile);
         const relativePath = path.relative(dirPath, markdownFile);
-        documents.forEach(doc => {
+        documents.forEach((doc) => {
           doc.metadata = {
             ...doc.metadata,
             source: markdownFile,
-            filename: filename,
-            relativePath: relativePath,
+            filename,
+            relativePath,
           };
         });
 
@@ -482,25 +482,27 @@ if (import.meta.url === `file://${process.argv[1]}`) {
   if (indexSingleFile) {
     // Index a single Markdown file
     log.info(`Indexing single Markdown file: ${markdownPath}`);
-    indexMarkdown(markdownPath, chunkSize, chunkOverlap)
-      .then(() => {
-        process.exit(0);
-      })
-      .catch(error => {
-        log.error('Indexing failed:', error);
-        process.exit(1);
-      });
+    try {
+      await indexMarkdown(markdownPath, chunkSize, chunkOverlap);
+      process.exit(0);
+    } catch (error) {
+      log.error('Indexing failed:', error);
+      process.exit(1);
+    }
   } else {
     // Index all Markdown files in the directory
     log.info(`Indexing all Markdown files in directory: ${markdownPath}`);
-    indexAllMarkdownFiles(markdownPath, chunkSize, chunkOverlap)
-      .then(indexedFiles => {
-        log.info(`Successfully indexed ${indexedFiles.length} Markdown files`);
-        process.exit(0);
-      })
-      .catch(error => {
-        log.error('Indexing failed:', error);
-        process.exit(1);
-      });
+    try {
+      const indexedFiles = await indexAllMarkdownFiles(
+        markdownPath,
+        chunkSize,
+        chunkOverlap
+      );
+      log.info(`Successfully indexed ${indexedFiles.length} Markdown files`);
+      process.exit(0);
+    } catch (error) {
+      log.error('Indexing failed:', error);
+      process.exit(1);
+    }
   }
 }

package/src/tools/index.ts

@@ -69,7 +69,7 @@ export default function registerTools(server: FastMCP): void {
           JSON.stringify(obj, (key, value) => {
             if (
               key &&
-              SENSITIVE_KEYS.some(k => key.toLowerCase().includes(k))
+              SENSITIVE_KEYS.some((k) => key.toLowerCase().includes(k))
             ) {
               return '[REDACTED]';
             }

package/src/tools/interactions/drag-and-drop.ts

@@ -28,7 +28,7 @@ async function performDragAndDrop(
         { type: 'pointerMove', duration: 0, x: startX, y: startY },
         { type: 'pointerDown', button: 0 },
         { type: 'pause', duration: longPressDuration },
-        { type: 'pointerMove', duration: duration, x: endX, y: endY },
+        { type: 'pointerMove', duration, x: endX, y: endY },
         { type: 'pause', duration: DROP_PAUSE_DURATION_MS },
         { type: 'pointerUp', button: 0 },
       ],

package/src/tools/interactions/long-press.ts

@@ -54,7 +54,7 @@ export default function longPress(server: FastMCP): void {
               actions: [
                 { type: 'pointerMove', duration: 0, x, y },
                 { type: 'pointerDown', button: 0 },
-                { type: 'pause', duration: duration },
+                { type: 'pause', duration },
                 { type: 'pointerUp', button: 0 },
               ],
             },
@@ -85,7 +85,7 @@ export default function longPress(server: FastMCP): void {
                 actions: [
                   { type: 'pointerMove', duration: 0, x, y },
                   { type: 'pointerDown', button: 0 },
-                  { type: 'pause', duration: duration },
+                  { type: 'pause', duration },
                   { type: 'pointerUp', button: 0 },
                 ],
               },

package/src/tools/ios/boot-simulator.ts

@@ -34,7 +34,7 @@ export default function bootSimulator(server: any): void {
         const simulators = await iosManager.listSimulators();
 
         // Find the simulator with the given UDID
-        const simulator = simulators.find(sim => sim.udid === udid);
+        const simulator = simulators.find((sim) => sim.udid === udid);
 
         if (!simulator) {
           throw new Error(

package/src/tools/ios/install-wda.ts

@@ -28,7 +28,7 @@ async function getLatestWDAVersion(): Promise<string> {
 
   const entries = await readdir(wdaCacheDir);
   const versions = await Promise.all(
-    entries.map(async dir => {
+    entries.map(async (dir) => {
       const dirPath = path.join(wdaCacheDir, dir);
       const stats = await stat(dirPath);
       return stats.isDirectory() ? dir : null;
@@ -37,10 +37,10 @@ async function getLatestWDAVersion(): Promise<string> {
 
   const filteredVersions = versions
     .filter((v): v is string => v !== null)
-    .sort((a, b) => {
+    .sort((a, b) =>
       // Simple version comparison - you might want to use semver for more complex versions
-      return b.localeCompare(a, undefined, { numeric: true });
-    });
+      b.localeCompare(a, undefined, { numeric: true })
+    );
 
   if (filteredVersions.length === 0) {
     throw new Error(

package/src/tools/ios/setup-wda.ts

@@ -6,8 +6,7 @@ import { exec } from 'child_process';
 import { promisify } from 'util';
 import path from 'path';
 import { access, mkdir, unlink } from 'fs/promises';
-import { constants } from 'fs';
-import { createWriteStream } from 'fs';
+import { constants, createWriteStream } from 'fs';
 import { pipeline } from 'stream/promises';
 import os from 'os';
 import axios from 'axios';
@@ -196,7 +195,7 @@ export default function setupWDA(server: any): void {
               text: `${JSON.stringify(
                 {
                   version: wdaVersion,
-                  platform: platform,
+                  platform,
                   architecture: archStr,
                   wdaAppPath: appPath,
                   wdaCachePath: `~/.cache/appium-mcp/wda/${wdaVersion}`,

package/src/tools/navigations/scroll-to-element.ts

@@ -24,7 +24,7 @@ const getValue = (xpath: string, expression: string): string => {
   // Extracts the value from an XPath expression.
   let start = xpath.indexOf(expression) + expression.length;
   start = xpath.indexOf("'", start) + 1;
-  let end = xpath.indexOf("'", start);
+  const end = xpath.indexOf("'", start);
   return xpath.substring(start, end);
 };
 
@@ -33,26 +33,30 @@ const transformXPath = (
 ): { strategy: string; selector: string } => {
   // normalize xpath expression by replacing " by '
   xpath = xpath.replace(/"/g, "'");
-  if (xpath.includes('@text='))
+  if (xpath.includes('@text=')) {
     return { strategy: 'text', selector: getValue(xpath, '@text=') };
+  }
 
-  if (xpath.includes('@content-desc='))
+  if (xpath.includes('@content-desc=')) {
     return {
       strategy: 'description',
       selector: getValue(xpath, '@content-desc='),
     };
+  }
 
-  if (xpath.includes('contains(@text,'))
+  if (xpath.includes('contains(@text,')) {
     return {
       strategy: 'textContains',
       selector: getValue(xpath, 'contains(@text,'),
     };
+  }
 
-  if (xpath.includes('contains(@content-desc,'))
+  if (xpath.includes('contains(@content-desc,')) {
     return {
       strategy: 'descriptionContains',
       selector: getValue(xpath, 'contains(@content-desc,'),
     };
+  }
 
   throw new Error(
     `Unsupported XPath expression: ${xpath}. Supported expressions are: @text, @content-desc, contains(@text, ...), contains(@content-desc, ...)`
@@ -63,9 +67,15 @@ const transformLocator = (
   strategy: string,
   selector: string
 ): { strategy: string; selector: string } => {
-  if (strategy === 'id') return { strategy: 'resourceId', selector };
-  if (strategy === 'xpath') return transformXPath(selector);
-  if (strategy === 'class name') return { strategy: 'className', selector };
+  if (strategy === 'id') {
+    return { strategy: 'resourceId', selector };
+  }
+  if (strategy === 'xpath') {
+    return transformXPath(selector);
+  }
+  if (strategy === 'class name') {
+    return { strategy: 'className', selector };
+  }
 
   return { strategy, selector };
 };
@@ -103,7 +113,7 @@ async function performiOSScroll(driver: any, direction: string): Promise<void> {
   const { width, height } = await driver.getWindowRect();
 
   await driver.execute('mobile: scroll', {
-    direction: direction,
+    direction,
     startX: width / 2,
     startY: direction === 'up' ? height * 0.3 : height * 0.7,
     endX: width / 2,

package/src/tools/navigations/scroll.ts

@@ -72,18 +72,18 @@ export default function scroll(server: any): void {
             ? await (driver as Client).executeScript('mobile: scroll', [
                 {
                   direction: args.direction,
-                  startX: startX,
-                  startY: startY,
+                  startX,
+                  startY,
                   endX: startX,
-                  endY: endY,
+                  endY,
                 },
               ])
             : await (driver as any).execute('mobile: scroll', {
                 direction: args.direction,
-                startX: startX,
-                startY: startY,
+                startX,
+                startY,
                 endX: startX,
-                endY: endY,
+                endY,
               });
         } else {
           throw new Error(

package/src/tools/navigations/swipe.ts

@@ -67,7 +67,7 @@ async function performAndroidSwipe(
         { type: 'pointerMove', duration: 0, x: startX, y: startY },
         { type: 'pointerDown', button: 0 },
         { type: 'pause', duration: 250 },
-        { type: 'pointerMove', duration: duration, x: endX, y: endY },
+        { type: 'pointerMove', duration, x: endX, y: endY },
         { type: 'pointerUp', button: 0 },
       ],
     },
@@ -102,7 +102,7 @@ async function performiOSSwipe(
           { type: 'pointerMove', duration: 0, x: startX, y: startY },
           { type: 'pointerDown', button: 0 },
           { type: 'pause', duration: 200 },
-          { type: 'pointerMove', duration: duration, x: endX, y: endY },
+          { type: 'pointerMove', duration, x: endX, y: endY },
           { type: 'pause', duration: 50 },
           { type: 'pointerUp', button: 0 },
         ],