@anhth2/spec-driven-dev-plugin 0.10.0 → 0.11.0

package/core/skills/qc/qa-reviewer/script/e2e.md

@@ -0,0 +1,87 @@
+---
+version: 1.0
+updated: 2026-06-11
+ported_from: ai-automation-qc-base
+---
+
+# Review Test Script — E2E Journey
+
+Review Python pytest script cho test end-to-end và đánh giá chất lượng code.
+
+## Khi nào trigger
+- "review script E2E cho [Feature]" / "check code E2E quality"
+- Sau khi qa-runner sinh script E2E, trước khi merge
+
+## Khi KHÔNG trigger
+- Review script functional 1 màn → `script/functional`
+- Review script integration → `script/integration`
+- Review session note → `script/exploratory`
+
+---
+
+## Phase 1 — Clarify
+
+1. Đọc file Python test E2E được chỉ định (thường `tests/<project>/e2e/test_*.py`)
+2. Đọc các Page Object liên quan (nhiều PO cho nhiều màn)
+3. Đọc TC E2E Markdown gốc để so sánh coverage journey
+
+---
+
+## Phase 2 — Review
+
+Đánh giá theo 6 tiêu chí:
+
+A. JOURNEY COVERAGE:
+- Mỗi journey trong TC E2E có test function tương ứng không?
+- Test function có traverse đủ các màn/module của journey không?
+- Main flow / alternate flow / exception flow đủ không?
+
+B. CROSS-MODULE DATA INTEGRITY:
+- Data nhập ở Page A có được verify ở Page B/DB/hệ thống ngoài không?
+- Assertion sau mỗi chặng (không chỉ assert ở bước cuối)?
+- Không bỏ qua bước trung gian để "shortcut" đến màn cuối?
+
+C. FIXTURE & PRECONDITION:
+- Precondition phức tạp (nhiều entity) có fixture riêng, không inline trong test?
+- Fixture tạo data → có teardown/cleanup tương ứng (yield + cleanup)?
+- Không hardcode ID của entity đã tạo ở bước trước → dùng biến trả về fixture?
+- Fixture scope đúng (`function` cho E2E, không dùng `session`)?
+
+D. WAIT & TIMING:
+- Timeout đủ dài cho navigation giữa màn (≥15s cho `networkidle`)?
+- Không `time.sleep()`; dùng `wait_for_url` / `wait_for_load_state` / `expect(...).to_be_visible`?
+- Không timeout ngắn (<5s) cho API call cross-module?
+
+E. ASSERTION DEPTH:
+- Không chỉ assert URL cuối; phải assert nội dung tại mỗi chặng?
+- Verify đúng data flow: giá trị nhập màn A xuất hiện đúng ở màn B?
+- Negative journey (thất bại giữa chừng): assert đúng màn dừng lại + thông báo lỗi?
+
+F. CONVENTION:
+- Marker `@pytest.mark.e2e` + `@pytest.mark.smoke` (nếu critical journey)?
+- Marker phân loại = tên journey; docstring = TC ID + mô tả journey?
+- `with step(...)` (`from utils.steps`) rõ từng chặng (màn nào → hành động nào)?
+- Docstring ghi TC ID + mô tả journey ngắn?
+
+---
+
+## Checklist chi tiết
+
+### Page Objects
+- Mỗi màn có PO riêng; test không gọi `page.click()` / `page.fill()` trực tiếp.
+- Selector constants khai báo đầu class; không rải rác trong test.
+- Action method `return self` để chain; assertion method gọi `take_screenshot()` cuối.
+
+### Test file
+- Fixture `logged_in_page` hoặc fixture composite (vd `logged_in_as_teacher`) làm base.
+- Không dùng biến global chia sẻ state giữa test function.
+- Compile & collect trước submit: `python3 -m py_compile` + `pytest --collect-only -q`.
+- Số journey collect = số TC E2E trong `.md`.
+
+---
+
+## Output
+
+Mỗi issue: `file:line` | severity | mô tả | suggestion fix.
+Severity: 🔴 fix ngay (missing cleanup, dữ liệu không verify cross-module, shortcut journey) · 🟠 quan trọng (timeout thiếu, assertion chỉ ở bước cuối) · 🟡 nhỏ (style, docstring/title sai).
+Score: A/B/C/D · Top 5 issue cần fix trước merge.

