fix(auth/logout): return data for not-authenticated instead of throwing

A no-op logout when not authenticated is more user-friendly than an
error. Matches the previous behavior where exit code was 0.

- Not-authenticated: returns { loggedOut: false, message } (exit 0)
- Env-token-active: still throws AuthError (can't proceed)
- Formatter handles both loggedOut:true and loggedOut:false cases

Author: BYK

src/commands/auth/logout.ts

@@ -20,6 +20,8 @@ import { formatLogoutResult } from "../../lib/formatters/human.js";
 export type LogoutResult = {
   /** Whether logout actually cleared credentials */
   loggedOut: boolean;
+  /** Informational message when no action was taken */
+  message?: string;
   /** Path where credentials were stored (when loggedOut is true) */
   configPath?: string;
 };
@@ -36,9 +38,9 @@ export const logoutCommand = buildCommand({
   },
   async func(this: SentryContext): Promise<{ data: LogoutResult }> {
     if (!(await isAuthenticated())) {
-      throw new AuthError("not_authenticated", undefined, {
-        skipAutoAuth: true,
-      });
+      return {
+        data: { loggedOut: false, message: "Not currently authenticated." },
+      };
     }
 
     if (isEnvTokenActive()) {

src/lib/formatters/human.ts

@@ -1643,6 +1643,9 @@ type LogoutResult = import("../../commands/auth/logout.js").LogoutResult;
  * @returns Rendered terminal string
  */
 export function formatLogoutResult(data: LogoutResult): string {
+  if (!data.loggedOut) {
+    return renderMarkdown(data.message ?? "Not currently authenticated.");
+  }
   const lines: string[] = [];
   lines.push(`${colorTag("green", "✓")} Logged out successfully.`);
   if (data.configPath) {

test/commands/auth/logout.test.ts

@@ -81,25 +81,14 @@ describe("logoutCommand.func", () => {
     getDbPathSpy.mockRestore();
   });
 
-  test("not authenticated: throws AuthError", async () => {
+  test("not authenticated: returns loggedOut false with message", async () => {
     isAuthenticatedSpy.mockResolvedValue(false);
-    const { context } = createContext();
-
-    await expect(func.call(context, {})).rejects.toThrow(AuthError);
-    expect(clearAuthSpy).not.toHaveBeenCalled();
-  });
+    const { context, getOutput } = createContext();
 
-  test("not authenticated: error has skipAutoAuth", async () => {
-    isAuthenticatedSpy.mockResolvedValue(false);
-    const { context } = createContext();
+    await func.call(context, {});
 
-    try {
-      await func.call(context, {});
-      expect.unreachable("should have thrown");
-    } catch (err) {
-      expect(err).toBeInstanceOf(AuthError);
-      expect((err as AuthError).skipAutoAuth).toBe(true);
-    }
+    expect(clearAuthSpy).not.toHaveBeenCalled();
+    expect(getOutput()).toContain("Not currently authenticated");
   });
 
   test("OAuth token: clears auth and writes success to stdout", async () => {