Skip to content

fix(ai): add Homebrew + Linuxbrew fallbacks for Claude CLI detection#2897

Merged
abose merged 3 commits into
mainfrom
ai
May 12, 2026
Merged

fix(ai): add Homebrew + Linuxbrew fallbacks for Claude CLI detection#2897
abose merged 3 commits into
mainfrom
ai

Conversation

@abose
Copy link
Copy Markdown
Member

@abose abose commented May 12, 2026

Apple Silicon Macs running brew install --cask claude-code land the binary at /opt/homebrew/bin/claude, which we weren't checking. On macOS, when Phoenix is launched from Finder/Dock the inherited PATH is the minimal /usr/bin:/bin:/usr/sbin:/sbin so which claude misses anything in shell-managed dirs — making the fallback list the only detection path that works.

Also reorder so the docs-recommended installs (~/.local/bin from install.sh, /usr/local/bin) come first and alternative installs (Homebrew, Linuxbrew) come last.

abose added 3 commits May 12, 2026 22:50
Apple Silicon Macs running `brew install --cask claude-code` land the
binary at /opt/homebrew/bin/claude, which we weren't checking. On
macOS, when Phoenix is launched from Finder/Dock the inherited PATH
is the minimal /usr/bin:/bin:/usr/sbin:/sbin so `which claude` misses
anything in shell-managed dirs — making the fallback list the only
detection path that works.

Also reorder so the docs-recommended installs (~/.local/bin from
install.sh, /usr/local/bin) come first and alternative installs
(Homebrew, Linuxbrew) come last.
@abose abose merged commit 6134a6a into main May 12, 2026
7 of 11 checks passed
@abose abose deleted the ai branch May 12, 2026 18:02
@sonarqubecloud
Copy link
Copy Markdown

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.

1 participant