package/core/skills/qc/qa-reviewer/script/exploratory.md

@@ -0,0 +1,45 @@
+---
+version: 1.0
+updated: 2026-06-11
+ported_from: ai-automation-qc-base
+---
+
+# Review Session Note — Exploratory
+ 
+Review session note sau khi test, coaching QC cải thiện kỹ năng.
+ 
+## Khi nào trigger
+- "review session note" / sau khi QC hoàn thành exploratory session
+- Khi lead/senior muốn đánh giá chất lượng session của junior
+ 
+## Khi KHÔNG trigger
+- Review Python script → dùng qa-reviewer/script/functional
+- Review charter → dùng qa-reviewer/test-case/exploratory
+ 
+---
+ 
+## Phase 1 — Read
+ 
+Đọc session note .md được chỉ định.
+ 
+---
+ 
+## Phase 2 — Review
+ 
+7 tiêu chí:
+1. CHARTER COMPLETION: Đi đúng charter? Coverage %?
+2. NOTE QUALITY: Ratio #TEST / #BUG ≥ 3:1? Mỗi #TEST có mô tả action + observation?
+3. BUG QUALITY: Steps rõ? Expected/Actual cụ thể? Severity hợp lý?
+4. OBSERVATION DEPTH: Có dùng FEW HICCUPPS? Hay chỉ test surface?
+5. TIME ALLOCATION: Setup <15%, Test >70%, Investigation <15%?
+6. FOLLOW-UP: #QUESTION gửi dev? #IDEA move vào ideas/?
+7. DEBRIEF: Có Summary cuối? Coverage, risks, next recommendation?
+ 
+---
+ 
+## Output
+ 
+Score: A (excellent) / B (good) / C (needs improvement) / D (redo)
+Mỗi tiêu chí: ✅/⚠️/❌ + feedback cụ thể
+Top 3 improvement suggestions (coaching tone)
+Đề xuất charter tiếp theo nếu chưa đủ coverage

package/core/skills/qc/qa-reviewer/script/functional.md

