From 2830b9873abe8f99281232ed6f30e8d72fc8cace Mon Sep 17 00:00:00 2001 From: Tu Shaokun <2801884530@qq.com> Date: Sun, 30 Nov 2025 15:35:08 +0800 Subject: [PATCH] =?UTF-8?q?feat:=20=E5=9C=A8=E5=AE=A1=E6=9F=A5=E6=B5=81?= =?UTF-8?q?=E7=A8=8B=E4=B8=AD=E6=B7=BB=E5=8A=A0=E6=9E=B6=E6=9E=84/?= =?UTF-8?q?=E6=80=A7=E8=83=BD=E5=AE=A1=E6=9F=A5=E8=A7=A6=E5=8F=91=E7=82=B9?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Phase 2 添加 Architecture Review Guide 引用 - 新增 Performance Assessment 检查项 - 明确列出关键检查点(SOLID、N+1、复杂度等) --- SKILL.md | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/SKILL.md b/SKILL.md index d251831..dca634b 100644 --- a/SKILL.md +++ b/SKILL.md @@ -102,8 +102,13 @@ Before diving into code, understand: ### Phase 2: High-Level Review (5-10 minutes) 1. **Architecture & Design** - Does the solution fit the problem? -2. **File Organization** - Are new files in the right places? -3. **Testing Strategy** - Are there tests covering edge cases? + - For significant changes, consult [Architecture Review Guide](reference/architecture-review-guide.md) + - Check: SOLID principles, coupling/cohesion, anti-patterns +2. **Performance Assessment** - Are there performance concerns? + - For performance-critical code, consult [Performance Review Guide](reference/performance-review-guide.md) + - Check: Algorithm complexity, N+1 queries, memory usage +3. **File Organization** - Are new files in the right places? +4. **Testing Strategy** - Are there tests covering edge cases? ### Phase 3: Line-by-Line Review (10-20 minutes)