Skip to content

fix(doctor): check global agent-browser when local install not found#16125

Closed
ms-alan wants to merge 1 commit into
NousResearch:mainfrom
ms-alan:fix/ISSUE-15951-doctor-agent-browser
Closed

fix(doctor): check global agent-browser when local install not found#16125
ms-alan wants to merge 1 commit into
NousResearch:mainfrom
ms-alan:fix/ISSUE-15951-doctor-agent-browser

Conversation

@ms-alan
Copy link
Copy Markdown
Contributor

@ms-alan ms-alan commented Apr 26, 2026

Closes #15951

When agent-browser is globally installed via 'npm install -g agent-browser' but not present in the local node_modules, doctor falsely warns that it's not installed. Add shutil.which('agent-browser') as a fallback check after the local path check.

When agent-browser is globally installed via 'npm install -g agent-browser'
but not present in the local node_modules, doctor falsely warns that it's
not installed. Add shutil.which('agent-browser') as a fallback check after
the local path check.

Closes NousResearch#15951
@alt-glitch alt-glitch added type/bug Something isn't working comp/cli CLI entry point, hermes_cli/, setup wizard tool/browser Browser automation (CDP, Playwright) P3 Low — cosmetic, nice to have labels Apr 26, 2026
@alt-glitch
Copy link
Copy Markdown
Collaborator

Likely duplicate of #15971 — same shutil.which() fallback for globally-installed agent-browser in doctor check. Both fix #15951.

@teknium1
Copy link
Copy Markdown
Contributor

teknium1 commented May 4, 2026

Salvaged via #19671 onto current main. Original commit's author email was empty (local git config) so it was re-attributed to your GitHub public email during salvage. Thanks @ms-alan!

@teknium1 teknium1 closed this May 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard P3 Low — cosmetic, nice to have tool/browser Browser automation (CDP, Playwright) type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: doctor reports agent-browser not installed when it is globally installed

3 participants