review-pr

Files (1)

• SKILL.md

SKILL.md (first lines)

# Pull Request Review

Review the current branch's changes compared to the base branch.

## Steps

1. Determine the base branch:
   - Try `main`, then `master`, then ask
   - Run `git log --oneline <base>..HEAD` to see all commits in this PR
2. Run `git diff <base>...HEAD` to see the full diff
3. For large diffs, review file-by-file using `git diff <base>...HEAD -- <file>`
4. 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
5. 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

From SeifBenayed/cloclo, MIT.

More from SeifBenayed/cloclo