Merge branch 'adding-pi-harness'

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