Code Review Skill
Workflow
- •Context: Read PR description and identify scope.
- •Review: Focus on what matters (see priorities below).
- •Verify: Run linting and tests.
- •Feedback: Be constructive, explain rationale.
Priorities
- •PR Description: Must be filled with clear context and purpose.
- •Simplicity: The simplest solution that works. No over-engineering.
- •Clean Code: Readable, well-named, no duplication.
- •Security: No hardcoded secrets, validate inputs, handle errors.
- •Tests: Appropriate coverage for the change.