@@ -0,0 +1,101 @@
+---
+version: 1.0
+updated: 2026-06-11
+ported_from: ai-automation-qc-base
+---
+
+# Review Test Script — Functional
+ 
+Review Python pytest script và đánh giá chất lượng code.
+ 
+## Khi nào trigger
+- "review script cho [Feature]" / "check code quality"
+- Sau khi qa-runner xong, trước khi merge vào main
+ 
+## Khi KHÔNG trigger
+- Review TC nghiệp vụ → dùng qa-reviewer
+- Review exploratory session note → dùng qa-reviewer/script/exploratory
+ 
+---
+ 
+## Phase 1 — Clarify
+ 
+1. Đọc file Python test được chỉ định
+2. Đọc Page Object liên quan
+3. Đọc TC markdown gốc (để so sánh coverage)
+ 
+---
+ 
+## Phase 2 — Review
+ 
+Đánh giá theo 6 tiêu chí:
+ 
+A. LOCATOR:
+- XPath dài/phức tạp → đề xuất data-testid / role
+- Locator phụ thuộc text tiếng Việt → fragile khi đổi copy
+- Locator nằm trong Page Object, KHÔNG hardcode trong test
+- Dùng .first()/[n] → không reliable
+ 
+B. WAIT & TIMING:
+- time.sleep() → phải đổi sang Playwright auto-wait / expect
+- Timeout quá ngắn (<3s) hoặc quá dài (>30s)
+ 
+C. TEST INDEPENDENCE:
+- Hardcode user ID/data ID → fragile
+- Phụ thuộc thứ tự chạy (test_b cần test_a)
+- Thiếu cleanup/teardown khi tạo data
+- Global state
+ 
+D. ASSERTION:
+- assert True / not error → vô nghĩa
+- Chỉ assert URL, không assert content → thiếu
+- Thiếu negative assertion
+ 
+E. CONVENTION:
+- Marker đúng (@pytest.mark.functional + platform)?
+- Tên: test_<feature>_<scenario>_<expected>?
+- Docstring có TC_ID?
+- Fixture từ conftest, không tự tạo browser?
+ 
+F. PAGE OBJECT:
+- Test gọi page.click() trực tiếp → phải qua PO
+- PO method quá dài (>15 dòng) → tách
+ 
+---
+
+## Checklist chi tiết review code
+
+### Phần 1 — Page Object (`pages/<feature>_page.py`)
+- Kế thừa `BasePage`; selector constants `UPPER_SNAKE_CASE` tập trung đầu class (không rải rác).
+- Tách 3 lớp: Locators (`_private()` chỉ trả `Locator`) → Actions (`public()` + `return self`, KHÔNG decorator) → Assertions (`assert_*()` + `expect()` + `take_screenshot()` cuối).
+- Wait: KHÔNG `time.sleep()`; `wait_for_timeout()` chỉ khi cần (≤1000ms); sau nav/submit dùng `wait_for_load_state("networkidle")`; không `wait_for_timeout` ngay sau `networkidle`.
+- KHÔNG hardcode URL/credential/timeout → `Env.*`, `CONFIG`.
+- Column index dùng named constant (`COL_TTHAI = 7`), không magic number `cells[7]`.
+- Assertion: dùng `expect()` cho element; `assert` Python cho logic (count/regex); screenshot tên `TC<NNN>_mo_ta`; KHÔNG silent fail → `raise AssertionError`, không `log.warning`.
+- Lỗi assertion hay gặp: "ordered" phải verify thứ tự thật (`last_pos`); đúng cột; Counter ≠ Pagination total (không assert `==`); P0 strict / P1 `_or_empty`.
+- Public API: có `get_row_count()`, `get_all_rows()` thay vì test gọi `_private()`; helper trả data là method thường (không cần bọc step).
+
+### Phần 2 — Test file (`tests/test_<feature>.py`)
+- Đủ class `TestFeatureUI` / `TestFeatureFunctional` / `TestFeatureNegative`; fixture scope `function`, base `logged_in_page`.
+- KHÔNG Allure: phân loại/độ ưu tiên qua pytest markers (`smoke`/`regression`/domain); docstring/`__doc__` ghi `TC_FEATURE_NNN – …`; Priority khớp marker (P0→smoke, P1/P2→regression).
+- Marker: ≥1 category (`smoke`/`regression`) + 1 domain (`ui`/`filter`/`search`/`pagination`/`action`/`negative`); đã đăng ký `pytest.ini`.
+- Step: bọc bước bằng `with step("…")` (`from utils.steps import step`) rõ Action/Verify; KHÔNG step rỗng `: pass`.
+- Isolation: độc lập thứ tự; không global state; không gọi `_private()` từ test.
+- Logic nghiệp vụ nghi ngờ → `@pytest.mark.xfail(strict=False, reason=...)` giải thích rõ.
+
+### Phần 4 — Tổng quát
+- Compile & collect bắt buộc trước submit:
+  `python3 -m py_compile pages/<f>_page.py tests/test_<f>.py` · `pytest tests/test_<f>.py --collect-only -q`
+- Số test collect = số TC trong `.md`; không warning marker chưa đăng ký.
+- Naming: class `PascalCase`+`Page`; method `test_TC<NNN>_<snake>`; constant `UPPER_SNAKE`; locator `_snake()`; action `verb_noun()`; assert `assert_condition()`.
+
+### Lỗi phổ biến (TC_TRUCLOP)
+empty step `:pass` · test gọi `_private()` · Counter≠Pagination → xfail · "ordered" không verify thứ tự · magic `cells[7]` · silent fail → raise · Reset thiếu domain marker · `wait_for_timeout` thừa sau networkidle · orphan selector constant.
+
+---
+ 
+## Output
+ 
+Mỗi issue: file:line | severity | mô tả | suggestion fix.
+Severity: 🔴 fix ngay (sai logic, POM breach, silent fail) · 🟠 quan trọng (fragile selector, wrong column, missing assert) · 🟡 nhỏ (style, thừa wait, orphan constant).
+Score: A/B/C/D · Top 5 issue cần fix trước.

