[SAN-331] rename render ea sandbox to render ea sandboxes (#530)

* cmd: add routing tests for sandboxes rename

* cmd: rename sandbox parent command to sandboxes

* cmd: error on unknown ea subcommands

Removing the sandbox command does not by itself make `render ea sandbox` fail. Cobra reports unknown commands only for the root; a nested parent like ea otherwise falls through to printing its help with a zero exit. Add cobra.NoArgs plus a help RunE so ea rejects any unknown subcommand, including the removed singular sandbox form, with a non-zero exit.

The routing test drops --help because the help flag short-circuits arg validation.

* cmd: update sandbox examples to plural form

GitOrigin-RevId: c3b3439f3505494b8a23769d4afbc9aaaf90c954

Author: render-tej

cmd/earlyaccess.go

@@ -13,6 +13,13 @@ func newEarlyAccessCmd() *cobra.Command {
 		Long:  `These commands are in early access and are subject to change.`,
 		Example: `  # List early access object storage resources
   render ea objects list --region=oregon`,
+		// Reject unknown subcommands with a non-zero exit. Cobra flags unknown
+		// commands only for the root, so a nested parent needs NoArgs, plus a
+		// RunE so arg validation runs (it also shows help for a bare "render ea").
+		Args: cobra.NoArgs,
+		RunE: func(cmd *cobra.Command, _ []string) error {
+			return cmd.Help()
+		},
 	}
 }
 

cmd/sandbox.go

