halo-agent 1.1.0 → 1.2.0

package/index.js

@@ -291,6 +291,122 @@ async function runInit() {
   console.log('Run "halo-agent start" to begin.\n');
 }
 
+// Is Chrome running without the --remote-debugging-port=9222 flag?
+// Returns true ONLY when we're certain a restart is needed.
+async function detectChromeRunningWithoutDebug() {
+  const { execSync } = require('child_process');
+  try {
+    if (process.platform === 'darwin') {
+      execSync('pgrep -x "Google Chrome"', { stdio: 'ignore' });
+      const psOut = execSync('ps aux', { encoding: 'utf8' });
+      return !psOut.includes('--remote-debugging-port=9222');
+    }
+    if (process.platform === 'win32') {
+      const out = execSync('tasklist /FI "IMAGENAME eq chrome.exe"', { encoding: 'utf8' });
+      if (!/chrome\.exe/i.test(out)) return false;
+      // Windows tasklist doesn't show full cmdline; use wmic as a fallback.
+      try {
+        const wmic = execSync('wmic process where "name=\'chrome.exe\'" get CommandLine', { encoding: 'utf8' });
+        return !wmic.includes('--remote-debugging-port=9222');
+      } catch { return true; } // can't introspect — safer to offer restart
+    }
+    // Linux
+    try { execSync('pgrep -x chrome', { stdio: 'ignore' }); } catch { return false; }
+    try {
+      const ps = execSync('ps -eo args', { encoding: 'utf8' });
+      return !ps.includes('--remote-debugging-port=9222');
+    } catch { return true; }
+  } catch {
+    return false; // pgrep failed → Chrome not running
+  }
+}
+
+// Terminal Y/n prompt for the consent before we touch Chrome.
+function offerChromeRestart() {
+  return new Promise((resolve) => {
+    console.log('');
+    console.log('Chrome is running, but without the debug flag the agent needs.');
+    console.log('I can restart Chrome for you — your tabs will reopen automatically');
+    console.log('(if your "On startup" setting is "Continue where you left off").');
+    console.log('');
+    const readline = require('readline');
+    const rl = readline.createInterface({ input: process.stdin, output: process.stdout });
+    rl.question('Restart Chrome now? [Y/n] ', (ans) => {
+      rl.close();
+      const a = (ans || '').trim().toLowerCase();
+      resolve(a === '' || a === 'y' || a === 'yes');
+    });
+  });
+}
+
+// Graceful quit (preserves tabs), wait until the process is actually gone,
+// then relaunch with the debug flag. Cross-platform.
+async function restartChromeWithDebugFlag() {
+  const { execSync, spawnSync } = require('child_process');
+  const { launchChrome, isChromeDebuggable } = require('./browser');
+
+  console.log('Restarting Chrome…');
+  if (process.platform === 'darwin') {
+    // AppleScript graceful quit — respects Chrome's restore-tabs behavior.
+    try {
+      spawnSync('osascript', ['-e', 'tell application "Google Chrome" to quit'], { stdio: 'ignore' });
+    } catch { /* fall through to pkill if osascript missing */ }
+  } else if (process.platform === 'win32') {
+    // Windows: send WM_CLOSE via taskkill (no /F so it's graceful).
+    try { spawnSync('taskkill', ['/IM', 'chrome.exe'], { stdio: 'ignore' }); } catch {}
+  } else {
+    try { spawnSync('pkill', ['-x', 'chrome'], { stdio: 'ignore' }); } catch {}
+  }
+
+  // Poll until Chrome is gone (up to 10s). If it lingers, the user has a
+  // "leave site?" prompt or modal — we surface that rather than force-killing.
+  const gone = await waitForChromeGone(10_000);
+  if (!gone) {
+    console.error('Chrome didn’t quit (you may have an unsaved-changes prompt).');
+    console.error('Click through any prompts in Chrome, then run `halo-agent start` again.');
+    process.exit(1);
+  }
+
+  // Relaunch with the debug flag + Default profile. Uses the existing
+  // browser.js logic so the path-finding stays in one place.
+  await new Promise(r => setTimeout(r, 800)); // brief breath
+  launchChrome();
+
+  // Wait for the new Chrome to expose CDP. Up to ~15s.
+  for (let i = 0; i < 30; i++) {
+    if (await isChromeDebuggable()) {
+      console.log('Chrome is back, debugging enabled.');
+      return;
+    }
+    await new Promise(r => setTimeout(r, 500));
+  }
+  console.error('Chrome relaunched but CDP didn’t come up in time.');
+  console.error('Try running `halo-agent start` again — Chrome may have just been slow.');
+  process.exit(1);
+}
+
+async function waitForChromeGone(timeoutMs) {
+  const { execSync } = require('child_process');
+  const probe = () => {
+    try {
+      if (process.platform === 'darwin') execSync('pgrep -x "Google Chrome"', { stdio: 'ignore' });
+      else if (process.platform === 'win32') {
+        const out = execSync('tasklist /FI "IMAGENAME eq chrome.exe"', { encoding: 'utf8' });
+        if (!/chrome\.exe/i.test(out)) return false;
+      } else execSync('pgrep -x chrome', { stdio: 'ignore' });
+      return true; // pgrep succeeded → still running
+    } catch {
+      return false; // pgrep failed → process gone
+    }
+  };
+  const start = Date.now();
+  while (Date.now() - start < timeoutMs) {
+    if (!probe()) return true;
+    await new Promise(r => setTimeout(r, 300));
+  }
+  return false;
+}
+
 async function runStart() {
   const config = loadConfig();
   if (!config || !config.token) {
@@ -301,33 +417,22 @@ async function runStart() {
   console.log('\nHALO Agent starting...');
   console.log('Connecting to your Chrome browser...\n');
 
-  // Pre-check: if Chrome is running WITHOUT debug port, warn immediately
+  // Pre-check: Chrome must be running with --remote-debugging-port=9222 for
+  // CDP to work. If it's running WITHOUT that flag, Chrome can't have it
+  // added retroactively — it's a Chrome architecture limit. We offer to
+  // restart Chrome gracefully (preserves tabs via Chrome's "Continue where
+  // you left off" setting) and relaunch with the flag.
   const { isChromeDebuggable } = require('./browser');
   const alreadyDebuggable = await isChromeDebuggable();
   if (!alreadyDebuggable) {
-    const { execSync } = require('child_process');
-    let chromeRunning = false;
-    let chromeHasDebugFlag = false;
-    try {
-      execSync('pgrep -x "Google Chrome"', { stdio: 'ignore' });
-      chromeRunning = true;
-    } catch {}
-    if (chromeRunning) {
-      try {
-        const psOut = execSync('ps aux', { encoding: 'utf8' });
-        chromeHasDebugFlag = psOut.includes('--remote-debugging-port=9222');
-      } catch {}
-    }
-    if (chromeRunning && !chromeHasDebugFlag) {
-      console.error('Chrome is already running WITHOUT remote debugging enabled.\n');
-      console.error('You need to fully quit Chrome first, then relaunch it with the debug flag.\n');
-      console.error('Run these commands:\n');
-      console.error('  pkill -a -i "Google Chrome"');
-      console.error('  sleep 2');
-      console.error('  "/Applications/Google Chrome.app/Contents/MacOS/Google Chrome" --remote-debugging-port=9222 --profile-directory=Default &');
-      console.error('  sleep 4');
-      console.error('  node index.js start\n');
-      process.exit(1);
+    const needsRestart = await detectChromeRunningWithoutDebug();
+    if (needsRestart) {
+      const ok = await offerChromeRestart();
+      if (!ok) {
+        console.error('\nOK — leaving Chrome alone. Run `halo-agent start` again after you’ve quit Chrome.');
+        process.exit(1);
+      }
+      await restartChromeWithDebugFlag();
     }
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "halo-agent",
-  "version": "1.1.0",
+  "version": "1.2.0",
   "description": "HALO local apply agent — auto-fills job applications using your real Chrome session",
   "main": "index.js",
   "bin": {