[PR #5205] fix: use basename for shell detection to support non-standard paths #11293

Closed
opened 2026-02-16 18:16:06 -05:00 by yindo · 0 comments
Owner

Original Pull Request: https://github.com/anomalyco/opencode/pull/5205

State: closed
Merged: Yes


The shell exclusion check for fish and nu was hardcoded to specific paths (/bin/fish, /usr/bin/fish, etc.), which failed for users with shells installed in non-standard locations (e.g., /opt/homebrew/bin/fish, ~/.nix-profile/bin/nu).

Now uses path.basename() to check just the shell name regardless of install path.

Resolves: #5190

**Original Pull Request:** https://github.com/anomalyco/opencode/pull/5205 **State:** closed **Merged:** Yes --- The shell exclusion check for fish and nu was hardcoded to specific paths (/bin/fish, /usr/bin/fish, etc.), which failed for users with shells installed in non-standard locations (e.g., /opt/homebrew/bin/fish, ~/.nix-profile/bin/nu). Now uses path.basename() to check just the shell name regardless of install path. Resolves: #5190
yindo added the pull-request label 2026-02-16 18:16:06 -05:00
yindo closed this issue 2026-02-16 18:16:06 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: anomalyco/opencode#11293