feat(autofix): Stop PR iteration from an @sentry comment - #122477
Draft
alexsohn1126 wants to merge 5 commits into
Draft
feat(autofix): Stop PR iteration from an @sentry comment#122477alexsohn1126 wants to merge 5 commits into
@sentry comment#122477alexsohn1126 wants to merge 5 commits into
Conversation
github-actions
Bot
added
Scope: Frontend
Contributor
|
🚨 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 |
Add an `@sentry stop iterating` command on top-level pull request comments. The command calls the existing `pause_pr_iteration`, which was previously unreachable from production code. The two-word phrase avoids a collision with real feedback such as `@sentry stop putting long comments in the diff`, which still iterates. The command runs the same gates as an iteration: the feature flag, the GitHub provider check, and repo write access. It acknowledges with a 👍 and a comment, and reports a failure with a 😕 when the run has no SeerRun row to mark.
alexsohn1126
force-pushed
the
alexsohn/cw-1709-stop-pr-iteration-from-comment
branch
from
August 24, 2026 16:25
2606843 to
918e502
Compare
alexsohn1126
removed
the
Scope: Frontend
… alexsohn/cw-1709-stop-pr-iteration-from-comment
… alexsohn/cw-1709-stop-pr-iteration-from-comment
Cut the stop confirmation to one sentence. Split the footer onto two lines and drop the review caveat. Also assert the iterate task's unauthorized metric name, which nothing covered.
A blank line closes the markdown block, so the second line rendered outside the sub tag. Wrap each line in its own tag.
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
1 participant
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.
Adds an
@sentry stop iteratingcommand on top-level pull request comments. It calls the existingpause_pr_iteration, which had no production caller until now.Two words, not one:
@sentry stop putting long comments in the diffis real feedback and still iterates.The command runs the same gates as an iteration — feature flag, GitHub provider, repo write access. It acknowledges with a 👍 and a comment, and reports a failure with a 😕 when the run has no
SeerRunrow to mark.Stacked on #122086.
CW-1709