Review

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

RhythmMittal19 f557052 1.0 KB Updated

File contents

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 ❌

RhythmMittal19/claude-config/tree/main/skills/review commit f55705267f

Frequently asked questions

npx skillmds@latest add rhythmmittal19/review