Skip to content

feat: add 'Append Only' shell option to axiom-configure#62

Open
mswell wants to merge 1 commit intoattacksurge:masterfrom
mswell:feat/shell-append-only-option
Open

feat: add 'Append Only' shell option to axiom-configure#62
mswell wants to merge 1 commit intoattacksurge:masterfrom
mswell:feat/shell-append-only-option

Conversation

@mswell
Copy link
Copy Markdown

@mswell mswell commented Mar 26, 2026

Summary

  • Add a new "Append Only" option to the shell setup menu in axiom-configure
  • For users with custom .zshrc/.bashrc setups who don't want their config backed up, replaced, or a new shell installed
  • The option only appends the 2-3 required lines (PATH + completion) to the existing shell config
  • Adds bashcompinit automatically for zsh users (fixes command not found: complete error with ax-completion.sh)
  • Idempotent: skips if lines are already present
  • Also available via --shell append flag for unattended installs

Menu before:

1) Bash  2) Zsh  3) Oh My Zsh

Menu after:

1) Bash  2) Zsh  3) Oh My Zsh  4) Append Only

Test plan

  • Select "Append Only" during interactive install — verify only PATH and completion lines are added
  • Run again — verify it detects lines already present and skips
  • Test with --shell append flag for unattended mode
  • Verify zsh users get bashcompinit line added (fixes complete command compatibility)
  • Verify existing options (Bash, Zsh, Oh My Zsh) still work unchanged

Add a new shell setup option that only appends the required Ax lines
to the user's existing shell config without backing up, replacing,
or installing a new shell. Ideal for users with custom .zshrc/.bashrc
setups who don't want their config modified.

The option:
- Detects current shell (zsh/bash) automatically
- Adds Ax PATH and completion to the existing rc file
- Adds bashcompinit for zsh compatibility with ax-completion.sh
- Skips if lines are already present (idempotent)
- Available via interactive menu ("Append Only") or --shell append flag
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