mirror of
https://github.com/coder/coder.git
synced 2026-09-24 15:04:27 +08:00
chore: rename boundary command to agent-firewall (#25889)
Renames the `coder boundary` CLI subcommand to `coder agent-firewall` as part of the Boundaries → Agent Firewall rebrand. `coder boundary` is retained as a hidden, deprecated alias that prints a deprecation notice to stderr before running. Both commands use separate builder functions backed by the same boundary base command and license verification logic. Closes https://linear.app/codercom/issue/AIGOV-236 <details><summary>Implementation notes</summary> **Approach:** Two separate `*serpent.Command` objects (not `Aliases`) so the deprecated `boundary` path can print a stderr warning while `agent-firewall` stays clean. **Changes:** - `enterprise/cli/boundary.go`: Split old `boundary()` into `buildAgentFirewallCmd()` and `buildBoundaryAliasCmd()`. Error messages in `verifyLicense` now reference "agent-firewall". - `enterprise/cli/root.go`: Register both commands. - `cli/root.go`: Update YAML-only option validation bypass for the new command name. - Tests: Rename to `TestAgentFirewallSubcommand`, add `TestBoundaryAlias`, update license verification tests to use `agent-firewall`. - Golden files and CLI reference docs regenerated. - `docs/ai-coder/agent-firewall/version.md` and `docs/manifest.json` updated. </details> > Generated with [Coder Agents](https://coder.com/agents) by @SasSwart
This commit is contained in:
+5
-4
@@ -343,10 +343,11 @@ func (r *RootCmd) Command(subcommands []*serpent.Command) (*serpent.Command, err
|
||||
// support links.
|
||||
return
|
||||
}
|
||||
if cmd.Name() == "boundary" {
|
||||
// The boundary command is integrated from the boundary package
|
||||
// and has YAML-only options (e.g., allowlist from config file)
|
||||
// that don't have flags or env vars.
|
||||
if cmd.Name() == "agent-firewall" || cmd.Name() == "boundary" {
|
||||
// The agent-firewall command (and its "boundary" alias) is
|
||||
// integrated from the boundary package and has YAML-only
|
||||
// options (e.g., allowlist from config file) that don't
|
||||
// have flags or env vars.
|
||||
return
|
||||
}
|
||||
merr = errors.Join(
|
||||
|
||||
Reference in New Issue
Block a user