@damphuquy/agent-init 1.4.4 → 2.0.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.
@@ -1,24 +1,101 @@
1
1
  # Subagent Delegation & Orchestration Protocol
2
2
 
3
- <orchestration_protocol version="1.0">
3
+ <orchestration_protocol version="2.0">
4
4
 
5
5
  <description>
6
- Guidelines for delegating, isolating, and coordinating subagents during complex tasks.
6
+ Engineering guidelines for delegating, isolating, and coordinating subagents during complex, multi-phase tasks. Project-agnostic.
7
7
  </description>
8
8
 
9
- ## 1. Delegation Principles
10
- <delegation_rules>
11
- <rule id="explicit_scope">
12
- Always specify exact files, function signatures, and expected structured output formats.
9
+ ---
10
+
11
+ ## 1. Delegation Criteria & Workload Triage
12
+
13
+ <delegation_criteria>
14
+ Subagents should be spawned deliberately for bounded, high-leverage subtasks. Do NOT spawn subagents for trivial steps.
15
+
16
+ ### When to Delegate to a Subagent:
17
+ - **Isolated Research Spikes:** Exploring unfamiliar libraries, auditing legacy subsystems, or reading documentation without polluting the parent agent's context window.
18
+ - **Dedicated Quality Audits:** Independent security vulnerability audits, accessibility reviews (a11y), or strict lint/formatting passes.
19
+ - **Orthogonal Test Authoring:** Writing standalone unit test fixtures or integration harnesses for a frozen interface contract.
20
+ - **Parallel Independent Slices:** Implementing non-overlapping, orthogonal components that touch distinct file boundaries.
21
+
22
+ ### When Parent Agent MUST Retain Control (Do NOT Delegate):
23
+ - **Architectural Trade-offs & Decisions:** Authoring `decision.md` options and Gate 1 sign-offs.
24
+ - **Master Task Contract & Planning:** Defining `task.md`, vertical slice contracts in `plan.md`, and Gate 2 sign-offs.
25
+ - **Quality Gates & User Approval:** Conducting Gate G3 review sign-off and final `handoff.md` generation.
26
+ - **Interactive User Clarification:** Any prompt requiring direct user guidance or requirement resolution.
27
+ </delegation_criteria>
28
+
29
+ ---
30
+
31
+ ## 2. Context Containment & Scoping Rules
32
+
33
+ <context_containment>
34
+ <rule id="explicit_file_manifest">
35
+ Always provide the subagent with an explicit list of file paths to inspect or modify. Never prompt a subagent with open-ended instructions like "explore the project" or "look around the codebase".
36
+ </rule>
37
+
38
+ <rule id="port_and_interface_focus">
39
+ Constrain the subagent's inputs to relevant port interfaces, domain entities, and accompanying test suites. Keep infrastructure noise out of the subagent prompt.
40
+ </rule>
41
+
42
+ <rule id="structured_output_contract">
43
+ Always demand a structured output format from the subagent (e.g. unified diff, bulleted findings categorized by Confirmed/Observed/Hypothesized, or JSON/markdown table).
44
+ </rule>
45
+ </context_containment>
46
+
47
+ ---
48
+
49
+ ## 3. Concurrency, Isolation & Write Permissions
50
+
51
+ <concurrency_and_isolation>
52
+ <rule id="zero_write_collision">
53
+ Multiple subagents must NEVER be given write access to the same files or shared mutable database tables concurrently. Overlapping writes cause silent regressions and merge conflicts.
54
+ </rule>
55
+
56
+ <rule id="read_only_by_default">
57
+ Default subagents to read-only mode whenever possible (e.g. research, code exploration, audit). Only grant write permissions when the target files are strictly isolated to that subagent.
58
+ </rule>
59
+
60
+ <rule id="workspace_isolation">
61
+ If subagents support isolated workspaces (e.g. branch or worktree mode), use them for speculative spike experiments to ensure the parent working tree remains clean.
62
+ </rule>
63
+ </concurrency_and_isolation>
64
+
65
+ ---
66
+
67
+ ## 4. State Synchronization & Parent Re-Integration
68
+
69
+ <state_synchronization>
70
+ <rule id="parent_owns_master_state">
71
+ Subagents must NEVER directly edit the parent's master task artifacts (`task.md`, `state.md`, `review.md`). Only the parent agent reconciles subagent outputs into master state files.
72
+ </rule>
73
+
74
+ <rule id="verification_before_acceptance">
75
+ When a subagent returns modified source code or findings, the parent agent must inspect the diff and execute the slice's designated verifier command before accepting the work.
76
+ </rule>
77
+
78
+ <rule id="evidence_crystallization">
79
+ Extract verified facts from subagent reports and append them to `state.md > <verification_evidence>`. Discard transient subagent conversation logs to preserve context hygiene.
80
+ </rule>
81
+ </state_synchronization>
82
+
83
+ ---
84
+
85
+ ## 5. Reactive Coordination, Timeouts & Failure Recovery
86
+
87
+ <coordination_and_recovery>
88
+ <rule id="no_polling_loops">
89
+ Never implement sleep-and-poll loops (`while true; sleep 5; check_status`) to monitor subagents. Rely on the system's reactive message wakeup mechanism to resume execution upon subagent completion.
13
90
  </rule>
14
91
 
