chore(experiments): remove dead experiment feature flag constants#60606
Merged
Conversation
Remove four experiment-related feature flag constants with no remaining code usage: - EXPERIMENT_INTERVAL_TIMESERIES / experiments-interval-timeseries (no flag in app, unreferenced) - EXPERIMENTS_TEMPLATES / experiments-templates (no flag in app, unreferenced) - AA_TEST_BAYESIAN_NEW / aa-test-bayesian-new (internal A/A harness, unreferenced in code) - AA_TEST_BAYESIAN_LEGACY / aa-test-bayesian-legacy (internal A/A harness, unreferenced in code)
Contributor
|
Reviews (1): Last reviewed commit: "chore(experiments): remove dead experime..." | Re-trigger Greptile |
Contributor
|
Size Change: 0 B Total Size: 80.7 MB ℹ️ View Unchanged
|
Contributor
|
🎭 Playwright report · View test results →
These issues are not necessarily caused by your changes. |
jurajmajerik
approved these changes
May 29, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
Four experiment-related feature flag constants in
constants.tsxare dead: two have no flag in the app and no code references; two (aa-test-bayesian-new/-legacy) are internal A/A-test harness flags never referenced in code.Changes
Remove the four unused constants:
EXPERIMENT_INTERVAL_TIMESERIES(experiments-interval-timeseries) — no flag, no usageEXPERIMENTS_TEMPLATES(experiments-templates) — no flag, no usageAA_TEST_BAYESIAN_NEW(aa-test-bayesian-new) — unreferenced in codeAA_TEST_BAYESIAN_LEGACY(aa-test-bayesian-legacy) — unreferenced in codeCode-only cleanup. The two
aa-test-bayesian-*flags remain live in the app (still attached to experiments); deleting them there is intentionally out of scope.How did you test this code?
Agent-authored. Verified no remaining constant or slug references in
.ts/.tsx/.py(only inert recorded payloads in error-tracking__mocks__fixtures). No automated tests run.🤖 Agent context
Authored by Claude Code. Flag states were cross-checked against the live project via the PostHog MCP before removal: the two missing slugs don't exist in the project, and the two A/A flags are unreferenced in code, making all four safe to drop. Graduating the fully-rolled-out flags and the disabled
experiments-sync-queriesflag were identified but deliberately left for separate PRs.