@educa-corp/sdd-framework 0.6.0 → 0.7.0
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.
- package/bin/gate-trace.js +25 -2
- package/bin/index.js +32 -5
- package/bin/lint-trace.js +41 -0
- package/bin/self-check.js +430 -3
- package/bin/trace-schema.json +391 -30
- package/core/FRAMEWORK_VERSION +1 -1
- package/{commands/extend-prd.md → core/commands/amend-prd.md} +205 -173
- package/core/commands/dev-run-test.md +47 -9
- package/core/commands/extend-prd.md +39 -12
- package/core/commands/generate-bdd.md +43 -4
- package/core/commands/generate-code.md +33 -0
- package/core/commands/generate-tech-docs.md +34 -2
- package/core/commands/qc-run-test.md +29 -3
- package/core/commands/refine-prd.md +13 -2
- package/core/commands/review-context.md +43 -8
- package/core/commands/sync.md +105 -1
- package/core/commands/validate-traces.md +284 -11
- package/core/rules/workflow.md +34 -0
- package/core/steps/context-loader.md +26 -5
- package/core/templates/feature.template +1 -1
- package/docs/02-concepts/architecture.md +36 -0
- package/docs/04-reference/commands.md +148 -134
- package/docs/04-reference/trace-schema.md +39 -0
- package/docs/explain/02b-extend-prd.md +1 -1
- package/docs/explain/02c-amend-prd.md +152 -0
- package/docs/explain/28-sync.md +25 -0
- package/docs/explain/README.md +136 -135
- package/package.json +1 -8
- package/commands/debug.md +0 -529
- package/commands/debug.tmpl +0 -260
- package/commands/define-product.md +0 -438
- package/commands/define-product.tmpl +0 -225
- package/commands/dev-gen-test.md +0 -700
- package/commands/dev-gen-test.tmpl +0 -490
- package/commands/dev-run-test.md +0 -435
- package/commands/dev-run-test.tmpl +0 -225
- package/commands/dev-smoke-test.md +0 -374
- package/commands/dev-smoke-test.tmpl +0 -217
- package/commands/extend-prd.tmpl +0 -273
- package/commands/fix-bug.md +0 -519
- package/commands/fix-bug.tmpl +0 -197
- package/commands/generate-architecture.md +0 -354
- package/commands/generate-architecture.tmpl +0 -197
- package/commands/generate-bdd.md +0 -923
- package/commands/generate-bdd.tmpl +0 -590
- package/commands/generate-code.md +0 -859
- package/commands/generate-code.tmpl +0 -649
- package/commands/generate-design-spec.md +0 -737
- package/commands/generate-design-spec.tmpl +0 -524
- package/commands/generate-prd.md +0 -722
- package/commands/generate-prd.tmpl +0 -226
- package/commands/generate-spec-manifest.md +0 -321
- package/commands/generate-spec-manifest.tmpl +0 -164
- package/commands/generate-tech-docs.md +0 -920
- package/commands/generate-tech-docs.tmpl +0 -273
- package/commands/learn.md +0 -399
- package/commands/learn.tmpl +0 -130
- package/commands/map-testids.md +0 -238
- package/commands/map-testids.tmpl +0 -81
- package/commands/propose-scenario.md +0 -359
- package/commands/propose-scenario.tmpl +0 -202
- package/commands/qc-analyze.md +0 -269
- package/commands/qc-analyze.tmpl +0 -112
- package/commands/qc-design-test.md +0 -226
- package/commands/qc-design-test.tmpl +0 -69
- package/commands/qc-plan.md +0 -206
- package/commands/qc-plan.tmpl +0 -49
- package/commands/qc-report.md +0 -217
- package/commands/qc-report.tmpl +0 -60
- package/commands/qc-review.md +0 -210
- package/commands/qc-review.tmpl +0 -53
- package/commands/qc-run-test.md +0 -326
- package/commands/qc-run-test.tmpl +0 -116
- package/commands/refine-prd.md +0 -653
- package/commands/refine-prd.tmpl +0 -281
- package/commands/report-bug.md +0 -305
- package/commands/report-bug.tmpl +0 -148
- package/commands/review-code.md +0 -415
- package/commands/review-code.tmpl +0 -146
- package/commands/review-context.md +0 -902
- package/commands/review-context.tmpl +0 -530
- package/commands/review-tech-docs.md +0 -561
- package/commands/review-tech-docs.tmpl +0 -404
- package/commands/setup-ai-first.md +0 -602
- package/commands/setup-ai-first.tmpl +0 -450
- package/commands/sync.md +0 -430
- package/commands/sync.tmpl +0 -429
- package/commands/update-framework.md +0 -203
- package/commands/update-framework.tmpl +0 -202
- package/commands/validate-traces.md +0 -1077
- package/commands/validate-traces.tmpl +0 -920
- package/hooks/data-guard.js +0 -232
- package/hooks/settings.json +0 -19
- package/modules/android-compose/module.yaml +0 -13
- package/modules/android-compose/stack-profile.yaml +0 -57
- package/modules/angular/architecture-snippets/component-patterns.md +0 -187
- package/modules/angular/module.yaml +0 -6
- package/modules/angular/stack-profile.yaml +0 -38
- package/modules/context-engineering/architecture-snippets/context-design.md +0 -119
- package/modules/context-engineering/module.yaml +0 -9
- package/modules/context-engineering/stack-profile.yaml +0 -61
- package/modules/dotnet/architecture-snippets/clean-arch.md +0 -160
- package/modules/dotnet/module.yaml +0 -6
- package/modules/dotnet/stack-profile.yaml +0 -50
- package/modules/flutter/module.yaml +0 -14
- package/modules/flutter/stack-profile.yaml +0 -59
- package/modules/golang/architecture-snippets/domain-layout.md +0 -283
- package/modules/golang/module.yaml +0 -6
- package/modules/golang/stack-profile.yaml +0 -40
- package/modules/ios-swiftui/module.yaml +0 -13
- package/modules/ios-swiftui/stack-profile.yaml +0 -55
- package/modules/java-spring/architecture-snippets/layered-arch.md +0 -201
- package/modules/java-spring/module.yaml +0 -15
- package/modules/java-spring/stack-profile.yaml +0 -28
- package/modules/nextjs/architecture-snippets/app-router-patterns.md +0 -269
- package/modules/nextjs/module.yaml +0 -14
- package/modules/nextjs/stack-profile.yaml +0 -74
- package/modules/nuxt/module.yaml +0 -14
- package/modules/nuxt/stack-profile.yaml +0 -58
- package/modules/phaser-game/architecture-snippets/phaser-scene-patterns.md +0 -646
- package/modules/phaser-game/module.yaml +0 -15
- package/modules/phaser-game/stack-profile.yaml +0 -90
- package/modules/php-laravel/architecture-snippets/service-repository.md +0 -302
- package/modules/php-laravel/module.yaml +0 -15
- package/modules/php-laravel/stack-profile.yaml +0 -56
- package/modules/qc-playwright/stack-profile.yaml +0 -66
- package/modules/react/architecture-snippets/hooks-query-patterns.md +0 -254
- package/modules/react/module.yaml +0 -14
- package/modules/react/stack-profile.yaml +0 -63
- package/modules/react-native/module.yaml +0 -14
- package/modules/react-native/stack-profile.yaml +0 -56
- package/modules/vue/module.yaml +0 -14
- package/modules/vue/stack-profile.yaml +0 -65
- package/rules/data-protection.md +0 -80
- package/rules/workflow.md +0 -99
- package/skills/code/SKILL.md +0 -19
- package/skills/code/SKILL.tmpl +0 -19
- package/skills/debug/SKILL.md +0 -19
- package/skills/debug/SKILL.tmpl +0 -19
- package/skills/design-spec/SKILL.md +0 -11
- package/skills/design-spec/SKILL.tmpl +0 -11
- package/skills/discovery/SKILL.md +0 -14
- package/skills/discovery/SKILL.tmpl +0 -14
- package/skills/prd/SKILL.md +0 -19
- package/skills/prd/SKILL.tmpl +0 -19
- package/skills/qc/qa-analyst/DOC_GAPS.template.md +0 -63
- package/skills/qc/qa-analyst/acceptance-criteria.md +0 -60
- package/skills/qc/qa-analyst/business-rules.md +0 -59
- package/skills/qc/qa-analyst/data-flow.md +0 -64
- package/skills/qc/qa-analyst/spec-breakdown.md +0 -61
- package/skills/qc/qa-designer/e2e/journey.md +0 -41
- package/skills/qc/qa-designer/exploratory/charter.md +0 -68
- package/skills/qc/qa-designer/exploratory/explore-to-functional.md +0 -43
- package/skills/qc/qa-designer/functional/api.md +0 -45
- package/skills/qc/qa-designer/functional/gui-feature.md +0 -46
- package/skills/qc/qa-designer/functional/gui-screen.md +0 -52
- package/skills/qc/qa-designer/integration/api.md +0 -42
- package/skills/qc/qa-designer/integration/db.md +0 -39
- package/skills/qc/qa-designer/integration/gui.md +0 -40
- package/skills/qc/qa-designer/integration/kafka.md +0 -40
- package/skills/qc/qa-designer/non-functional.md +0 -40
- package/skills/qc/qa-planner/test-plan.md +0 -120
- package/skills/qc/qa-reviewer/script/e2e.md +0 -87
- package/skills/qc/qa-reviewer/script/exploratory.md +0 -45
- package/skills/qc/qa-reviewer/script/functional.md +0 -101
- package/skills/qc/qa-reviewer/script/integration.md +0 -91
- package/skills/qc/qa-reviewer/script/non-functional.md +0 -126
- package/skills/qc/qa-reviewer/test-case/e2e.md +0 -73
- package/skills/qc/qa-reviewer/test-case/exploratory.md +0 -43
- package/skills/qc/qa-reviewer/test-case/functional.md +0 -76
- package/skills/qc/qa-reviewer/test-case/integration.md +0 -69
- package/skills/qc/qa-reviewer/test-case/non-functional.md +0 -73
- package/skills/qc/qa-runner/e2e.md +0 -49
- package/skills/qc/qa-runner/exploratory/session.md +0 -36
- package/skills/qc/qa-runner/functional/api.md +0 -35
- package/skills/qc/qa-runner/functional/gui-feature.md +0 -51
- package/skills/qc/qa-runner/functional/gui-screen.md +0 -55
- package/skills/qc/qa-runner/integration.md +0 -47
- package/skills/qc/qa-runner/non-functional.md +0 -49
- package/skills/qc/qa-runner/report/report.md +0 -37
- package/skills/setup-ai-first/SKILL.md +0 -19
- package/skills/setup-ai-first/SKILL.tmpl +0 -19
- package/skills/spec/SKILL.md +0 -19
- package/skills/spec/SKILL.tmpl +0 -19
- package/skills/test/SKILL.md +0 -18
- package/skills/test/SKILL.tmpl +0 -18
- package/steps/business-language.md +0 -56
- package/steps/capture-lesson.md +0 -112
- package/steps/context-loader.md +0 -406
- package/steps/gate.md +0 -151
- package/steps/report-footer.md +0 -125
- package/steps/review-fanout.md +0 -159
- package/steps/spawn-agent.md +0 -129
- package/steps/trace-mirror.md +0 -53
- package/templates/README.md +0 -70
- package/templates/architecture.template.md +0 -394
- package/templates/ci/trace-gate.yml +0 -146
- package/templates/design-spec.template.md +0 -217
- package/templates/feature.template +0 -123
- package/templates/hooks/pre-push +0 -61
- package/templates/platform-guide.template.md +0 -145
- package/templates/prd.template.md +0 -283
- package/templates/product-definition.template.md +0 -188
- package/templates/project-context.yaml +0 -212
- package/templates/tech-design.template.md +0 -490
package/commands/refine-prd.md
DELETED
|
@@ -1,653 +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
|
-
# Gate — Quy trình vào chuẩn cho mọi lệnh
|
|
8
|
-
|
|
9
|
-
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ó.
|
|
10
|
-
|
|
11
|
-
## Bước 0 — Kiểm tra chế độ Sub-Agent
|
|
12
|
-
|
|
13
|
-
Trước tiên, kiểm tra xem `$ARGUMENTS` có phải là payload JSON từ một orchestrator hay không:
|
|
14
|
-
|
|
15
|
-
1. Thử parse `$ARGUMENTS` dưới dạng JSON.
|
|
16
|
-
2. Nếu parse thành công **và** chứa `"_agent_mode": true`:
|
|
17
|
-
- **Bỏ qua hoàn toàn Bước 1, 2 và 3 của Gate này.**
|
|
18
|
-
- Đặt target file = `payload.target_file`
|
|
19
|
-
- Đặt loaded context = `payload.context` (KHÔNG chạy context-loader.md)
|
|
20
|
-
- Đặt phạm vi UC = `payload.uc_id` (chỉ xử lý UC này)
|
|
21
|
-
- Đặt line range = `payload.uc_section` (chỉ đọc đúng section đó của PRD)
|
|
22
|
-
- Đặt dimension = `payload.dimension` nếu có (lệnh review per-UC: chỉ review đúng lăng kính này)
|
|
23
|
-
- Đi thẳng tới phần logic riêng của lệnh.
|
|
24
|
-
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).
|
|
25
|
-
|
|
26
|
-
## Bước 0-B — Ghi nhận Model *(KHÔNG chặn)*
|
|
27
|
-
|
|
28
|
-
*Bỏ qua nếu `_agent_mode: true` (sub-agent — orchestrator đã ghi nhận rồi).*
|
|
29
|
-
|
|
30
|
-
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
|
|
31
|
-
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
|
|
32
|
-
model Opus, gắn thêm cảnh báo ngay ở dòng đó.
|
|
33
|
-
|
|
34
|
-
**KHÔNG hỏi người dùng. KHÔNG chờ. KHÔNG dừng.**
|
|
35
|
-
|
|
36
|
-
> **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
|
|
37
|
-
> `⚙️ MODEL CHECK` rồi chờ `Y/S/N`. Ba vấn đề cùng chỉ một hướng:
|
|
38
|
-
> **(1)** nó hỏi người dùng thứ mà **agent đã biết chính xác**;
|
|
39
|
-
> **(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
|
|
40
|
-
> gì phát hiện;
|
|
41
|
-
> **(3)** **cả `Y` lẫn `S` đều đi tiếp** — cách duy nhất để nó dừng là tự nguyện gõ `N`.
|
|
42
|
-
> 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
|
|
43
|
-
> 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
|
|
44
|
-
> 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.
|
|
45
|
-
>
|
|
46
|
-
> Khai báo trong report **mạnh hơn** hỏi: đúng nguồn (agent, không phải người), và nằm
|
|
47
|
-
> **cạnh kết quả** để cân nhắc, thay vì nằm trước khi có kết quả để bấm cho xong.
|
|
48
|
-
|
|
49
|
-
**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;
|
|
50
|
-
model nhỏ hơn dễ bỏ sót edge case và vi phạm kiến trúc. Đổi: `/model` → chọn Opus.
|
|
51
|
-
|
|
52
|
-
## Bước 1 — Xác định Target File
|
|
53
|
-
|
|
54
|
-
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.)
|
|
55
|
-
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.
|
|
56
|
-
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/`):
|
|
57
|
-
- **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 đó.
|
|
58
|
-
- **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.)*
|
|
59
|
-
- **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:
|
|
60
|
-
- `$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`.
|
|
61
|
-
- `$ARGUMENTS` là **TICKET-ID** → glob trực tiếp như trên.
|
|
62
|
-
- Chưa biết domain → `{specs_dir}/*/*/tech-docs/{TICKET-ID}-tech-design.md`.
|
|
63
|
-
- 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.
|
|
64
|
-
*(Đừ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`.)*
|
|
65
|
-
- **Lệnh design-spec**: `{specs_dir}/{domain}/*/design-spec/{TICKET-ID}*.md`.
|
|
66
|
-
|
|
67
|
-
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.
|
|
68
|
-
3. Nếu `$ARGUMENTS` rỗng hoặc không tìm thấy file khớp:
|
|
69
|
-
- 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).
|
|
70
|
-
- 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)"
|
|
71
|
-
- Chờ người dùng chọn rồi mới tiếp tục.
|
|
72
|
-
|
|
73
|
-
## Bước 2 — Chạy Context Loader
|
|
74
|
-
|
|
75
|
-
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`.
|
|
76
|
-
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.
|
|
77
|
-
|
|
78
|
-
## Bước 3 — CHECKPOINT
|
|
79
|
-
|
|
80
|
-
*Bỏ qua nếu `_agent_mode: true`.*
|
|
81
|
-
|
|
82
|
-
### 3a — Lệnh này có phải chặn không?
|
|
83
|
-
|
|
84
|
-
| Mức | Lệnh nào | `--yes` bỏ qua được? |
|
|
85
|
-
|---|---|:---:|
|
|
86
|
-
| **Không chặn** | Lệnh read-only: `/review-code` · `/validate-traces` · `/debug` · `/review-context` · `/review-tech-docs` | — (vốn không có) |
|
|
87
|
-
| **Chặn thường** | Mọi lệnh sinh/sửa artifact | ✅ |
|
|
88
|
-
| **Chặn CỨNG** | Ghi đè file đã tồn tại · `--resume` áp findings · migrate · prune | ❌ **không bao giờ** |
|
|
89
|
-
|
|
90
|
-
`--yes` trong `$ARGUMENTS` → bỏ qua CHECKPOINT mức *chặn thường*. (Bước 1 đã tách mọi token
|
|
91
|
-
`--` 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
|
|
92
|
-
headless: `claude -p "/generate-code UC1 --yes"`.
|
|
93
|
-
|
|
94
|
-
> **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: …*`
|
|
95
|
-
> 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
|
|
96
|
-
> **nghĩa là gì**.
|
|
97
|
-
> Nguồn máy đọc: `bin/trace-schema.json` → `gate.checkpoint_levels`; `self-check` **R11** fail
|
|
98
|
-
> 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.
|
|
99
|
-
> *(Lệnh không có dòng nào = mức **chặn thường**, mặc định.)*
|
|
100
|
-
|
|
101
|
-
> **Mức *không chặn* là thực thi đúng miễn trừ mà `rules/workflow.md` đã cấp từ trước** —
|
|
102
|
-
> trước G41 file đó viết *"read-only commands may skip CHECKPOINT"* còn gate thì luôn đòi.
|
|
103
|
-
> 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.
|
|
104
|
-
|
|
105
|
-
### 3b — In gì
|
|
106
|
-
|
|
107
|
-
**KHÔNG lặp lại những gì `[CTX LOADED]` vừa in.** Recap của context-loader (Bước 7) đã hiện
|
|
108
|
-
Stack · Platform · Layers · CLAUDE.md · Dict · Entities · Lessons · Service · Status ngay phía
|
|
109
|
-
trên. CHECKPOINT chỉ thêm **một** thông tin mới là `Target`.
|
|
110
|
-
|
|
111
|
-
**Mọi thứ sạch** — recap báo `Status: FULL`, không cờ nào bật → in đúng hai dòng:
|
|
112
|
-
|
|
113
|
-
```
|
|
114
|
-
CHECKPOINT — Target: {resolved file path}
|
|
115
|
-
Tiếp tục? (Y/N)
|
|
116
|
-
```
|
|
117
|
-
|
|
118
|
-
**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:
|
|
119
|
-
|
|
120
|
-
```
|
|
121
|
-
CHECKPOINT
|
|
122
|
-
🔴 Service : unresolved — {lý do context-loader đã ghi}
|
|
123
|
-
⚠️ CLAUDE.md: service overlay THIẾU — dùng root (code sinh ra có thể sai stack)
|
|
124
|
-
⚠️ Target : resolve bằng wildcard — {n} file khớp, chọn {file}
|
|
125
|
-
⚠️ Module : not configured — code sinh ra sẽ dùng default
|
|
126
|
-
Status : PARTIAL — thiếu: {danh sách}
|
|
127
|
-
Target : {resolved file path}
|
|
128
|
-
Tiếp tục? (Y/N)
|
|
129
|
-
```
|
|
130
|
-
|
|
131
|
-
### 3c — Cờ nào bật, cờ nào KHÔNG
|
|
132
|
-
|
|
133
|
-
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
|
|
134
|
-
đ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:
|
|
135
|
-
|
|
136
|
-
| Bật cờ khi | Nguồn | Mức |
|
|
137
|
-
|---|---|:---:|
|
|
138
|
-
| `active_service = unresolved` | context-loader Bước 2b/2c/Fallback | 🔴 |
|
|
139
|
-
| `Status = MINIMAL` | recap Bước 7 | 🔴 |
|
|
140
|
-
| `Status = PARTIAL` | recap Bước 7 | ⚠️ |
|
|
141
|
-
| CLAUDE.md thiếu, hoặc service overlay thiếu | context-loader Bước 3 | ⚠️ |
|
|
142
|
-
| Target resolve qua wildcard, hoặc nhiều file khớp mà lệnh tự chọn | Bước 1 ở trên | ⚠️ |
|
|
143
|
-
| `module` không cấu hình | recap Bước 7 | ⚠️ |
|
|
144
|
-
|
|
145
|
-
**KHÔNG bật cờ cho:** `Lessons: chưa có` · `Dict: missing` · `Entities: missing`. Đó là
|
|
146
|
-
*"dự án chưa điền"*, không phải *"có gì đó sai"* — chúng ở lại trong recap.
|
|
147
|
-
|
|
148
|
-
> **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**,
|
|
149
|
-
> 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ư
|
|
150
|
-
> 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.
|
|
151
|
-
|
|
152
|
-
### 3d — Chờ trả lời
|
|
153
|
-
|
|
154
|
-
- "Y" → tiếp tục sang các bước riêng của lệnh.
|
|
155
|
-
- "N" → dừng, hỏi người dùng muốn thay đổi gì.
|
|
156
|
-
- Có `--yes` và mức *chặn thường* → coi như "Y", **nhưng vẫn IN khối CHECKPOINT** nếu có cờ
|
|
157
|
-
🔴/⚠️ (không chặn ≠ không báo — người đọc log sau này vẫn cần thấy).
|
|
158
|
-
|
|
159
|
-
|
|
160
|
-
*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.*
|
|
161
|
-
|
|
162
|
-
### Bước 0-C — Resume mode routing *(riêng /refine-prd, chạy ngay sau Bước 0 của Gate)*
|
|
163
|
-
|
|
164
|
-
Nếu `$ARGUMENTS` chứa `--resume`:
|
|
165
|
-
- Tách `--resume` ra, phần còn lại là `raw_target` (file path hoặc prd-slug).
|
|
166
|
-
- **Nạp minimal context:** đọc `.agent/project-context.yaml`, trích xuất `paths.refinement_dir`
|
|
167
|
-
(default: `.agent/review` nếu không có hoặc file không tồn tại).
|
|
168
|
-
- Chạy **Bước 1** với `raw_target` để phân giải `prd-slug` và `target_file`.
|
|
169
|
-
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.
|
|
170
|
-
- **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.
|
|
171
|
-
- → Nhảy thẳng tới **Resume Mode** ở cuối lệnh này.
|
|
172
|
-
|
|
173
|
-
Nếu `$ARGUMENTS` không chứa `--resume` → tiếp tục luồng review bình thường bên dưới.
|
|
174
|
-
|
|
175
|
-
## Context
|
|
176
|
-
**BẮT BUỘC — đọc `.agent/steps/context-loader.md` và thực thi TOÀN BỘ quy trình trong đó**,
|
|
177
|
-
rồi mới tiếp tục phần bên dưới.
|
|
178
|
-
|
|
179
|
-
Bỏ qua bước này thì `{paths.*}`, `{tech_stack.*}`, `{conventions.*}`, guardrail từ
|
|
180
|
-
`project-lessons`, và routing service (chế độ umbrella) đều **chưa được phân giải** — mọi
|
|
181
|
-
placeholder bên dưới sẽ rỗng và lệnh sẽ đọc/ghi sai chỗ.
|
|
182
|
-
|
|
183
|
-
---
|
|
184
|
-
|
|
185
|
-
## Ngôn ngữ nghiệp vụ *(áp khi viết/áp fix — gồm cả Resume Mode Phase 2)*
|
|
186
|
-
# Business Language Guard — chặn thuật ngữ kỹ thuật rò vào tài liệu nghiệp vụ
|
|
187
|
-
|
|
188
|
-
Tài liệu nghiệp vụ (PRD, product-definition) mô tả **WHAT** — chỉ ngôn ngữ nghiệp vụ. Guard này chạy **mỗi khi viết hoặc sửa** prose (gen mới, áp fix `--resume`, hiệu chỉnh): **quét và xử lý** các thuật ngữ kỹ thuật/UI phổ thông bên dưới **trước khi ghi**.
|
|
189
|
-
|
|
190
|
-
> Guard này là **baseline framework**, chạy **song song** với Banned Terms của `business-dictionary.md` (cơ chế dictionary giữ nguyên; project vẫn bổ sung term đặc thù vào đó). Khi cả hai cùng áp, ưu tiên bản chuẩn của dictionary nếu có.
|
|
191
|
-
|
|
192
|
-
## Bản đồ xử lý (4 nhóm)
|
|
193
|
-
|
|
194
|
-
**Nhóm 1 — Tương tác/triển khai → DIỄN ĐẠT LẠI sang nghiệp vụ (giữ nguyên nghĩa):**
|
|
195
|
-
|
|
196
|
-
| Kỹ thuật/UI | Cách nói nghiệp vụ |
|
|
197
|
-
|---|---|
|
|
198
|
-
| re-render / render lại / reload / refresh (màn) | "hiển thị lại {tên màn}" |
|
|
199
|
-
| timeout | "quá thời gian chờ" |
|
|
200
|
-
| lỗi mạng / network error | "lỗi kết nối" |
|
|
201
|
-
| UI / giao diện (khi chỉ một màn) | "màn" / "màn hình" |
|
|
202
|
-
| click / tap | "bấm" / "chọn" |
|
|
203
|
-
| popup / modal (nếu chỉ là khái niệm hiển thị) | "hộp thoại" / "thông báo" |
|
|
204
|
-
| disable / enable (nút) | "khoá" / "mở" thao tác |
|
|
205
|
-
| redirect / navigate | "chuyển tới {màn}" |
|
|
206
|
-
|
|
207
|
-
**Nhóm 2 — Visual thuần → CHUYỂN Design Spec (bỏ khỏi PRD, ghi nhận lại):**
|
|
208
|
-
`spinner`, `loading indicator`, `animation`, `fade/slide`, màu sắc, font, layout pixel, micro-interaction → *"Chi tiết visual này thuộc Design Spec — ghi nhận để tạo Design Spec sau."*
|
|
209
|
-
|
|
210
|
-
**Nhóm 3 — Backend/contract thuần → BỎ khỏi PRD (thuộc Tech Docs):**
|
|
211
|
-
`API`, `endpoint`, `token/JWT`, `HTTP status`, tên class/bảng/cột DB, query, payload, header.
|
|
212
|
-
*(Ngoại lệ DUY NHẤT: Appendix "Existing API Contract" khi `API Source: existing` — xem Platform Strategy.)*
|
|
213
|
-
|
|
214
|
-
**Nhóm 4 — Ẩn dụ dữ liệu/cài đặt cho trạng thái nghiệp vụ → XÉT THEO NGHĨA (KHÔNG phải bảng thay thế):**
|
|
215
|
-
|
|
216
|
-
Các từ như `cờ / flag`, `biến / trường / field`, `giá trị / value`, `trả về / return`, `đọc / ghi (cờ)` **đa nghĩa** — kỹ thuật ở ngữ cảnh này, nghiệp vụ ở ngữ cảnh khác. **ĐỪNG thay máy móc.** Một từ chỉ là leak khi **cả hai** điều sau đúng:
|
|
217
|
-
1. Nó chỉ một **artifact lưu trữ/cơ chế** (cờ, biến, trường, giá trị-trả-về, đọc/ghi) đứng thay cho một **trạng thái/khái niệm nghiệp vụ**; VÀ
|
|
218
|
-
2. Khái niệm đó **đã có tên nghiệp vụ** (trong business-dictionary hoặc hiển nhiên).
|
|
219
|
-
|
|
220
|
-
→ Cả hai đúng: viết lại theo **tên nghiệp vụ**, ưu tiên term chuẩn trong business-dictionary.
|
|
221
|
-
→ Từ **tự nó là khái niệm nghiệp vụ**: **GIỮ NGUYÊN**.
|
|
222
|
-
|
|
223
|
-
| Reframe (là leak) | Giữ nguyên (nghiệp vụ thật) |
|
|
224
|
-
|---|---|
|
|
225
|
-
| "cờ tình trạng = chưa làm" → "con *chưa làm khảo sát*" (có term Tình trạng khảo sát) | "giá trị đơn hàng", "giá trị hợp đồng" |
|
|
226
|
-
| "cờ trả giá trị lạ" → "không đọc được tình trạng khảo sát" | "khách trả về sản phẩm" (hoàn hàng) |
|
|
227
|
-
| "đọc cờ thất bại" → "không xác định được tình trạng" | "trả kết quả học tập cho phụ huynh" |
|
|
228
|
-
|
|
229
|
-
**Neo an toàn:** lái theo business-dictionary — nếu đang diễn giải một khái niệm **đã có entry** thì dùng đúng term đó. Hỏi *"khái niệm này có tên nghiệp vụ chưa"*, KHÔNG hỏi *"từ này có bị cấm không"*.
|
|
230
|
-
|
|
231
|
-
**Luật code-format:** trong prose nghiệp vụ, **không bọc backtick/`code`** quanh giá trị/trạng thái nghiệp vụ (`chưa làm`, `đã nộp`) — code-format báo hiệu "token kỹ thuật". Dùng *nghiêng* hoặc "trong ngoặc kép". Backtick chỉ dành cho định danh code/kỹ thuật thật.
|
|
232
|
-
|
|
233
|
-
## Quy tắc áp dụng
|
|
234
|
-
- Quét toàn bộ text sắp ghi (User Story, AC, BR, Business Logic, Scope, Edge Cases, Assumptions…).
|
|
235
|
-
- Nhóm 1 → thay tại chỗ, giữ nguyên nghĩa nghiệp vụ. **Đồng bộ cách diễn đạt** với chỗ đã có sẵn trong cùng tài liệu (vd nếu "quá thời gian chờ" đã dùng ở một BR → dùng nhất quán ở mọi nơi).
|
|
236
|
-
- Nhóm 2 → gỡ khỏi prose nghiệp vụ + nhắc chuyển Design Spec.
|
|
237
|
-
- Nhóm 3 → gỡ khỏi PRD (trừ ngoại lệ brownfield).
|
|
238
|
-
- Nhóm 4 → **xét ngữ cảnh, KHÔNG thay máy móc**: chỉ reframe khi là ẩn dụ dữ liệu cho một khái niệm đã có tên nghiệp vụ (ưu tiên term dictionary); **giữ nguyên** khi từ mang nghĩa nghiệp vụ thật. Đồng thời bỏ backtick khỏi giá trị nghiệp vụ trong prose.
|
|
239
|
-
- Nếu term không có trong bản đồ nhưng rõ ràng là tên kỹ thuật/triển khai → vẫn diễn đạt lại theo tinh thần Nhóm 1, đừng để lọt.
|
|
240
|
-
|
|
241
|
-
**Checklist (dùng ở Quality Checklist của lệnh):** 0 thuật ngữ kỹ thuật/UI (re-render, UI, timeout, spinner, API/endpoint/token…) trong prose nghiệp vụ — đã diễn đạt lại (Nhóm 1) / chuyển Design Spec (Nhóm 2) / bỏ về Tech Docs (Nhóm 3); 0 ẩn dụ dữ liệu cho trạng thái đã có tên nghiệp vụ (cờ/giá trị/đọc-ghi khi là artifact — Nhóm 4) và 0 backtick bọc giá trị nghiệp vụ.
|
|
242
|
-
|
|
243
|
-
|
|
244
|
-
---
|
|
245
|
-
|
|
246
|
-
## Quy trình Review
|
|
247
|
-
# Review Fan-Out toàn diện + Hội tụ về độ đầy đủ
|
|
248
|
-
|
|
249
|
-
**Vì sao có cái này:** Một lượt review đơn không bao giờ liệt kê hết mọi vấn đề cùng lúc — model
|
|
250
|
-
dừng ở mức "đủ" findings, nên mỗi vòng review sau lại lòi ra vấn đề *mới*
|
|
251
|
-
(đập chuột chũi). Quy trình này ép review **hội tụ trong một lần chạy lệnh**:
|
|
252
|
-
fan out song song theo các chiều review, rồi lặp một critic độ-đầy-đủ cho tới khi một
|
|
253
|
-
vòng không sinh thêm gì mới, *trước khi* ghi file findings.
|
|
254
|
-
|
|
255
|
-
Lệnh gọi cung cấp hai thứ bắt buộc + hai tuỳ chọn:
|
|
256
|
-
- **DIMENSIONS** — danh sách các chiều review để fan out
|
|
257
|
-
(`/refine-prd` → 3 lăng kính; `/review-context` → các P-check hoặc B-check).
|
|
258
|
-
- **FINDINGS SCHEMA** — dạng YAML mà mỗi finding phải theo (định nghĩa trong lệnh).
|
|
259
|
-
- **GRANULARITY** *(tuỳ chọn, mặc định `auto`)* — `auto`: chọn độ mịn fan-out theo bảng ngưỡng kích thước ở Phase 1 (hành vi cũ). `per-uc`: **LUÔN** fan-out theo từng UC, **bỏ qua ngưỡng** — dùng cho review cần độ đầy đủ cao (`/refine-prd` truyền cái này để lần đầu đã quét sâu). Lệnh không truyền → `auto` → hành vi không đổi.
|
|
260
|
-
- **CHANGED_SCOPE** *(tuỳ chọn)* — danh sách UC/section đã thay đổi (review **delta**). Nếu được truyền, Phase 1 chỉ fan-out trên các phạm vi này + PRD-global; Phase 2 critic vẫn quét **toàn doc** làm lưới an toàn. Không truyền → quét toàn bộ như thường.
|
|
261
|
-
|
|
262
|
-
> **Bỏ qua ở chế độ sub-agent:** Nếu Gate Bước 0 đã set `_agent_mode: true`, toàn bộ
|
|
263
|
-
> quy trình này bị **bỏ qua** — orchestrator đã chạy sẵn một dimension/UC cho mỗi
|
|
264
|
-
> sub-agent. Chạy các check của lệnh trực tiếp trên section đã giới hạn và trả về findings.
|
|
265
|
-
|
|
266
|
-
---
|
|
267
|
-
|
|
268
|
-
## Phase 1 — Quét dimension song song
|
|
269
|
-
|
|
270
|
-
**Bao nhiêu sub-agent:** *số lượng* agent không phải là đòn bẩy độ đầy đủ — bề rộng được
|
|
271
|
-
cố định bởi taxonomy DIMENSION (thêm agent vào cùng một dimension chỉ tìm lại cùng vấn đề),
|
|
272
|
-
còn *độ sâu* thuộc về vòng lặp critic ở Phase 2.
|
|
273
|
-
|
|
274
|
-
**Nếu `GRANULARITY = per-uc`:** **bỏ qua bảng ngưỡng dưới đây**, luôn dùng độ mịn **DIMENSION × phạm vi UC** (kể cả PRD nhỏ) — đảm bảo quét sâu, không bỏ sót ngay lần đầu. (Cái giá: nhiều agent hơn cho PRD nhỏ — chấp nhận để lần đầu đầy đủ.)
|
|
275
|
-
|
|
276
|
-
**Nếu `GRANULARITY = auto`** (mặc định): chọn **độ mịn fan-out** theo kích thước target, tái dùng ngưỡng của `steps/spawn-agent.md`:
|
|
277
|
-
|
|
278
|
-
| Kích thước target | Độ mịn | Số agent |
|
|
279
|
-
|-------------|-------------|-------------|
|
|
280
|
-
| ≤ 3 UC **và** ≤ 300 dòng | một agent cho mỗi DIMENSION trên cả file | = số dimension |
|
|
281
|
-
| > 3 UC **hoặc** > 300 dòng | một agent cho mỗi **DIMENSION × phạm vi UC** (các UC + một phạm vi PRD-global), gom batch để vừa giới hạn agent | `dimensions × (UCs + 1)`, có cap (xem dưới) |
|
|
282
|
-
|
|
283
|
-
Độ mịn lớn hơn giữ context của mỗi sub-agent nhỏ và quét nó vét cạn trên một
|
|
284
|
-
UC duy nhất — chính là điều ngăn bỏ sót trên các PRD lớn.
|
|
285
|
-
|
|
286
|
-
> **Các section global (không thuộc UC) — bắt buộc ở chế độ `DIMENSION × UC`.** Mỗi agent per-UC chỉ
|
|
287
|
-
> thấy một UC, nên các section toàn-PRD không thuộc UC nào (scope, success metric,
|
|
288
|
-
> problem statement, terminology, glossary, changelog) sẽ không được quét. Khi nào
|
|
289
|
-
> fan out theo UC, cũng phải thêm một phạm vi **"PRD-global"** (các section không thuộc UC, finding nhận
|
|
290
|
-
> `uc_id: ""`) bên cạnh danh sách UC. Nên số agent tự nhiên là `dimensions × (UCs + 1)`.
|
|
291
|
-
> (Không cần ở chế độ whole-file — ở đó mỗi agent đã thấy các section global rồi.)
|
|
292
|
-
|
|
293
|
-
### Agent cap — gom batch các UC khi fan-out quá rộng
|
|
294
|
-
|
|
295
|
-
`dimensions × (UCs + 1)` có thể bùng nổ trên PRD lớn (vd 6 check × (8 UC + 1) = 54
|
|
296
|
-
agent). Giới hạn mỗi wave ở **`AGENT_CAP = 12`** agent và gom batch các phạm vi UC cho vừa:
|
|
297
|
-
|
|
298
|
-
1. Dựng danh sách phạm vi = `[UC1, UC2, …, UCn, PRD-global]` (độ dài `UCs + 1`).
|
|
299
|
-
- **Nếu `CHANGED_SCOPE` được truyền (review delta):** danh sách phạm vi = `[các UC trong CHANGED_SCOPE] + [PRD-global]` (chỉ các UC đã đổi + global), KHÔNG phải tất cả UC. Số agent tụt theo đó.
|
|
300
|
-
2. Tính số-phạm-vi-mỗi-bucket: `groups = max(1, floor(AGENT_CAP / dimensions))`.
|
|
301
|
-
- Nếu `groups ≥ UCs + 1` → không cần batch, chạy một agent cho mỗi `DIMENSION × scope`.
|
|
302
|
-
- Else chia danh sách phạm vi thành `groups` bucket liền kề kích thước xấp xỉ bằng nhau
|
|
303
|
-
(giữ `PRD-global` ở bucket riêng nếu vừa; nếu không thì gắn vào bucket cuối).
|
|
304
|
-
Mỗi agent khi đó xử lý **một DIMENSION trên một bucket UC**.
|
|
305
|
-
3. Kích thước wave kết quả = `dimensions × groups ≤ AGENT_CAP`.
|
|
306
|
-
|
|
307
|
-
Một agent đã batch review nhiều UC cùng lúc — vẫn giới hạn chặt hơn nhiều so với cả
|
|
308
|
-
file, nên độ phủ vẫn cao. `AGENT_CAP` là núm chỉnh duy nhất; tăng nếu host cho phép
|
|
309
|
-
concurrency nhiều hơn, giảm để tiết kiệm token. Chế độ whole-file (≤ 3 UC) không bao giờ chạm cap.
|
|
310
|
-
|
|
311
|
-
Spawn các sub-agent đã chọn bằng Agent tool (gửi trong một message duy nhất để chúng
|
|
312
|
-
chạy đồng thời). Mỗi sub-agent nhận một **context window mới** và quét phạm vi của nó
|
|
313
|
-
chỉ qua **một** dimension duy nhất — độ phủ sâu hơn một session phải tung hứng mọi
|
|
314
|
-
dimension cùng lúc (tránh lost-in-the-middle).
|
|
315
|
-
|
|
316
|
-
Template prompt cho sub-agent (điền vào các ngoặc):
|
|
317
|
-
|
|
318
|
-
```
|
|
319
|
-
You are a {DIMENSION_NAME} reviewer. Read the full target file at {target_file}.
|
|
320
|
-
Scope: review ONLY through the {DIMENSION_NAME} lens/check — {DIMENSION_DESCRIPTION}.
|
|
321
|
-
Be exhaustive: scan every section, every UC, every AC/BR/scenario. Do not stop early.
|
|
322
|
-
Project context (terminology, entities, architecture):
|
|
323
|
-
{slim_context — banned terms, canonical entities, layer order, domains}
|
|
324
|
-
|
|
325
|
-
Return a JSON array of findings, each:
|
|
326
|
-
{ "dimension": "{DIMENSION_NAME}", "severity": "critical|major|minor",
|
|
327
|
-
"section": "...", "uc_id": "...", "quote": "<verbatim ≤120 chars>",
|
|
328
|
-
"finding": "...", "suggestion": "...", "auto_fixable": true|false }
|
|
329
|
-
Return [] if this dimension is clean. Return ONLY the JSON array.
|
|
330
|
-
```
|
|
331
|
-
|
|
332
|
-
Gom mảng findings của mọi sub-agent vào một danh sách hợp nhất `ALL_FINDINGS`.
|
|
333
|
-
|
|
334
|
-
---
|
|
335
|
-
|
|
336
|
-
## Phase 2 — Vòng lặp hội tụ critic độ-đầy-đủ
|
|
337
|
-
|
|
338
|
-
Đây là bước chống đập-chuột-chũi. Lặp cho tới khi **hai vòng liên tiếp thêm 0 finding
|
|
339
|
-
mới**, hoặc tới cap cứng **3 vòng**, cái nào đến trước:
|
|
340
|
-
|
|
341
|
-
> **Lưu ý delta:** kể cả khi `CHANGED_SCOPE` giới hạn Phase 1 vào các UC đã đổi, completeness-critic ở Phase 2 **vẫn đọc TOÀN bộ doc** — đây là lưới an toàn bắt các vấn đề mà một fix ở UC đã đổi có thể làm lộ ra ở chỗ khác.
|
|
342
|
-
|
|
343
|
-
1. Spawn một sub-agent **completeness-critic** bằng Agent tool. Cho nó:
|
|
344
|
-
- toàn bộ target file (`{target_file}`),
|
|
345
|
-
- danh sách findings đã ghi nhận dưới dạng **slim JSON** — chỉ 3 fields cốt lõi
|
|
346
|
-
đủ để critic nhận ra trùng lặp (không cần `quote`, `suggestion`, `auto_fixable`, `severity`):
|
|
347
|
-
```json
|
|
348
|
-
[
|
|
349
|
-
{ "uc_id": "...", "section": "...", "finding": "..." },
|
|
350
|
-
...
|
|
351
|
-
]
|
|
352
|
-
```
|
|
353
|
-
Nếu `ALL_FINDINGS` vượt 60 items, rút gọn `finding` xuống còn 80 ký tự đầu mỗi item.
|
|
354
|
-
- cùng slim context (banned terms, canonical entities, layer order, domains).
|
|
355
|
-
Prompt nó:
|
|
356
|
-
```
|
|
357
|
-
Here is a document and a list of issues already found. Read the WHOLE document.
|
|
358
|
-
List ONLY real, additional issues NOT already in the list — gaps, ambiguities,
|
|
359
|
-
contradictions, missing edge/negative paths, coverage holes, terminology drift,
|
|
360
|
-
structural omissions, and any issue that a fix to an existing finding would expose.
|
|
361
|
-
ALSO flag ROLE-BOUNDARY / altitude violations (you are NOT limited to adding detail):
|
|
362
|
-
content sitting in the WRONG section — detailed mechanism (retry counts, timeouts, flag
|
|
363
|
-
names/owners, error branches) written INSIDE an acceptance criterion or a scope line
|
|
364
|
-
instead of the Business Rule/Logic section; an AC that merely restates its referenced BR
|
|
365
|
-
(same content, converged); a term definition crammed into In/Out Scope. For these, the
|
|
366
|
-
suggestion must be to MOVE the detail to its proper section (AC keeps only the observable
|
|
367
|
-
outcome + BR ref) — NOT to delete it, and NOT to add more detail.
|
|
368
|
-
Do NOT repeat anything already listed. Return the same finding JSON shape, or [] if
|
|
369
|
-
nothing new.
|
|
370
|
-
```
|
|
371
|
-
2. Thêm bất kỳ finding thực sự mới (chưa có trong `ALL_FINDINGS`) vào danh sách.
|
|
372
|
-
3. Nếu vòng này trả 0 finding mới → tăng bộ đếm dry-round; ngược lại reset về 0.
|
|
373
|
-
4. Dừng khi bộ đếm dry-round đạt 2, hoặc sau tổng cộng 3 vòng.
|
|
374
|
-
|
|
375
|
-
Ghi lại `convergence_rounds` (số vòng critic đã chạy) cho report.
|
|
376
|
-
|
|
377
|
-
---
|
|
378
|
-
|
|
379
|
-
## Phase 3 — Dedup, giải quyết xung đột, merge
|
|
380
|
-
|
|
381
|
-
Các sub-agent chạy **mù với nhau** (độc lập = độ phủ đa dạng). Chúng không bao giờ
|
|
382
|
-
trao đổi hay điều hoà giữa chúng — mọi xử lý trùng/xung đột diễn ra **ở đây trong
|
|
383
|
-
orchestrator**, nơi thấy toàn bộ tập findings.
|
|
384
|
-
|
|
385
|
-
1. **Khử trùng lặp** `ALL_FINDINGS`: hai finding là trùng nếu cùng nhắm tới cùng
|
|
386
|
-
`section` + `uc_id` và mô tả cùng một vấn đề gốc. Giữ cái có `suggestion`
|
|
387
|
-
phong phú hơn; nếu khác nhau về severity, giữ severity **cao hơn**.
|
|
388
|
-
2. **Giải quyết xung đột** — nhóm các finding còn lại theo `section` + `uc_id` và kiểm tra
|
|
389
|
-
mâu thuẫn (hai finding có `suggestion` không thể cùng áp dụng, hoặc đề xuất sửa ngược nhau cho cùng một chỗ):
|
|
390
|
-
- Nếu hai đề xuất có thể **merge** thành một bản sửa mạch lạc → merge thành một finding duy nhất.
|
|
391
|
-
- Nếu chúng **loại trừ lẫn nhau** → phát ra **một** finding nêu cả hai phương án
|
|
392
|
-
và set `auto_fixable: false` với `status: "needs_discussion"` (PRD) /
|
|
393
|
-
`status: "pending"` (review) để con người chọn — không bao giờ âm thầm bỏ một bên.
|
|
394
|
-
- Nếu một finding bị **vô hiệu** bởi finding khác (vd một finding cấu trúc nói một section
|
|
395
|
-
bị thiếu, nhưng một finding khác trích dẫn nội dung từ chính section đó) → bỏ cái không hợp lệ.
|
|
396
|
-
3. **Sắp xếp** theo severity (critical → major → minor), rồi theo thứ tự `section` trong file.
|
|
397
|
-
4. **Gán ID ổn định** `F001, F002, …` theo thứ tự đã sắp đó.
|
|
398
|
-
5. Map `dimension` của mỗi finding vào field schema của lệnh
|
|
399
|
-
(`lens` cho `/refine-prd`; `check_id` cho `/review-context`).
|
|
400
|
-
6. Ghi **một** file findings duy nhất theo FINDINGS SCHEMA mà lệnh định nghĩa.
|
|
401
|
-
|
|
402
|
-
Trong report cuối của lệnh, thêm một dòng:
|
|
403
|
-
```
|
|
404
|
-
Convergence: {convergence_rounds} vòng critic — file findings đã đầy đủ; chạy lại sẽ lòi ra 0 vấn đề mới.
|
|
405
|
-
```
|
|
406
|
-
|
|
407
|
-
|
|
408
|
-
---
|
|
409
|
-
|
|
410
|
-
## Phân tích — 3 lăng kính (fan out cả ba, rồi hội tụ)
|
|
411
|
-
|
|
412
|
-
Chạy review qua **Quy trình Review** ở trên (`steps/review-fanout.md`).
|
|
413
|
-
|
|
414
|
-
**Tham số truyền vào Quy trình Review:**
|
|
415
|
-
- `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.
|
|
416
|
-
- `CHANGED_SCOPE` — xác định theo chế độ full/delta ngay dưới đây.
|
|
417
|
-
|
|
418
|
-
**Chọn full vs delta** *(mặc định: lần đầu FULL, lần sau DELTA)*:
|
|
419
|
-
1. Tách `--full` khỏi `$ARGUMENTS` nếu có.
|
|
420
|
-
2. Kiểm tra `{paths.refinement_dir}/{prd-slug}-findings.yaml`:
|
|
421
|
-
- **Không tồn tại** (lần đầu review PRD này) → **FULL**: KHÔNG truyền `CHANGED_SCOPE`.
|
|
422
|
-
- **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ộ).
|
|
423
|
-
- **Tồn tại** + KHÔNG có `--full` → so `prd_version` trong findings cũ với `| **Version** |` của PRD hiện tại:
|
|
424
|
-
- **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ộ."`
|
|
425
|
-
- **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:
|
|
426
|
-
- **`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.
|
|
427
|
-
- **`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."`
|
|
428
|
-
|
|
429
|
-
**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
|
|
430
|
-
toàn bộ PRD qua đúng lăng kính của nó:
|
|
431
|
-
|
|
432
|
-
<!-- ─────────────────────────────────────────────────────────────────────────────
|
|
433
|
-
LĂNG KÍNH BỊ TẮT (DISABLED — KHÔNG dùng, KHÔNG fan-out agent cho lăng kính này):
|
|
434
|
-
|
|
435
|
-
- **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.
|
|
436
|
-
|
|
437
|
-
CÁCH THÊM LẠI QA (khi cần bật lại giai đoạn sau):
|
|
438
|
-
1. Chuyển bullet QA ở trên ra khỏi block comment này, đặt lên đầu danh sách DIMENSIONS.
|
|
439
|
-
2. Đổi "3 lăng kính"/"cả ba" → "4 lăng kính"/"cả bốn" ở: dòng tiêu đề (# /refine-prd),
|
|
440
|
-
heading "## Phân tích", câu "DIMENSIONS = N lăng kính", và steps/review-fanout.md.
|
|
441
|
-
3. Thêm "QA" lại vào enum của `lens:` trong FINDINGS SCHEMA + vào `by_lens`.
|
|
442
|
-
4. Thêm "QA" lại vào ghi chú "phán đoán DEV/SA/PO" ở LƯU Ý của schema.
|
|
443
|
-
5. Rebuild: node bin/build.js
|
|
444
|
-
───────────────────────────────────────────────────────────────────────────── -->
|
|
445
|
-
|
|
446
|
-
> **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ụ.**
|
|
447
|
-
> 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*.
|
|
448
|
-
|
|
449
|
-
- **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.
|
|
450
|
-
- **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ụ.
|
|
451
|
-
- **Lăng kính PO**: Scope đã khoanh vùng? Priority rõ chưa? Success metric đã định nghĩa? Rủi ro scope creep?
|
|
452
|
-
|
|
453
|
-
Vòng lặp completeness-critic (Phase 2) là cái đảm bảo file findings đầy đủ trong một lần chạy
|
|
454
|
-
— chạy lại `/refine-prd` sẽ lòi ra **0 finding mới**. Map mỗi dimension vào field `lens`
|
|
455
|
-
của schema dưới đây.
|
|
456
|
-
|
|
457
|
-
## File Output
|
|
458
|
-
|
|
459
|
-
Suy ra tên file output từ PRD slug:
|
|
460
|
-
- File PRD: `{paths.specs_dir}/payment/create-invoice/PAY01-create-invoice.md` → output: `{paths.refinement_dir}/create-invoice-findings.yaml`
|
|
461
|
-
- Quy tắc: lấy tên folder cha của PRD (prd-slug), thêm hậu tố `-findings.yaml`
|
|
462
|
-
|
|
463
|
-
Trước khi ghi file, xác định `recommendation` từ findings đã dedup và sắp xếp:
|
|
464
|
-
- `by_severity.critical ≥ 1` → `BLOCKED`
|
|
465
|
-
- `by_severity.critical = 0` và `by_severity.major ≥ 1` → `NEEDS_REVISION`
|
|
466
|
-
- `by_severity.critical = 0` và `by_severity.major = 0` → `APPROVED_WITH_MINOR_CHANGES`
|
|
467
|
-
|
|
468
|
-
Đả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.
|
|
469
|
-
|
|
470
|
-
Ghi `{paths.refinement_dir}/{prd-slug}-findings.yaml`:
|
|
471
|
-
|
|
472
|
-
```yaml
|
|
473
|
-
prd_source: "{paths.specs_dir}/{domain}/{prd-slug}/{TICKET-ID}-{prd-slug}.md"
|
|
474
|
-
prd_version: "{đọc | **Version** | từ metadata PRD lúc sinh findings — dùng để chọn full/delta lần chạy sau}"
|
|
475
|
-
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")
|
|
476
|
-
generated_at: "{ISO datetime}"
|
|
477
|
-
status: "pending_review"
|
|
478
|
-
|
|
479
|
-
findings:
|
|
480
|
-
- id: "F001"
|
|
481
|
-
lens: "DEV" # DEV | SA | PO
|
|
482
|
-
severity: "major" # critical | major | minor
|
|
483
|
-
section: "§2. Acceptance Criteria" # nhãn heading/section dạng người đọc
|
|
484
|
-
uc_id: "{TICKET-ID}-UC{N}" # UC mà finding thuộc về; "" nếu PRD-global (scope, metrics, problem statement)
|
|
485
|
-
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ự}"
|
|
486
|
-
finding: "{mô tả gap hoặc vấn đề}"
|
|
487
|
-
suggestion: "{đề xuất cải thiện cụ thể, hành động được}"
|
|
488
|
-
resolution_edge_cases: # CHỈ điền cho critical/major; minor → để [] (bỏ qua)
|
|
489
|
-
# Phân tích bậc-hai (advisory, KHÔNG chặn): nếu áp `suggestion` này thì có thể đẻ ra
|
|
490
|
-
# 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,
|
|
491
|
-
# 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.
|
|
492
|
-
- "{edge case có thể phát sinh nếu chốt phương án này}"
|
|
493
|
-
auto_fixable: false
|
|
494
|
-
# true = AI tự tin cao vào suggestion này; Review Board có thể hiển thị nút "quick accept"
|
|
495
|
-
# false = cần human đọc kỹ và ghi quyết định trước khi accept
|
|
496
|
-
# Resume Mode luôn áp dụng theo status (accepted|modified), bất kể auto_fixable.
|
|
497
|
-
# LƯU Ý: /refine-prd CỐ Ý không có `--fix` mode (khác /review-context) — finding 3 lăng kính
|
|
498
|
-
# là phán đoán DEV/SA/PO, phải qua người duyệt ở Board; auto_fixable ở đây CHỈ là gợi ý
|
|
499
|
-
# quick-accept cho Board, KHÔNG để máy tự áp.
|
|
500
|
-
status: "pending"
|
|
501
|
-
applied_via: ""
|
|
502
|
-
# 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):
|
|
503
|
-
# pending → finding mới, chưa được review
|
|
504
|
-
# accepted → reviewer chấp nhận suggestion gốc → Resume sẽ apply
|
|
505
|
-
# modified → reviewer đã sửa suggestion trong Review Board → Resume dùng suggestion đã sửa
|
|
506
|
-
# rejected → reviewer bác bỏ, không apply
|
|
507
|
-
# needs_discussion → hai suggestion xung đột, CHẶN --resume (bỏ qua + cảnh báo) tới khi con người quyết
|
|
508
|
-
# deferred → cố ý hoãn (không chặn), bỏ qua lượt này, xem lại sau
|
|
509
|
-
# applied → đã áp vào PRD; applied_via ghi cách áp: "resume" (qua --resume) — /review-context còn dùng "fix" (qua --fix)
|
|
510
|
-
|
|
511
|
-
summary:
|
|
512
|
-
total_findings: {N}
|
|
513
|
-
by_severity: { critical: {N}, major: {N}, minor: {N} }
|
|
514
|
-
by_lens: { DEV: {N}, SA: {N}, PO: {N} }
|
|
515
|
-
recommendation: "APPROVED_WITH_MINOR_CHANGES | NEEDS_REVISION | BLOCKED"
|
|
516
|
-
# Rule: critical ≥ 1 → BLOCKED
|
|
517
|
-
# critical = 0, major ≥ 1 → NEEDS_REVISION
|
|
518
|
-
# critical = 0, major = 0 → APPROVED_WITH_MINOR_CHANGES
|
|
519
|
-
```
|
|
520
|
-
|
|
521
|
-
> **Field định vị (`quote` + `uc_id`) — bắt buộc cho source-jump của Review Board.**
|
|
522
|
-
> Với mỗi finding, copy một đoạn `quote` **nguyên văn** thẳng từ PRD tại đúng chỗ
|
|
523
|
-
> 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
|
|
524
|
-
> `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
|
|
525
|
-
> finding trong Review Board và nhảy thẳng tới đúng vị trí trong PRD nguồn.
|
|
526
|
-
|
|
527
|
-
> **`resolution_edge_cases` — phân tích bậc-hai (chỉ critical/major).**
|
|
528
|
-
> 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ì
|
|
529
|
-
> đẻ 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).
|
|
530
|
-
> Ghi vào `resolution_edge_cases` để PO **thấy trước khi accept**. Đây là **advisory** — KHÔNG chặn, KHÔNG tự
|
|
531
|
-
> tạo finding; PO đọc rồi quyết. Finding `minor` → để `[]`.
|
|
532
|
-
> *(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`
|
|
533
|
-
> 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.)*
|
|
534
|
-
|
|
535
|
-
## Report
|
|
536
|
-
|
|
537
|
-
**Đọc `.agent/steps/report-footer.md`** và áp đúng khuôn footer trong đó (Status Badge ·
|
|
538
|
-
Output Artifacts · Next) cho report cuối, kèm khối bên dưới.
|
|
539
|
-
|
|
540
|
-
Dùng footer chuẩn với hai field bổ sung `Findings` và `Review` đặt ngay sau `Status`:
|
|
541
|
-
|
|
542
|
-
```
|
|
543
|
-
---
|
|
544
|
-
Status : {badge}
|
|
545
|
-
Findings : {total} | 🔴 Critical: {N} | 🟡 Major: {N} | 🟢 Minor: {N}
|
|
546
|
-
Review : {paths.refinement_dir}/{prd-slug}-findings.yaml
|
|
547
|
-
Output Artifacts:
|
|
548
|
-
created {paths.refinement_dir}/{prd-slug}-findings.yaml (findings từ 3-lens review)
|
|
549
|
-
Pipeline : Discovery → [PRD ◀ bạn ở đây] → Design Spec → BDD → Tech Design → Code → Dev Self-Check → QC → Trace Audit
|
|
550
|
-
Vòng review: [① phân tích ◀] → ② Review Board → ③ --resume
|
|
551
|
-
Next : Mở trong Review Board (chuột phải vào file) → Cập nhật PRD
|
|
552
|
-
→ /review-context {prd-file} ← kiểm tra chất lượng PRD trước khi sinh BDD
|
|
553
|
-
→ khi sạch critical, PO đặt | **Status** | approved | → /generate-bdd {prd-file}
|
|
554
|
-
```
|
|
555
|
-
|
|
556
|
-
Nếu có finding nào có `status: "needs_discussion"`, thêm warning block sau footer:
|
|
557
|
-
|
|
558
|
-
```
|
|
559
|
-
⚠️ {N} finding(s) cần quyết định của bạn trước khi --resume:
|
|
560
|
-
Mở {paths.refinement_dir}/{prd-slug}-findings.yaml
|
|
561
|
-
Với mỗi finding có status "needs_discussion":
|
|
562
|
-
1. Đọc cả hai phương án trong `suggestion`
|
|
563
|
-
2. Chọn một (hoặc viết phương án khác) vào `suggestion`
|
|
564
|
-
3. Đổi `status` → "accepted"
|
|
565
|
-
Sau đó chạy: /refine-prd {prd-file} --resume
|
|
566
|
-
```
|
|
567
|
-
*(Bỏ warning block này nếu không có `needs_discussion` finding nào.)*
|
|
568
|
-
|
|
569
|
-
---
|
|
570
|
-
|
|
571
|
-
## Resume Mode — Áp dụng Findings & Bump Version
|
|
572
|
-
|
|
573
|
-
*Đượ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.*
|
|
574
|
-
|
|
575
|
-
### Phase 1 — Đọc các finding được chấp nhận
|
|
576
|
-
|
|
577
|
-
1. Kiểm tra `{paths.refinement_dir}/{prd-slug}-findings.yaml` có tồn tại không.
|
|
578
|
-
Nếu không → báo lỗi rõ ràng và dừng:
|
|
579
|
-
```
|
|
580
|
-
❌ Không tìm thấy findings file tại {paths.refinement_dir}/{prd-slug}-findings.yaml
|
|
581
|
-
Hãy chạy /refine-prd {prd-file} trước để tạo findings, rồi mới --resume.
|
|
582
|
-
```
|
|
583
|
-
2. Đọc file findings.
|
|
584
|
-
3. Gom tất cả finding có `status: "accepted"` hoặc `status: "modified"`.
|
|
585
|
-
4. Nếu không có finding nào được chấp nhận → báo "No accepted findings. PRD unchanged." và dừng.
|
|
586
|
-
5. Nếu còn finding nào có `status: "needs_discussion"` → cảnh báo (không dừng):
|
|
587
|
-
```
|
|
588
|
-
⚠️ {N} finding(s) chưa được giải quyết (needs_discussion) — sẽ bị bỏ qua lần apply này.
|
|
589
|
-
Giải quyết chúng trong Review Board rồi chạy lại --resume để apply.
|
|
590
|
-
```
|
|
591
|
-
|
|
592
|
-
### Phase 2 — Áp dụng thay đổi vào PRD
|
|
593
|
-
|
|
594
|
-
Với mỗi finding được chấp nhận, theo thứ tự severity (critical → major → minor):
|
|
595
|
-
- Đi tới section PRD được chỉ định bởi `finding.section`.
|
|
596
|
-
- Á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.
|
|
597
|
-
- **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.
|
|
598
|
-
- **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.
|
|
599
|
-
- 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.
|
|
600
|
-
|
|
601
|
-
### Phase 2.5 — Cập nhật file findings
|
|
602
|
-
|
|
603
|
-
Với mỗi finding đã áp dụng (status là `accepted` hoặc `modified`):
|
|
604
|
-
- Đặt `status: "applied"` + `applied_via: "resume"` trong `{paths.refinement_dir}/{prd-slug}-findings.yaml`.
|
|
605
|
-
|
|
606
|
-
Cập nhật `status: "applied"` ở **root level** của file findings (không phải `summary.status`).
|
|
607
|
-
# Lifecycle file-level: pending_review → applied | partially_applied
|
|
608
|
-
# partially_applied khi có finding bị rejected hoặc needs_discussion còn sót lại.
|
|
609
|
-
|
|
610
|
-
### Phase 3 — Bump version & ghi entry changelog
|
|
611
|
-
|
|
612
|
-
1. Đọc giá trị `| **Version** |` hiện tại từ bảng metadata của PRD.
|
|
613
|
-
2. Xác định loại bump từ tập finding được chấp nhận:
|
|
614
|
-
- **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.
|
|
615
|
-
- **minor** (x.Y → x.Y+1): làm rõ, thêm edge case, tinh chỉnh wording AC/BR.
|
|
616
|
-
3. Tính chuỗi version mới.
|
|
617
|
-
4. Cập nhật metadata PRD:
|
|
618
|
-
- `| **Version** | {new_version} |`
|
|
619
|
-
- `| **Updated** | {today YYYY-MM-DD} |`
|
|
620
|
-
- `| **Status** | draft |` ← reset về draft, phải được duyệt lại
|
|
621
|
-
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):
|
|
622
|
-
- Thêm row mới lên **đầu** bảng:
|
|
623
|
-
```
|
|
624
|
-
| {new_version} | {today} | {tóm tắt — KÈM UC/AC/BR bị ảnh hưởng, vd "UC2: sửa BR5; thêm AC7"} |
|
|
625
|
-
```
|
|
626
|
-
*(Nêu rõ UC/AC/BR giúp `/generate-bdd` Version Check biết scenario nào cần cập nhật.)*
|
|
627
|
-
- 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)`
|
|
628
|
-
- **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:
|
|
629
|
-
```
|
|
630
|
-
# Change Log (lịch sử) — {TICKET-ID}
|
|
631
|
-
> 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.
|
|
632
|
-
|
|
633
|
-
| Version | Date | Changes |
|
|
634
|
-
|---------|------|---------|
|
|
635
|
-
```
|
|
636
|
-
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").
|
|
637
|
-
|
|
638
|
-
### Phase 4 — Report
|
|
639
|
-
|
|
640
|
-
```
|
|
641
|
-
/refine-prd Đã áp dụng — {PRD name}
|
|
642
|
-
Version : {old_version} → {new_version}
|
|
643
|
-
Applied : {N} findings ({critical} critical, {major} major, {minor} minor)
|
|
644
|
-
Changes :
|
|
645
|
-
- {change 1}
|
|
646
|
-
- {change 2}
|
|
647
|
-
|
|
648
|
-
💡 Vừa áp {N} phương án (gồm bản PO tự sửa) → chạy lại /refine-prd {prd-file}
|
|
649
|
-
(delta tự động) để lòi edge case mà các phương án vừa áp có thể tạo ra.
|
|
650
|
-
⚠️ Status đã reset về draft (sửa rồi phải duyệt lại). BDD có thể đã lỗi thời. Chạy:
|
|
651
|
-
/review-context {prd-file} ← kiểm tra chất lượng PRD trước
|
|
652
|
-
→ PO đặt | **Status** | approved | khi hài lòng → /generate-bdd {prd-file}
|
|
653
|
-
```
|