15
- <rule id="context_containment">
16
- Do not grant unbounded repo access; constrain subagent context to relevant test suites and port contracts.
92
+ <rule id="retry_budget_enforcement">
93
+ If a subagent encounters a tool error or recurring failure, enforce the 3-attempt retry budget. If exhausted, record the failure signature in `state.md > <failure_memory>` and escalate to human guidance.
17
94
  </rule>
18
95
 
19
- <rule id="no_polling">
20
- Rely on the system's reactive message wakeup mechanism when subagents complete their execution.
96
+ <rule id="teardown_and_cleanup">
97
+ Terminate idle or failed subagents cleanly. Ensure any temporary scratch files or experiment branches created by the subagent are deleted before completing the slice.
21
98
  </rule>
22
- </delegation_rules>
99
+ </coordination_and_recovery>
23
100
 
24
101
  </orchestration_protocol>
@@ -24,6 +24,7 @@ process/
24
24
  │ ├── review-template.md.seed # Artifact giai đoạn Review (Gate 3)
25
25
  │ ├── handoff-template.md.seed # Tóm tắt bàn giao cuối cùng (ngắn gọn)
26
26
  │ ├── cancellation-template.md.seed # Biên bản hủy task lưu giữ tri thức
27
+ │ ├── pause-template.md.seed # Biên bản tạm dừng task & đóng băng an toàn
27
28
  │ ├── results-template.tsv.seed # Bảng theo dõi số liệu benchmark & định lượng
28
29
  │ └── program-template.md.seed # Bản thiết kế chương trình lớn nhiều giai đoạn
29
30
  ├── context/ # Cơ sở tri thức bền vững & router định tuyến
@@ -45,6 +46,7 @@ process/
45
46
  │ │ ├── review.md # Biên bản nghiệm thu kiểm thử
46
47
  │ │ ├── handoff.md # Tóm tắt bàn giao
47
48
  │ │ ├── results.tsv # (Tùy chọn) Theo dõi benchmark & metric
49
+ │ │ ├── paused.md # (Nếu tạm dừng) Biên bản đóng băng & điều kiện mở lại
48
50
  │ │ └── cancelled.md # (Nếu hủy) Lưu vết nguyên nhân & hoàn nguyên
49
51
  │ ├── completed/ # Lưu trữ các task đã hoàn thành
50
52
  │ └── backlog/ # Ghi chú task tồn đọng: {note_slug}_NOTE_{dd-mm-yy}.md
@@ -107,8 +109,10 @@ handoff.md ← (Complete) Tóm tắt bàn giao ngắn gọn
107
109
  | Thực thi | mã nguồn + kiểm thử + `state.md` |
108
110
  | Nghiệm thu | `review.md` |
109
111
  | Bàn giao | `handoff.md` |
112
+ | Tạm dừng / Đóng băng | `paused.md` (đóng băng sạch sẽ & checklist mở lại khi bị chặn) |
110
113
  | Hủy task | `cancelled.md` (lưu giữ tri thức khi hủy bỏ) |
111
114
  | Benchmark / Định lượng | `results.tsv` (theo dõi hiệu năng và số liệu đánh giá) |
115
+ | Chương trình lớn | `program.md` (bản thiết kế chương trình lớn nhiều giai đoạn) |
112
116
  | Gate 1 | Phê duyệt trong `decision.md` |
113
117
  | Gate 2 | Phê duyệt trong `plan.md` |
114
118
  | Gate 3 | Phê duyệt trong `review.md` |
@@ -121,6 +121,10 @@ handoff.md ← Tóm tắt bàn giao cuối cùng (ngắn gọn)
121
121
  | Thực thi | mã nguồn + kiểm thử + `state.md` |
122
122
  | Nghiệm thu | `review.md` |
123
123
  | Bàn giao | `handoff.md` |
124
+ | Tạm dừng / Đóng băng | `paused.md` (đóng băng sạch sẽ & checklist mở lại) |
125
+ | Hủy task | `cancelled.md` (lưu giữ tri thức khi hủy bỏ) |
126
+ | Benchmark / Định lượng | `results.tsv` (theo dõi hiệu năng và số liệu đánh giá) |
127
+ | Chương trình lớn | `program.md` (bản thiết kế chương trình lớn nhiều giai đoạn) |
124
128
  | Gate 1 | Phê duyệt trong `decision.md` |
125
129
  | Gate 2 | Phê duyệt trong `plan.md` |
126
130
  | Gate 3 | Phê duyệt trong `review.md` |
@@ -6,9 +6,11 @@
6
6
  PHẦN 0 — KIỂM SOÁT TASK (bản lưu vết trạng thái master)
7
7
  ════════════════════════════════════════════ -->
8
8
  <task_control>
9
- <status>BACKLOG</status> <!-- BACKLOG | ACTIVE | BLOCKED | REVIEW | COMPLETED -->
9
+ <status>BACKLOG</status> <!-- BACKLOG | ACTIVE | BLOCKED | PAUSED | REVIEW | COMPLETED | CANCELLED -->
10
10
  <spec_level>S1</spec_level> <!-- S0=phác thảo | S1=đã định nghĩa | S2=đã kiểm chứng | S3=đã khóa -->
11
+ <priority>P2</priority> <!-- P0=khẩn cấp | P1=cao | P2=bình thường | P3=thấp -->
11
12
  <risk>MEDIUM</risk> <!-- LOW | MEDIUM | HIGH -->
13
+ <estimated_story_points>2</estimated_story_points> <!-- 1 SP ≈ 2-4 giờ làm việc kỹ thuật tập trung -->
12
14
  <working_mode>PAIR</working_mode> <!-- MANUAL | PAIR | DELEGATED | DIAGNOSE-ONLY -->
13
15
  <current_phase>RESEARCH</current_phase> <!-- RESEARCH | INNOVATE | PLAN | EXECUTE | REVIEW -->
14
16
  <owner>@engineer</owner>
@@ -63,6 +65,11 @@
63
65
  - `tests/[duong/dan/toi/test]` — [Khẳng định mục tiêu và test fixtures]
64
66
  </target_files>
65
67
 
68
+ <context_groups>
69
+ <!-- Tham chiếu tới các nhóm ngữ cảnh đã đăng ký trong process/context/all-context.md -->
70
+ - [planning | tests | protocols | domain-specific-group]
71
+ </context_groups>
72
+
66
73
  <source_of_truth>
67
74
  <requirement>[Liên kết tới tài liệu yêu cầu hoặc đặc tả]</requirement>
68
75
  <architecture>[Liên kết tới tài liệu kiến trúc hoặc ADR]</architecture>
@@ -154,6 +161,7 @@
154
161
  - [ ] Chạy verifier ngay sau mỗi slice.
155
162
  - [ ] Kiểm tra git diff sau mỗi slice.
156
163
  - [ ] Cập nhật `state.md` sau mỗi slice.
164
+ - [ ] (Tùy chọn) Ghi nhận số liệu định lượng vào `results.tsv` nếu task có đo lường benchmark/hiệu năng.
157
165
  </phase>
158
166
 
159
167
  <phase name="Review" order="5">
@@ -26,7 +26,7 @@
26
26
  <group id="seeds">
27
27
  <title>Cẩm Nang Mẫu Chuẩn & Seeds</title>
28
28
  <path>[`../_seeds/_GUIDE.md`](../_seeds/_GUIDE.md)</path>
29
- <scope>Blueprints mẫu cho task, research, decision, plan, state, review, handoff và program. Đầy đủ chuỗi artifact và lệnh khởi tạo.</scope>
29
+ <scope>Blueprints mẫu cho task, research, decision, plan, state, review, handoff, tạm dừng task, hủy task, benchmark định lượng và program. Đầy đủ chuỗi artifact và lệnh khởi tạo.</scope>
30
30
  </group>
31
31
 
32
32
  <group id="tests">
@@ -1,31 +1,87 @@
1
1
  # Quy Chuẩn Lập Kế Hoạch & Hiệu Chỉnh Năng Lực (Planning Context)
2
2
 
3
- <planning_context version="1.0">
3
+ <planning_context version="2.0">
4
4
 
5
5
  <overview>
6
- Hướng dẫn ước lượng Story Point, phân rã lát cắt dọc và hiệu chỉnh năng lực thực thi.
6
+ Hướng dẫn kỹ thuật về ước lượng Story Point, phân rã lát cắt dọc (Vertical Slicing), điểm hoàn nguyên rollback và hiệu chỉnh năng lực thực thi.
7
7
  </overview>
8
8
 
9
- ## 1. Tiêu Chuẩn Chất Lượng (Quality Standards)
9
+ ---
10
+
11
+ ## 1. Tiêu Chuẩn Chất Lượng (Tiêu Chuẩn INVEST)
12
+
10
13
  <quality_standards>
11
14
  <standard name="INVEST">
12
- - **Độc lập (Independent):** Có thể bàn giao mà không bị chặn bởi các story khác.
13
- - **Thương lượng được (Negotiable):** Chi tiết triển khai có thể tinh chỉnh linh hoạt.
14
- - **Có giá trị (Valuable):** Mang lại giá trị đo lường được cho người dùng hoặc doanh nghiệp.
15
- - **Ước lượng được (Estimable):** Phạm vi đủ rõ để ước lượng khối lượng công việc.
16
- - **Nhỏ gọn (Small):** Hoàn thành vừa vặn trong 1-3 ngày làm việc của kỹ sư.
17
- - **Kiểm chứng được (Testable):** Tiêu chí nghiệm thu (AC) cụ thể, đo lường được.
18
- </standard>
19
-
20
- <standard name="Vertical Slicing (Cắt Lát Dọc)">
21
- Tránh chia việc theo các tầng ngang độc lập (horizontal silos). Bàn giao từng lát cắt dọc hoàn chỉnh xuyên suốt qua API, logic nghiệp vụ và lưu trữ dữ liệu.
15
+ - **Độc lập (Independent):** Có thể bàn giao và kiểm chứng độc lập mà không bị phụ thuộc vòng vo vào story khác.
16
+ - **Thương lượng được (Negotiable):** Chi tiết kỹ thuật và phương án tiếp cận có thể linh hoạt đánh giá trong giai đoạn Innovate.
17
+ - **Có giá trị (Valuable):** Mang lại năng lực hệ thống ràng hoặc tiến bộ kỹ thuật đo lường được.
18
+ - **Ước lượng được (Estimable):** Phạm vi đủ rõ và danh sách file cần sửa đủ hẹp để tính toán khối lượng công việc.
19
+ - **Nhỏ gọn (Small):** Hoàn thành vừa vặn trong 13 ngày làm việc của kỹ sư (hoặc 1–3 Story Points).
20
+ - **Kiểm chứng được (Testable):** Kèm theo các Tiêu chí nghiệm thu (AC) ràng, kiểm chứng được (`- [ ]`).
22
21
  </standard>