package/core/skills/qc/qa-reviewer/script/integration.md

@@ -0,0 +1,91 @@
+---
+version: 1.0
+updated: 2026-06-11
+ported_from: ai-automation-qc-base
+---
+
+# Review Test Script — Integration
+
+Review Python pytest script cho test tích hợp (GUI↔Backend, API, DB) và đánh giá chất lượng code.
+
+## Khi nào trigger
+- "review script integration cho [Feature]" / "check code tích hợp"
+- Sau khi qa-runner sinh script integration, trước khi merge
+
+## Khi KHÔNG trigger
+- Review script functional 1 màn/endpoint → `script/functional`
+- Review script E2E xuyên nhiều module → `script/e2e`
+- Review session note → `script/exploratory`
+
+---
+
+## Phase 1 — Clarify
+
+1. Đọc file Python test integration được chỉ định
+2. Đọc Page Object và API/DB utility liên quan
+3. Đọc TC integration Markdown gốc để so sánh coverage
+
+---
+
+## Phase 2 — Review
+
+Đánh giá theo 6 tiêu chí:
+
+A. NO MOCK ON INTEGRATION:
+- Không mock API/DB trong integration test (mock → đây là unit test, không phải integration)?
+- Dùng real network call / real DB query / `page.expect_response()` để capture API thật?
+- Network interception chỉ dùng để **observe** (verify request), không để **stub** response?
+
+B. BACKEND STATE VERIFICATION:
+- Sau action UI, có verify trạng thái backend không (DB query / API GET để re-fetch)?
+- **GUI↔Backend:** verify cả request gửi đúng (method/URL/payload) lẫn render UI đúng?
+- **DB:** dùng trực tiếp DB fixture/util để query `bảng.cột = giá trị`; không chỉ verify qua UI?
+- **API:** verify response schema + status code + downstream effect (DB/event)?
+
+C. ERROR STATE COVERAGE:
+- Có test 4xx/5xx response → UI hiển thị message đúng?
+- Có test empty state / loading state / timeout state?
+- Concurrency test: mô tả rõ số request đồng thời; assert không race condition (vd unique constraint giữ)?
+
+D. DATA SETUP & CLEANUP:
+- Data test được tạo qua fixture (không hardcode ID)?
+- Fixture `yield` + teardown xóa/rollback data sau mỗi test?
+- Không dùng data production hoặc shared data giữa các test?
+
+E. WAIT & TIMING:
+- Không `time.sleep()`; chờ API response bằng `page.expect_response()` hoặc `wait_for_response()`?
+- Sau action có side-effect backend, chờ đủ trước khi assert state (vd `wait_for_load_state("networkidle")`)?
+- Timeout đủ cho network round-trip (≥10s)?
+
+F. CONVENTION:
+- Marker `@pytest.mark.integration` + sub-domain (`gui`, `api`, `db`)?
+- Marker phân loại = loại tích hợp; docstring = TC ID + điểm tích hợp?
+- `with step(...)` (`from utils.steps`) rõ hành động → API call → verify response/DB?
+- Helper DB/API truy cập trong `utils/`, không rải trong test file?
+
+---
+
+## Checklist chi tiết
+
+### GUI↔Backend
+- Dùng `page.expect_response("**/api/endpoint")` để capture và assert request/response.
+- Assert: status code + response payload + UI change sau response.
+
+### API integration
+- Dùng `requests` hoặc Playwright API context; không dùng UI để trigger API call.
+- Assert schema với JSON schema validator hoặc `assert key in response.json()`.
+
+### DB integration
+- DB fixture trả connection/cursor; cleanup `DELETE WHERE id = created_id`.
+- `assert cursor.fetchone()["column"] == expected_value`; không hardcode row position.
+
+### Compile & collect
+`python3 -m py_compile` + `pytest --collect-only -q`; số test collect = số TC integration `.md`.
+
+---
+
+## Output
+
+Mỗi issue: `file:line` | severity | mô tả | suggestion fix.
+Severity: 🔴 fix ngay (mock thay real call, missing DB verify, no cleanup) · 🟠 quan trọng (thiếu error state, hardcode ID, timeout ngắn) · 🟡 nhỏ (style, marker thiếu sub-domain).
+Score: A/B/C/D · Top 5 issue cần fix trước merge.

