fix(cli): trim the id stored by contexts add

cubic: the empty-check trimmed the id but the raw value was saved, so a
whitespace-padded id would be stored and later resolved as a bogus context id.
Persist the normalized (trimmed) id and echo it back.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: shrey150

packages/cli/src/commands/cloud/contexts/add.ts

@@ -38,7 +38,10 @@ export default class ContextsAdd extends BrowseCommand {
     if (!isValidContextName(args.name)) {
       fail(`Invalid context name "${args.name}". ${contextNameRequirement()}`);
     }
-    if (args.id.trim().length === 0) {
+    // Normalize before storing so whitespace-padded input isn't saved and later
+    // resolved as a bogus context id.
+    const id = args.id.trim();
+    if (id.length === 0) {
       fail("Context ID cannot be empty.");
     }
     if (!flags.force && (await getContextAlias(args.name))) {
@@ -49,9 +52,9 @@ export default class ContextsAdd extends BrowseCommand {
     }
 
     await saveContextAlias(args.name, {
-      id: args.id,
+      id,
       createdAt: new Date().toISOString(),
     });
-    outputJson({ name: args.name, id: args.id });
+    outputJson({ name: args.name, id });
   }
 }

packages/cli/tests/contexts-named.test.ts

@@ -213,11 +213,16 @@ describe("named contexts (end to end through the CLI)", () => {
       expect(dup.exitCode).not.toBe(0);
       expect(dup.stderr).toContain("already exists locally");
 
+      // Padded id + --force: repoints and stores the trimmed id.
       const forced = await runCli(
-        ["cloud", "contexts", "add", "team-login", newId, "--force"],
+        ["cloud", "contexts", "add", "team-login", `  ${newId}  `, "--force"],
         { env },
       );
       expect(forced.exitCode).toBe(0);
+      expect(JSON.parse(forced.stdout)).toEqual({
+        name: "team-login",
+        id: newId,
+      });
       expect(
         JSON.parse(await readFile(storePath, "utf8")).contexts,
       ).toMatchObject({ "team-login": { id: newId } });