fix(json): flatten view command JSON output for --fields filtering (#495)

## Summary

`issue view`, `event view`, and `trace view` commands yielded wrapper
objects (e.g., `{ issue, event, trace }`) as their `CommandOutput` data.
When `--fields` was used, `filterFields` looked for field names at the
top level of this wrapper, so `--fields shortId,title` returned `{}`
because those fields lived inside the nested `issue` object.

## Fix

Add `jsonTransform` to all three view commands that spreads the primary
entity as the top-level object:
- **issue view**: `{ ...issue, event, trace }`
- **event view**: `{ ...event, trace }`
- **trace view**: `{ ...summary, spans }`

This makes `--fields shortId,title` work directly on issue properties,
while enrichment data remains accessible via `--fields event.id` or
`--fields trace.traceId`.

## Test plan
- `bun test test/commands/trace/view.func.test.ts` — updated assertions
for flattened JSON structure
- Manual: `sentry issue view <id> --json --fields shortId,title,count`
should return filtered fields instead of `{}`

Author: BYK

AGENTS.md

@@ -24,6 +24,7 @@ import { openInBrowser } from "../../lib/browser.js";
 import { buildCommand } from "../../lib/command.js";
 import { ApiError, ContextError, ResolutionError } from "../../lib/errors.js";
 import { formatEventDetails } from "../../lib/formatters/index.js";
+import { filterFields } from "../../lib/formatters/json.js";
 import { CommandOutput } from "../../lib/formatters/output.js";
 import {
   applyFreshFlag,
@@ -77,6 +78,27 @@ function formatEventView(data: EventViewData): string {
   return parts.join("\n");
 }
 
+/**
+ * Transform event view data for JSON output.
+ *
+ * Flattens the event as the primary object so that `--fields eventID,title`
+ * works directly on event properties. The `trace` enrichment data is
+ * attached as a nested key, accessible via `--fields trace.traceId`.
+ *
+ * Without this transform, `--fields eventID` would return `{}` because
+ * the raw yield shape is `{ event, trace }` and `eventID` lives inside `event`.
+ */
+function jsonTransformEventView(
+  data: EventViewData,
+  fields?: string[]
+): unknown {
+  const { event, trace } = data;
+  const result: Record<string, unknown> = { ...event, trace };
+  if (fields && fields.length > 0) {
+    return filterFields(result, fields);
+  }
+  return result;
+}
 /** Usage hint for ContextError messages */
 const USAGE_HINT = "sentry event view <org>/<project> <event-id>";
 
@@ -387,7 +409,7 @@ export const viewCommand = buildCommand({
   },
   output: {
     human: formatEventView,
-    jsonExclude: ["spanTreeLines"],
+    jsonTransform: jsonTransformEventView,
   },
   parameters: {
     positional: {

src/commands/event/view.ts

@@ -15,6 +15,7 @@ import {
   isPlainOutput,
   muted,
 } from "../../lib/formatters/index.js";
+import { filterFields } from "../../lib/formatters/json.js";
 import { CommandOutput } from "../../lib/formatters/output.js";
 import {
   applyFreshFlag,
@@ -84,6 +85,30 @@ function formatIssueView(data: IssueViewData): string {
   return parts.join("\n");
 }
 
+/**
+ * Transform issue view data for JSON output.
+ *
+ * Flattens the issue as the primary object so that `--fields shortId,title`
+ * works directly on issue properties. The `event` and `trace` enrichment
+ * data are attached as nested keys, accessible via `--fields event.id`
+ * or `--fields trace.traceId`.
+ *
+ * Without this transform, `--fields shortId` would return `{}` because
+ * the raw yield shape is `{ issue, event, trace }` and `shortId` lives
+ * inside `issue`.
+ */
+function jsonTransformIssueView(
+  data: IssueViewData,
+  fields?: string[]
+): unknown {
+  const { issue, event, trace } = data;
+  const result: Record<string, unknown> = { ...issue, event, trace };
+  if (fields && fields.length > 0) {
+    return filterFields(result, fields);
+  }
+  return result;
+}
+
 export const viewCommand = buildCommand({
   docs: {
     brief: "View details of a specific issue",
@@ -104,7 +129,7 @@ export const viewCommand = buildCommand({
   },
   output: {
     human: formatIssueView,
-    jsonExclude: ["spanTreeLines"],
+    jsonTransform: jsonTransformIssueView,
   },
   parameters: {
     positional: issueIdPositional,

src/commands/issue/view.ts

@@ -19,6 +19,7 @@ import {
   formatSimpleSpanTree,
   formatTraceSummary,
 } from "../../lib/formatters/index.js";
+import { filterFields } from "../../lib/formatters/json.js";
 import { CommandOutput } from "../../lib/formatters/output.js";
 import {
   applyFreshFlag,
@@ -117,6 +118,28 @@ export function formatTraceView(data: TraceViewData): string {
   return parts.join("\n");
 }
 
+/**
+ * Transform trace view data for JSON output.
+ *
+ * Flattens the summary as the primary object so that `--fields traceId,duration`
+ * works directly on summary properties. The raw `spans` array is preserved as
+ * a nested key, accessible via `--fields spans`.
+ *
+ * Without this transform, `--fields traceId` would return `{}` because
+ * the raw yield shape is `{ summary, spans }` and `traceId` lives inside `summary`.
+ */
+function jsonTransformTraceView(
+  data: TraceViewData,
+  fields?: string[]
+): unknown {
+  const { summary, spans } = data;
+  const result: Record<string, unknown> = { ...summary, spans };
+  if (fields && fields.length > 0) {
+    return filterFields(result, fields);
+  }
+  return result;
+}
+
 export const viewCommand = buildCommand({
   docs: {
     brief: "View details of a specific trace",
@@ -130,7 +153,7 @@ export const viewCommand = buildCommand({
   },
   output: {
     human: formatTraceView,
-    jsonExclude: ["spanTreeLines"],
+    jsonTransform: jsonTransformTraceView,
   },
   parameters: {
     positional: {

src/commands/trace/view.ts

@@ -159,9 +159,9 @@ describe("viewCommand.func", () => {
 
     const output = stdoutWrite.mock.calls.map((c) => c[0]).join("");
     const parsed = JSON.parse(output);
-    expect(parsed).toHaveProperty("summary");
+    expect(parsed).toHaveProperty("traceId");
     expect(parsed).toHaveProperty("spans");
-    expect(parsed.summary.traceId).toBe("aaaa1111bbbb2222cccc3333dddd4444");
+    expect(parsed.traceId).toBe("aaaa1111bbbb2222cccc3333dddd4444");
     expect(parsed.spans).toHaveLength(1);
   });
 

test/commands/trace/view.func.test.ts

@@ -175,7 +175,8 @@ describe("sentry trace view", () => {
 
     expect(result.exitCode).toBe(0);
     const data = JSON.parse(result.stdout);
-    expect(data).toHaveProperty("summary");
+    // jsonTransform flattens summary to top level (no wrapper)
+    expect(data).toHaveProperty("traceId");
     expect(data).toHaveProperty("spans");
   });