23
22
  </quality_standards>
24
23
 
25
- ## 2. Hiệu Chỉnh Năng Lực (Capacity Calibration)
24
+ ---
25
+
26
+ ## 2. Kỷ Luật Cắt Lát Dọc (Vertical Slicing Discipline)
27
+
28
+ <vertical_slicing>
29
+ ### Lát Cắt Dọc (Vertical Slice) vs Tầng Ngang (Horizontal Silo)
30
+ - **Quy tắc Lát Cắt Dọc:** Mỗi lát cắt bắt buộc phải đi xuyên suốt toàn bộ các tầng kỹ thuật cần thiết (ví dụ: Thực thể Domain $\rightarrow$ Service/Logic $\rightarrow$ Port Adapter/API $\rightarrow$ Bài kiểm thử tự động) để mang lại một gia số hành vi hoàn chỉnh, kiểm chứng được.
31
+ - **Phản mẫu Tầng Ngang (Anti-Pattern):** Tuyệt đối không chia việc theo tầng ngang (ví dụ: Slice 1: tạo toàn bộ bảng DB, Slice 2: viết toàn bộ logic service, Slice 3: tạo toàn bộ API endpoint). Các tầng ngang không thể kiểm chứng độc lập, khiến toàn hệ thống ở trạng thái dở dang hoặc không thể chạy test cho đến tận slice cuối cùng.
32
+
33
+ ### Luồng Triển Khai Lát Cắt Dọc Mẫu
34
+ ```text
35
+ [Slice 1: Lõi tối thiểu xuyên suốt] ──► Domain Entity + Repo giả lập bộ nhớ + Endpoint tối thiểu + Test pass
36
+ [Slice 2: Lưu trữ & Ranh giới] ──► Adapter Database thật + File Migration + Kiểm thử tích hợp
37
+ [Slice 3: Trường hợp biên & Xử lý] ──► Chuẩn hóa đầu vào nghiêm ngặt + Bắt lỗi + Test trường hợp biên
38
+ ```
39
+ </vertical_slicing>
40
+
41
+ ---
42
+
43
+ ## 3. Quy Mô Lát Cắt, Khả Năng Kiểm Chứng & Checkpoint
44
+
45
+ <slice_sizing>
46
+ <rule id="slice_cardinality">
47
+ Một task tiêu chuẩn chỉ nên chứa từ **2 đến 5 lát cắt dọc**. Nếu một task đòi hỏi hơn 5 slice, bán kính ảnh hưởng của nó quá lớn và bắt buộc phải phân rã thành Chương trình nhiều giai đoạn (`program-template.md.seed`).
48
+ </rule>
49
+
50
+ <rule id="autonomous_verifier">
51
+ Mỗi slice trong `plan.md` bắt buộc phải có lệnh kiểm chứng `<verifier>` tường minh, tự động chạy được (ví dụ: `npm run test -- test/duong/dan/test.js` hoặc `pytest tests/unit/test_slice.py`). Một slice CHƯA ĐƯỢC COI LÀ XONG nếu lệnh verifier chưa exit với mã 0.
52
+ </rule>
53
+
54
+ <rule id="atomic_checkpoint">
55
+ Commit hoặc đánh dấu checkpoint git sau khi mỗi slice vượt qua verifier. Điều này đảm bảo lịch sử git tinh gọn và cho phép hoàn nguyên ngay lập tức nếu các slice sau gặp lỗi không thể cứu vãn.
56
+ </rule>
57
+ </slice_sizing>
58
+
59
+ ---
60
+
61
+ ## 4. Chiến Lược Hoàn Nguyên (Rollback Strategy) Cho Từng Slice
62
+
63
+ <rollback_strategy>
64
+ Mỗi slice trong `plan.md` BẮT BUỘC phải chỉ định một điểm hoàn nguyên cụ thể `<rollback_point>`:
65
+ - **Checkpoint Git:** `git checkout -- <allowed_files>` hoặc commit hash đảo ngược.
66
+ - **Checkpoint Stash:** `git stash pop` hoặc branch WIP riêng biệt.
67
+ - **Rollback Dữ liệu / Schema:** Script down-migration hoặc xóa container test tạm thời.
68
+
69
+ Nếu Agent cạn kiệt ngân sách thử lại 3 lần cho một slice, Agent phải thực thi rollback point trước khi dừng lại và xin chỉ thị từ con người.
70
+ </rollback_strategy>
71
+
72
+ ---
73
+
74
+ ## 5. Hiệu Chỉnh Năng Lực & Story Point (Capacity Calibration)
75
+
26
76
  <capacity_calibration>
