Code Review Best Practices

Systematic approach to reviewing code for quality, security, and maintainability

majiayu000 c6d4e0c 2 files · 1.6 KB Updated 567 repo stars

File contents

Code Review Skill

When reviewing code, follow this comprehensive checklist:

1. Correctness

  • Does the code do what it's supposed to do?
  • Are edge cases handled?
  • Is the logic correct?
  • Are there any off-by-one errors?

2. Security

  • Input validation present?
  • No SQL injection vulnerabilities?
  • No XSS vulnerabilities?
  • Secrets not hardcoded?
  • Proper authentication/authorization?

3. Performance

  • No unnecessary loops or iterations?
  • Efficient data structures used?
  • Database queries optimized?
  • No N+1 query problems?

4. Maintainability

  • Code is readable and self-documenting?
  • Functions/methods are focused (single responsibility)?
  • Appropriate naming conventions?
  • No magic numbers/strings?

5. Testing

  • Unit tests included?
  • Edge cases tested?
  • Tests are meaningful (not just for coverage)?

6. Documentation

  • Public APIs documented?
  • Complex logic explained?
  • README updated if needed?

Review Feedback Guidelines

  • Be constructive and specific
  • Explain the "why" behind suggestions
  • Distinguish between required changes and suggestions
  • Acknowledge good practices

majiayu000/claude-skill-registry-data/tree/main/development/code-review-best-practices commit c6d4e0c529

Frequently asked questions

npx skillmds add majiayu000/code-review-best-practices