Skip to content

fix(seer): Decouple create-pr setting from stopping point#112646

Merged
ryan953 merged 3 commits intomasterfrom
ryan953/fix-decouple-createpr-stoppingpoint
Apr 10, 2026
Merged

fix(seer): Decouple create-pr setting from stopping point#112646
ryan953 merged 3 commits intomasterfrom
ryan953/fix-decouple-createpr-stoppingpoint

Conversation

@ryan953
Copy link
Copy Markdown
Member

@ryan953 ryan953 commented Apr 10, 2026

This decouples the Create PR setting from the Stopping Point value, and adds some optimistic updates to make the ui feel snappier.

This hook applies to the Autofix Project details page only, no other imports.

@ryan953 ryan953 requested review from a team as code owners April 10, 2026 04:13
@ryan953 ryan953 requested a review from billyvg April 10, 2026 04:13
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 10, 2026
const tuning = stoppingPoint === 'off' ? ('off' as const) : ('medium' as const);
ProjectsStore.onUpdateSuccess({...project, autofixAutomationTuning: tuning});

if (stoppingPoint !== 'off' && previousPreference?.preference) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why do we not update when it's off?

Copy link
Copy Markdown
Member Author

@ryan953 ryan953 Apr 10, 2026

Choose a reason for hiding this comment

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

added a comment to answer

TL/DR I hate these api endpoints... can't wait until they're changed

optionall, nullable, nested objects, but also things spread across ProjectOptions and the SeerPreferences model. fml

@ryan953 ryan953 merged commit 5afae00 into master Apr 10, 2026
65 checks passed
@ryan953 ryan953 deleted the ryan953/fix-decouple-createpr-stoppingpoint branch April 10, 2026 20:24
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.

2 participants