Skip to content

fix: edit tool error after undo#4923

Closed
AdiY00 wants to merge 5 commits intoanomalyco:devfrom
AdiY00:fix/edit-error-after-revert
Closed

fix: edit tool error after undo#4923
AdiY00 wants to merge 5 commits intoanomalyco:devfrom
AdiY00:fix/edit-error-after-revert

Conversation

@AdiY00
Copy link
Copy Markdown
Contributor

@AdiY00 AdiY00 commented Nov 30, 2025

Prevents unnecessary read/write cycles when editing files after undo / redo.
From edit -> error "modified externally" -> read -> edit (extra read and edit in context + extra time) to just edit.

  • Update FileTime for reverted files in revert() and unrevert() so the edit tool sees them as freshly read rather than externally modified

*Restoring the old read time would be more semantically correct but adds complexity without functional benefit.

AdiY00 and others added 3 commits November 30, 2025 21:07
Prevents unnecessary write -> error "modified externaly" -> read ->
write cycles when editing files after undo.

- Update FileTime for reverted files in `revert()` and `unrevert()` so
the edit tool sees them as freshly read rather than externally modified

*Restoring the old read time would be more "semantically correct" but
adds complexity without functional benefit.
@github-actions
Copy link
Copy Markdown
Contributor

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@github-actions
Copy link
Copy Markdown
Contributor

Closing this pull request because it has had no updates for more than 60 days. If you plan to continue working on it, feel free to reopen or open a new PR.

@github-actions github-actions bot closed this Mar 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants