@anhth2/spec-driven-dev-plugin 0.9.2 → 0.11.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/commands/debug.md +50 -20
- package/commands/define-product.md +49 -19
- package/commands/{generate-tests.md → dev-gen-test.md} +85 -23
- package/commands/{generate-tests.tmpl → dev-gen-test.tmpl} +18 -4
- package/{core/commands/run-tests.md → commands/dev-run-test.md} +102 -21
- package/commands/{run-tests.tmpl → dev-run-test.tmpl} +35 -2
- package/commands/{smoke-test.md → dev-smoke-test.md} +54 -24
- package/commands/{smoke-test.tmpl → dev-smoke-test.tmpl} +5 -5
- package/commands/fix-bug.md +50 -20
- package/commands/generate-bdd.md +78 -21
- package/commands/generate-bdd.tmpl +11 -2
- package/commands/generate-code.md +123 -23
- package/commands/generate-code.tmpl +56 -4
- package/commands/generate-design-spec.md +142 -47
- package/commands/generate-design-spec.tmpl +93 -28
- package/commands/generate-prd.md +49 -19
- package/commands/generate-spec-manifest.md +49 -19
- package/commands/generate-tech-docs.md +50 -20
- package/commands/generate-tech-docs.tmpl +1 -1
- package/commands/learn.md +50 -20
- package/commands/propose-scenario.md +50 -20
- package/commands/propose-scenario.tmpl +1 -1
- package/commands/qc-analyze.md +514 -0
- package/commands/qc-analyze.tmpl +71 -0
- package/commands/qc-design-test.md +510 -0
- package/commands/qc-design-test.tmpl +67 -0
- package/commands/qc-plan.md +492 -0
- package/commands/qc-plan.tmpl +49 -0
- package/commands/qc-report.md +491 -0
- package/commands/qc-report.tmpl +48 -0
- package/commands/qc-review.md +496 -0
- package/commands/qc-review.tmpl +53 -0
- package/commands/qc-run-test.md +538 -0
- package/commands/qc-run-test.tmpl +77 -0
- package/commands/refine-prd.md +203 -24
- package/commands/refine-prd.tmpl +16 -5
- package/commands/report-bug.md +49 -19
- package/commands/review-code.md +51 -21
- package/commands/review-code.tmpl +1 -1
- package/commands/review-context.md +198 -20
- package/commands/review-context.tmpl +11 -1
- package/commands/review-tech-docs.md +49 -19
- package/commands/setup-ai-first.md +14 -7
- package/commands/sync.md +30 -20
- package/commands/sync.tmpl +16 -13
- package/commands/update-framework.md +14 -7
- package/commands/validate-traces.md +106 -45
- package/commands/validate-traces.tmpl +57 -26
- package/core/FRAMEWORK_VERSION +1 -1
- package/core/commands/debug.md +50 -20
- package/core/commands/define-product.md +49 -19
- package/core/commands/{generate-tests.md → dev-gen-test.md} +85 -23
- package/{commands/run-tests.md → core/commands/dev-run-test.md} +102 -21
- package/core/commands/{smoke-test.md → dev-smoke-test.md} +54 -24
- package/core/commands/fix-bug.md +50 -20
- package/core/commands/generate-bdd.md +78 -21
- package/core/commands/generate-code.md +123 -23
- package/core/commands/generate-design-spec.md +142 -47
- package/core/commands/generate-prd.md +49 -19
- package/core/commands/generate-spec-manifest.md +49 -19
- package/core/commands/generate-tech-docs.md +50 -20
- package/core/commands/learn.md +50 -20
- package/core/commands/propose-scenario.md +50 -20
- package/core/commands/qc-analyze.md +514 -0
- package/core/commands/qc-design-test.md +510 -0
- package/core/commands/qc-plan.md +492 -0
- package/core/commands/qc-report.md +491 -0
- package/core/commands/qc-review.md +496 -0
- package/core/commands/qc-run-test.md +538 -0
- package/core/commands/refine-prd.md +203 -24
- package/core/commands/report-bug.md +49 -19
- package/core/commands/review-code.md +51 -21
- package/core/commands/review-context.md +198 -20
- package/core/commands/review-tech-docs.md +49 -19
- package/core/commands/setup-ai-first.md +14 -7
- package/core/commands/sync.md +30 -20
- package/core/commands/update-framework.md +14 -7
- package/core/commands/validate-traces.md +106 -45
- package/core/modules/qc-playwright/stack-profile.yaml +65 -0
- package/core/skills/code/SKILL.md +63 -26
- package/core/skills/debug/SKILL.md +78 -34
- package/core/skills/design-spec/SKILL.md +49 -19
- package/core/skills/discovery/SKILL.md +49 -19
- package/core/skills/prd/SKILL.md +28 -14
- package/core/skills/qc/qa-analyst/DOC_GAPS.template.md +63 -0
- package/core/skills/qc/qa-analyst/acceptance-criteria.md +56 -0
- package/core/skills/qc/qa-analyst/business-rules.md +55 -0
- package/core/skills/qc/qa-analyst/data-flow.md +60 -0
- package/core/skills/qc/qa-analyst/spec-breakdown.md +57 -0
- package/core/skills/qc/qa-designer/e2e/journey.md +41 -0
- package/core/skills/qc/qa-designer/exploratory/charter.md +68 -0
- package/core/skills/qc/qa-designer/exploratory/explore-to-functional.md +43 -0
- package/core/skills/qc/qa-designer/functional/api.md +45 -0
- package/core/skills/qc/qa-designer/functional/gui-feature.md +46 -0
- package/core/skills/qc/qa-designer/functional/gui-screen.md +52 -0
- package/core/skills/qc/qa-designer/integration/api.md +42 -0
- package/core/skills/qc/qa-designer/integration/db.md +39 -0
- package/core/skills/qc/qa-designer/integration/gui.md +40 -0
- package/core/skills/qc/qa-designer/integration/kafka.md +40 -0
- package/core/skills/qc/qa-designer/non-functional.md +40 -0
- package/core/skills/qc/qa-planner/test-plan.md +120 -0
- package/core/skills/qc/qa-reviewer/script/e2e.md +87 -0
- package/core/skills/qc/qa-reviewer/script/exploratory.md +45 -0
- package/core/skills/qc/qa-reviewer/script/functional.md +101 -0
- package/core/skills/qc/qa-reviewer/script/integration.md +91 -0
- package/core/skills/qc/qa-reviewer/script/non-functional.md +126 -0
- package/core/skills/qc/qa-reviewer/test-case/e2e.md +73 -0
- package/core/skills/qc/qa-reviewer/test-case/exploratory.md +43 -0
- package/core/skills/qc/qa-reviewer/test-case/functional.md +76 -0
- package/core/skills/qc/qa-reviewer/test-case/integration.md +69 -0
- package/core/skills/qc/qa-reviewer/test-case/non-functional.md +73 -0
- package/core/skills/qc/qa-runner/e2e.md +49 -0
- package/core/skills/qc/qa-runner/exploratory/session.md +36 -0
- package/core/skills/qc/qa-runner/functional/api.md +35 -0
- package/core/skills/qc/qa-runner/functional/gui-feature.md +51 -0
- package/core/skills/qc/qa-runner/functional/gui-screen.md +55 -0
- package/core/skills/qc/qa-runner/integration.md +47 -0
- package/core/skills/qc/qa-runner/non-functional.md +49 -0
- package/core/skills/qc/qa-runner/report/report.md +37 -0
- package/core/skills/setup-ai-first/SKILL.md +14 -7
- package/core/skills/spec/SKILL.md +28 -14
- package/core/skills/test/SKILL.md +121 -54
- package/core/steps/capture-lesson.md +1 -1
- package/core/steps/context-loader.md +35 -12
- package/core/steps/report-footer.md +14 -7
- package/core/steps/review-fanout.md +138 -0
- package/core/steps/spawn-agent.md +1 -1
- package/core/steps/trace-mirror.md +18 -0
- package/core/templates/design-spec.template.md +16 -8
- package/core/templates/project-context.yaml +8 -0
- package/docs/01-getting-started/README.md +19 -0
- package/docs/01-getting-started/core-concepts.md +102 -0
- package/docs/01-getting-started/installation.md +154 -0
- package/docs/01-getting-started/quickstart.md +85 -0
- package/docs/02-guides/README.md +27 -0
- package/docs/02-guides/developer/README.md +46 -0
- package/docs/02-guides/developer/bdd-and-trace.md +123 -0
- package/docs/02-guides/developer/commands.md +76 -0
- package/docs/02-guides/developer/pr-checklist.md +15 -0
- package/docs/02-guides/developer/scenarios.md +448 -0
- package/docs/02-guides/developer/workflow.md +59 -0
- package/docs/02-guides/product-owner/README.md +77 -0
- package/docs/02-guides/product-owner/commands.md +30 -0
- package/docs/02-guides/product-owner/handoff-checklist.md +42 -0
- package/docs/02-guides/product-owner/prd-writing-rules.md +45 -0
- package/docs/02-guides/product-owner/scenarios.md +357 -0
- package/docs/02-guides/qc-automation.md +92 -0
- package/docs/02-guides/tester/README.md +72 -0
- package/docs/02-guides/tester/bug-reporting.md +117 -0
- package/docs/02-guides/tester/reading-specs.md +79 -0
- package/docs/02-guides/tester/scenarios.md +186 -0
- package/docs/02-guides/tester/spec-manifest.md +124 -0
- package/docs/02-guides/tester/test-checklist.md +31 -0
- package/docs/02-guides/tester/workflow.md +79 -0
- package/docs/03-concepts/README.md +19 -0
- package/docs/03-concepts/architecture.md +243 -0
- package/docs/03-concepts/pipeline.md +249 -0
- package/docs/03-concepts/traceability.md +148 -0
- package/docs/04-operations/README.md +33 -0
- package/docs/04-operations/bug-flow.md +321 -0
- package/docs/04-operations/publishing.md +137 -0
- package/docs/04-operations/sync-and-update.md +328 -0
- package/docs/05-reference/README.md +29 -0
- package/docs/05-reference/commands.md +229 -0
- package/docs/05-reference/modules.md +110 -0
- package/docs/05-reference/trace-schema.md +146 -0
- package/docs/README.md +51 -0
- package/modules/qc-playwright/stack-profile.yaml +65 -0
- package/package.json +2 -2
- package/skills/code/SKILL.md +63 -26
- package/skills/debug/SKILL.md +78 -34
- package/skills/debug/SKILL.tmpl +1 -1
- package/skills/design-spec/SKILL.md +49 -19
- package/skills/discovery/SKILL.md +49 -19
- package/skills/prd/SKILL.md +28 -14
- package/skills/qc/qa-analyst/DOC_GAPS.template.md +63 -0
- package/skills/qc/qa-analyst/acceptance-criteria.md +56 -0
- package/skills/qc/qa-analyst/business-rules.md +55 -0
- package/skills/qc/qa-analyst/data-flow.md +60 -0
- package/skills/qc/qa-analyst/spec-breakdown.md +57 -0
- package/skills/qc/qa-designer/e2e/journey.md +41 -0
- package/skills/qc/qa-designer/exploratory/charter.md +68 -0
- package/skills/qc/qa-designer/exploratory/explore-to-functional.md +43 -0
- package/skills/qc/qa-designer/functional/api.md +45 -0
- package/skills/qc/qa-designer/functional/gui-feature.md +46 -0
- package/skills/qc/qa-designer/functional/gui-screen.md +52 -0
- package/skills/qc/qa-designer/integration/api.md +42 -0
- package/skills/qc/qa-designer/integration/db.md +39 -0
- package/skills/qc/qa-designer/integration/gui.md +40 -0
- package/skills/qc/qa-designer/integration/kafka.md +40 -0
- package/skills/qc/qa-designer/non-functional.md +40 -0
- package/skills/qc/qa-planner/test-plan.md +120 -0
- package/skills/qc/qa-reviewer/script/e2e.md +87 -0
- package/skills/qc/qa-reviewer/script/exploratory.md +45 -0
- package/skills/qc/qa-reviewer/script/functional.md +101 -0
- package/skills/qc/qa-reviewer/script/integration.md +91 -0
- package/skills/qc/qa-reviewer/script/non-functional.md +126 -0
- package/skills/qc/qa-reviewer/test-case/e2e.md +73 -0
- package/skills/qc/qa-reviewer/test-case/exploratory.md +43 -0
- package/skills/qc/qa-reviewer/test-case/functional.md +76 -0
- package/skills/qc/qa-reviewer/test-case/integration.md +69 -0
- package/skills/qc/qa-reviewer/test-case/non-functional.md +73 -0
- package/skills/qc/qa-runner/e2e.md +49 -0
- package/skills/qc/qa-runner/exploratory/session.md +36 -0
- package/skills/qc/qa-runner/functional/api.md +35 -0
- package/skills/qc/qa-runner/functional/gui-feature.md +51 -0
- package/skills/qc/qa-runner/functional/gui-screen.md +55 -0
- package/skills/qc/qa-runner/integration.md +47 -0
- package/skills/qc/qa-runner/non-functional.md +49 -0
- package/skills/qc/qa-runner/report/report.md +37 -0
- package/skills/setup-ai-first/SKILL.md +14 -7
- package/skills/spec/SKILL.md +28 -14
- package/skills/test/SKILL.md +121 -54
- package/skills/test/SKILL.tmpl +9 -9
- package/steps/capture-lesson.md +1 -1
- package/steps/context-loader.md +35 -12
- package/steps/report-footer.md +14 -7
- package/steps/review-fanout.md +138 -0
- package/steps/spawn-agent.md +1 -1
- package/steps/trace-mirror.md +18 -0
- package/templates/design-spec.template.md +16 -8
- package/templates/project-context.yaml +8 -0
- package/ARCHITECTURE.md +0 -247
|
@@ -0,0 +1,448 @@
|
|
|
1
|
+
[📚 Docs](../../README.md) › [Guides](../README.md) › [Developer](README.md) › Tình huống thực tế
|
|
2
|
+
|
|
3
|
+
# Tình Huống Thực Tế
|
|
4
|
+
|
|
5
|
+
- [1. Nhận PRD + BDD mới và bắt đầu work](#tình-huống-1-nhận-prd--bdd-mới-và-bắt-đầu-work)
|
|
6
|
+
- [2. Đọc và hiểu System BDD (BE dev)](#tình-huống-2-đọc-và-hiểu-system-bdd-be-dev)
|
|
7
|
+
- [3. Đọc Web/App BDD (FE/App dev)](#tình-huống-3-đọc-webapp-bdd-feapp-dev)
|
|
8
|
+
- [4. PRD thay đổi mid-sprint](#tình-huống-4-prd-thay-đổi-mid-sprint)
|
|
9
|
+
- [4b. Chờ API design — BE + FE/App đồng thuận](#tình-huống-4b-chờ-api-design--be--feapp-đồng-thuận)
|
|
10
|
+
- [5. Nhận bug report từ Tester](#tình-huống-5-nhận-bug-report-từ-tester)
|
|
11
|
+
- [6. Nhận Design Spec + BDD từ PO (FE/App)](#tình-huống-6-nhận-design-spec--bdd-từ-po-feapp)
|
|
12
|
+
- [7b. Brownfield — API đã tồn tại](#tình-huống-7b-brownfield--api-đã-tồn-tại-trên-hệ-thống-cũ)
|
|
13
|
+
- [7. Setup service submodule (Umbrella)](#tình-huống-7-setup-service-submodule-umbrella-mode)
|
|
14
|
+
- [8. Validate traces trước khi tạo PR lớn](#tình-huống-8-validate-traces-trước-khi-tạo-pr-lớn)
|
|
15
|
+
|
|
16
|
+
## Tình huống 1: Nhận PRD + BDD mới và bắt đầu work
|
|
17
|
+
|
|
18
|
+
**Bối cảnh:** PO thông báo PRD `FT-042-checkout.md` và BDD đã approved, sẵn sàng implement.
|
|
19
|
+
```
|
|
20
|
+
1. git submodule update --remote my-project-specs
|
|
21
|
+
(lấy PRD + BDD mới nhất từ PO)
|
|
22
|
+
|
|
23
|
+
2. /review-context my-project-specs/specs/prd/payment/FT-042-checkout.md
|
|
24
|
+
→ Kiểm tra @trace.status = approved (không code khi còn draft)
|
|
25
|
+
→ Đọc kỹ AC, UC, BR
|
|
26
|
+
|
|
27
|
+
3. Đọc BDD tương ứng theo platform của mình:
|
|
28
|
+
FE/Web: my-project-specs/specs/bdd/payment/web/FT-042-UC*.feature
|
|
29
|
+
App: my-project-specs/specs/bdd/payment/app/FT-042-UC*.feature
|
|
30
|
+
BE: my-project-specs/specs/bdd/payment/system/FT-042-UC*.feature
|
|
31
|
+
|
|
32
|
+
4. Nếu có thắc mắc về PRD hoặc BDD → hỏi PO ngay, không tự suy diễn
|
|
33
|
+
Ví dụ: "BR5 trong System BDD nói 'kiểm tra giới hạn thanh toán' — limit này
|
|
34
|
+
có khác nhau theo tier user không? BDD không chỉ rõ."
|
|
35
|
+
|
|
36
|
+
5. Bắt đầu: /generate-tech-docs dựa trên BDD
|
|
37
|
+
```
|
|
38
|
+
**Lưu ý:** Nếu `/review-context` báo P0 warning (domain không match config) → **dừng lại**, báo PO/DevOps fix config trước.
|
|
39
|
+
|
|
40
|
+
## Tình huống 2: Đọc và hiểu System BDD (BE dev)
|
|
41
|
+
|
|
42
|
+
**Bối cảnh:** BE dev nhận thông báo BDD đã sẵn sàng tại `specs/bdd/auth/system/`.
|
|
43
|
+
|
|
44
|
+
**System BDD tập trung vào:** API contracts được tổng hợp từ FE + App BDD · Business rule enforcement tại system level · Data contracts (request/response shape) · Cross-platform consistency.
|
|
45
|
+
```
|
|
46
|
+
# Đọc file BDD (không generate):
|
|
47
|
+
my-project-specs/specs/bdd/auth/system/FT-001-UC1-login-system.feature
|
|
48
|
+
```
|
|
49
|
+
```gherkin
|
|
50
|
+
# Ví dụ System BDD do PO gen (tổng hợp từ web + app BDD)
|
|
51
|
+
Feature: User Authentication — System Contract
|
|
52
|
+
# @trace.prd: FT-001
|
|
53
|
+
# @trace.platform: system
|
|
54
|
+
|
|
55
|
+
Scenario: Successful login returns token and profile
|
|
56
|
+
Given a registered user with valid credentials
|
|
57
|
+
When the system receives a login request
|
|
58
|
+
Then the system returns an auth token
|
|
59
|
+
And the system returns the user profile
|
|
60
|
+
And the session is valid for 3600 seconds
|
|
61
|
+
|
|
62
|
+
Scenario: Account locked after 5 failed attempts
|
|
63
|
+
Given a user with 4 failed login attempts
|
|
64
|
+
When the system receives a 5th failed login
|
|
65
|
+
Then the system locks the account for 30 minutes
|
|
66
|
+
And the system signals the locked state with remaining time
|
|
67
|
+
```
|
|
68
|
+
BE dev dùng System BDD để: thiết kế API endpoint + response schema (`/generate-tech-docs`) · generate code skeleton (`/generate-code`) · viết integration tests (`/dev-gen-test`).
|
|
69
|
+
|
|
70
|
+
## Tình huống 3: Đọc Web/App BDD (FE/App dev)
|
|
71
|
+
|
|
72
|
+
**Bối cảnh:** FE dev nhận thông báo BDD web đã sẵn sàng tại `specs/bdd/auth/web/`.
|
|
73
|
+
```
|
|
74
|
+
# Đọc file BDD (không generate):
|
|
75
|
+
my-project-specs/specs/bdd/auth/web/FT-001-UC1-login-web.feature
|
|
76
|
+
```
|
|
77
|
+
```gherkin
|
|
78
|
+
# Ví dụ Web BDD do PO gen (vocabulary: clicks, sees, navigates)
|
|
79
|
+
Scenario: User sees error after wrong password
|
|
80
|
+
Given user is on the Login screen
|
|
81
|
+
When user submits login with wrong password
|
|
82
|
+
Then user sees "Sai mật khẩu" error
|
|
83
|
+
And the password field is cleared
|
|
84
|
+
|
|
85
|
+
Scenario: Account locked — countdown shown
|
|
86
|
+
Given user has submitted wrong password 5 times
|
|
87
|
+
Then user sees "Tài khoản bị khoá. Thử lại sau 29:45"
|
|
88
|
+
And the countdown decrements every second
|
|
89
|
+
```
|
|
90
|
+
FE dev dùng Web BDD để:
|
|
91
|
+
1. Thiết kế component spec + API integration plan (`/generate-tech-docs`)
|
|
92
|
+
2. Gen UI + mock adapter từ System BDD contract (`/generate-code --phase=ui`) → Mock adapter trả về fixture data đúng với BDD `Then` clauses → Tester test toàn bộ FE flow ngay, không cần chờ BE
|
|
93
|
+
3. [Trong khi đó — tham gia review API contract, sign-off T7 gate]
|
|
94
|
+
4. Khi sign-off done → wire real API (`/generate-code --phase=integration`)
|
|
95
|
+
5. Viết E2E tests với Playwright/Cypress (`/dev-gen-test`)
|
|
96
|
+
|
|
97
|
+
## Tình huống 4: PRD thay đổi mid-sprint
|
|
98
|
+
|
|
99
|
+
**Bối cảnh:** PO update PRD `FT-042` từ v1.0 → v1.1 khi dev đang code.
|
|
100
|
+
```
|
|
101
|
+
PO notify: "FT-042 updated — BR7 thay đổi giới hạn từ 5tr → 10tr"
|
|
102
|
+
│
|
|
103
|
+
▼
|
|
104
|
+
Dev chạy:
|
|
105
|
+
/review-context specs/prd/payment/FT-042-checkout.md
|
|
106
|
+
→ Xem diff từ v1.0 sang v1.1 (agent highlight thay đổi)
|
|
107
|
+
│
|
|
108
|
+
▼
|
|
109
|
+
Đánh giá impact:
|
|
110
|
+
- BDD bị ảnh hưởng? → thông báo PO để PO update BDD trong spec repo, rồi pull lại
|
|
111
|
+
- Tech Docs bị ảnh hưởng? → update API spec
|
|
112
|
+
- Code bị ảnh hưởng? → update logic + tests
|
|
113
|
+
│
|
|
114
|
+
▼
|
|
115
|
+
/validate-traces
|
|
116
|
+
→ Đảm bảo không có trace nào còn trỏ về spec cũ
|
|
117
|
+
```
|
|
118
|
+
**Nguyên tắc:** Không merge code khi traces broken. Fix traces trước.
|
|
119
|
+
|
|
120
|
+
## Tình huống 4b: Chờ API design — BE + FE/App đồng thuận
|
|
121
|
+
|
|
122
|
+
**Bối cảnh:** System BDD đã gen, BE dev bắt đầu `/generate-tech-docs` nhưng FE/App chưa confirm API contract. **Trạng thái tech docs trong thời gian này:** `@trace.status: in-review`.
|
|
123
|
+
```
|
|
124
|
+
BE dev: /generate-tech-docs auth/FT-001-UC1
|
|
125
|
+
# Umbrella mode (có spec_source): output nằm trong SPEC REPO chung
|
|
126
|
+
→ Output: free-trial-specs/specs/tech-docs/auth/FT-001-UC1-tech-design.md
|
|
127
|
+
# Single-service (không có spec_source): output nằm tại project root
|
|
128
|
+
→ Output: specs/tech-docs/auth/FT-001-UC1-tech-design.md
|
|
129
|
+
→ @trace.status: draft
|
|
130
|
+
→ @trace.sign_off: { be_team: done, fe_team: pending, app_team: pending, sa: pending }
|
|
131
|
+
→ Publish: commit + push file lên spec repo (2-layer) để FE/App `/sync` đọc được
|
|
132
|
+
|
|
133
|
+
BE dev: /review-tech-docs free-trial-specs/specs/tech-docs/auth/FT-001-UC1-tech-design.md
|
|
134
|
+
→ Chạy T1–T7 (bao gồm T7 cross-team contract check)
|
|
135
|
+
→ Report: "Sign-off gate: 🔒 BLOCKED — pending: fe_team, app_team, sa"
|
|
136
|
+
```
|
|
137
|
+
**FE dev review API contract:**
|
|
138
|
+
```
|
|
139
|
+
# FE dev mở tech-design file → xem API contract section
|
|
140
|
+
# Xác nhận: response fields có đủ cho web BDD expectations không?
|
|
141
|
+
# Nếu ok → thêm comment hoặc báo BE dev cập nhật sign_off
|
|
142
|
+
```
|
|
143
|
+
Khi FE/App confirm xong → BE dev update header:
|
|
144
|
+
```yaml
|
|
145
|
+
# @trace.sign_off:
|
|
146
|
+
# be_team: done
|
|
147
|
+
# fe_team: done ← FE đã confirm
|
|
148
|
+
# app_team: done ← App đã confirm
|
|
149
|
+
# sa: done ← SA đã approve
|
|
150
|
+
```
|
|
151
|
+
```
|
|
152
|
+
BE dev: /review-tech-docs --resume {tech-design-file}
|
|
153
|
+
→ Sign-off gate: ✅ READY
|
|
154
|
+
→ @trace.status: approved
|
|
155
|
+
→ BE có thể chạy /generate-code
|
|
156
|
+
→ FE chạy /generate-code --phase=integration để wire API thật
|
|
157
|
+
```
|
|
158
|
+
```
|
|
159
|
+
# FE — sau khi sign-off gate approved:
|
|
160
|
+
/generate-code --phase=integration auth/FT-001-UC1
|
|
161
|
+
→ Reads existing mock adapter interface ({UC-ID}ApiPort)
|
|
162
|
+
→ Generates real API adapter với calls đến endpoints trong tech-doc
|
|
163
|
+
→ Flips DI/env flag: service dùng real adapter thay mock
|
|
164
|
+
→ Mock adapter giữ lại cho unit test
|
|
165
|
+
```
|
|
166
|
+
**Nguyên tắc:**
|
|
167
|
+
- `/generate-code` (không phase flag) cho BE trả về warning nếu tech docs status là `in-review` hoặc `draft`.
|
|
168
|
+
- FE dùng `--phase=ui` được ngay sau khi đọc BDD — không cần chờ.
|
|
169
|
+
- FE dùng `--phase=integration` chỉ sau khi sign-off gate `approved`.
|
|
170
|
+
|
|
171
|
+
## Tình huống 5: Nhận bug report từ Tester
|
|
172
|
+
|
|
173
|
+
**Bối cảnh:** Tester gửi bug report theo đúng format spec-driven, có đầy đủ spec context.
|
|
174
|
+
```
|
|
175
|
+
Bug ID : BUG-20260605-003
|
|
176
|
+
Feature : FT-001 — User Login
|
|
177
|
+
Service : BE
|
|
178
|
+
Severity : Major
|
|
179
|
+
|
|
180
|
+
Spec context:
|
|
181
|
+
PRD : specs/prd/auth/FT-001-login.md (v1.0)
|
|
182
|
+
BDD : free-trial-be/specs/bdd/auth/FT-001-login.feature
|
|
183
|
+
→ Scenario: "Lock account after 5 failed attempts"
|
|
184
|
+
Tech Doc : free-trial-specs/specs/tech-docs/auth/FT-001-auth-api.md
|
|
185
|
+
|
|
186
|
+
AC bị vi phạm:
|
|
187
|
+
AC3: "Sai password 5 lần liên tiếp → khoá tài khoản 30 phút"
|
|
188
|
+
|
|
189
|
+
BDD Scenario bị fail:
|
|
190
|
+
Given : user has 0 failed attempts
|
|
191
|
+
When : login with wrong password 5 times
|
|
192
|
+
Then : 5th attempt returns 423 Locked AND retry_after = 1800
|
|
193
|
+
|
|
194
|
+
Actual: 5th attempt trả 401, không có retry_after, tài khoản không bị khoá
|
|
195
|
+
```
|
|
196
|
+
|
|
197
|
+
**Bước 1 — Tìm code implement scenario bị fail:**
|
|
198
|
+
```
|
|
199
|
+
Đọc theo thứ tự: PRD → BDD → Code
|
|
200
|
+
|
|
201
|
+
PRD AC3: "5 lần sai → khoá 30 phút" → rõ ràng ✅
|
|
202
|
+
BDD SC3: "Then 423 Locked, retry_after=1800" → đúng theo PRD ✅
|
|
203
|
+
Code: ??? → kiểm tra tiếp
|
|
204
|
+
```
|
|
205
|
+
Chạy:
|
|
206
|
+
```
|
|
207
|
+
/fix-bug "BUG-20260605-003: FT-001-UC2-BR3 — account not locked after 5 failures
|
|
208
|
+
PRD: specs/prd/auth/FT-001-login.md
|
|
209
|
+
BDD: free-trial-be/specs/bdd/auth/FT-001-login.feature"
|
|
210
|
+
```
|
|
211
|
+
Agent sẽ: đọc BDD scenario được chỉ định · tìm code implement scenario đó (theo `@trace.bdd`) · so sánh logic thực tế vs spec · propose fix có giải thích.
|
|
212
|
+
|
|
213
|
+
**Bước 2 — Xác định bug thuộc layer nào:** Đọc theo thứ tự PRD → BDD → Code để tìm chỗ lệch. Có 3 khả năng:
|
|
214
|
+
|
|
215
|
+
| PRD | BDD | Code | → Fix ở đâu |
|
|
216
|
+
|---|---|---|---|
|
|
217
|
+
| ✅ rõ | ✅ đúng | ❌ sai | Fix code |
|
|
218
|
+
| ✅ rõ | ❌ sai | ❌ sai | Fix BDD + code |
|
|
219
|
+
| ❌ mơ hồ | bất kỳ | bất kỳ | Hỏi PO trước, không tự fix |
|
|
220
|
+
|
|
221
|
+
> Flow đầy đủ cho cả 6 cases (bao gồm PRD change, Design Spec bug, env bug) và cách phối hợp với PO/Tester: xem [Operations › Bug Flow](../../04-operations/bug-flow.md).
|
|
222
|
+
|
|
223
|
+
**Bước 3 — Sau khi fix:**
|
|
224
|
+
```
|
|
225
|
+
/validate-traces
|
|
226
|
+
→ Đảm bảo @trace.bdd trong code vẫn trỏ đúng BDD scenario
|
|
227
|
+
→ Không có trace broken
|
|
228
|
+
|
|
229
|
+
/dev-run-test
|
|
230
|
+
→ BDD pass = fix đúng theo spec
|
|
231
|
+
|
|
232
|
+
Notify tester:
|
|
233
|
+
"BUG-20260605-003 fixed — deploy to staging [link commit/PR]
|
|
234
|
+
Root cause: Case A — code dùng > thay vì >=
|
|
235
|
+
BDD: không đổi (spec đã đúng)
|
|
236
|
+
Re-test: FT-001-UC2-SC3"
|
|
237
|
+
```
|
|
238
|
+
|
|
239
|
+
## Tình huống 6: Nhận Design Spec + BDD từ PO (FE/App)
|
|
240
|
+
|
|
241
|
+
**Bối cảnh:** PO tạo Design Spec + BDD web cho tính năng checkout — FE cần implement.
|
|
242
|
+
```
|
|
243
|
+
PO thông báo: "FT-042 Design Spec + BDD đã sẵn sàng"
|
|
244
|
+
│
|
|
245
|
+
▼
|
|
246
|
+
git submodule update --remote my-project-specs
|
|
247
|
+
|
|
248
|
+
FE dev đọc:
|
|
249
|
+
- my-project-specs/specs/prd/payment/FT-042-checkout.md (business rules)
|
|
250
|
+
- my-project-specs/specs/design-spec/payment/FT-042-*.md (screens, components)
|
|
251
|
+
- my-project-specs/specs/bdd/payment/web/FT-042-UC*.feature (BDD đã gen sẵn)
|
|
252
|
+
│
|
|
253
|
+
▼
|
|
254
|
+
Bật Figma Dev Mode MCP (nếu dùng Figma — để FE codegen chính xác):
|
|
255
|
+
→ Mở Figma DESKTOP app + enable Dev Mode MCP server (local, http://127.0.0.1:3845)
|
|
256
|
+
→ `/generate-code` (FE/App UI) tự detect server local + prompt nếu chưa bật
|
|
257
|
+
→ Dùng real tokens, components, Code Connect thay vì web link → codegen sát design hơn
|
|
258
|
+
│
|
|
259
|
+
▼
|
|
260
|
+
/generate-tech-docs payment/FT-042-UC1
|
|
261
|
+
→ Gen component spec, API integration plan dựa trên Design Spec + BDD
|
|
262
|
+
│
|
|
263
|
+
▼
|
|
264
|
+
/generate-code payment/FT-042-UC1 --phase=ui
|
|
265
|
+
→ Gen UI components + mock API adapter (fixture từ System BDD Then clauses)
|
|
266
|
+
→ FE codegen đọc Figma Dev Mode MCP local nếu đang bật (tokens/components/Code Connect)
|
|
267
|
+
→ Tester có thể test FE ngay
|
|
268
|
+
│ [trong khi đó: tham gia review API contract — T7 sign-off gate]
|
|
269
|
+
▼
|
|
270
|
+
[Nhận thông báo: sign-off gate approved]
|
|
271
|
+
│
|
|
272
|
+
▼
|
|
273
|
+
/generate-code payment/FT-042-UC1 --phase=integration
|
|
274
|
+
→ Wire real API adapter thay thế mock
|
|
275
|
+
→ /dev-gen-test payment/FT-042-UC1
|
|
276
|
+
→ /review-code {files-changed}
|
|
277
|
+
→ /dev-run-test
|
|
278
|
+
```
|
|
279
|
+
**Lưu ý:** BE không cần đọc Design Spec — chỉ đọc System BDD tại `specs/bdd/{domain}/system/`.
|
|
280
|
+
|
|
281
|
+
## Tình huống 7b: Brownfield — API đã tồn tại trên hệ thống cũ
|
|
282
|
+
|
|
283
|
+
**Bối cảnh:** PO viết PRD cho feature mới nhưng BE API đã có sẵn trên hệ thống cũ, chưa có tài liệu. PO khai báo luôn trong PRD thay vì thiết kế lại.
|
|
284
|
+
|
|
285
|
+
**Dấu hiệu nhận ra:** PRD Metadata có `| **API Source** | existing |` · PRD có section "Existing API Contract" với bảng endpoint + response.
|
|
286
|
+
|
|
287
|
+
**Dev workflow (đơn giản hơn greenfield):**
|
|
288
|
+
```
|
|
289
|
+
git submodule update --remote my-project-specs
|
|
290
|
+
|
|
291
|
+
1. /review-context → đọc PRD + BDD
|
|
292
|
+
→ BDD system đã dùng contract sẵn có từ PRD (không synthesis)
|
|
293
|
+
→ @trace.api_source: existing trong BDD header
|
|
294
|
+
|
|
295
|
+
2. /generate-tech-docs {feature-file}
|
|
296
|
+
→ Mode: Reverse-document
|
|
297
|
+
→ §2 API Endpoints: mô tả lại API đã tồn tại từ bảng PRD
|
|
298
|
+
→ Ghi chú gaps nếu contract thực tế khác BDD expectations
|
|
299
|
+
|
|
300
|
+
3. /review-tech-docs {tech-design-file}
|
|
301
|
+
→ T7 sign-off gate: tự động SKIP (không có API design mới)
|
|
302
|
+
→ Chỉ review T1–T6 (architecture, entity, BDD traceability, ...)
|
|
303
|
+
→ Approved nhanh hơn
|
|
304
|
+
|
|
305
|
+
4. /generate-code {feature-file} ← không cần --phase
|
|
306
|
+
→ API đã live, gen real adapter trực tiếp
|
|
307
|
+
```
|
|
308
|
+
|
|
309
|
+
**Điểm khác biệt so với greenfield:**
|
|
310
|
+
|
|
311
|
+
| | Greenfield | Brownfield (API existing) |
|
|
312
|
+
|---|---|---|
|
|
313
|
+
| System BDD | Synthesize từ FE + App BDD | Dùng PRD contract trực tiếp |
|
|
314
|
+
| T7 gate | Bắt buộc | Tự động skip |
|
|
315
|
+
| `--phase=ui` | Cần nếu BE chưa ready | Không cần |
|
|
316
|
+
| `generate-tech-docs` | Design mới | Reverse-document |
|
|
317
|
+
|
|
318
|
+
## Tình huống 7: Setup service submodule (Umbrella mode)
|
|
319
|
+
|
|
320
|
+
**Bối cảnh:** Project dùng umbrella repo. Dev được assign vào service `mass-product-be`.
|
|
321
|
+
|
|
322
|
+
**Setup lần đầu:**
|
|
323
|
+
```bash
|
|
324
|
+
# 1. Clone umbrella repo
|
|
325
|
+
git clone {umbrella-repo-url} mass-product
|
|
326
|
+
cd mass-product
|
|
327
|
+
|
|
328
|
+
# 2. Mở Claude Code TẠI umbrella root (QUAN TRỌNG)
|
|
329
|
+
code . ← hoặc claude .
|
|
330
|
+
|
|
331
|
+
# 3. Chạy một lệnh duy nhất — setup toàn bộ
|
|
332
|
+
/sync
|
|
333
|
+
# → tự detect setup mode (submodule chưa init)
|
|
334
|
+
# → git pull + git submodule update --init --recursive --remote
|
|
335
|
+
# → validate service configs (cảnh báo nếu thiếu .agent/project-context.yaml)
|
|
336
|
+
# → sync Living Docs panel
|
|
337
|
+
|
|
338
|
+
# 4. Framework tự detect umbrella mode từ project-context.yaml
|
|
339
|
+
# Khi chạy /review-context với PRD có @trace.domain: be
|
|
340
|
+
# → tự động route tới mass-product-be/specs/bdd/
|
|
341
|
+
```
|
|
342
|
+
|
|
343
|
+
**Update hằng ngày — cũng chỉ 1 lệnh:**
|
|
344
|
+
```bash
|
|
345
|
+
/sync
|
|
346
|
+
# → git pull + submodule update --remote
|
|
347
|
+
# → refresh Living Docs
|
|
348
|
+
```
|
|
349
|
+
|
|
350
|
+
**project-context.yaml của umbrella:**
|
|
351
|
+
```yaml
|
|
352
|
+
setup:
|
|
353
|
+
mode: umbrella
|
|
354
|
+
spec_source: "mass-product-spec"
|
|
355
|
+
services:
|
|
356
|
+
be:
|
|
357
|
+
path: "mass-product-be"
|
|
358
|
+
module: "NestJS"
|
|
359
|
+
specs_dir: "mass-product-be/specs/bdd"
|
|
360
|
+
web:
|
|
361
|
+
path: "mass-product-web"
|
|
362
|
+
module: "NextJS"
|
|
363
|
+
specs_dir: "mass-product-web/specs/bdd"
|
|
364
|
+
```
|
|
365
|
+
|
|
366
|
+
> **Tech-docs (API contract):** không khai trong `services`. Khi `setup.spec_source` được set, BE tech-design (API contract) **LUÔN** nằm tại `{spec_source}/specs/tech-docs` (vd `mass-product-spec/specs/tech-docs`) — context-loader tự route `tech_docs_dir` về đó. BE generate xong → commit + push lên spec repo; FE/App đọc qua `/sync` + `/generate-code --phase=integration`. Per-service tech-docs **chỉ** khi KHÔNG có `spec_source`.
|
|
367
|
+
>
|
|
368
|
+
> **Bắt buộc:** Mỗi service submodule cũng cần file `.agent/project-context.yaml` riêng. Framework đọc file này (context-loader Step 1.6) để lấy đúng `test_command` và `build_command` khi `/dev-run-test` hoặc `/dev-gen-test` chạy từ umbrella root.
|
|
369
|
+
|
|
370
|
+
**project-context.yaml của từng service submodule:**
|
|
371
|
+
```yaml
|
|
372
|
+
# mass-product-be/.agent/project-context.yaml
|
|
373
|
+
tech_stack:
|
|
374
|
+
language: "TypeScript"
|
|
375
|
+
framework: "NestJS"
|
|
376
|
+
module: "nestjs"
|
|
377
|
+
|
|
378
|
+
conventions:
|
|
379
|
+
test_command: "npm test"
|
|
380
|
+
build_command: "npm run build"
|
|
381
|
+
|
|
382
|
+
paths:
|
|
383
|
+
trace_dir: ".trace"
|
|
384
|
+
```
|
|
385
|
+
```yaml
|
|
386
|
+
# mass-product-web/.agent/project-context.yaml
|
|
387
|
+
tech_stack:
|
|
388
|
+
language: "TypeScript"
|
|
389
|
+
framework: "Next.js 14"
|
|
390
|
+
module: "nextjs"
|
|
391
|
+
|
|
392
|
+
conventions:
|
|
393
|
+
test_command: "npx vitest run"
|
|
394
|
+
build_command: "npm run build"
|
|
395
|
+
|
|
396
|
+
paths:
|
|
397
|
+
trace_dir: ".trace"
|
|
398
|
+
```
|
|
399
|
+
|
|
400
|
+
Khi `/dev-run-test` chạy từ umbrella root cho một UC thuộc domain `be`:
|
|
401
|
+
1. Step 1.5 detect `service_root = "mass-product-be"`
|
|
402
|
+
2. Step 1.6 load `mass-product-be/.agent/project-context.yaml` → `test_command = "npm test"`
|
|
403
|
+
3. Lệnh test chạy: `cd mass-product-be && npm test`
|
|
404
|
+
|
|
405
|
+
**Commit 2 lớp (bắt buộc):**
|
|
406
|
+
```bash
|
|
407
|
+
# Lớp 1: Commit trong service submodule
|
|
408
|
+
cd mass-product-be
|
|
409
|
+
git add specs/bdd/auth/FT-001-login.feature
|
|
410
|
+
git commit -m "feat(bdd): add login BDD scenarios — FT-001"
|
|
411
|
+
git push origin feature/ft-001-login
|
|
412
|
+
|
|
413
|
+
# Lớp 2: Update pointer tại umbrella
|
|
414
|
+
cd .. ← về umbrella root
|
|
415
|
+
git add mass-product-be
|
|
416
|
+
git commit -m "chore: update mass-product-be submodule pointer — FT-001 BDD"
|
|
417
|
+
git push
|
|
418
|
+
```
|
|
419
|
+
**Không commit lớp 2 → umbrella repo vẫn trỏ về commit cũ của service.**
|
|
420
|
+
|
|
421
|
+
## Tình huống 8: Validate traces trước khi tạo PR lớn
|
|
422
|
+
|
|
423
|
+
**Bối cảnh:** Dev refactor module Auth — đổi tên `AuthService` → `IdentityService`.
|
|
424
|
+
```
|
|
425
|
+
Sau khi refactor xong:
|
|
426
|
+
|
|
427
|
+
/validate-traces
|
|
428
|
+
│
|
|
429
|
+
▼
|
|
430
|
+
Agent kiểm tra:
|
|
431
|
+
- BDD có @trace.module: AuthService → BROKEN (class không còn tồn tại)
|
|
432
|
+
- Code comments @trace.bdd: FT-001-UC1-SC1 → còn hợp lệ không?
|
|
433
|
+
- Tech Docs mention "AuthService" → stale reference
|
|
434
|
+
│
|
|
435
|
+
▼
|
|
436
|
+
Report:
|
|
437
|
+
❌ BROKEN specs/bdd/auth/FT-001-login.feature @trace.module: AuthService (not found)
|
|
438
|
+
❌ BROKEN specs/tech-docs/auth/FT-001-auth-api.md "AuthService" referenced 7 times
|
|
439
|
+
✅ OK src/identity/identity.service.ts @trace.bdd: FT-001-UC1-SC1
|
|
440
|
+
│
|
|
441
|
+
▼
|
|
442
|
+
Fix: Update @trace.module và references → re-run /validate-traces → all green → tạo PR
|
|
443
|
+
```
|
|
444
|
+
**Quy tắc:** PR không được merge khi còn broken traces.
|
|
445
|
+
|
|
446
|
+
---
|
|
447
|
+
|
|
448
|
+
← [Workflow](workflow.md) · Tiếp theo: [Checklist trước khi tạo PR](pr-checklist.md)
|
|
@@ -0,0 +1,59 @@
|
|
|
1
|
+
[📚 Docs](../../README.md) › [Guides](../README.md) › [Developer](README.md) › Workflow
|
|
2
|
+
|
|
3
|
+
# Workflow Cơ Bản
|
|
4
|
+
|
|
5
|
+
```
|
|
6
|
+
Nhận thông báo PRD + BDD mới từ PO
|
|
7
|
+
│
|
|
8
|
+
▼
|
|
9
|
+
git submodule update --remote my-project-specs
|
|
10
|
+
(lấy spec mới nhất, bao gồm cả BDD đã được PO gen)
|
|
11
|
+
│
|
|
12
|
+
▼
|
|
13
|
+
/review-context {prd-file}
|
|
14
|
+
→ Kiểm tra @trace.domain, @trace.status = approved
|
|
15
|
+
→ Đọc hiểu AC, UC, BR trong PRD
|
|
16
|
+
→ Đọc BDD tương ứng trong specs/bdd/{domain}/{platform}/
|
|
17
|
+
→ Nếu có gì không rõ: hỏi PO TRƯỚC khi tiếp tục
|
|
18
|
+
│
|
|
19
|
+
▼
|
|
20
|
+
(Đọc BDD từ spec submodule — KHÔNG tự generate BDD)
|
|
21
|
+
FE/Web: my-project-specs/specs/bdd/{domain}/web/{TICKET-ID}-UC*.feature
|
|
22
|
+
App: my-project-specs/specs/bdd/{domain}/app/{TICKET-ID}-UC*.feature
|
|
23
|
+
BE: my-project-specs/specs/bdd/{domain}/system/{TICKET-ID}-UC*.feature
|
|
24
|
+
│
|
|
25
|
+
▼
|
|
26
|
+
/generate-tech-docs {prd-file}
|
|
27
|
+
→ Gen API spec, DB schema, sequence diagram dựa trên BDD
|
|
28
|
+
→ /review-tech-docs để verify chất lượng
|
|
29
|
+
│
|
|
30
|
+
▼
|
|
31
|
+
# BE (hoặc full-stack không cần mock split):
|
|
32
|
+
/generate-code {bdd-file}
|
|
33
|
+
→ Gen code skeleton theo BDD + tech docs
|
|
34
|
+
→ Đảm bảo @trace.bdd comment trong code
|
|
35
|
+
|
|
36
|
+
# FE/App (2 phases — không cần chờ BE):
|
|
37
|
+
/generate-code {bdd-file} --phase=ui # Phase 1: UI + mock adapter
|
|
38
|
+
→ Tester test FE ngay (không cần BE ready)
|
|
39
|
+
/generate-code {bdd-file} --phase=integration # Phase 2: sau khi sign-off done
|
|
40
|
+
→ Wire real API thay thế mock
|
|
41
|
+
│
|
|
42
|
+
▼
|
|
43
|
+
/dev-gen-test {bdd-file}
|
|
44
|
+
→ Gen unit test (dev self-check, không phải coverage chính thức)
|
|
45
|
+
→ /dev-run-test để verify → ghi dev_selftest (pass/fail) + dev_selftest_at vào trace TSV
|
|
46
|
+
→ /validate-traces (hoặc /sync) để push trace lên spec-module Living Docs ({spec_source}/.living-docs/)
|
|
47
|
+
│
|
|
48
|
+
▼
|
|
49
|
+
/review-code {files}
|
|
50
|
+
→ 4 lens: Security / Performance / Architecture / Test Coverage
|
|
51
|
+
→ Fix issues trước khi tạo PR
|
|
52
|
+
│
|
|
53
|
+
▼
|
|
54
|
+
Tạo PR → notify PO/SA review
|
|
55
|
+
```
|
|
56
|
+
|
|
57
|
+
---
|
|
58
|
+
|
|
59
|
+
← [BDD & Trace System](bdd-and-trace.md) · Tiếp theo: [Tình huống thực tế](scenarios.md)
|
|
@@ -0,0 +1,77 @@
|
|
|
1
|
+
[📚 Docs](../../README.md) › [Guides](../README.md) › Product Owner / BA
|
|
2
|
+
|
|
3
|
+
# Hướng Dẫn PO/BA — Spec-Driven Dev
|
|
4
|
+
|
|
5
|
+
Tài liệu dành cho **Product Owner (PO)** và **Business Analyst (BA)** — vai trò, commands, các tình huống thực tế, và quy tắc viết PRD.
|
|
6
|
+
|
|
7
|
+
## Mục Lục
|
|
8
|
+
|
|
9
|
+
| Trang | Nội dung |
|
|
10
|
+
|---|---|
|
|
11
|
+
| [Commands](commands.md) | Bảng lệnh cho PO/BA · project lessons · xử lý feedback tester |
|
|
12
|
+
| [Tình huống thực tế](scenarios.md) | 11 scenario: tính năng mới, design spec, BDD, PRD thay đổi, conflict, brownfield, ... |
|
|
13
|
+
| [Quy tắc viết PRD](prd-writing-rules.md) | Platform-agnostic · testable AC · negative path · BR numbering |
|
|
14
|
+
| [Checklist handoff](handoff-checklist.md) | Checklist verify trước khi thông báo dev team bắt đầu |
|
|
15
|
+
|
|
16
|
+
## Vai Trò PO/BA Trong Framework
|
|
17
|
+
|
|
18
|
+
PO/BA là người duy nhất viết và approve tài liệu đầu vào + BDD:
|
|
19
|
+
|
|
20
|
+
```
|
|
21
|
+
PO/BA Dev team
|
|
22
|
+
────────────────────── ──────────────────────────────
|
|
23
|
+
/define-product đọc product-definition + BDD
|
|
24
|
+
/generate-prd → /review-context (xác nhận PRD)
|
|
25
|
+
/refine-prd (SA review) /generate-tech-docs
|
|
26
|
+
/review-context /generate-code + tests
|
|
27
|
+
/generate-design-spec → /review-code / /dev-run-test
|
|
28
|
+
/generate-bdd (web)
|
|
29
|
+
/generate-bdd (app)
|
|
30
|
+
/generate-bdd (system)
|
|
31
|
+
```
|
|
32
|
+
|
|
33
|
+
**PO/BA chịu trách nhiệm:**
|
|
34
|
+
- Đảm bảo PRD platform-agnostic (không chứa UI details hay API specs)
|
|
35
|
+
- Đặt `@trace.domain` đúng để dev team routing hoạt động
|
|
36
|
+
- Cập nhật `@trace.status: approved` trước khi handoff
|
|
37
|
+
- **Generate BDD cho tất cả platforms** (web, app, system) trong spec repo
|
|
38
|
+
- Thông báo dev team khi có PRD hoặc BDD mới/update
|
|
39
|
+
|
|
40
|
+
**PO/BA KHÔNG cần quan tâm:**
|
|
41
|
+
- Tech docs — dev team tự generate
|
|
42
|
+
- Source code — hoàn toàn dev team
|
|
43
|
+
|
|
44
|
+
### Tại sao BDD thuộc trách nhiệm của PO?
|
|
45
|
+
|
|
46
|
+
Đặt BDD trong spec repo giúp **tổng hợp toàn bộ tài liệu nghiệp vụ về một chỗ**.
|
|
47
|
+
|
|
48
|
+
```
|
|
49
|
+
PRD (WHAT + WHY) → BDD (HOW verified) → Code (HOW built)
|
|
50
|
+
↑ PO/BA ↑ PO/BA ↑ Dev
|
|
51
|
+
spec repo spec repo dev repo
|
|
52
|
+
```
|
|
53
|
+
|
|
54
|
+
> **Sau khi BDD approved → QC chạy pipeline tự động:** Khi BDD của một UC được approve, QC team chạy bộ lệnh native `/qc-analyze → /qc-plan → /qc-design-test → /qc-review → /qc-run-test → /qc-report` — đọc official `.feature`, generate + run Playwright/pytest, rồi ghi `qc_status` (official QC coverage) vào Living Docs. PO không chạy QC, nhưng nên biết bước này tồn tại ở downstream. Chi tiết: [QC Automation Guide](../qc-automation.md).
|
|
55
|
+
|
|
56
|
+
**3 lý do chính BDD thuộc PO:**
|
|
57
|
+
|
|
58
|
+
**1. Tổng hợp tài liệu nghiệp vụ tại một nơi** — Khi BDD nằm trong spec repo cùng PRD, mọi spec nghiệp vụ đều ở một chỗ — dễ review, audit, và generate báo cáo. Dev team đọc spec từ submodule, không cần tự suy ra scenarios.
|
|
59
|
+
|
|
60
|
+
**2. BDD viết ở mức nghiệp vụ — PO hiểu được** — BDD trong spec repo mô tả **hành vi hệ thống** bằng ngôn ngữ nghiệp vụ, không phải implementation detail:
|
|
61
|
+
|
|
62
|
+
| Platform | BDD trong spec repo (PO viết) | Test code (Dev viết) |
|
|
63
|
+
|---|---|---|
|
|
64
|
+
| Web | `When user submits login → sees dashboard` | `await page.click('#login-btn')` |
|
|
65
|
+
| App | `When user taps Login → navigates to Home` | `element(by.id('login')).tap()` |
|
|
66
|
+
| System | `When login request received → returns auth token` | `POST /api/auth/login assertions` |
|
|
67
|
+
|
|
68
|
+
**3. System BDD là nguồn sự thật cho BE** — tổng hợp từ FE + App BDD:
|
|
69
|
+
- FE BDD: "User expects auth token để truy cập tiếp"
|
|
70
|
+
- App BDD: "App expects user profile sau khi đăng nhập"
|
|
71
|
+
- System BDD: "BE phải trả `{ token, user_profile }` để thỏa mãn cả 2 platform"
|
|
72
|
+
|
|
73
|
+
> Xem [Concepts › Traceability](../../03-concepts/traceability.md) để hiểu trace chain PRD → BDD → Code.
|
|
74
|
+
|
|
75
|
+
---
|
|
76
|
+
|
|
77
|
+
*Xem thêm:* [Developer Guide](../developer/README.md) · [Tester Guide](../tester/README.md) · [QC Automation Guide](../qc-automation.md) · [Reference › Commands](../../05-reference/commands.md)
|
|
@@ -0,0 +1,30 @@
|
|
|
1
|
+
[📚 Docs](../../README.md) › [Guides](../README.md) › [Product Owner](README.md) › Commands
|
|
2
|
+
|
|
3
|
+
# Commands Dành Cho PO/BA
|
|
4
|
+
|
|
5
|
+
| Command | Mục đích | Khi nào dùng |
|
|
6
|
+
|---|---|---|
|
|
7
|
+
| `/setup-ai-first` | Khởi tạo spec repo | 1 lần duy nhất khi bắt đầu project |
|
|
8
|
+
| `/define-product` | Tạo product definition | Trước khi viết PRD — capture ý tưởng tính năng |
|
|
9
|
+
| `/generate-prd` | Tạo PRD từ product definition | Sau khi product definition đủ rõ |
|
|
10
|
+
| `/refine-prd` | Review PRD qua 4 lens (QA/DEV/SA/PO) — fan-out song song + critic loop → findings đầy đủ trong 1 lần chạy | Trước khi share với dev team |
|
|
11
|
+
| `/review-context` | Review chất lượng + consistency của PRD (P-B check groups fan-out song song) | Sau refine, trước khi approve |
|
|
12
|
+
| `/review-context --fix` | Auto-fix các lỗi nhỏ trong PRD | Khi có nhiều lỗi minor tự sửa được |
|
|
13
|
+
| `/review-context --resume` | Apply các fix sau Review Board | Sau khi review findings |
|
|
14
|
+
| `/generate-design-spec` | Tạo Design Spec cho FE/App (cần Figma link node-level `?node-id=` cho từng screen) | Sau khi PRD approved, trước khi generate BDD |
|
|
15
|
+
| `/generate-bdd` | Tạo BDD cho web / app / system | Sau khi PRD + Design Spec approved |
|
|
16
|
+
| `/learn {text}` | Ghi lại lỗi AI hay lặp khi gen PRD/BDD thành guardrail | Khi AI lặp lại cùng kiểu sai trong PRD/BDD |
|
|
17
|
+
|
|
18
|
+
> Danh mục đầy đủ mọi command: [Reference › Commands](../../05-reference/commands.md).
|
|
19
|
+
|
|
20
|
+
> **Brownfield shortcut:** Nếu API đã tồn tại trên hệ thống cũ, thêm `API Source: existing` vào PRD Metadata + điền section "Existing API Contract". BDD generation sẽ dùng contract đó trực tiếp, T7 sign-off gate tự động được bỏ qua. Chi tiết: [Tình huống 11](scenarios.md#tình-huống-11--brownfield-api-đã-tồn-tại-trên-hệ-thống-cũ).
|
|
21
|
+
|
|
22
|
+
> **Project Lessons:** Nếu AI cứ lặp lại một kiểu sai khi gen PRD/BDD (vd: quên negative path, dùng sai thuật ngữ), chạy `/learn "AI hay X, đúng phải Y"` (category `prd`/`bdd`). Lesson được nạp vào đầu mọi lệnh → AI không lặp lại. Commit file lessons để cả team dùng chung.
|
|
23
|
+
|
|
24
|
+
> **Xử lý feedback từ tester:** Tester `/report-bug` và `/propose-scenario` commit vào `{spec_source}/feedback/` của spec repo. PO/Dev **thấy chúng khi chạy `/sync`** (dòng `📥 New tester feedback` liệt kê bug + proposal mới kéo về). PO review proposal:
|
|
25
|
+
> - **Map vào AC sẵn có** → coverage gap thật → để Dev thêm scenario vào `.feature` (hoặc regen).
|
|
26
|
+
> - Là **PRD change request** (behavior chưa có trong PRD) → PO thêm/sửa AC → bump version → `/generate-bdd` lại.
|
|
27
|
+
|
|
28
|
+
---
|
|
29
|
+
|
|
30
|
+
← [Product Owner Guide](README.md) · Tiếp theo: [Tình huống thực tế](scenarios.md)
|
|
@@ -0,0 +1,42 @@
|
|
|
1
|
+
[📚 Docs](../../README.md) › [Guides](../README.md) › [Product Owner](README.md) › Checklist Handoff
|
|
2
|
+
|
|
3
|
+
# Checklist Handoff Cho Dev Team
|
|
4
|
+
|
|
5
|
+
Trước khi thông báo dev team bắt đầu, kiểm tra:
|
|
6
|
+
|
|
7
|
+
**Metadata:**
|
|
8
|
+
- [ ] `@trace.id` có và đúng format (vd: FEAT-01)
|
|
9
|
+
- [ ] `@trace.domain` có và khớp với domain names đã thống nhất với dev team
|
|
10
|
+
- [ ] `@trace.status: approved` (không phải draft/in-review)
|
|
11
|
+
- [ ] `@trace.version` có (vd: 1.0)
|
|
12
|
+
|
|
13
|
+
**Nội dung PRD:**
|
|
14
|
+
- [ ] Tất cả `{{PLACEHOLDER}}` đã được điền
|
|
15
|
+
- [ ] Ít nhất 1 UC với format `{TICKET-ID}-UC1`
|
|
16
|
+
- [ ] Mỗi UC có: Description, Actors, Preconditions, AC, BR
|
|
17
|
+
- [ ] Có cả happy path và error cases trong AC
|
|
18
|
+
- [ ] Không có UI details (button colors, API paths) trong PRD
|
|
19
|
+
- [ ] Changelog section có ít nhất 1 entry
|
|
20
|
+
|
|
21
|
+
**Framework check:**
|
|
22
|
+
- [ ] `/review-context` cho kết quả `APPROVED` (0 critical findings)
|
|
23
|
+
- [ ] Không có P3 conflict với PRD khác trong cùng domain
|
|
24
|
+
|
|
25
|
+
**Cho FE/App thêm:**
|
|
26
|
+
- [ ] Design Spec đã tạo và `@trace.status: approved` (không còn `draft`)
|
|
27
|
+
- [ ] Mỗi screen có Figma link node-level (`?node-id=`) — không screen nào bị flag ❌ Missing
|
|
28
|
+
- [ ] Designer đã sign-off (gate này bị BLOCKED nếu còn screen thiếu node-id link)
|
|
29
|
+
|
|
30
|
+
**BDD (bắt buộc trước khi handoff):**
|
|
31
|
+
- [ ] BDD web đã gen: `specs/bdd/{domain}/web/{TICKET-ID}-UC*.feature`
|
|
32
|
+
- [ ] BDD app đã gen: `specs/bdd/{domain}/app/{TICKET-ID}-UC*.feature` (nếu project có app)
|
|
33
|
+
- [ ] System BDD đã gen: `specs/bdd/{domain}/system/{TICKET-ID}-UC*.feature`
|
|
34
|
+
- [ ] Tất cả BDD files đã được review và không có `MISSING` trong Coverage Matrix
|
|
35
|
+
|
|
36
|
+
**Git:**
|
|
37
|
+
- [ ] Đã commit và push toàn bộ (`specs/prd/`, `specs/design-spec/`, `specs/bdd/`) lên spec repo
|
|
38
|
+
- [ ] Thông báo dev team domain name và BDD path để họ cập nhật `git submodule update --remote`
|
|
39
|
+
|
|
40
|
+
---
|
|
41
|
+
|
|
42
|
+
← [Quy tắc viết PRD](prd-writing-rules.md)
|