appium-mcp 1.8.0 → 1.8.2

package/eslint.config.js

@@ -1,45 +1,19 @@
-import js from '@eslint/js';
-import typescript from '@typescript-eslint/eslint-plugin';
-import typescriptParser from '@typescript-eslint/parser';
-import prettier from 'eslint-plugin-prettier';
-import prettierConfig from 'eslint-config-prettier';
+import appiumConfig from '@appium/eslint-config-appium-ts';
 
 export default [
-  js.configs.recommended,
+  ...appiumConfig,
   {
     files: ['src/**/*.ts'],
-    languageOptions: {
-      parser: typescriptParser,
-      parserOptions: {
-        ecmaVersion: 'latest',
-        sourceType: 'module',
-      },
-      globals: {
-        console: 'readonly',
-        process: 'readonly',
-        Buffer: 'readonly',
-        __dirname: 'readonly',
-        __filename: 'readonly',
-        global: 'readonly',
-        module: 'readonly',
-        require: 'readonly',
-        exports: 'readonly',
-      },
-    },
-    plugins: {
-      '@typescript-eslint': typescript,
-      prettier: prettier,
-    },
     rules: {
-      ...typescript.configs.recommended.rules,
-      ...prettierConfig.rules,
-      'prettier/prettier': 'error',
+      'no-console': 'off',
+      'import/no-named-as-default': 'off',
+      'import/no-named-as-default-member': 'off',
       '@typescript-eslint/no-unused-vars': 'off',
+      '@typescript-eslint/no-non-null-assertion': 'off',
       '@typescript-eslint/explicit-function-return-type': 'off',
       '@typescript-eslint/explicit-module-boundary-types': 'off',
       '@typescript-eslint/no-explicit-any': 'off',
       '@typescript-eslint/ban-ts-comment': 'off',
-      'no-undef': 'off', // TypeScript handles this
       'no-case-declarations': 'off',
     },
   },

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "appium-mcp",
   "mcpName": "io.github.appium/appium-mcp",
-  "version": "1.8.0",
+  "version": "1.8.2",
   "type": "module",
   "repository": {
     "type": "git",
@@ -62,7 +62,6 @@
     "webdriver": "^9.23.0"
   },
   "devDependencies": {
-    "@appium/tsconfig": "^1.0.0",
     "@appium/eslint-config-appium-ts": "^2.0.0",
     "@jest/globals": "^30.2.0",
     "@semantic-release/changelog": "^6.0.3",
@@ -70,9 +69,7 @@
     "@types/jest": "^29.5.12",
     "@types/lodash": "^4.17.17",
     "@types/node": "^22.15.18",
-    "@typescript-eslint/eslint-plugin": "^8.34.0",
     "conventional-changelog-conventionalcommits": "^9.1.0",
-    "eslint-plugin-prettier": "^5.4.1",
     "jest": "^29.7.0",
     "prettier": "^3.5.3",
     "semantic-release": "^25.0.2",

package/server.json

@@ -3,12 +3,12 @@
   "name": "io.github.appium/appium-mcp",
   "title": "MCP Appium - Mobile Development and Automation Server",
   "description": "MCP server for Appium mobile automation on iOS and Android devices with test creation tools.",
-  "version": "1.8.0",
+  "version": "1.8.2",
   "packages": [
     {
       "registryType": "npm",
       "identifier": "appium-mcp",
-      "version": "1.8.0",
+      "version": "1.8.2",
       "transport": {
         "type": "stdio"
       }

package/src/devicemanager/ios-manager.ts

@@ -81,7 +81,7 @@ export class IOSManager {
    */
   public async listBootedSimulators(): Promise<IOSDevice[]> {
     const allSimulators = await this.listSimulators();
-    return allSimulators.filter(simulator => simulator.state === 'Booted');
+    return allSimulators.filter((simulator) => simulator.state === 'Booted');
   }
 
   /**
@@ -98,7 +98,7 @@ export class IOSManager {
       const devices = await utilities.getConnectedDevices();
       return devices.map((udid: string) => ({
         name: udid, // We'll use UDID as name for now
-        udid: udid,
+        udid,
         type: 'real' as const,
       }));
     } catch (error) {

package/src/index.ts

@@ -7,7 +7,7 @@ import log from './logger.js';
 const args = process.argv.slice(2);
 const useHttpStream = args.includes('--httpStream');
 const port =
-  args.find(arg => arg.startsWith('--port='))?.split('=')[1] || '8080';
+  args.find((arg) => arg.startsWith('--port='))?.split('=')[1] || '8080';
 
 async function startServer(): Promise<void> {
   log.info('Starting MCP Appium MCP Server...');

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/screenshot.test.ts

@@ -7,6 +7,7 @@ import {
   jest,
 } from '@jest/globals';
 import { join, isAbsolute } from 'path';
+import * as os from 'node:os';
 
 /**
  * Local implementation of resolveScreenshotDir for testing.
@@ -17,7 +18,7 @@ function resolveScreenshotDir(): string {
   const screenshotDir = process.env.SCREENSHOTS_DIR;
 
   if (!screenshotDir) {
-    return process.cwd();
+    return os.tmpdir();
   }
 
   if (isAbsolute(screenshotDir)) {
@@ -95,15 +96,15 @@ describe('resolveScreenshotDir', () => {
     }
   });
 
-  test('should return process.cwd() when SCREENSHOTS_DIR is not set', () => {
+  test('should return os.tmpdir() when SCREENSHOTS_DIR is not set', () => {
     const result = resolveScreenshotDir();
-    expect(result).toBe(cwd);
+    expect(result).toBe(os.tmpdir());
   });
 
-  test('should return process.cwd() when SCREENSHOTS_DIR is empty string', () => {
+  test('should return os.tmpdir() when SCREENSHOTS_DIR is empty string', () => {
     process.env.SCREENSHOTS_DIR = '';
     const result = resolveScreenshotDir();
-    expect(result).toBe(cwd);
+    expect(result).toBe(os.tmpdir());
   });
 
   test('should return absolute path as-is when SCREENSHOTS_DIR is absolute', () => {

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 },
                 ],
               },