Code Review Guide

Review Priorities (In Order)

1. Correctness - Does it work? Does it handle edge cases?
2. Security - Vulnerabilities, data exposure, injection risks
3. Performance - Obvious inefficiencies, scaling concerns
4. Maintainability - Readability, complexity, documentation
5. Style - Formatting, naming (lowest priority if linter exists)

What to Look For

Correctness

• Logic handles all cases (including edge cases)
• Error handling is appropriate
• Async operations handled correctly
• State mutations are intentional
• Tests cover the changes