feat(issue-list): redesign table to match Sentry web UI

Author: BYK

AGENTS.md

@@ -227,6 +227,7 @@ List issues in a project
 - `--json - Output JSON`
 - `-c, --cursor <value> - Pagination cursor for <org>/ or multi-target modes (use "last" to continue)`
 - `-f, --fresh - Bypass cache and fetch fresh data`
+- `--compact - Single-line rows for compact output`
 
 **Examples:**
 
@@ -646,6 +647,7 @@ List issues in a project
 - `--json - Output JSON`
 - `-c, --cursor <value> - Pagination cursor for <org>/ or multi-target modes (use "last" to continue)`
 - `-f, --fresh - Bypass cache and fetch fresh data`
+- `--compact - Single-line rows for compact output`
 
 ### Orgs
 

plugins/sentry-cli/skills/sentry-cli/SKILL.md

@@ -30,6 +30,22 @@ export type FileData = {
   fnhMax: number;
 };
 
+/**
+ * Per-input DA line tracking for phantom line filtering.
+ *
+ * Bun's coverage instrumentation generates DA entries for transitively imported
+ * modules, including lines that don't correspond to executable code. These
+ * "phantom" lines always have 0 hits and only appear in one input (typically
+ * the isolated test suite). We track which inputs contain each DA line so we
+ * can exclude 0-hit lines that appear in only one input.
+ */
+type PerInputDA = {
+  /** How many input files included this DA line */
+  inputCount: number;
+  /** Whether any input had hits > 0 */
+  hasHits: boolean;
+};
+
 function createFileData(): FileData {
   return {
     da: new Map(),
@@ -51,6 +67,8 @@ if (files.length === 0) {
 const merged = new Map<string, FileData>();
 /** Track insertion order of source files */
 const fileOrder: string[] = [];
+/** Per-source-file, per-line tracking of which inputs contributed each DA line */
+const daInputTracking = new Map<string, Map<number, PerInputDA>>();
 
 for (const filePath of files) {
   const content = await Bun.file(filePath).text();
@@ -63,6 +81,7 @@ for (const filePath of files) {
       if (!merged.has(currentSF)) {
         merged.set(currentSF, createFileData());
         fileOrder.push(currentSF);
+        daInputTracking.set(currentSF, new Map());
       }
       data = merged.get(currentSF) ?? null;
     } else if (line.startsWith("DA:") && data) {
@@ -72,6 +91,18 @@ for (const filePath of files) {
       if (!data.da.has(lineNo) || hits > (data.da.get(lineNo) ?? 0)) {
         data.da.set(lineNo, hits);
       }
+
+      // Track which inputs contributed this DA line for phantom filtering
+      const tracking = daInputTracking.get(currentSF);
+      if (tracking) {
+        const existing = tracking.get(lineNo);
+        if (existing) {
+          existing.inputCount += 1;
+          existing.hasHits = existing.hasHits || hits > 0;
+        } else {
+          tracking.set(lineNo, { inputCount: 1, hasHits: hits > 0 });
+        }
+      }
     } else if (line.startsWith("FN:") && data) {
       const val = line.slice(3);
       const comma = val.indexOf(",");
@@ -156,16 +187,29 @@ for (const sf of fileOrder) {
     out.push(`FNH:${data.fnhMax}`);
   }
 
-  // DA lines sorted by line number + compute LF/LH
+  // DA lines sorted by line number + compute LF/LH.
+  // Filter out phantom lines: 0-hit DA entries that only appeared in a single
+  // input file. These are artifacts of Bun's coverage instrumentation for
+  // transitively imported modules and inflate the uncovered line count.
+  const tracking = daInputTracking.get(sf);
   const sortedLines = [...data.da.entries()].sort((a, b) => a[0] - b[0]);
+  let lf = 0;
   let lh = 0;
   for (const [lineNo, hits] of sortedLines) {
+    if (hits === 0 && files.length > 1 && tracking) {
+      const info = tracking.get(lineNo);
+      if (info && info.inputCount === 1 && !info.hasHits) {
+        // Phantom line: 0 hits, only in one input, never had hits — skip
+        continue;
+      }
+    }
     out.push(`DA:${lineNo},${hits}`);
+    lf += 1;
     if (hits > 0) {
       lh += 1;
     }
   }
-  out.push(`LF:${data.da.size}`);
+  out.push(`LF:${lf}`);
   out.push(`LH:${lh}`);
 
   // BRDA lines + compute BRF/BRH

script/merge-lcov.ts

@@ -84,6 +84,7 @@ type ListFlags = {
   readonly json: boolean;
   readonly cursor?: string;
   readonly fresh: boolean;
+  readonly compact: boolean;
 };
 
 /** @internal */ export type SortValue = "date" | "new" | "freq" | "user";
@@ -438,7 +439,7 @@ async function fetchIssuesForTarget(
     const { issues, nextCursor } = await listIssuesAllPages(
       target.org,
       target.project,
-      { ...options, projectId: target.projectId }
+      { ...options, projectId: target.projectId, groupStatsPeriod: "auto" }
     );
     return { target, issues, hasMore: !!nextCursor, nextCursor };
   });
@@ -735,6 +736,7 @@ async function fetchOrgAllIssues(
       perPage,
       sort: flags.sort,
       statsPeriod: flags.period,
+      groupStatsPeriod: "auto",
     });
     return { issues: response.data, nextCursor: response.nextCursor };
   }
@@ -745,6 +747,7 @@ async function fetchOrgAllIssues(
     limit: flags.limit,
     sort: flags.sort,
     statsPeriod: flags.period,
+    groupStatsPeriod: "auto",
     onPage,
   });
   return { issues, nextCursor };
@@ -826,7 +829,7 @@ async function handleOrgAllIssues(options: OrgAllIssuesOptions): Promise<void> {
       isMultiProject: true,
     },
   }));
