Skip to content

Use GitHub's merge base for contract review comparisons - #43

Merged
spolu merged 1 commit into
spolu:mainfrom
fontanierh:codex/fix-review-comparison-merge-base
Nov 7, 2026
Merged

spolu merged 1 commit into
spolu:mainfrom
fontanierh:codex/fix-review-comparison-merge-base

Conversation

@fontanierh

@fontanierh fontanierh commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Git and GitHub can choose different starting points when comparing a PR. The bot can then try to comment on a line outside GitHub's diff, making the whole review fail with 422: Line could not be resolved.

This change makes review setup and posting use GitHub's comparison of the saved commits. Findings that cannot be posted on a diff line stay in the review body, with links to the source.

Tests:

  • Added two tests for a history with two merge bases. Both fail before the fix and pass after it.
  • Existing formatting, type, lint, build, CLI tests and contract checks pass.

@spolu spolu left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fontanierh

Copy link
Copy Markdown
Contributor Author

(I can't actually merge on your repo @spolu)

@spolu
spolu merged commit 9d27bbd into spolu:main Nov 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants