Skip to content

fix(seer): Linkify short IDs with multi-hyphen project slugs#113431

Merged
JoshFerge merged 1 commit intomasterfrom
joshferge/fix/seer-explorer-multi-hyphen-short-id-links
Apr 20, 2026
Merged

fix(seer): Linkify short IDs with multi-hyphen project slugs#113431
JoshFerge merged 1 commit intomasterfrom
joshferge/fix/seer-explorer-multi-hyphen-short-id-links

Conversation

@JoshFerge
Copy link
Copy Markdown
Member

@JoshFerge JoshFerge commented Apr 20, 2026

Fix Seer explorer issue-short-ID linkification so IDs with multi-hyphen project slugs (e.g. FRONTEND-REACT-59A, BACKEND-FLASK-F2, BACKEND-RUBY-ON-RAILS-58) produce correct links.

The regex in linkifyIssueShortIds only allowed a single -[A-Z0-9]+ suffix, so anything after the second hyphen was dropped from the link. FRONTEND-REACT-59A was rendered as a link to /issues/FRONTEND-REACT/ with a dangling -59A in the text — broken navigation from Seer responses.

Allow one or more trailing -[A-Z0-9]+ segments so the full short ID is captured. Existing short IDs (PROJECT-1, JAVASCRIPT-2SDJ, INTERNAL-4K) continue to match.

Surfaced while demoing the Seer agent — links in agent output were pointing at the wrong (or non-existent) issues. Adds utils.spec.tsx covering single- and multi-hyphen IDs plus the existing code-block / markdown-link / URL exclusion paths.

Agent transcript: https://claudescope.sentry.dev/share/RxwksDlKS0reKAGgdnK0yVb--9ijSdmRNb5bmM29wYo

The Seer explorer post-processes LLM markdown to turn issue short IDs
like PROJECT-1 into clickable links. The regex only allowed a single
hyphen-suffix, so multi-hyphen slugs such as FRONTEND-REACT-59A or
BACKEND-RUBY-ON-RAILS-58 were truncated to FRONTEND-REACT /
BACKEND-RUBY-ON-RAILS, producing broken links to the wrong issue.

Allow one or more trailing `-[A-Z0-9]+` segments so the full short ID
is captured. Adds tests covering single- and multi-hyphen IDs,
lowercase rejection, and the existing code-block / markdown-link /
URL exclusion paths.

Co-Authored-By: Claude <noreply@anthropic.com>

Agent transcript: https://claudescope.sentry.dev/share/K6uQSc6tQK_7ERCo6j8hcBt7uHdP0arYesqC5ltNJDE
@JoshFerge JoshFerge requested a review from a team as a code owner April 20, 2026 14:20
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 20, 2026
Copy link
Copy Markdown
Contributor

@chromy chromy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@JoshFerge JoshFerge merged commit 43c9057 into master Apr 20, 2026
67 checks passed
@JoshFerge JoshFerge deleted the joshferge/fix/seer-explorer-multi-hyphen-short-id-links branch April 20, 2026 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants