Skip to content

fix(cli): include plugins info in shell completions - #794

Merged
kevincodex1 merged 1 commit into
Gitlawb:mainfrom
Ayush7614:feat/completions-plugins-info
Jul 24, 2026
Merged

fix(cli): include plugins info in shell completions#794
kevincodex1 merged 1 commit into
Gitlawb:mainfrom
Ayush7614:feat/completions-plugins-info

Conversation

@Ayush7614

@Ayush7614 Ayush7614 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add info to the plugins / plugin shell-completion tree
  • Cover it in completion-tree tests

zero plugins info landed in #773, but zero completions still omitted it.

Closes #791

Test plan

  • go test ./internal/cli/ -run 'Completions|CompletionTree' -count=1
  • go vet ./internal/cli/

Summary by CodeRabbit

  • Bug Fixes
    • Added shell completion support for the plugins info command.
    • Updated completions for both plugin and plugins command paths to include all available subcommands and aliases.

zero plugins info landed in Gitlawb#773 but the completions tree still only
offered list/add/remove, so generated bash/zsh/fish scripts omitted it.
@coderabbitai

coderabbitai Bot commented Jul 23, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 845d3a2d-95e5-4803-8c58-c6c7f7c17232

📥 Commits

Reviewing files that changed from the base of the PR and between d9b882e and 3cdf800.

📒 Files selected for processing (2)
  • internal/cli/completions.go
  • internal/cli/completions_test.go

Walkthrough

The plugin shell completion tree now exposes info alongside existing plugin subcommands, with coverage for both plugins and plugin command paths.

Changes

Plugin shell completions

Layer / File(s) Summary
Add info completion and assertions
internal/cli/completions.go, internal/cli/completions_test.go
The plugins completion node includes info, and tests verify expected candidates for both plugins and plugin paths.

Estimated code review effort: 2 (Simple) | ~5 minutes

Possibly related PRs

  • Gitlawb/zero#764: Established the completion command-tree generator and coverage assertions extended by this change.
  • Gitlawb/zero#773: Added the zero plugins info CLI command represented in shell completions.

Suggested reviewers: anandh8x, kevincodex1

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly states the main change: adding plugins info to shell completions.
Linked Issues check ✅ Passed The change adds info to the plugins completion tree and covers it in tests, matching #791.
Out of Scope Changes check ✅ Passed No unrelated code paths or extra features appear in the reported changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@kevincodex1 kevincodex1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@kevincodex1
kevincodex1 merged commit fa8734f into Gitlawb:main Jul 24, 2026
7 checks passed
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.

fix(cli): include plugins info in shell completions

2 participants