@educa-corp/sdd-framework 0.9.7 → 0.9.8

package/core/modules/qc-playwright/stack-profile.yaml

@@ -1,66 +0,0 @@
-# QC automation module — Python + pytest-playwright + Page Object
-# Used by the /qc-* commands (the official QC automation pipeline ported from the QC team).
-# This is the QC test-authoring/execution stack, independent of the dev implementation
-# module (java-spring, react, flutter, …). Selected via tech_stack.qc_module or per /qc-* run.
-
-build:
-  test: "python3 -m pytest"
-  e2e: "python3 -m pytest -m e2e"
-  report: "python3 -m pytest --html=reports/<feature>/report.html --self-contained-html"
-  show_trace: "python3 -m playwright show-trace <test-results/<nodeid>/trace.zip>"
-
-architecture:
-  style: "Page Object Model over pytest-playwright — Markdown test-case first, Python second"
-  key_rules:
-    - "Markdown-first: never generate Python until a reviewed .Test.md exists for the feature"
-    - "No Allure, no hand-written dashboard, no record_video — use Playwright Trace + pytest-html"
-    - "No hard-coded URL/credential/timeout — read from Env.* and CONFIG[...]"
-    - "No time.sleep() — use Playwright auto-wait / expect()"
-    - "Each test independent via pytest-playwright fixtures (page / logged_in_page / …)"
-    - "Page Object extends slim BasePage; split 3 layers: locators _x(), actions verb_noun(), assertions assert_x() using expect()"
-    - "Locator priority: data-testid → role → label/text → CSS → avoid XPath"
-    - "test-id values come from the FE tech-design Test Selectors contract (tech-doc gộp cấp PRD: {TICKET-ID}-tech-design.md, bảng Test Selectors §4.5.6 — lọc theo cột 'Phục vụ SC') — prefer them (no runtime scan); fall back to role/text only when an actionable element has no test-id there, and note the gap"
-    - "Group tests by (role, account) so login/logout never interleaves across roles"
-    - "Cover 100% of TCs in the .Test.md — every TC ends Pass/Fail/Skip, none left Draft"
-  folder_structure: |
-    {qc_artifact_dir}test-cases/   ← test-case Markdown (.Test.md) — source of truth (mặc định docs/, lộ ra ngoài)
-    pages/                         ← Page Object Model
-    │   ├── base_page.py           ← slim BasePage (click/fill/wait/screenshot)
-    │   └── <feature>_page.py
-    tests/                         ← pytest scripts, 1-1 with test-cases/
-    │   ├── conftest.py            ← fixtures: browser, page, logged_in_page, tracing
-    │   └── <project>/test_<feature>.py
-    utils/                         ← config_loader, logger, steps, test_ordering, report helpers
-    test_data/                     ← JSON datasets
-    config/config.yaml             ← browser, timeout, video/screenshot/trace toggles
-    reports/  test-results/        ← generated (gitignored): html report, trace.zip, screenshots
-
-coding_standards:
-  naming:
-    test_case_id: "TC_<FEATURE>_<NNN>"
-    test_class: "TestFeatureHappyCase"
-    test_function: "test_TC<NNN>_<snake_case>"
-    page_object: "<feature>_page.py with <Feature>Page class extending BasePage"
-    files:
-      test_case_md: "{qc_artifact_dir}test-cases/TC_<FEATURE>.Test.md"
-      page_object: "pages/<feature>_page.py"
-      test_script: "tests/<project>/test_<feature>.py"
-  patterns:
-    steps: "wrap steps with `with step(\"…\")` (from utils.steps import step)"
-    assertions: "Playwright expect() — never bare assert on dynamic UI"
-    fixtures: "auth fixtures register via register_auth_fixtures([...]) in project conftest"
-    fail_triage: "classify each FAIL as script-bug (fix selector/logic) vs product-gap (keep FAIL + evidence, never fake-pass)"
-
-testing:
-  layers: "functional (gui-screen / gui-feature / api), integration (api/db/gui/kafka), e2e (journey), non-functional, exploratory"
-  runner: "pytest-playwright; trace via context.tracing.start in conftest"
-  report: "pytest-html (--html ... --self-contained-html) + Playwright Trace viewer"
-
-trace_tags:
-  # QC tests map back to the framework's scenarios — drives qc_status in the trace TSV.
-  verifies: "# @trace.verifies={UC-ID}-SC{N}"
-  source: "# @trace.source=specs/{domain}/{prd-slug}/bdd/{platform}/{UC-ID}-{slug}.feature"
-  test_type: "# @trace.test_type=functional|integration|e2e|non-functional"
-
-# qc_status: /qc-run-script writes pass|fail|skip|not_run + qc_run_at into {trace_dir}/{UC-ID}.tsv
-# (parallel to dev_selftest), surfaced in Living Docs as the OFFICIAL QC automation result.

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

