@dynamicu/chromedebug-mcp 2.3.0 → 2.4.0

package/src/chrome-controller.js

@@ -7,6 +7,7 @@ import { SessionManager } from './services/session-manager.js';
 import { BrowserDaemon } from './services/browser-daemon.js';
 import { FailoverManager } from './services/failover-manager.js';
 import { getExtensionPath } from './utils/extension-path.js';
+import logger from './utils/logger.js';
 
 export class ChromeController {
   constructor() {
@@ -104,10 +105,10 @@ export class ChromeController {
         this.setActiveTab(targetId);
       }
 
-      console.log(`[TabRegistry] Registered tab ${targetId} for session ${this.sessionId}`);
+      logger.debug(`[TabRegistry] Registered tab ${targetId} for session ${this.sessionId}`);
       return tabData;
     } catch (error) {
-      console.error(`[TabRegistry] Failed to register tab:`, error);
+      logger.error(`[TabRegistry] Failed to register tab:`, error);
       return null;
     }
   }
@@ -120,7 +121,7 @@ export class ChromeController {
     if (tabData) {
       this.activeTargetId = targetId;
       this.page = tabData.page; // Maintain backward compatibility
-      console.log(`[TabRegistry] Active tab set to ${targetId}`);
+      logger.debug(`[TabRegistry] Active tab set to ${targetId}`);
       return true;
     }
     return false;
@@ -158,7 +159,7 @@ export class ChromeController {
     const removed = this.tabRegistry.delete(targetId);
 
     if (removed) {
-      console.log(`[TabRegistry] Unregistered tab ${targetId}`);
+      logger.debug(`[TabRegistry] Unregistered tab ${targetId}`);
 
       // If this was the active tab, select a new active tab
       if (wasActive) {
@@ -182,7 +183,7 @@ export class ChromeController {
     this.tabRegistry.clear();
     this.activeTargetId = null;
     this.page = null;
-    console.log(`[TabRegistry] Cleared all tabs for session ${this.sessionId}`);
+    logger.debug(`[TabRegistry] Cleared all tabs for session ${this.sessionId}`);
   }
 
   /**
@@ -208,7 +209,7 @@ export class ChromeController {
         // Only handle page targets (tabs)
         if (targetInfo.type === 'page') {
           try {
-            console.log(`[TargetEvents] New tab created: ${targetInfo.targetId}`);
+            logger.debug(`[TargetEvents] New tab created: ${targetInfo.targetId}`);
 
             // Wait a bit for the page to be ready
             setTimeout(async () => {
@@ -224,11 +225,11 @@ export class ChromeController {
                   await this.registerTab(newPage, targetInfo.targetId);
                 }
               } catch (error) {
-                console.error(`[TargetEvents] Error registering new tab:`, error);
+                logger.error(`[TargetEvents] Error registering new tab:`, error);
               }
             }, 100);
           } catch (error) {
-            console.error(`[TargetEvents] Error handling target creation:`, error);
+            logger.error(`[TargetEvents] Error handling target creation:`, error);
           }
         }
       });
@@ -236,7 +237,7 @@ export class ChromeController {
       // Listen for tab destruction
       browserClient.on('Target.targetDestroyed', (params) => {
         const { targetId } = params;
-        console.log(`[TargetEvents] Tab destroyed: ${targetId}`);
+        logger.debug(`[TargetEvents] Tab destroyed: ${targetId}`);
         this.unregisterTab(targetId);
       });
 
@@ -249,15 +250,15 @@ export class ChromeController {
           if (tabData) {
             tabData.url = targetInfo.url;
             tabData.title = targetInfo.title || 'Untitled';
-            console.log(`[TargetEvents] Tab info updated: ${targetInfo.targetId} - ${targetInfo.title}`);
+            logger.debug(`[TargetEvents] Tab info updated: ${targetInfo.targetId} - ${targetInfo.title}`);
           }
         }
       });
 
       this.targetEventListenersSetup = true;
-      console.log(`[TabRegistry] Target event listeners setup complete for session ${this.sessionId}`);
+      logger.debug(`[TabRegistry] Target event listeners setup complete for session ${this.sessionId}`);
     } catch (error) {
-      console.error(`[TabRegistry] Failed to setup target event listeners:`, error);
+      logger.error(`[TabRegistry] Failed to setup target event listeners:`, error);
     }
   }
 
@@ -266,13 +267,13 @@ export class ChromeController {
    */
   async initializeTabRegistry() {
     if (!this.browser) {
-      console.warn(`[TabRegistry] Cannot initialize - no browser connection`);
+      logger.warn(`[TabRegistry] Cannot initialize - no browser connection`);
       return;
     }
 
     try {
       const pages = await this.browser.pages();
-      console.log(`[TabRegistry] Initializing with ${pages.length} existing tabs`);
+      logger.debug(`[TabRegistry] Initializing with ${pages.length} existing tabs`);
 
       for (const page of pages) {
         await this.registerTab(page);
@@ -281,9 +282,9 @@ export class ChromeController {
       // Set up event listeners after registry initialization
       await this.setupTargetEventListeners();
 
-      console.log(`[TabRegistry] Initialization complete - ${this.tabRegistry.size} tabs registered`);
+      logger.debug(`[TabRegistry] Initialization complete - ${this.tabRegistry.size} tabs registered`);
     } catch (error) {
-      console.error(`[TabRegistry] Failed to initialize tab registry:`, error);
+      logger.error(`[TabRegistry] Failed to initialize tab registry:`, error);
     }
   }
 
@@ -323,7 +324,7 @@ export class ChromeController {
           }
         }
       } catch (error) {
-        console.error(`[TabRegistry] Error finding valid page:`, error);
+        logger.error(`[TabRegistry] Error finding valid page:`, error);
       }
     }
 
@@ -373,7 +374,7 @@ export class ChromeController {
       if (!options.allowCrossSession) {
         throw new Error(`Cross-session tab access denied. Tab belongs to session ${tabData.sessionId}, current session is ${this.sessionId}`);
       }
-      console.warn(`[TabRegistry] Cross-session tab switch: ${tabData.sessionId} → ${this.sessionId}`);
+      logger.warn(`[TabRegistry] Cross-session tab switch: ${tabData.sessionId} → ${this.sessionId}`);
     }
 
     // Verify the page is still valid
@@ -385,7 +386,7 @@ export class ChromeController {
     // Switch to the tab
     const success = this.setActiveTab(targetId);
     if (success) {
-      console.log(`[TabRegistry] Switched to tab ${targetId} (Session: ${tabData.sessionId})`);
+      logger.debug(`[TabRegistry] Switched to tab ${targetId} (Session: ${tabData.sessionId})`);
       return {
         success: true,
         targetId,
@@ -416,7 +417,7 @@ export class ChromeController {
       const tabData = await this.registerTab(newPage);
 
       if (tabData) {
-        console.log(`[TabRegistry] Created new tab ${tabData.targetId} for session ${this.sessionId}`);
+        logger.debug(`[TabRegistry] Created new tab ${tabData.targetId} for session ${this.sessionId}`);
         return {
           success: true,
           targetId: tabData.targetId,
@@ -427,7 +428,7 @@ export class ChromeController {
 
       throw new Error('Failed to register new tab');
     } catch (error) {
-      console.error(`[TabRegistry] Error creating new tab:`, error);
+      logger.error(`[TabRegistry] Error creating new tab:`, error);
       throw error;
     }
   }
@@ -442,14 +443,14 @@ export class ChromeController {
         // Security: Validate that processId is a valid number and contains no suspicious characters
         const pidStr = String(this.processId).trim();
         if (!/^\d+$/.test(pidStr)) {
-          console.error('Invalid or suspicious process ID for Chrome process:', this.processId);
+          logger.error('Invalid or suspicious process ID for Chrome process:', this.processId);
           this.processId = null;
           return;
         }
         
         const pid = parseInt(pidStr, 10);
         if (isNaN(pid) || pid <= 0 || pid > 4194304) {
-          console.error('Invalid process ID for Chrome process:', this.processId);
+          logger.error('Invalid process ID for Chrome process:', this.processId);
           this.processId = null;
           return;
         }
@@ -478,7 +479,7 @@ export class ChromeController {
           if (error.code === 'ESRCH') {
             // Process already dead, ignore
           } else if (error.code === 'EPERM') {
-            console.error('Permission denied killing Chrome process:', pid);
+            logger.error('Permission denied killing Chrome process:', pid);
           } else {
             throw error;
           }
@@ -487,7 +488,7 @@ export class ChromeController {
         this.processId = null;
       }
     } catch (error) {
-      console.error('Error killing Chrome process:', error.message);
+      logger.error('Error killing Chrome process:', error.message);
     }
   }
 
@@ -513,7 +514,7 @@ export class ChromeController {
     try {
       // Use Node.js fs.rm() instead of shell commands to prevent injection
       await fs.rm(resolvedPath, { recursive: true, force: true });
-      console.error(`Cleaned up directory: ${resolvedPath}`);
+      logger.error(`Cleaned up directory: ${resolvedPath}`);
     } catch (error) {
       if (error.code === 'ENOENT') {
         // Directory doesn't exist, ignore
@@ -542,23 +543,23 @@ export class ChromeController {
               try {
                 process.kill(pid, 0);
                 // Process exists, skip this directory
-                console.error(`Skipping active ChromeDebug MCP directory: ${dir} (PID ${pid} is still running)`);
+                logger.error(`Skipping active ChromeDebug MCP directory: ${dir} (PID ${pid} is still running)`);
               } catch (e) {
                 if (e.code === 'ESRCH') {
                   // Process doesn't exist, safe to clean up
                   const dirPath = path.join(tmpDir, dir);
-                  console.error(`Cleaning up stale ChromeDebug MCP directory: ${dir} (PID ${pid} no longer exists)`);
+                  logger.error(`Cleaning up stale ChromeDebug MCP directory: ${dir} (PID ${pid} no longer exists)`);
                   await fs.rm(dirPath, { recursive: true, force: true });
                 }
               }
             }
           }
         } catch (error) {
-          console.error(`Error processing directory ${dir}:`, error.message);
+          logger.error(`Error processing directory ${dir}:`, error.message);
         }
       }
     } catch (error) {
-      console.error('Error during stale profile cleanup:', error.message);
+      logger.error('Error during stale profile cleanup:', error.message);
     }
   }
 
@@ -584,7 +585,7 @@ export class ChromeController {
           return sharedResult;
         }
       } catch (error) {
-        console.warn('[ChromeController] Shared browser unavailable, falling back to individual browser:', error.message);
+        logger.warn('[ChromeController] Shared browser unavailable, falling back to individual browser:', error.message);
       }
 
       // Fallback to individual browser launch
@@ -594,9 +595,9 @@ export class ChromeController {
       if (sessionData) {
         try {
           await this.sessionManager.forceCleanupSession(sessionData.sessionId);
-          console.error(`Cleaned up session after launch failure: ${sessionData.sessionId}`);
+          logger.error(`Cleaned up session after launch failure: ${sessionData.sessionId}`);
         } catch (cleanupError) {
-          console.error(`Failed to clean up session: ${sessionData.sessionId}`, cleanupError.message);
+          logger.error(`Failed to clean up session: ${sessionData.sessionId}`, cleanupError.message);
         }
       }
 
@@ -616,13 +617,13 @@ export class ChromeController {
    * @returns {Promise<Object>} Connection result
    */
   async connectToSharedBrowser() {
-    console.log('[ChromeController] Attempting to connect to shared browser daemon...');
+    logger.debug('[ChromeController] Attempting to connect to shared browser daemon...');
 
     // Check if we should use fallback based on failover manager
     const shouldUseFallback = await this.failoverManager.shouldUseFallback('connect');
     if (shouldUseFallback) {
       // Instead of immediately failing, try to auto-start a daemon if none exists
-      console.log('[ChromeController] Failover manager recommends fallback, but checking for auto-start opportunity...');
+      logger.debug('[ChromeController] Failover manager recommends fallback, but checking for auto-start opportunity...');
     }
 
     // Try to find a running daemon by checking common ports
@@ -657,7 +658,7 @@ export class ChromeController {
         const isListening = await checkPort();
         if (isListening) {
           daemonPort = port;
-          console.log(`[ChromeController] Found potential daemon on port ${port}`);
+          logger.debug(`[ChromeController] Found potential daemon on port ${port}`);
           break;
         }
       } catch (error) {
@@ -667,10 +668,10 @@ export class ChromeController {
 
     if (!daemonPort) {
       try {
-        console.log('[ChromeController] No existing daemon found, starting new daemon...');
+        logger.debug('[ChromeController] No existing daemon found, starting new daemon...');
         daemonPort = await this.autoStartDaemon();
       } catch (error) {
-        console.warn('[ChromeController] Daemon auto-start failed, falling back to individual browser:', error.message);
+        logger.warn('[ChromeController] Daemon auto-start failed, falling back to individual browser:', error.message);
         // Now honor the failover manager's recommendation if auto-start also failed
         if (shouldUseFallback) {
           throw new Error('Failover manager recommends using individual browser after auto-start failed');
@@ -712,7 +713,7 @@ export class ChromeController {
     // Notify failover manager of successful connection
     await this.failoverManager.recordSuccessfulCheck(Date.now());
 
-    console.log(`[ChromeController] Successfully connected to shared browser daemon on port ${daemonPort}`);
+    logger.debug(`[ChromeController] Successfully connected to shared browser daemon on port ${daemonPort}`);
 
     return {
       success: true,
@@ -730,7 +731,7 @@ export class ChromeController {
    * @returns {Promise<Object>} Launch result
    */
   async launchIndividualBrowser() {
-    console.log('[ChromeController] Launching individual browser instance...');
+    logger.debug('[ChromeController] Launching individual browser instance...');
 
     let sessionData = null;
 
@@ -743,8 +744,8 @@ export class ChromeController {
       this.userDataDir = sessionData.profilePath;
       this.sessionId = sessionData.sessionId;
       
-      console.error(`Launching Chrome with debugging port ${this.debugPort}`);
-      console.error(`User data directory: ${this.userDataDir}`);
+      logger.error(`Launching Chrome with debugging port ${this.debugPort}`);
+      logger.error(`User data directory: ${this.userDataDir}`);
       
       // Detect Chrome executable path
       let executablePath;
@@ -762,7 +763,7 @@ export class ChromeController {
         executablePath = '/usr/bin/google-chrome';
       }
       
-      console.error(`Using Chrome executable: ${executablePath}`);
+      logger.error(`Using Chrome executable: ${executablePath}`);
       
       // Get Chrome extension path with fallback logic
       const extensionPath = getExtensionPath();
@@ -784,7 +785,7 @@ export class ChromeController {
           `--load-extension=${extensionPath}`
         );
       } else {
-        console.warn('[ChromeController] ChromeDebug extension not found. Visual selection features will be unavailable.');
+        logger.warn('[ChromeController] ChromeDebug extension not found. Visual selection features will be unavailable.');
       }
 
       // First, try to launch Chrome with a simpler configuration
@@ -798,7 +799,7 @@ export class ChromeController {
           timeout: 60000
         });
       } catch (launchError) {
-        console.error('Simple launch failed, trying with additional flags...');
+        logger.error('Simple launch failed, trying with additional flags...');
 
         // Build fallback args with more flags
         const fallbackArgs = [
@@ -845,7 +846,7 @@ export class ChromeController {
       
       // Set up disconnect handler
       this.browser.on('disconnected', () => {
-        console.error('Browser disconnected');
+        logger.error('Browser disconnected');
         this.browser = null;
         this.page = null;
         this.client = null;
@@ -899,7 +900,7 @@ export class ChromeController {
       
       // Get the WebSocket endpoint for future connections
       const wsEndpoint = this.browser.wsEndpoint();
-      console.error(`Chrome WebSocket endpoint: ${wsEndpoint}`);
+      logger.error(`Chrome WebSocket endpoint: ${wsEndpoint}`);
       
       return {
         success: true,
@@ -917,9 +918,9 @@ export class ChromeController {
       if (sessionData) {
         try {
           await this.sessionManager.forceCleanupSession(sessionData.sessionId);
-          console.error(`Cleaned up session after launch failure: ${sessionData.sessionId}`);
+          logger.error(`Cleaned up session after launch failure: ${sessionData.sessionId}`);
         } catch (cleanupError) {
-          console.error(`Failed to clean up session: ${sessionData.sessionId}`, cleanupError.message);
+          logger.error(`Failed to clean up session: ${sessionData.sessionId}`, cleanupError.message);
         }
       }
       
@@ -967,7 +968,7 @@ export class ChromeController {
   async autoStartDaemon() {
     // Prevent concurrent daemon starts using class-level lock
     if (ChromeController.daemonStartPromise) {
-      console.log('[ChromeController] Daemon start already in progress, waiting...');
+      logger.debug('[ChromeController] Daemon start already in progress, waiting...');
       return await ChromeController.daemonStartPromise;
     }
 
@@ -994,14 +995,14 @@ export class ChromeController {
       this.isDaemonOwner = true;
 
       const daemonPort = this.sharedDaemon.config.daemonPort;
-      console.log(`[ChromeController] Started shared browser daemon on port ${daemonPort}`);
+      logger.debug(`[ChromeController] Started shared browser daemon on port ${daemonPort}`);
 
       // Set up cleanup handlers for this specific daemon owner
       this.setupDaemonCleanup();
 
       return daemonPort;
     } catch (error) {
-      console.error('[ChromeController] Failed to start daemon:', error.message);
+      logger.error('[ChromeController] Failed to start daemon:', error.message);
       this.sharedDaemon = null;
       this.isDaemonOwner = false;
       throw new Error(`Could not start shared browser daemon: ${error.message}`);
@@ -1016,11 +1017,11 @@ export class ChromeController {
 
     const cleanupHandler = async () => {
       if (this.isDaemonOwner && this.sharedDaemon) {
-        console.log('[ChromeController] Cleaning up auto-started daemon...');
+        logger.debug('[ChromeController] Cleaning up auto-started daemon...');
         try {
           await this.sharedDaemon.gracefulShutdown();
         } catch (error) {
-          console.error('[ChromeController] Error stopping daemon:', error.message);
+          logger.error('[ChromeController] Error stopping daemon:', error.message);
         }
         this.sharedDaemon = null;
         this.isDaemonOwner = false;
@@ -1040,7 +1041,7 @@ export class ChromeController {
         await this.close();
       }
       
-      console.error(`Attempting to connect to Chrome on port ${port}...`);
+      logger.error(`Attempting to connect to Chrome on port ${port}...`);
       
       // Try connecting with a timeout
       this.browser = await this.withTimeout(
@@ -1053,10 +1054,10 @@ export class ChromeController {
       );
       
       this.debugPort = port;
-      console.error(`Successfully connected to existing Chrome on port ${port}`);
+      logger.error(`Successfully connected to existing Chrome on port ${port}`);
       
       this.browser.on('disconnected', () => {
-        console.error('Browser disconnected');
+        logger.error('Browser disconnected');
         this.browser = null;
         this.page = null;
         this.client = null;
@@ -1647,7 +1648,7 @@ export class ChromeController {
   }
 
   async forceReset() {
-    console.error('Force resetting Chrome...');
+    logger.error('Force resetting Chrome...');
 
     await this.killOwnChromeProcess();
 
@@ -1658,9 +1659,9 @@ export class ChromeController {
     if (this.sessionManager && this.sessionId) {
       try {
         await this.sessionManager.forceCleanupSession(this.sessionId);
-        console.log(`[ChromeController] Force cleanup completed for session ${this.sessionId}`);
+        logger.debug(`[ChromeController] Force cleanup completed for session ${this.sessionId}`);
       } catch (error) {
-        console.error('[ChromeController] Error during force cleanup:', error.message);
+        logger.error('[ChromeController] Error during force cleanup:', error.message);
       }
     }
 
@@ -1686,7 +1687,7 @@ export class ChromeController {
       try {
         await this.browser.close();
       } catch (e) {
-        console.error('Error closing browser:', e.message);
+        logger.error('Error closing browser:', e.message);
       } finally {
         this.browser = null;
         this.page = null;
@@ -1699,9 +1700,9 @@ export class ChromeController {
     if (this.sessionManager) {
       try {
         await this.sessionManager.cleanup();
-        console.log(`[ChromeController] Session cleanup completed for ${this.sessionId}`);
+        logger.debug(`[ChromeController] Session cleanup completed for ${this.sessionId}`);
       } catch (error) {
-        console.error('[ChromeController] Error during session cleanup:', error.message);
+        logger.error('[ChromeController] Error during session cleanup:', error.message);
       }
     }
 
@@ -1717,11 +1718,11 @@ export class ChromeController {
   async cleanup() {
     // Clean up auto-started daemon first
     if (this.isDaemonOwner && this.sharedDaemon) {
-      console.log('[ChromeController] Cleaning up auto-started daemon during cleanup...');
+      logger.debug('[ChromeController] Cleaning up auto-started daemon during cleanup...');
       try {
         await this.sharedDaemon.gracefulShutdown();
       } catch (error) {
-        console.error('[ChromeController] Error stopping daemon during cleanup:', error.message);
+        logger.error('[ChromeController] Error stopping daemon during cleanup:', error.message);
       }
       this.sharedDaemon = null;
       this.isDaemonOwner = false;
@@ -1987,7 +1988,7 @@ export class ChromeController {
       // Merge function traces into actions if provided
       let allActions = [...actions];
       if (functionTraces && functionTraces.length > 0) {
-        console.log(`[ChromeController] Merging ${functionTraces.length} function traces into actions`);
+        logger.debug(`[ChromeController] Merging ${functionTraces.length} function traces into actions`);
         // Function traces are already in the correct format from the Chrome extension
         allActions = [...actions, ...functionTraces];
         // Sort all actions by timestamp to maintain chronological order
@@ -2005,10 +2006,10 @@ export class ChromeController {
       // Store logs if provided
       if (includeLogs && logs && logs.length > 0) {
         const logsResult = database.storeWorkflowLogs(workflowId, logs);
-        console.log(`Stored ${logsResult.storedCount} logs for workflow ${sessionId}`);
+        logger.debug(`Stored ${logsResult.storedCount} logs for workflow ${sessionId}`);
       }
       
-      console.log(`Stored workflow recording ${sessionId} with ${actionsResult.storedCount} actions (including ${functionTraces ? functionTraces.length : 0} function traces)`);
+      logger.debug(`Stored workflow recording ${sessionId} with ${actionsResult.storedCount} actions (including ${functionTraces ? functionTraces.length : 0} function traces)`);
       
       return {
         success: true,
@@ -2019,7 +2020,7 @@ export class ChromeController {
         functionTracesStored: functionTraces ? functionTraces.length : 0
       };
     } catch (error) {
-      console.error('Error storing workflow recording:', error);
+      logger.error('Error storing workflow recording:', error);
       throw error;
     }
   }
@@ -2029,7 +2030,7 @@ export class ChromeController {
       const { database } = await import('./database.js');
       return database.getWorkflowRecording(sessionId);
     } catch (error) {
-      console.error('Error retrieving workflow recording:', error);
+      logger.error('Error retrieving workflow recording:', error);
       throw error;
     }
   }
@@ -2040,7 +2041,7 @@ export class ChromeController {
       const recordings = database.listWorkflowRecordings();
       return { recordings };
     } catch (error) {
-      console.error('Error listing workflow recordings:', error);
+      logger.error('Error listing workflow recordings:', error);
       throw error;
     }
   }
@@ -2062,7 +2063,7 @@ export class ChromeController {
         recording: recording
       };
     } catch (error) {
-      console.error('Error playing workflow recording:', error);
+      logger.error('Error playing workflow recording:', error);
       throw error;
     }
   }
@@ -2094,7 +2095,7 @@ export class ChromeController {
         recording: fullRecording
       };
     } catch (error) {
-      console.error('Error playing workflow by name:', error);
+      logger.error('Error playing workflow by name:', error);
       throw error;
     }
   }
@@ -2110,7 +2111,7 @@ export class ChromeController {
       );
       return result;
     } catch (error) {
-      console.error('Error saving restore point:', error);
+      logger.error('Error saving restore point:', error);
       throw error;
     }
   }
@@ -2121,7 +2122,7 @@ export class ChromeController {
       const { database } = await import('./database.js');
       return database.getRestorePoint(restorePointId);
     } catch (error) {
-      console.error('Error getting restore point:', error);
+      logger.error('Error getting restore point:', error);
       throw error;
     }
   }
@@ -2132,7 +2133,7 @@ export class ChromeController {
       const { database } = await import('./database.js');
       return database.listRestorePoints(workflowId);
     } catch (error) {
-      console.error('Error listing restore points:', error);
+      logger.error('Error listing restore points:', error);
       throw error;
     }
   }
@@ -2146,7 +2147,7 @@ export class ChromeController {
       const result = deleteStmt.run(restorePointId);
       return { success: result.changes > 0 };
     } catch (error) {
-      console.error('Error deleting restore point:', error);
+      logger.error('Error deleting restore point:', error);
       throw error;
     }
   }
@@ -2157,7 +2158,7 @@ export class ChromeController {
       const { database } = await import('./database.js');
       return database.getFrameSessionInfo(sessionId);
     } catch (error) {
-      console.error('Error getting frame session info:', error);
+      logger.error('Error getting frame session info:', error);
       throw error;
     }
   }
@@ -2167,7 +2168,7 @@ export class ChromeController {
       const { database } = await import('./database.js');
       return database.getFrame(sessionId, frameIndex);
     } catch (error) {
-      console.error('Error getting frame:', error);
+      logger.error('Error getting frame:', error);
       throw error;
     }
   }
@@ -2177,7 +2178,7 @@ export class ChromeController {
       const { database } = await import('./database.js');
       return database.getFrameScreenshot(sessionId, frameIndex, includeMetadata);
     } catch (error) {
-      console.error('Error getting frame screenshot:', error);
+      logger.error('Error getting frame screenshot:', error);
       throw error;
     }
   }
@@ -2187,7 +2188,7 @@ export class ChromeController {
       const { database } = await import('./database.js');
       return database.getScreenInteractions(sessionId);
     } catch (error) {
-      console.error('Error retrieving screen interactions:', error);
+      logger.error('Error retrieving screen interactions:', error);
       throw error;
     }
   }
@@ -2197,7 +2198,7 @@ export class ChromeController {
       const { database } = await import('./database.js');
       return database.getFrameInteractions(sessionId, frameIndex);
     } catch (error) {
-      console.error('Error retrieving frame interactions:', error);
+      logger.error('Error retrieving frame interactions:', error);
       throw error;
     }
   }
@@ -2207,7 +2208,7 @@ export class ChromeController {
       const { database } = await import('./database.js');
       return database.getFrameSession(sessionId);
     } catch (error) {
-      console.error('Error getting frame session:', error);
+      logger.error('Error getting frame session:', error);
       throw error;
     }
   }
@@ -2219,21 +2220,21 @@ export class ChromeController {
 
       // After storing frames, attempt to associate any deferred logs
       if (result && result.id) {
-        console.log(`[storeFrameBatch] Frames stored successfully, checking for deferred logs...`);
+        logger.debug(`[storeFrameBatch] Frames stored successfully, checking for deferred logs...`);
         try {
           const deferredResult = await database.associateDeferredLogs(sessionId);
           if (deferredResult.success && deferredResult.logsAssociated > 0) {
-            console.log(`[storeFrameBatch] Associated ${deferredResult.logsAssociated} deferred logs with new frames`);
+            logger.debug(`[storeFrameBatch] Associated ${deferredResult.logsAssociated} deferred logs with new frames`);
           }
         } catch (deferredError) {
-          console.error('[storeFrameBatch] Error associating deferred logs:', deferredError);
+          logger.error('[storeFrameBatch] Error associating deferred logs:', deferredError);
           // Don't fail the frame storage if deferred log association fails
         }
       }
 
       return result;
     } catch (error) {
-      console.error('Error storing frame batch:', error);
+      logger.error('Error storing frame batch:', error);
       throw error;
     }
   }
@@ -2243,7 +2244,7 @@ export class ChromeController {
       const { database } = await import('./database.js');
       return database.associateLogsWithFrames(sessionId, logs);
     } catch (error) {
-      console.error('Error associating logs with frames:', error);
+      logger.error('Error associating logs with frames:', error);
       throw error;
     }
   }
@@ -2253,7 +2254,7 @@ export class ChromeController {
       const { database } = await import('./database.js');
       return database.streamLogsToFrames(sessionId, logs);
     } catch (error) {
-      console.error('Error streaming logs to frames:', error);
+      logger.error('Error streaming logs to frames:', error);
       throw error;
     }
   }
@@ -2281,7 +2282,7 @@ export class ChromeController {
       const { database } = await import('./database.js');
       return database.storeSnapshot(sessionId, frameData, userNote);
     } catch (error) {
-      console.error('Error storing snapshot:', error);
+      logger.error('Error storing snapshot:', error);
       throw error;
     }
   }
@@ -2291,7 +2292,7 @@ export class ChromeController {
       const { database } = await import('./database.js');
       return database.getSnapshots(limit);
     } catch (error) {
-      console.error('Error getting snapshots:', error);
+      logger.error('Error getting snapshots:', error);
       throw error;
     }
   }
@@ -2334,7 +2335,7 @@ export class ChromeController {
         message: `Snapshot created successfully with ID: ${result}`
       };
     } catch (error) {
-      console.error('Error taking snapshot:', error);
+      logger.error('Error taking snapshot:', error);
       throw error;
     }
   }
@@ -2345,7 +2346,7 @@ export class ChromeController {
       const { database } = await import('./database.js');
       return database.listRecordings();
     } catch (error) {
-      console.error('Error listing frame sessions:', error);
+      logger.error('Error listing frame sessions:', error);
       throw error;
     }
   }
@@ -2393,7 +2394,7 @@ export class ChromeController {
         timestamp: log.timestamp
       }));
     } catch (error) {
-      console.error('Error searching frame logs:', error);
+      logger.error('Error searching frame logs:', error);
       throw error;
     }
   }
@@ -2463,7 +2464,7 @@ export class ChromeController {
         logs
       };
     } catch (error) {
-      console.error('Error getting paginated frame logs:', error);
+      logger.error('Error getting paginated frame logs:', error);
       throw error;
     }
   }
@@ -2478,7 +2479,7 @@ export class ChromeController {
         return { success: false, error: 'Recording not found' };
       }
     } catch (error) {
-      console.error('Error deleting recording:', error);
+      logger.error('Error deleting recording:', error);
       throw error;
     }
   }
@@ -2493,7 +2494,7 @@ export class ChromeController {
         return { success: false, error: 'Workflow recording not found' };
       }
     } catch (error) {
-      console.error('Error deleting workflow recording:', error);
+      logger.error('Error deleting workflow recording:', error);
       throw error;
     }
   }
@@ -2541,7 +2542,7 @@ export class ChromeController {
       if (workflow.url) {
         const currentUrl = currentPage.url();
         if (currentUrl !== workflow.url) {
-          console.log(`Navigating to workflow URL: ${workflow.url}`);
+          logger.debug(`Navigating to workflow URL: ${workflow.url}`);
           await currentPage.goto(workflow.url, { waitUntil: 'networkidle2' });
           await new Promise(resolve => setTimeout(resolve, 1000)); // Wait for page to settle
         }
@@ -2550,7 +2551,7 @@ export class ChromeController {
       // Execute each action
       for (const action of actions) {
         try {
-          console.log(`Executing action ${executedActions + 1}/${actions.length}: ${action.type}`);
+          logger.debug(`Executing action ${executedActions + 1}/${actions.length}: ${action.type}`);
           
           // Calculate delay based on speed
           const delay = action.timestamp && actions.indexOf(action) > 0 
@@ -2566,13 +2567,13 @@ export class ChromeController {
             case 'click':
               if (action.selector) {
                 const fixedSelector = this.fixSelector(action.selector);
-                console.log(`Fixed selector: ${fixedSelector} (original: ${action.selector})`);
+                logger.debug(`Fixed selector: ${fixedSelector} (original: ${action.selector})`);
                 
                 try {
                   await currentPage.waitForSelector(fixedSelector, { timeout: 5000 });
                   await currentPage.click(fixedSelector);
                 } catch (selectorError) {
-                  console.error(`Failed with CSS selector, trying coordinates fallback`);
+                  logger.error(`Failed with CSS selector, trying coordinates fallback`);
                   if (action.x && action.y) {
                     await currentPage.mouse.click(action.x, action.y);
                   } else {
@@ -2587,7 +2588,7 @@ export class ChromeController {
             case 'input':
               if (action.selector) {
                 const fixedSelector = this.fixSelector(action.selector);
-                console.log(`Fixed input selector: ${fixedSelector} (original: ${action.selector})`);
+                logger.debug(`Fixed input selector: ${fixedSelector} (original: ${action.selector})`);
                 
                 await currentPage.waitForSelector(fixedSelector, { timeout: 5000 });
                 await currentPage.click(fixedSelector); // Focus the element
@@ -2625,17 +2626,17 @@ export class ChromeController {
               // Wait for navigation to complete
               await currentPage.waitForNavigation({ waitUntil: 'networkidle2', timeout: 10000 }).catch(() => {
                 // Navigation might not happen, continue
-                console.log('Navigation timeout, continuing...');
+                logger.debug('Navigation timeout, continuing...');
               });
               break;
 
             default:
-              console.warn(`Unknown action type: ${action.type}`);
+              logger.warn(`Unknown action type: ${action.type}`);
           }
 
           executedActions++;
         } catch (error) {
-          console.error(`Error executing action ${executedActions + 1}:`, error);
+          logger.error(`Error executing action ${executedActions + 1}:`, error);
           lastError = error.message;
           // Continue with next action on error
         }
@@ -2651,7 +2652,7 @@ export class ChromeController {
         error: lastError
       };
     } catch (error) {
-      console.error('Error playing workflow:', error);
+      logger.error('Error playing workflow:', error);
       throw error;
     }
   }