refactor: unify all command functions as async generators

All command functions now use async generator signatures. The framework
iterates each yielded value through the existing OutputConfig rendering
pipeline. Non-streaming commands yield once and return; streaming
commands (log list --follow) yield multiple times.

Key changes:
- buildCommand: func returns AsyncGenerator, wrapper uses for-await-of
- All ~27 command files: async func → async *func, return → yield
- log/list follow mode: drainStreamingOutput replaced by yield delegation
- Delete streaming-command.ts (151 lines) — absorbed into buildCommand
- output.ts: extracted applyJsonExclude/writeTransformedJson helpers,
  support undefined suppression for streaming text-only chunks

No new dependencies. Net -131 lines.

Author: BYK

AGENTS.md

@@ -1155,7 +1155,7 @@ export const apiCommand = buildCommand({
       n: "dry-run",
     },
   },
-  async func(this: SentryContext, flags: ApiFlags, endpoint: string) {
+  async *func(this: SentryContext, flags: ApiFlags, endpoint: string) {
     const { stdin } = this;
 
     const normalizedEndpoint = normalizeEndpoint(endpoint);
@@ -1168,14 +1168,15 @@ export const apiCommand = buildCommand({
 
     // Dry-run mode: preview the request that would be sent
     if (flags["dry-run"]) {
-      return {
+      yield {
         data: {
           method: flags.method,
           url: resolveRequestUrl(normalizedEndpoint, params),
           headers: resolveEffectiveHeaders(headers, body),
           body: body ?? null,
         },
       };
+      return;
     }
 
     const verbose = flags.verbose && !flags.silent;
@@ -1210,6 +1211,7 @@ export const apiCommand = buildCommand({
       throw new OutputError(response.body);
     }
 
-    return { data: response.body };
+    yield { data: response.body };
+    return;
   },
 });

src/commands/api.ts

@@ -48,7 +48,8 @@ export const loginCommand = buildCommand({
       },
     },
   },
-  async func(this: SentryContext, flags: LoginFlags): Promise<void> {
+  // biome-ignore lint/correctness/useYield: void generator — writes to stdout directly, will be migrated to yield pattern later
+  async *func(this: SentryContext, flags: LoginFlags) {
     // Check if already authenticated
     if (await isAuthenticated()) {
       if (isEnvTokenActive()) {

src/commands/auth/login.ts

@@ -36,11 +36,12 @@ export const logoutCommand = buildCommand({
   parameters: {
     flags: {},
   },
-  async func(this: SentryContext): Promise<{ data: LogoutResult }> {
+  async *func(this: SentryContext) {
     if (!(await isAuthenticated())) {
-      return {
+      yield {
         data: { loggedOut: false, message: "Not currently authenticated." },
       };
+      return;
     }
 
     if (isEnvTokenActive()) {
@@ -55,11 +56,12 @@ export const logoutCommand = buildCommand({
     const configPath = getDbPath();
     await clearAuth();
 
-    return {
+    yield {
       data: {
         loggedOut: true,
         configPath,
       },
     };
+    return;
   },
 });

src/commands/auth/logout.ts

@@ -68,7 +68,7 @@ Examples:
       },
     },
   },
-  async func(this: SentryContext, flags: RefreshFlags) {
+  async *func(this: SentryContext, flags: RefreshFlags) {
     // Env var tokens can't be refreshed
     if (isEnvTokenActive()) {
       const envVar = getActiveEnvVarName();
@@ -104,6 +104,7 @@ Examples:
         : undefined,
     };
 
-    return { data: payload };
+    yield { data: payload };
+    return;
   },
 });

src/commands/auth/refresh.ts

@@ -155,7 +155,7 @@ export const statusCommand = buildCommand({
     },
     aliases: FRESH_ALIASES,
   },
-  async func(this: SentryContext, flags: StatusFlags) {
+  async *func(this: SentryContext, flags: StatusFlags) {
     applyFreshFlag(flags);
 
     const auth = getAuthConfig();
@@ -189,6 +189,7 @@ export const statusCommand = buildCommand({
       verification: await verifyCredentials(),
     };
 
-    return { data };
+    yield { data };
+    return;
   },
 });

src/commands/auth/status.ts

@@ -20,7 +20,9 @@ export const tokenCommand = buildCommand({
       "when stdout is not a TTY (e.g., when piped).",
   },
   parameters: {},
-  func(this: SentryContext): void {
+  // biome-ignore lint/correctness/useYield: void generator — writes to stdout directly
+  // biome-ignore lint/suspicious/useAwait: sync body but async generator required by buildCommand
+  async *func(this: SentryContext) {
     const { stdout } = this;
 
     const token = getAuthToken();

src/commands/auth/token.ts

@@ -43,7 +43,7 @@ export const whoamiCommand = buildCommand({
     },
     aliases: FRESH_ALIASES,
   },
-  async func(this: SentryContext, flags: WhoamiFlags) {
+  async *func(this: SentryContext, flags: WhoamiFlags) {
     applyFreshFlag(flags);
 
     if (!(await isAuthenticated())) {
@@ -65,6 +65,7 @@ export const whoamiCommand = buildCommand({
       // Cache update failure is non-essential — user identity was already fetched.
     }
 
-    return { data: user };
+    yield { data: user };
+    return;
   },
 });

src/commands/auth/whoami.ts

@@ -42,12 +42,12 @@ export const feedbackCommand = buildCommand({
       },
     },
   },
-  async func(
+  async *func(
     this: SentryContext,
     // biome-ignore lint/complexity/noBannedTypes: Stricli requires empty object for commands with no flags
     _flags: {},
     ...messageParts: string[]
-  ): Promise<{ data: FeedbackResult }> {
+  ) {
     const message = messageParts.join(" ");
 
     if (!message.trim()) {
@@ -66,11 +66,12 @@ export const feedbackCommand = buildCommand({
     // Flush to ensure feedback is sent before process exits
     const sent = await Sentry.flush(3000);
 
-    return {
+    yield {
       data: {
         sent,
         message,
       },
     };
+    return;
   },
 });

src/commands/cli/feedback.ts

@@ -678,7 +678,7 @@ export const fixCommand = buildCommand({
       },
     },
   },
-  async func(this: SentryContext, flags: FixFlags) {
+  async *func(this: SentryContext, flags: FixFlags) {
     const dbPath = getDbPath();
     const dryRun = flags["dry-run"];
 
@@ -734,6 +734,7 @@ export const fixCommand = buildCommand({
       throw new OutputError(result);
     }
 
-    return { data: result };
+    yield { data: result };
+    return;
   },
 });