@@ -1,95 +0,0 @@
----
-version: 1.0
-updated: 2026-06-11
-ported_from: ui-automation-testing
----
-
-# 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).
-**Điểm `XX/100`** — ánh xạ mức độ sang điểm trừ: 🔴 = `FAIL` (−5đ) · 🟠 = `WARN` (−2đ) ·
-🟡 = ghi nhận, không trừ. ≥80 đạt · 60–79 cần cải thiện · <60 không đạt.
-**Verdict:** `≥80` VÀ không còn 🔴 → **`APPROVED`**; ngược lại **`NEEDS_FIX`**.
-
-**Ghi vào `{qc_artifact_dir}test-cases/REVIEW_<FEATURE>.md`** — thêm một hàng vào bảng Tổng quan
-(cột `Tầng` phân biệt vai soát-code với vai soát-kịch-bản; **không ghi đè** hàng của vai kia).
-Khuôn + quy tắc ghi: `../shared/review-file-template.md`.
-
-**KHÔNG tự sửa code** — chỉ nêu `file:line` + cách sửa. Người sửa là `/qc-design-script`.

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

@@ -1,109 +0,0 @@
----
-version: 1.0
-updated: 2026-06-11
-ported_from: ui-automation-testing
----
-
-# 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).
-**Điểm `XX/100`** — ánh xạ mức độ sang điểm trừ: 🔴 = `FAIL` (−5đ) · 🟠 = `WARN` (−2đ) ·
-🟡 = ghi nhận, không trừ. ≥80 đạt · 60–79 cần cải thiện · <60 không đạt.
-**Verdict:** `≥80` VÀ không còn 🔴 → **`APPROVED`**; ngược lại **`NEEDS_FIX`**.
-
-**Ghi vào `{qc_artifact_dir}test-cases/REVIEW_<FEATURE>.md`** — thêm một hàng vào bảng Tổng quan
-(cột `Tầng` phân biệt vai soát-code với vai soát-kịch-bản; **không ghi đè** hàng của vai kia).
-Khuôn + quy tắc ghi: `../shared/review-file-template.md`.
-
-**KHÔNG tự sửa code** — chỉ nêu `file:line` + cách sửa. Người sửa là `/qc-design-script`.

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

@@ -1,99 +0,0 @@
----
-version: 1.0
-updated: 2026-06-11
-ported_from: ui-automation-testing
----
-
-# 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).
-**Điểm `XX/100`** — ánh xạ mức độ sang điểm trừ: 🔴 = `FAIL` (−5đ) · 🟠 = `WARN` (−2đ) ·
-🟡 = ghi nhận, không trừ. ≥80 đạt · 60–79 cần cải thiện · <60 không đạt.
-**Verdict:** `≥80` VÀ không còn 🔴 → **`APPROVED`**; ngược lại **`NEEDS_FIX`**.
-
-**Ghi vào `{qc_artifact_dir}test-cases/REVIEW_<FEATURE>.md`** — thêm một hàng vào bảng Tổng quan
-(cột `Tầng` phân biệt vai soát-code với vai soát-kịch-bản; **không ghi đè** hàng của vai kia).
-Khuôn + quy tắc ghi: `../shared/review-file-template.md`.
-
-**KHÔNG tự sửa code** — chỉ nêu `file:line` + cách sửa. Người sửa là `/qc-design-script`.

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

@@ -1,134 +0,0 @@
----
-version: 1.0
-updated: 2026-06-11
-ported_from: ui-automation-testing
----
-
-# 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).
-**Điểm `XX/100`** — ánh xạ mức độ sang điểm trừ: 🔴 = `FAIL` (−5đ) · 🟠 = `WARN` (−2đ) ·
-🟡 = ghi nhận, không trừ. ≥80 đạt · 60–79 cần cải thiện · <60 không đạt.
-**Verdict:** `≥80` VÀ không còn 🔴 → **`APPROVED`**; ngược lại **`NEEDS_FIX`**.
-
-**Ghi vào `{qc_artifact_dir}test-cases/REVIEW_<FEATURE>.md`** — thêm một hàng vào bảng Tổng quan
-(cột `Tầng` phân biệt vai soát-code với vai soát-kịch-bản; **không ghi đè** hàng của vai kia).
-Khuôn + quy tắc ghi: `../shared/review-file-template.md`.
-
-**KHÔNG tự sửa code** — chỉ nêu `file:line` + cách sửa. Người sửa là `/qc-design-script`.

