You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
getAgentBrowserPath resolves the agent-browser bin script, but the resolved path is handed straight to Bun.spawn / execFileSync in execCommand (src/runner.ts). On Windows that does not work for two reasons:
Only the extensionless shim is probed. The resolver checks <node_modules>/.bin/agent-browser. On Windows, npm/bun generate agent-browser.cmd (and .ps1) alongside it; the extensionless file is a shell script that Windows will not execute.
The JavaScript-bin fallback has no interpreter. When the shim is missing, the resolver falls back to the package's own bin script (agent-browser@0.9.3 ships a JS file). Windows does not honour the shebang line, so spawning that path directly fails.
Suggested fix
Teach the resolver to return a command descriptor rather than a bare path — something like { command, prependArgs } — so the launch site can:
prefer agent-browser.cmd on win32 when it exists, and
invoke a JS bin script through process.execPath (node <script>) instead of relying on shebang handling.
Keep the current direct-execution path unchanged on non-Windows platforms.
Notes
Reported by CodeRabbit on PR feat: CLI surface adapter #16 as an outside-diff comment, labelled Stability & Availability · Major · Heavy lift.
Activity