SeifBenayed Pull Request Review
Review the current branch's changes compared to the base branch.
Steps
Determine the base branch:
- Try
main, thenmaster, then ask - Run
git log --oneline <base>..HEADto see all commits in this PR
- Try
Run
git diff <base>...HEADto see the full diffFor large diffs, review file-by-file using
git diff <base>...HEAD -- <file>For each changed file, check for:
- Bugs: logic errors, off-by-one, null/undefined access, race conditions
- Security: injection, XSS, auth bypass, hardcoded secrets, SSRF
- Performance: N+1 queries, unbounded loops, missing indexes, memory leaks
- Error handling: missing try/catch, swallowed errors, unclear error messages
- Style: inconsistency with the rest of the codebase
- Tests: missing tests for new functionality or edge cases
Provide a structured review:
Summary — What the PR does (1-2 sentences)
Issues — Bugs, security, or correctness problems that should be fixed before merge
Suggestions — Non-blocking improvements worth considering
Verdict — Approve / Request Changes / Needs Discussion
Be specific — reference file paths and line numbers. Suggest concrete fixes, not vague advice.
$ARGUMENTS