Skip to content

fix(tools): use numeric issue IDs after lookup - #1328

Merged
dcramer merged 2 commits into
mainfrom
fix/1327-numeric-issue-followups
Sep 23, 2026
Merged

dcramer merged 2 commits into
mainfrom
fix/1327-numeric-issue-followups

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Use the resolved numeric issue ID for event, external-link, and Seer requests in get_issue_details and get_sentry_resource. This avoids 404s from short-ID resolution for legacy mixed-case project slugs. Displayed short IDs are unchanged.

Regression tests cover issue IDs, URLs, and event lookups when short-ID requests return 404.

Fixes #1327.

via David Cramer.

--

View Junior Session [Sentry]

Keep event, external-link, and Seer lookups on the resolved numeric issue ID so legacy short-ID resolution cannot break a successful lookup.

Co-Authored-By: David Cramer <david@sentry.io>
@dcramer
dcramer marked this pull request as ready for review September 23, 2026 17:52
@github-actions github-actions Bot added the risk: medium PR risk score: medium label Sep 23, 2026
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>
@sentry-junior sentry-junior Bot changed the title fix(tools): use resolved numeric IDs for issue detail requests fix(tools): use numeric issue IDs after lookup Sep 23, 2026
@dcramer
dcramer merged commit 89544ff into main Sep 23, 2026
22 of 23 checks passed
@dcramer
dcramer deleted the fix/1327-numeric-issue-followups branch September 23, 2026 18:35
BYK pushed a commit that referenced this pull request Sep 30, 2026
mr-danya pushed a commit to mr-danya/sentry-mcp that referenced this pull request Oct 6, 2026
mr-danya pushed a commit to mr-danya/sentry-mcp that referenced this pull request Oct 6, 2026

This branch was successfully deployed

1 active deployment
Actions — 7d05d15a Deployed Sep 23, 2026 by sentry-junior[bot] via eval #1126
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: medium PR risk score: medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

get_sentry_resource 404s for issues whose short IDs don't resolve, even when called with a numeric issue ID

1 participant