@dynamicu/chromedebug-mcp 2.3.0 → 2.3.1

package/src/database.js

@@ -4,6 +4,7 @@ import path from 'path';
 import { fileURLToPath } from 'url';
 import fs from 'fs';
 import { ProjectManager } from './services/project-manager.js';
+import logger from './utils/logger.js';
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 
@@ -24,7 +25,7 @@ function getDatabasePath() {
     fs.mkdirSync(dbDir, { recursive: true });
   }
   
-  console.log(`[Database] Using global database: ${globalDbPath}`);
+  logger.debug(`[Database] Using global database: ${globalDbPath}`);
   return globalDbPath;
 }
 
@@ -52,7 +53,7 @@ class ChromePilotDatabase {
     // Create tables
     this.createTables();
     this.initialized = true;
-    console.log(`ChromeDebug MCP database initialized at: ${DB_PATH}`);
+    logger.debug(`ChromeDebug MCP database initialized at: ${DB_PATH}`);
   }
 
   createTables() {
@@ -89,11 +90,11 @@ class ChromePilotDatabase {
     // Add interactions column to existing frames table if it doesn't exist
     try {
       this.db.exec(`ALTER TABLE frames ADD COLUMN interactions TEXT DEFAULT '[]'`);
-      console.log('[Database] Added interactions column to frames table');
+      logger.debug('[Database] Added interactions column to frames table');
     } catch (error) {
       // Column already exists, ignore error
       if (!error.message.includes('duplicate column name')) {
-        console.warn('[Database] Unexpected error adding interactions column:', error.message);
+        logger.warn('[Database] Unexpected error adding interactions column:', error.message);
       }
     }
 
@@ -214,41 +215,41 @@ class ChromePilotDatabase {
 
       if (!hasComponent) {
         this.db.exec('ALTER TABLE workflow_actions ADD COLUMN component TEXT');
-        console.log('[Database] Added component column to workflow_actions');
+        logger.debug('[Database] Added component column to workflow_actions');
       }
       if (!hasArgs) {
         this.db.exec('ALTER TABLE workflow_actions ADD COLUMN args TEXT');
-        console.log('[Database] Added args column to workflow_actions');
+        logger.debug('[Database] Added args column to workflow_actions');
       }
       if (!hasStack) {
         this.db.exec('ALTER TABLE workflow_actions ADD COLUMN stack TEXT');
-        console.log('[Database] Added stack column to workflow_actions');
+        logger.debug('[Database] Added stack column to workflow_actions');
       }
 
       // Add enhanced click tracking columns
       if (!hasElementHtml) {
         this.db.exec('ALTER TABLE workflow_actions ADD COLUMN element_html TEXT');
-        console.log('[Database] Added element_html column to workflow_actions');
+        logger.debug('[Database] Added element_html column to workflow_actions');
       }
       if (!hasComponentData) {
         this.db.exec('ALTER TABLE workflow_actions ADD COLUMN component_data TEXT');
-        console.log('[Database] Added component_data column to workflow_actions');
+        logger.debug('[Database] Added component_data column to workflow_actions');
       }
       if (!hasEventHandlers) {
         this.db.exec('ALTER TABLE workflow_actions ADD COLUMN event_handlers TEXT');
-        console.log('[Database] Added event_handlers column to workflow_actions');
+        logger.debug('[Database] Added event_handlers column to workflow_actions');
       }
       if (!hasElementState) {
         this.db.exec('ALTER TABLE workflow_actions ADD COLUMN element_state TEXT');
-        console.log('[Database] Added element_state column to workflow_actions');
+        logger.debug('[Database] Added element_state column to workflow_actions');
       }
       if (!hasPerformanceMetrics) {
         this.db.exec('ALTER TABLE workflow_actions ADD COLUMN performance_metrics TEXT');
-        console.log('[Database] Added performance_metrics column to workflow_actions');
+        logger.debug('[Database] Added performance_metrics column to workflow_actions');
       }
     } catch (error) {
       // Columns might already exist, that's ok
-      console.log('[Database] Function trace columns already exist or migration failed:', error.message);
+      logger.debug('[Database] Function trace columns already exist or migration failed:', error.message);
     }
 
     // Add enhanced columns to screen_interactions table
@@ -263,27 +264,27 @@ class ChromePilotDatabase {
       // Add enhanced click tracking columns to screen_interactions
       if (!hasScreenElementHtml) {
         this.db.exec('ALTER TABLE screen_interactions ADD COLUMN element_html TEXT');
-        console.log('[Database] Added element_html column to screen_interactions');
+        logger.debug('[Database] Added element_html column to screen_interactions');
       }
       if (!hasScreenComponentData) {
         this.db.exec('ALTER TABLE screen_interactions ADD COLUMN component_data TEXT');
-        console.log('[Database] Added component_data column to screen_interactions');
+        logger.debug('[Database] Added component_data column to screen_interactions');
       }
       if (!hasScreenEventHandlers) {
         this.db.exec('ALTER TABLE screen_interactions ADD COLUMN event_handlers TEXT');
-        console.log('[Database] Added event_handlers column to screen_interactions');
+        logger.debug('[Database] Added event_handlers column to screen_interactions');
       }
       if (!hasScreenElementState) {
         this.db.exec('ALTER TABLE screen_interactions ADD COLUMN element_state TEXT');
-        console.log('[Database] Added element_state column to screen_interactions');
+        logger.debug('[Database] Added element_state column to screen_interactions');
       }
       if (!hasScreenPerformanceMetrics) {
         this.db.exec('ALTER TABLE screen_interactions ADD COLUMN performance_metrics TEXT');
-        console.log('[Database] Added performance_metrics column to screen_interactions');
+        logger.debug('[Database] Added performance_metrics column to screen_interactions');
       }
     } catch (error) {
       // Columns might already exist, that's ok
-      console.log('[Database] Screen interactions enhanced columns already exist or migration failed:', error.message);
+      logger.debug('[Database] Screen interactions enhanced columns already exist or migration failed:', error.message);
     }
 
     // Restore points table
@@ -461,12 +462,12 @@ class ChromePilotDatabase {
     const hasScreenshotSettings = columns.some(col => col.name === 'screenshot_settings');
     
     if (!hasNameColumn) {
-      console.log('[Database] Migrating workflow_recordings table to add name column');
+      logger.debug('[Database] Migrating workflow_recordings table to add name column');
       this.db.exec(`ALTER TABLE workflow_recordings ADD COLUMN name TEXT`);
     }
     
     if (!hasScreenshotSettings) {
-      console.log('[Database] Migrating workflow_recordings table to add screenshot_settings column');
+      logger.debug('[Database] Migrating workflow_recordings table to add screenshot_settings column');
       this.db.exec(`ALTER TABLE workflow_recordings ADD COLUMN screenshot_settings TEXT`);
     }
     
@@ -475,7 +476,7 @@ class ChromePilotDatabase {
     const hasScreenshotData = actionColumns.some(col => col.name === 'screenshot_data');
     
     if (!hasScreenshotData) {
-      console.log('[Database] Migrating workflow_actions table to add screenshot_data column');
+      logger.debug('[Database] Migrating workflow_actions table to add screenshot_data column');
       this.db.exec(`ALTER TABLE workflow_actions ADD COLUMN screenshot_data TEXT`);
     }
     
@@ -484,7 +485,7 @@ class ChromePilotDatabase {
     const hasSequenceNumber = logColumns.some(col => col.name === 'sequence_number');
     
     if (!hasSequenceNumber) {
-      console.log('[Database] Migrating console_logs table to add sequence_number column');
+      logger.debug('[Database] Migrating console_logs table to add sequence_number column');
       this.db.exec(`ALTER TABLE console_logs ADD COLUMN sequence_number INTEGER DEFAULT 0`);
     }
 
@@ -495,17 +496,17 @@ class ChromePilotDatabase {
     const hasName = recordingsColumns.some(col => col.name === 'name');
 
     if (!hasRecordingType) {
-      console.log('[Database] Migrating recordings table to add recording_type column');
+      logger.debug('[Database] Migrating recordings table to add recording_type column');
       this.db.exec(`ALTER TABLE recordings ADD COLUMN recording_type TEXT DEFAULT 'continuous'`);
     }
 
     if (!hasUserNote) {
-      console.log('[Database] Migrating recordings table to add user_note column');
+      logger.debug('[Database] Migrating recordings table to add user_note column');
       this.db.exec(`ALTER TABLE recordings ADD COLUMN user_note TEXT`);
     }
 
     if (!hasName) {
-      console.log('[Database] Migrating recordings table to add name column');
+      logger.debug('[Database] Migrating recordings table to add name column');
       this.db.exec(`ALTER TABLE recordings ADD COLUMN name TEXT`);
     }
   }
@@ -523,9 +524,9 @@ class ChromePilotDatabase {
     const recordingId = sessionId;
     const timestamp = Date.now();
 
-    console.log(`[Database] Storing recording: ${recordingId} for session: ${sessionId}, type: ${recordingType}, name: ${name || 'none'}`);
+    logger.debug(`[Database] Storing recording: ${recordingId} for session: ${sessionId}, type: ${recordingType}, name: ${name || 'none'}`);
     const result = stmt.run(recordingId, sessionId, type, timestamp, recordingType, userNote, name);
-    console.log(`[Database] Recording stored with changes: ${result.changes}, lastInsertRowid: ${result.lastInsertRowid}`);
+    logger.debug(`[Database] Recording stored with changes: ${result.changes}, lastInsertRowid: ${result.lastInsertRowid}`);
 
     return recordingId;
   }
@@ -562,7 +563,7 @@ class ChromePilotDatabase {
       this.storeFrameBatch(sessionId, [frameData]);
     }
 
-    console.log(`[Database] Snapshot stored: ${recordingId} with note: ${userNote || 'none'}`);
+    logger.debug(`[Database] Snapshot stored: ${recordingId} with note: ${userNote || 'none'}`);
     return recordingId;
   }
 
@@ -590,26 +591,26 @@ class ChromePilotDatabase {
     // Use base ID instead of prefixed ID
     const recordingId = sessionId;
 
-    console.log(`[storeFrameBatch] DEATH SPIRAL FIX: Processing ${frames.length} frames for session ${sessionId}`);
+    logger.debug(`[storeFrameBatch] DEATH SPIRAL FIX: Processing ${frames.length} frames for session ${sessionId}`);
 
     // Ensure recording exists (but don't overwrite existing)
     const existingRecording = this.getRecording(sessionId);
     if (!existingRecording) {
       this.storeRecording(sessionId, 'frame_capture', 'continuous', null, name);
     } else {
-      console.log(`[storeFrameBatch] Using existing recording: ${existingRecording.id}`);
+      logger.debug(`[storeFrameBatch] Using existing recording: ${existingRecording.id}`);
       // Update name if provided and different
       if (name && name !== existingRecording.name) {
         const updateStmt = this.db.prepare(`UPDATE recordings SET name = ? WHERE id = ?`);
         updateStmt.run(name, sessionId);
-        console.log(`[storeFrameBatch] Updated recording name to: ${name}`);
+        logger.debug(`[storeFrameBatch] Updated recording name to: ${name}`);
       }
     }
     
     // Check existing frame count BEFORE insertion
     const preCountStmt = this.db.prepare(`SELECT COUNT(*) as count FROM frames WHERE recording_id = ?`);
     const preCount = preCountStmt.get(recordingId).count;
-    console.log(`[storeFrameBatch] PRE-INSERT: ${preCount} existing frames in database`);
+    logger.debug(`[storeFrameBatch] PRE-INSERT: ${preCount} existing frames in database`);
     
     // Use INSERT OR REPLACE instead of INSERT OR IGNORE to handle updates
     const frameStmt = this.db.prepare(`
@@ -626,7 +627,7 @@ class ChromePilotDatabase {
         // Use frame.index from extension (properly tracked per session), fallback to i for backward compatibility
         const frameIndex = frame.index !== undefined ? frame.index : i;
         
-        console.log(`[storeFrameBatch] Inserting frame ${frameIndex}: timestamp=${frame.timestamp}, absoluteTimestamp=${frame.absoluteTimestamp}`);
+        logger.debug(`[storeFrameBatch] Inserting frame ${frameIndex}: timestamp=${frame.timestamp}, absoluteTimestamp=${frame.absoluteTimestamp}`);
         
         try {
           // Serialize interactions as JSON
@@ -643,18 +644,18 @@ class ChromePilotDatabase {
           
           if (result.changes > 0) {
             insertedCount++;
-            console.log(`[storeFrameBatch] SUCCESS: Frame ${frameIndex} inserted/updated`);
+            logger.debug(`[storeFrameBatch] SUCCESS: Frame ${frameIndex} inserted/updated`);
           } else {
             skippedCount++;
-            console.log(`[storeFrameBatch] WARNING: Frame ${frameIndex} not changed`);
+            logger.debug(`[storeFrameBatch] WARNING: Frame ${frameIndex} not changed`);
           }
         } catch (error) {
-          console.error(`[storeFrameBatch] ERROR inserting frame ${frameIndex}:`, error.message);
+          logger.error(`[storeFrameBatch] ERROR inserting frame ${frameIndex}:`, error.message);
           throw error; // Fail fast on SQL errors
         }
       }
       
-      console.log(`[storeFrameBatch] Transaction complete: ${insertedCount} inserted, ${skippedCount} skipped`);
+      logger.debug(`[storeFrameBatch] Transaction complete: ${insertedCount} inserted, ${skippedCount} skipped`);
       
       // Update total frames count
       const updateStmt = this.db.prepare(`
@@ -668,13 +669,13 @@ class ChromePilotDatabase {
       // Debug: Check the actual count after update
       const finalCountStmt = this.db.prepare(`SELECT COUNT(*) as count FROM frames WHERE recording_id = ?`);
       const finalCount = finalCountStmt.get(recordingId).count;
-      console.log(`[storeFrameBatch] FINAL: ${finalCount} frames in database (was ${preCount})`);
+      logger.debug(`[storeFrameBatch] FINAL: ${finalCount} frames in database (was ${preCount})`);
       
       return { insertedCount, skippedCount, finalCount };
     });
     
     const result = transaction(frames);
-    console.log(`[storeFrameBatch] DEATH SPIRAL RESOLVED: ${result.insertedCount}/${frames.length} frames stored successfully`);
+    logger.debug(`[storeFrameBatch] DEATH SPIRAL RESOLVED: ${result.insertedCount}/${frames.length} frames stored successfully`);
     
     // Verify what was actually stored
     const verifyStmt = this.db.prepare(`
@@ -683,10 +684,10 @@ class ChromePilotDatabase {
       ORDER BY frame_index ASC
     `);
     const storedFrames = verifyStmt.all(recordingId);
-    console.log(`[storeFrameBatch] VERIFICATION: ${storedFrames.length} frames with indices: [${storedFrames.map(f => f.frame_index).join(', ')}]`);
+    logger.debug(`[storeFrameBatch] VERIFICATION: ${storedFrames.length} frames with indices: [${storedFrames.map(f => f.frame_index).join(', ')}]`);
     
     if (storedFrames.length === 0) {
-      console.error(`[storeFrameBatch] DEATH SPIRAL DETECTED: NO FRAMES STORED! Check SQL constraints and frame data.`);
+      logger.error(`[storeFrameBatch] DEATH SPIRAL DETECTED: NO FRAMES STORED! Check SQL constraints and frame data.`);
       throw new Error(`Frame storage failed: 0 frames stored for session ${sessionId}`);
     }
     
@@ -716,13 +717,13 @@ class ChromePilotDatabase {
     `);
     const frames = framesStmt.all(recordingId);
     
-    console.log(`[associateLogsWithFrames] Found ${frames.length} frames for log association`);
-    console.log(`[associateLogsWithFrames] Frame indices found: [${frames.map(f => f.frame_index).join(', ')}]`);
+    logger.debug(`[associateLogsWithFrames] Found ${frames.length} frames for log association`);
+    logger.debug(`[associateLogsWithFrames] Frame indices found: [${frames.map(f => f.frame_index).join(', ')}]`);
     
     // Debug: Check total frames before processing
     const countStmt = this.db.prepare(`SELECT COUNT(*) as count FROM frames WHERE recording_id = ?`);
     const frameCount = countStmt.get(recordingId).count;
-    console.log(`[associateLogsWithFrames] Total frames in database before log association: ${frameCount}`);
+    logger.debug(`[associateLogsWithFrames] Total frames in database before log association: ${frameCount}`);
     
     if (frames.length === 0) {
       return { success: false, message: 'No frames found for session' };
@@ -775,11 +776,11 @@ class ChromePilotDatabase {
     
     const associatedCount = transaction(logs, frames);
     
-    console.log(`Associated ${associatedCount} of ${logs.length} logs with frames`);
+    logger.debug(`Associated ${associatedCount} of ${logs.length} logs with frames`);
     
     // Debug: Check total frames after processing
     const finalFrameCount = countStmt.get(recordingId).count;
-    console.log(`[associateLogsWithFrames] Total frames in database after log association: ${finalFrameCount}`);
+    logger.debug(`[associateLogsWithFrames] Total frames in database after log association: ${finalFrameCount}`);
     
     return {
       success: true,
@@ -796,7 +797,7 @@ class ChromePilotDatabase {
     // Use base ID instead of prefixed ID
     const recordingId = sessionId;
     
-    console.log(`[streamLogsToFrames] Processing ${logs.length} logs for session: ${sessionId}`);
+    logger.debug(`[streamLogsToFrames] Processing ${logs.length} logs for session: ${sessionId}`);
     
     // Get all frames sorted by timestamp for efficient association
     const framesStmt = this.db.prepare(`
@@ -808,7 +809,7 @@ class ChromePilotDatabase {
     const frames = framesStmt.all(recordingId);
     
     if (frames.length === 0) {
-      console.log(`[streamLogsToFrames] No frames found for session: ${sessionId}, storing ${logs.length} logs as deferred`);
+      logger.debug(`[streamLogsToFrames] No frames found for session: ${sessionId}, storing ${logs.length} logs as deferred`);
       return this.storeDeferredLogs(sessionId, logs);
     }
     
@@ -882,7 +883,7 @@ class ChromePilotDatabase {
     
     const associatedCount = transaction(logs, frames);
     
-    console.log(`[streamLogsToFrames] Streamed ${associatedCount} of ${logs.length} logs to frames`);
+    logger.debug(`[streamLogsToFrames] Streamed ${associatedCount} of ${logs.length} logs to frames`);
     
     return {
       success: true,
@@ -897,7 +898,7 @@ class ChromePilotDatabase {
   storeDeferredLogs(sessionId, logs) {
     this.init();
     
-    console.log(`[storeDeferredLogs] Storing ${logs.length} deferred logs for session: ${sessionId}`);
+    logger.debug(`[storeDeferredLogs] Storing ${logs.length} deferred logs for session: ${sessionId}`);
     
     const deferredLogStmt = this.db.prepare(`
       INSERT INTO deferred_logs (session_id, level, message, timestamp, sequence_number, args)
@@ -918,7 +919,7 @@ class ChromePilotDatabase {
           );
           stored++;
         } catch (error) {
-          console.error(`[storeDeferredLogs] Failed to store deferred log:`, error, log);
+          logger.error(`[storeDeferredLogs] Failed to store deferred log:`, error, log);
         }
       }
       return stored;
@@ -926,7 +927,7 @@ class ChromePilotDatabase {
     
     const storedCount = transaction(logs);
     
-    console.log(`[storeDeferredLogs] Stored ${storedCount} of ${logs.length} deferred logs`);
+    logger.debug(`[storeDeferredLogs] Stored ${storedCount} of ${logs.length} deferred logs`);
     
     return {
       success: true,
@@ -941,7 +942,7 @@ class ChromePilotDatabase {
   associateDeferredLogs(sessionId) {
     this.init();
     
-    console.log(`[associateDeferredLogs] Checking for deferred logs for session: ${sessionId}`);
+    logger.debug(`[associateDeferredLogs] Checking for deferred logs for session: ${sessionId}`);
     
     // Get deferred logs for this session
     const deferredLogsStmt = this.db.prepare(`
@@ -953,11 +954,11 @@ class ChromePilotDatabase {
     const deferredLogs = deferredLogsStmt.all(sessionId);
     
     if (deferredLogs.length === 0) {
-      console.log(`[associateDeferredLogs] No deferred logs found for session: ${sessionId}`);
+      logger.debug(`[associateDeferredLogs] No deferred logs found for session: ${sessionId}`);
       return { success: true, logsAssociated: 0 };
     }
     
-    console.log(`[associateDeferredLogs] Found ${deferredLogs.length} deferred logs, attempting to associate with frames`);
+    logger.debug(`[associateDeferredLogs] Found ${deferredLogs.length} deferred logs, attempting to associate with frames`);
     
     // Convert deferred logs back to the format expected by streamLogsToFrames
     const logs = deferredLogs.map(log => ({
@@ -975,7 +976,7 @@ class ChromePilotDatabase {
       // Successfully associated - clean up deferred logs
       const cleanupStmt = this.db.prepare(`DELETE FROM deferred_logs WHERE session_id = ?`);
       cleanupStmt.run(sessionId);
-      console.log(`[associateDeferredLogs] Successfully associated ${result.logsStreamed} logs and cleaned up deferred storage`);
+      logger.debug(`[associateDeferredLogs] Successfully associated ${result.logsStreamed} logs and cleaned up deferred storage`);
       
       return {
         success: true,
@@ -984,7 +985,7 @@ class ChromePilotDatabase {
       };
     }
     
-    console.log(`[associateDeferredLogs] Frames still not available for session: ${sessionId}`);
+    logger.debug(`[associateDeferredLogs] Frames still not available for session: ${sessionId}`);
     return { success: false, message: 'Frames still not available' };
   }
 
@@ -992,13 +993,13 @@ class ChromePilotDatabase {
   getRecording(recordingId) {
     this.init();
     
-    console.log(`[getRecording] Looking for recording with ID: ${recordingId}`);
+    logger.debug(`[getRecording] Looking for recording with ID: ${recordingId}`);
     
     // Handle both old format (frame_<id>) and new format (base id)
     let baseId = recordingId;
     if (recordingId.startsWith('frame_')) {
       baseId = recordingId.substring(6); // Remove 'frame_' prefix
-      console.log(`[getRecording] Converting old format ID ${recordingId} to base ID: ${baseId}`);
+      logger.debug(`[getRecording] Converting old format ID ${recordingId} to base ID: ${baseId}`);
     }
     
     // Priority order: exact id match, then session_id match with frames, then any session_id match
@@ -1020,11 +1021,11 @@ class ChromePilotDatabase {
     const recording = stmt.get(recordingId, baseId, recordingId, baseId, recordingId, baseId, recordingId);
     
     if (recording) {
-      console.log(`[getRecording] Found recording with ID: ${recording.id}, total_frames: ${recording.total_frames}`);
+      logger.debug(`[getRecording] Found recording with ID: ${recording.id}, total_frames: ${recording.total_frames}`);
       return recording;
     }
     
-    console.log(`[getRecording] No recording found for: ${recordingId} or base ID: ${baseId}`);
+    logger.debug(`[getRecording] No recording found for: ${recordingId} or base ID: ${baseId}`);
     return null;
   }
 
@@ -1086,7 +1087,7 @@ class ChromePilotDatabase {
     try {
       embeddedInteractions = JSON.parse(frame.interactions || '[]');
     } catch (error) {
-      console.warn(`[getFrame] Failed to parse embedded interactions for frame ${frameIndex}:`, error);
+      logger.warn(`[getFrame] Failed to parse embedded interactions for frame ${frameIndex}:`, error);
       embeddedInteractions = [];
     }
 
@@ -1122,7 +1123,7 @@ class ChromePilotDatabase {
       WHERE recording_id = ?
     `);
     const frameStats = allFramesStmt.get(recording.id);
-    console.log(`[getFrameSession] Database stats for ${recording.id}: ${frameStats.count} frames, indices ${frameStats.min_index} to ${frameStats.max_index}`);
+    logger.debug(`[getFrameSession] Database stats for ${recording.id}: ${frameStats.count} frames, indices ${frameStats.min_index} to ${frameStats.max_index}`);
     
     const framesStmt = this.db.prepare(`
       SELECT frame_index, timestamp, absolute_timestamp, image_data, interactions
@@ -1132,8 +1133,8 @@ class ChromePilotDatabase {
     `);
     const frames = framesStmt.all(recording.id);
     
-    console.log(`[getFrameSession] Found ${frames.length} frames in database for session ${sessionId}`);
-    console.log(`[getFrameSession] Frame indices: [${frames.map(f => f.frame_index).join(', ')}]`);
+    logger.debug(`[getFrameSession] Found ${frames.length} frames in database for session ${sessionId}`);
+    logger.debug(`[getFrameSession] Frame indices: [${frames.map(f => f.frame_index).join(', ')}]`);
     
     // Get logs and interactions for all frames
     for (const frame of frames) {
@@ -1172,7 +1173,7 @@ class ChromePilotDatabase {
       try {
         embeddedInteractions = JSON.parse(frame.interactions || '[]');
       } catch (error) {
-        console.warn(`[getFrameSession] Failed to parse embedded interactions for frame ${frame.frame_index}:`, error);
+        logger.warn(`[getFrameSession] Failed to parse embedded interactions for frame ${frame.frame_index}:`, error);
         embeddedInteractions = [];
       }
 
@@ -1316,7 +1317,7 @@ class ChromePilotDatabase {
   storeWorkflowRecording(sessionId, url, title, includeLogs = false, name = null, screenshotSettings = null) {
     this.init();
     
-    console.log('[Database] Storing workflow recording with name:', name, 'sessionId:', sessionId);
+    logger.debug('[Database] Storing workflow recording with name:', name, 'sessionId:', sessionId);
     
     const stmt = this.db.prepare(`
       INSERT OR REPLACE INTO workflow_recordings (id, session_id, name, url, title, timestamp, include_logs, screenshot_settings, updated_at)
@@ -1360,7 +1361,7 @@ class ChromePilotDatabase {
           if (!data) return null;
           let jsonStr = typeof data === 'string' ? data : JSON.stringify(data);
           if (jsonStr.length > maxSize) {
-            console.warn(`[Database] Truncating data field from ${jsonStr.length} to ${maxSize} bytes`);
+            logger.warn(`[Database] Truncating data field from ${jsonStr.length} to ${maxSize} bytes`);
             jsonStr = jsonStr.substring(0, maxSize - 50) + '...[TRUNCATED]';
           }
           return jsonStr;
@@ -1676,7 +1677,7 @@ class ChromePilotDatabase {
       const deleteRecordingsStmt = this.db.prepare(`DELETE FROM workflow_recordings`);
       const result = deleteRecordingsStmt.run();
       
-      console.log(`[Database] Cleared workflow recordings: ${result.changes} recordings deleted`);
+      logger.debug(`[Database] Cleared workflow recordings: ${result.changes} recordings deleted`);
       return result.changes;
     });
     
@@ -1715,7 +1716,7 @@ class ChromePilotDatabase {
       
       return { success: true, restorePointId };
     } catch (error) {
-      console.error('Error storing restore point:', error);
+      logger.error('Error storing restore point:', error);
       return { success: false, error: error.message };
     }
   }
@@ -1785,7 +1786,7 @@ class ChromePilotDatabase {
     `);
     
     stmt.run(pid, mode);
-    console.log(`Registered server instance: PID ${pid}, mode: ${mode}`);
+    logger.debug(`Registered server instance: PID ${pid}, mode: ${mode}`);
   }
   
   async getSingleServerInstance() {
@@ -1824,7 +1825,7 @@ class ChromePilotDatabase {
         // Process is dead, remove it
         const deleteStmt = this.db.prepare(`DELETE FROM server_instances WHERE pid = ?`);
         deleteStmt.run(instance.pid);
-        console.log(`Cleaned up dead server instance: PID ${instance.pid}`);
+        logger.debug(`Cleaned up dead server instance: PID ${instance.pid}`);
       }
     }
   }
@@ -1836,7 +1837,7 @@ class ChromePilotDatabase {
     const result = stmt.run(pid);
     
     if (result.changes > 0) {
-      console.log(`Unregistered server instance: PID ${pid}`);
+      logger.debug(`Unregistered server instance: PID ${pid}`);
     }
   }
 
@@ -1893,7 +1894,7 @@ class ChromePilotDatabase {
       transaction();
       return { success: true, count: interactions.length };
     } catch (error) {
-      console.error('Error storing screen interactions:', error);
+      logger.error('Error storing screen interactions:', error);
       return { success: false, error: error.message };
     }
   }
@@ -1907,7 +1908,7 @@ class ChromePilotDatabase {
     let baseId = recordingId;
     if (recordingId.startsWith('frame_')) {
       baseId = recordingId.substring(6); // Remove 'frame_' prefix
-      console.log(`[getScreenInteractions] Converting old format ID ${recordingId} to base ID: ${baseId}`);
+      logger.debug(`[getScreenInteractions] Converting old format ID ${recordingId} to base ID: ${baseId}`);
     }
 
     // Priority order: exact recording_id match, then baseId match
@@ -1972,11 +1973,11 @@ class ChromePilotDatabase {
     const interactions = stmt.all(frameTimestamp, recordingId, lowerBound, upperBound);
 
     // Diagnostic logging for interaction association debugging
-    console.log(`[getInteractionsForFrame] DEBUGGING - recordingId: ${recordingId}, frameTimestamp: ${frameTimestamp}, window: ±${windowMs}ms`);
-    console.log(`[getInteractionsForFrame] Search range: ${lowerBound} to ${upperBound}`);
-    console.log(`[getInteractionsForFrame] Found ${interactions.length} interactions`);
+    logger.debug(`[getInteractionsForFrame] DEBUGGING - recordingId: ${recordingId}, frameTimestamp: ${frameTimestamp}, window: ±${windowMs}ms`);
+    logger.debug(`[getInteractionsForFrame] Search range: ${lowerBound} to ${upperBound}`);
+    logger.debug(`[getInteractionsForFrame] Found ${interactions.length} interactions`);
     if (interactions.length > 0) {
-      console.log(`[getInteractionsForFrame] First interaction:`, interactions[0]);
+      logger.debug(`[getInteractionsForFrame] First interaction:`, interactions[0]);
     }
 
     // Convert to InteractionData format with metadata