package/core/skills/qc/qa-reviewer/script/non-functional.md

@@ -0,0 +1,126 @@
+---
+version: 1.0
+updated: 2026-06-11
+ported_from: ai-automation-qc-base
+---
+
+# Review Test Script — Non-Functional
+
+Review Python pytest script cho test phi chức năng (performance, security, accessibility, compatibility) và đánh giá chất lượng code.
+
+## Khi nào trigger
+- "review script non-functional cho [Feature]" / "check code performance/security/accessibility"
+- Sau khi qa-runner sinh script non-functional, trước khi merge
+
+## Khi KHÔNG trigger
+- Review script functional → `script/functional`
+- Review script integration → `script/integration`
+- Review session note → `script/exploratory`
+
+---
+
+## Phase 1 — Clarify
+
+1. Đọc file Python test non-functional được chỉ định
+2. Xác định loại: performance / security / accessibility / compatibility
+3. Đọc TC non-functional Markdown gốc để đối chiếu ngưỡng + công cụ đo
+
+---
+
+## Phase 2 — Review
+
+Đánh giá theo 6 tiêu chí:
+
+A. THRESHOLD ASSERTION — tiêu chí quan trọng nhất:
+- Assertion có dùng **giá trị ngưỡng cụ thể** không? (`assert elapsed < 2.0`, `assert violations == []`)
+- Không dùng assertion mơ hồ: `assert response` / `assert "ok" in text` / `assert True`?
+- Ngưỡng khớp với TC Markdown gốc (không tự đặt giá trị khác)?
+
+B. THEO LOẠI:
+- **Performance:**
+  - Đo thời gian bằng `page.wait_for_load_state` + `performance.timing` hoặc `time.perf_counter()` (không `time.sleep()`)?
+  - Có parametrize tải mục tiêu (concurrent users / data volume)?
+  - `pytest-benchmark` hoặc custom fixture đo rõ ràng?
+  - Margin hợp lý (không `assert elapsed < 0.001` quá strict)?
+- **Security:**
+  - Payload injection được lưu trong fixture/constant, không inline magic string?
+  - Test không thực sự tấn công server production; dùng môi trường test?
+  - Assert bị chặn đúng: status 4xx, message lỗi, KHÔNG tạo được record?
+  - PII test: assert response KHÔNG chứa SĐT/email raw?
+- **Accessibility:**
+  - Dùng `axe-playwright` (`AxeBuilder`) hoặc `pytest-axe`?
+  - Assert `violations == []` hoặc filter đúng WCAG level (`wcag2a`, `wcag2aa`)?
+  - Không assert bằng element count / class name (không liên quan accessibility)?
+- **Compatibility:**
+  - Parametrize `@pytest.mark.parametrize` trên browser/device/viewport?
+  - Mỗi parameter = 1 target trong TC Markdown?
+  - Dùng `playwright_browser_type` fixture, không hardcode `chromium`?
+
+C. ENVIRONMENT GUARD:
+- Test cần môi trường đặc biệt (load server, scanner) có `@pytest.mark.skipif` nếu env không đủ?
+- Credentials/endpoint load test không hardcode → `Env.*` / `CONFIG`?
+- Test security không gọi endpoint production?
+
+D. DATA SETUP & TEARDOWN:
+- Data lớn (performance) có fixture tạo trước, teardown sau?
+- Không để lại data/artifact sau test (security test không tạo record rác)?
+
+E. WAIT & TIMING:
+- Không `time.sleep()` cho wait UI; dùng Playwright auto-wait?
+- Đo elapsed time chính xác: bắt đầu/kết thúc đo rõ ràng, không bao gồm fixture setup?
+
+F. CONVENTION:
+- Marker `@pytest.mark.non_functional` + sub-domain (`performance`/`security`/`accessibility`/`compatibility`)?
+- **KHÔNG Allure** (đã gỡ): không `@allure.*`; bọc bước bằng `with step("…")` (`from utils.steps import step`)?
+- `with step(...)` rõ: setup tải → trigger → measure → assert ngưỡng?
+- Docstring ghi TC ID + ngưỡng mục tiêu + công cụ đo?
+- Report = Playwright Trace (`test-results/<nodeid>/trace.zip`, `playwright show-trace`) + pytest-html (`--html=… --self-contained-html`); không tham chiếu report tự viết/Allure?
+
+---
+
+## Checklist chi tiết
+
+### Performance
+```python
+# ✅ Đúng
+start = time.perf_counter()
+page.goto(Env.BASE_URL + "/list")
+page.wait_for_load_state("networkidle")
+elapsed = time.perf_counter() - start
+assert elapsed < 2.0, f"Load time {elapsed:.2f}s > 2.0s threshold"
+```
+
+### Security
+```python
+# ✅ Đúng — payload trong constant, assert bị block
+INJECTION_PAYLOADS = ["<script>alert(1)</script>", "' OR 1=1--"]
+# assert response.status == 400 hoặc record không tồn tại
+```
+
+### Accessibility
+```python
+# ✅ Đúng — axe-playwright
+from axe_playwright_python.sync_playwright import Axe
+results = Axe().run(page)
+assert results.violations_count == 0, results.generate_report()
+```
+
+### Compatibility
+```python
+# ✅ Đúng — parametrize browser
+@pytest.mark.parametrize("browser_name", ["chromium", "firefox", "webkit"])
+def test_compatibility(browser_name, playwright):
+    browser = getattr(playwright, browser_name).launch()
+    ...
+```
+
+### Compile & collect
+`python3 -m py_compile` + `pytest --collect-only -q`; số test collect = số TC non-functional `.md`.
+
+---
+
+## Output
+
+Mỗi issue: `file:line` | severity | mô tả | suggestion fix.
+Severity: 🔴 fix ngay (assertion không có ngưỡng, gọi production endpoint, hardcode credential) · 🟠 quan trọng (ngưỡng không khớp TC, thiếu parametrize, không teardown data) · 🟡 nhỏ (style, marker thiếu sub-domain, docstring thiếu ngưỡng).
+Score: A/B/C/D · Top 5 issue cần fix trước merge.

