From 898b52edcb47bcb3e9d6106e74ca73e74ea01e70 Mon Sep 17 00:00:00 2001 From: sillylaird Date: Thu, 3 Sep 2026 00:33:59 +0000 Subject: import live www.sillylaird.ca webroot --- .../references/code-review-best-practices.md | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) create mode 100644 .agents/skills/code-review-excellence/references/code-review-best-practices.md (limited to '.agents/skills/code-review-excellence/references/code-review-best-practices.md') 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. -- cgit v1.2.3