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

package/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/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/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/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/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/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.
+

package/skills/qc/qa-reviewer/test-case/integration.md

@@ -0,0 +1,69 @@
+---
+version: 1.0
+updated: 2026-06-11
+ported_from: ai-automation-qc-base
+---
+
+# Review Test Case — Integration
+
+Review bộ TC tích hợp (GUI↔Backend, API, DB) và đánh giá chất lượng.
+
+## Khi nào trigger
+- "review TC integration cho [Feature]" / "check TC tích hợp"
+- Sau khi qa-designer/integration/* xong, trước khi qa-runner
+
+## Khi KHÔNG trigger
+- Review TC functional 1 màn/endpoint → `test-case/functional`
+- Review E2E xuyên nhiều module → `test-case/e2e`
+
+---
+
+## Phase 1 — Clarify
+
+1. Đọc tất cả TC integration trong folder chỉ định; xác định loại: GUI↔Backend / API / DB
+2. Đọc REQUIREMENT_ANALYSIS + DOC_GAPS nếu có
+3. Xác định chuỗi tích hợp (caller → component → downstream)
+
+---
+
+## Phase 2 — Review
+
+Đánh giá theo 4 tiêu chí:
+
+A. COVERAGE:
+- Mỗi điểm tích hợp (API call / DB write / event) trong chuỗi có TC tương ứng?
+- Đủ nhóm: happy → empty state / null → lỗi server (4xx/5xx) → failure/timeout → concurrency?
+- TC GUI↔Backend: có verify cả request gửi đúng lẫn render response đúng?
+
+B. INTEGRATION DEPTH (theo loại):
+- **GUI↔Backend:** Expected ghi rõ API liên quan + biểu hiện UI; state loading/empty/error đủ.
+- **API:** schema request/response đúng; failure/retry/timeout có TC; định tuyến phụ thuộc điều kiện có Decision Table.
+- **DB:** Expected ghi `bảng.cột = giá trị` cụ thể; có TC soft-delete, audit log, unique constraint; có hướng dẫn cleanup/query kiểm tra.
+
+C. QUALITY:
+- Expected CỤ THỂ (không "phản hồi đúng", "lưu thành công" chung chung)?
+- Test Data đủ (endpoint, payload mẫu, bảng/cột kiểm tra, điều kiện định tuyến)?
+- TC concurrency: mô tả rõ 2+ request đồng thời + kết quả kỳ vọng?
+
+D. FORMAT & TRACE:
+- Metadata đủ; Trace `[BR-xx]` hoặc `⚠️ Chưa có Business Rule`; `🚫 Block: [GAP-xx]` nếu bị chặn?
+- Cuối file có Trace matrix + bảng TC block?
+- Steps phân biệt `[Action]`/`[Verify]`; Expected 1 bullet; KHÔNG `✅/❌` inline.
+
+---
+
+## Checklist format file `.md`
+
+- **Test Data:** dạng list; ghi rõ endpoint/payload/bảng cần kiểm tra.
+- **Expected (DB):** phải có dạng `bảng.cột = giá trị` + query kiểm tra + cleanup instruction.
+- **Expected (API):** ghi mã HTTP + cấu trúc response + downstream effect.
+- **Expected (GUI):** ghi API gọi + biểu hiện UI (message/state/list cập nhật).
+
+---
+
+## Output
+
+Mỗi tiêu chí: ✅ PASS | ⚠️ PARTIAL | ❌ MISSING + evidence (TC ID / điểm tích hợp)
+Score: A (excellent) / B (good) / C (needs improvement) / D (redo)
+Điểm tích hợp thiếu TC; TC Expected mờ nhạt; TC DB thiếu cleanup.
+Kết luận: sẵn sàng cho `qa-runner` chưa.

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

@@ -0,0 +1,73 @@
+---
+version: 1.0
+updated: 2026-06-11
+ported_from: ai-automation-qc-base
+---
+
+# Review Test Case — Non-Functional
+
+Review bộ TC phi chức năng (performance, security, accessibility, compatibility) và đánh giá chất lượng.
+
+## Khi nào trigger
+- "review TC non-functional cho [Feature]" / "check TC hiệu năng/bảo mật/accessibility"
+- Sau khi qa-designer/non-functional xong, trước khi qa-runner
+
+## Khi KHÔNG trigger
+- Review TC chức năng → `test-case/functional`
+- Review TC tích hợp → `test-case/integration`
+
+---
+
+## Phase 1 — Clarify
+
+1. Đọc tất cả TC non-functional trong folder chỉ định; xác định loại: performance / security / accessibility / compatibility
+2. Đọc REQUIREMENT_ANALYSIS để lấy SLA, ngưỡng, môi trường mục tiêu
+3. Xác định công cụ đo đã được thống nhất (load tool, scanner, axe, lighthouse…)
+
+---
+
+## Phase 2 — Review
+
+Đánh giá theo 4 tiêu chí:
+
+A. COVERAGE (theo loại):
+- **Performance:** đủ scenario tải mục tiêu (normal / peak / max-data / concurrency / pagination)?
+- **Security:** có TC cho authZ/role (truy cập trái phép), injection, PII không lộ, session/timeout, rate limit?
+- **Accessibility:** có TC cho keyboard nav, focus order, label/aria, contrast (WCAG level rõ)?
+- **Compatibility:** liệt kê đủ browser/device/độ phân giải mục tiêu; mỗi target có TC riêng?
+
+B. MEASURABILITY — tiêu chí quan trọng nhất:
+- Expected có **ngưỡng pass cụ thể** không? (vd `< 2s`, `0 critical issues`, `WCAG 2.1 AA`)
+- Expected **không được** dùng: "nhanh", "ổn định", "hiển thị đúng", "bảo mật tốt"
+- Có ghi công cụ đo kèm theo ngưỡng không?
+- Performance TC: ghi rõ số user đồng thời, kích thước data, thời gian tải tối đa?
+- Security TC: ghi rõ payload thử nghiệm (OWASP top 10 input mẫu)?
+
+C. ENVIRONMENT & DATA:
+- TC cần môi trường đặc biệt (load test server, staging) có ghi chú rõ không?
+- TC cần data lớn / pre-populated data có hướng dẫn chuẩn bị?
+- Cleanup / teardown sau mỗi TC (đặc biệt security test)?
+
+D. FORMAT & TRACE:
+- Metadata đủ; Trace `[BR-xx]` / SLA source rõ; `🚫 Block: [GAP-xx]` nếu bị chặn?
+- Steps phân biệt `[Action]`/`[Verify]`; Expected 1 bullet; KHÔNG `✅/❌` inline?
+- Cuối file có Trace matrix + bảng TC block?
+
+---
+
+## Checklist format file `.md`
+
+- **Expected:** PHẢI có `ngưỡng + đơn vị + công cụ đo` — không có → ❌ MISSING.
+- **Performance TC:** ghi `concurrent users`, `data volume`, `target response time`.
+- **Security TC:** ghi `attack vector`, `input payload`, `expected block/response`.
+- **Accessibility TC:** ghi `WCAG version + level`, `tool (axe/lighthouse)`.
+- **Compatibility TC:** ghi `browser/OS version + device/resolution`.
+
+---
+
+## Output
+
+Mỗi tiêu chí: ✅ PASS | ⚠️ PARTIAL | ❌ MISSING + evidence (TC ID)
+Score: A (excellent) / B (good) / C (needs improvement) / D (redo)
+Danh sách TC Expected mờ nhạt (thiếu ngưỡng); loại non-functional thiếu coverage.
+Kết luận: sẵn sàng cho `qa-runner` chưa; ghi rõ TC nào cần môi trường đặc biệt.