Repository navigation
fix(ai-review): review exact changes with complete PR context - #13
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR reviews could inspect files from a different ref than the requested head and compare against the base branch tip, including unrelated changes when a branch was behind. Check out the exact head and derive the PR diff from its merge base; commit reviews retain parent-to-commit comparison.
Prepare PR title/body and independently paginated review threads/comments before analysis. Tighten the prompt to validate concrete failure scenarios, omit speculative comments, consider intentional tradeoffs, and avoid repeating findings already covered by the discussion.
review_instructionsnow supplements the core rules.Intentional decisions and tradeoffs
gpt-5.6-terra), effort, CLI/action pins, and directAPPROVEposting remain unchanged.Validation
python3 -m unittest discover -s tests -v: 5 regression tests passed (branch behind main, parent-to-commit review, incorrect checkout, changed PR head, prompt/checkout contract, and multi-page API response handling).actionlint1.7.7: passed.git diff --check: passed.Live Codex analysis was not invoked; this PR validates workflow plumbing and prompt generation without posting test reviews.