Skip to content

chore(flags): Remove 5 dead feature flags (batch 3)#108559

Merged
wedamija merged 1 commit intomasterfrom
flag-burner/dead/batch-3
Feb 19, 2026
Merged

chore(flags): Remove 5 dead feature flags (batch 3)#108559
wedamija merged 1 commit intomasterfrom
flag-burner/dead/batch-3

Conversation

@wedamija
Copy link
Member

Summary

Remove unused feature flag registrations identified by the scan_flags management command. These flags have zero code references and no flagpole configuration.

Flags removed:

  • organizations:profiling-flamegraph-always-use-direct-chunks
  • organizations:related-events
  • organizations:release-comparison-performance
  • organizations:reprocessing-v2
  • organizations:transaction-name-mark-scrubbed-as-sanitized

Test plan

  • These flags have no code usage, test references, or flagpole config — removing the registration is the only change needed.

Remove the following unused feature flag registrations:
- organizations:profiling-flamegraph-always-use-direct-chunks
- organizations:related-events
- organizations:release-comparison-performance
- organizations:reprocessing-v2
- organizations:transaction-name-mark-scrubbed-as-sanitized

These flags have zero code references and no flagpole configuration.
Identified by the scan_flags management command.
@wedamija wedamija requested a review from a team February 19, 2026 19:39
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Feb 19, 2026
Copy link
Contributor

@cursor cursor bot left a comment

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.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

manager.add("organizations:transaction-metrics-extraction", OrganizationFeature, FeatureHandlerStrategy.INTERNAL, api_expose=False)
# Enable feature to load explore link for transaction widgets
manager.add("organizations:transaction-widget-deprecation-explore-view", OrganizationFeature, FeatureHandlerStrategy.FLAGPOLE, api_expose=True)
# Mark URL transactions scrubbed by regex patterns as "sanitized".
Copy link
Contributor

Choose a reason for hiding this comment

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

Stale test snapshot still references removed feature flag

Low Severity

The organizations:transaction-name-mark-scrubbed-as-sanitized flag (which had default=True) is removed here, but tests/sentry/relay/snapshots/test_config/test_get_project_config/MONOLITH.pysnap still references it. Since this flag was enabled by default, it was included in the project config snapshot. Removing the registration without updating the snapshot will cause the test_get_project_config test to fail.

Fix in Cursor Fix in Web

@wedamija wedamija merged commit 949a9c4 into master Feb 19, 2026
100 checks passed
@wedamija wedamija deleted the flag-burner/dead/batch-3 branch February 19, 2026 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments