Skip to content

BUILD-646: Add band build sip-trunk command - #59

Merged
noahg1 merged 2 commits into
mainfrom
BUILD-646
Nov 9, 2026
Merged

noahg1 merged 2 commits into
mainfrom
BUILD-646

Conversation

@noahg1

@noahg1 noahg1 commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Adds band sip trunk create --transport udp|tls, wrapping the new Build endpoint POST /v1/build/accounts/{accountId}/sip-trunks (spec: Bandwidth/api-specs#2696).
  • New cmdutil.BuildAPIClient (/v1/build, honors --environment / BW_API_URL) with a test seam.
  • --transport is required; missing/invalid exits 6 before any request. 409 exits 4 (already exists, in progress, or provisioning not finished).
  • Provisioning is async (202 PROVISIONING) and no status endpoint exists yet, so no --wait; no --if-not-exists since the 409 is ambiguous (documented in AGENTS.md).
  • README and AGENTS.md updated.

Test plan

  • go build ./... && go test ./...
  • Golden plain output, transport normalization, flag-error exit 6, conflict exit 4
  • Smoke test against stage once the Build change is deployed

Wraps POST /v1/build/accounts/{accountId}/sip-trunks (api-specs PR #2696) with a required --transport udp|tls flag. 409 maps to exit 4; provisioning is async so there is no --wait.
@noahg1
noahg1 requested review from a team as code owners November 9, 2026 15:00
@bwappsec

bwappsec commented Nov 9, 2026 •

Copy link
Copy Markdown

✅ Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues
✅ Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Drops the new build command group in favor of the existing sip group, matching how Build registration lives under account.
@noahg1
noahg1 merged commit 96bf54f into main Nov 9, 2026
8 checks passed
@noahg1
noahg1 deleted the BUILD-646 branch November 9, 2026 15:39
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.

4 participants