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

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/util ./pkg/cmd
  • make fmtcheck
  • make vet
  • make lint
  • make fast-build
  • bash -n test-ports-e2e.sh

@callen-bot
callen-bot force-pushed the agent/close-instance-ports branch from 09a02d3 to 61da52e Compare August 14, 2026 20:20
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