refactor: remove stdout/stderr plumbing from commands

Commands no longer receive or pass stdout/stderr Writer references.
Interactive and diagnostic output now routes through the project's
consola logger (→ stderr), keeping stdout reserved for structured
command output.

Changes:
- org-list.ts: remove stdout from HandlerContext and DispatchOptions
- list-command.ts, project/list.ts, issue/list.ts, trace/logs.ts:
  stop threading stdout to dispatchOrgScopedList
- issue/list.ts: convert partial-failure stderr.write to logger.warn
- log/list.ts: convert follow-mode banner and onDiagnostic to logger
- auth/login.ts, interactive-login.ts: remove stdout/stderr params,
  use logger for all UI output (QR code, URLs, progress dots)
- clipboard.ts: remove stdout param from setupCopyKeyListener
- trial/start.ts: use logger for billing URL and QR code display
- help.ts: printCustomHelp returns string, caller writes to process.stdout
- formatters/log.ts: rename displayTraceLogs → formatTraceLogs (returns string)
- formatters/output.ts: extract formatFooter helper from writeFooter

Remaining stdout usage:
- auth/token.ts: intentional raw stdout for pipe compatibility
- help.ts: process.stdout.write for help text (like git --help)
- trace/logs.ts: process.stdout.write (pending OutputConfig migration)

Author: BYK

AGENTS.md

@@ -103,11 +103,7 @@ const autoAuthMiddleware: ErrorMiddleware = async (next, args) => {
           : "Authentication required. Starting login flow...\n\n"
       );
 
-      const loginSuccess = await runInteractiveLogin(
-        process.stdout,
-        process.stderr,
-        process.stdin
-      );
+      const loginSuccess = await runInteractiveLogin(process.stdin);
 
       if (loginSuccess) {
         process.stderr.write("\nRetrying command...\n\n");

src/bin.ts

@@ -104,7 +104,7 @@ export const loginCommand = buildCommand({
       },
     },
   },
