@dynamicu/chromedebug-mcp 2.5.14 → 2.5.15

package/src/services/session-manager.js

@@ -47,9 +47,9 @@ export class SessionManager {
       if (this.sessionData && this.sessionData.portLockFile) {
         try {
           await releasePortLock(this.sessionData.portLockFile);
-          console.log(`[SessionManager] Released port lock on process exit for session ${this.sessionData.sessionId}`);
+          console.error(`[SessionManager] Released port lock on process exit for session ${this.sessionData.sessionId}`);
         } catch (error) {
-          console.warn('[SessionManager] Error releasing port lock on exit:', error);
+          console.error('[SessionManager] Error releasing port lock on exit:', error);
         }
       }
     };
@@ -72,12 +72,12 @@ export class SessionManager {
       const projectInfo = this.projectManager.detectProject();
       if (projectInfo) {
         await this.projectManager.initializeProjectStructure();
-        console.log(`[SessionManager] Using project-local storage: ${this.projectManager.getCurrentMode()}`);
+        console.error(`[SessionManager] Using project-local storage: ${this.projectManager.getCurrentMode()}`);
       } else {
-        console.log('[SessionManager] Using global storage mode');
+        console.error('[SessionManager] Using global storage mode');
       }
     } catch (error) {
-      console.warn('[SessionManager] Failed to initialize project mode, falling back to global:', error.message);
+      console.error('[SessionManager] Failed to initialize project mode, falling back to global:', error.message);
     } finally {
       this._projectModeInitialized = true;
     }
@@ -136,7 +136,7 @@ export class SessionManager {
         portLockFile: portLockResult.lockFile // Store lock file for cleanup
       };
 
-      console.log(`[SessionManager] Created session ${sessionId} with profile: ${profilePath}`);
+      console.error(`[SessionManager] Created session ${sessionId} with profile: ${profilePath}`);
       
       // Start heartbeat automatically
       this.startHeartbeat();
@@ -155,7 +155,7 @@ export class SessionManager {
   async updateHeartbeat() {
     if (this.sessionData) {
       this.sessionData.lastHeartbeat = new Date();
-      console.log(`[SessionManager] Updated heartbeat for session ${this.sessionData.sessionId}`);
+      console.error(`[SessionManager] Updated heartbeat for session ${this.sessionData.sessionId}`);
     }
   }
 
@@ -171,7 +171,7 @@ export class SessionManager {
       await this.updateHeartbeat();
     }, this.HEARTBEAT_INTERVAL);
 
