feat: update @sentry/api to 0.133.0 and adopt pagination improvements (#915)

Update `@sentry/api` from 0.113.0 to 0.133.0 and adopt the new
pagination type improvements. This eliminates 5 unsafe `as { cursor?:
string }` casts, replaces 2 manual pagination loops with `autoPaginate`,
and adds `prevCursor` support to `PaginatedResponse`.

## Changes

- **`@sentry/api` ^0.133.0** — includes pagination wrappers
(`fetchPage_*`, `paginateAll_*`, `paginateUpTo_*`), widened query types
(`cursor` + `per_page` now typed), and `prevCursor` in link header
parsing
- **infrastructure.ts** — `PaginatedResponse<T>` gains `prevCursor?:
string`; `unwrapPaginatedResult` passes it through
- **projects.ts** — manual 30-line pagination loop → `autoPaginate`
one-liner; casts cleaned up
- **repositories.ts** — manual 15-line pagination loop → `autoPaginate`;
casts cleaned up
- **teams.ts, releases.ts, issues.ts, events.ts** — `as { cursor?:
string }` casts narrowed/removed; multi-line result casts collapsed

## What's NOT migrated (follow-up)

`traces.ts`, `discover.ts`, and `dashboards.ts` widget queries stay on
raw `apiRequestToRegion` — they use Zod validation schemas and the SDK
wrappers' error handling (plain `Error`) is incompatible with the CLI's
`ApiError`/`AuthError` pipeline.

## Testing

- `bun test test/lib/api-client.coverage.test.ts` — 91/91 pass
- `npx tsc --noEmit` — clean (no new errors)

<!--
## Plan
1. Bump @sentry/api from 0.113.0 to 0.133.0
2. Add prevCursor to PaginatedResponse type and unwrapPaginatedResult
3. Replace manual pagination loops in listProjects() and
listAllRepositories() with autoPaginate
4. Clean up as { cursor?: string } casts — cursor and per_page are now
in SDK query types
5. Remove unnecessary multi-line result casts
6. Keep CLI's own unwrapPaginatedResult (not SDK's) for
ApiError/AuthError preservation
7. Leave traces.ts/discover.ts/dashboards.ts raw call sites for
follow-up (Zod validation + error type incompatibility)
-->

---------

Co-authored-by: Sentry Bot <bot@sentry.io>

Author: MathurAditya724

bun.lock

@@ -11,7 +11,7 @@
     "@biomejs/biome": "2.3.8",
     "@clack/prompts": "^0.11.0",
     "@mastra/client-js": "^1.4.0",
-    "@sentry/api": "^0.113.0",
+    "@sentry/api": "^0.133.0",
     "@sentry/node-core": "10.50.0",
     "@sentry/sqlish": "^1.0.0",
     "@stricli/auto-complete": "^1.2.4",

package.json

@@ -237,9 +237,7 @@ export async function listIssueEvents(
     });
 
     const paginated = unwrapPaginatedResult(
-      result as
-        | { data: IssueEvent[]; error: undefined }
-        | { data: undefined; error: unknown },
+      result,
       "Failed to list issue events"
     );
 

src/lib/api/events.ts

@@ -15,6 +15,7 @@ import { extractRequiredScopes } from "../api-scope.js";
 import { getActiveEnvVarName, isEnvTokenActive } from "../db/auth.js";
 import { getEnv } from "../env.js";
 import { ApiError, AuthError, stringifyUnknown } from "../errors.js";
