fix(auth): guard setUserInfo in whoami against DB write failures

Cache update is a non-essential side effect — a read-only filesystem or
corrupted DB must not prevent the command from displaying the user identity
that was already successfully fetched from the API.

Author: BYK

src/commands/auth/whoami.ts

@@ -44,13 +44,18 @@ export const whoamiCommand = buildCommand({
 
     const user = await getCurrentUser();
 
-    // Keep cached user info up to date
-    setUserInfo({
-      userId: user.id,
-      email: user.email,
-      username: user.username,
-      name: user.name,
-    });
+    // Keep cached user info up to date. Non-fatal: display must succeed even
+    // if the DB write fails (read-only filesystem, corrupted database, etc.).
+    try {
+      setUserInfo({
+        userId: user.id,
+        email: user.email,
+        username: user.username,
+        name: user.name,
+      });
+    } catch {
+      // Cache update failure is non-essential — user identity was already fetched.
+    }
 
     if (flags.json) {
       writeJson(stdout, {

test/commands/auth/whoami.test.ts

@@ -165,6 +165,20 @@ describe("whoamiCommand.func", () => {
         email: "jane@example.com",
       });
     });
+
+    test("still displays identity when DB cache write fails", async () => {
+      isAuthenticatedSpy.mockResolvedValue(true);
+      getCurrentUserSpy.mockResolvedValue(FULL_USER);
+      setUserInfoSpy.mockImplementation(() => {
+        throw new Error("read-only filesystem");
+      });
+
+      const { context, getOutput } = createContext();
+      // Must not throw — output must still be shown
+      await func.call(context, { json: false });
+
+      expect(getOutput()).toContain("Jane Doe");
+    });
   });
 
   describe("--json output", () => {