Skip to content

#1648 - AlignedString.resolve returns an empty interval for the first character after a deletion - #1649

Merged
reckart merged 1 commit into
mainfrom
bugfix/1648-AlignedString.resolve-returns-an-empty-interval-for-the-first-character-after-a-deletion
Aug 9, 2026
Merged

reckart merged 1 commit into
mainfrom
bugfix/1648-AlignedString.resolve-returns-an-empty-interval-for-the-first-character-after-a-deletion

Conversation

@reckart

@reckart reckart commented Aug 9, 2026

Copy link
Copy Markdown
Member

What's in the PR

  • Fix AlignedString.resolve() to correctly handle single-character intervals at anchor positions by preventing anchor-inclusive segment lookup for endSeg in non-empty intervals
  • Add regression tests for resolve() correctness: testResolveSingleCharacterAtStart, testResolveSingleCharacterAtInnerAnchor, testResolveRoundTripsForEveryCharacter, testResolveRoundTripsForEveryCharacterWithAnchors

How to test manually

  • See issue description

Automatic testing

  • PR includes unit tests

Documentation

  • PR updates documentation

… character after a deletion

- Fix AlignedString.resolve() to correctly handle single-character intervals at anchor positions by preventing anchor-inclusive segment lookup for endSeg in non-empty intervals
- Add regression tests for resolve() correctness: testResolveSingleCharacterAtStart, testResolveSingleCharacterAtInnerAnchor, testResolveRoundTripsForEveryCharacter, testResolveRoundTripsForEveryCharacterWithAnchors
@reckart reckart added this to the 3.0.0 milestone Aug 9, 2026
@reckart reckart self-assigned this Aug 9, 2026
@reckart reckart added this to Kanban Aug 9, 2026
@reckart reckart added 🐛Bug Something isn't working Module-api.transform labels Aug 9, 2026
@github-project-automation github-project-automation Bot moved this to In progress in Kanban Aug 9, 2026
@reckart
reckart merged commit d8ae08d into main Aug 9, 2026
5 checks passed
@reckart
reckart deleted the bugfix/1648-AlignedString.resolve-returns-an-empty-interval-for-the-first-character-after-a-deletion branch August 9, 2026 15:10
@github-project-automation github-project-automation Bot moved this from In progress to Done in Kanban Aug 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛Bug Something isn't working Module-api.transform

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant