Skip to content

LLM-25262: file deletion patch processing - #58

Merged
dtim merged 1 commit into
mainfrom
dtim/LLM-25262-delete-file-patch-processing
Mar 16, 2026
Merged

LLM-25262: file deletion patch processing#58
dtim merged 1 commit into
mainfrom
dtim/LLM-25262-delete-file-patch-processing

Conversation

@dtim

@dtim dtim commented Mar 10, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@dtim
dtim force-pushed the dtim/LLM-25262-delete-file-patch-processing branch from 6263d9e to 4785f42 Compare March 10, 2026 17:16
@dtim
dtim requested review from nizienko and slapoguzov March 10, 2026 17:18
@slapoguzov
slapoguzov requested review from KirillAlekhnovich and removed request for nizienko March 13, 2026 14:36
Codex can delete the file after the patch has been
created. To avoid accessing non-existing files,
file update changes with the kind 'delete' are
processed as follows.
- If the file still exists, its old content
  is read directly from the file. The new content
  is always set to an empty string (it's consistent
  with the IntelliJ Idea ACP client handling
  of file deletion).
- If the file can't be read, the old content
  is restored from the patch itself.
@dtim
dtim force-pushed the dtim/LLM-25262-delete-file-patch-processing branch from 4785f42 to 9a74246 Compare March 13, 2026 15:59
@dtim
dtim merged commit 5b3a9ac into main Mar 16, 2026
3 checks passed
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