-
Notifications
You must be signed in to change notification settings - Fork 698
feat(amazonq): Enable users to edit code files directly on the diff view #7886
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: feature/editable_diffview
Are you sure you want to change the base?
feat(amazonq): Enable users to edit code files directly on the diff view #7886
Conversation
⏳ I'm reviewing this pull request for security vulnerabilities and code quality issues. I'll provide an update when I'm done |
|
✅ I finished the code review, and didn't find any security or code quality issues. |
What happens when user clicks the diff view again after editing and closing it? |
const originalContent = params.originalFileContent ?? '' | ||
|
||
// Clean up any existing temp files first | ||
for (const tempFile of tempDiffFiles) { |
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.
we tried using a temp file before, but this causes issues like the temp file now showing up in recent files in the IDE
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.
This file is only created at OS/tmp and cleaned up immediately after the diff view is closed. It will not show up at the VSC's workspace file list or work tree!
5dfa590
to
c722999
Compare
Problem
Currently users couldn't edit code files on the diff view provided by Amazon Q. Users have to open those files in the workspace to edit them. This is very inconvenient.
Solution
Enable users to edit code files directly on the diff view.
Screen.Recording.2025-08-13.at.2.16.11.PM.mov
feature/x
branches will not be squash-merged at release time.