package/core/skills/qc/qa-runner/e2e.md

@@ -1,49 +0,0 @@
----
-version: 1.0
-updated: 2026-06-11
-ported_from: ui-automation-testing
----
-
-# Gen Script — E2E Journey
-
-Skill **tự chứa**: convert journey `.Test.md` → Python pytest + Playwright đầu→cuối xuyên nhiều màn/module. Chỉ cần load file này.
-
-## Khi nào trigger
-- Convert TC E2E journey (output qa-designer e2e/journey) sang script; TC đã Reviewed
-
-## Khi KHÔNG trigger
-- Test 1 màn/field → `functional/*` · 1 điểm tích hợp → `integration.md`
-
-## Quy ước script (bám CLAUDE.md)
-- Tiền đề: `.Test.md` đã Reviewed. KHÔNG hardcode → `Env.*`/`CONFIG`; KHÔNG `time.sleep()` → `expect()`.
-- Dùng chuỗi Page Object xuyên các màn; verify points (V1…Vn) thành các `assert_*()` rõ ràng.
-- **Tiền điều kiện qua fixture** (tài khoản role, dữ liệu lớp/buổi); **cleanup** sau journey; mỗi journey độc lập.
-- Cần verify hệ thống ngoài (DB/CRM) → client/fixture riêng. Bọc mỗi chặng `with step("…")` (`from utils.steps import step`, **KHÔNG Allure**); assertion `expect()`; marker `@pytest.mark.e2e`.
-- Naming: class `TestFeatureE2E`, method `test_E2E_<NN>_<snake>`.
-
-## Phase 1 — Clarify
-Đọc journey `.Test.md` · các màn/PO + hệ thống verify · tài khoản role & data cần dựng · điểm cleanup.
-
-## Phase 2 — Generate
-Mỗi journey → 1 test; bọc từng bước `with step("…")`; cuối journey verify đủ V1…Vn (tạo/mã/định tuyến/đồng bộ/danh sách).
-Journey còn phụ thuộc gap → tạo test `@pytest.mark.skip(reason="GAP-UC{N}-{nnn}")` hoặc xfail.
-
-## Phase 3 — Verify
-`py_compile` + `pytest --collect-only -q` · chạy (môi trường staging + CRM) · cập nhật Status TC.
-**Phân loại FAIL — 3 nhãn, luật ở `/qc-run-script` §Chạy lại trước khi kết luận** (chạy lại ×2 trước, rồi người xác nhận; **không chép lại luật ở đây**). Đặc thù E2E: journey fail vì 1 bước feature chưa wire = `product-gap` (giữ FAIL/skip + bằng chứng), không phải lỗi script; sai selector/state → `script-bug`. Journey dài qua nhiều bước **dễ ra `flaky` hơn test đơn lẻ** — một bước chậm bất thường là đủ; nên đừng vội gọi `product-gap` khi chưa chạy lại.
-
-## Output
-Script `tests/<project>/e2e/test_<feature>.py` + Page Object/client tái dùng. Bàn giao `qa-reviewer`.
-
-## Phase 4 — Report (bắt buộc sau khi chạy test)
-Report = **Playwright Trace viewer + pytest-html** (KHÔNG Allure, KHÔNG dashboard tự viết).
-
-1. Chạy test kèm pytest-html (trace đã bật sẵn ở conftest → mỗi test có `test-results/<nodeid>/trace.zip`):
-   ```bash
-   pytest tests/<project>/e2e/test_<feature>.py --html=reports/<feature>/report.html --self-contained-html
-   ```
-2. Gửi cho người dùng:
-   - HTML report: `reports/<feature>/report.html` (self-contained, mở trực tiếp).
-   - Trace từng test (debug step-by-step): `python3 -m playwright show-trace test-results/<nodeid>/trace.zip`.
-   - Tóm tắt: **TOTAL / PASS / FAIL / SKIP** + duration.
-3. TC Fail → mở trace tương ứng để xem timeline/DOM snapshot/network, phân loại theo **3 nhãn** (`script-bug` · `product-gap` · `flaky`) — luật đầy đủ + bước chạy lại ×2 ở `/qc-run-script` §Chạy lại trước khi kết luận, **không chép lại ở đây**; ghi mô tả lỗi tiếng Việt dễ hiểu vào Status/khối kết quả của `.Test.md`.

