Limit apply diff results to changed ranges

[szgupta]

Description

This fixes the apply-diff/edit-file result payload regression where accepted file edits were reported back to the LLM as full post-edit file contents instead of only the changed ranges plus context.

Root cause: remote apply-diff support switched the result source to accepted editor-buffer contents, but the request-file-edits success path converted those buffers directly into UpdatedFileContext entries with line_range: None. That bypassed the existing changed-line ranges computed by CodeDiffView, so the server received and rendered full files. The fix keeps the editor-buffer source needed for remote correctness, but slices it using FileLocations.lines expanded by the restored 10-line context window.

Changes

• Restores the 10-line apply-diff result context behavior.
• Builds UpdatedFileContext fragments from accepted editor-buffer content using changed ranges when available.
• Preserves the full-file fallback when no changed ranges are present.
• Adds unit tests for ranged results and the no-range fallback.

Validation

• rustup run 1.92.0 cargo fmt --manifest-path /workspace/warp/app/Cargo.toml --check
• MACOSX_DEPLOYMENT_TARGET=10.14 rustup run 1.92.0 cargo test --manifest-path /workspace/warp/Cargo.toml -p warp updated_file_contexts_from_editor_buffers
• MACOSX_DEPLOYMENT_TARGET=10.14 rustup run 1.92.0 cargo clippy --manifest-path /workspace/warp/Cargo.toml -p warp --all-targets -- -D warnings

Co-Authored-By: Oz oz-agent@warp.dev

Conversation: https://staging.warp.dev/conversation/2b2a0786-45a7-4d14-ac07-f300fa1aed27
Run: https://oz.staging.warp.dev/runs/019e81ed-8cc0-7ab3-a2d8-4734728412c9
This PR was generated with Oz.

[oz-for-oss]

@szgupta

I'm starting a first review of this pull request.

You can view the conversation on Warp.

I completed the review and no human review was requested for this pull request.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

[oz-for-oss]

Overview

This PR changes request-file-edits success results to build ranged UpdatedFileContext entries from accepted editor-buffer contents, with tests for ranged output and no-range fallback.

Concerns

• The ranged slicing path currently uses editor diff ranges as if they were 1-based file-context ranges, which can report line 0 and return the wrong content slice/context window.

Verdict

Found: 0 critical, 1 important, 0 suggestions

Request changes

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz