# Review

> Review all uncommitted changes against project standards before committing

- Skill: `smartwhale8/review` (Agent Skill)
- Install (CLI): `npx skillmds@latest add smartwhale8/review`
- Raw SKILL.md: https://api.skillmd.com/api/skills/smartwhale8/review/raw
- Safety review: pending (external: skill-scanner PASS, skillspector PASS)
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Coding & Dev Tools
- Author: smartwhale8 (https://skillmd.com/u/smartwhale8)
- Updated: 2026-09-10
- Page: https://skillmd.com/skills/smartwhale8/review

---


Review all uncommitted changes in this project. $ARGUMENTS

## Process

1. **Gather changes**: Run `git diff --staged` for staged changes, or `git diff` if nothing is staged. Also check `git status` for new untracked files.

2. **Check each changed file against the project rules** (`.claude/rules/*.md`):

   ### Code Quality
   - Is there dead code, unused imports, or commented-out code?
   - Are there bandaid fixes or workarounds instead of root-cause solutions?
   - Is anything over-engineered for what it needs to do?

   ### Architecture
   - Does this follow existing patterns in the codebase?
   - Is there duplicate logic that should use an existing shared component or utility?
   - Are dependencies pointing in the right direction (inward, not outward)?

   ### Consistency
   - Does the code match the style and structure of similar files in the project?
   - For frontend: are spacing, colors, and component usage consistent with the rest of the UI?
   - For backend: are error handling, response format, and validation patterns consistent?

   ### Reuse
   - Could any new component, function, or pattern already exist in the codebase? Search before approving.
   - If something similar exists, flag it — it should be reused or extracted into a shared utility.

   ### Security
   - Any hardcoded secrets, credentials, or API keys?
   - Is user input validated at the boundary?
   - Are there SQL injection, XSS, or other OWASP risks?

   ### Performance
   - Any N+1 query patterns (database calls inside loops)?
   - Any unbounded queries missing LIMIT/pagination?
   - Frontend: unnecessary re-renders, missing debounce on inputs?

3. **Verdict**: Provide one of:
   - **Ready to commit** — no issues found
   - **Minor issues** — list them, but committing is acceptable after acknowledging
   - **Needs changes** — list specific issues that must be fixed before committing

For each issue, provide: the file, the line(s), what's wrong, and what the fix should be.