Skip to content

Fix AI Organize CLI discovery in desktop app#36

Open
ajia1206 wants to merge 1 commit into
alchaincyf:masterfrom
ajia1206:codex/ai-organize-cli-discovery
Open

Fix AI Organize CLI discovery in desktop app#36
ajia1206 wants to merge 1 commit into
alchaincyf:masterfrom
ajia1206:codex/ai-organize-cli-discovery

Conversation

@ajia1206

Copy link
Copy Markdown

Summary

  • Fix AI Organize CLI discovery when FanBox is launched as a macOS desktop app.
  • Add common user CLI locations, including ~/.local/bin, Homebrew, and nvm Node bins, before shell-based lookup.
  • Return an absolute, shell-quoted agent command so the embedded terminal does not depend on its own PATH.

Root Cause

FanBox launched from the GUI does not inherit the user's interactive terminal PATH. The existing zsh -lc "command -v claude" check could miss CLIs installed under user-managed paths even though they worked in Terminal.

Validation

  • node --check server.js
  • git diff --check origin/master...HEAD
  • npm run check:vendor-patch
  • Started server.js with env -i and minimal PATH on FANBOX_PORT=5567
  • curl /api/organize/launch returned ok:true with '/Users/getui/.local/bin/claude' ...

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