@educa-corp/sdd-framework 0.1.0

package/commands/review-code.tmpl

@@ -0,0 +1,104 @@
+# /review-code — Code Review chỉ-đọc
+
+**CẢNH BÁO: READ-ONLY. Chỉ report, KHÔNG sửa.**
+
+## Gate
+{{include:steps/gate.md}}
+
+*Lưu ý: Với lệnh này, target ở Bước 1 là một UC-ID, file path, hoặc tên branch hiện tại. Bỏ qua sang nạp context ngay sau khi phân giải target.*
+
+## Context
+{{include:steps/context-loader.md}}
+
+---
+
+## Pre-Review Scan
+
+*(Tách khỏi CHECKPOINT của Gate — phần này liệt kê file và scenario thực tế trong scope.)*
+
+Quét tìm file implementation và scenario, rồi hiện:
+
+```
+Review Scope — {UC-ID}
+──────────────────────────────────────
+UC        : {UC-ID}
+Files     : {danh sách file gắn tag @trace.implements={UC-ID}}
+Scenarios : {N} scenario trong file .feature
+
+Tiếp tục review? (Y/N)
+```
+
+Chờ "Y" rõ ràng trước khi tiếp tục.
+
+---
+
+## Review Dimensions
+
+### 1. Traceability
+- [ ] Mỗi controller endpoint có tag `@trace.implements`?
+- [ ] Mỗi test file có tag `@trace.verifies`?
+- [ ] Không có tag `@trace` ở sai layer?
+- [ ] `{paths.trace_dir}/{domain}/{prd-slug}/{UC-ID}-{platform}.tsv` cập nhật chưa? (nếu stale → chạy `/validate-traces {UC-ID}` trước, rồi chạy lại review này)
+
+### 2. Layer Architecture (từ CLAUDE.md §2)
+- [ ] Mỗi class ở đúng layer?
+- [ ] Phụ thuộc giữa layer đi đúng chiều?
+- [ ] Không layer nào bị bypass?
+
+### 3. Coding Standards (từ CLAUDE.md §3)
+- [ ] Tuân theo naming convention?
+- [ ] Response wrapper dùng nhất quán?
+- [ ] Exception được throw (không bị nuốt)?
+- [ ] Không magic number, không dữ liệu nhạy cảm trong log?
+- [ ] Annotation transaction đúng?
+
+### 4. Spec Compliance
+- [ ] Mỗi scenario trong .feature có implementation?
+- [ ] Không có endpoint không tài liệu (code không có spec backing)?
+
+## Output
+
+{{include:steps/report-footer.md}}
+
+```
+/review-code Report — {UC-ID}
+Critical: {X} | Major: {Y} | Minor: {Z}
+
+### Critical
+| # | File | Line | Issue | Suggested Fix |
+
+### Major
+| # | File | Line | Issue | Suggested Fix |
+
+### Minor
+| # | File | Line | Issue | Suggested Fix |
+
+Output Artifacts: none (read-only)
+
+Verdict: APPROVED ✅ | NEEDS_FIX ❌
+
+Nếu APPROVED ✅:
+  Next: /dev-gen-test {UC-ID}
+
+Nếu NEEDS_FIX ❌:
+  - Fix nhỏ (1–3 dòng, không đổi logic) → fix inline → chạy lại /review-code {UC-ID}
+  - Vấn đề logic / kiến trúc              → /fix-bug {TICKET_ID}
+  - Spec mismatch (code ≠ scenario)       → /generate-code {feature-file}  (gen lại UC bị ảnh hưởng)
+```
+
+---
+
+## Đề xuất ghi Lessons (tuỳ chọn)
+
+Với mỗi finding **Critical/Major** thể hiện một **lỗi AI lặp lại** trong quá trình sinh code
+(một vi phạm kiến trúc/standards mà AI có khả năng làm lại — KHÔNG phải typo một lần hay nguyên nhân bên ngoài), hỏi:
+
+```
+Finding "{issue}" trông như một lỗi lặp lại.
+Ghi nó thành project lesson để /generate-code không lặp lại? (Y/N)
+```
+
+Nếu `Y` → chạy quy trình capture bên dưới với `category=code-gen`, `source=/review-code {UC-ID}`,
+`scope` = domain hoặc file glob bị ảnh hưởng.
+
+{{include:steps/capture-lesson.md}}