Feat/polish completions doctor ci#12
Merged
Merged
Conversation
New:
- cli/internal/cmd/doctor.go: --doctor health check (chafa, fastfetch,
pokemon-colorscripts, terminal graphics protocol, config, manifest,
assets on disk, fastfetch config, rc file integration)
- cli/internal/cmd/completions.go: tab completion for --car, --brand,
--decade, --category, --powertrain, --country, --set-mode. Loads
manifest lazily via sync.Once. Includes both slugs and alias keys
as --car completions.
- cli/internal/cmd/list.go: grouped output with counts + year ranges
for --list-brands ('Ferrari 91 cars 1947-2024'), --list-decades
(with vintage/modern indicator), --list-categories (sorted by count)
- .github/workflows/ci.yml: go build+vet, python py_compile,
shellcheck on all install scripts
Fixed:
- uninstall.sh loop-start grep now also matches
'hypercar-installer-disabled' so files where the block was stripped
manually still get processed for pokemon restoration.
- uninstall.sh restore regex now handles leading whitespace (was
breaking when install.sh preserved indentation via \\1).
Changed:
- install.sh drops completion files into XDG dirs for bash/zsh/fish
after binary install. Non-fatal — installs succeed even if a shell
is missing.
- uninstall.sh removes the completion files."
IMPORTANT:
- The shell completion scripts still have issues and will be fixed in a follow-up commit.
- Added the pokemon mixing option while installing. - Removed the shell completion logic from the install script and the completions.go file.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
New:
pokemon-colorscripts, terminal graphics protocol, config, manifest,
assets on disk, fastfetch config, rc file integration)
for --list-brands ('Ferrari 91 cars 1947-2024'), --list-decades
(with vintage/modern indicator), --list-categories (sorted by count)
shellcheck on all install scripts
Fixed:
'hypercar-installer-disabled' so files where the block was stripped
manually still get processed for pokemon restoration.
breaking when install.sh preserved indentation via \1).
Changed:
is missing.