1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
|
---
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
|
