Skip to content

Conversation

hassoncs
Copy link
Contributor

@hassoncs hassoncs commented Oct 16, 2025

Fix bug where the agent couldn't edit files properly when git diff views were open by ensuring only file:// scheme documents are matched in editor detection.

I'll open this fix for Roo too so the // kilocode_change markers are pending:
RooCodeInc/Roo-Code#8676

@hassoncs hassoncs requested a review from kevinvandijk October 16, 2025 00:22
Copy link

changeset-bot bot commented Oct 16, 2025

🦋 Changeset detected

Latest commit: d93fadf

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
kilo-code Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Chris Hasson added 2 commits October 16, 2025 09:25
Fix bug where the agent couldn't edit files properly when git diff views were open by ensuring only file:// scheme documents are matched in editor detection.
@hassoncs hassoncs force-pushed the fix-edits-dont-apply-again branch from 1ebed8c to bf0dec7 Compare October 16, 2025 16:25
… (#8676)

* fix(editor): prevent file editing issues when git diff views are open

Add scheme checks to ensure only file:// URIs are matched when finding editors,
avoiding issues with git diffs and other schemes. Includes error logging for
failed editor lookups.

* Remove the warnings

* fix(editor): enforce file:// scheme in editor lookups to prevent git diff issues

---------

Co-authored-by: daniel-lxs <ricciodaniel98@gmail.com>
@hassoncs hassoncs force-pushed the fix-edits-dont-apply-again branch from bf0dec7 to d93fadf Compare October 16, 2025 16:26
@hassoncs hassoncs marked this pull request as ready for review October 16, 2025 16:26
@hassoncs hassoncs merged commit d71f1d6 into main Oct 16, 2025
22 of 23 checks passed
@hassoncs hassoncs deleted the fix-edits-dont-apply-again branch October 16, 2025 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants