Skip to content

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

Open
fontanierh wants to merge 1 commit into
spolu:mainfrom
fontanierh:codex/fix-review-comparison-merge-base
Open

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

Conversation

@fontanierh

@fontanierh fontanierh commented Oct 21, 2026 •

Copy link
Copy Markdown

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.

This branch has not been deployed

No deployments
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.

1 participant