27
- <unit>1 Story Point ≈ 2-4 giờ làm việc kỹ thuật tập trung</unit>
28
- <max_task_size>3-5 Story Points (các task lớn hơn bắt buộc phải phân rã nhỏ hơn)</max_task_size>
77
+ <unit>1 Story Point (SP) ≈ 24 giờ làm việc kỹ thuật tập trung</unit>
78
+ <scale>
79
+ - **1 SP:** Thay đổi thẳng thắn, phạm vi hẹp và rõ ràng (sửa 1-3 files, 1-2 slices).
80
+ - **2 SP:** Tính năng tiêu chuẩn hoặc refactor có phạm vi (sửa 3-5 files, 2-3 slices).
81
+ - **3 SP:** Tác vụ phức tạp vừa phải tác động đến hợp đồng domain và lưu trữ (sửa 4-7 files, 3-4 slices).
82
+ - **5 SP:** Quy mô tối đa cho một task active đơn lẻ trong `process/features/active/`.
83
+ - **> 5 SP:** Quá lớn. Bắt buộc phải tách thành nhiều task độc lập hoặc tổ chức dưới `program-template.md.seed`.
84
+ </scale>
29
85
  </capacity_calibration>
30
86
 
31
87
  </planning_context>
@@ -1,56 +1,113 @@
1
1
  # Tiêu Chuẩn Kiểm Thử & Chiến Lược Khung Thử Nghiệm (Testing Context)
2
2
 
3
- <testing_context version="1.0">
3
+ <testing_context version="2.0">
4
4
 
5
5
  <overview>
6
- Hướng dẫn hiệu chỉnh kim tự tháp kiểm thử, chính sách cách ly, quy ước mock và khung kiểm tra tự động xuyên suốt hệ thống.
6
+ Hướng dẫn hiệu chỉnh kim tự tháp kiểm thử, chính sách cách ly, quy ước mock, vệ sinh fixture dữ liệu và khung kiểm tra tự động xuyên suốt hệ thống.
7
7
  </overview>
8
8
 
9
- ## 1. Kim Tự Tháp Kiểm Thử & Phân Loại (Test Pyramid & Classification)
9
+ ---
10
+
11
+ ## 1. Kim Tự Tháp Kiểm Thử & Phân Loại Tầng (Test Pyramid & Classification)
12
+
10
13
  <test_pyramid>
11
14
  <layer name="Unit Tests (Kiểm thử đơn vị)" path="tests/unit/">
12
- - Phạm vi: Logic nghiệp vụ thuần túy, domain models, hàm tiện ích, phân nhánh trường hợp biên.
13
- - Tốc độ thực thi: Rất nhanh (< 50ms mỗi test), không I/O mạng / đĩa / ngoại vi.
14
- - Chính sách mock: Mock toàn bộ ranh giới I/O, client bên ngoàigiao diện cổng (port interfaces).
15
+ - **Phạm vi:** Logic nghiệp vụ thuần túy, mô hình domain, hàm tiện ích, rẽ nhánh trường hợp biên.
16
+ - **Tốc độ & Ràng buộc:** Cực nhanh (< 50ms mỗi test). Không gọi mạng, không đọc/ghi đĩa I/O, không kết nối database thật.
17
+ - **Chính sách Mock:** Mock toàn bộ giao diện cổng (port interfaces), client ngoại viranh giới I/O.
15
18
  </layer>
16
19
 
17
20
  <layer name="Integration Tests (Kiểm thử tích hợp)" path="tests/integration/">
18
- - Phạm vi: Lưu trữ cơ sở dữ liệu, adapter gọi API bên ngoài, cấu hình framework.
19
- - Mức độ cách ly: Test containers tạm thời (ví dụ: Testcontainers) hoặc database test cách ly riêng cho từng suite.
20
- - Dọn dẹp (Teardown): Phải đảm bảo không rỉ dữ liệu hoặc trạng thái giữa các lần chạy test.
21
+ - **Phạm vi:** Adapter lưu trữ repository, truy vấn cơ sở dữ liệu, file migration, wrapper gọi HTTP client, cấu hình framework.
22
+ - **Cách ly:** Test containers tạm thời (ví dụ: Testcontainers), in-memory database, hoặc schema cách ly riêng cho mỗi worker chạy test.
23
+ - **Dọn dẹp (Teardown):** Bắt buộc phải đảm bảo dọn dẹp dữ liệu 100%, không để rò rỉ trạng thái giữa các lần chạy.
21
24
  </layer>
22
25
 
23
26
  <layer name="End-to-End Tests (Kiểm thử đầu-cuối)" path="tests/e2e/">
24
- - Phạm vi: Các luồng trải nghiệm người dùng trọng yếu, toàn bộ vòng đời HTTP/gRPC API, regression smoke tests.
25
- - Kiểm chứng: Xác thực tính toàn vẹn hợp đồng đầu-cuối xuyên suốt các subsystem.
27
+ - **Phạm vi:** Các luồng trải nghiệm người dùng trọng yếu, toàn bộ vòng đời API công khai, luồng xác thực auth hoàn chỉnh, regression smoke tests.
28
+ - **Kiểm chứng:** Xác thực tính toàn vẹn hợp đồng đầu-cuối xuyên suốt toàn bộ hệ thống.
26
29
  </layer>
27
30
  </test_pyramid>
28
31
 
