refactor: replace NodeJS.WriteStream with simple Writer type

Add a simple Writer interface ({ write(s: string): void }) to avoid
dependency on Node.js-specific types. This is more idiomatic for Bun
while maintaining testability through the context pattern.

Files updated:
- types/index.ts: Add Writer type
- formatters/json.ts: Use Writer
- All command files: Use Writer instead of NodeJS.WriteStream

Author: betegon

packages/cli/src/commands/api.ts

@@ -8,6 +8,7 @@
 import { buildCommand } from "@stricli/core";
 import type { SentryContext } from "../context.js";
 import { rawApiRequest } from "../lib/api-client.js";
+import type { Writer } from "../types/index.js";
 
 type HttpMethod = "GET" | "POST" | "PUT" | "DELETE" | "PATCH";
 
@@ -124,7 +125,7 @@ function parseHeaders(headers: string[]): Record<string, string> {
  * Write response headers to stdout
  */
 function writeResponseHeaders(
-  stdout: NodeJS.WriteStream,
+  stdout: Writer,
   status: number,
   headers: Headers
 ): void {
@@ -138,7 +139,7 @@ function writeResponseHeaders(
 /**
  * Write response body to stdout
  */
-function writeResponseBody(stdout: NodeJS.WriteStream, body: unknown): void {
+function writeResponseBody(stdout: Writer, body: unknown): void {
   if (body === null || body === undefined) {
     return;
   }

packages/cli/src/commands/auth/status.ts

@@ -15,7 +15,7 @@ import {
   readConfig,
 } from "../../lib/config.js";
 import { formatExpiration, maskToken } from "../../lib/formatters/human.js";
-import type { SentryConfig } from "../../types/index.js";
+import type { SentryConfig, Writer } from "../../types/index.js";
 
 type StatusFlags = {
   readonly showToken: boolean;
@@ -25,7 +25,7 @@ type StatusFlags = {
  * Write token information
  */
 function writeTokenInfo(
-  stdout: NodeJS.WriteStream,
+  stdout: Writer,
   config: SentryConfig,
   showToken: boolean
 ): void {
@@ -46,7 +46,7 @@ function writeTokenInfo(
 /**
  * Write default settings
  */
-async function writeDefaults(stdout: NodeJS.WriteStream): Promise<void> {
+async function writeDefaults(stdout: Writer): Promise<void> {
   const defaultOrg = await getDefaultOrganization();
   const defaultProject = await getDefaultProject();
 
@@ -66,7 +66,7 @@ async function writeDefaults(stdout: NodeJS.WriteStream): Promise<void> {
 /**
  * Verify credentials by fetching organizations
  */
-async function verifyCredentials(stdout: NodeJS.WriteStream): Promise<void> {
+async function verifyCredentials(stdout: Writer): Promise<void> {
   stdout.write("\nVerifying credentials...\n");
 
   try {

packages/cli/src/commands/event/get.ts

@@ -26,7 +26,7 @@ type GetFlags = {
  * @param detectedFrom - Optional source description for auto-detection
  */
 function writeHumanOutput(
-  stdout: NodeJS.WriteStream,
+  stdout: Writer,
   event: SentryEvent,
   detectedFrom?: string
 ): void {

packages/cli/src/commands/issue/get.ts

@@ -43,7 +43,7 @@ async function tryGetLatestEvent(
  * Write human-readable issue output
  */
 function writeHumanOutput(
-  stdout: NodeJS.WriteStream,
+  stdout: Writer,
   issue: SentryIssue,
   event?: SentryEvent
 ): void {

packages/cli/src/commands/issue/list.ts

@@ -48,7 +48,7 @@ function parseSort(value: string): SortValue {
  * Write the issue list header with column titles.
  */
 function writeListHeader(
-  stdout: NodeJS.WriteStream,
+  stdout: Writer,
   org: string,
   project: string,
   count: number
@@ -61,10 +61,7 @@ function writeListHeader(
 /**
  * Write formatted issue rows to stdout.
  */
-function writeIssueRows(
-  stdout: NodeJS.WriteStream,
-  issues: SentryIssue[]
-): void {
+function writeIssueRows(stdout: Writer, issues: SentryIssue[]): void {
   for (const issue of issues) {
     stdout.write(`${formatIssueRow(issue)}\n`);
   }
@@ -73,7 +70,7 @@ function writeIssueRows(
 /**
  * Write footer with usage tip.
  */
-function writeListFooter(stdout: NodeJS.WriteStream): void {
+function writeListFooter(stdout: Writer): void {
   stdout.write(
     "\nTip: Use 'sentry issue get <SHORT_ID>' to view issue details.\n"
   );

packages/cli/src/commands/org/get.ts

@@ -23,7 +23,7 @@ type GetFlags = {
  * @param detectedFrom - Optional source description if org was auto-detected
  */
 function writeHumanOutput(
-  stdout: NodeJS.WriteStream,
+  stdout: Writer,
   org: Parameters<typeof formatOrgDetails>[0],
   detectedFrom?: string
 ): void {

packages/cli/src/commands/project/get.ts

@@ -24,7 +24,7 @@ type GetFlags = {
  * @param detectedFrom - Optional source description if project was auto-detected
  */
 function writeHumanOutput(
-  stdout: NodeJS.WriteStream,
+  stdout: Writer,
   project: Parameters<typeof formatProjectDetails>[0],
   detectedFrom?: string
 ): void {

packages/cli/src/commands/project/list.ts

@@ -82,15 +82,15 @@ function filterByPlatform(
 /**
  * Write the column header row for project list output.
  */
-function writeHeader(stdout: NodeJS.WriteStream, slugWidth: number): void {
+function writeHeader(stdout: Writer, slugWidth: number): void {
   stdout.write(`${"SLUG".padEnd(slugWidth)}  ${"PLATFORM".padEnd(20)}  NAME\n`);
 }
 
 /**
  * Write formatted project rows to stdout.
  */
 function writeRows(
-  stdout: NodeJS.WriteStream,
+  stdout: Writer,
   projects: ProjectWithOrg[],
   slugWidth: number,
   showOrg: boolean

packages/cli/src/lib/formatters/json.ts

@@ -2,6 +2,8 @@
  * JSON output utilities
  */
 
+import type { Writer } from "../../types/index.js";
+
 /**
  * Format data as pretty-printed JSON
  */
@@ -12,6 +14,6 @@ export function formatJson<T>(data: T): string {
 /**
  * Output JSON to a write stream
  */
-export function writeJson<T>(stream: NodeJS.WriteStream, data: T): void {
+export function writeJson<T>(stream: Writer, data: T): void {
   stream.write(`${formatJson(data)}\n`);
 }

packages/cli/src/types/index.ts

@@ -24,3 +24,14 @@ export type {
   SentryOrganization,
   SentryProject,
 } from "./sentry.js";
+
+// I/O types
+
+/**
+ * Simple writer interface for output streams.
+ * Compatible with process.stdout, process.stderr, and test mocks.
+ * Avoids dependency on Node.js-specific types like NodeJS.WriteStream.
+ */
+export type Writer = {
+  write(data: string): void;
+};