-  // biome-ignore lint/correctness/useYield: void generator — writes to stdout directly, will be migrated to yield pattern later
+  // biome-ignore lint/correctness/useYield: void generator — all output goes to stderr via logger, will be migrated to yield pattern later
   async *func(this: SentryContext, flags: LoginFlags) {
     // Check if already authenticated and handle re-authentication
     if (await isAuthenticated()) {
@@ -168,15 +168,9 @@ export const loginCommand = buildCommand({
       // Non-fatal: cache directory may not exist
     }
 
-    const { stdout, stderr } = this;
-    const loginSuccess = await runInteractiveLogin(
-      stdout,
-      stderr,
-      process.stdin,
-      {
-        timeout: flags.timeout * 1000,
-      }
-    );
+    const loginSuccess = await runInteractiveLogin(process.stdin, {
+      timeout: flags.timeout * 1000,
+    });
 
     if (!loginSuccess) {
       // Error already displayed by runInteractiveLogin - just set exit code

src/commands/auth/login.ts

@@ -32,11 +32,9 @@ export const helpCommand = buildCommand({
   // biome-ignore lint/complexity/noBannedTypes: Stricli requires empty object for commands with no flags
   // biome-ignore lint/correctness/useYield: void generator — delegates to Stricli help system
   async *func(this: SentryContext, _flags: {}, ...commandPath: string[]) {
-    const { stdout } = this;
-
     // No args: show branded help
     if (commandPath.length === 0) {
-      await printCustomHelp(stdout);
+      process.stdout.write(await printCustomHelp());
       return;
     }
 

src/commands/help.ts

@@ -38,7 +38,6 @@ import {
 } from "../../lib/errors.js";
 import {
   type IssueTableRow,
-  muted,
   shouldAutoCompact,
   writeIssueTable,
 } from "../../lib/formatters/index.js";
@@ -57,6 +56,7 @@ import {
   parseCursorFlag,
   targetPatternExplanation,
 } from "../../lib/list-command.js";
+import { logger } from "../../lib/logger.js";
 import {
   dispatchOrgScopedList,
   jsonTransformListResult,
@@ -871,7 +871,6 @@ async function handleOrgAllIssues(
 
 /** Options for {@link handleResolvedTargets}. */
 type ResolvedTargetsOptions = {
-  stderr: Writer;
   parsed: ReturnType<typeof parseOrgProjectArg>;
   flags: ListFlags;
   cwd: string;
@@ -890,7 +889,7 @@ type ResolvedTargetsOptions = {
 async function handleResolvedTargets(
   options: ResolvedTargetsOptions
 ): Promise<IssueListResult> {
-  const { stderr, parsed, flags, cwd, setContext } = options;
+  const { parsed, flags, cwd, setContext } = options;
 
   const { targets, footer, skippedSelfHosted, detectedDsns } =
     await resolveTargetsFromParsedArg(parsed, cwd);
@@ -1094,10 +1093,8 @@ async function handleResolvedTargets(
     const failedNames = failures
       .map(({ target: t }) => `${t.org}/${t.project}`)
       .join(", ");
-    stderr.write(
-      muted(
-        `\nNote: Failed to fetch issues from ${failedNames}. Showing results from ${validResults.length} project(s).\n`
-      )
+    logger.warn(
+      `Failed to fetch issues from ${failedNames}. Showing results from ${validResults.length} project(s).`
     );
   }
 
@@ -1318,7 +1315,7 @@ export const listCommand = buildListCommand("issue", {
   },
   async *func(this: SentryContext, flags: ListFlags, target?: string) {
     applyFreshFlag(flags);
-    const { stdout, stderr, cwd, setContext } = this;
+    const { cwd, setContext } = this;
 
     const parsed = parseOrgProjectArg(target);
 
@@ -1341,13 +1338,11 @@ export const listCommand = buildListCommand("issue", {
       handleResolvedTargets({
         ...ctx,
         flags,
-        stderr,
         setContext,
       });
 
     const result = (await dispatchOrgScopedList({
       config: issueListMeta,
-      stdout,
       cwd,
       flags,
       parsed,

src/commands/issue/list.ts

@@ -33,13 +33,13 @@ import {
   FRESH_FLAG,
   TARGET_PATTERN_NOTE,
 } from "../../lib/list-command.js";
+import { logger } from "../../lib/logger.js";
 import {
   resolveOrg,
   resolveOrgProjectFromArg,
 } from "../../lib/resolve-target.js";
 import { validateTraceId } from "../../lib/trace-id.js";
 import { getUpdateNotification } from "../../lib/version-check.js";
-import type { Writer } from "../../types/index.js";
 
 type ListFlags = {
   readonly limit: number;
@@ -227,13 +227,13 @@ function abortableSleep(ms: number, signal: AbortSignal): Promise<void> {
  *
  * Unlike the old callback-based approach, this does NOT include
  * stdout/stderr. All stdout output flows through yielded chunks;
- * stderr diagnostics use the `onDiagnostic` callback.
+ * diagnostics are reported via the `onDiagnostic` callback.
  */
 type FollowGeneratorConfig<T extends LogLike> = {
   flags: ListFlags;
   /** Whether to show the trace-ID column in table output */
   includeTrace: boolean;
-  /** Report diagnostic/error messages (caller writes to stderr) */
+  /** Report diagnostic/error messages (caller logs via logger) */
   onDiagnostic: (message: string) => void;
   /**
    * Fetch logs with the given time window.
@@ -321,8 +321,8 @@ async function fetchPoll<T extends LogLike>(
  * - `data` chunks contain raw log arrays for JSONL serialization
  *
  * The generator handles SIGINT via AbortController for clean shutdown.
- * It never touches stdout/stderr directly — all output flows through
- * yielded chunks and the `onDiagnostic` callback.
+ * It never touches stdout directly — all data output flows through
+ * yielded chunks and diagnostics use the `onDiagnostic` callback.
  *
  * @throws {AuthError} if the API returns an authentication error
  */
@@ -455,25 +455,20 @@ async function executeTraceSingleFetch(
 }
 
 /**
- * Write the follow-mode banner to stderr. Suppressed in JSON mode.
+ * Write the follow-mode banner via logger. Suppressed in JSON mode.
  * Includes poll interval, Ctrl+C hint, and update notification.
  */
-function writeFollowBanner(
-  stderr: Writer,
-  flags: ListFlags,
-  bannerText: string
-): void {
+function writeFollowBanner(flags: ListFlags, bannerText: string): void {
   if (flags.json) {
     return;
   }
   const pollInterval = flags.follow ?? DEFAULT_POLL_INTERVAL;
-  stderr.write(`${bannerText} (poll interval: ${pollInterval}s)\n`);
-  stderr.write("Press Ctrl+C to stop.\n");
+  logger.info(`${bannerText} (poll interval: ${pollInterval}s)`);
+  logger.info("Press Ctrl+C to stop.");
   const notification = getUpdateNotification();
   if (notification) {
-    stderr.write(notification);
+    logger.info(notification);
   }
-  stderr.write("\n");
 }
 
 // ---------------------------------------------------------------------------
@@ -631,23 +626,18 @@ export const listCommand = buildListCommand("log", {
       setContext([org], []);
 
       if (flags.follow) {
-        const { stderr } = this;
         const traceId = flags.trace;
 
-        // Banner (stderr, suppressed in JSON mode)
-        writeFollowBanner(
-          stderr,
-          flags,
-          `Streaming logs for trace ${traceId}...`
-        );
+        // Banner (suppressed in JSON mode)
+        writeFollowBanner(flags, `Streaming logs for trace ${traceId}...`);
 
         // Track IDs of logs seen without timestamp_precise so they are
         // shown once but not duplicated on subsequent polls.
         const seenWithoutTs = new Set<string>();
         const generator = generateFollowLogs({
           flags,
           includeTrace: false,
-          onDiagnostic: (msg) => stderr.write(msg),
+          onDiagnostic: (msg) => logger.warn(msg),
           fetch: (statsPeriod) =>
             listTraceLogs(org, traceId, {
               query: flags.query,
@@ -704,14 +694,12 @@ export const listCommand = buildListCommand("log", {
       setContext([org], [project]);
 
       if (flags.follow) {
-        const { stderr } = this;
-
-        writeFollowBanner(stderr, flags, "Streaming logs...");
+        writeFollowBanner(flags, "Streaming logs...");
 
         const generator = generateFollowLogs({
           flags,
           includeTrace: true,
-          onDiagnostic: (msg) => stderr.write(msg),
+          onDiagnostic: (msg) => logger.warn(msg),
           fetch: (statsPeriod, afterTimestamp) =>
             listLogs(org, project, {
               query: flags.query,

src/commands/log/list.ts

@@ -594,13 +594,12 @@ export const listCommand = buildListCommand("project", {
   },
   async *func(this: SentryContext, flags: ListFlags, target?: string) {
     applyFreshFlag(flags);
-    const { stdout, cwd } = this;
+    const { cwd } = this;
 
     const parsed = parseOrgProjectArg(target);
 
     const result = await dispatchOrgScopedList({
       config: projectListMeta,
-      stdout,
       cwd,
       flags,
       parsed,

src/commands/project/list.ts

@@ -10,7 +10,7 @@ import { validateLimit } from "../../lib/arg-parsing.js";
 import { openInBrowser } from "../../lib/browser.js";
 import { buildCommand } from "../../lib/command.js";
 import { ContextError } from "../../lib/errors.js";
-import { displayTraceLogs } from "../../lib/formatters/index.js";
+import { formatTraceLogs } from "../../lib/formatters/index.js";
 import {
   applyFreshFlag,
   FRESH_ALIASES,
@@ -176,10 +176,10 @@ export const logsCommand = buildCommand({
       q: "query",
     },
   },
-  // biome-ignore lint/correctness/useYield: void generator — writes to stdout directly, will be migrated to yield pattern later
+  // biome-ignore lint/correctness/useYield: void generator — early returns for web mode
   async *func(this: SentryContext, flags: LogsFlags, ...args: string[]) {
     applyFreshFlag(flags);
-    const { stdout, cwd, setContext } = this;
+    const { cwd, setContext } = this;
 
     const { traceId, orgArg } = parsePositionalArgs(args);
 
@@ -206,16 +206,17 @@ export const logsCommand = buildCommand({
       query: flags.query,
     });
 
-    displayTraceLogs({
-      stdout,
-      logs,
-      traceId,
-      limit: flags.limit,
-      asJson: flags.json,
-      fields: flags.fields,
-      emptyMessage:
-        `No logs found for trace ${traceId} in the last ${flags.period}.\n\n` +
-        `Try a longer period: sentry trace logs --period 30d ${traceId}\n`,
-    });
+    process.stdout.write(
+      formatTraceLogs({
+        logs,
+        traceId,
+        limit: flags.limit,
+        asJson: flags.json,
+        fields: flags.fields,
+        emptyMessage:
+          `No logs found for trace ${traceId} in the last ${flags.period}.\n\n` +
+          `Try a longer period: sentry trace logs --period 30d ${traceId}\n`,
+      })
+    );
   },
 });

src/commands/trace/logs.ts

@@ -143,11 +143,7 @@ export const startCommand = buildCommand({
 
     // Plan trial: no API to start it — open billing page instead
     if (parsed.name === "plan") {
-      const planResult = await handlePlanTrial(
-        orgSlug,
-        this.stdout,
-        flags.json ?? false
-      );
+      const planResult = await handlePlanTrial(orgSlug, flags.json ?? false);
       yield commandOutput(planResult);
       return;
     }
@@ -181,19 +177,19 @@ export const startCommand = buildCommand({
 /**
  * Show URL + QR code and prompt to open browser if interactive.
  *
+ * Display text goes to stderr via consola — stdout is reserved for
+ * structured command output.
+ *
  * @returns true if browser was opened, false otherwise
  */
-async function promptOpenBillingUrl(
-  url: string,
-  stdout: { write: (s: string) => unknown }
-): Promise<boolean> {
+async function promptOpenBillingUrl(url: string): Promise<boolean> {
   const log = logger.withTag("trial");
 
-  stdout.write(`\n  ${url}\n\n`);
+  log.log(`\n  ${url}\n`);
 
   // Show QR code so mobile/remote users can scan
   const qr = await generateQRCode(url);
-  stdout.write(`${qr}\n`);
+  log.log(qr);
 
   // Prompt to open browser if interactive TTY
   if (isatty(0) && isatty(1)) {
@@ -236,7 +232,6 @@ type PlanTrialResult = {
  */
 async function handlePlanTrial(
   orgSlug: string,
-  stdout: { write: (s: string) => unknown },
   json: boolean
 ): Promise<PlanTrialResult> {
   const log = logger.withTag("trial");
@@ -269,7 +264,7 @@ async function handlePlanTrial(
     log.info(
       `The ${currentPlan} → Business plan trial must be activated in the Sentry UI.`
     );
-    opened = await promptOpenBillingUrl(url, stdout);
+    opened = await promptOpenBillingUrl(url);
   }
 
   return {