---
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.

## 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