Skip to content

Use gh pr diff for valid-line filter; fix preexisting issue format#308

Merged
jth-nw merged 1 commit intomainfrom
dev
Feb 24, 2026
Merged

Use gh pr diff for valid-line filter; fix preexisting issue format#308
jth-nw merged 1 commit intomainfrom
dev

Conversation

@jth-nw
Copy link
Collaborator

@jth-nw jth-nw commented Feb 24, 2026

  • Replace local git diff (--unified=10) with gh pr diff to get the exact diff GitHub uses. Local unified=10 was accepting lines 4-10 away from changes that GitHub's 3-line context window excludes, causing "Line could not be resolved" 422 errors.
  • Clarify prompt: both review sections must use a flat list with no subheadings or extra nesting, matching the format in instructions.

- Replace local git diff (--unified=10) with gh pr diff to get the
  exact diff GitHub uses. Local unified=10 was accepting lines 4-10
  away from changes that GitHub's 3-line context window excludes,
  causing "Line could not be resolved" 422 errors.
- Clarify prompt: both review sections must use a flat list with no
  subheadings or extra nesting, matching the format in instructions.

Generated with AI

Co-Authored-By: Claude Code <ai@netwrix.com>
@jth-nw jth-nw merged commit 9587847 into main Feb 24, 2026
6 of 8 checks passed
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