Skip to content

fix(zsh): Starship prompt ownership with Oh My Zsh - #12

Merged
sempervent merged 5 commits into
masterfrom
fix/zsh-starship-prompt-ownership
Oct 1, 2026
Merged

sempervent merged 5 commits into
masterfrom
fix/zsh-starship-prompt-ownership

Conversation

@sempervent

Copy link
Copy Markdown
Owner

Summary

  • Initialize Starship for Zsh based on prompt_starship_precmd hook presence, not inherited STARSHIP_SHELL.
  • Add regression tests for inherited env, reload idempotence, OMZ stub, fallback, and local override.
  • Extend ./scripts/check.sh with a Prompt section that probes STARSHIP_SHELL=bash.

Test plan

  • ./scripts/ci/lint.sh
  • ./scripts/ci/test.sh (local)
  • bash scripts/tests/zsh_prompt_starship_test.sh
  • Interactive STARSHIP_SHELL=bash zsh -ic shows Starship hooks
  • GitHub Actions CI green

sempervent added 5 commits September 30, 2026 23:19
Initialize Starship for Zsh when prompt_starship_precmd hooks are absent,
instead of trusting inherited STARSHIP_SHELL from parent shells or tmux.
Add isolated OMZ stub tests for STARSHIP_SHELL inheritance, idempotent
reload, fallback prompt, and local PROMPT override behavior.
Report Starship hook registration and config deployment; probe with
inherited STARSHIP_SHELL=bash to catch the regression on health checks.
Avoid hard-coding patch numbers so release bumps do not break macOS smoke.
@sempervent
sempervent merged commit 996d4bc into master Oct 1, 2026
11 checks passed
@sempervent
sempervent deleted the fix/zsh-starship-prompt-ownership branch October 1, 2026 03:40
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