feat(autofix): Add retry buttons to Autofix v3 step cards#112290
Draft
isaacwang-sentry wants to merge 17 commits intomasterfrom
Draft
feat(autofix): Add retry buttons to Autofix v3 step cards#112290isaacwang-sentry wants to merge 17 commits intomasterfrom
isaacwang-sentry wants to merge 17 commits intomasterfrom
Conversation
Add a new Flagpole feature flag to gate "Retry from step" buttons on Autofix v3 cards. This is primarily useful for local Seer testing, allowing developers to re-run individual autofix steps without restarting the entire flow. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
github-actions
bot
added
the
Scope: Frontend
isaacwang-sentry
force-pushed
the
isaacwang-sentry/feat/autofix-retry-from-step-ui
branch
from
cab7212 to
e7f6ee2
Compare
Pass insert_index from the API serializer through trigger_autofix_explorer to client.continue_run. When provided, Seer truncates blocks after that index, enabling retry-from-step to properly reset downstream steps. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
isaacwang-sentry
force-pushed
the
isaacwang-sentry/feat/autofix-retry-from-step-ui
branch
from
e7f6ee2 to
0eca46e
Compare
The retry buttons are no longer gated behind a feature flag, so this registration is unnecessary. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
isaacwang-sentry
force-pushed
the
isaacwang-sentry/feat/autofix-retry-from-step-ui
branch
from
6c067db to
183a444
Compare
The test_stopping_point test checks exact kwargs passed to trigger_autofix_explorer. Add the new insert_index=None parameter to match the updated function signature. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
isaacwang-sentry
force-pushed
the
isaacwang-sentry/feat/autofix-retry-from-step-ui
branch
from
0c3efba to
194045f
Compare
isaacwang-sentry
force-pushed
the
isaacwang-sentry/feat/autofix-retry-from-step-ui
branch
from
bb98231 to
0750db9
Compare
aliu39
reviewed
Apr 7, 2026
| trailingItems={ | ||
| section.status === 'completed' && artifact?.data ? ( | ||
| <RetryButton | ||
| onClick={() => startStep('root_cause', runId, undefined, section.blockIndex)} |
Member
There was a problem hiding this comment.
right now it seems like "restart from scratch" calls startStep with runId=undefined, which creates a fresh run state that's then picked up by group_ai_autofix endpoint. This could be intentional so we have the record of the previous run for debugging. Could we do that here?
Suggested change
| onClick={() => startStep('root_cause', runId, undefined, section.blockIndex)} | |
| onClick={() => startStep('root_cause')} |
frontend behavior should look the same since blockIndex is 0. Thoughts on removing the header button now that we have these?
Verify that insert_index from the POST request is threaded through to trigger_autofix_explorer for retry-from-step functionality. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add a "Retry" icon button to completed step cards (Plan, Code Changes, Pull Requests) in the v3 Explorer drawer. Clicking the button re-runs that step on the existing run without requiring feedback text. Gated behind the `autofix-retry-from-step` feature flag so it only appears for local Seer testing. RootCauseCard is excluded since the header's "Start Over" button already covers that case. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
When retrying from a step, pass the section's block index as insert_index so Seer truncates all blocks after that point. This ensures downstream sections (e.g. Code Changes after retrying Plan) are removed and the normal NextStep flow resumes after the retried step completes. Changes: - Add blockIndex to AutofixSection, tracked in getOrderedAutofixSections - Add insertIndex parameter to startStep - Pass section.blockIndex in retry button handlers Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Retry buttons are now always visible on all completed step cards including Root Cause. The feature flag check has been removed since this is intended as a standard feature, not gated functionality. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Only show the synthetic PR and coding-agent sections when a completed code_changes section exists. After retrying an earlier step, blocks are truncated but repo_pr_states persists on the run state, causing the PR card to remain visible and preventing NextStep buttons from appearing. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- NextStep now skips synthetic sections (pull_request, coding_agents) when finding the last section, so correct next-step buttons appear even when stale PR data persists - CodeChangesNextStep shows "Yes, update the PR" when a PR already exists, "Yes, draft a PR" otherwise - PullRequestsCard shows "Update" button when code has changed since the PR was last pushed, "View" link when in sync Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The PR card now only appears when all PRs match the current code changes (commit SHA comparison). This prevents the PR card from showing above the "Are you happy with these code changes?" prompt. - Added areAllPRsInSync() helper for commit SHA comparison - PR section hidden when code is out of sync or no PR exists yet - CodeChangesNextStep returns null when all PRs are in sync Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
When the user clicks "Yes, draft/update a PR", immediately show a "Creating PR..." or "Updating PR..." message instead of leaving the NextStep prompt visible during the backend roundtrip. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
When user clicks to create/update PR, immediately hide the NextStep prompt and show the PR card with a disabled "Creating/Updating PR" button. The button text distinguishes between new PRs and updates. The PR section now also appears during the 'creating' status, not just when fully in sync. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
When PR creation fails, show a clickable "Retry PR" button instead of a disabled "Failed to create PR" button. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add missing blockIndex property to makeSection helpers in test files and pr_commit_shas to the autofixSection PR test to match the new sync-aware PR section logic. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Remove the no-op refresh button from PullRequestsCard header and the dead isOutOfSync/Update code. The error state now shows "Failed to create PR — Retry" as a clickable button. Closed PRs are handled by the normal rerun flow (Seer reopens them automatically). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Show the PR section when any PR has an error status, so the "Failed to create PR — Retry" button is visible. Stale errors after retrying earlier steps are cleared on the Seer side. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
isaacwang-sentry
force-pushed
the
isaacwang-sentry/feat/autofix-retry-from-step-ui
branch
from
4e806e5 to
292ccf9
Compare
isaacwang-sentry
added a commit
that referenced
this pull request
Apr 7, 2026
#112287) Thread `insert_index` through the explorer autofix API so the frontend can request a retry-from-step that truncates all downstream blocks. When provided, Seer does `blocks = blocks[:insert_index]` before re-running the step, ensuring downstream sections (e.g. Code Changes after retrying Plan) are properly removed. Changes: - Add `insert_index` field to `ExplorerAutofixRequestSerializer` - Pass it through `trigger_autofix_explorer()` → `client.continue_run()` A follow-up frontend PR (#112290) adds retry buttons that use this parameter. --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Base automatically changed from
isaacwang-sentry/feat/autofix-retry-from-step-flag
to
master
April 7, 2026 22:37
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
2 participants
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.
Add "Retry" icon buttons to completed step cards (Plan, Code Changes, Pull Requests) in
the Autofix v3 Explorer drawer. Clicking the button re-runs that step on the existing run
without requiring feedback text — useful for iterating on Seer changes locally.
Gated behind the
autofix-retry-from-stepfeature flag registered in #112287.RootCauseCard is excluded since the header "Start Over" button already covers that case.
Depends on #112287