halo-agent 2.2.3 → 2.2.4

package/browser.js

@@ -135,7 +135,7 @@ async function connectToChrome(retries = 10, opts = {}) {
               ctx = fresh.contexts()[0] || await fresh.newContext();
               conn.context = ctx;
             }
-            const page = await ctx.newPage();
+            let page = await ctx.newPage();
             if (url) {
               try {
                 await page.goto(url, { waitUntil: 'domcontentloaded', timeout: 60000 });
@@ -153,6 +153,39 @@ async function connectToChrome(retries = 10, opts = {}) {
                   await page.waitForTimeout(8000);
                 }
               }
+
+              // VERIFY-AFTER-GOTO. connectOverCDP against the user's REAL Chrome
+              // has a tab-handle race: ctx.newPage() may hand back a different
+              // tab than the one that actually navigated (e.g. Chrome already
+              // had a restored New Tab open). Symptom: the agent then scans
+              // about:blank / chrome://newtab forever while the target URL is
+              // loaded in a sibling tab. Self-correct: if our handle isn't on
+              // the target host, find the tab that IS and adopt it.
+              try {
+                const here = page.url();
+                const targetHost = new URL(url).hostname;
+                const onTarget = (u) => { try { return new URL(u).hostname === targetHost; } catch { return false; } };
+                if (!onTarget(here)) {
+                  // Look across all pages in this context for the one that navigated.
+                  const pages = ctx.pages();
+                  const match = pages.find((p) => onTarget(p.url()));
+                  if (match && match !== page) {
+                    console.warn(`[halo-agent] newPage handle was on "${here.slice(0, 40)}" but target loaded in another tab — adopting it.`);
+                    // Close the stray blank tab we opened so it doesn't litter.
+                    if (/^about:blank$|^chrome:\/\/newtab/i.test(here)) {
+                      try { await page.close(); } catch {}
+                    }
+                    page = match;
+                    await page.bringToFront().catch(() => {});
+                  } else if (/^about:blank$|^chrome:\/\/newtab/i.test(here)) {
+                    // Our tab is genuinely blank and nothing else navigated —
+                    // the goto silently no-op'd. Retry the navigation on THIS tab.
+                    console.warn('[halo-agent] Tab still blank after goto — retrying navigation on this handle...');
+                    await page.goto(url, { waitUntil: 'commit', timeout: 30000 }).catch(() => {});
+                    await page.waitForTimeout(4000);
+                  }
+                }
+              } catch { /* best-effort self-correction; fall through */ }
             }
             return page;
           },

package/index.js

@@ -479,7 +479,7 @@ async function runStart() {
 
   let version = '';
   try { version = require('./package.json').version; } catch {}
-  ui.banner(version);
+  ui.bigBanner(version); // hero frame — the one you'd screenshot
 
   // The agent runs its own isolated Chrome instance (separate --user-data-dir
   // at ~/.halo-agent/chrome-profile) so it never collides with the user's

package/package.json

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

package/ui.js

@@ -73,6 +73,26 @@ function banner(version) {
   console.log('');
 }
 
+// Big block-letter wordmark for the hero moment (`start`) — the frame you'd
+// screenshot for a LinkedIn post. Five-row ASCII HALO in ember, tagline +
+// version underneath. Falls back to the compact boxed banner when color is
+// off (logs/CI) so the auto-start service log doesn't get a wall of blocks.
+const HALO_ART = [
+  '██   ██  █████  ██      ██████ ',
+  '██   ██ ██   ██ ██      ██  ██ ',
+  '███████ ███████ ██      ██  ██ ',
+  '██   ██ ██   ██ ██      ██  ██ ',
+  '██   ██ ██   ██ ███████ ██████ ',
+];
+function bigBanner(version) {
+  if (!useColor) { banner(version); return; }
+  const ver = version ? gray('· agent v' + version) : gray('· agent');
+  console.log('');
+  for (const row of HALO_ART) console.log('  ' + ember(row));
+  console.log('  ' + gray('auto-apply, in your own Chrome') + '   ' + ver);
+  console.log('');
+}
+
 // Affordance helper: the obvious next command, rendered consistently so the
 // user always knows what to type next. Shown after pair/init/etc.
 function nextStep(cmd, note) {
@@ -123,5 +143,5 @@ function spinner(message) {
 module.exports = {
   bold, dim, red, green, yellow, blue, magenta, cyan, gray, ember,
   sym, success, fail, info, warn, muted, step,
-  banner, spinner, nextStep, hint, commandTable, visLen,
+  banner, bigBanner, spinner, nextStep, hint, commandTable, visLen,
 };