29
- ## 2. Chính Sách Khung Kiểm Thử (Test Harness Policies)
30
- <test_policies>
31
- <policy name="Determinism (Tính tiền định)">
32
- Tuyệt đối không có test chập chờn (flaky tests). Test không được phụ thuộc vào thời gian thực tế của đồng hồ hệ thống (sử dụng frozen clock hoặc fake timer) hay thứ tự thực thi ngẫu nhiên.
33
- </policy>
32
+ ---
33
+
34
+ ## 2. Ranh Giới Mocking & Các Phản Mẫu (Mocking Boundaries)
35
+
36
+ <mocking_boundaries>
37
+ <rule id="mock_at_architectural_boundary">
38
+ **Mock tại ranh giới kiến trúc, không bao giờ mock ở lõi.** Chỉ mock hạ tầng bên ngoài: HTTP API bên thứ ba, message broker, dịch vụ email, hoặc đồng hồ hệ thống.
39
+ </rule>
40
+
41
+ <rule id="do_not_mock_domain_or_sut">
42
+ Tuyệt đối không mock Hệ thống đang được kiểm thử (System Under Test - SUT) hoặc các thực thể domain / value objects thuần túy. Nếu một thực thể domain khó khởi tạo nếu thiếu mock, thiết kế đó đang bị thắt chặt và cần được tái cấu trúc (refactor).
43
+ </rule>
44
+
45
+ <rule id="verify_mock_interactions_sparingly">
46
+ Ưu tiên khẳng định dựa trên giá trị trả về và thay đổi trạng thái quan sát được hơn là kiểm tra số lần gọi hàm nội bộ (`toHaveBeenCalledTimes`), tránh việc gắn chặt test vào chi tiết triển khai ngầm.
47
+ </rule>
48
+ </mocking_boundaries>
49
+
50
+ ---
51
+
52
+ ## 3. Vệ Sinh Fixture Kiểm Thử & Data Factory (Fixture Hygiene)
53
+
54
+ <fixture_hygiene>
55
+ <rule id="data_factories_over_preseeded_db">
56
+ Sử dụng test factories (hoặc builder functions) rõ ràng với giá trị mặc định hợp lý thay vì phụ thuộc vào các file dump SQL seed tĩnh dùng chung dễ vỡ.
57
+ </rule>
58
+
59
+ <rule id="no_shared_mutable_fixtures">
60
+ Mỗi bài test bắt buộc phải tự khởi tạo dữ liệu của riêng mình. Tuyệt đối không chia sẻ trạng thái biến đổi hoặc biến test toàn cục giữa các bài test.
61
+ </rule>
62
+
63
+ <rule id="deterministic_teardown">
64
+ Luôn dọn sạch bản ghi database, tệp tạm thời hoặc đăng ký mock trong các hook `afterEach` / `teardown` để ngăn chặn lỗi dây chuyền sang các bài test khác.
65
+ </rule>
66
+ </fixture_hygiene>
67
+
68
+ ---
69
+
70
+ ## 4. Tính Tiền Định & Không Chấp Nhận Test Chập Chờn (Determinism)
71
+
72
+ <determinism>
73
+ <rule id="frozen_clock">
74
+ Tuyệt đối không sử dụng đồng hồ thực tế của hệ thống (`Date.now()`, `datetime.now()`) trong các khẳng định nhạy cảm về thời gian. Sử dụng frozen clock hoặc fake timer để đảm bảo kết quả tái lập tuyệt đối.
75
+ </rule>
76
+
77
+ <rule id="order_independence">
78
+ Các bài test bắt buộc phải chạy pass khi chạy riêng lẻ hoặc chạy theo thứ tự ngẫu nhiên (`--randomize`). Không bao giờ phụ thuộc vào việc bài test này phải chạy trước bài test kia.
79
+ </rule>
80
+
81
+ <rule id="no_arbitrary_sleeps">
82
+ Tuyệt đối không dùng lệnh chờ tùy tiện `sleep(1000)` trong test bất đồng bộ. Sử dụng helper kiểm tra định kỳ có timeout (`waitFor`, `eventually`) để hoàn thành ngay khi điều kiện kỳ vọng được thỏa mãn.
83
+ </rule>
84
+ </determinism>
85
+
86
+ ---
87
+
88
+ ## 5. Độ Chính Xác Của Khẳng Định (Assertion Precision)
89
+
90
+ <assertion_precision>
91
+ <rule id="exact_assertions">
92
+ Tránh các khẳng định mơ hồ (ví dụ: `expect(res).toBeTruthy()`). Luôn khẳng định cấu trúc giá trị chính xác, mã trạng thái HTTP và mã lỗi cụ thể.
93
+ </rule>
34
94
 
35
- <policy name="Clean State (Trạng thái sạch)">
36
- Mỗi bài test phải tự khởi tạo dữ liệu seed hoặc fixtures riêng của mình. Không bao giờ giả định sẵn các dòng dữ liệu trong database.
37
- </policy>
95
+ <rule id="actionable_failure_messages">
96
+ Khi viết assertion tùy biến, cung cấp thông báo lỗi ràng chỉ ra dữ liệu đầu vào điều kiện bất biến nào đã bị vi phạm.
97
+ </rule>
98
+ </assertion_precision>
38
99
 
39
- <policy name="Assertion Precision (Độ chính xác của khẳng định)">
40
- Tránh các khẳng định mơ hồ (ví dụ: `expect(res).toBeTruthy()`). Luôn khẳng định cấu trúc chính xác kỳ vọng, mã trạng thái HTTP (status code) và loại lỗi.
41
- </policy>
100
+ ---
42
101
 