-  writeIssueTable(stdout, issuesWithOpts, true);
+  writeIssueTable(stdout, issuesWithOpts, { compact: flags.compact });
 
   if (hasMore) {
     stdout.write(`\nShowing ${issues.length} issues (more available)\n`);
@@ -1088,7 +1091,7 @@ async function handleResolvedTargets(
       : `Issues from ${validResults.length} projects`;
 
   writeListHeader(stdout, title);
-  writeIssueTable(stdout, issuesWithOptions, isMultiProject);
+  writeIssueTable(stdout, issuesWithOptions, { compact: flags.compact });
 
   let footerMode: "single" | "multi" | "none" = "none";
   if (isMultiProject) {
@@ -1198,6 +1201,11 @@ export const listCommand = buildListCommand("issue", {
         optional: true,
       },
       fresh: FRESH_FLAG,
+      compact: {
+        kind: "boolean",
+        brief: "Single-line rows for compact output",
+        default: false,
+      },
     },
     aliases: {
       ...LIST_BASE_ALIASES,

src/commands/issue/list.ts

@@ -1149,6 +1149,8 @@ export async function listIssuesPaginated(
      *  instead of `project:<slug>` search syntax, avoiding "not actively
      *  selected" errors. */
     projectId?: number;
+    /** Controls the time resolution of inline stats data. "auto" adapts to statsPeriod. */
+    groupStatsPeriod?: "" | "14d" | "24h" | "auto";
   } = {}
 ): Promise<PaginatedResponse<SentryIssue[]>> {
   // When we have a numeric project ID, use the `project` query param (Array<number>)
@@ -1175,6 +1177,7 @@ export async function listIssuesPaginated(
       limit: options.perPage ?? 25,
       sort: options.sort,
       statsPeriod: options.statsPeriod,
+      groupStatsPeriod: options.groupStatsPeriod,
     },
   });
 
@@ -1221,6 +1224,8 @@ export async function listIssuesAllPages(
     statsPeriod?: string;
     /** Numeric project ID for direct project selection via query param. */
     projectId?: number;
+    /** Controls the time resolution of inline stats data. "auto" adapts to statsPeriod. */
+    groupStatsPeriod?: "" | "14d" | "24h" | "auto";
     /** Resume pagination from this cursor instead of starting from the beginning. */
     startCursor?: string;
     /** Called after each page is fetched. Useful for progress indicators. */
@@ -1247,6 +1252,7 @@ export async function listIssuesAllPages(
       sort: options.sort,
       statsPeriod: options.statsPeriod,
       projectId: options.projectId,
+      groupStatsPeriod: options.groupStatsPeriod,
     });
 
     allResults.push(...response.data);