blob: 6629d522b24f07bddd046643d2cd2cb3051b5b2e (
plain)
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
|
---
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
|
