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

package/core/skills/qc/qa-designer/integration/kafka.md

@@ -0,0 +1,40 @@
+---
+version: 1.0
+updated: 2026-06-11
+ported_from: ai-automation-qc-base
+---
+
+# Test Case — Integration Kafka (Message/Event)
+
+Skill **tự chứa** để viết TC tích hợp qua Kafka: producer phát event đúng, consumer xử lý
+đúng, đảm bảo ordering/idempotency/retry. Chỉ cần load file này.
+
+## Khi nào trigger
+- Action sinh event Kafka (vd tạo ticket → phát event sang service/CRM); verify topic/payload/thứ tự/khử trùng
+
+## Khi KHÔNG trigger
+- Tích hợp đồng bộ qua API → `integration/api` · verify DB → `integration/db`
+
+---
+
+## Format file TC (bắt buộc)
+- Metadata **list**: Title · Feature · Priority · Status(Draft) · Author(AI) · Tags · **Trace** `[BR-xx](REQUIREMENT_ANALYSIS.md#3-business-rules)` (không có → `⚠️ Chưa có Business Rule`) · **🚫 Block** `[GAP-xx](DOC_GAPS.md)`.
+- **Test Data** dạng list (payload) · **Steps** `[Action]`/`[Verify]` · **Expected** 1 bullet nêu **topic + field payload / hành vi consumer**.
+- Cuối file: Trace matrix + bảng TC block · bỏ nội dung gạch ngang.
+
+## Kỹ thuật áp dụng
+- **Message/event:** verify topic, key, payload schema, điều kiện phát.
+- **Ordering & idempotency:** thứ tự theo key, message trùng/out-of-order.
+- **Error/Retry:** consumer lỗi → retry/DLQ.
+
+## Phase 1 — Clarify
+Topic/key/payload + điều kiện phát (khi nào phát/không) · consumer xử lý gì + side-effect + idempotent ·
+yêu cầu ordering · xử lý trùng/out-of-order/lỗi (retry, DLQ).
+
+## Phase 2 — Write
+Nhóm TC: phát đúng topic+payload (happy) → điều kiện không phát → consumer xử lý đúng → message trùng (idempotent)
+→ out-of-order → consumer lỗi → retry/DLQ → ordering theo key. Setup/teardown consumer test.
+Mỗi TC bám Format; trace BR; gap chặn → 🚫 Block.
+
+## Output
+File TC trong `docs/<project>/<feature>/`. Mỗi TC ghi topic, key, payload cần verify + hành vi consumer. Bàn giao `qa-reviewer`.

package/core/skills/qc/qa-designer/non-functional.md

@@ -0,0 +1,40 @@
+---
+version: 1.0
+updated: 2026-06-11
+ported_from: ai-automation-qc-base
+---
+
+# Test Case — Non-Functional
+
+Skill **tự chứa** để viết TC phi chức năng: performance, security, accessibility, compatibility.
+Trọng tâm "hệ thống hoạt động TỐT thế nào". Chỉ cần load file này.
+
+## Khi nào trigger
+- Yêu cầu hiệu năng/tải, bảo mật, accessibility, tương thích; feature có SLA, dữ liệu nhạy cảm, hoặc data lớn
+
+## Khi KHÔNG trigger
+- Kiểm thử chức năng theo đặc tả → `functional/*` · tích hợp module → `integration/*`
+
+---
+
+## Format file TC (bắt buộc)
+- Metadata **list**: Title · Feature · Priority · Status(Draft) · Author(AI) · Tags · **Trace** `[BR-xx](REQUIREMENT_ANALYSIS.md#3-business-rules)` (không có → `⚠️ Chưa có Business Rule`) · **🚫 Block** `[GAP-xx](DOC_GAPS.md)`.
+- **Test Data** dạng list · **Steps** `[Action]`/`[Verify]` · **Expected** 1 bullet có **ngưỡng đo cụ thể** (không "nhanh/ổn định").
+- Cuối file: Trace matrix + bảng TC block · bỏ nội dung gạch ngang.
+
+## Kỹ thuật / loại
+- **Performance:** response time dưới tải mục tiêu; danh sách lớn (max data); pagination; concurrency.
+- **Security:** authZ/role (truy cập trái phép → chặn), injection, PII (SĐT) không lộ, session/timeout, rate limit.
+- **Accessibility:** keyboard nav, focus, label/aria, contrast (WCAG).
+- **Compatibility:** trình duyệt/thiết bị/độ phân giải mục tiêu.
+
+## Phase 1 — Clarify
+Loại non-functional + tiêu chí đo (ngưỡng cụ thể: thời gian, số user, WCAG level) · môi trường & công cụ
+(load tool, scanner, axe/lighthouse) · dữ liệu/tải mẫu.
+
+## Phase 2 — Write
+Mỗi TC bám Format; **Expected có ngưỡng pass + công cụ đo**; đánh dấu TC cần môi trường/data đặc biệt.
+Trace BR; gap chặn → 🚫 Block.
+
+## Output
+File TC non-functional trong `docs/<project>/<feature>/`. Mỗi TC ghi tiêu chí đo + ngưỡng + công cụ. Bàn giao `qa-reviewer`.

