test: simplify numeric issue ID regression coverage

Reuse the shared event fixtures, replace the numeric autofix handler list with one route, and shorten the API guidance.

Co-Authored-By: David Cramer <david@sentry.io>

Author: sentry-junior[bot]

docs/contributing/api-patterns.md

@@ -65,10 +65,8 @@ await api.issues.update({
 
 ### Resolved Issue IDs
 
-After resolving an issue, use `String(issue.id)` for follow-up API requests,
-including events, external links, and Seer analysis. Do not switch back to
-`issue.shortId`: short-ID resolution can fail for legacy mixed-case project
-slugs even when numeric-ID lookup succeeds. Keep short IDs for display.
+Use `String(issue.id)` for API requests after resolving an issue. Short IDs can
+fail to resolve for legacy mixed-case project slugs; keep them for display.
 
 ### Multi-Region Support
 

packages/mcp-core/src/tools/catalog/get-issue-details.numeric-id.test.ts

@@ -19,8 +19,6 @@ const context = {
   userId: "1",
 };
 
-// Short IDs for legacy mixed-case project slugs can fail to resolve even though
-// the numeric ID works. Reject every short-ID follow-up, including optional data.
 describe("issue details with an unresolvable short ID", () => {
   it.each([
     {
@@ -65,39 +63,26 @@ describe("issue details with an unresolvable short ID", () => {
         ),
     },
   ])("loads $name and enrichment by numeric ID", async ({ call }) => {
-    const requestedIssueIds: string[] = [];
     const base = `https://sentry.io/api/0/organizations/${organizationSlug}`;
     mswServer.use(
       http.get(`${base}/issues/`, () => HttpResponse.json([issueFixture])),
-      http.get(`${base}/issues/:issueId/*`, ({ params, request }) => {
-        const requestedId = String(params.issueId);
-        requestedIssueIds.push(requestedId);
-        if (requestedId !== issueId) {
-          return HttpResponse.json(
-            { detail: "The requested resource does not exist" },
-            { status: 404 },
-          );
-        }
-        const path = new URL(request.url).pathname;
-        if (path.includes("/events/")) {
-          return HttpResponse.json(eventsFixture);
-        }
-        if (path.endsWith("/external-issues/")) {
-          return HttpResponse.json([
-            {
-              id: "123",
-              issueId,
-              serviceType: "github",
-              displayName: "example/app#123",
-              webUrl: "https://github.com/example/app/issues/123",
-            },
-          ]);
-        }
-        if (path.endsWith("/autofix/")) {
-          return HttpResponse.json(autofixStateFixture);
-        }
-        return HttpResponse.json(issueFixture);
-      }),
+      http.get(`${base}/issues/${issueFixture.shortId}/*`, () =>
+        HttpResponse.json({ detail: "Not found" }, { status: 404 }),
+      ),
+      http.get(`${base}/issues/${issueId}/autofix/`, () =>
+        HttpResponse.json(autofixStateFixture),
+      ),
+      http.get(`${base}/issues/${issueId}/external-issues/`, () =>
+        HttpResponse.json([
+          {
+            id: "123",
+            issueId,
+            serviceType: "github",
+            displayName: "example/app#123",
+            webUrl: "https://github.com/example/app/issues/123",
+          },
+        ]),
+      ),
     );
 
     const result = await call();
@@ -106,7 +91,5 @@ describe("issue details with an unresolvable short ID", () => {
     expect(result).toContain(eventsFixture.id);
     expect(result).toContain("## Seer Analysis");
     expect(result).toContain("**example/app#123** (github)");
-    expect(requestedIssueIds.length).toBeGreaterThanOrEqual(3);
-    expect(new Set(requestedIssueIds)).toEqual(new Set([issueId]));
   });
 });

packages/mcp-server-mocks/src/index.ts

@@ -1334,20 +1334,11 @@ export const restHandlers = buildHandlers([
     path: "/api/0/organizations/sentry-mcp-evals/issues/CLOUDFLARE-MCP-41/autofix/",
     fetch: () => HttpResponse.json({ autofix: null }),
   },
-  // Issue details use the resolved numeric ID for enrichment requests.
-  ...[
-    issueFixture.id,
-    "7890123456", // PERF-N1-001
-    "123456", // DEFAULT-001 / CONTEXT-001
-    "123456789", // TEAM-ISSUE-001
-    "6898891101", // MCP-SERVER-EQE
-    "7777777777", // FUTURE-TYPE-001
-    "4256774711", // BLOG-CSP-4XC
-  ].map((issueId) => ({
-    method: "get" as const,
-    path: `/api/0/organizations/sentry-mcp-evals/issues/${issueId}/autofix/`,
+  {
+    method: "get",
+    path: "/api/0/organizations/sentry-mcp-evals/issues/:issueId(\\d+)/autofix/",
     fetch: () => HttpResponse.json({ autofix: null }),
-  })),
+  },
   {
     method: "post",
     path: "/api/0/organizations/sentry-mcp-evals/issues/CLOUDFLARE-MCP-42/autofix/",