feat(flags): Remove graduated team-replay-2 feature flags#108812
feat(flags): Remove graduated team-replay-2 feature flags#108812
Conversation
github-actions
bot
added
Scope: Frontend
|
🚨 Warning: This pull request contains Frontend and Backend changes! It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently. Have questions? Please ask in the |
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
…r-release, sentry-toolbar-ui) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Remove all features.has() checks and @with_feature decorators for flags that are now always enabled: integrations-external-projects-async-lookup, integrations-github-project-management, migrate-azure-devops-integration, resolve-in-semver-release, sentry-toolbar-ui, service-hooks-outbox, expanded-sentry-apps-webhooks, sentry-app-manual-token-refresh. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
wedamija
force-pushed
the
flag-burner/ga/team-replay-2
branch
from
7886038 to
fafa4f9
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
| ): | ||
| self._update_service_hooks_via_outbox() | ||
| else: | ||
| self._update_service_hooks_via_task() |
There was a problem hiding this comment.
Service hook updates may time out
Medium Severity
SentryAppUpdater._update_service_hooks now always uses the outbox path, which synchronously iterates all SentryAppInstallations and .save()s one ControlOutbox per installation. For widely-installed/published apps, this can significantly increase request latency and reintroduce timeout risk that the removed task-based path previously avoided.
These flags belong to the ecosystem PR (#108813), not team-replay-2: - integrations-external-projects-async-lookup - integrations-github-project-management - migrate-azure-devops-integration - service-hooks-outbox - expanded-sentry-apps-webhooks - sentry-app-manual-token-refresh Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2 participants
Summary
Removes 2 feature flags owned by team-replay that have been enabled at 100% via flagpole with no conditions:
organizations:resolve-in-semver-releaseorganizations:sentry-toolbar-uiThese flags have been fully rolled out and their behavior is now the default.
Test plan
🤖 Generated with Claude Code