# Review

> Review current branch changes for bugs, security, and quality before merging.

- Skill: `rhythmmittal19/review` (Agent Skill)
- Install (CLI): `npx skillmds@latest add rhythmmittal19/review`
- Raw SKILL.md: https://api.skillmd.com/api/skills/rhythmmittal19/review/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Security
- Author: RhythmMittal19 (https://skillmd.com/u/rhythmmittal19)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/rhythmmittal19/review

---


## Branch Info
!`git branch --show-current`

## Files Changed
!`git diff --name-only main...HEAD 2>/dev/null || git diff --name-only HEAD~3`

## Full Diff
!`git diff main...HEAD 2>/dev/null || git diff HEAD~3`

Review every changed file systematically:

1. **Bugs**: Logic errors, off-by-one, null/undefined access, race conditions
2. **Security**: XSS, injection, exposed secrets, unsafe deserialization, missing auth checks
3. **Types**: TypeScript strict violations, `any` usage, missing null checks
4. **Tests**: Is new functionality tested? Are edge cases covered?
5. **Performance**: N+1 queries, unnecessary re-renders, missing memoization, large bundle imports
6. **Style**: Does it follow existing patterns in the codebase?

For each issue found:
- File and line reference
- Severity (🔴 blocker / 🟡 should fix / 🟢 nice to have)
- Specific fix suggestion

End with summary: ship it ✅ or needs changes ❌