@dynamicu/chromedebug-mcp 2.5.14 → 2.6.1

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

@@ -5,32 +5,36 @@
 
 import { startHttpServer, startWebSocketServer } from './http-server.js';
 import { removePortFile } from './port-discovery.js';
+import { interceptStdout } from './logger.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('');
+// CRITICAL: Intercept stdout FIRST to prevent MCP JSON-RPC pollution
+interceptStdout();
+
+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 +44,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',