Skip to content

fix(search-events): Gate Seer translation on a rollout flag - #1447

Merged
DominikB2014 merged 2 commits into
mainfrom
fix/search-events-drop-seer-translate-flag
Oct 9, 2026
Merged

DominikB2014 merged 2 commits into
mainfrom
fix/search-events-drop-seer-translate-flag

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

gen-ai-search-agent-translate was removed after reaching GA, so the MCP server can no longer rely on it to determine access to Seer search-agent translation. This PR gates Seer use in search_events on a new mcp-search-events-seer-translate organization flag instead, allowing a gradual rollout. AI-feature opt-outs and the existing fallback to the embedded agent remain in place.

via @DominikB2014.

--

View Junior Session [Sentry]

Co-Authored-By: Dominik Buszowiecki <dominik.buszowiecki@sentry.io>
@DominikB2014
DominikB2014 requested a review from a team October 8, 2026 14:25
@DominikB2014
DominikB2014 marked this pull request as ready for review October 8, 2026 14:26
@github-actions github-actions Bot added the risk: medium PR risk score: medium label Oct 8, 2026
The old gen-ai-search-agent-translate flag was removed from Sentry, so
search_events never used Seer. Check the new
mcp-search-events-seer-translate flag instead so Seer translation can be
rolled out gradually rather than at 100%.

Co-Authored-By: Dominik Buszowiecki <44422760+DominikB2014@users.noreply.github.com>
@sentry-junior sentry-junior Bot changed the title fix(search-events): Stop gating Seer on removed translate flag fix(search-events): Gate Seer translation on a rollout flag Oct 9, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit a133931. Configure here.

Comment thread packages/mcp-core/src/tools/support/search-events/seer.ts
DominikB2014 added a commit to getsentry/sentry that referenced this pull request Oct 9, 2026
Registers `organizations:mcp-search-events-seer-translate` so the MCP
`search_events` tool can roll out Seer search agent translation
gradually. The last time this went to 100% we saw 502s, so this time it
starts at 10%.

It uses `api_expose=True` because the MCP server checks it in the
organization's `features` list (`include_feature_flags=1`), not through
`features.has`.

The rollout config is in getsentry/sentry-options-automator, and the MCP
change that reads the flag is getsentry/toolkit#1447.

<!-- junior-request-attribution:start -->
via @DominikB2014.
<!-- junior-request-attribution:end -->

<!-- junior-session-footer:start -->
<!-- junior-conversation-id:slack%3AD0BAH48EJL8%3A1791563344.182649 -->

--

[View Junior
Session](https://junior-prod.sentry.dev/conversations/slack%3AD0BAH48EJL8%3A1791563344.182649)
[[Sentry]](https://sentry.sentry.io/explore/conversations/slack%3AD0BAH48EJL8%3A1791563344.182649/?project=4510944073809921)

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

Co-authored-by: sentry-junior[bot] <264270552+sentry-junior[bot]@users.noreply.github.com>
Co-authored-by: Dominik Buszowiecki <44422760+DominikB2014@users.noreply.github.com>
@DominikB2014
DominikB2014 merged commit 5badeb6 into main Oct 9, 2026
34 checks passed
@DominikB2014
DominikB2014 deleted the fix/search-events-drop-seer-translate-flag branch October 9, 2026 19:01

This branch was successfully deployed

1 active deployment
Actions — a133931a Deployed Oct 9, 2026 by sentry-junior[bot] via eval #1284
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.

2 participants