Fall back on aggressive line number stripping in diffs

[mrubens]

Context

We're seeing a lot of errors applying diffs in the Gemini models. When we look closely, a lot of them are related to not stripping line numbers correctly in the search and replace blocks.

Implementation

What this change does is add another more aggressive second pass if we fail to apply a diff with the standard approach. This second pass will more aggressively strip leading pipes+spaces from both search and replace, even if there's not a line number before it or not every line has a line number.

---

Important

Add aggressive line number stripping fallback in applyDiff() to handle inconsistent line numbers in diffs.

• Behavior:
  • Adds aggressive line number stripping fallback in applyDiff() in multi-search-replace.ts.
  • Handles cases where line numbers are inconsistent or missing in diffs.
• Functions:
  • Updates stripLineNumbers() in extract-text.ts to support aggressive mode for line number stripping.
• Tests:
  • Adds tests for aggressive line number stripping in multi-search-replace.test.ts and extract-text.test.ts.
  • Removes a test that expected failure when not all lines had numbers in multi-search-replace.test.ts.

^{This description was created by for 112a1a0. It will automatically update as commits are pushed.}

[changeset-bot]

⚠️ No Changeset found

Latest commit: 112a1a0

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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