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

src/commands/api.ts

@@ -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/auth/login.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/logout.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/refresh.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/status.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/token.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/whoami.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/cli/feedback.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/fix.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;
   },
 });

src/commands/cli/setup.ts

@@ -105,7 +105,7 @@ async function handlePathModification(
   shell: ShellInfo,
   env: NodeJS.ProcessEnv,
   emit: Logger
-): Promise<void> {
+) {
   const alreadyInPath = isInPath(binaryDir, env.PATH);
 
   if (alreadyInPath) {
@@ -235,7 +235,7 @@ async function handleCompletions(
  * Only produces output when the skill file is freshly created. Subsequent
  * runs (e.g. after upgrade) silently update without printing.
  */
-async function handleAgentSkills(homeDir: string, emit: Logger): Promise<void> {
+async function handleAgentSkills(homeDir: string, emit: Logger) {
   const location = await installAgentSkills(homeDir, CLI_VERSION);
 
   if (location?.created) {
@@ -276,7 +276,7 @@ async function bestEffort(
   stepName: string,
   fn: () => void | Promise<void>,
   warn: WarnLogger
-): Promise<void> {
+) {
   try {
     await fn();
   } catch (error) {
@@ -301,7 +301,7 @@ type ConfigStepOptions = {
  * Each step is independently guarded so a failure in one (e.g. DB permission
  * error) doesn't prevent the others from running.
  */
-async function runConfigurationSteps(opts: ConfigStepOptions): Promise<void> {
+async function runConfigurationSteps(opts: ConfigStepOptions) {
   const { flags, binaryPath, binaryDir, homeDir, env, emit, warn } = opts;
   const shell = detectShell(env.SHELL, homeDir, env.XDG_CONFIG_HOME);
 
@@ -441,7 +441,7 @@ export const setupCommand = buildCommand({
       },
     },
   },
-  async func(this: SentryContext, flags: SetupFlags): Promise<void> {
+  async *func(this: SentryContext, flags: SetupFlags) {
     const { process, homeDir } = this;
 
     const emit: Logger = (msg: string) => {