Code Review

Code Review Skill

quannv1198 Updated

File contents

Code Review Skill

  • Review code for correctness, bugs, security issues, performance, and maintainability.
  • Understand the existing architecture and conventions before judging or changing code.
  • Prioritize real problems over style preferences.
  • Check edge cases, error handling, null/undefined handling, and race conditions.
  • Look for unnecessary complexity, duplicated logic, and dead code.
  • Check API contracts, data flow, state management, and side effects.
  • Identify potential performance problems and unnecessary re-renders.
  • Check TypeScript type safety and unsafe assumptions.
  • Check React hooks, dependencies, component lifecycle, and rendering behavior when applicable.
  • Check accessibility and user-facing behavior when applicable.
  • Do not suggest changes that are purely subjective unless they improve consistency or readability.
  • Prefer minimal, practical fixes over large refactors.
  • Clearly explain each finding with the problem, impact, and recommended fix.
  • Rank findings by severity: critical, high, medium, low.
  • If the code is correct, say so instead of inventing issues.

quannv1198/agent-skills/tree/main/code-review commit 7e244a856f

Frequently asked questions

npx skillmds@latest add quannv1198/code-review