[diffs/edit] Fix batch edit ordering sensitivity#991
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
pierre/packages/diffs/test/README.md
Line 106 in 2c2f975
After removing the batch-edit ordering bullet and converting its test.failing case to a passing test, the category counts in this section now add up to 29, but the heading still advertises 30. This README is the triage list for pinned failing tests, so the stale total will mislead future cleanup tracking unless it is decremented with the removed item.
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
Fixed batch edit ordering sensitivity.
Equal-start edits now sort by end offset, placing inserts before deletes regardless of caller order.