43
- <policy name="Fast Feedback (Phản hồi nhanh)">
44
- Toàn bộ bộ unit test phải hoàn thành trong vòng dưới 30 giây. Các bộ integration test chậm phải được phân vùng rõ ràng.
45
- </policy>
46
- </test_policies>
102
+ ## 6. Ánh Xạ Lệnh Kiểm Tra (Verification Commands Mapping)
47
103
 
48
- ## 3. Ánh Xạ Lệnh Kiểm Tra (Verification Commands Mapping)
49
104
  <verification_commands>
50
105
  <!-- Giữ đồng bộ với <validation_commands> trong AGENTS.md -->
51
- <command type="unit">Chạy bộ kiểm thử unit test nhanh</command>
52
- <command type="integration">Chạy kiểm thử tích hợp integration/e2e</command>
53
- <command type="coverage">Chạy kiểm tra độ bao phủ test coverage</command>
106
+ <command type="unit">Chạy bộ kiểm thử unit test nhanh (< 30 giây toàn bộ)</command>
107
+ <command type="integration">Chạy kiểm thử tích hợp integration/e2e cách ly</command>
108
+ <command type="typecheck">Chạy kiểm tra kiểu tĩnh (0 lỗi, strict mode)</command>
109
+ <command type="lint">Chạy linter và kiểm tra format (0 cảnh báo)</command>
110
+ <command type="coverage">Chạy báo cáo độ bao phủ test coverage và ngưỡng tối thiểu</command>
54
111
  </verification_commands>
55
112
 
56
113
  </testing_context>
@@ -1,28 +1,117 @@
1
- # Tiêu Chuẩn Triển Khai & Khung Kiểm Thử (Implementation Standards)
1
+ # Tiêu Chuẩn Triển Khai & Khung Kỹ Thuật (Implementation Standards)
2
2
 
3
- <implementation_standards version="1.0">
3
+ <implementation_standards version="2.0">
4
4
 
5
5
  <description>
6
- Quy chuẩn chất lượng kỹ thuật, an toàn kiểu dữ liệu (type safety)kiểm thử cho codebase này.
6
+ Quy chuẩn chất lượng kỹ thuật, an toàn kiểu dữ liệu, kiến trúc sạch thực thi phòng thủ cho codebase. Dùng chung cho mọi dự án.
7
7
  </description>
8
8
 
9
- ## 1. An Toàn Kiểu Dữ Liệu & Vệ Sinh Code (Type Safety & Code Hygiene)
10
- <conventions>
9
+ ---
10
+
11
+ ## 1. An Toàn Kiểu Dữ Liệu & Phân Tích Tĩnh (Type Safety & Static Analysis)
12
+
13
+ <type_safety>
11
14
  <rule id="strict_typing">
12
- Toàn bộ các hàm, phương thức và thuộc tính class bắt buộc phải có chú thích kiểu dữ liệu (type annotations) rõ ràng. Sử dụng union types hiện đại (`int | None` hoặc `string | null`) tránh dùng `Any`/`any` không định kiểu trừ khi giao tiếp với payload bên ngoài thô.
15
+ Toàn bộ các hàm, phương thức và thuộc tính class bắt buộc phải có chú thích kiểu dữ liệu (type annotations) rõ ràng. Sử dụng union types hiện đại (`int | None` hoặc `string | null`). Tránh dùng `Any`/`any` không định kiểu trừ khi giải dữ liệu đầu vào thô chưa xác thực tại cổng biên.
16
+ </rule>
17
+
18
+ <rule id="explicit_return_types">
19
+ Luôn khai báo kiểu trả về tường minh cho các hàm và phương thức public để áp đặt hợp đồng lúc biên dịch (compile-time) và ngăn ngừa hiện tượng suy luận kiểu bị mở rộng vô ý.
20
+ </rule>
21
+
22
+ <rule id="no_suppression">
23
+ Tuyệt đối không tắt lỗi linter, formatter hoặc type-checker bằng comment bỏ qua nội dòng (`# type: ignore`, `eslint-disable`, `@SuppressWarnings`) để vượt qua cổng kiểm tra giả tạo. Phải xử lý triệt để nguyên nhân gốc rễ.
13
24
  </rule>
25
+ </type_safety>
26
+
27
+ ---
28
+
29
+ ## 2. Kiến Trúc Sạch & Độ Thuần Khiết Domain (Clean Architecture & Domain Purity)
30
+
31
+ <clean_architecture>
32
+ <rule id="dependency_rule">
33
+ Chiều phụ thuộc (dependency) chỉ được phép hướng vào lõi Domain bên trong. Các tầng domain bên trong tuyệt đối không import hay phụ thuộc vào framework hạ tầng, database hoặc UI bên ngoài.
34
+ </rule>
35
+
14
36
  <rule id="domain_purity">
15
- nguồn trong các tầng domain phải thuần khiết với 0 phụ thuộc vào hạ tầng bên ngoài (zero external infrastructure dependencies).
37
+ Logic nghiệp vụ cốt lõi, thực thể domain và value objects phải thuần khiết 100% với 0 tác dụng phụ (zero side-effects), không thực hiện I/O tệp hệ thống và không gọi mạng.
16
38
  </rule>
