@dynamicu/chromedebug-mcp 2.6.3 → 2.6.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dynamicu/chromedebug-mcp",
-  "version": "2.6.3",
+  "version": "2.6.4",
   "description": "ChromeDebug MCP - MCP server that provides full control over a Chrome browser instance for debugging and automation with AI assistants like Claude Code",
   "main": "src/index.js",
   "type": "module",

package/src/cli.js

@@ -81,6 +81,11 @@ For details: https://github.com/dynamicupgrade/ChromeDebug#windows-setup
       verbose: args.verbose
     });
 
+    // Kill other instances unless --allow-multiple is set
+    if (!args.allowMultiple) {
+      await sessionManager.killOtherInstances();
+    }
+
     // Register this process for tracking
     await registerProcess(process.pid, 'mcp-server');
 

package/src/index.js

@@ -152,7 +152,8 @@ class ChromePilotApp {
       sessionId: null,
       noCleanup: false,
       verbose: false,
-      dbPath: null
+      dbPath: null,
+      allowMultiple: false
     };
 
     const argv = process.argv.slice(2);
@@ -191,6 +192,9 @@ class ChromePilotApp {
         case '--verbose':
           args.verbose = true;
           break;
+        case '--allow-multiple':
+          args.allowMultiple = true;
+          break;
         case '--help':
           this.showHelp();
           process.exit(0);
@@ -237,6 +241,7 @@ Options:
   --db-path PATH      Set custom database file path (default: ~/.chromedebug/chrome-pilot.db)
   --no-cleanup        Skip cleanup of dead processes on startup
   --verbose           Enable verbose logging including session info
+  --allow-multiple    Allow multiple ChromeDebug instances (default: kill others on startup)
   --help              Show this help message
 
 Session Isolation:

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

@@ -446,6 +446,99 @@ export class UnifiedSessionManager {
     return sessions;
   }
 
+  /**
+   * Kill all other ChromeDebug instances
+   * Uses session files to find PIDs, then kills them gracefully
+   * @returns {Array} Array of killed PIDs
+   */
+  async killOtherInstances() {
+    const sessions = await UnifiedSessionManager.findActiveSessions();
+    const currentPid = process.pid;
+    const killedPids = [];
+
+    for (const session of sessions) {
+      if (session.pid === currentPid) continue; // Don't kill ourselves
+
+      try {
+        // Check if process exists
+        process.kill(session.pid, 0); // Signal 0 = check existence
+
+        // Verify this is actually a ChromeDebug process
+        const isChromeDebug = await this.isChromeDebugProcess(session.pid);
+        if (!isChromeDebug) {
+          console.error(`Skipping PID ${session.pid} - not a ChromeDebug process`);
+          continue;
+        }
+
+        console.error(`Killing ChromeDebug instance: PID ${session.pid} (session ${session.sessionId})`);
+
+        // Send SIGTERM for graceful shutdown
+        process.kill(session.pid, 'SIGTERM');
+        killedPids.push(session.pid);
+
+        // Wait 2 seconds, then SIGKILL if still alive
+        setTimeout(() => {
+          try {
+            process.kill(session.pid, 0); // Check if still alive
+            console.error(`  Process ${session.pid} didn't terminate, sending SIGKILL`);
+            process.kill(session.pid, 'SIGKILL');
+          } catch (err) {
+            // Process already dead, good
+          }
+        }, 2000);
+
+        // Clean up session file
+        const sessionFile = path.join(this.sessionDir, `${session.sessionId}.json`);
+        try {
+          await fs.promises.unlink(sessionFile);
+        } catch (err) {
+          // Ignore errors
+        }
+
+      } catch (error) {
+        if (error.code === 'ESRCH') {
+          // Process doesn't exist, just clean up the session file
+          const sessionFile = path.join(this.sessionDir, `${session.sessionId}.json`);
+          try {
+            await fs.promises.unlink(sessionFile);
+          } catch (err) {
+            // Ignore errors
+          }
+        }
+        // Ignore other errors (permission denied, etc.)
+      }
+    }
+
+    if (killedPids.length > 0) {
+      console.error(`Killed ${killedPids.length} other ChromeDebug instance(s)`);
+    }
+
+    return killedPids;
+  }
+
+  /**
+   * Verify if a PID belongs to a ChromeDebug process
+   * @param {number} pid - Process ID to check
+   * @returns {boolean} True if it's a ChromeDebug process
+   */
+  async isChromeDebugProcess(pid) {
+    try {
+      const { execSync } = await import('child_process');
+
+      // Get process command line
+      const cmd = execSync(`ps -p ${pid} -o command=`, { encoding: 'utf8' }).trim();
+
+      // Check if it's a ChromeDebug process (node running chromedebug or chrome-pilot)
+      return cmd.includes('chromedebug') ||
+             cmd.includes('chrome-pilot') ||
+             cmd.includes('ChromePilot') ||
+             cmd.includes('ChromeDebug');
+    } catch (error) {
+      // If we can't determine, be safe and don't kill
+      return false;
+    }
+  }
+
   /**
    * Clean up stale sessions
    */