+import { logger } from "../logger.js";
 import { resolveOrgRegion } from "../region.js";
 import {
   getApiBaseUrl,
@@ -188,8 +189,17 @@ export function unwrapPaginatedResult<T>(
 ): PaginatedResponse<T> {
   const response = (result as { response?: Response }).response;
   const data = unwrapResult(result, context);
-  const { nextCursor } = parseLinkHeader(response?.headers.get("link") ?? null);
-  return { data, nextCursor };
+  const { nextCursor, prevCursor } = parseLinkHeader(
+    response?.headers.get("link") ?? null
+  );
+  const out: PaginatedResponse<T> = { data };
+  if (nextCursor !== undefined) {
+    out.nextCursor = nextCursor;
+  }
+  if (prevCursor !== undefined) {
+    out.prevCursor = prevCursor;
+  }
+  return out;
 }
 
 /**
@@ -270,6 +280,8 @@ export type PaginatedResponse<T> = {
   data: T;
   /** Cursor for fetching the next page (undefined if no more pages) */
   nextCursor?: string;
+  /** Cursor for the previous page (undefined on the first page) */
+  prevCursor?: string;
 };
 
 /**
@@ -316,7 +328,11 @@ export async function autoPaginate<T>(
     cursor = result.nextCursor;
   }
 
-  // Safety limit reached — return what we have, no nextCursor
+  // Safety limit reached — warn and return what we have, no nextCursor
+  logger.warn(
+    `Pagination limit reached (${MAX_PAGINATION_PAGES} pages, ${allRows.length} items). ` +
+      "Results may be incomplete."
+  );
   return { data: allRows.slice(0, limit) };
 }
 

src/lib/api/infrastructure.ts

@@ -27,14 +27,14 @@ import {
 } from "../db/project-cache.js";
 import { getCachedOrganizations } from "../db/regions.js";
 import { type AuthGuardSuccess, withAuthGuard } from "../errors.js";
-import { logger } from "../logger.js";
 import { getApiBaseUrl } from "../sentry-client.js";
 import { buildProjectUrl } from "../sentry-urls.js";
 import { isAllDigits } from "../utils.js";
 
 import {
   API_MAX_PER_PAGE,
   apiRequestToRegion,
+  autoPaginate,
   getOrgSdkConfig,
   MAX_PAGINATION_PAGES,
   ORG_FANOUT_CONCURRENCY,
@@ -54,38 +54,23 @@ import { getUserRegions, listOrganizations } from "./organizations.js";
  */
 export async function listProjects(orgSlug: string): Promise<SentryProject[]> {
   const config = await getOrgSdkConfig(orgSlug);
-  const allResults: SentryProject[] = [];
-  let cursor: string | undefined;
 
-  for (let page = 0; page < MAX_PAGINATION_PAGES; page++) {
+  const { data: allResults } = await autoPaginate(async (cursor) => {
     const result = await listAnOrganization_sProjects({
       ...config,
       path: { organization_id_or_slug: orgSlug },
-      // per_page is supported by Sentry's pagination framework at runtime
-      // but not yet in the OpenAPI spec
-      query: { cursor, per_page: API_MAX_PER_PAGE } as { cursor?: string },
+      query: { cursor, per_page: API_MAX_PER_PAGE } as {
+        cursor?: string;
+        per_page?: number;
+      },
     });
-
-    const { data, nextCursor } = unwrapPaginatedResult<SentryProject[]>(
+    return unwrapPaginatedResult<SentryProject[]>(
       result as
         | { data: SentryProject[]; error: undefined }
         | { data: undefined; error: unknown },
       "Failed to list projects"
     );
-    allResults.push(...data);
-
-    if (!nextCursor) {
-      break;
-    }
-    cursor = nextCursor;
-
-    if (page === MAX_PAGINATION_PAGES - 1) {
-      logger.warn(
-        `Pagination limit reached (${MAX_PAGINATION_PAGES} pages, ${allResults.length} items). ` +
-          "Results may be incomplete for this organization."
-      );
-    }
-  }
+  }, MAX_PAGINATION_PAGES * API_MAX_PER_PAGE);
 
   // Populate project cache for shell completions (best-effort).
   // Mirrors how listOrganizations() calls setOrgRegions().
@@ -121,7 +106,7 @@ export async function listProjectsPaginated(
     query: {
       cursor: options.cursor,
       per_page: options.perPage ?? API_MAX_PER_PAGE,
-    } as { cursor?: string },
+    } as { cursor?: string; per_page?: number },
   });
 
   return unwrapPaginatedResult<SentryProject[]>(

src/lib/api/projects.ts

@@ -84,7 +84,17 @@ export async function listReleasesPaginated(
       environment: options.environment,
       statsPeriod: options.statsPeriod,
       status: options.status,
-    } as { cursor?: string },
+    } as {
+      cursor?: string;
+      per_page?: number;
+      query?: string;
+      sort?: string;
+      health?: number;
+      project?: number[];
+      environment?: string[];
+      statsPeriod?: string;
+      status?: string;
+    },
   });
 
   return unwrapPaginatedResult<SentryRelease[]>(

src/lib/api/releases.ts

@@ -12,6 +12,7 @@ import { logger } from "../logger.js";
 
 import {
   API_MAX_PER_PAGE,
+  autoPaginate,
   getOrgSdkConfig,
   MAX_PAGINATION_PAGES,
   type PaginatedResponse,
@@ -59,12 +60,10 @@ export async function listRepositoriesPaginated(
   const result = await listAnOrganization_sRepositories({
     ...config,
     path: { organization_id_or_slug: orgSlug },
-    // per_page is supported by Sentry's pagination framework at runtime
-    // but not yet in the OpenAPI spec
     query: {
       cursor: options.cursor,
       per_page: options.perPage ?? 25,
-    } as { cursor?: string },
+    } as { cursor?: string; per_page?: number },
   });
 
   return unwrapPaginatedResult<SentryRepository[]>(
@@ -79,33 +78,24 @@ export async function listRepositoriesPaginated(
  * List **all** repositories in an organization by walking every page.
  *
  * Used by the offline repo cache and anywhere else we need the complete
- * set (not just the first page). Stops at {@link MAX_PAGINATION_PAGES}
- * as a safety net for pathological cases.
+ * set (not just the first page). Bounded by `autoPaginate`'s
+ * {@link MAX_PAGINATION_PAGES} safety limit.
  *
  * @param orgSlug - Organization slug
  * @returns All Sentry-registered repositories across all pages
  */
 export async function listAllRepositories(
   orgSlug: string
 ): Promise<SentryRepository[]> {
-  const all: SentryRepository[] = [];
-  let cursor: string | undefined;
-  for (let page = 0; page < MAX_PAGINATION_PAGES; page++) {
-    const { data, nextCursor } = await listRepositoriesPaginated(orgSlug, {
-      cursor,
-      perPage: API_MAX_PER_PAGE,
-    });
-    all.push(...data);
-    if (!nextCursor) {
-      return all;
-    }
-    cursor = nextCursor;
-  }
-  log.warn(
-    `Stopped paginating repositories for '${orgSlug}' after ${MAX_PAGINATION_PAGES} pages — ` +
-      "some repos may be missing from the cache."
+  const { data } = await autoPaginate(
+    (cursor) =>
+      listRepositoriesPaginated(orgSlug, {
+        cursor,
+        perPage: API_MAX_PER_PAGE,
+      }),
+    MAX_PAGINATION_PAGES * API_MAX_PER_PAGE
   );
-  return all;
+  return data;
 }
 
 /**

src/lib/api/repositories.ts

@@ -57,20 +57,13 @@ export async function listTeamsPaginated(
   const result = await listAnOrganization_sTeams({
     ...config,
     path: { organization_id_or_slug: orgSlug },
-    // per_page is supported by Sentry's pagination framework at runtime
-    // but not yet in the OpenAPI spec
     query: {
       cursor: options.cursor,
       per_page: options.perPage ?? 25,
-    } as { cursor?: string },
+    } as { cursor?: string; per_page?: number },
   });
 
-  return unwrapPaginatedResult<SentryTeam[]>(
-    result as
-      | { data: SentryTeam[]; error: undefined }
-      | { data: undefined; error: unknown },
-    "Failed to list teams"
-  );
+  return unwrapPaginatedResult<SentryTeam[]>(result, "Failed to list teams");
 }
 
 /**