@educa-corp/sdd-framework 0.6.0 → 0.7.1

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (223) hide show
  1. package/bin/gate-trace.js +25 -2
  2. package/bin/index.js +32 -5
  3. package/bin/lint-trace.js +41 -0
  4. package/bin/self-check.js +430 -3
  5. package/bin/trace-schema.json +418 -31
  6. package/core/FRAMEWORK_VERSION +1 -1
  7. package/{commands/extend-prd.md → core/commands/amend-prd.md} +206 -173
  8. package/core/commands/dev-run-test.md +48 -10
  9. package/core/commands/extend-prd.md +39 -12
  10. package/core/commands/generate-bdd.md +52 -10
  11. package/core/commands/generate-code.md +35 -2
  12. package/core/commands/generate-tech-docs.md +36 -4
  13. package/core/commands/map-testids.md +1 -1
  14. package/core/commands/qc-run-test.md +29 -3
  15. package/core/commands/refine-prd.md +13 -2
  16. package/core/commands/review-context.md +43 -8
  17. package/core/commands/sync.md +105 -1
  18. package/core/commands/validate-traces.md +289 -16
  19. package/core/rules/workflow.md +34 -0
  20. package/core/steps/context-loader.md +27 -6
  21. package/core/templates/feature.template +1 -1
  22. package/core/templates/project-context.yaml +3 -3
  23. package/core/templates/tech-design.template.md +2 -2
  24. package/docs/02-concepts/architecture.md +37 -1
  25. package/docs/02-concepts/overview.md +1 -1
  26. package/docs/02-concepts/pipeline-steps/02-specification.md +13 -7
  27. package/docs/02-concepts/pipeline-steps/07-dev-selftest.md +2 -0
  28. package/docs/02-concepts/pipeline-steps/08-qc-automation.md +1 -0
  29. package/docs/02-concepts/pipeline-steps/09-validate-traces.md +34 -3
  30. package/docs/02-concepts/pipeline-steps/10-feedback-loop.md +10 -1
  31. package/docs/02-concepts/traceability.md +187 -183
  32. package/docs/03-guides/architect.md +13 -4
  33. package/docs/03-guides/developer.md +1 -0
  34. package/docs/03-guides/product-owner.md +89 -72
  35. package/docs/03-guides/tester-qa.md +81 -81
  36. package/docs/04-reference/commands.md +148 -134
  37. package/docs/04-reference/trace-schema.md +45 -1
  38. package/docs/explain/02b-extend-prd.md +1 -1
  39. package/docs/explain/02c-amend-prd.md +152 -0
  40. package/docs/explain/06-generate-bdd.md +1 -1
  41. package/docs/explain/13-dev-run-test.md +15 -1
  42. package/docs/explain/19-qc-run-test.md +91 -87
  43. package/docs/explain/21-validate-traces.md +79 -75
  44. package/docs/explain/28-sync.md +25 -0
  45. package/docs/explain/README.md +136 -135
  46. package/package.json +1 -8
  47. package/commands/debug.md +0 -529
  48. package/commands/debug.tmpl +0 -260
  49. package/commands/define-product.md +0 -438
  50. package/commands/define-product.tmpl +0 -225
  51. package/commands/dev-gen-test.md +0 -700
  52. package/commands/dev-gen-test.tmpl +0 -490
  53. package/commands/dev-run-test.md +0 -435
  54. package/commands/dev-run-test.tmpl +0 -225
  55. package/commands/dev-smoke-test.md +0 -374
  56. package/commands/dev-smoke-test.tmpl +0 -217
  57. package/commands/extend-prd.tmpl +0 -273
  58. package/commands/fix-bug.md +0 -519
  59. package/commands/fix-bug.tmpl +0 -197
  60. package/commands/generate-architecture.md +0 -354
  61. package/commands/generate-architecture.tmpl +0 -197
  62. package/commands/generate-bdd.md +0 -923
  63. package/commands/generate-bdd.tmpl +0 -590
  64. package/commands/generate-code.md +0 -859
  65. package/commands/generate-code.tmpl +0 -649
  66. package/commands/generate-design-spec.md +0 -737
  67. package/commands/generate-design-spec.tmpl +0 -524
  68. package/commands/generate-prd.md +0 -722
  69. package/commands/generate-prd.tmpl +0 -226
  70. package/commands/generate-spec-manifest.md +0 -321
  71. package/commands/generate-spec-manifest.tmpl +0 -164
  72. package/commands/generate-tech-docs.md +0 -920
  73. package/commands/generate-tech-docs.tmpl +0 -273
  74. package/commands/learn.md +0 -399
  75. package/commands/learn.tmpl +0 -130
  76. package/commands/map-testids.md +0 -238
  77. package/commands/map-testids.tmpl +0 -81
  78. package/commands/propose-scenario.md +0 -359
  79. package/commands/propose-scenario.tmpl +0 -202
  80. package/commands/qc-analyze.md +0 -269
  81. package/commands/qc-analyze.tmpl +0 -112
  82. package/commands/qc-design-test.md +0 -226
  83. package/commands/qc-design-test.tmpl +0 -69
  84. package/commands/qc-plan.md +0 -206
  85. package/commands/qc-plan.tmpl +0 -49
  86. package/commands/qc-report.md +0 -217
  87. package/commands/qc-report.tmpl +0 -60
  88. package/commands/qc-review.md +0 -210
  89. package/commands/qc-review.tmpl +0 -53
  90. package/commands/qc-run-test.md +0 -326
  91. package/commands/qc-run-test.tmpl +0 -116
  92. package/commands/refine-prd.md +0 -653
  93. package/commands/refine-prd.tmpl +0 -281
  94. package/commands/report-bug.md +0 -305
  95. package/commands/report-bug.tmpl +0 -148
  96. package/commands/review-code.md +0 -415
  97. package/commands/review-code.tmpl +0 -146
  98. package/commands/review-context.md +0 -902
  99. package/commands/review-context.tmpl +0 -530
  100. package/commands/review-tech-docs.md +0 -561
  101. package/commands/review-tech-docs.tmpl +0 -404
  102. package/commands/setup-ai-first.md +0 -602
  103. package/commands/setup-ai-first.tmpl +0 -450
  104. package/commands/sync.md +0 -430
  105. package/commands/sync.tmpl +0 -429
  106. package/commands/update-framework.md +0 -203
  107. package/commands/update-framework.tmpl +0 -202
  108. package/commands/validate-traces.md +0 -1077
  109. package/commands/validate-traces.tmpl +0 -920
  110. package/hooks/data-guard.js +0 -232
  111. package/hooks/settings.json +0 -19
  112. package/modules/android-compose/module.yaml +0 -13
  113. package/modules/android-compose/stack-profile.yaml +0 -57
  114. package/modules/angular/architecture-snippets/component-patterns.md +0 -187
  115. package/modules/angular/module.yaml +0 -6
  116. package/modules/angular/stack-profile.yaml +0 -38
  117. package/modules/context-engineering/architecture-snippets/context-design.md +0 -119
  118. package/modules/context-engineering/module.yaml +0 -9
  119. package/modules/context-engineering/stack-profile.yaml +0 -61
  120. package/modules/dotnet/architecture-snippets/clean-arch.md +0 -160
  121. package/modules/dotnet/module.yaml +0 -6
  122. package/modules/dotnet/stack-profile.yaml +0 -50
  123. package/modules/flutter/module.yaml +0 -14
  124. package/modules/flutter/stack-profile.yaml +0 -59
  125. package/modules/golang/architecture-snippets/domain-layout.md +0 -283
  126. package/modules/golang/module.yaml +0 -6
  127. package/modules/golang/stack-profile.yaml +0 -40
  128. package/modules/ios-swiftui/module.yaml +0 -13
  129. package/modules/ios-swiftui/stack-profile.yaml +0 -55
  130. package/modules/java-spring/architecture-snippets/layered-arch.md +0 -201
  131. package/modules/java-spring/module.yaml +0 -15
  132. package/modules/java-spring/stack-profile.yaml +0 -28
  133. package/modules/nextjs/architecture-snippets/app-router-patterns.md +0 -269
  134. package/modules/nextjs/module.yaml +0 -14
  135. package/modules/nextjs/stack-profile.yaml +0 -74
  136. package/modules/nuxt/module.yaml +0 -14
  137. package/modules/nuxt/stack-profile.yaml +0 -58
  138. package/modules/phaser-game/architecture-snippets/phaser-scene-patterns.md +0 -646
  139. package/modules/phaser-game/module.yaml +0 -15
  140. package/modules/phaser-game/stack-profile.yaml +0 -90
  141. package/modules/php-laravel/architecture-snippets/service-repository.md +0 -302
  142. package/modules/php-laravel/module.yaml +0 -15
  143. package/modules/php-laravel/stack-profile.yaml +0 -56
  144. package/modules/qc-playwright/stack-profile.yaml +0 -66
  145. package/modules/react/architecture-snippets/hooks-query-patterns.md +0 -254
  146. package/modules/react/module.yaml +0 -14
  147. package/modules/react/stack-profile.yaml +0 -63
  148. package/modules/react-native/module.yaml +0 -14
  149. package/modules/react-native/stack-profile.yaml +0 -56
  150. package/modules/vue/module.yaml +0 -14
  151. package/modules/vue/stack-profile.yaml +0 -65
  152. package/rules/data-protection.md +0 -80
  153. package/rules/workflow.md +0 -99
  154. package/skills/code/SKILL.md +0 -19
  155. package/skills/code/SKILL.tmpl +0 -19
  156. package/skills/debug/SKILL.md +0 -19
  157. package/skills/debug/SKILL.tmpl +0 -19
  158. package/skills/design-spec/SKILL.md +0 -11
  159. package/skills/design-spec/SKILL.tmpl +0 -11
  160. package/skills/discovery/SKILL.md +0 -14
  161. package/skills/discovery/SKILL.tmpl +0 -14
  162. package/skills/prd/SKILL.md +0 -19
  163. package/skills/prd/SKILL.tmpl +0 -19
  164. package/skills/qc/qa-analyst/DOC_GAPS.template.md +0 -63
  165. package/skills/qc/qa-analyst/acceptance-criteria.md +0 -60
  166. package/skills/qc/qa-analyst/business-rules.md +0 -59
  167. package/skills/qc/qa-analyst/data-flow.md +0 -64
  168. package/skills/qc/qa-analyst/spec-breakdown.md +0 -61
  169. package/skills/qc/qa-designer/e2e/journey.md +0 -41
  170. package/skills/qc/qa-designer/exploratory/charter.md +0 -68
  171. package/skills/qc/qa-designer/exploratory/explore-to-functional.md +0 -43
  172. package/skills/qc/qa-designer/functional/api.md +0 -45
  173. package/skills/qc/qa-designer/functional/gui-feature.md +0 -46
  174. package/skills/qc/qa-designer/functional/gui-screen.md +0 -52
  175. package/skills/qc/qa-designer/integration/api.md +0 -42
  176. package/skills/qc/qa-designer/integration/db.md +0 -39
  177. package/skills/qc/qa-designer/integration/gui.md +0 -40
  178. package/skills/qc/qa-designer/integration/kafka.md +0 -40
  179. package/skills/qc/qa-designer/non-functional.md +0 -40
  180. package/skills/qc/qa-planner/test-plan.md +0 -120
  181. package/skills/qc/qa-reviewer/script/e2e.md +0 -87
  182. package/skills/qc/qa-reviewer/script/exploratory.md +0 -45
  183. package/skills/qc/qa-reviewer/script/functional.md +0 -101
  184. package/skills/qc/qa-reviewer/script/integration.md +0 -91
  185. package/skills/qc/qa-reviewer/script/non-functional.md +0 -126
  186. package/skills/qc/qa-reviewer/test-case/e2e.md +0 -73
  187. package/skills/qc/qa-reviewer/test-case/exploratory.md +0 -43
  188. package/skills/qc/qa-reviewer/test-case/functional.md +0 -76
  189. package/skills/qc/qa-reviewer/test-case/integration.md +0 -69
  190. package/skills/qc/qa-reviewer/test-case/non-functional.md +0 -73
  191. package/skills/qc/qa-runner/e2e.md +0 -49
  192. package/skills/qc/qa-runner/exploratory/session.md +0 -36
  193. package/skills/qc/qa-runner/functional/api.md +0 -35
  194. package/skills/qc/qa-runner/functional/gui-feature.md +0 -51
  195. package/skills/qc/qa-runner/functional/gui-screen.md +0 -55
  196. package/skills/qc/qa-runner/integration.md +0 -47
  197. package/skills/qc/qa-runner/non-functional.md +0 -49
  198. package/skills/qc/qa-runner/report/report.md +0 -37
  199. package/skills/setup-ai-first/SKILL.md +0 -19
  200. package/skills/setup-ai-first/SKILL.tmpl +0 -19
  201. package/skills/spec/SKILL.md +0 -19
  202. package/skills/spec/SKILL.tmpl +0 -19
  203. package/skills/test/SKILL.md +0 -18
  204. package/skills/test/SKILL.tmpl +0 -18
  205. package/steps/business-language.md +0 -56
  206. package/steps/capture-lesson.md +0 -112
  207. package/steps/context-loader.md +0 -406
  208. package/steps/gate.md +0 -151
  209. package/steps/report-footer.md +0 -125
  210. package/steps/review-fanout.md +0 -159
  211. package/steps/spawn-agent.md +0 -129
  212. package/steps/trace-mirror.md +0 -53
  213. package/templates/README.md +0 -70
  214. package/templates/architecture.template.md +0 -394
  215. package/templates/ci/trace-gate.yml +0 -146
  216. package/templates/design-spec.template.md +0 -217
  217. package/templates/feature.template +0 -123
  218. package/templates/hooks/pre-push +0 -61
  219. package/templates/platform-guide.template.md +0 -145
  220. package/templates/prd.template.md +0 -283
  221. package/templates/product-definition.template.md +0 -188
  222. package/templates/project-context.yaml +0 -212
  223. package/templates/tech-design.template.md +0 -490
