Skip to content

ux: hidden cmds tabcomplete - #701

Open
avirtopeanu-ionos wants to merge 1 commit into
masterfrom
ux/hidden-cmds-tabcomplete
Open

ux: hidden cmds tabcomplete#701
avirtopeanu-ionos wants to merge 1 commit into
masterfrom
ux/hidden-cmds-tabcomplete

Conversation

@avirtopeanu-ionos

Copy link
Copy Markdown
Contributor

Allow tab completions for the hidden compute commands.

@avirtopeanu-ionos
avirtopeanu-ionos requested review from a team, adeatcu-ionos, cavramoniu-ionos, cristiGuranIonos, mimihalescu and rmocanu-ionos and a lite review from Copilot and removed request for a team August 14, 2026 10:01
@avirtopeanu-ionos
avirtopeanu-ionos changed the base branch from master to feat/inmemorydbv2 August 14, 2026 10:01
@avirtopeanu-ionos
avirtopeanu-ionos removed the request for review from Copilot August 14, 2026 10:01
Base automatically changed from feat/inmemorydbv2 to master August 14, 2026 10:03
Copilot AI lite review requested due to automatic review settings August 14, 2026 12:24
@avirtopeanu-ionos
avirtopeanu-ionos force-pushed the ux/hidden-cmds-tabcomplete branch from 63bf100 to c0704cf Compare August 14, 2026 12:24
@sonarqubecloud

Copy link
Copy Markdown

Copilot AI 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.

Pull request overview

This PR improves CLI usability by enabling tab-completion suggestions for the hidden root-level compute alias commands (kept for backward compatibility), without making them visible in help/docs.

Changes:

  • Collects hidden compute alias command names (and their short descriptions) during root command setup.
  • Adds a rootCmd.Command.ValidArgsFunction to return completion candidates for those hidden aliases when completing the first token.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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.

2 participants