Pup - the AI's own browser

@@ -1771,37 +1771,37 @@ async function osBackgroundWatchdog(pid, cdpPort, restoreHwnd) {       '    [WinWd]::AttachThreadInput($pupTid, $myTid, $false) | Out-Null',       '  }',       '}',-      // 1. Wait for the window to EXIST (the event, not a guessed delay). Re-resolve the pid-      //    list every ~500ms (child browser pids can appear late).+      // 1. Wait for the window to EXIST (the event, not a guessed delay). BestWindow (cheap+      //    EnumWindows) runs @50ms; the EXPENSIVE pid-list resolution (Get-NetTCPConnection ++      //    Get-CimInstance) refreshes only every ~500ms — running it every tick overran the+      //    process budget when several windows opened at once (the wiki3-4 'error').+      'function ResolvePids {',+      '  $l = New-Object System.Collections.ArrayList',+      '  if ($targetPid -ne 0) { [void]$l.Add([int]$targetPid) }',+      '  if ($cdpPort -ne 0) { try { $own = Get-NetTCPConnection -LocalPort $cdpPort -State Listen -ErrorAction SilentlyContinue | Select-Object -First 1 -ExpandProperty OwningProcess; if ($own) { [void]$l.Add([int]$own) } } catch {} }',+      '  $anchors = @($l.ToArray())',+      '  foreach ($tp in $anchors) { Get-CimInstance Win32_Process -Filter "ParentProcessId=$tp" -ErrorAction SilentlyContinue | Where-Object { $_.Name -in @("chrome.exe","msedge.exe") } | ForEach-Object { [void]$l.Add([int]$_.ProcessId) } }',+      '  return ,[int[]]@($l.ToArray())',+      '}',       '$pup = [IntPtr]::Zero',+      '$pids = ResolvePids',       '$deadline = [Environment]::TickCount + 4000',+      '$nextResolve = [Environment]::TickCount + 500',       'while ([Environment]::TickCount -lt $deadline) {',-      '  $pidList = New-Object System.Collections.ArrayList',-      '  if ($targetPid -ne 0) { [void]$pidList.Add([int]$targetPid) }',-      '  if ($cdpPort -ne 0) { try { $own = Get-NetTCPConnection -LocalPort $cdpPort -State Listen -ErrorAction SilentlyContinue | Select-Object -First 1 -ExpandProperty OwningProcess; if ($own) { [void]$pidList.Add([int]$own) } } catch {} }',-      '  $anchors = @($pidList.ToArray())',-      '  foreach ($tp in $anchors) { Get-CimInstance Win32_Process -Filter "ParentProcessId=$tp" -ErrorAction SilentlyContinue | Where-Object { $_.Name -in @("chrome.exe","msedge.exe") } | ForEach-Object { [void]$pidList.Add([int]$_.ProcessId) } }',-      '  $pup = [WinWd]::BestWindow([int[]]@($pidList.ToArray()))',+      '  $pup = [WinWd]::BestWindow($pids)',       '  if ($pup -ne [IntPtr]::Zero) { break }',+      '  if ([Environment]::TickCount -ge $nextResolve) { $pids = ResolvePids; $nextResolve = [Environment]::TickCount + 500 }',       '  Start-Sleep -Milliseconds 50',       '}',       'if ($pup -eq [IntPtr]::Zero) { Write-Output "no-window"; exit }',-      // 2. Deactivate FIRST (while still off-screen), THEN park on-screen at z-bottom.+      // 2. Park ONCE: hand back focus only if Chrome somehow took it (with --no-startup-window+      //    the OS denies it the foreground, so this is normally a no-op), then move on-screen+      //    at z-bottom. NO stability loop — there is no self-raise to fight anymore, and any+      //    watch loop inevitably fights a user's early taskbar click (John proved it: clicking+      //    within the first couple of seconds got yanked back). Park, verify, EXIT.       'HandBack $pup',       '[WinWd]::SetWindowPos($pup, $BOTTOM, 0, 0, $scrW, $scrH, $SWP) | Out-Null',-      // 3. Stability watch: react to Chrome re-grabs within 50ms; exit after 500ms of calm.-      '$calmSince = [Environment]::TickCount',-      '$watchEnd = [Environment]::TickCount + 2000',-      'while ([Environment]::TickCount -lt $watchEnd) {',-      '  $fg = [WinWd]::GetForegroundWindow()',-      '  if ($fg -eq $pup) {',-      '    HandBack $pup',-      '    [WinWd]::SetWindowPos($pup, $BOTTOM, 0, 0, $scrW, $scrH, $SWP) | Out-Null',-      '    $calmSince = [Environment]::TickCount',-      '  } elseif (([Environment]::TickCount - $calmSince) -ge 500) { break }',-      '  Start-Sleep -Milliseconds 50',-      '}',-      // 4. Self-verify and exit. Nothing runs after this — the user owns the window now.+      // 3. Self-verify and exit. Nothing runs after this — the user owns the window now.       '$fg2 = [WinWd]::GetForegroundWindow()',       'if ($fg2 -eq $pup) { Write-Output "still-foreground" } else { Write-Output "backgrounded" }',     ].join('\r\n');@@ -2317,11 +2317,18 @@ async function getOrLaunchBrowser(profileName, { freshProfile = false, strictPer       // so browser_screenshot/eval/record all work, but the user NEVER sees it pop up and       // it never covers their active window. foreground:true launches maximized on-screen       // as before. browser_raise_os_window moves it on-screen when the user asks to watch.+      // v1.8.51: `--no-startup-window` is THE fix for the focus-steal war. Windows only grants+      // a process the right to take the foreground at launch. With this flag the browser+      // process starts with NO window at all — the window is created LATER via CDP (newPage),+      // by which time Chrome is a background process and the OS itself DENIES it the+      // foreground. So: no steal is possible, no watchdog/timer war, and a user's taskbar+      // click can never be fought (nothing is running to fight it). The off-screen position+      // args stay as belt-and-suspenders for the brief moment before the park.       const chromeArgs = [         ...launchOpts.args.filter(a => a !== '--remote-debugging-port=0' && (foreground || a !== '--start-maximized')),         `--remote-debugging-port=${debugPort}`,         `--user-data-dir=${userDataDir}`,-        ...(foreground ? [] : ['--window-position=-32000,-32000', '--window-size=1280,800']),+        ...(foreground ? [] : ['--no-startup-window', '--window-position=-32000,-32000', '--window-size=1280,800']),       ];       console.log(`Chrome launch attempt ${attemptNum} for profile "${profileName}" on CDP port ${debugPort}`);       // v1.8.34: snapshot the user's foreground window right before launch (background only)@@ -3484,17 +3491,15 @@ const server = http.createServer(async (req, res) => {             // innerWidth/innerHeight deterministic, which is what fullPage/element shots need.             try { await session.page.setViewport({ width: Math.round(w), height: Math.round(h), deviceScaleFactor: Number(args.deviceScaleFactor) || 1 }); } catch (e) {}           }-          // v1.8.26: re-assert background AFTER any sizing (CDP setWindowBounds can-          // re-activate the window). Use the real minimize-based backgrounder (drops-          // focus), not the old z-order-only one, resolving the window via the browser's-          // pid + CDP port.-          if (!foreground) {+          // v1.8.51: re-assert background ONLY after an explicit sizing call (CDP+          // setWindowBounds can re-activate the window). For a plain open this re-assert is+          // GONE — it duplicated the launch park and was one more thing that could fight a+          // user's early taskbar click. The respectUser foreground check keeps even the sized+          // path from yanking a window the user has up.+          if (!foreground && sized) {             try {               const be = browsers.get(args.profile);-              // v1.8.44: if the user deliberately raised this window to watch it, don't yank it-              // back on a resize either — honor the same latch the open-time retries set.-              if (be && be.browser && be.browser._userForeground) { /* user is watching — leave it */ }-              else if (be && be.browser) be.browser._bgResult = osBackgroundWindowByPid(be.browser._detachedPid, be.browser._cdpPort, be.browser._priorForeground, true);+              if (be && be.browser) be.browser._bgResult = osBackgroundWindowByPid(be.browser._detachedPid, be.browser._cdpPort, be.browser._priorForeground, true);               else osSendWindowToBack(args.profile);             } catch (e) {}           }