feat(trace): add cursor pagination to trace list

[BYK]

Summary

sentry trace list previously had no pagination — --limit was the only way to get more results, with no way to continue to the next page.

This PR adds --cursor / -c pagination and extracts shared infrastructure from duplicated code in issue list.

Usage

sentry trace list my-org/my-project           # first page
sentry trace list my-org/my-project -c last   # next page
sentry trace list my-org/my-project -c last --sort duration  # preserves flags

Changes

Commit 1: feat(trace) — Add cursor pagination

src/lib/api-client.ts

• listTransactions() returns PaginatedResponse<TransactionListItem[]> instead of a raw array
• Forwards cursor to the Sentry events API; extracts next cursor from the Link header

src/commands/trace/list.ts

• New --cursor / -c flag: "last" resumes from stored cursor; bare integers rejected early
• Cursor storage via composite context key (host + org/project + sort + query)
• JSON output: { data, nextCursor?, hasMore } envelope (consistent with issue list)
• Human output: Next page: sentry trace list <target> -c last footer when more exist

Commit 2: refactor — Extract shared pagination infrastructure

src/lib/db/pagination.ts — new buildPaginationContextKey(type, scope, params?)

• Generalizes the host:…|type:…[|key:value]* pattern hand-rolled in 3 places
• buildOrgContextKey now delegates to it (backward compatible)

src/lib/list-command.ts — new parseCursorFlag()

• Extracts the digits-rejecting cursor parser duplicated in issue list and trace list
• LIST_CURSOR_FLAG updated to use it — all commands get cursor validation for free

src/commands/trace/list.ts — uses LIST_CURSOR_FLAG and buildPaginationContextKey directly

src/commands/issue/list.ts — uses parseCursorFlag and buildPaginationContextKey; removes local ALL_DIGITS_RE and manual context key construction in handleOrgAllIssues

[github-actions]

Semver Impact of This PR

🟡 Minor (new features)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

---

New Features ✨

Trace

• Add cursor pagination to trace list by BYK in #324

• Add sentry trace logs subcommand (Add CLI support for fetching logs associated with a trace #247) by BYK in #311

Other

• (api) Add --data/-d flag and auto-detect JSON body in fields by BYK in #320
• (formatters) Render all terminal output as markdown by BYK in #297
• (issue-list) Global limit with fair distribution, compound cursor, and richer progress by BYK in #306

Bug Fixes 🐛

Api

• Use numeric project ID to avoid "not actively selected" error by betegon in #312
• Use limit param for issues endpoint page size by BYK in #309
• Auto-correct ':' to '=' in --field values with a warning by BYK in #302

Formatters

• Expand streaming table to fill terminal width by betegon in #314
• Fix HTML entities and escaped underscores in table output by betegon in #313

Other

• (ci) Generate JUnit XML to silence codecov-action warnings by BYK in #300
• (nightly) Push to GHCR from artifacts dir so layer titles are bare filenames by BYK in #301
• (region) Resolve DSN org prefix at resolution layer by BYK in #316
• (test) Handle 0/-0 in getComparator anti-symmetry property test by BYK in #308
• (trace-logs) Timestamp_precise is a number, not a string by BYK in #323

Internal Changes 🔧

Api

• Upgrade @sentry/api to 0.21.0, remove raw HTTP pagination workarounds by BYK in #321
• Wire listIssuesPaginated through @sentry/api SDK for type safety by BYK in #310

---

_{🤖 This preview updates automatically when you update the PR.}

[github-actions]

Codecov Results 📊

✅ 2431 passed | Total: 2431 | Pass Rate: 100% | Execution Time: 0ms

📊 Comparison with Base Branch

Metric	Change
Total Tests	📈 +23
Passed Tests	📈 +23
Failed Tests	—
Skipped Tests	—

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 3137 uncovered lines.
✅ Project coverage is 81.23%. Comparing base (base) to head (head).

Files with missing lines (3)

File	Patch %	Lines
api-client.ts	77.47%	⚠️ 201 Missing
list.ts	86.97%	⚠️ 96 Missing
list.ts	99.26%	⚠️ 1 Missing

Coverage diff

@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    80.84%    81.23%    +0.39%
==========================================
  Files          121       121         —
  Lines        16662     16711       +49
  Branches         0         0         —
==========================================
+ Hits         13470     13574      +104
- Misses        3192      3137       -55
- Partials         0         0         —

---

Generated by Codecov Action

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

[BYK]

Good catch on the escaping inconsistency — fixed in the latest commit by escaping sort in buildMultiTargetContextKey via escapeContextKeyValue for consistency with buildPaginationContextKey.

On the incomplete refactoring: buildMultiTargetContextKey is intentionally separate. It handles multiple targets (fan-out across orgs/projects), encodes them as a sorted comma-joined fingerprint, and uses the type:multi: discriminant — a fundamentally different shape than the single-scope buildPaginationContextKey(type, scope, params?) signature. Delegating to it would require adding a multi-target path to the shared helper, which would mix two distinct concerns. The PR description was slightly imprecise about this.