Pull Request Review

Perform a thorough code review covering quality, security, performance, and maintainability.

Review Checklist

Code Quality

• Naming: clear, descriptive, consistent with codebase conventions
• Functions: single responsibility, reasonable length (<50 lines)
• Error handling: all failure paths covered, no swallowed exceptions
• Types: proper TypeScript/type annotations where applicable
• DRY: no unnecessary duplication
• Dead code: nothing unused or commented out

Security (see also: security-review skill)

• No secrets or credentials in code
• Input validation on all user-facing endpoints
• Parameterized queries (no string concatenation for SQL)
• Proper auth/authz checks