package/core/skills/qc/qa-runner/functional/api.md

@@ -1,35 +0,0 @@
----
-version: 1.0
-updated: 2026-06-11
-ported_from: ui-automation-testing
----
-
-# Gen Script — Functional API
-
-Skill **tự chứa**: convert `.Test.md` API → Python pytest (requests/httpx), không qua UI. Chỉ cần load file này.
-
-## Khi nào trigger
-- Convert TC API (output qa-designer functional/api) sang script; TC đã Reviewed
-
-## Khi KHÔNG trigger
-- Test qua UI → `functional/gui-screen.md` · tích hợp đa thành phần → `integration.md`
-
-## Quy ước script (bám CLAUDE.md)
-- Tiền đề: `.Test.md` đã Reviewed. KHÔNG hardcode base URL/token → `Env.*`/`CONFIG`.
-- Client API gói trong helper/fixture (base url, auth header); KHÔNG rải request rời rạc trong test.
-- Assertion: `assert` cho status code + field response (jsonpath); `with step(...)` (`from utils.steps`) cho mỗi call.
-- Marker `@pytest.mark.api` + category; fixture `function`; test độc lập (tạo→cleanup data qua API/teardown).
-- Naming: class `TestFeatureAPI`, method `test_TC<NNN>_<snake>`.
-
-## Phase 1 — Clarify
-Đọc `.Test.md` · base url/auth/role · client/fixture đã có chưa · data setup/cleanup.
-
-## Phase 2 — Generate
-Mỗi TC → 1 test gọi endpoint với request từ Test Data; assert **status code + field body** theo Expected.
-Nhóm happy/validation/auth/not-found/edge. Data từ `test_data/`.
-
-## Phase 3 — Verify
-`py_compile` + `pytest --collect-only -q` · chạy · cập nhật Status TC · in mapping.
-
-## Output
-Script `tests/<project>/functional/api/test_<feature>.py` + API client/fixture nếu mới. Bàn giao `qa-reviewer`.

package/core/skills/qc/qa-runner/functional/gui-feature.md