-    console.log(`[SessionManager] Started heartbeat for session ${this.sessionId}`);
+    console.error(`[SessionManager] Started heartbeat for session ${this.sessionId}`);
   }
 
   /**
@@ -181,7 +181,7 @@ export class SessionManager {
     if (this.heartbeatInterval) {
       clearInterval(this.heartbeatInterval);
       this.heartbeatInterval = null;
-      console.log(`[SessionManager] Stopped heartbeat for session ${this.sessionId}`);
+      console.error(`[SessionManager] Stopped heartbeat for session ${this.sessionId}`);
     }
   }
 
@@ -240,7 +240,7 @@ export class SessionManager {
   setProcessId(processId) {
     if (this.sessionData) {
       this.sessionData.processId = processId;
-      console.log(`[SessionManager] Set process ID ${processId} for session ${this.sessionId}`);
+      console.error(`[SessionManager] Set process ID ${processId} for session ${this.sessionId}`);
     }
   }
 
@@ -279,11 +279,11 @@ export class SessionManager {
         if (this.projectManager.isProjectModeAvailable()) {
           const projectSessionsDir = path.resolve(this.projectManager.getProjectSessionsDir());
           if (!resolvedPath.startsWith(projectSessionsDir)) {
-            console.warn(`[SessionManager] Refusing to clean unsafe project directory: ${dirPath}`);
+            console.error(`[SessionManager] Refusing to clean unsafe project directory: ${dirPath}`);
             return false;
           }
         } else {
-          console.warn(`[SessionManager] Cannot clean project directory when not in project mode: ${dirPath}`);
+          console.error(`[SessionManager] Cannot clean project directory when not in project mode: ${dirPath}`);
           return false;
         }
       } else {
@@ -293,13 +293,13 @@ export class SessionManager {
         
         if (!resolvedPath.startsWith(resolvedTmpDir) || 
             !path.basename(resolvedPath).startsWith('chromedebug-')) {
-          console.warn(`[SessionManager] Refusing to clean unsafe global directory: ${dirPath}`);
+          console.error(`[SessionManager] Refusing to clean unsafe global directory: ${dirPath}`);
           return false;
         }
       }
 
       await fs.rm(resolvedPath, { recursive: true, force: true });
-      console.log(`[SessionManager] Cleaned up ${isProjectLocal ? 'project-local' : 'global'} directory: ${resolvedPath}`);
+      console.error(`[SessionManager] Cleaned up ${isProjectLocal ? 'project-local' : 'global'} directory: ${resolvedPath}`);
       return true;
     } catch (error) {
       if (error.code === 'ENOENT') {
@@ -336,7 +336,7 @@ export class SessionManager {
             isProjectLocal: true
           })));
         } catch (error) {
-          console.warn('[SessionManager] Could not read project sessions directory:', error.message);
+          console.error('[SessionManager] Could not read project sessions directory:', error.message);
         }
       }
       
@@ -350,7 +350,7 @@ export class SessionManager {
         isProjectLocal: false
       })));
 
-      console.log(`[SessionManager] Found ${dirsToCheck.length} directories to evaluate`);
+      console.error(`[SessionManager] Found ${dirsToCheck.length} directories to evaluate`);
 
       for (const dirInfo of dirsToCheck) {
         try {
@@ -403,7 +403,7 @@ export class SessionManager {
         }
       }
 
-      console.log(`[SessionManager] Cleanup complete: ${cleanupResult.cleanedSessions.length} cleaned, ${cleanupResult.activeSessionsRemaining} active remaining`);
+      console.error(`[SessionManager] Cleanup complete: ${cleanupResult.cleanedSessions.length} cleaned, ${cleanupResult.activeSessionsRemaining} active remaining`);
     } catch (error) {
       console.error('[SessionManager] Error during stale session cleanup:', error);
       throw error;
@@ -437,7 +437,7 @@ export class SessionManager {
               process.kill(this.sessionData.processId, 'SIGKILL');
             }
           } catch (error) {
-            console.warn(`[SessionManager] Error killing process ${this.sessionData.processId}:`, error);
+            console.error(`[SessionManager] Error killing process ${this.sessionData.processId}:`, error);
           }
         }
 
@@ -447,9 +447,9 @@ export class SessionManager {
         if (this.sessionData.portLockFile) {
           try {
             await releasePortLock(this.sessionData.portLockFile);
-            console.log(`[SessionManager] Released port lock for session ${sessionId}`);
+            console.error(`[SessionManager] Released port lock for session ${sessionId}`);
           } catch (error) {
-            console.warn(`[SessionManager] Error releasing port lock for session ${sessionId}:`, error);
+            console.error(`[SessionManager] Error releasing port lock for session ${sessionId}:`, error);
           }
         }
         
@@ -486,7 +486,7 @@ export class SessionManager {
    * Cleanup method called when SessionManager is destroyed
    */
   async cleanup() {
-    console.log(`[SessionManager] Cleaning up session manager for ${this.sessionId}`);
+    console.error(`[SessionManager] Cleaning up session manager for ${this.sessionId}`);
     
     this.stopHeartbeat();
     

package/src/services/session-registry.js

@@ -39,7 +39,7 @@ export class PersistentRegistry {
       ...options
     };
 
-    console.log(`[SessionRegistry] Initialized with registry directory: ${this.registryDir}`);
+    console.error(`[SessionRegistry] Initialized with registry directory: ${this.registryDir}`);
   }
 
   /**
@@ -59,7 +59,7 @@ export class PersistentRegistry {
       await this.cleanupStaleSessions();
 
       this.initialized = true;
-      console.log(`[SessionRegistry] Initialized successfully with ${this.sessions.size} existing sessions`);
+      console.error(`[SessionRegistry] Initialized successfully with ${this.sessions.size} existing sessions`);
     } catch (error) {
       console.error('[SessionRegistry] Initialization failed:', error);
       throw error;
@@ -107,7 +107,7 @@ export class PersistentRegistry {
       // Persist to disk
       await this.saveSessions();
 
-      console.log(`[SessionRegistry] Updated session ${sessionId}`);
+      console.error(`[SessionRegistry] Updated session ${sessionId}`);
     });
   }
 
@@ -135,7 +135,7 @@ export class PersistentRegistry {
       try {
         await fs.access(sessionData.profilePath);
       } catch (error) {
-        console.warn(`[SessionRegistry] Profile no longer exists for session ${sessionId}, removing from registry`);
+        console.error(`[SessionRegistry] Profile no longer exists for session ${sessionId}, removing from registry`);
         await this.removeSession(sessionId);
         return null;
       }
@@ -161,7 +161,7 @@ export class PersistentRegistry {
     await this.withFileLock(async () => {
       this.sessions.delete(sessionId);
       await this.saveSessions();
-      console.log(`[SessionRegistry] Removed session ${sessionId}`);
+      console.error(`[SessionRegistry] Removed session ${sessionId}`);
     });
 
     return true;
@@ -241,7 +241,7 @@ export class PersistentRegistry {
             const lockAge = Date.now() - existingLockData.timestamp;
 
             if (lockAge > this.config.lockTimeoutMs) {
-              console.warn('[SessionRegistry] Removing stale lock file');
+              console.error('[SessionRegistry] Removing stale lock file');
               await fs.unlink(this.lockFile);
               continue; // Retry
             }
@@ -253,14 +253,14 @@ export class PersistentRegistry {
             } catch (processError) {
               if (processError.code === 'ESRCH') {
                 // Process doesn't exist, remove stale lock
-                console.warn('[SessionRegistry] Removing lock file from dead process');
+                console.error('[SessionRegistry] Removing lock file from dead process');
                 await fs.unlink(this.lockFile);
                 continue; // Retry
               }
             }
           } catch (parseError) {
             // Corrupted lock file, remove it
-            console.warn('[SessionRegistry] Removing corrupted lock file');
+            console.error('[SessionRegistry] Removing corrupted lock file');
             await fs.unlink(this.lockFile);
             continue; // Retry
           }
@@ -284,7 +284,7 @@ export class PersistentRegistry {
       await fs.unlink(this.lockFile);
     } catch (error) {
       if (error.code !== 'ENOENT') {
-        console.warn('[SessionRegistry] Error releasing lock:', error);
+        console.error('[SessionRegistry] Error releasing lock:', error);
       }
     }
   }
@@ -310,13 +310,13 @@ export class PersistentRegistry {
         }
       }
 
-      console.log(`[SessionRegistry] Loaded ${this.sessions.size} sessions from disk`);
+      console.error(`[SessionRegistry] Loaded ${this.sessions.size} sessions from disk`);
     } catch (error) {
       if (error.code === 'ENOENT') {
-        console.log('[SessionRegistry] No existing registry file found, starting fresh');
+        console.error('[SessionRegistry] No existing registry file found, starting fresh');
         this.sessions.clear();
       } else {
-        console.warn('[SessionRegistry] Error loading sessions, attempting backup recovery:', error);
+        console.error('[SessionRegistry] Error loading sessions, attempting backup recovery:', error);
         await this.loadFromBackup();
       }
     }
@@ -339,7 +339,7 @@ export class PersistentRegistry {
       await fs.writeFile(tempFile, data, 'utf-8');
       await fs.rename(tempFile, this.registryFile);
 
-      console.log(`[SessionRegistry] Saved ${this.sessions.size} sessions to disk`);
+      console.error(`[SessionRegistry] Saved ${this.sessions.size} sessions to disk`);
     } catch (error) {
       console.error('[SessionRegistry] Error saving sessions:', error);
       throw error;
@@ -364,7 +364,7 @@ export class PersistentRegistry {
       await this.cleanupOldBackups();
     } catch (error) {
       if (error.code !== 'ENOENT') {
-        console.warn('[SessionRegistry] Error creating backup:', error);
+        console.error('[SessionRegistry] Error creating backup:', error);
       }
     }
   }
@@ -381,7 +381,7 @@ export class PersistentRegistry {
         .reverse(); // Most recent first
 
       if (backupFiles.length === 0) {
-        console.log('[SessionRegistry] No backup files found, starting fresh');
+        console.error('[SessionRegistry] No backup files found, starting fresh');
         this.sessions.clear();
         return;
       }
@@ -397,7 +397,7 @@ export class PersistentRegistry {
         }
       }
 
-      console.log(`[SessionRegistry] Recovered ${this.sessions.size} sessions from backup: ${backupFiles[0]}`);
+      console.error(`[SessionRegistry] Recovered ${this.sessions.size} sessions from backup: ${backupFiles[0]}`);
     } catch (error) {
       console.error('[SessionRegistry] Error loading from backup:', error);
       this.sessions.clear();
@@ -423,10 +423,10 @@ export class PersistentRegistry {
       }
 
       if (filesToDelete.length > 0) {
-        console.log(`[SessionRegistry] Cleaned up ${filesToDelete.length} old backup files`);
+        console.error(`[SessionRegistry] Cleaned up ${filesToDelete.length} old backup files`);
       }
     } catch (error) {
-      console.warn('[SessionRegistry] Error cleaning up old backups:', error);
+      console.error('[SessionRegistry] Error cleaning up old backups:', error);
     }
   }
 
@@ -456,7 +456,7 @@ export class PersistentRegistry {
         await this.saveSessions();
       });
 
-      console.log(`[SessionRegistry] Cleaned up ${staleSessions.length} stale sessions`);
+      console.error(`[SessionRegistry] Cleaned up ${staleSessions.length} stale sessions`);
     }
   }
 
@@ -478,7 +478,7 @@ export class PersistentRegistry {
    * Cleanup method for graceful shutdown
    */
   async cleanup() {
-    console.log('[SessionRegistry] Performing final save...');
+    console.error('[SessionRegistry] Performing final save...');
 
     if (this.sessions.size > 0) {
       await this.withFileLock(async () => {
@@ -486,6 +486,6 @@ export class PersistentRegistry {
       });
     }
 
-    console.log('[SessionRegistry] Cleanup completed');
+    console.error('[SessionRegistry] Cleanup completed');
   }
 }