package/core/skills/qc/qa-reviewer/test-case/e2e.md

@@ -0,0 +1,73 @@
+---
+version: 1.0
+updated: 2026-06-11
+ported_from: ai-automation-qc-base
+---
+
+# Review Test Case — E2E Journey
+
+Review bộ TC end-to-end và đánh giá chất lượng.
+
+## Khi nào trigger
+- "review TC E2E cho [Feature]" / "check coverage E2E"
+- Sau khi qa-designer/e2e/journey xong, trước khi qa-runner
+
+## Khi KHÔNG trigger
+- Review TC functional 1 màn → `test-case/functional`
+- Review charter exploratory → `test-case/exploratory`
+
+---
+
+## Phase 1 — Clarify
+
+1. Đọc tất cả TC E2E (ID dạng `E2E-<FEATURE>-NN`) trong folder chỉ định
+2. Đọc TEST_PLAN để lấy danh sách journey + trục bao phủ dự kiến
+3. Đọc REQUIREMENT_ANALYSIS nếu có để đối chiếu BR
+
+---
+
+## Phase 2 — Review
+
+Đánh giá theo 4 tiêu chí:
+
+A. COVERAGE (quan trọng nhất):
+- Mỗi journey trong TEST_PLAN có TC tương ứng không?
+- Trục bao phủ (vd role × loại × bộ phận) có đủ không? (Decision Table)
+- Còn alternate flow / exception flow nào chưa có TC?
+
+B. END-TO-END INTEGRITY:
+- Steps có xuyên suốt qua các màn/module liên quan không?
+- Expected có verify **toàn chuỗi**: tạo → định tuyến → đồng bộ → hiển thị danh sách?
+- Data nhập màn A → hiện đúng màn B/DB/hệ thống ngoài không?
+- Verify points chung (V1…Vn) áp nhất quán cho mọi journey?
+
+C. INDEPENDENCE & PRECONDITION:
+- Mỗi journey có tiền điều kiện rõ (data + tài khoản/role) không?
+- Journey phụ thuộc nhau thứ tự? (vi phạm test isolation)
+- Có hướng dẫn cleanup/teardown sau mỗi journey?
+
+D. FORMAT & TRACE:
+- ID đúng dạng `E2E-<FEATURE>-NN`?
+- Mỗi TC có `Trace: [BR-xx]`; nếu không có BR phải ghi `⚠️ Chưa có Business Rule`?
+- TC phụ thuộc gap có dòng `🚫 Block: [GAP-xx]`?
+- Cuối file có Trace matrix + bảng TC block?
+- Expected là 1 bullet tổng hợp (không `✅ PASS/❌ FAIL`, không "hoạt động bình thường")?
+
+---
+
+## Checklist format file `.md`
+
+- **Metadata:** Title · Feature · Priority · Status(Draft) · Author · Tags · Trace · Block — mỗi trường 1 dòng.
+- **Test Data:** dạng list (không bảng); ghi cả tài khoản/role cần dùng.
+- **Steps:** phân biệt `[Action]`/`[Verify]`; KHÔNG có `- *Expected:* ...` sau từng bước.
+- **Expected:** 1 bullet; liệt kê chuỗi verify point rõ ràng.
+- **KHÔNG** có section `#### Python Test Mapping`.
+
+---
+
+## Output
+
+Mỗi tiêu chí: ✅ PASS | ⚠️ PARTIAL | ❌ MISSING + evidence (TC ID / journey)
+Score: A (excellent) / B (good) / C (needs improvement) / D (redo)
+Danh sách journey thiếu TC; TC cần sửa Expected; TC vi phạm isolation.
+Kết luận: sẵn sàng cho `qa-runner` chưa.

