feat(log): support custom attributes in log view via --fields

getLogs and getLogsBatch now accept extraFields, which are merged with
the default DETAILED_LOG_FIELDS before the API request — same pattern
as listLogs. formatLogDetails renders a Custom Attributes section for
any fields that were explicitly requested. The KNOWN_DETAIL_FIELDS
blocklist approach was avoided; instead extraFields flows through
LogViewData so the formatter only shows what was asked for.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>

Author: betegon

src/commands/log/view.ts

@@ -399,6 +399,8 @@ type LogViewData = {
   logs: DetailedSentryLog[];
   /** Org slug — needed by human formatter for trace URLs, also useful context in JSON */
   orgSlug: string;
+  /** Custom fields requested via --fields, passed to the detail formatter */
+  extraFields?: string[];
 };
 
 /**
@@ -416,7 +418,7 @@ function formatLogViewHuman(data: LogViewData): string {
     if (parts.length > 0) {
       parts.push("\n---\n");
     }
-    parts.push(formatLogDetails(entry, data.orgSlug));
+    parts.push(formatLogDetails(entry, data.orgSlug, data.extraFields));
   }
   return parts.join("\n");
 }
@@ -496,7 +498,12 @@ export const viewCommand = buildCommand({
     }
 
     // Fetch all requested log entries
-    const logs = await getLogs(target.org, target.project, logIds);
+    const logs = await getLogs(
+      target.org,
+      target.project,
+      logIds,
+      flags.fields
+    );
 
     if (logs.length === 0) {
       throwNotFoundError(logIds, target.org, target.project);
@@ -508,7 +515,11 @@ export const viewCommand = buildCommand({
       ? `Detected from ${target.detectedFrom}`
       : undefined;
 
-    yield new CommandOutput({ logs, orgSlug: target.org });
+    yield new CommandOutput({
+      logs,
+      orgSlug: target.org,
+      extraFields: flags.fields,
+    });
     return { hint };
   },
 });

src/lib/api/logs.ts

@@ -162,16 +162,24 @@ async function getLogsBatch(
   orgSlug: string,
   projectSlug: string,
   batchIds: string[],
-  config: Awaited<ReturnType<typeof getOrgSdkConfig>>
+  config: Awaited<ReturnType<typeof getOrgSdkConfig>>,
+  extraFields?: string[]
 ): Promise<DetailedSentryLog[]> {
   const query = `project:${projectSlug} sentry.item_id:[${batchIds.join(",")}]`;
 
+  const fields = extraFields?.length
+    ? [
+        ...DETAILED_LOG_FIELDS,
+        ...extraFields.filter((f) => !DETAILED_LOG_FIELDS.includes(f)),
+      ]
+    : DETAILED_LOG_FIELDS;
+
   const result = await queryExploreEventsInTableFormat({
     ...config,
     path: { organization_id_or_slug: orgSlug },
     query: {
       dataset: "logs",
-      field: DETAILED_LOG_FIELDS,
+      field: fields,
       query,
       per_page: batchIds.length,
       statsPeriod: LOG_RETENTION_PERIOD,
@@ -199,13 +207,14 @@ async function getLogsBatch(
 export async function getLogs(
   orgSlug: string,
   projectSlug: string,
-  logIds: string[]
+  logIds: string[],
+  extraFields?: string[]
 ): Promise<DetailedSentryLog[]> {
   const config = await getOrgSdkConfig(orgSlug);
 
   // Single batch — no splitting needed
   if (logIds.length <= API_MAX_PER_PAGE) {
-    return getLogsBatch(orgSlug, projectSlug, logIds, config);
+    return getLogsBatch(orgSlug, projectSlug, logIds, config, extraFields);
   }
 
   // Split into batches of API_MAX_PER_PAGE and fetch in parallel
@@ -215,7 +224,9 @@ export async function getLogs(
   }
 
   const results = await Promise.all(
-    batches.map((batch) => getLogsBatch(orgSlug, projectSlug, batch, config))
+    batches.map((batch) =>
+      getLogsBatch(orgSlug, projectSlug, batch, config, extraFields)
+    )
   );
 
   return results.flat();

src/lib/formatters/log.ts

@@ -287,12 +287,14 @@ function formatSeverityLabel(severity: string | null | undefined): string {
  *
  * @param log - The detailed log entry to format
  * @param orgSlug - Organization slug for building trace URLs
+ * @param extraFields - Custom fields requested via --fields, shown in Custom Attributes section
  * @returns Rendered terminal string
  */
 // biome-ignore lint/complexity/noExcessiveCognitiveComplexity: log detail formatting requires multiple conditional sections
 export function formatLogDetails(
   log: DetailedSentryLog,
-  orgSlug: string
+  orgSlug: string,
+  extraFields?: string[]
 ): string {
   const logId = log["sentry.item_id"];
   const lines: string[] = [];
@@ -394,5 +396,16 @@ export function formatLogDetails(
     lines.push(mdKvTable(otelRows, "OpenTelemetry"));
   }
 
+  // Custom Attributes — fields explicitly requested via --fields
+  if (extraFields?.length) {
+    const customRows: [string, string][] = extraFields
+      .filter((f) => log[f] !== null && log[f] !== undefined)
+      .map((f) => [f, String(log[f])]);
+    if (customRows.length > 0) {
+      lines.push("");
+      lines.push(mdKvTable(customRows, "Custom Attributes"));
+    }
+  }
+
   return renderMarkdown(lines.join("\n"));
 }

test/commands/log/view.func.test.ts

@@ -193,7 +193,12 @@ describe("viewCommand.func", () => {
       );
 
       // getLogs should have been called with both IDs
-      expect(getLogsSpy).toHaveBeenCalledWith("my-org", "proj", [ID1, ID2]);
+      expect(getLogsSpy).toHaveBeenCalledWith(
+        "my-org",
+        "proj",
+        [ID1, ID2],
+        undefined
+      );
 
       const output = stdoutWrite.mock.calls.map((c) => c[0]).join("");
       const parsed = JSON.parse(output);
@@ -295,9 +300,12 @@ describe("viewCommand.func", () => {
       await func.call(context, { json: true, web: false }, "my-project", ID1);
 
       expect(resolveProjectBySlugSpy).toHaveBeenCalled();
-      expect(getLogsSpy).toHaveBeenCalledWith("resolved-org", "resolved-proj", [
-        ID1,
-      ]);
+      expect(getLogsSpy).toHaveBeenCalledWith(
+        "resolved-org",
+        "resolved-proj",
+        [ID1],
+        undefined
+      );
     });
 
     test("org/ target (org-all) throws ContextError", async () => {
@@ -327,9 +335,12 @@ describe("viewCommand.func", () => {
       await func.call(context, { json: false, web: false }, ID1);
 
       expect(resolveOrgAndProjectSpy).toHaveBeenCalled();
-      expect(getLogsSpy).toHaveBeenCalledWith("detected-org", "detected-proj", [
-        ID1,
-      ]);
+      expect(getLogsSpy).toHaveBeenCalledWith(
+        "detected-org",
+        "detected-proj",
+        [ID1],
+        undefined
+      );
 
       // Human output should include the detected-from hint
       const output = stdoutWrite.mock.calls.map((c) => c[0]).join("");