Skip to content

diffmap: Add diff position mapper#1280

Open
abhinav wants to merge 1 commit into
mainfrom
branch-comment-diffmap
Open

diffmap: Add diff position mapper#1280
abhinav wants to merge 1 commit into
mainfrom
branch-comment-diffmap

Conversation

@abhinav

@abhinav abhinav commented Jun 14, 2026

Copy link
Copy Markdown
Owner

Comment workflows need to translate working-tree line numbers into
diff positions before a forge can accept inline comment requests.

The diff mapper parses unified diff hunks into a small mapping API that
callers can use without depending on any forge implementation.
Keeping this package independent lets forge comment support and CLI
state management adopt the mapper separately.

Extracted from #1162

Comment workflows need to translate working-tree line numbers into
diff positions before a forge can accept inline comment requests.

The diff mapper parses unified diff hunks into a small mapping API that
callers can use without depending on any forge implementation.
Keeping this package independent lets forge comment support and CLI
state management adopt the mapper separately.

Extracted from #1162
@abhinav

abhinav commented Jun 14, 2026

Copy link
Copy Markdown
Owner Author

This change is part of the following stack:

Change managed by git-spice.

@abhinav abhinav added the skip changelog PRs that don't need a changelog. label Jun 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip changelog PRs that don't need a changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants