appium-mcp 1.1.5 → 1.1.8

package/src/tools/locators.ts

@@ -12,6 +12,7 @@
 import { z } from 'zod';
 import { getDriver } from './session-store.js';
 import { generateAllElementLocators } from '../locators/generate-all-locators.js';
+import log from '../locators/logger.js';
 
 export default function generateLocators(server: any): void {
   server.addTool({
@@ -34,8 +35,6 @@ export default function generateLocators(server: any): void {
           );
         }
 
-        console.log('Getting page source');
-
         try {
           // Get the page source from the driver
           const pageSource = await driver.getPageSource();
@@ -71,11 +70,11 @@ export default function generateLocators(server: any): void {
             ],
           };
         } catch (parseError: any) {
-          console.error('Error parsing XML:', parseError);
+          log.error('Error parsing XML:', parseError);
           throw new Error(`Failed to parse XML: ${parseError.message}`);
         }
       } catch (error: any) {
-        console.error('Error getting page source:', error);
+        log.error('Error getting page source:', error);
         throw new Error(`Failed to get page source: ${error.message}`);
       }
     },

package/src/tools/scroll.ts

@@ -1,5 +1,6 @@
 import { z } from 'zod';
 import { getDriver, getPlatformName } from './session-store.js';
+import log from '../locators/logger.js';
 
 export default function scroll(server: any): void {
   server.addTool({
@@ -25,7 +26,7 @@ export default function scroll(server: any): void {
 
       try {
         const { width, height } = await driver.getWindowSize();
-        console.log('Device screen size:', { width, height });
+        log.info('Device screen size:', { width, height });
         const startX = Math.floor(width / 2);
         // calculate start and end Y positions for scrolling depending on the direction
         // startY is at 80% of the height, endY is at 20% of the height for downward scroll
@@ -42,8 +43,8 @@ export default function scroll(server: any): void {
             ? Math.floor(height * 0.2)
             : Math.floor(height * 0.8);
 
-        console.log('Going to scroll from:', { startX, startY });
-        console.log('Going to scroll to:', { startX, endY });
+        log.info('Going to scroll from:', { startX, startY });
+        log.info('Going to scroll to:', { startX, endY });
 
         if (getPlatformName(driver) === 'Android') {
           await driver.performActions([
@@ -60,7 +61,7 @@ export default function scroll(server: any): void {
               ],
             },
           ]);
-          console.log('Scroll action completed successfully.');
+          log.info('Scroll action completed successfully.');
         } else if (getPlatformName(driver) === 'iOS') {
           await driver.execute('mobile: scroll', {
             direction: args.direction,

package/src/tools/session-store.ts

@@ -1,5 +1,6 @@
 import { AndroidUiautomator2Driver } from 'appium-uiautomator2-driver';
 import { XCUITestDriver } from 'appium-xcuitest-driver';
+import log from '../locators/logger.js';
 
 let driver: any = null;
 let sessionId: string | null = null;
@@ -33,13 +34,13 @@ export function hasActiveSession(): boolean {
 export async function safeDeleteSession(): Promise<boolean> {
   // Check if there's no session to delete
   if (!driver || !sessionId) {
-    console.log('No active session to delete.');
+    log.info('No active session to delete.');
     return false;
   }
 
   // Check if deletion is already in progress
   if (isDeletingSession) {
-    console.log('Session deletion already in progress, skipping...');
+    log.info('Session deletion already in progress, skipping...');
     return false;
   }
 
@@ -47,17 +48,17 @@ export async function safeDeleteSession(): Promise<boolean> {
   isDeletingSession = true;
 
   try {
-    console.log('Deleting current session');
+    log.info('Deleting current session');
     await driver.deleteSession();
 
     // Clear the session from store
     driver = null;
     sessionId = null;
 
-    console.log('Session deleted successfully.');
+    log.info('Session deleted successfully.');
     return true;
   } catch (error) {
-    console.error('Error deleting session:', error);
+    log.error('Error deleting session:', error);
     throw error;
   } finally {
     // Always release lock