Skip to content

ref(flags): Remove preprod-size-analysis-pr-comments frontend gate - #129452

Open
cursor[bot] wants to merge 1 commit into
masterfrom
claude-rm-preprod-size-analysis-pr-comments
Open

cursor[bot] wants to merge 1 commit into
masterfrom
claude-rm-preprod-size-analysis-pr-comments

Conversation

@cursor

@cursor cursor Bot commented Nov 1, 2026

Copy link
Copy Markdown
Contributor

Remove the organizations:preprod-size-analysis-pr-comments gate in the mobile builds settings. This flag was rolled out to GA in the automator in #8928 on 8/3/26 (conditions: [], rollout: 100) and hasn't been touched since, so the size-analysis PR comment rules panel now always renders.

This is the frontend half, split from the backend read removal per the frontend/backend deploy split. Same shape as #120633, which removed the sibling preprod-build-distribution-pr-comments gate from this exact file.

Follow-up PRs remove the automator value and then the temporary.py registration, in that order.

Stale flag cleanup; flagged by the feature-flag scanner as ga-ready-to-graduate.

Open in Web View Automation 

The flag has been at rollout 100 for all orgs in the automator since
#8928 (8/3/26), so the size-analysis PR comment rules panel always
renders.

Co-authored-by: charlie.luo <charlie.luo@sentry.io>

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@getsentry/emerge-tools — stale feature flag cleanup for organizations:preprod-size-analysis-pr-comments, 1 of 4.

Merge order:

  1. #129452 (this one) — frontend gate in static/app/views/settings/project/preprod/index.tsx
  2. #129453 — backend reads in src/sentry/preprod/vcs/pr_comments/
  3. getsentry/sentry-options-automator#10416 — the flagpole value
  4. #129454 — the temporary.py registration

Evidence the collapse value is True: the flagpole block is conditions: [] / rollout: 100 and has been since automator#8928 on 8/3/26, its only edit after creation. Shape and wording follow #120633, which removed the sibling preprod-build-distribution-pr-comments gate from this same file.

Verified locally with pnpm run lint:js on the changed file and a full pnpm run typecheck; both clean. There is no index.spec.tsx, so no spec needed updating.

Note getsentry dispatch will fail on all three sentry PRs — it gates on the author having write permission and the author is cursor[bot]. A collaborator needs to add the Trigger: getsentry tests label.

View PR

Open in Web View Automation 

Sent by Cursor Automation: Feature flag cleanup

@cursor
cursor Bot marked this pull request as ready for review November 1, 2026 20:27
@cursor
cursor Bot requested a review from a team as a code owner November 1, 2026 20:27

This branch was successfully deployed

1 active deployment
Preview — 4c95e5e8 Deployed Nov 1, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant