diff options
| author | sillylaird <sillyfanboy@gmail.com> | 2026-09-03 00:33:59 +0000 |
|---|---|---|
| committer | sillylaird <sillyfanboy@gmail.com> | 2026-09-03 00:33:59 +0000 |
| commit | 898b52edcb47bcb3e9d6106e74ca73e74ea01e70 (patch) | |
| tree | 85c6ee5ad58b860144551184d4cf86b560c62b91 /.agents/skills/code-review-excellence/references/code-review-best-practices.md | |
| download | www-898b52edcb47bcb3e9d6106e74ca73e74ea01e70.tar.gz www-898b52edcb47bcb3e9d6106e74ca73e74ea01e70.zip | |
Diffstat (limited to '.agents/skills/code-review-excellence/references/code-review-best-practices.md')
| -rw-r--r-- | .agents/skills/code-review-excellence/references/code-review-best-practices.md | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/.agents/skills/code-review-excellence/references/code-review-best-practices.md b/.agents/skills/code-review-excellence/references/code-review-best-practices.md new file mode 100644 index 0000000..65c37bb --- /dev/null +++ b/.agents/skills/code-review-excellence/references/code-review-best-practices.md @@ -0,0 +1,19 @@ +# Code Review Best Practices + +## Default review order +1. Understand intent and scope. +2. Check architecture and correctness. +3. Check tests and failure handling. +4. Check security and performance risks. +5. Leave clear, prioritized comments. + +## Comment severity +- `blocking` - correctness, security, data loss, major maintainability issue +- `important` - should be fixed before merge if practical +- `nit` - polish only + +## Good reviewer habits +- summarize first, +- separate required changes from suggestions, +- quote the code path or failure mode, +- praise good decisions when they matter. |