fix(onboard): reject sandbox names that collide with global CLI commands

latenighthackathon · latenighthackathon · commit f078f7e199f1 · 2026-04-10T20:29:49.000-05:00
A sandbox named 'status', 'list', or 'debug' becomes inaccessible because the CLI router matches global commands before sandbox names. Add a reserved-name check in promptValidatedSandboxName that rejects names matching any global command and prompts the user to choose a different name. Closes #1731 Signed-off-by: latenighthackathon <latenighthackathon@users.noreply.github.com>
diff --git a/src/lib/onboard.ts b/src/lib/onboard.ts
@@ -2233,7 +2233,20 @@ async function promptValidatedSandboxName() {
     // Validate: RFC 1123 subdomain — lowercase alphanumeric and hyphens,
     // must start with a letter (not a digit) to satisfy Kubernetes naming.
     if (/^[a-z]([a-z0-9-]*[a-z0-9])?$/.test(sandboxName)) {
-      return sandboxName;
+      // Reject names that collide with global CLI commands.
+      // A sandbox named 'status' makes 'nemoclaw status connect' route to
+      // the global status command instead of the sandbox.
+      const RESERVED_NAMES = new Set([
+        "onboard", "list", "deploy", "setup", "setup-spark",
+        "start", "stop", "status", "debug", "uninstall",
+        "credentials", "help",
+      ]);
+      if (RESERVED_NAMES.has(sandboxName)) {
+        console.error(`  Reserved name: '${sandboxName}' is a NemoClaw CLI command.`);
+        console.error("  Choose a different name to avoid routing conflicts.");
+      } else {
+        return sandboxName;
+      }
     }
 
     console.error(`  Invalid sandbox name: '${sandboxName}'`);