@@ -6,7 +6,7 @@ import (
 
 func newSandboxCmd(children ...*cobra.Command) *cobra.Command {
 	cmd := &cobra.Command{
-		Use:   "sandbox",
+		Use:   "sandboxes",
 		Short: "Manage sandboxes",
 		Long: `Manage sandboxes for your Render workspace.
 
@@ -19,11 +19,11 @@ Available commands:
   stop     - Terminate a running sandbox
 
 Examples:
-  render ea sandbox create --base=render/sandbox-python
-  render ea sandbox exec sbx-abc123 -- echo hello
-  render ea sandbox list
-  render ea sandbox list --all
-  render ea sandbox stop trn-abc123 --confirm
+  render ea sandboxes create --base=render/sandbox-python
+  render ea sandboxes exec sbx-abc123 -- echo hello
+  render ea sandboxes list
+  render ea sandboxes list --all
+  render ea sandboxes stop trn-abc123 --confirm
 `,
 	}
 	cmd.AddCommand(children...)

cmd/sandbox_test.go

@@ -0,0 +1,80 @@
+package cmd
+
+import (
+	"bytes"
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+
+	renderapi "github.com/render-oss/cli/internal/fakes/renderapi"
+	"github.com/render-oss/cli/pkg/client"
+	"github.com/render-oss/cli/pkg/command"
+	"github.com/render-oss/cli/pkg/dependencies"
+)
+
+// executeSandboxCommand runs the CLI with the sandbox tree registered under
+// ea, mirroring executeSandboxGroupsCommand in sandboxgroups_test.go.
+func executeSandboxCommand(t *testing.T, server *renderapi.Server, args ...string) (CommandResult, error) {
+	t.Helper()
+	t.Setenv("RENDER_CLI_CONFIG_PATH", newTestConfigPath(t))
+	t.Setenv("RENDER_API_KEY", "test-api-key")
+
+	c, err := client.NewClientWithResponses(server.URL())
+	require.NoError(t, err)
+	deps := dependencies.New(c)
+	deps.DetectRuntimeSignals = func() (command.RuntimeSignals, error) {
+		return command.RuntimeSignals{StdinTTY: false, StdoutTTY: false, StderrTTY: false}, nil
+	}
+
+	root := newRootCmd()
+	ea := newEarlyAccessCmd()
+	root.AddCommand(ea)
+	setupSandboxCommands(ea, deps)
+	setupRootCmdPersistentRun(root, deps)
+
+	var stdout, stderr bytes.Buffer
+	root.SetOut(&stdout)
+	root.SetErr(&stderr)
+	root.SetArgs(args)
+
+	execErr := root.Execute()
+	return CommandResult{Stdout: stdout.String(), Stderr: stderr.String()}, execErr
+}
+
+// The parent command is plural per the PRD CLI spec and STYLE.md's
+// plural-resource rule.
+func TestSandboxes_PluralCommandResolves(t *testing.T) {
+	server := renderapi.NewServer(t)
+
+	result, err := executeSandboxCommand(t, server, "ea", "sandboxes", "--help")
+	require.NoError(t, err)
+	assert.Contains(t, result.Stdout, "Manage sandboxes")
+	assert.Contains(t, result.Stdout, "render ea sandboxes")
+}
+
+// The singular form is removed outright (no alias), per the #sandboxes-core
+// decision that pre-alpha backwards compatibility is not needed. This test
+// locks that in: if someone reintroduces "sandbox" as a name or alias, this
+// fails and forces a deliberate decision.
+//
+// No --help here on purpose: the help flag short-circuits before Cobra
+// validates args, so "ea sandbox --help" would print ea's help with no error.
+// The unknown-command error comes from ea's cobra.NoArgs validator.
+func TestSandboxes_SingularFormRemoved(t *testing.T) {
+	server := renderapi.NewServer(t)
+
+	_, err := executeSandboxCommand(t, server, "ea", "sandbox")
+	require.Error(t, err)
+	assert.Contains(t, err.Error(), "unknown command")
+}
+
+func TestSandboxes_HelpListsSubcommands(t *testing.T) {
+	server := renderapi.NewServer(t)
+
+	result, err := executeSandboxCommand(t, server, "ea", "sandboxes", "--help")
+	require.NoError(t, err)
+	for _, sub := range []string{"create", "exec", "list", "stop"} {
+		assert.Contains(t, result.Stdout, sub, "expected subcommand %q in help output", sub)
+	}
+}

cmd/sandboxcreate.go

@@ -48,9 +48,9 @@ func newSandboxCreateCmd(deps *dependencies.Dependencies) *cobra.Command {
 		Long: `Create a new sandbox in the current workspace.
 
 Examples:
-  render ea sandbox create
-  render ea sandbox create --plan=standard --region=oregon
-  render ea sandbox create --timeout=3600
+  render ea sandboxes create
+  render ea sandboxes create --plan=standard --region=oregon
+  render ea sandboxes create --timeout=3600
 `,
 	}
 

cmd/sandboxexec.go

@@ -43,8 +43,8 @@ Pass the command after a "--" separator so its own flags aren't parsed by the
 CLI.
 
 Examples:
-  render ea sandbox exec sbx-abc123 -- echo hello
-  render ea sandbox exec sbx-abc123 -- python script.py
+  render ea sandboxes exec sbx-abc123 -- echo hello
+  render ea sandboxes exec sbx-abc123 -- python script.py
 `,
 		Args: cobra.MinimumNArgs(2),
 	}

cmd/sandboxlist.go

@@ -23,11 +23,11 @@ func newSandboxListCmd(deps *dependencies.Dependencies) *cobra.Command {
 By default, terminated sandboxes are excluded. Use --all to include them, or --status to filter by specific statuses.
 
 Examples:
-  render ea sandbox list
-  render ea sandbox list --all
-  render ea sandbox list --status=running
-  render ea sandbox list --status=running --status=creating
-  render ea sandbox list -o json
+  render ea sandboxes list
+  render ea sandboxes list --all
+  render ea sandboxes list --status=running
+  render ea sandboxes list --status=running --status=creating
+  render ea sandboxes list -o json
 `,
 	}
 

cmd/sandboxstop.go

@@ -33,13 +33,13 @@ func newSandboxStopCmd(deps *dependencies.Dependencies) *cobra.Command {
 Without --confirm, this command previews what would be terminated and makes no
 changes. Pass --confirm to actually terminate the sandbox.`,
 		Example: `  # Preview termination (no changes made)
-  render ea sandbox stop trn-abc123
+  render ea sandboxes stop trn-abc123
 
   # Terminate the sandbox
-  render ea sandbox stop trn-abc123 --confirm
+  render ea sandboxes stop trn-abc123 --confirm
 
   # JSON output
-  render ea sandbox stop trn-abc123 --confirm --output json`,
+  render ea sandboxes stop trn-abc123 --confirm --output json`,
 	}
 
 	cmd.RunE = func(cmd *cobra.Command, args []string) error {