(MOT-3874) feat(shell): coder::apply-patch + worktree add/remove#412
(MOT-3874) feat(shell): coder::apply-patch + worktree add/remove#412ytallo wants to merge 1 commit into
Conversation
Adds the apply_patch (V4A) format codex-family models are trained on: *** Begin/End Patch with Add/Delete/Update File hunks, @@ context markers, optional Move to, and End of File pinning. seek_sequence and the replacement engine are ported from OpenAI's codex-rs apply-patch crate (Apache-2.0) so context matching (exact -> rstrip -> trim -> unicode-normalised) matches the reference; the parser is a compact re-implementation pinned by the reference test cases. All-or-nothing: every hunk resolves through the jail and computes before anything is written (context mismatch fails C210 with re-read guidance, filesystem untouched); writes land atomically per file and modified files return a bounded echo of the first changed region.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
skill-check — worker0 verified, 32 skipped (no docs/).
Four for four. Nicely done. |
|
Consolidated into #423. |
Adds the apply_patch (V4A) format codex-family models are trained on:
*** Begin/End Patch with Add/Delete/Update File hunks, @@ context
markers, optional Move to, and End of File pinning. seek_sequence and
the replacement engine are ported from OpenAI's codex-rs apply-patch
crate (Apache-2.0) so context matching (exact -> rstrip -> trim ->
unicode-normalised) matches the reference; the parser is a compact
re-implementation pinned by the reference test cases.
All-or-nothing: every hunk resolves through the jail and computes
before anything is written (context mismatch fails C210 with re-read
guidance, filesystem untouched); writes land atomically per file and
modified files return a bounded echo of the first changed region.
Refs MOT-3874