-
Notifications
You must be signed in to change notification settings - Fork 3.6k
feat: remove auto-accept edits setting #8310
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No issues found across 4 files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@uinstinct looks like just pending tests! GUI edit tests account for policies
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@uinstinct instead of hiding edit tools from policies, can we just get rid of the auto-accept agent edits setting entirely?
Description
Automatically accept all edit file tools (edit existing file, single find and replace, multi-edit) because accept/reject file diffs will be shown for them anyway.
resolves CON-4427
AI Code Review
@continue-review
Checklist
Screen recording or screenshot
[ When applicable, please include a short screen recording or screenshot - this makes it much easier for us as contributors to review and understand your changes. See this PR as a good example. ]
Tests
[ What tests were added or updated to ensure the changes work as expected? ]
Summary by cubic
Automatically auto-approve file edit tools and remove them from policy controls. Users still see diffs to accept or reject. Resolves CON-4427.