perf(cli): Skip adjacent-event lookups on project event details (#1712)

`@sentry/api` 0.423.0, which #1713 bumped on main, defaults
`collapse=adjacentEvents` on `getOrganizationIssueEvent` and
`getProjectEvent` (getsentry/sentry-api-schema#112 and #113). With it,
Sentry skips the next/previous-event lookups. Nothing in the CLI reads
`nextEventID` or `previousEventID`.

Requests that change:

- CLI `getEvent()` → `projects/{org}/{project}/events/{event_id}/`. This
is the explicit event-ID path in `sentry event view`, and it now sends
the param through the SDK default. #1705 skipped it because the project
endpoint didn't support `collapse` yet. getsentry/sentry#129123 added
that support. Until it deploys, Sentry ignores the unknown value.
- CLI `getLatestEvent()` keeps sending the param, but through the SDK
default now. The explicit `collapse` query #1705 added there is gone,
along with its URL assertion, since that would only test the SDK
default.

A tier test confirms that project event URLs with the param stay
`volatile`.

### Behavior notes

- Once getsentry/sentry#129123 deploys, `sentry event view
<org>/<project> <event-id> --json` will return
`nextEventID`/`previousEventID` as null. Issue latest-event lookups
already behave this way after #1705.
- Response-cache keys for project event URLs change, so existing entries
miss once and then fill again.

<!-- junior-session-footer:start -->
<!-- junior-conversation-id:agent%3Aea5efd697aa911af7986f685a5952441 -->

--

[View Junior Session in
Sentry](https://sentry.sentry.io/explore/conversations/agent%3Aea5efd697aa911af7986f685a5952441/?project=4510944073809921)

<!-- junior-session-footer:end -->

---------

Co-authored-by: sentry-junior[bot] <264270552+sentry-junior[bot]@users.noreply.github.com>
Co-authored-by: BYK <burak.kaya@sentry.io>

Author: sentry-junior[bot]

packages/cli/src/lib/api/events.ts

@@ -6,7 +6,6 @@
 
 import {
   type EventAttachmentDetailsResponse,
-  type GetOrganizationIssueEventData,
   getOrganizationIssueEvent,
   getProjectEvent,
   listOrganizationIssueEvents,
@@ -44,13 +43,6 @@ export async function getLatestEvent(
   issueId: string,
 ): Promise<SentryEvent> {
   const config = await getOrgSdkConfig(orgSlug);
-  // `collapse=adjacentEvents` lets Sentry skip the next/previous-event Snuba
-  // lookups; the CLI never reads `nextEventID` or `previousEventID`. The
-  // generated query type omits `collapse`, so the intersection keeps the
-  // server-supported parameter without weakening the generated client type.
-  const query: NonNullable<GetOrganizationIssueEventData["query"]> & {
-    collapse: string[];
-  } = { collapse: ["adjacentEvents"] };
 
   const result = await getOrganizationIssueEvent({
     ...config,
@@ -59,7 +51,6 @@ export async function getLatestEvent(
       issue_id: issueId,
       event_id: "latest",
     },
-    query,
   });
 
   return unwrapResult<SentryEvent>(result, "Failed to get latest event");

packages/cli/test/lib/api-client.coverage.test.ts

@@ -1006,9 +1006,6 @@ describe("events.ts", () => {
         const req = new Request(input!, init);
         expect(req.url).toContain("/organizations/test-org/issues/");
         expect(req.url).toContain("/events/latest/");
-        expect(new URL(req.url).searchParams.getAll("collapse")).toEqual([
-          "adjacentEvents",
-        ]);
         return new Response(JSON.stringify(event), {
           status: 200,
           headers: { "Content-Type": "application/json" },

packages/toolkit-core/src/response-cache/tiers.test.ts

@@ -34,6 +34,14 @@ describe("classifyUrl", () => {
     ).toBe("volatile");
   });
 
+  it("keeps project events volatile when adjacent events are collapsed", () => {
+    expect(
+      classifyUrl(
+        `${API}/projects/acme/backend/events/deadbeef/?collapse=adjacentEvents`,
+      ),
+    ).toBe("volatile");
+  });
+
   it("reuses full traces for at most 60 seconds, since spans can still arrive", () => {
     const url = `${API}/organizations/acme/trace/${TRACE_ID}/?limit=100`;
     expect(classifyUrl(url)).toBe("volatile");

packages/toolkit-core/src/response-cache/tiers.ts

@@ -49,8 +49,9 @@ const URL_TIER_PATTERNS: readonly [TtlTier, readonly RegExp[]][] = [
       /\/trace-logs\//,
       // Full traces gain spans as they arrive
       /\/trace\/[0-9a-f]{32}\//,
-      // Project event details embed `nextEventID`/`previousEventID` and user
-      // reports, and reprocessing can change them for the same ID
+      // Project event details embed user reports (and `nextEventID`/
+      // `previousEventID` unless collapsed), and reprocessing can change them
+      // for the same ID
       /\/projects\/[^/]+\/[^/]+\/events\/[^/?]+\/?(?:\?|$)/,
     ],
   ],