Skip to content

Close instance ports from the CLI - #446

Draft
callen-bot wants to merge 1 commit into
agent/open-instance-portfrom
agent/close-instance-ports
Draft

Close instance ports from the CLI#446
callen-bot wants to merge 1 commit into
agent/open-instance-portfrom
agent/close-instance-ports

Conversation

@callen-bot

@callen-bot callen-bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add brev ports close <instance-or-node> with a remove alias
  • present an interactive picker for human use and support exact --id <port_id> --approve targeting for automation
  • add --all to close every current mapping, with confirmation unless --approve is supplied
  • support managed instances and registered nodes, including partial-failure reporting
  • direct automation to brev ports ls <instance-or-node> --json for stable IDs

Why

PR #441 adds the read layer and PR #442 adds the create layer. This stacked follow-up completes the delete layer for port mappings.

Impact

Users can remove one selected mapping, deterministically remove one mapping by its unique API ID, or remove all mappings in a single command.

Stack

Validation

  • go test -race ./pkg/cmd/ports ./pkg/cmd
  • go build -o /tmp/brev-port-restack-close-brev .
  • help smoke tests for ports and ports close

@callen-bot
callen-bot force-pushed the agent/close-instance-ports branch from 09a02d3 to 61da52e Compare August 14, 2026 20:20
@callen-bot
callen-bot force-pushed the agent/close-instance-ports branch from 61da52e to 67ff4e9 Compare August 19, 2026 16:54
@callen-bot
callen-bot force-pushed the agent/close-instance-ports branch from 67ff4e9 to 020e241 Compare August 20, 2026 00:06
@callen-bot
callen-bot force-pushed the agent/close-instance-ports branch 3 times, most recently from aa61461 to 220d84d Compare August 20, 2026 20:13
@callen-bot
callen-bot force-pushed the agent/close-instance-ports branch from 220d84d to d68471e Compare August 20, 2026 20:43
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