@@ -1,281 +0,0 @@
1
- # /refine-prd — Phân tích PRD qua 3 lăng kính review
2
-
3
- ## Gate
4
-
5
- *Checkpoint: **chặn CỨNG** — --resume áp findings trực tiếp vào PRD. `--yes` KHÔNG bỏ qua được (gate Bước 3a).*
6
-
7
- {{include:steps/gate.md}}
8
-
9
- *Lưu ý: Với lệnh này, target file ở Bước 1 là một file PRD (`{TICKET-ID}-{prd-slug}.md` — file `.md` duy nhất ở gốc feature folder) dưới `{paths.specs_dir}/{domain}/{prd-slug}/`. Đọc toàn bộ PRD sau khi phân giải file.*
10
-
11
- ### Bước 0-C — Resume mode routing *(riêng /refine-prd, chạy ngay sau Bước 0 của Gate)*
12
-
13
- Nếu `$ARGUMENTS` chứa `--resume`:
14
- - Tách `--resume` ra, phần còn lại là `raw_target` (file path hoặc prd-slug).
15
- - **Nạp minimal context:** đọc `.agent/project-context.yaml`, trích xuất `paths.refinement_dir`
16
- (default: `.agent/review` nếu không có hoặc file không tồn tại).
17
- - Chạy **Bước 1** với `raw_target` để phân giải `prd-slug` và `target_file`.
18
- Nếu `raw_target` rỗng → liệt kê các file `*.yaml` trong `{paths.refinement_dir}/` và hỏi user chọn findings file nào.
19
- - **Bỏ qua Bước 0-B, 2, 3** — apply findings là tác vụ cơ học, không cần model check hay load full context.
20
- - → Nhảy thẳng tới **Resume Mode** ở cuối lệnh này.
21
-
22
- Nếu `$ARGUMENTS` không chứa `--resume` → tiếp tục luồng review bình thường bên dưới.
23
-
24
- ## Context
25
- {{include:steps/context-loader.md}}
26
-
27
- ---
28
-
29
- ## Ngôn ngữ nghiệp vụ *(áp khi viết/áp fix — gồm cả Resume Mode Phase 2)*
30
- {{include:steps/business-language.md}}
31
-
32
- ---
33
-
34
- ## Quy trình Review
35
- {{include:steps/review-fanout.md}}
36
-
37
- ---
38
-
39
- ## Phân tích — 3 lăng kính (fan out cả ba, rồi hội tụ)
40
-
41
- Chạy review qua **Quy trình Review** ở trên (`steps/review-fanout.md`).
42
-
43
- **Tham số truyền vào Quy trình Review:**
44
- - `GRANULARITY = per-uc` — LUÔN fan-out theo từng UC (bỏ ngưỡng cả-file), để **ngay lần đầu đã lòi phần nhiều issue**, không dồn sang lần sau.
45
- - `CHANGED_SCOPE` — xác định theo chế độ full/delta ngay dưới đây.
46
-
47
- **Chọn full vs delta** *(mặc định: lần đầu FULL, lần sau DELTA)*:
48
- 1. Tách `--full` khỏi `$ARGUMENTS` nếu có.
49
- 2. Kiểm tra `{paths.refinement_dir}/{prd-slug}-findings.yaml`:
50
- - **Không tồn tại** (lần đầu review PRD này) → **FULL**: KHÔNG truyền `CHANGED_SCOPE`.
51
- - **Tồn tại** + có `--full` → **FULL**: bỏ qua findings cũ, không truyền `CHANGED_SCOPE` (ép quét lại toàn bộ).
52
- - **Tồn tại** + KHÔNG có `--full` → so `prd_version` trong findings cũ với `| **Version** |` của PRD hiện tại:
53
- - **Bằng nhau** (PRD chưa đổi từ lần review trước) → DỪNG, báo: `"PRD chưa đổi từ v{X} (lần review gần nhất). Không có gì để review lại — dùng --full nếu vẫn muốn quét toàn bộ."`
54
- - **Khác** → kiểm tra `applied_to_version` trong findings cũ (version mà lần `--resume` gần nhất của CHÍNH lệnh này đã bump PRD tới) để biết ai gây ra thay đổi:
55
- - **`applied_to_version` có mặt VÀ `==` version PRD hiện tại** → PRD đổi đúng bằng phần lệnh này tự áp, không actor khác động vào → **DELTA**: `CHANGED_SCOPE` = { `uc_id`/`section` của các finding `status: applied` trong findings cũ } ∪ { UC có trong PRD hiện tại nhưng chưa từng xuất hiện ở findings cũ }. Truyền `CHANGED_SCOPE` này vào Quy trình Review.
56
- - **`applied_to_version` vắng mặt HOẶC `≠` version hiện tại** → PRD đã bị sửa bởi **actor khác** (lệnh `/review-context`, sửa tay…) sau lần resume này → KHÔNG tin được phạm vi hẹp → **FULL** (KHÔNG truyền `CHANGED_SCOPE`), kèm cảnh báo: `"PRD đổi ngoài tầm theo dõi của findings (applied_to_version={A} ≠ hiện tại={C}); quét lại toàn bộ để khỏi sót UC do người/lệnh khác sửa."`
57
-
58
- **DIMENSIONS** = 3 lăng kính dưới đây — fan out một sub-agent cho mỗi lăng kính, mỗi cái quét
59
- toàn bộ PRD qua đúng lăng kính của nó:
60
-
61
- <!-- ─────────────────────────────────────────────────────────────────────────────
62
- LĂNG KÍNH BỊ TẮT (DISABLED — KHÔNG dùng, KHÔNG fan-out agent cho lăng kính này):
63
-
64
- - **Lăng kính QA (tầng nghiệm thu)**: AC có nêu **outcome quan sát/kiểm được** chưa? — **KHÔNG** hỏi "AC đủ chi tiết chưa" (câu đó kéo cơ chế vào AC). Chi tiết cơ chế (số lần retry, timeout, tên/chủ cờ, nhánh lỗi vụn) thuộc **BR/BL**: nếu gap là cơ chế → suggestion phải **route sang BR/BL + AC ref**, KHÔNG phình AC. AC có lặp lại nội dung BR (trùng tầng) không → nếu có, đề xuất làm mỏng AC.
65
-
66
- CÁCH THÊM LẠI QA (khi cần bật lại giai đoạn sau):
67
- 1. Chuyển bullet QA ở trên ra khỏi block comment này, đặt lên đầu danh sách DIMENSIONS.
68
- 2. Đổi "3 lăng kính"/"cả ba" → "4 lăng kính"/"cả bốn" ở: dòng tiêu đề (# /refine-prd),
69
- heading "## Phân tích", câu "DIMENSIONS = N lăng kính", và steps/review-fanout.md.
70
- 3. Thêm "QA" lại vào enum của `lens:` trong FINDINGS SCHEMA + vào `by_lens`.
71
- 4. Thêm "QA" lại vào ghi chú "phán đoán DEV/SA/PO" ở LƯU Ý của schema.
72
- 5. Rebuild: node bin/build.js
73
- ───────────────────────────────────────────────────────────────────────────── -->
74
-
75
- > **Nguyên tắc chung cho DEV & SA — đọc bằng mắt kỹ thuật, VIẾT bằng lời nghiệp vụ.**
76
- > Hai lăng kính này dùng con mắt kỹ thuật để **phát hiện chỗ nghiệp vụ mô tả thiếu/mơ hồ/mâu thuẫn đến mức sẽ chặn triển khai** — mục tiêu là **làm rõ vấn đề nghiệp vụ để sau này xử lý được về mặt kỹ thuật**. **KHÔNG** đưa góc nhìn kỹ thuật vào PRD, **KHÔNG** đề xuất giải pháp/cơ chế kỹ thuật. Mọi `finding` và `suggestion` phải **thuần nghiệp vụ** (tuân Business Language Guard): mô tả *cái nghiệp vụ còn thiếu/chưa rõ* và *hỏi cần làm rõ gì*, chứ không nói *làm thế nào về kỹ thuật*.
77
-
78
- - **Lăng kính DEV (tầng cơ chế nghiệp vụ)**: Đọc luồng như một dev sắp build — BR (WHAT) + Business Logic (HOW nghiệp vụ) đã **đủ & không mơ hồ để triển khai mà không phải đoán** chưa? Soi: nhánh nghiệp vụ còn thiếu, điều kiện biên chưa nói, đường xử lý khi có lỗi/ngoại lệ bị bỏ ngỏ (vd: "khi thanh toán bị từ chối thì đơn hàng được xử lý ra sao?"), business rule mâu thuẫn hoặc mơ hồ. Diễn đạt gap bằng **ngôn ngữ nghiệp vụ + đặt câu hỏi làm rõ** — **KHÔNG** đề xuất cơ chế kỹ thuật (số lần thử lại, timeout, cấu trúc dữ liệu, API, thư viện) và **KHÔNG** nhồi chi tiết kỹ thuật vào PRD.
79
- - **Lăng kính SA (tính thông suốt & nhất quán nghiệp vụ)**: Đọc như một architect — toàn bộ luồng nghiệp vụ có **thông suốt & nhất quán** trên cả feature/domain không? Soi: tương tác nghiệp vụ giữa các UC/feature đã định nghĩa chưa, quan hệ giữa các thực thể nghiệp vụ (theo core-entities) có mâu thuẫn/bỏ trống không, trạng thái & vòng đời của đối tượng nghiệp vụ có nhất quán không, quy tắc **ai được làm gì / ai sở hữu gì** (nghiệp vụ đằng sau phân quyền) đã rõ chưa. **KHÔNG** phán về kiến trúc/security/mô hình dữ liệu kỹ thuật, **KHÔNG** đề xuất giải pháp kỹ thuật — chỉ nêu chỗ nghiệp vụ chưa nhất quán/chưa rõ bằng ngôn ngữ nghiệp vụ.
80
- - **Lăng kính PO**: Scope đã khoanh vùng? Priority rõ chưa? Success metric đã định nghĩa? Rủi ro scope creep?
81
-
82
- Vòng lặp completeness-critic (Phase 2) là cái đảm bảo file findings đầy đủ trong một lần chạy
83
- — chạy lại `/refine-prd` sẽ lòi ra **0 finding mới**. Map mỗi dimension vào field `lens`
84
- của schema dưới đây.
85
-
86
- ## File Output
87
-
88
- Suy ra tên file output từ PRD slug:
89
- - File PRD: `{paths.specs_dir}/payment/create-invoice/PAY01-create-invoice.md` → output: `{paths.refinement_dir}/create-invoice-findings.yaml`
90
- - Quy tắc: lấy tên folder cha của PRD (prd-slug), thêm hậu tố `-findings.yaml`
91
-
92
- Trước khi ghi file, xác định `recommendation` từ findings đã dedup và sắp xếp:
93
- - `by_severity.critical ≥ 1` → `BLOCKED`
94
- - `by_severity.critical = 0` và `by_severity.major ≥ 1` → `NEEDS_REVISION`
95
- - `by_severity.critical = 0` và `by_severity.major = 0` → `APPROVED_WITH_MINOR_CHANGES`
96
-
97
- Đảm bảo `{paths.refinement_dir}/` tồn tại — tạo thư mục nếu chưa có trước khi ghi file.
98
-
99
- Ghi `{paths.refinement_dir}/{prd-slug}-findings.yaml`:
100
-
101
- ```yaml
102
- prd_source: "{paths.specs_dir}/{domain}/{prd-slug}/{TICKET-ID}-{prd-slug}.md"
103
- prd_version: "{đọc | **Version** | từ metadata PRD lúc sinh findings — dùng để chọn full/delta lần chạy sau}"
104
- applied_to_version: "" # để trống lúc phân tích; --resume sẽ ghi version PRD vừa bump tới. Lần delta sau: nếu version PRD hiện tại ≠ giá trị này → PRD bị actor khác sửa → fallback FULL (xem "Chọn full vs delta")
105
- generated_at: "{ISO datetime}"
106
- status: "pending_review"
107
-
108
- findings:
109
- - id: "F001"
110
- lens: "DEV" # DEV | SA | PO
111
- severity: "major" # critical | major | minor
112
- section: "§2. Acceptance Criteria" # nhãn heading/section dạng người đọc
113
- uc_id: "{TICKET-ID}-UC{N}" # UC mà finding thuộc về; "" nếu PRD-global (scope, metrics, problem statement)
114
- quote: "{trích đoạn nguyên văn copy CHÍNH XÁC từ PRD tại vị trí lỗi, ≤120 ký tự}"
115
- finding: "{mô tả gap hoặc vấn đề}"
116
- suggestion: "{đề xuất cải thiện cụ thể, hành động được}"
117
- resolution_edge_cases: # CHỈ điền cho critical/major; minor → để [] (bỏ qua)
118
- # Phân tích bậc-hai (advisory, KHÔNG chặn): nếu áp `suggestion` này thì có thể đẻ ra
119
- # edge case / side-effect gì — path lỗi mới, va chạm với BR/UC khác, trạng thái biên,
120
- # hệ luỵ cross-section. PO đọc để cân nhắc trước khi accept; nếu muốn xử lý → tạo finding mới.
121
- - "{edge case có thể phát sinh nếu chốt phương án này}"
122
- auto_fixable: false
123
- # true = AI tự tin cao vào suggestion này; Review Board có thể hiển thị nút "quick accept"
124
- # false = cần human đọc kỹ và ghi quyết định trước khi accept
125
- # Resume Mode luôn áp dụng theo status (accepted|modified), bất kể auto_fixable.
126
- # LƯU Ý: /refine-prd CỐ Ý không có `--fix` mode (khác /review-context) — finding 3 lăng kính
127
- # là phán đoán DEV/SA/PO, phải qua người duyệt ở Board; auto_fixable ở đây CHỈ là gợi ý
128
- # quick-accept cho Board, KHÔNG để máy tự áp.
129
- status: "pending"
130
- applied_via: ""
131
- # Vòng đời finding-level (CHUNG với /review-context — Review Board đọc cả hai loại file nên enum phải khớp):
132
- # pending → finding mới, chưa được review
133
- # accepted → reviewer chấp nhận suggestion gốc → Resume sẽ apply
134
- # modified → reviewer đã sửa suggestion trong Review Board → Resume dùng suggestion đã sửa
135
- # rejected → reviewer bác bỏ, không apply
136
- # needs_discussion → hai suggestion xung đột, CHẶN --resume (bỏ qua + cảnh báo) tới khi con người quyết
137
- # deferred → cố ý hoãn (không chặn), bỏ qua lượt này, xem lại sau
138
- # applied → đã áp vào PRD; applied_via ghi cách áp: "resume" (qua --resume) — /review-context còn dùng "fix" (qua --fix)
139
-
140
- summary:
141
- total_findings: {N}
142
- by_severity: { critical: {N}, major: {N}, minor: {N} }
143
- by_lens: { DEV: {N}, SA: {N}, PO: {N} }
144
- recommendation: "APPROVED_WITH_MINOR_CHANGES | NEEDS_REVISION | BLOCKED"
145
- # Rule: critical ≥ 1 → BLOCKED
146
- # critical = 0, major ≥ 1 → NEEDS_REVISION
147
- # critical = 0, major = 0 → APPROVED_WITH_MINOR_CHANGES
148
- ```
149
-
150
- > **Field định vị (`quote` + `uc_id`) — bắt buộc cho source-jump của Review Board.**
151
- > Với mỗi finding, copy một đoạn `quote` **nguyên văn** thẳng từ PRD tại đúng chỗ
152
- > lỗi xảy ra — KHÔNG diễn giải lại; nó được so khớp với tài liệu để định vị dòng. Đặt
153
- > `uc_id` là Use Case sở hữu (hoặc `""` cho finding PRD-global). Hai field này cho phép reviewer click một
154
- > finding trong Review Board và nhảy thẳng tới đúng vị trí trong PRD nguồn.
155
-
156
- > **`resolution_edge_cases` — phân tích bậc-hai (chỉ critical/major).**
157
- > Với mỗi finding `critical`/`major`, sau khi viết `suggestion`, nghĩ tiếp: *nếu PO chốt phương án này thì
158
- > đẻ ra edge case / side-effect gì?* (path lỗi mới, va chạm BR/UC khác, trạng thái biên, hệ luỵ cross-section).
159
- > Ghi vào `resolution_edge_cases` để PO **thấy trước khi accept**. Đây là **advisory** — KHÔNG chặn, KHÔNG tự
160
- > tạo finding; PO đọc rồi quyết. Finding `minor` → để `[]`.
161
- > *(Phần phản hồi cho phương án PO **tự sửa** (`modified`) đến ở vòng sau: sau `--resume`, lần `/refine-prd`
162
- > kế chạy delta sẽ quét lại UC đã đổi + critic toàn-doc → tự lòi edge case mà phương án đó tạo ra.)*
163
-
164
- ## Report
165
-
166
- {{include:steps/report-footer.md}}
167
-
168
- Dùng footer chuẩn với hai field bổ sung `Findings` và `Review` đặt ngay sau `Status`:
169
-
170
- ```
171
- ---
172
- Status : {badge}
173
- Findings : {total} | 🔴 Critical: {N} | 🟡 Major: {N} | 🟢 Minor: {N}
174
- Review : {paths.refinement_dir}/{prd-slug}-findings.yaml
175
- Output Artifacts:
176
- created {paths.refinement_dir}/{prd-slug}-findings.yaml (findings từ 3-lens review)
177
- Pipeline : Discovery → [PRD ◀ bạn ở đây] → Design Spec → BDD → Tech Design → Code → Dev Self-Check → QC → Trace Audit
178
- Vòng review: [① phân tích ◀] → ② Review Board → ③ --resume
179
- Next : Mở trong Review Board (chuột phải vào file) → Cập nhật PRD
180
- → /review-context {prd-file} ← kiểm tra chất lượng PRD trước khi sinh BDD
181
- → khi sạch critical, PO đặt | **Status** | approved | → /generate-bdd {prd-file}
182
- ```
183
-
184
- Nếu có finding nào có `status: "needs_discussion"`, thêm warning block sau footer:
185
-
186
- ```
187
- ⚠️ {N} finding(s) cần quyết định của bạn trước khi --resume:
188
- Mở {paths.refinement_dir}/{prd-slug}-findings.yaml
189
- Với mỗi finding có status "needs_discussion":
190
- 1. Đọc cả hai phương án trong `suggestion`
191
- 2. Chọn một (hoặc viết phương án khác) vào `suggestion`
192
- 3. Đổi `status` → "accepted"
193
- Sau đó chạy: /refine-prd {prd-file} --resume
194
- ```
195
- *(Bỏ warning block này nếu không có `needs_discussion` finding nào.)*
196
-
197
- ---
198
-
199
- ## Resume Mode — Áp dụng Findings & Bump Version
200
-
201
- *Được route tới từ Bước 0-C — Resume mode routing. `prd-slug` và `target_file` đã được phân giải ở Bước 1 trước khi nhảy vào đây.*
202
-
203
- ### Phase 1 — Đọc các finding được chấp nhận
204
-
205
- 1. Kiểm tra `{paths.refinement_dir}/{prd-slug}-findings.yaml` có tồn tại không.
206
- Nếu không → báo lỗi rõ ràng và dừng:
207
- ```
208
- ❌ Không tìm thấy findings file tại {paths.refinement_dir}/{prd-slug}-findings.yaml
209
- Hãy chạy /refine-prd {prd-file} trước để tạo findings, rồi mới --resume.
210
- ```
211
- 2. Đọc file findings.
212
- 3. Gom tất cả finding có `status: "accepted"` hoặc `status: "modified"`.
213
- 4. Nếu không có finding nào được chấp nhận → báo "No accepted findings. PRD unchanged." và dừng.
214
- 5. Nếu còn finding nào có `status: "needs_discussion"` → cảnh báo (không dừng):
215
- ```
216
- ⚠️ {N} finding(s) chưa được giải quyết (needs_discussion) — sẽ bị bỏ qua lần apply này.
217
- Giải quyết chúng trong Review Board rồi chạy lại --resume để apply.
218
- ```
219
-
220
- ### Phase 2 — Áp dụng thay đổi vào PRD
221
-
222
- Với mỗi finding được chấp nhận, theo thứ tự severity (critical → major → minor):
223
- - Đi tới section PRD được chỉ định bởi `finding.section`.
224
- - Áp dụng `finding.suggestion`. Với finding `status: "modified"`, người đã sửa sẵn `finding.suggestion` trong Review Board — giá trị đã sửa đó CHÍNH LÀ bản fix cần áp dụng.
225
- - **Giữ ĐÚNG TẦNG + gọn khi áp fix (altitude):** AC = **outcome quan sát được + ref BR**, KHÔNG chứa cơ chế. Nếu fix là **chi tiết cơ chế/rule** (retry N, timeout, tên/chủ cờ, nhánh lỗi) → **ghi vào bảng BR/BL của UC** (hoặc tạo BR mới), AC chỉ **ref**; KHÔNG inline vào câu AC. Nếu fix làm rõ **≥2 điều kiện/nhánh** ở đúng tầng của nó → **tách bullet con** (mỗi ý một dòng ` - …`) hoặc **AC/BR mới**, đừng nối mệnh đề vào câu cũ. Một AC = một tiêu chí kiểm chứng; đừng để AC lặp lại nội dung BR.
226
- - **Chạy Business Language Guard trên text mới TRƯỚC khi ghi** (xem section "Ngôn ngữ nghiệp vụ"): diễn đạt lại / gỡ thuật ngữ kỹ thuật-UI để fix không tự kéo theo term kỹ thuật vào PRD.
227
- - Không thay đổi bất kỳ section nào không được tham chiếu bởi một finding được chấp nhận.
228
-
229
- ### Phase 2.5 — Cập nhật file findings
230
-
231
- Với mỗi finding đã áp dụng (status là `accepted` hoặc `modified`):
232
- - Đặt `status: "applied"` + `applied_via: "resume"` trong `{paths.refinement_dir}/{prd-slug}-findings.yaml`.
233
-
234
- Cập nhật `status: "applied"` ở **root level** của file findings (không phải `summary.status`).
235
- # Lifecycle file-level: pending_review → applied | partially_applied
236
- # partially_applied khi có finding bị rejected hoặc needs_discussion còn sót lại.
237
-
238
- ### Phase 3 — Bump version & ghi entry changelog
239
-
240
- 1. Đọc giá trị `| **Version** |` hiện tại từ bảng metadata của PRD.
241
- 2. Xác định loại bump từ tập finding được chấp nhận:
242
- - **major** (X.0 → X+1.0 kế tiếp): thêm/xoá Use Case, tái cấu trúc scope, thay đổi BR breaking.
243
- - **minor** (x.Y → x.Y+1): làm rõ, thêm edge case, tinh chỉnh wording AC/BR.
244
- 3. Tính chuỗi version mới.
245
- 4. Cập nhật metadata PRD:
246
- - `| **Version** | {new_version} |`
247
- - `| **Updated** | {today YYYY-MM-DD} |`
248
- - `| **Status** | draft |` ← reset về draft, phải được duyệt lại
249
- 5. Cập nhật `# Change Log` của PRD — **bảng phẳng 1 dòng/version, cửa sổ trượt 5 entry** (nếu gặp format cũ `### v{X}` block → chuẩn hoá sang bảng phẳng khi cập nhật):
250
- - Thêm row mới lên **đầu** bảng:
251
- ```
252
- | {new_version} | {today} | {tóm tắt — KÈM UC/AC/BR bị ảnh hưởng, vd "UC2: sửa BR5; thêm AC7"} |
253
- ```
254
- *(Nêu rõ UC/AC/BR giúp `/generate-bdd` Version Check biết scenario nào cần cập nhật.)*
255
- - Cập nhật dòng đầu section: `> Hiện tại: **v{new_version}** ({today}) · Lịch sử đầy đủ → [changelog](./changelog/{TICKET-ID}-{prd-slug}.changelog.md)`
256
- - **Rollover (giữ PRD gọn — đây là chuẩn chung, /review-context cũng theo):** nếu bảng `# Change Log` có **> 5 row** → chuyển **mọi row vượt 5** (cũ nhất) sang **đầu** bảng của file kho `{paths.specs_dir}/{domain}/{prd-slug}/changelog/{TICKET-ID}-{prd-slug}.changelog.md` (giữ thứ tự mới→cũ); PRD chỉ giữ **5 row gần nhất**. Tạo thư mục `changelog/` + file kho nếu chưa có, theo skeleton:
257
- ```
258
- # Change Log (lịch sử) — {TICKET-ID}
259
- > PRD: [../{TICKET-ID}-{prd-slug}.md](../{TICKET-ID}-{prd-slug}.md) — 5 version gần nhất nằm trong PRD; đây là phần cũ hơn.
260
-
261
- | Version | Date | Changes |
262
- |---------|------|---------|
263
- ```
264
- 6. Ghi `applied_to_version: "{new_version}"` ở **root level** của findings — đóng dấu "PRD đổi tới version này là do lệnh này áp", để lần review delta sau phân biệt được thay đổi của chính mình với thay đổi do actor khác (xem "Chọn full vs delta").
265
-
266
- ### Phase 4 — Report
267
-
268
- ```
269
- /refine-prd Đã áp dụng — {PRD name}
270
- Version : {old_version} → {new_version}
271
- Applied : {N} findings ({critical} critical, {major} major, {minor} minor)
272
- Changes :
273
- - {change 1}
274
- - {change 2}
275
-
276
- 💡 Vừa áp {N} phương án (gồm bản PO tự sửa) → chạy lại /refine-prd {prd-file}
277
- (delta tự động) để lòi edge case mà các phương án vừa áp có thể tạo ra.
278
- ⚠️ Status đã reset về draft (sửa rồi phải duyệt lại). BDD có thể đã lỗi thời. Chạy:
279
- /review-context {prd-file} ← kiểm tra chất lượng PRD trước
280
- → PO đặt | **Status** | approved | khi hài lòng → /generate-bdd {prd-file}
281
- ```
@@ -1,305 +0,0 @@
1
- # /report-bug — File một Bug có trace-spec (cho Tester & QC)
2
-
3
- Dành cho **tester và QC** — gồm cả **product-gap** lòi ra từ pipeline `/qc-*`
4
- (`/qc-run-test` FAIL phân loại product-gap, hoặc một spec-defect blocker `DOC_GAPS` từ
5
- `/qc-analyze`). Sinh một bug report có cấu trúc với đầy đủ spec context, phân loại layer
6
- khả nghi, và lưu lại để handoff cho team dev.
7
-
8
- **READ-ONLY trên spec và code.** Lệnh này không bao giờ sửa PRD, BDD, tech-docs, hay source.
9
- Fix là việc của dev (`/fix-bug`); viết scenario là `/propose-scenario`.
10
-
11
- Usage: `/report-bug {UC-ID hoặc TICKET} {mô tả ngắn}`
12
- Ví dụ: `/report-bug FT-001 tài khoản khoá sau 6 lần login fail, spec nói 5`
13
-
14
- ## Gate
15
- # Gate — Quy trình vào chuẩn cho mọi lệnh
16
-
17
- Mọi lệnh PHẢI chạy gate này trước khi thực thi phần logic riêng của nó.
18
-
19
- ## Bước 0 — Kiểm tra chế độ Sub-Agent
20
-
21
- Trước tiên, kiểm tra xem `$ARGUMENTS` có phải là payload JSON từ một orchestrator hay không:
22
-
23
- 1. Thử parse `$ARGUMENTS` dưới dạng JSON.
24
- 2. Nếu parse thành công **và** chứa `"_agent_mode": true`:
25
- - **Bỏ qua hoàn toàn Bước 1, 2 và 3 của Gate này.**
26
- - Đặt target file = `payload.target_file`
27
- - Đặt loaded context = `payload.context` (KHÔNG chạy context-loader.md)
28
- - Đặt phạm vi UC = `payload.uc_id` (chỉ xử lý UC này)
29
- - Đặt line range = `payload.uc_section` (chỉ đọc đúng section đó của PRD)
30
- - Đặt dimension = `payload.dimension` nếu có (lệnh review per-UC: chỉ review đúng lăng kính này)
31
- - Đi thẳng tới phần logic riêng của lệnh.
32
- 3. Nếu `$ARGUMENTS` không phải JSON hoặc không có `_agent_mode` → tiếp tục sang Bước 1 (chế độ thường).
33
-
34
- ## Bước 0-B — Ghi nhận Model *(KHÔNG chặn)*
35
-
36
- *Bỏ qua nếu `_agent_mode: true` (sub-agent — orchestrator đã ghi nhận rồi).*
37
-
38
- Ghi lại **model mà bạn — agent đang chạy lệnh này — thực sự đang dùng**, rồi mang nó vào
39
- dòng `Model:` của report cuối (xem `report-footer`). Nếu bạn biết mình **không** phải một
40
- model Opus, gắn thêm cảnh báo ngay ở dòng đó.
41
-
42
- **KHÔNG hỏi người dùng. KHÔNG chờ. KHÔNG dừng.**
43
-
44
- > **Vì sao bước này từng là prompt chặn, và vì sao bỏ (GAPS-v3 G41):** bản cũ hiện khối
45
- > `⚙️ MODEL CHECK` rồi chờ `Y/S/N`. Ba vấn đề cùng chỉ một hướng:
46
- > **(1)** nó hỏi người dùng thứ mà **agent đã biết chính xác**;
47
- > **(2)** câu trả lời **không kiểm chứng được** — gõ `Y` xong vẫn đang chạy Haiku thì không
48
- > gì phát hiện;
49
- > **(3)** **cả `Y` lẫn `S` đều đi tiếp** — cách duy nhất để nó dừng là tự nguyện gõ `N`.
50
- > Tức nó **không chặn được ai**, mà tốn một lần chặn ở **mọi** lệnh. Một feature đi hết
51
- > pipeline dùng 20 lệnh; 30/32 lệnh chạy gate. Hai mươi lần bấm cho một tín hiệu tự-khai
52
- > không kiểm chứng được — và chính cái giá đó làm mòn CHECKPOINT ở Bước 3, cổng có giá trị thật.
53
- >
54
- > Khai báo trong report **mạnh hơn** hỏi: đúng nguồn (agent, không phải người), và nằm
55
- > **cạnh kết quả** để cân nhắc, thay vì nằm trước khi có kết quả để bấm cho xong.
56
-
57
- **Vẫn khuyến nghị Opus:** phân tích spec, review kiến trúc và sinh code đòi hỏi suy luận sâu;
58
- model nhỏ hơn dễ bỏ sót edge case và vi phạm kiến trúc. Đổi: `/model` → chọn Opus.
59
-
60
- ## Bước 1 — Xác định Target File
61
-
62
- 0. **Tách cờ trước khi resolve target.** `$ARGUMENTS` có thể lẫn các `--flag` (vd `--phase=integration`, `--comment`, `--fix`). **Loại bỏ mọi token bắt đầu bằng `--`** ra khỏi phần dùng để tìm target — chỉ giữ phần path/UC-ID/ticket. (Các flag đó do phần logic riêng của lệnh parse ở bước sau, KHÔNG phải tên file.)
63
- 1. Nếu `$ARGUMENTS` (đã tách cờ) được cung cấp và trỏ tới một file tồn tại → dùng trực tiếp làm target.
64
- 2. Nếu `$ARGUMENTS` là một **UC-ID / ticket ID / tên rút gọn** (không có path) → phân giải thành file bằng cách glob theo bố cục feature-package. `{prd-slug}` lúc này **chưa biết**, nên dùng wildcard `*` cho segment đó, và `**` đệ quy dưới `bdd/` để phủ hết các thư mục con theo platform (`bdd/web/`, `bdd/app/`, `bdd/system/`):
65
- - **Lệnh BDD** (target là `.feature`): `{specs_dir}/{domain}/*/bdd/**/{UC-ID}*.feature` — hoặc `{specs_dir}/*/*/bdd/**/{UC-ID}*.feature` nếu domain cũng chưa biết. Nếu lệnh ngụ ý một platform/scope cụ thể (vd: system tech-doc cần BDD `system/`), ưu tiên kết quả trong thư mục con platform đó.
66
- - **Lệnh PRD** (target là file PRD `{TICKET-ID}-{prd-slug}.md` — file `.md` duy nhất ở gốc feature folder, cạnh `bdd/`): `{specs_dir}/{domain}/*/{TICKET-ID}*.md` nếu biết TICKET-ID; nếu không, `{specs_dir}/{domain}/*/*.md` (khớp feature folder có id tương ứng), hoặc `{specs_dir}/*/*/*.md` nếu domain cũng chưa biết. *(Glob `*/*.md` ở cấp gốc folder chỉ khớp PRD — tech-docs/design-spec `.md` nằm sâu hơn trong thư mục con.)*
67
- - **Lệnh tech-docs** — target là tech-doc **gộp cấp PRD** `{TICKET-ID}-tech-design.md` (MỘT doc phủ nhiều UC; danh sách UC nằm ở `@trace.ucs`). Vì tên file mang `{TICKET-ID}` chứ **không** mang `{UC-ID}`, phải tách trước khi glob:
68
- - `$ARGUMENTS` là **UC-ID** (`{TICKET-ID}-UC{N}`) → lấy `{TICKET-ID}` = phần **trước** `-UC`, rồi glob `{specs_dir}/{domain}/*/tech-docs/{TICKET-ID}-tech-design.md`.
69
- - `$ARGUMENTS` là **TICKET-ID** → glob trực tiếp như trên.
70
- - Chưa biết domain → `{specs_dir}/*/*/tech-docs/{TICKET-ID}-tech-design.md`.
71
- - Vẫn không khớp → glob rộng `{specs_dir}/*/*/tech-docs/*tech-design*.md` rồi liệt kê để người dùng chọn.
72
- *(Đừng glob `{UC-ID}*-tech-design*.md` — nó nở thành `FT-001-UC1*-tech-design*.md` và **không bao giờ** khớp `FT-001-tech-design.md`.)*
73
- - **Lệnh design-spec**: `{specs_dir}/{domain}/*/design-spec/{TICKET-ID}*.md`.
74
-
75
- Khi một file khớp: đặt nó làm target **và** ghi lại `domain` + `prd_slug` từ path của nó (theo quy tắc trích xuất trong `context-loader.md` Bước 1 — `prd_slug` = segment đầu tiên sau `{specs_dir}/{domain}/`). Mọi path mà lệnh đọc/ghi về sau (BDD/tech-docs/design-spec/trace cùng cấp) đều dùng **`prd_slug` đã phân giải đó**, nên tất cả artifact nằm chung một feature package. Nếu nhiều file khớp (vd: nhiều platform), chọn theo platform/scope của lệnh hoặc liệt kê ra và hỏi.
76
- 3. Nếu `$ARGUMENTS` rỗng hoặc không tìm thấy file khớp:
77
- - Liệt kê các file trong thư mục liên quan của lệnh này (vd: `specs/*/*/*.md` — file PRD ở gốc mỗi feature folder — cho lệnh PRD, `specs/*/*/bdd/**/*.feature` cho lệnh BDD).
78
- - Hiển thị danh sách cho người dùng và hỏi: "Bạn muốn làm việc với file nào? (Nhập số thứ tự hoặc tên file)"
79
- - Chờ người dùng chọn rồi mới tiếp tục.
80
-
81
- ## Bước 2 — Chạy Context Loader
82
-
83
- Nạp toàn bộ context của dự án bằng cách làm theo quy trình trong `steps/context-loader.md`.
84
- Lưu toàn bộ context đã nạp vào bộ nhớ để dùng xuyên suốt phiên làm việc của lệnh.
85
-
86
- ## Bước 3 — CHECKPOINT
87
-
88
- *Bỏ qua nếu `_agent_mode: true`.*
89
-
90
- ### 3a — Lệnh này có phải chặn không?
91
-
92
- | Mức | Lệnh nào | `--yes` bỏ qua được? |
93
- |---|---|:---:|
94
- | **Không chặn** | Lệnh read-only: `/review-code` · `/validate-traces` · `/debug` · `/review-context` · `/review-tech-docs` | — (vốn không có) |
95
- | **Chặn thường** | Mọi lệnh sinh/sửa artifact | ✅ |
96
- | **Chặn CỨNG** | Ghi đè file đã tồn tại · `--resume` áp findings · migrate · prune | ❌ **không bao giờ** |
97
-
98
- `--yes` trong `$ARGUMENTS` → bỏ qua CHECKPOINT mức *chặn thường*. (Bước 1 đã tách mọi token
99
- `--` khỏi phần resolve target, nên cờ này không ảnh hưởng việc tìm file.) Mở đường chạy
100
- headless: `claude -p "/generate-code UC1 --yes"`.
101
-
102
- > **KHÔNG tự suy mức từ bảng này.** Mỗi lệnh **tự khai** mức của nó ở một dòng `*Checkpoint: …*`
103
- > ngay dưới `## Gate` của chính nó — đọc dòng đó, đừng suy diễn. Bảng trên chỉ giải thích ba mức
104
- > **nghĩa là gì**.
105
- > Nguồn máy đọc: `bin/trace-schema.json` → `gate.checkpoint_levels`; `self-check` **R11** fail
106
- > build nếu nhãn trong file lệnh lệch với schema, hoặc nếu một lệnh `hard`/`none` thiếu nhãn.
107
- > *(Lệnh không có dòng nào = mức **chặn thường**, mặc định.)*
108
-
109
- > **Mức *không chặn* là thực thi đúng miễn trừ mà `rules/workflow.md` đã cấp từ trước** —
110
- > trước G41 file đó viết *"read-only commands may skip CHECKPOINT"* còn gate thì luôn đòi.
111
- > Hai file cùng được nạp vào mọi lệnh mà nói ngược nhau; agent theo cái nào là tuỳ lúc.
112
-
113
- ### 3b — In gì
114
-
115
- **KHÔNG lặp lại những gì `[CTX LOADED]` vừa in.** Recap của context-loader (Bước 7) đã hiện
116
- Stack · Platform · Layers · CLAUDE.md · Dict · Entities · Lessons · Service · Status ngay phía
117
- trên. CHECKPOINT chỉ thêm **một** thông tin mới là `Target`.
118
-
119
- **Mọi thứ sạch** — recap báo `Status: FULL`, không cờ nào bật → in đúng hai dòng:
120
-
121
- ```
122
- CHECKPOINT — Target: {resolved file path}
123
- Tiếp tục? (Y/N)
124
- ```
125
-
126
- **Có bất thường** → thêm một dòng cho **mỗi** trạng thái, nặng nhất lên đầu:
127
-
128
- ```
129
- CHECKPOINT
130
- 🔴 Service : unresolved — {lý do context-loader đã ghi}
131
- ⚠️ CLAUDE.md: service overlay THIẾU — dùng root (code sinh ra có thể sai stack)
132
- ⚠️ Target : resolve bằng wildcard — {n} file khớp, chọn {file}
133
- ⚠️ Module : not configured — code sinh ra sẽ dùng default
134
- Status : PARTIAL — thiếu: {danh sách}
135
- Target : {resolved file path}
136
- Tiếp tục? (Y/N)
137
- ```
138
-
139
- ### 3c — Cờ nào bật, cờ nào KHÔNG
140
-
141
- Mỗi dòng ⚠️/🔴 phải ứng với một trạng thái **context-loader đã tính rồi** — không phát minh
142
- điều kiện mới, chỉ mang thứ đang bị giấu lên chỗ người dùng phải quyết định:
143
-
144
- | Bật cờ khi | Nguồn | Mức |
145
- |---|---|:---:|
146
- | `active_service = unresolved` | context-loader Bước 2b/2c/Fallback | 🔴 |
147
- | `Status = MINIMAL` | recap Bước 7 | 🔴 |
148
- | `Status = PARTIAL` | recap Bước 7 | ⚠️ |
149
- | CLAUDE.md thiếu, hoặc service overlay thiếu | context-loader Bước 3 | ⚠️ |
150
- | Target resolve qua wildcard, hoặc nhiều file khớp mà lệnh tự chọn | Bước 1 ở trên | ⚠️ |
151
- | `module` không cấu hình | recap Bước 7 | ⚠️ |
152
-
153
- **KHÔNG bật cờ cho:** `Lessons: chưa có` · `Dict: missing` · `Entities: missing`. Đó là
154
- *"dự án chưa điền"*, không phải *"có gì đó sai"* — chúng ở lại trong recap.
155
-
156
- > **Nguyên tắc một câu:** cờ dành cho thứ **framework không chắc chắn hoặc đã phải đoán**,
157
- > không dành cho thứ **người dùng chưa làm**. Đẩy hết mọi thứ lên thì CHECKPOINT lại đầy như
158
- > cũ, và ta quay về đúng chỗ xuất phát: một cổng luôn giống nhau thì bị lướt qua.
159
-
160
- ### 3d — Chờ trả lời
161
-
162
- - "Y" → tiếp tục sang các bước riêng của lệnh.
163
- - "N" → dừng, hỏi người dùng muốn thay đổi gì.
164
- - Có `--yes` và mức *chặn thường* → coi như "Y", **nhưng vẫn IN khối CHECKPOINT** nếu có cờ
165
- 🔴/⚠️ (không chặn ≠ không báo — người đọc log sau này vẫn cần thấy).
166
-
167
-
168
- *Lưu ý: Với lệnh này, target ở Bước 1 là một UC-ID / TICKET từ `$ARGUMENTS`. Phân giải file `.feature` khớp nếu có; nếu không, tiếp tục — bug vẫn file được.*
169
-
170
- ## Context
171
- **BẮT BUỘC — đọc `.agent/steps/context-loader.md` và thực thi TOÀN BỘ quy trình trong đó**,
172
- rồi mới tiếp tục phần bên dưới.
173
-
174
- Bỏ qua bước này thì `{paths.*}`, `{tech_stack.*}`, `{conventions.*}`, guardrail từ
175
- `project-lessons`, và routing service (chế độ umbrella) đều **chưa được phân giải** — mọi
176
- placeholder bên dưới sẽ rỗng và lệnh sẽ đọc/ghi sai chỗ.
177
-
178
- ---
179
-
180
- ## Step 1 — Phân giải Spec Context
181
-
182
- Định vị chuỗi spec cho UC/TICKET của bug. Ưu tiên `spec-manifest.yaml` (setup umbrella/tester) nếu có; else dùng `paths` đã phân giải.
183
-
184
- Thu thập và lưu:
185
- - `prd_path` + `Version` hiện tại của PRD
186
- - `bdd_path` (file `.feature` của UC này) + tiêu đề **scenario fail** cụ thể, nếu có cái khớp
187
- - `tech_doc_path` (nếu có)
188
- - `service` / domain
189
-
190
- Nếu không có scenario `.feature` nào khớp behavior báo cáo → set `coverage_gap = true` (behavior chưa được BDD test).
191
-
192
- ## Step 2 — Thu thập chi tiết Bug
193
-
194
- Nếu chưa có trong `$ARGUMENTS`, hỏi tester (một prompt gọn):
195
-
196
- ```
197
- File một bug — trả lời những gì bạn biết:
198
- 1. Xảy ra ở đâu? (endpoint / screen / flow)
199
- 2. Các bước tái hiện?
200
- 3. Expected (theo spec) vs Actual?
201
- 4. Error / log / status code?
202
- 5. Environment? (staging / prod / local)
203
- ```
204
-
205
- ## Step 3 — Xác định AC bị vi phạm
206
-
207
- Đọc PRD acceptance criteria cho UC này. Khớp bug với **AC** cụ thể nó vi phạm
208
- (vd `AC3: "5 lần login fail → khoá 30m"`). Trích nguyên văn. Nếu behavior không map AC nào →
209
- ghi "No AC covers this" và coi như một PRD gap khả dĩ.
210
-
211
- ## Step 4 — Phân loại Layer khả nghi (BUG_FLOW)
212
-
213
- Áp dụng bảng quyết định BUG_FLOW để gợi ý root cause khả năng ở đâu — cái này route bug:
214
-
215
- | Nếu… | Layer khả nghi | Route tới |
216
- |------|-------------|----------|
217
- | Code mâu thuẫn một BDD scenario | **Code bug** (Case 1) | Dev → `/fix-bug` |
218
- | BDD scenario mâu thuẫn PRD AC | **BDD bug** (Case 2) | Dev/PO fix BDD |
219
- | PRD AC mơ hồ / im lặng | **PRD ambiguity** (Case 3) | PO làm rõ PRD |
220
- | Behavior đúng nhưng **không scenario nào phủ** | **BDD coverage gap** | Tester → `/propose-scenario` |
221
- | UI ≠ Design Spec | **Design Spec bug** (Case 5) | Dev/Designer |
222
- | Chỉ Env / data | **Environment** (Case 6) | DevOps |
223
-
224
- Nêu phân loại như một gợi ý (dev confirm trong `/fix-bug`).
225
-
226
- ## Step 5 — Ghi Report
227
-
228
- Gán `BUG-{today YYYYMMDD}-{NN}` (NN = sequence kế tiếp trong các report có sẵn).
229
- Ghi vào `{paths.bug_reports_dir}/{BUG-ID}.md` (phân giải về `{spec_source}/feedback/bug-reports/` ở umbrella mode; tạo dir nếu cần) theo cấu trúc trong Output block, và cũng in nó ra để dán vào Jira/Slack.
230
-
231
- ## Step 5.5 — Backfill trace (link pending-view)
232
-
233
- Nếu Step 1 khớp một scenario fail cụ thể `{UC-ID}-SC{N}`, xác định **sổ platform** cần cập nhật (trace tách theo platform: `{UC-ID}-{platform}.tsv`):
234
- - `{platform}` = platform mà bug xảy ra (từ ngữ cảnh test/feature fail — `web`/`app`/`system`). Nếu biết → dùng thẳng.
235
- - Nếu **không rõ platform**: glob `{paths.trace_dir}/{domain}/{prd-slug}/{UC-ID}-*.tsv` tìm sổ nào có `sc_id` = `{UC-ID}-SC{N}`. Khớp đúng **một** sổ → dùng nó. Khớp **nhiều** sổ (cùng số SC ở nhiều platform, là 2 scenario khác nhau) → **hỏi tester bug thuộc platform nào** rồi mới ghi.
236
-
237
- Nếu tìm được đúng row trong sổ `{paths.trace_dir}/{domain}/{prd-slug}/{UC-ID}-{platform}.tsv`, cập nhật **chỉ** row đó để view "waiting-on" của PO/PM trỏ
238
- tới bug này — giữ nguyên mọi cột khác (gồm `qc_status`):
239
- - `qc_blocked_by` = `{BUG-ID}`
240
- - `qc_owner` = `dev` nếu layer khả nghi ∈ {Code, BDD, Design Spec, Env} · `po` nếu layer khả nghi = PRD ambiguity
241
-
242
- Skip âm thầm nếu không SC nào khớp hoặc không có file/row trace (bug vẫn file được). Đây là
243
- lần ghi duy nhất lệnh này làm tới operational state — nó vẫn **không bao giờ** sửa PRD/BDD/code.
244
-
245
- ## Step 6 — Handoff (để PO/Dev thực sự thấy)
246
-
247
- Report chỉ tới PO/Dev nếu được **commit và push lên spec repo dùng chung**. File local là dead drop.
248
-
249
- Xác định repo sở hữu `{paths.bug_reports_dir}`:
250
- - Umbrella mode → spec submodule tại `{spec_source}`
251
- - Single-service → repo hiện tại
252
-
253
- Rồi commit + push **repo đó**:
254
-
255
- ```bash
256
- cd {spec_source} # umbrella: spec submodule; single-service: bỏ
257
- git add feedback/bug-reports/{BUG-ID}.md
258
- git commit -m "qa(bug): {BUG-ID} — {short description}"
259
- git push # → PO/Dev thấy nó ở lần /sync tiếp theo
260
- ```
261
-
262
- - Nếu tester không có quyền push spec repo → mở PR / MR thay vì, hoặc đưa file cho người sở hữu. In fallback này.
263
- - In chính xác các lệnh nếu bạn không tự chạy chúng.
264
-
265
- > PO/Dev được thông báo qua routine bình thường: `/sync` liệt kê các bug report vừa pull về.
266
-
267
- ---
268
-
269
- ## Output
270
-
271
- **Đọc `.agent/steps/report-footer.md`** và áp đúng khuôn footer trong đó (Status Badge ·
272
- Output Artifacts · Next) cho report cuối, kèm khối bên dưới.
273
-
274
- ```
275
- 🐞 {BUG-ID} → {paths.bug_reports_dir}/{BUG-ID}.md (trong spec repo dùng chung)
276
-
277
- Feature : {UC-ID} — {feature name} | Service: {service} | Severity: {Critical|Major|Minor}
278
- State : 🟢 Open (lifecycle: Open → Fixed → Closed — set `Fixed` bởi /fix-bug, `Closed` sau khi /qc-run-test re-verify pass)
279
-
280
- Spec context
281
- PRD : {prd_path} (v{prd_version})
282
- BDD : {bdd_path} → Scenario: "{scenario title}" {hoặc: ⚠️ no scenario covers this}
283
- Tech Doc : {tech_doc_path}
284
-
285
- AC bị vi phạm
286
- {AC-N}: "{AC text}"
287
-
288
- Expected (theo spec) : {expected}
289
- Actual : {actual}
290
- Steps to reproduce : {1..N}
291
- Environment : {env}
292
-
293
- Layer khả nghi : {Code | BDD | PRD | Design Spec | Env} → {route}
294
-
295
- Handoff : {✅ committed + pushed to spec repo | ⚠️ chạy git command ở trên / mở PR}
296
-
297
- ---
298
- Status : ✅ Complete (read-only trên specs/code — chỉ ghi file feedback)
299
- Output Artifacts: created {paths.bug_reports_dir}/{BUG-ID}.md (pushed to shared spec repo)
300
- Next :
301
- - PO/Dev sẽ thấy nó ở lần /sync tiếp theo
302
- - Code bug → dev chạy /fix-bug {BUG-ID}
303
- - PRD ambiguity → PO (BUG_FLOW Case 3)
304
- - Coverage gap → /propose-scenario {UC-ID}
305
- ```