Skip to content

Bugfix/issue #122 fish shell support#14

Merged
Lisztos merged 7 commits intomainfrom
bugfix/issue-122-fish-shell-support
Mar 25, 2026
Merged

Bugfix/issue #122 fish shell support#14
Lisztos merged 7 commits intomainfrom
bugfix/issue-122-fish-shell-support

Conversation

@Lisztos
Copy link
Copy Markdown
Owner

@Lisztos Lisztos commented Mar 24, 2026

Ported from parcadei/Continuous-Claude-v3#125 by @stephenfeather


Summary

  • Detect fish shell and set CLAUDE_OPC_DIR/LOOGLE_HOME using fish-compatible syntax (set -gx instead of export)
  • Added a verification step at the end of the wizard to check for CLAUDE_OPC_DIR
  • Updated math features guide with fish instructions

Known Issues (from upstream review)

  • Verification step suggests source command which doesn't work correctly with fish's config.fish
  • Missing directory creation for ~/.config/fish/ before writing to config.fish
  • No tests provided

Test plan

  • Run wizard with fish shell — verify set -gx CLAUDE_OPC_DIR written to config.fish
  • Run wizard with bash/zsh — verify export syntax unchanged
  • Verification step detects missing env var

stephenfeather and others added 7 commits January 19, 2026 12:16
…zard

- Detect fish shell and set CLAUDE_OPC_DIR/LOOGLE_HOME using fish-compatible syntax
- Added a verification step at the end of the wizard to check for CLAUDE_OPC_DIR
- Updated math features guide with fish instructions

Closes parcadei#122
- Create ~/.config/fish/ directory if it doesn't exist (fish creates
  it lazily, so fresh installs would silently skip writing config)
- Apply same guard to LOOGLE_HOME block
- Use dot (.) instead of source for fish in verification step, since
  source is not a fish builtin
@Lisztos Lisztos merged commit 172f4c1 into main Mar 25, 2026
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.

2 participants