diff options
Diffstat (limited to 'modules/development/pi-coding-agent/skills/code-review')
| -rw-r--r-- | modules/development/pi-coding-agent/skills/code-review/SKILL.md | 44 |
1 files changed, 44 insertions, 0 deletions
diff --git a/modules/development/pi-coding-agent/skills/code-review/SKILL.md b/modules/development/pi-coding-agent/skills/code-review/SKILL.md new file mode 100644 index 0000000..bdbacd0 --- /dev/null +++ b/modules/development/pi-coding-agent/skills/code-review/SKILL.md @@ -0,0 +1,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 |