package/core/skills/qc/qa-planner/test-plan.md

@@ -0,0 +1,120 @@
+---
+version: 1.0
+updated: 2026-06-11
+ported_from: ai-automation-qc-base
+---
+
+# Lập Test Plan
+
+Tổng hợp **output của qa-analyst** thành **Test Plan** cho một feature.
+
+**Đầu vào (bắt buộc, chỉ 2 nguồn):**
+1. `docs/<project>/<feature>/REQUIREMENT_ANALYSIS.md` — chức năng, BR-xx, AC-xx, data flow (qa-analyst).
+2. `docs/<project>/<feature>/DOC_GAPS.md` — bảng gap GAP-xx, mức độ, gap Blocker (qa-analyst).
+
+## Khi nào trigger
+- "lập test plan cho [Feature]" / "viết test plan"
+- Sau khi qa-analyst xong (đã có REQUIREMENT_ANALYSIS + DOC_GAPS)
+- Trước khi qa-designer thiết kế chi tiết TC — test plan là khung định hướng
+
+## Khi KHÔNG trigger
+- Chưa có REQUIREMENT_ANALYSIS / DOC_GAPS → chạy qa-analyst trước
+- Thiết kế test case chi tiết (.Test.md) → dùng qa-designer
+- Bóc tách yêu cầu/spec, lập danh sách gap → dùng qa-analyst
+
+---
+
+## Phase 1 — Thu thập đầu vào
+
+1. Đọc `REQUIREMENT_ANALYSIS.md`: nắm chức năng, các BR-xx và AC-xx, data flow,
+   integration/failure point.
+2. Đọc `DOC_GAPS.md`: lấy danh sách gap, đặc biệt **gap Blocker còn Open** → đây là
+   nguồn cho cột "Phụ thuộc" và cho Entry criteria.
+3. Map mỗi nhóm BR sang **layer test** của qa-designer: functional/gui-screen,
+   gui-feature, api, integration, e2e/journey, non-functional.
+4. **Bỏ qua nội dung gạch ngang** (đã loại ở qa-analyst) — không đưa vào plan.
+
+---
+
+## Phase 2 — Lập Test Plan
+
+Điền đủ template bên dưới. Nguyên tắc:
+- Mỗi **vùng test** map về BR-xx cụ thể, gắn **Layer + Loại test + Priority + ước lượng TC**;
+  hiển thị **rule chi tiết** ngay trong bảng (cột riêng, dùng `<br>` cho nhiều rule).
+- **Priority theo rủi ro suy ra từ BR + gap:** core function / logic định tuyến / sinh
+  mã / tiền-dữ liệu = P0.
+- Vùng/journey còn phụ thuộc **gap Blocker** → ghi rõ cột "Phụ thuộc" (GAP#); Entry
+  criteria yêu cầu đóng các gap đó trước khi thiết kế TC.
+- Liệt kê **E2E journey** đầy đủ (mỗi journey: tiền điều kiện, kết quả/định tuyến kỳ
+  vọng, BR, phụ thuộc, priority) + bộ **verify point chung** sau submit.
+- Mục Rủi ro: rút trực tiếp từ gap Blocker + các BR logic phức tạp.
+
+---
+
+## Output — Template `TEST_PLAN.md`
+
+Đặt tại `docs/<project>/<feature>/TEST_PLAN.md`:
+
+```markdown
+# Test Plan – <Feature>
+
+| Trường | Giá trị |
+|---|---|
+| Feature / Project / Module | … |
+| Người lập | qa-planner |
+| Ngày / Phiên bản | … |
+| Nguồn | REQUIREMENT_ANALYSIS · DOC_GAPS |
+
+## 1. Mục tiêu
+Mục tiêu test của feature (1–3 câu).
+
+## 2. Phạm vi
+- **In scope:** chức năng/BR được test.
+- **Out of scope:** phần để tài liệu/feature khác; nội dung gạch ngang đã loại.
+
+## 3. Test items theo vùng & độ ưu tiên
+Bảng — mỗi vùng kèm rule chi tiết + metadata:
+
+| # | Vùng test | Rule chi tiết | Layer (qa-designer) | Loại test | Pri | Ước lượng |
+|---|---|---|---|---|---|---|
+| ① | <vùng> | **BR-xx:** … <br>**BR-yy:** … | functional/gui-screen | Functional/Negative | P0 | n |
+| … | … | … | … | … | … | … |
+
+> Tổng ước lượng sơ bộ: ~N test case (+ E2E).
+
+### 3.1 Danh sách E2E đầy đủ
+Verify points chung (sau Submit): V1 tạo thành công · V2 mã/ID đúng · V3 hiển thị danh
+sách · V4… · Trục bao phủ: <các chiều tổ hợp>.
+
+| ID | Journey | Tiền điều kiện | Kết quả kỳ vọng | BR | Phụ thuộc | Pri |
+|---|---|---|---|---|---|---|
+| E2E-XX-01 | … | … | … | BR-… | GAP# | P0 |
+
+## 4. Cách tiếp cận (Test approach)
+Kỹ thuật áp dụng: EP+BVA, Decision Table (cho logic điều kiện), state/lookup,
+integration, negative/exploratory; tự động hoá theo `CLAUDE.md` (Playwright + pytest-playwright + Trace + pytest-html).
+
+## 5. Tiêu chí Vào / Ra
+- **Entry:** gap Blocker (trong DOC_GAPS) đã Answered; doc phụ thuộc sẵn sàng; môi trường + tài khoản role.
+- **Exit:** pass P0=100%, P1≥95%; không còn defect Blocker/Critical; mọi BR/AC được trace; báo cáo pytest-html + Playwright Trace.
+
+## 6. Rủi ro (risk-based)
+| Rủi ro | Ảnh hưởng | Mức | Giảm thiểu |
+(rút từ gap Blocker + BR logic phức tạp)
+
+## 7. Dữ liệu & Môi trường
+Tài khoản các role, dữ liệu mẫu (biên/edge), môi trường staging.
+
+## 8. Deliverables
+Test case `.Test.md`, script Playwright/pytest + Page Object, báo cáo pytest-html + Playwright Trace.
+
+## 9. Lịch trình (milestone phụ thuộc gap)
+| Mốc | Điều kiện | Trạng thái |
+M0 phân tích → M1 đóng gap → M2 thiết kế TC → M3 review TC → M4 chạy → M5 review script.
+
+## 10. Trạng thái hiện tại
+Blocked/Ready + phần có thể làm sớm (không phụ thuộc gap Blocker).
+```
+
+Kết thúc bằng: tóm tắt số vùng test + tổng TC ước lượng + trạng thái (Ready/Blocked dựa
+trên gap Blocker còn Open) và gợi ý bước kế tiếp (bàn giao qa-designer khi đã Ready).

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.