package/src/services/unified-session-manager.js

@@ -81,10 +81,10 @@ export class UnifiedSessionManager {
       this.initialized = true;
 
       if (this.verbose) {
-        console.log(`Session initialized: ${this.sessionId}`);
-        console.log(`  PID: ${this.state.pid}`);
-        console.log(`  Port: ${this.state.port}`);
-        console.log(`  Session file: ${this.sessionFile}`);
+        console.error(`Session initialized: ${this.sessionId}`);
+        console.error(`  PID: ${this.state.pid}`);
+        console.error(`  Port: ${this.state.port}`);
+        console.error(`  Session file: ${this.sessionFile}`);
       }
 
       return this.state;
@@ -338,7 +338,7 @@ export class UnifiedSessionManager {
     await this.updateSessionFile();
 
     if (this.verbose) {
-      console.log(`Registered process: PID ${pid} (${type}) [Session: ${this.sessionId}]`);
+      console.error(`Registered process: PID ${pid} (${type}) [Session: ${this.sessionId}]`);
     }
   }
 
@@ -350,7 +350,7 @@ export class UnifiedSessionManager {
     await this.updateSessionFile();
 
     if (this.verbose) {
-      console.log(`Unregistered process: PID ${pid} [Session: ${this.sessionId}]`);
+      console.error(`Unregistered process: PID ${pid} [Session: ${this.sessionId}]`);
     }
   }
 
@@ -365,7 +365,7 @@ export class UnifiedSessionManager {
         // Handle specific error cases that indicate session cleanup
         if (error.code === 'ENOENT' || error.message.includes('Session file does not exist')) {
           if (this.verbose) {
-            console.log(`Session file removed, stopping heartbeat for session ${this.sessionId}`);
+            console.error(`Session file removed, stopping heartbeat for session ${this.sessionId}`);
           }
           this.stopHeartbeat();
           return;
@@ -373,7 +373,7 @@ export class UnifiedSessionManager {
 
         // Handle permission errors gracefully
         if (error.code === 'EPERM' || error.code === 'EACCES' || error.message.includes('Permission denied')) {
-          console.warn(`Heartbeat permission error for session ${this.sessionId}:`, error.message);
+          console.error(`Heartbeat permission error for session ${this.sessionId}:`, error.message);
           this.stopHeartbeat();
           return;
         }
@@ -396,7 +396,7 @@ export class UnifiedSessionManager {
       this.initialized = false;
 
       if (this.verbose) {
-        console.log(`Heartbeat stopped for session ${this.sessionId}`);
+        console.error(`Heartbeat stopped for session ${this.sessionId}`);
       }
     }
   }
@@ -471,7 +471,7 @@ export class UnifiedSessionManager {
             cleanedCount++;
 
             if (this.verbose) {
-              console.log(`Cleaned up stale session: ${sessionData.sessionId}`);
+              console.error(`Cleaned up stale session: ${sessionData.sessionId}`);
             }
           }
         } catch (error) {
@@ -487,7 +487,7 @@ export class UnifiedSessionManager {
       await this.cleanupStalePortLocks();
 
       if (cleanedCount > 0 && this.verbose) {
-        console.log(`Cleaned up ${cleanedCount} stale sessions`);
+        console.error(`Cleaned up ${cleanedCount} stale sessions`);
       }
     } catch (error) {
       // Directory might not exist
@@ -539,7 +539,7 @@ export class UnifiedSessionManager {
 
     const cleanup = async (reason) => {
       if (this.verbose) {
-        console.log(`Session ${this.sessionId} exiting (${reason}), cleaning up...`);
+        console.error(`Session ${this.sessionId} exiting (${reason}), cleaning up...`);
       }
       await this.cleanup();
     };
@@ -617,7 +617,7 @@ export class UnifiedSessionManager {
       } catch {}
 
       if (this.verbose) {
-        console.log(`Session ${this.sessionId} cleaned up`);
+        console.error(`Session ${this.sessionId} cleaned up`);
       }
     } catch (error) {
       console.error(`Error during session cleanup:`, error.message);

package/src/standalone-server.js

@@ -6,31 +6,31 @@
 import { startHttpServer, startWebSocketServer } from './http-server.js';
 import { removePortFile } from './port-discovery.js';
 
-console.log('╔════════════════════════════════════════════════════════════════╗');
-console.log('║              Chrome Debug HTTP Server                          ║');
-console.log('║                                                                ║');
-console.log('║  This server enables Chrome extension features like:           ║');
-console.log('║  - Frame recording                                             ║');
-console.log('║  - Workflow recording                                          ║');
-console.log('║  - Visual element selection                                    ║');
-console.log('║                                                                ║');
-console.log('║  Keep this running while using Chrome Debug through Claude     ║');
-console.log('╚════════════════════════════════════════════════════════════════╝');
-console.log('');
+console.error('╔════════════════════════════════════════════════════════════════╗');
+console.error('║              Chrome Debug HTTP Server                          ║');
+console.error('║                                                                ║');
+console.error('║  This server enables Chrome extension features like:           ║');
+console.error('║  - Frame recording                                             ║');
+console.error('║  - Workflow recording                                          ║');
+console.error('║  - Visual element selection                                    ║');
+console.error('║                                                                ║');
+console.error('║  Keep this running while using Chrome Debug through Claude     ║');
+console.error('╚════════════════════════════════════════════════════════════════╝');
+console.error('');
 
 async function main() {
   try {
     // Start HTTP server
     const httpPort = await startHttpServer();
-    console.log(`✓ HTTP server started on port ${httpPort}`);
+    console.error(`✓ HTTP server started on port ${httpPort}`);
     
     // Start WebSocket server
     await startWebSocketServer();
-    console.log('✓ WebSocket server started');
+    console.error('✓ WebSocket server started');
     
-    console.log('');
-    console.log('Server is ready! Chrome extension features are now available.');
-    console.log('Press Ctrl+C to stop the server.');
+    console.error('');
+    console.error('Server is ready! Chrome extension features are now available.');
+    console.error('Press Ctrl+C to stop the server.');
     
   } catch (error) {
     console.error('Failed to start server:', error.message);
@@ -40,7 +40,7 @@ async function main() {
 
 // Cleanup on exit
 process.on('SIGINT', () => {
-  console.log('\nShutting down server...');
+  console.error('\nShutting down server...');
   removePortFile();
   process.exit(0);
 });

package/src/utils/extension-path.js

@@ -52,18 +52,18 @@ export function getExtensionPath(config = {}) {
     const resolvedPath = path.resolve(searchPath);
 
     if (validateExtensionPath(resolvedPath)) {
-      console.log(`[ExtensionPath] ✓ Found ChromeDebug extension at: ${resolvedPath}`);
+      console.error(`[ExtensionPath] ✓ Found ChromeDebug extension at: ${resolvedPath}`);
       return resolvedPath;
     }
   }
 
-  console.warn('[ExtensionPath] ⚠ ChromeDebug extension not found in any standard location');
-  console.warn('[ExtensionPath] Searched locations:');
-  searchPaths.filter(Boolean).forEach(p => console.warn(`[ExtensionPath]   - ${p}`));
-  console.warn('[ExtensionPath]');
-  console.warn('[ExtensionPath] To specify a custom location, use:');
-  console.warn('[ExtensionPath]   • Environment variable: CHROMEDEBUG_EXTENSION_PATH=/path/to/extension');
-  console.warn('[ExtensionPath]   • Config file: Set "extensionPath" in config/chrome-pilot-config.json');
+  console.error('[ExtensionPath] ⚠ ChromeDebug extension not found in any standard location');
+  console.error('[ExtensionPath] Searched locations:');
+  searchPaths.filter(Boolean).forEach(p => console.error(`[ExtensionPath]   - ${p}`));
+  console.error('[ExtensionPath]');
+  console.error('[ExtensionPath] To specify a custom location, use:');
+  console.error('[ExtensionPath]   • Environment variable: CHROMEDEBUG_EXTENSION_PATH=/path/to/extension');
+  console.error('[ExtensionPath]   • Config file: Set "extensionPath" in config/chrome-pilot-config.json');
 
   return null;
 }
@@ -114,7 +114,7 @@ export function validateExtensionPath(extensionPath) {
 
     // Verify it's the ChromeDebug extension
     if (!manifest.name?.includes('ChromeDebug')) {
-      console.warn(`[ExtensionPath] ⚠ Extension at ${extensionPath} is not ChromeDebug (found: ${manifest.name})`);
+      console.error(`[ExtensionPath] ⚠ Extension at ${extensionPath} is not ChromeDebug (found: ${manifest.name})`);
       return false;
     }
 

package/src/utils.js

@@ -88,7 +88,7 @@ async function handleExistingLock(lockFile, port) {
       // Try to remove stale lock
       try {
         await fs.promises.unlink(lockFile);
-        console.log(`Cleaned up stale port lock for port ${port}`);
+        console.error(`Cleaned up stale port lock for port ${port}`);
         return true;
       } catch (cleanupError) {
         // Another process might have cleaned it up, that's fine
@@ -127,7 +127,7 @@ async function cleanupStaleLocks(lockDir) {
         
         if (lockAge > STALE_LOCK_THRESHOLD) {
           await fs.promises.unlink(lockFile);
-          console.log(`Cleaned up stale lock file: ${file}`);
+          console.error(`Cleaned up stale lock file: ${file}`);
         }
       } catch (error) {
         // Corrupted lock file, remove it

package/src/validation/schemas.js

@@ -269,8 +269,8 @@ export function createValidator(schema, target = 'body') {
     
     if (error) {
       // Diagnostic logging for validation failures
-      console.log(`[Validation] Failed for ${target}:`, JSON.stringify(data, null, 2));
-      console.log(`[Validation] Errors:`, error.details.map(d => ({ field: d.path.join('.'), message: d.message, value: d.context?.value })));
+      console.error(`[Validation] Failed for ${target}:`, JSON.stringify(data, null, 2));
+      console.error(`[Validation] Errors:`, error.details.map(d => ({ field: d.path.join('.'), message: d.message, value: d.context?.value })));
 
       return res.status(400).json({
         error: 'Validation failed',