17
- <rule id="immutability">
18
- Ưu tiên các mô hình bất biến (immutable models) cho các sự kiện domain (domain events), value objects và DTOs.
39
+
40
+ <rule id="ports_and_adapters">
41
+ Định nghĩa giao diện trừu tượng (ports) trong tầng application/domain cho lưu trữ dữ liệu, API bên ngoài và nhắn tin. Triển khai các driver cụ thể (adapters) độc quyền trong các package hạ tầng (infrastructure).
42
+ </rule>
43
+
44
+ <rule id="no_premature_abstraction">
45
+ Không tự ý đưa vào các lớp trừu tượng mang tính suy diễn, dynamic factory hoặc lớp trung gian bắc cầu không cần thiết cho các tiện ích đơn chức năng đơn giản.
46
+ </rule>
47
+ </clean_architecture>
48
+
49
+ ---
50
+
51
+ ## 3. Xử Lý Lỗi & Nguyên Tắc Thất Bại Sớm (Error Handling & Fail-Fast)
52
+
53
+ <error_handling>
54
+ <rule id="fail_fast_at_boundaries">
55
+ Kiểm tra hợp lệ toàn bộ tham số, payload và biến môi trường ngay tại điểm vào của hệ thống. Từ chối dữ liệu sai lệch ngay lập tức trước khi gọi vào logic nghiệp vụ.
56
+ </rule>
57
+
58
+ <rule id="no_silent_swallowing">
59
+ Tuyệt đối không sử dụng `catch {}` rỗng, `except: pass` hoặc bắt ngoại lệ mà không ghi log. Mọi ngoại lệ bị bắt phải được xử lý, chuyển đổi thành lỗi domain có định kiểu hoặc lan truyền (propagate) sạch sẽ.
60
+ </rule>
61
+
62
+ <rule id="typed_domain_errors">
63
+ Tạo các class lỗi domain rõ ràng kèm mã lỗi đọc được bằng máy (ví dụ: `ResourceNotFoundError`, `ValidationError`, `ConcurrencyConflictError`) thay vì ném ra các exception chung chung.
19
64
  </rule>
20
- </conventions>
65
+ </error_handling>
21
66
 
22
- ## 2. Kiến Trúc Kiểm Thử (Test Architecture)
23
- <test_structure>
24
- - `tests/unit/`: Các bài test nhanh, cách ly cho domain models, logic cốt lõi và mock adapters.
25
- - `tests/integration/`: Các bài test luồng đầu-cuối với dịch vụ kiểm thử hoặc database nội bộ.
26
- </test_structure>
67
+ ---
68
+
69
+ ## 4. Bất Biến & An Toàn Khi Chạy Đồng Thời (Immutability & Concurrency Safety)
70
+
71
+ <concurrency_and_immutability>
72
+ <rule id="immutable_data_structures">
73
+ Ưu tiên các cấu trúc dữ liệu bất biến (`dataclass(frozen=True)`, `readonly` interface, hoặc `Readonly<T>`) cho các sự kiện domain, cấu hình và DTO.
74
+ </rule>
75
+
76
+ <rule id="no_shared_mutable_state">
77
+ Tránh sử dụng biến toàn cục có thể thay đổi (mutable globals), cache trạng thái cấp module hoặc singleton có trường biến đổi. Truyền dependency rõ ràng thông qua hàm khởi tạo (Dependency Injection).
78
+ </rule>
79
+
80
+ <rule id="safe_async_lifecycle">
81
+ Luôn xử lý dọn dẹp và từ chối lỗi bất đồng bộ (async error rejection/cancellation) cẩn thận. Thiết lập timeout rõ ràng cho mọi lệnh gọi mạng hoặc truy vấn cơ sở dữ liệu.
82
+ </rule>
83
+ </concurrency_and_immutability>
84
+
85
+ ---
86
+
87
+ ## 5. Giám Sát & Vệ Sinh Ghi Log (Observability & Logging Hygiene)
88
+
89
+ <observability_hygiene>
90
+ <rule id="structured_logging">
91
+ Sử dụng ghi log có cấu trúc (structured logging) kèm ngữ cảnh dạng key-value (`task_id`, `user_id`, `component`) thay vì nối chuỗi văn bản thuần túy.
92
+ </rule>
93
+
94
+ <rule id="no_credential_leakage">
95
+ Không bao giờ ghi log dữ liệu nhạy cảm: mật khẩu, tokens, API keys, cookies hoặc thông tin nhận dạng cá nhân (PII).
96
+ </rule>
97
+
98
+ <rule id="clean_production_diff">
99
+ Xóa sạch toàn bộ debug log tạm thời (`console.log`, `print()`, `debugger`, `dump()`) trước khi đề nghị ký duyệt Gate G3.
100
+ </rule>
101
+ </observability_hygiene>
102
+
103
+ ---
104
+
105
+ ## 6. Quản Lý Tài Nguyên Phòng Thủ & Dọn Dẹp (Resource Management & Teardown)
106
+
107
+ <resource_management>
108
+ <rule id="deterministic_teardown">
109
+ Luôn quản lý con trỏ tệp (file handles), kết nối mạng socket và transaction database bằng các cấu trúc giải phóng tiền định (`try...finally`, Python `with`, TypeScript `using`, hoặc RAII nguyên bản của ngôn ngữ).
110
+ </rule>
111
+
112
+ <rule id="isolated_test_state">
113
+ Mỗi fixture kiểm thử phải tự tạo và tự dọn sạch trạng thái tạm thời của mình. Không để sót lại dữ liệu rác sau khi chạy xong test.
114
+ </rule>
115
+ </resource_management>
27
116
 
28
117
  </implementation_standards>