refactor: api command uses JSON-only output for both normal and dry-run

The api command is a JSON proxy — its output should always be JSON,
not conditionally switch to human-readable markdown in dry-run mode.

- Revert output config to 'json' (flag-only, no human formatter)
- Dry-run writes JSON directly via writeJson (imperative, since
  flag-only mode doesn't intercept returns)
- Remove formatDryRunRequest and its markdown imports entirely
- Remove formatDryRunRequest test suite (6 tests)

Author: BYK

src/commands/api.ts

@@ -10,12 +10,6 @@ import { buildSearchParams, rawApiRequest } from "../lib/api-client.js";
 import { buildCommand } from "../lib/command.js";
 import { ValidationError } from "../lib/errors.js";
 import { writeJson } from "../lib/formatters/json.js";
-import {
-  escapeMarkdownInline,
-  mdKvTable,
-  renderMarkdown,
-  safeCodeSpan,
-} from "../lib/formatters/markdown.js";
 import { validateEndpoint } from "../lib/input-validation.js";
 import { getDefaultSdkConfig } from "../lib/sentry-client.js";
 import type { Writer } from "../types/index.js";
@@ -991,45 +985,6 @@ export function buildDryRunRequest(
   };
 }
 
-/**
- * Format a dry-run request preview as rendered markdown.
- *
- * Uses the standard mdKvTable layout for consistency with other commands.
- *
- * @internal Exported for testing
- */
-export function formatDryRunRequest(request: DryRunRequest): string {
-  const lines: string[] = [];
-  lines.push(
-    `## <muted>Dry run</muted> — ${escapeMarkdownInline(request.method)} ${escapeMarkdownInline(request.url)}`
-  );
-  lines.push("");
-
-  const kvRows: [string, string][] = [
-    ["Method", request.method],
-    ["URL", request.url],
-  ];
-
-  const headerEntries = Object.entries(request.headers);
-  if (headerEntries.length > 0) {
-    kvRows.push([
-      "Headers",
-      headerEntries.map(([k, v]) => `${k}: ${v}`).join(", "),
-    ]);
-  }
-
-  if (request.body !== null) {
-    const bodyStr =
-      typeof request.body === "string"
-        ? request.body
-        : JSON.stringify(request.body, null, 2);
-    kvRows.push(["Body", safeCodeSpan(bodyStr)]);
-  }
-
-  lines.push(mdKvTable(kvRows));
-  return renderMarkdown(lines.join("\n"));
-}
-
 /**
  * Handle response output based on flags
  * @internal Exported for testing
@@ -1197,7 +1152,7 @@ export async function resolveBody(
 // Command Definition
 
 export const apiCommand = buildCommand({
-  output: { json: true, human: formatDryRunRequest },
+  output: "json",
   docs: {
     brief: "Make an authenticated API request",
     fullDescription:
@@ -1322,13 +1277,13 @@ export const apiCommand = buildCommand({
 
     // Dry-run mode: show the resolved request without sending it
     if (flags["dry-run"]) {
-      return {
-        data: buildDryRunRequest(flags.method, normalizedEndpoint, {
-          params,
-          headers,
-          body,
-        }),
-      };
+      const request = buildDryRunRequest(flags.method, normalizedEndpoint, {
+        params,
+        headers,
+        body,
+      });
+      writeJson(stdout, request, flags.fields);
+      return;
     }
 
     // Verbose mode: show request details (unless silent)

test/commands/api.test.ts

@@ -15,10 +15,8 @@ import {
   buildQueryParams,
   buildQueryParamsFromFields,
   buildRawQueryParams,
-  type DryRunRequest,
   dataToQueryParams,
   extractJsonBody,
-  formatDryRunRequest,
   handleResponse,
   normalizeEndpoint,
   normalizeFields,
@@ -1829,75 +1827,3 @@ describe("buildDryRunRequest", () => {
     expect(request.body).toBe('{"raw":"string"}');
   });
 });
-
-describe("formatDryRunRequest", () => {
-  const req = (overrides: Partial<DryRunRequest> = {}): DryRunRequest => ({
-    method: "GET",
-    url: "https://sentry.io/api/0/organizations/",
-    headers: {},
-    body: null,
-    ...overrides,
-  });
-
-  test("includes method and URL", () => {
-    const output = formatDryRunRequest(req());
-
-    expect(output).toContain("GET");
-    expect(output).toContain("https://sentry.io/api/0/organizations/");
-    expect(output).toContain("Dry run");
-  });
-
-  test("includes headers", () => {
-    const output = formatDryRunRequest(
-      req({
-        method: "POST",
-        url: "https://sentry.io/api/0/issues/",
-        headers: {
-          "Content-Type": "application/json",
-          Authorization: "Bearer token",
-        },
-      })
-    );
-
-    expect(output).toContain("Content-Type: application/json");
-    expect(output).toContain("Authorization: Bearer token");
-  });
-
-  test("includes JSON body", () => {
-    const output = formatDryRunRequest(
-      req({
-        method: "PUT",
-        url: "https://sentry.io/api/0/issues/123/",
-        body: { status: "resolved" },
-      })
-    );
-
-    expect(output).toContain("Body");
-    expect(output).toContain('"status": "resolved"');
-  });
-
-  test("includes string body as-is", () => {
-    const output = formatDryRunRequest(
-      req({
-        method: "POST",
-        url: "https://sentry.io/api/0/events/",
-        body: "raw-body-content",
-      })
-    );
-
-    expect(output).toContain("Body");
-    expect(output).toContain("raw-body-content");
-  });
-
-  test("omits body when null", () => {
-    const output = formatDryRunRequest(req());
-
-    expect(output).not.toContain("Body");
-  });
-
-  test("omits headers row when empty", () => {
-    const output = formatDryRunRequest(req());
-
-    expect(output).not.toContain("Headers");
-  });
-});