← ClaudeAtlas

reviewlisted

Review code changes for flaws, bugs, security issues, and potential problems
Jewgah/claude-code-skills · ★ 2 · Code & Development · score 76
Install: claude install-skill Jewgah/claude-code-skills
# Code Review Perform a thorough code review of the current changes. Act as a senior developer reviewing a pull request. ## Steps 1. **Gather changes based on scope**: - If `$ARGUMENTS` contains "staged": only `git diff --cached` - If `$ARGUMENTS` contains "unstaged": only `git diff` - If `$ARGUMENTS` contains "last-commit": `git rev-parse HEAD~1 >/dev/null 2>&1 && git diff HEAD~1 || git diff $(git hash-object -t tree /dev/null)` (falls back to the empty tree on a repo's first commit) - Otherwise: both `git diff` and `git diff --cached` (all pending changes) - Also run `git status` for context 2. **Read full files**: For every file that has changes, read the COMPLETE file (not just the diff) to understand the full context around the modifications. 3. **Analyze each change** against this checklist: ### Logic & Correctness - Are there logic errors, off-by-one mistakes, or wrong conditions? - Are edge cases handled (null, undefined, empty arrays, missing fields)? - Do loops and iterations behave correctly? - Are return values used correctly? ### Security - Any injection risks (SQL, command, XSS)? - Is user input validated and sanitized? - Are there exposed secrets or credentials? - Are auth checks present where needed? ### Data & State - Are data operations correct (correct methods, field names, query parameters)? - Is state managed correctly (component state, query cache)? - Are race conditions possible? - Is data transfor