package/core/skills/qc/qa-reviewer/test-case/exploratory.md

@@ -0,0 +1,43 @@
+---
+version: 1.0
+updated: 2026-06-11
+ported_from: ai-automation-qc-base
+---
+
+# Review Charter — Exploratory
+ 
+Review chất lượng test charter trước khi QC chạy session.
+ 
+## Khi nào trigger
+- "review charter cho [Feature]"
+- Sau khi qa-designer/exploratory/charter xong, trước khi chạy session
+ 
+## Khi KHÔNG trigger
+- Review TC functional → dùng qa-reviewer/test-case/functional
+- Review session note → dùng qa-reviewer/script/exploratory
+ 
+---
+ 
+## Phase 1 — Read
+ 
+Đọc tất cả charter .md trong folder chỉ định + feature description nếu có.
+ 
+---
+ 
+## Phase 2 — Review
+ 
+7 tiêu chí:
+1. FOCUS: Charter đủ focus chưa? (quá rộng "Explore checkout" → cần narrow)
+2. RISK ALIGNMENT: Target vào risk cao nhất? (tiền → Money Tour, input → Saboteur)
+3. SFDIPOT COVERAGE: Tập hợp charter cover đủ 7 dimension?
+4. TOUR FIT: Tour phù hợp charter?
+5. OVERLAP: 2 charter cover cùng area? → merge
+6. MISSING: Area nào chưa có charter?
+7. TIME BUDGET: Tổng time-box fit resource QC?
+ 
+---
+ 
+## Output
+ 
+Mỗi charter: ✅ PASS | 🔧 REWORK | ✂️ SPLIT | 🔗 MERGE + feedback cụ thể
+Overall assessment + charter bổ sung nếu có gap

