Skip to content

Conversation

@benbrandt
Copy link
Member

@benbrandt benbrandt commented Aug 26, 2025

Previously, we wouldn't finalize the diff if an error occurred during editing or the tool call was canceled.

Release Notes:

  • N/A

There were cases before where we would exit after creating the diff and
wouldn't finalize it
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Aug 26, 2025
@benbrandt benbrandt changed the title agent2: always finalize diffs from the edit tool agent2: Always finalize diffs from the edit tool Aug 26, 2025
@as-cii as-cii enabled auto-merge (squash) August 26, 2025 09:19
@as-cii as-cii merged commit b249593 into main Aug 26, 2025
21 checks passed
@as-cii as-cii deleted the finalize-diffs branch August 26, 2025 09:46
JosephTLyons pushed a commit that referenced this pull request Aug 26, 2025
Previously, we wouldn't finalize the diff if an error occurred during
editing or the tool call was canceled.

Release Notes:

- N/A

---------

Co-authored-by: Antonio Scandurra <[email protected]>
tidely pushed a commit to tidely/zed that referenced this pull request Sep 10, 2025
Previously, we wouldn't finalize the diff if an error occurred during
editing or the tool call was canceled.

Release Notes:

- N/A

---------

Co-authored-by: Antonio Scandurra <[email protected]>
FrGoIs pushed a commit to FrGoIs/zed that referenced this pull request Sep 29, 2025
Previously, we wouldn't finalize the diff if an error occurred during
editing or the tool call was canceled.

Release Notes:

- N/A

---------

Co-authored-by: Antonio Scandurra <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants