---
name: code-review
description: "Review diffs for correctness, edge cases, regressions, API compatibility, maintainability, and missing tests."
---

# Code Review Skill

Review diffs for correctness, edge cases, regressions, API compatibility, maintainability, and missing tests.

## Subagents
When you need to delegate sub‑tasks, use the `pi-subagents` skill.

**Example:** For a large pull request, you could assign one **scout** to read the changed files in the `api` layer, another **scout** to inspect the `database` migrations, and a **reviewer** to check each area for correctness, edge cases, and API compatibility—then aggregate their findings into a single review.

*You may adapt the delegation pattern to fit the exact requirements of the codebase.*

## Checklist

### Correctness
- [ ] Logic matches requirements
- [ ] Edge cases handled
- [ ] No off-by-one errors
- [ ] Proper error handling

### Regressions
- [ ] Existing tests pass
- [ ] No broken functionality
- [ ] Migration path for breaking changes

### API Compatibility
- [ ] Public interfaces preserved
- [ ] Version compatibility maintained
- [ ] Deprecation warnings where needed

### Maintainability
- [ ] Code is readable
- [ ] Appropriate abstraction level
- [ ] No unnecessary complexity
- [ ] Consistent with codebase style

### Tests
- [ ] New functionality tested
- [ ] Edge cases covered
- [ ] Regression tests added