Skip to content

fix(cli): generate native zsh completions#481

Draft
piiq wants to merge 1 commit intosteipete:mainfrom
piiq:codex/fix-zsh-completion-script
Draft

fix(cli): generate native zsh completions#481
piiq wants to merge 1 commit intosteipete:mainfrom
piiq:codex/fix-zsh-completion-script

Conversation

@piiq
Copy link

@piiq piiq commented Mar 27, 2026

Summary

  • replace the zsh completion output with a native _gog zsh completer
  • keep the implementation small by calling gog __complete directly from zsh
  • update the existing completion test to expect the native zsh marker

Why

The current gog completion zsh output is a bash compatibility shim using bashcompinit, which is awkward in zsh and does not install as a normal native zsh completion script.

Validation

  • go test ./internal/cmd
  • go test ./...
  • go run ./cmd/gog completion zsh | sed -n '1,40p'

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