Skip to content

fix: detect uv installed via curl script#4478

Merged
laithrw merged 3 commits into
browser-use:mainfrom
ishan0102:main
Mar 23, 2026
Merged

fix: detect uv installed via curl script#4478
laithrw merged 3 commits into
browser-use:mainfrom
ishan0102:main

Conversation

@ishan0102
Copy link
Copy Markdown
Contributor

@ishan0102 ishan0102 commented Mar 23, 2026

Fixes bug where browser-use CLI installation failed if the user had installed via curl and note Homebrew.

install_uv checked command -v uv before adding ~/.local/bin to PATH. This worked for Homebrew but not for curl-based installs, because Homebrew installs to a directory already in PATH.


Summary by cubic

Fixes browser-use CLI installs failing when uv was installed via the curl script, and makes the installer script executable. We now add common uv paths to PATH before detection so it works for both curl and Homebrew installs.

  • Bug Fixes
    • Prepend $HOME/.local/bin and $HOME/.cargo/bin to PATH before command -v uv; drop the redundant export after install.
    • Mark browser_use/skill_cli/install.sh as executable so it can run directly.

Written for commit fae8036. Summary will update on new commits.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 23, 2026

CLA assistant check
All committers have signed the CLA.

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file


Since this is your first cubic review, here's how it works:

  • cubic automatically reviews your code and comments on bugs and improvements
  • Teach cubic by replying to its comments. cubic learns from your replies and gets better over time
  • Add one-off context when rerunning by tagging @cubic-dev-ai with guidance or docs links (including llms.txt)
  • Ask questions if you need clarification on any suggestion

@laithrw laithrw merged commit 32e69f3 into browser-use:main Mar 23, 2026
88 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants