perf(api): collapse stats on issue detail endpoints to save 100-300ms

Switch getIssueInOrg and getIssueByShortId from @sentry/api SDK to raw
apiRequestToRegion() to enable passing the collapse query parameter.
The SDK types declare query?: never on retrieveAnIssue and
resolveAShortId, blocking query params entirely.

Add ISSUE_DETAIL_COLLAPSE constant that collapses stats, lifetime,
filtered, and unhandled fields on all single-issue API calls. These
fields are never displayed in issue view, explain, or plan commands.
The count, userCount, firstSeen, and lastSeen fields remain unaffected
as they are top-level fields outside the collapsed sub-objects.

Also add collapse to resolveSelector() which calls listIssuesPaginated
with perPage: 1 — previously fetched all fields for a single issue
that only needed identity data.

Estimated savings: 100-300ms per issue detail request by skipping
expensive Snuba/ClickHouse queries on the backend.

Author: BYK

AGENTS.md

@@ -880,4 +880,6 @@ mock.module("./some-module", () => ({
 
 <!-- lore:019cc325-d322-7e6e-86cc-93010b71abee -->
 * **Testing Stricli command func() bodies via spyOn mocking**: Stricli/Bun test patterns: (1) Command func tests: \`const func = await cmd.loader()\`, then \`func.call(mockContext, flags, ...args)\`. \`loader()\` return type union causes LSP errors — false positives that pass \`tsc\`. File naming: \`\*.func.test.ts\`. (2) ESM prevents \`vi.spyOn\` on Node built-in exports. Workaround: test subclass that overrides the method calling the built-in. (3) Follow-mode uses \`setTimeout\`-based scheduling; test with \`interceptSigint()\` helper. \`Bun.sleep()\` has no AbortSignal so \`setTimeout\`/\`clearTimeout\` required.
+
+* **@sentry/api SDK blocks query params on issue detail endpoints**: The \`retrieveAnIssue\` and \`resolveAShortId\` SDK functions have \`query?: never\` in their TypeScript types, preventing callers from passing \`collapse\` or other query parameters. The Sentry backend DOES accept \`collapse\` on these endpoints (same Django view base class as the list endpoint), but the OpenAPI spec omits it. The CLI works around this by using raw \`apiRequestToRegion()\` instead of the SDK for \`getIssueInOrg\` and \`getIssueByShortId\`. Upstream issue filed at https://github.com/getsentry/sentry-api-schema/. If the schema is fixed, these functions can switch back to the SDK.
 <!-- End lore-managed section -->

src/commands/issue/utils.ts

@@ -11,6 +11,7 @@ import {
   getIssue,
   getIssueByShortId,
   getIssueInOrg,
+  ISSUE_DETAIL_COLLAPSE,
   type IssueSort,
   listIssuesPaginated,
   listOrganizations,
@@ -137,7 +138,9 @@ async function tryResolveFromAlias(
   }
 
   const resolvedShortId = expandToFullShortId(suffix, projectEntry.projectSlug);
-  const issue = await getIssueByShortId(projectEntry.orgSlug, resolvedShortId);
+  const issue = await getIssueByShortId(projectEntry.orgSlug, resolvedShortId, {
+    collapse: ISSUE_DETAIL_COLLAPSE,
+  });
   return { org: projectEntry.orgSlug, issue };
 }
 
@@ -182,7 +185,9 @@ async function resolveProjectSearchFallback(
   const matchedOrg = matchedProject?.orgSlug;
   if (matchedOrg && matchedProject) {
     const retryShortId = expandToFullShortId(suffix, matchedProject.slug);
-    const issue = await getIssueByShortId(matchedOrg, retryShortId);
+    const issue = await getIssueByShortId(matchedOrg, retryShortId, {
+      collapse: ISSUE_DETAIL_COLLAPSE,
+    });
     return { org: matchedOrg, issue };
   }
 
@@ -240,7 +245,9 @@ async function resolveProjectSearch(
     dsnTarget.project.toLowerCase() === projectSlug.toLowerCase()
   ) {
     const fullShortId = expandToFullShortId(suffix, dsnTarget.project);
-    const issue = await getIssueByShortId(dsnTarget.org, fullShortId);
+    const issue = await getIssueByShortId(dsnTarget.org, fullShortId, {
+      collapse: ISSUE_DETAIL_COLLAPSE,
+    });
     return { org: dsnTarget.org, issue };
   }
 
@@ -255,7 +262,11 @@ async function resolveProjectSearch(
   const results = await Promise.all(
     orgs.map((org) =>
       limit(() =>
-        withAuthGuard(() => tryGetIssueByShortId(org.slug, fullShortId))
+        withAuthGuard(() =>
+          tryGetIssueByShortId(org.slug, fullShortId, {
+            collapse: ISSUE_DETAIL_COLLAPSE,
+          })
+        )
       )
     )
   );
@@ -327,7 +338,9 @@ async function resolveSuffixOnly(
     );
   }
   const fullShortId = expandToFullShortId(suffix, target.project);
-  const issue = await getIssueByShortId(target.org, fullShortId);
+  const issue = await getIssueByShortId(target.org, fullShortId, {
+    collapse: ISSUE_DETAIL_COLLAPSE,
+  });
   return { org: target.org, issue };
 }
 
@@ -413,11 +426,13 @@ async function resolveSelector(
   const sort = SELECTOR_SORT_MAP[selector];
   const label = SELECTOR_LABELS[selector];
 
-  // Fetch just the top issue with the appropriate sort
+  // Fetch just the top issue with the appropriate sort.
+  // Collapse all non-essential fields since we only need the issue identity.
   const { data: issues } = await listIssuesPaginated(orgSlug, "", {
     sort,
     perPage: 1,
     query: "is:unresolved",
+    collapse: ISSUE_DETAIL_COLLAPSE,
   });
 
   const issue = issues[0];
@@ -483,8 +498,10 @@ async function resolveNumericIssue(
   const resolvedOrg = await resolveOrg({ cwd });
   try {
     const issue = resolvedOrg
-      ? await getIssueInOrg(resolvedOrg.org, id)
-      : await getIssue(id);
+      ? await getIssueInOrg(resolvedOrg.org, id, {
+          collapse: ISSUE_DETAIL_COLLAPSE,
+        })
+      : await getIssue(id, { collapse: ISSUE_DETAIL_COLLAPSE });
     // Extract org from the response permalink as a fallback so that callers
     // like resolveOrgAndIssueId (used by explain/plan) get the org slug even
     // when no org context was available before the fetch.
@@ -542,7 +559,9 @@ export async function resolveIssue(
       // Full context: org + project + suffix
       const org = await resolveEffectiveOrg(parsed.org);
       const fullShortId = expandToFullShortId(parsed.suffix, parsed.project);
-      const issue = await getIssueByShortId(org, fullShortId);
+      const issue = await getIssueByShortId(org, fullShortId, {
+        collapse: ISSUE_DETAIL_COLLAPSE,
+      });
       result = { org, issue };
       break;
     }
@@ -551,7 +570,9 @@ export async function resolveIssue(
       // Org + numeric ID — use org-scoped endpoint for proper region routing.
       const org = await resolveEffectiveOrg(parsed.org);
       try {
-        const issue = await getIssueInOrg(org, parsed.numericId);
+        const issue = await getIssueInOrg(org, parsed.numericId, {
+          collapse: ISSUE_DETAIL_COLLAPSE,
+        });
         result = { org, issue };
       } catch (err) {
         if (err instanceof ApiError && err.status === 404) {

src/lib/api-client.ts

@@ -48,6 +48,7 @@ export {
   getIssue,
   getIssueByShortId,
   getIssueInOrg,
+  ISSUE_DETAIL_COLLAPSE,
   type IssueCollapseField,
   type IssueSort,
   type IssuesPage,

src/lib/api/issues.ts

@@ -5,24 +5,21 @@
  */
 
 import type { ListAnOrganizationSissuesData } from "@sentry/api";
-import {
-  listAnOrganization_sIssues,
-  resolveAShortId,
-  retrieveAnIssue,
-} from "@sentry/api";
+import { listAnOrganization_sIssues } from "@sentry/api";
 
 import type { SentryIssue } from "../../types/index.js";
 
 import { ApiError } from "../errors.js";
+import { resolveOrgRegion } from "../region.js";
 
 import {
   API_MAX_PER_PAGE,
   apiRequest,
+  apiRequestToRegion,
   getOrgSdkConfig,
   MAX_PAGINATION_PAGES,
   type PaginatedResponse,
   unwrapPaginatedResult,
-  unwrapResult,
 } from "./infrastructure.js";
 
 /**
@@ -74,6 +71,24 @@ export function buildIssueListCollapse(options: {
   return collapse;
 }
 
+/**
+ * Collapse fields for single-issue detail endpoints.
+ *
+ * The CLI never displays stats (sparkline time-series), lifetime (aggregate
+ * sub-object), filtered (filtered counts), or unhandled (unhandled sub-object)
+ * in detail views (`issue view`, `issue explain`, `issue plan`).
+ * Collapsing these skips expensive Snuba queries, saving 100-300ms per request.
+ *
+ * Note: `count`, `userCount`, `firstSeen`, `lastSeen` are top-level fields
+ * and remain unaffected by collapsing.
+ */
+export const ISSUE_DETAIL_COLLAPSE: IssueCollapseField[] = [
+  "stats",
+  "lifetime",
+  "filtered",
+  "unhandled",
+];
+
 /**
  * List issues for a project with pagination control.
  *
@@ -237,58 +252,80 @@ export async function listIssuesAllPages(
  *
  * Uses the legacy unscoped endpoint — no org context or region routing.
  * Prefer {@link getIssueInOrg} when the org slug is known.
+ *
+ * @param issueId - Numeric issue ID
+ * @param options - Optional collapse fields to skip expensive backend queries
  */
-export function getIssue(issueId: string): Promise<SentryIssue> {
-  // The @sentry/api SDK's retrieveAnIssue requires org slug in path,
-  // but the legacy endpoint /issues/{id}/ works without org context.
-  // Use raw request for backward compatibility.
-  return apiRequest<SentryIssue>(`/issues/${issueId}/`);
+export function getIssue(
+  issueId: string,
+  options?: { collapse?: IssueCollapseField[] }
+): Promise<SentryIssue> {
+  return apiRequest<SentryIssue>(`/issues/${issueId}/`, {
+    params: options?.collapse ? { collapse: options.collapse } : undefined,
+  });
 }
 
 /**
  * Get a specific issue by numeric ID, scoped to an organization.
  *
- * Uses the org-scoped SDK endpoint with region-aware routing.
+ * Uses the org-scoped endpoint with region-aware routing.
  * Preferred over {@link getIssue} when the org slug is available.
  *
+ * Uses raw `apiRequestToRegion` instead of the SDK's `retrieveAnIssue`
+ * because the SDK types declare `query?: never`, blocking `collapse`
+ * and other query parameters. See: https://github.com/getsentry/sentry-api-schema/
+ *
  * @param orgSlug - Organization slug (used for region routing)
  * @param issueId - Numeric issue ID
+ * @param options - Optional collapse fields to skip expensive backend queries
  */
 export async function getIssueInOrg(
   orgSlug: string,
-  issueId: string
+  issueId: string,
+  options?: { collapse?: IssueCollapseField[] }
 ): Promise<SentryIssue> {
-  const config = await getOrgSdkConfig(orgSlug);
-  const result = await retrieveAnIssue({
-    ...config,
-    path: { organization_id_or_slug: orgSlug, issue_id: issueId },
-  });
-  return unwrapResult(result, "Failed to get issue") as unknown as SentryIssue;
+  const regionUrl = await resolveOrgRegion(orgSlug);
+  const { data } = await apiRequestToRegion<SentryIssue>(
+    regionUrl,
+    `/organizations/${orgSlug}/issues/${issueId}/`,
+    {
+      params: options?.collapse ? { collapse: options.collapse } : undefined,
+    }
+  );
+  return data;
 }
 
 /**
  * Get an issue by short ID (e.g., SPOTLIGHT-ELECTRON-4D).
  * Requires organization context to resolve the short ID.
  * Uses region-aware routing for multi-region support.
+ *
+ * Uses raw `apiRequestToRegion` instead of the SDK's `resolveAShortId`
+ * because the SDK types declare `query?: never`, blocking `collapse`
+ * and other query parameters. See: https://github.com/getsentry/sentry-api-schema/
+ *
+ * @param orgSlug - Organization slug
+ * @param shortId - Short ID (e.g., "CLI-G5", "SPOTLIGHT-ELECTRON-4D")
+ * @param options - Optional collapse fields to skip expensive backend queries
  */
 export async function getIssueByShortId(
   orgSlug: string,
-  shortId: string
+  shortId: string,
+  options?: { collapse?: IssueCollapseField[] }
 ): Promise<SentryIssue> {
   const normalizedShortId = shortId.toUpperCase();
-  const config = await getOrgSdkConfig(orgSlug);
+  const regionUrl = await resolveOrgRegion(orgSlug);
 
-  const result = await resolveAShortId({
-    ...config,
-    path: {
-      organization_id_or_slug: orgSlug,
-      issue_id: normalizedShortId,
-    },
-  });
-
-  let data: ReturnType<typeof unwrapResult>;
+  let data: { group?: SentryIssue };
   try {
-    data = unwrapResult(result, "Failed to resolve short ID");
+    const result = await apiRequestToRegion<{ group?: SentryIssue }>(
+      regionUrl,
+      `/organizations/${orgSlug}/shortids/${normalizedShortId}/`,
+      {
+        params: options?.collapse ? { collapse: options.collapse } : undefined,
+      }
+    );
+    data = result.data;
   } catch (error) {
     // Enrich 404 errors with actionable context. The generic
     // "Failed to resolve short ID: 404 Not Found" is the most common
@@ -308,16 +345,14 @@ export async function getIssueByShortId(
     throw error;
   }
 
-  // resolveAShortId returns a ShortIdLookupResponse with a group (issue)
-  const resolved = data as unknown as { group?: SentryIssue };
-  if (!resolved.group) {
+  if (!data.group) {
     throw new ApiError(
       `Short ID ${normalizedShortId} resolved but no issue group returned`,
       404,
       "Issue not found"
     );
   }
-  return resolved.group;
+  return data.group;
 }
 
 /**
@@ -333,10 +368,11 @@ export async function getIssueByShortId(
  */
 export async function tryGetIssueByShortId(
   orgSlug: string,
-  shortId: string
+  shortId: string,
+  options?: { collapse?: IssueCollapseField[] }
 ): Promise<SentryIssue | null> {
   try {
-    return await getIssueByShortId(orgSlug, shortId);
+    return await getIssueByShortId(orgSlug, shortId, options);
   } catch (error) {
     if (error instanceof ApiError && error.status === 404) {
       return null;