package/core/skills/qc/qa-reviewer/test-case/functional.md

@@ -0,0 +1,76 @@
+---
+version: 1.0
+updated: 2026-06-11
+ported_from: ai-automation-qc-base
+---
+
+# Review Test Case — Functional
+ 
+Review bộ functional TC và đánh giá chất lượng.
+ 
+## Khi nào trigger
+- "review TC cho [Feature]" / "check coverage"
+- Sau khi qa-designer xong, trước khi PO approve hoặc trước khi qa-runner
+ 
+## Khi KHÔNG trigger
+- Review Python script → dùng qa-reviewer
+- Review charter exploratory → dùng qa-reviewer/test-case/exploratory
+ 
+---
+ 
+## Phase 1 — Clarify
+ 
+1. Đọc tất cả .Test.md trong folder được chỉ định
+2. Đọc requirement/spec/user story nếu QC cung cấp
+3. Nếu không có requirement → review dựa trên TC content (vẫn check internal consistency)
+ 
+---
+ 
+## Phase 2 — Review
+ 
+Đánh giá theo 4 tiêu chí:
+ 
+A. COVERAGE (quan trọng nhất):
+- Requirement nào chưa được phủ?
+- TC nào không trace được về requirement? (TC dư)
+- Thiếu negative / edge / boundary test?
+ 
+B. TECHNIQUE:
+- Field có constraints → có EP + BVA không?
+- Logic AND/OR → có Decision Table không?
+- State → có transition hợp lệ VÀ forbidden không?
+- Thiếu implicit requirement? (rate limit, concurrent, audit, session, security)
+ 
+C. QUALITY:
+- Title rõ scenario + expected?
+- Expected result CỤ THỂ (2 QC test ra cùng kết luận)?
+- Test data có giá trị cụ thể (không placeholder)?
+- Mỗi TC chỉ verify 1 thing?
+- Priority đúng (core function = P0)?
+ 
+D. ANTI-PATTERN:
+- TC chung chung, không actionable?
+- Expected dạng "hiển thị đúng" (cảm tính)?
+- Hardcode credentials/URL?
+ 
+---
+
+## Checklist format file `.md` (đối chiếu skill layer qa-designer)
+
+- **Metadata:** đủ Title, Feature, Priority, Status, Author, Tags, **Trace**; Priority `P0/P1/P2` (không emoji); Status `Draft` text thuần; mỗi trường 1 dòng riêng.
+- **Trace:** mỗi TC link `[BR-xx](REQUIREMENT_ANALYSIS.md#3-business-rules)`; TC không có BR → phải có ⚠️ cảnh báo (không để trống/`—`).
+- **TC bị block:** TC phụ thuộc gap có dòng `🚫 Block: [GAP-xx]`; cuối file có **Trace matrix** (BR↔TC) + **bảng TC bị block** (Gap↔TC).
+- **Test Data:** dạng list (`- **Trường:** giá trị`), không dùng bảng.
+- **Test Steps:** phân biệt `[Action]`/`[Verify]`; KHÔNG có `- *Expected:* ...` sau mỗi bước.
+- **Expected Result:** 1 dòng bullet `-`, cụ thể; KHÔNG `✅ PASS/❌ FAIL`; không viết "hoạt động bình thường".
+- **KHÔNG** có section `#### Python Test Mapping`.
+- **1 concept/TC**; phân nhóm GUI / Functional.
+
+---
+ 
+## Output
+ 
+Mỗi tiêu chí: ✅ PASS | ⚠️ PARTIAL | ❌ MISSING + evidence cụ thể
+Score: A (excellent) / B (good) / C (needs improvement) / D (redo)
+Đề xuất TC cần thêm/sửa/xoá, sắp theo priority. Liệt kê TC thiếu Trace BR (⚠️) cần bổ sung.
+