Skip to content

feat(cli): add openshell gateway list subcommand - #1179

Merged
maxdubrinsky merged 1 commit into
mainfrom
md/1173-gateway-list
May 5, 2026
Merged

feat(cli): add openshell gateway list subcommand#1179
maxdubrinsky merged 1 commit into
mainfrom
md/1173-gateway-list

Conversation

@maxdubrinsky

Copy link
Copy Markdown
Collaborator

Summary

Wires up openshell gateway list as a first-class subcommand. Resolves
the dead reference at crates/openshell-bootstrap/src/errors.rs:248,
where the port-conflict recovery hint already pointed users at this
command.

Related Issue

Closes #1173

Changes

  • crates/openshell-cli/src/main.rs: add GatewayCommands::List variant
    and dispatch to the existing run::gateway_list(&cli.gateway).
  • docs/sandboxes/manage-gateways.mdx: replace the gateway select
    workaround with gateway list in the "List all registered gateways"
    example.

Testing

Manual: openshell gateway list renders the same table as gateway select
(no arg), and openshell gateway --help shows the new subcommand.

  • mise run pre-commit passes
  • Unit tests added/updated
  • E2E tests added/updated (if applicable)

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (if applicable)

Wires up a first-class `gateway list` command that calls the existing
`run::gateway_list()` helper. Resolves the dead reference in
`crates/openshell-bootstrap/src/errors.rs:248`, where the port-conflict
recovery hint already pointed users at this command.

Updates the manage-gateways doc to teach the new verb instead of the
`gateway select`-with-no-arg fallback.

Closes #1173

Signed-off-by: Max Dubrinsky <mdubrinsky@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented May 5, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@github-actions

github-actions Bot commented May 5, 2026

Copy link
Copy Markdown

All contributors have signed the DCO ✍️ ✅
Posted by the DCO Assistant Lite bot.

@github-actions

github-actions Bot commented May 5, 2026

Copy link
Copy Markdown

@maxdubrinsky

Copy link
Copy Markdown
Collaborator Author

I have read the DCO document and I hereby sign the DCO.

@maxdubrinsky
maxdubrinsky merged commit c0cc196 into main May 5, 2026
6 of 7 checks passed
@maxdubrinsky
maxdubrinsky deleted the md/1173-gateway-list branch May 5, 2026 21:28
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.

feat(cli): add openshell gateway list subcommand

2 participants