@@ -1,57 +0,0 @@
----
-version: 1.0
-updated: 2026-06-11
-ported_from: ui-automation-testing
----
-
-# Gen Script — Functional GUI Feature (đa màn hình)
-
-Skill **tự chứa**: convert `.Test.md` feature span ≥2 màn → Python pytest + Playwright. Chỉ cần load file này.
-
-## Khi nào trigger
-- Convert TC feature đa màn (output qa-designer gui-feature) sang script; TC đã Reviewed
-
-## Khi KHÔNG trigger
-- Gọn 1 màn → `functional/gui-screen.md` · API → `functional/api.md` · đầu-cuối + đồng bộ → `e2e.md`
-
-## Quy ước script (bám CLAUDE.md)
-- Tiền đề: `.Test.md` đã Reviewed. KHÔNG `time.sleep()` → `expect()`/auto-wait; KHÔNG hardcode → `Env.*`/`CONFIG`.
-- **Một Page Object / màn**; điều hướng giữa màn là action trả về PO màn kế (`return NextPage(self.page)`).
-- PO 3 lớp (kế thừa slim `BasePage`, **KHÔNG Allure**) + bọc bước `with step("…")` (`from utils.steps import step`); assertion `expect()`; mọi interaction qua PO; selector constants `UPPER_SNAKE`.
-- Marker + fixture pytest-playwright từ root `tests/conftest.py` (`page`/`logged_in_page`/`logged_in_page_gv`/`login_page`/`dashboard`); test độc lập + cleanup; naming `Test*{...}` / `test_TC<NNN>_<snake>`.
-
-## Phase 1 — Clarify
-Đọc `.Test.md` · liệt kê các màn/PO cần · state truyền giữa màn · fixture dựng tiền điều kiện (data qua nhiều bước).
-**Locator: đọc hợp đồng TRƯỚC, dò DOM là bước cuối.** Thứ tự bắt buộc (luật đầy đủ + lý do ở `/qc-design-script` §Role & stack — **không chép lại ở đây**):
-
-1. **Test-id contract** — bảng *Test Selectors* §4.5.6 của tech-doc gộp, lọc theo cột "Serves SC" khớp SC của UC này. TÊN thuộc tính đọc từ `@trace.testid_attr` ở header tech-doc (đừng suy từ platform). Feature đa màn: một UC chạm nhiều màn nhưng **vẫn một bảng §4.5.6** cho cả platform — lọc theo SC, không theo màn.
-2. **Role + accessible name** — cho element có action mà §4.5.6 chưa phủ.
-3. **Dò DOM** — CHỈ khi 1 và 2 đều không định vị được. Dump class/`aria-label`/role, rồi nhìn kết quả:
-   - **3a. Element ĐÃ mang test-id trong code** → **DỪNG, đừng tự dùng id nhặt được.** Đây là ca *code đi trước hợp đồng*: chạy `/map-testids {UC-ID}` để đưa id đó vào §4.5.6 (nhánh `existing` — reverse-document), rồi quay lại bậc 1.
-   - **3b. Element KHÔNG có test-id nào** (chỉ class/role) → mới dùng class/role: BEM `feature__el`; carousel dot thường `role="tab"` + class `--active` (không `aria-selected`). **VÀ ghi một GAP**: element nào, màn nào, thiếu test-id → đề nghị dev gắn rồi chạy `/map-testids` lại. Đừng im lặng sống với selector giòn.
-
-## Phase 2 — Generate
-**PHỦ HẾT 100%**: 1 test cho **MỌI** TC trong file (`grep -cE "^#{2,4} *TC_"` = số test phải sinh), KHÔNG chọn tập đại diện, KHÔNG để TC nào Draft; TC bất khả thi → `pytest.skip`/`xfail` + lý do.
-Bọc mỗi chặng bằng `with step("…")`; dùng chuỗi PO theo điều hướng.
-Phủ TC điều hướng forward/back/giữ-reset state. Data từ `test_data/`. Marker mới (`bva ep e2e`…) đăng ký `pytest.ini`.
-
-## Phase 3 — Verify
-`py_compile` + `pytest --collect-only -q` (**số collect = tổng TC**; thiếu → sinh nốt) · chạy · cập nhật Status TC (verify KHÔNG còn Draft) · in mapping.
-**Gom nhóm role/account** tự áp qua `utils/test_ordering.py` (root conftest); fixture auth mới → `register_auth_fixtures([...])`. ⚠️ Run dài bị **WSL suspend** có thể gây flaky login/timeout → re-run TC đó + merge report.
-**Phân loại FAIL — 3 nhãn, luật ở `/qc-run-script` §Chạy lại trước khi kết luận** (chạy lại ×2 trước, rồi người xác nhận; **không chép lại ở đây**). Đặc thù đa màn: sai selector/logic → `script-bug`, sửa & chạy lại; feature chưa wire/defect **đỏ nhất quán** → `product-gap`, giữ FAIL + ghi bằng chứng vào khối "Kết quả thực thi" đầu `.Test.md`, không fake-pass. **State truyền giữa màn là nguồn `flaky` phổ biến** — điều hướng nhanh hơn/chậm hơn một nhịp là đủ đổi kết quả; ghi nghi vấn đó vào phần nguyên nhân.
-
-## Output
-Script + nhiều Page Object (mỗi màn) trong `pages/<project>/...`. Bàn giao `qa-reviewer` (script).
-
-## Phase 4 — Report (bắt buộc sau khi chạy test)
-Report = **Playwright Trace viewer + pytest-html** (KHÔNG Allure, KHÔNG dashboard tự viết).
-
-1. Chạy test kèm pytest-html (trace đã bật sẵn ở conftest → mỗi test có `test-results/<nodeid>/trace.zip`):
-   ```bash
-   pytest tests/<project>/.../test_<feature>.py --html=reports/<feature>/report.html --self-contained-html
-   ```
-2. Gửi cho người dùng:
-   - HTML report: `reports/<feature>/report.html` (self-contained, mở trực tiếp).
-   - Trace từng test (debug step-by-step): `python3 -m playwright show-trace test-results/<nodeid>/trace.zip`.
-   - Tóm tắt: **TOTAL / PASS / FAIL / SKIP** + duration.
-3. TC Fail → mở trace tương ứng để xem timeline/DOM snapshot/network, phân loại theo **3 nhãn** (`script-bug` · `product-gap` · `flaky`) — luật đầy đủ + bước chạy lại ×2 ở `/qc-run-script` §Chạy lại trước khi kết luận, **không chép lại ở đây**; ghi mô tả lỗi tiếng Việt dễ hiểu vào Status/khối kết quả của `.Test.md`.