code-review

SKILL.md

---
name: code-review
description: Use when reviewing a diff or PR. Checks readability, edge cases, error handling, security, and tests in that order.
---

# Code review

When reviewing a change:

1. **Readability** — names match domain language? functions short enough to hold in head?
2. **Edge cases** — empty input, null, very long input, concurrent calls, network failure?
3. **Error handling** — every failure path produces a clear user message AND a useful log line?
4. **Security** — input validated? secrets in env (not code)? auth check on the right side of the boundary?
5. **Tests** — at least one happy path + one edge case covered?

Comment AT MOST 5 things. Pick the most impactful — review fatigue is real.

الوسوم