Skip to content

fix(discord-bot): stop treating Sentry short ids as Jira keys - #415

Merged
patroza merged 2 commits into
fork/devfrom
t3code/fix-sentry-link-detection
Aug 20, 2026
Merged

patroza merged 2 commits into
fork/devfrom
t3code/fix-sentry-link-detection

Conversation

@patroza

@patroza patroza commented Aug 20, 2026

Copy link
Copy Markdown
Owner

Why

Sentry short ids (SCANNER-313) have the same shape as Jira keys (PROJ-123). The Discord bot extracted those tokens from sentry.io URLs and Sentry alert embeds, pinned them under Jira, and injected jira: SCANNER-313 into agent turns — so the agent treated an obvious Sentry issue as a Jira ticket.

What

  • Do not treat sentry.io URLs, Sentry-bot authors, or Sentry Discord embeds as Jira. Atlassian browse / selectedIssue URLs are still extracted.
  • Drop already-stored false positives on pin refresh and backfill (and skip mining our own Omegent Info pin, which echoed the misclassified key).
  • Persist sentry.io issue URLs and render them as Sentry on the thread-info pin (above Jira).

Test plan

  • vp test run jiraLinks, sentryLinks, threadInfoPin, ThreadLinkStore, threadContext, MentionRouter
  • apps/discord-bot typecheck
  • After deploy: Sentry alert / pasted https://*.sentry.io/issues/SCANNER-313 should pin Sentry, not Jira

opened by Patrick Roza in chat thread Discord · what happened here?

Sentry short ids (SCANNER-313) match the Jira PROJ-123 shape, so the
thread-info pin labelled sentry.io issues as Jira and injected
jira: SCANNER-313 into agent turns.

Skip sentry.io URLs, Sentry-bot authors, and Sentry embeds when
extracting Jira keys; drop already-stored false positives on pin
refresh/backfill; persist sentry issue URLs and render them under
Sentry on the pin.
Label Sentry Discord embeds with the footer short id (SCANNER-313)
instead of the numeric issue path, pin them under **Sentry**, and
re-inject the URLs as `sentry:` on later turns so they are not treated
as Jira keys.
@patroza
patroza marked this pull request as ready for review August 20, 2026 08:51
@patroza
patroza merged commit 9b4a23a into fork/dev Aug 20, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant