sdtk-kit 0.3.0

package/assets/toolkit/toolkit/skills/sdtk-screen-design-spec/scripts/validate_flow_action_spec_numbering.py

@@ -0,0 +1,249 @@
+from __future__ import annotations
+
+import argparse
+import re
+import sys
+from dataclasses import dataclass
+from pathlib import Path
+
+
+TABLE_ROW_RE = re.compile(r"^\|.*\|\s*$")
+TABLE_SEP_RE = re.compile(r"^\|\s*:?-+:?\s*(\|\s*:?-+:?\s*)+\|\s*$")
+HEADING_RE = re.compile(r"^(#{1,6})\s+(.*\S)\s*$")
+VI_DIACRITIC_RE = re.compile(r"[À-ỹ]")
+VI_PHRASE_RE = re.compile(
+    r"\b("
+    r"thuc thi|hien thi|cap nhat|bo sung|can xac nhan|trong anh|"
+    r"duoc tra kem|tao lich|theo user|muc global|thay the cho|bao gom"
+    r")\b",
+    re.IGNORECASE,
+)
+INLINE_HEADING_RE = re.compile(r".+\s#{2,}\s+\S")
+
+
+@dataclass(frozen=True)
+class Occurrence:
+    number: int
+    section: str
+    line_no: int
+    raw_line: str
+
+
+def _split_table_row(line: str) -> list[str]:
+    # Keep it simple: markdown tables in our spec are plain and don't escape pipes.
+    parts = [p.strip() for p in line.strip().strip("|").split("|")]
+    return parts
+
+
+def _read_text(path: Path) -> str:
+    # Prefer UTF-8; fallback to cp1252-ish if user saved without UTF-8.
+    for enc in ("utf-8", "utf-8-sig", "cp1252"):
+        try:
+            return path.read_text(encoding=enc)
+        except UnicodeDecodeError:
+            continue
+    return path.read_text(errors="replace")
+
+
+def collect_hygiene_issues(md_text: str, *, en_check: bool) -> dict[str, list[tuple[int, str]]]:
+    issues: dict[str, list[tuple[int, str]]] = {
+        "encoding": [],
+        "vi_diacritic": [],
+        "vi_phrase": [],
+        "inline_heading": [],
+    }
+    lines = md_text.splitlines()
+    in_code_block = False
+
+    for idx, line in enumerate(lines, start=1):
+        stripped = line.strip()
+        if stripped.startswith("```"):
+            in_code_block = not in_code_block
+            continue
+        if in_code_block:
+            continue
+
+        if "�" in line or "ↁE" in line:
+            issues["encoding"].append((idx, line))
+
+        # A common corruption symptom: merged headings inside normal text lines.
+        if INLINE_HEADING_RE.search(line) and not stripped.startswith("#"):
+            issues["inline_heading"].append((idx, line))
+
+        if en_check:
+            lowered = stripped.lower()
+            if "original text" in lowered:
+                continue
+            if VI_DIACRITIC_RE.search(line):
+                issues["vi_diacritic"].append((idx, line))
+            elif VI_PHRASE_RE.search(line):
+                issues["vi_phrase"].append((idx, line))
+
+    return issues
+
+
+def parse_action_tables(md_text: str) -> list[Occurrence]:
+    occurrences: list[Occurrence] = []
+    current_section = ""
+    lines = md_text.splitlines()
+
+    i = 0
+    while i < len(lines):
+        line = lines[i]
+
+        m = HEADING_RE.match(line)
+        if m:
+            current_section = m.group(2).strip()
+
+        # Detect table header
+        if i + 1 < len(lines) and TABLE_ROW_RE.match(line) and TABLE_ROW_RE.match(lines[i + 1]):
+            header = _split_table_row(line)
+            sep = lines[i + 1]
+
+            if not TABLE_SEP_RE.match(sep):
+                i += 1
+                continue
+
+            # Only validate action tables: must contain No + Action + Description
+            lowered = [h.lower() for h in header]
+            if "no" not in lowered and "no." not in lowered:
+                i += 1
+                continue
+            if "action" not in lowered or "description" not in lowered:
+                i += 1
+                continue
+
+            try:
+                no_index = lowered.index("no")
+            except ValueError:
+                no_index = lowered.index("no.")
+
+            # Consume rows until table ends
+            j = i + 2
+            while j < len(lines) and TABLE_ROW_RE.match(lines[j]):
+                row = _split_table_row(lines[j])
+                if len(row) <= no_index:
+                    j += 1
+                    continue
+                raw_no = row[no_index]
+                try:
+                    number = int(raw_no)
+                except ValueError:
+                    j += 1
+                    continue
+                occurrences.append(
+                    Occurrence(
+                        number=number,
+                        section=current_section or "(unknown section)",
+                        line_no=j + 1,  # 1-based
+                        raw_line=lines[j],
+                    )
+                )
+                j += 1
+
+            i = j
+            continue
+
+        i += 1
+
+    return occurrences
+
+
+def main() -> int:
+    parser = argparse.ArgumentParser(description="Validate numbering in flow-action spec tables.")
+    parser.add_argument("--spec", required=True, help="Path to *_FLOW_ACTION_SPEC.md")
+    parser.add_argument(
+        "--en-check",
+        action="store_true",
+        help="Enable EN artifact hygiene checks (Vietnamese leftovers + encoding corruption).",
+    )
+    args = parser.parse_args()
+
+    try:
+        sys.stdout.reconfigure(encoding="utf-8")  # type: ignore[attr-defined]
+        sys.stderr.reconfigure(encoding="utf-8")  # type: ignore[attr-defined]
+    except Exception:
+        pass
+
+    spec_path = Path(args.spec)
+    if not spec_path.exists():
+        print(f"[ERROR] Not found: {spec_path}", file=sys.stderr)
+        return 2
+
+    text = _read_text(spec_path)
+    occ = parse_action_tables(text)
+    if not occ:
+        print("[WARN] No action tables detected (| No | Action | Description | ... |).")
+        return 0
+
+    by_number: dict[int, list[Occurrence]] = {}
+    for o in occ:
+        by_number.setdefault(o.number, []).append(o)
+
+    duplicates = {n: os for n, os in by_number.items() if len(os) > 1}
+
+    max_seen = -1
+    decreases: list[tuple[Occurrence, int]] = []
+    for o in occ:
+        if o.number < max_seen:
+            decreases.append((o, max_seen))
+        max_seen = max(max_seen, o.number)
+
+    has_issues = bool(duplicates or decreases)
+    hygiene = collect_hygiene_issues(text, en_check=args.en_check)
+    if hygiene["encoding"] or hygiene["inline_heading"]:
+        has_issues = True
+    if args.en_check and (hygiene["vi_diacritic"] or hygiene["vi_phrase"]):
+        has_issues = True
+
+    print(f"Checked: {spec_path}")
+    print(f"- Total numbered rows: {len(occ)}")
+    print(f"- Unique numbers: {len(by_number)}")
+    print(f"- EN hygiene check enabled: {args.en_check}")
+
+    if duplicates:
+        print("\n[FAIL] Duplicate numbers found:")
+        for n in sorted(duplicates.keys()):
+            print(f"- No {n}:")
+            for o in duplicates[n]:
+                print(f"  - {o.section} (line {o.line_no})")
+
+    if decreases:
+        print("\n[FAIL] Numbering decreases/resets detected (global numbering policy):")
+        for o, prev_max in decreases:
+            print(f"- No {o.number} after max {prev_max}: {o.section} (line {o.line_no})")
+
+    if hygiene["encoding"]:
+        print("\n[FAIL] Encoding/mojibake markers detected:")
+        for line_no, line in hygiene["encoding"][:20]:
+            print(f"- line {line_no}: {line.strip()}")
+
+    if hygiene["inline_heading"]:
+        print("\n[FAIL] Inline/merged heading patterns detected:")
+        for line_no, line in hygiene["inline_heading"][:20]:
+            print(f"- line {line_no}: {line.strip()}")
+
+    if args.en_check and hygiene["vi_diacritic"]:
+        print("\n[FAIL] Vietnamese diacritic text detected in EN artifact:")
+        for line_no, line in hygiene["vi_diacritic"][:20]:
+            print(f"- line {line_no}: {line.strip()}")
+
+    if args.en_check and hygiene["vi_phrase"]:
+        print("\n[FAIL] Vietnamese phrase patterns detected in EN artifact:")
+        for line_no, line in hygiene["vi_phrase"][:20]:
+            print(f"- line {line_no}: {line.strip()}")
+
+    if not has_issues:
+        print("\n[OK] Numbering and text hygiene checks passed.")
+        return 0
+
+    print("\nHint: Use global numbering across the full document.")
+    print("- Do not repeat No values in any action table.")
+    print("- Do not reset numbering per screen/dialog.")
+    print("- Keep EN artifacts free of Vietnamese leftovers and encoding corruption.")
+    print("- Keep headings/sections on separate lines (avoid merged markdown blocks).")
+    return 1
+
+
+if __name__ == "__main__":
+    raise SystemExit(main())

package/assets/toolkit/toolkit/skills/sdtk-test-case-spec/SKILL.md

@@ -0,0 +1,65 @@
+---
+name: sdtk-test-case-spec
+description: Generate screen-based QA test-case markdown (`[FEATURE_KEY]_TEST_CASE.md`) in Excel-aligned layout (Statistic + per-screen UTC/ITC worksheets). Use when QA needs reusable test design artifacts before or during execution.
+---
+
+# SDTK Test Case Spec
+
+## Outputs
+- `docs/qa/[FEATURE_KEY]_TEST_CASE.md`
+- Optional project variant:
+  - `docs/en/qa/[FEATURE_KEY]_TEST_CASE.md` (only when project uses `docs/en/**`)
+
+## Required Inputs
+- Feature key (`FEATURE_KEY`)
+- Screen/flow references:
+  - `docs/specs/[FEATURE_KEY]_FLOW_ACTION_SPEC.md`
+  - `docs/specs/BA_SPEC_[FEATURE_KEY].md`
+- API reference:
+  - `docs/api/[FEATURE_KEY]_ENDPOINTS.md`
+  - (optional) `docs/api/[FeaturePascal]_API.yaml`
+- Clarification source:
+  - `docs/specs/Q&A.md` or `docs/en/specs/Q&A.md` when available
+
+## Core Rules
+1. Apply `./references/TEST_CASE_CREATION_RULES.md` first.
+2. Keep section order fixed (Statistic -> Abbreviations -> Scope -> References -> Environment -> Coverage -> Screen-based UTC/ITC -> OQ -> STC/UAT note).
+3. Use screen-first layout to mirror worksheet structure:
+   - each screen can have `[SCREEN_KEY]_UTC` and `[SCREEN_KEY]_ITC`.
+4. Keep one unified 18-column schema for UTC/ITC rows.
+5. Keep stable case IDs; do not renumber only because of regrouping.
+6. Track unresolved decisions via `OQ-xx`.
+
+## Procedure
+1. Resolve output path:
+   - default `docs/qa/[FEATURE_KEY]_TEST_CASE.md`
+   - use `docs/en/qa/...` only if the repo already follows `docs/en/**`.
+2. Build/refresh `Statistic Summary (Excel-aligned)`:
+   - include UT total, IT total, grand total.
+3. Build `Feature Coverage Matrix` from flow/action and API docs.
+4. Split UTC/ITC by screen sections (`screen-first`), not by test type only.
+5. Fill conflict/permission/error-path cases from Q&A decisions and API contracts.
+6. Record unresolved items in section `Open Questions`.
+7. Validate:
+   - counts in summary match table rows
+   - no placeholder tokens like `??`
+   - out-of-scope boundaries are explicit
+
+## Role Integration
+- Primary owner: `/qa`.
+- `/qa` uses this skill to design/update reusable test-case specs.
+- `sdtk-qa` still owns release decision artifact (`QA_RELEASE_REPORT_*`).
+
+## Notes
+- This skill is for test-case specification artifacts, not test execution logs.
+- For release approval and defect triage, continue with `sdtk-qa`.
+
+## Validator Script
+- `scripts/validate_test_case_spec.py`
+
+### Typical command
+```powershell
+python "toolkit/skills/sdtk-test-case-spec/scripts/validate_test_case_spec.py" `
+  --file "docs/qa/[FEATURE_KEY]_TEST_CASE.md"
+```
+

package/assets/toolkit/toolkit/skills/sdtk-test-case-spec/references/TEST_CASE_CREATION_RULES.md

@@ -0,0 +1,129 @@
+# TEST CASE CREATION RULES
+## Canonical rules for `[FEATURE_KEY]_TEST_CASE.md`
+
+**Version:** 1.0.0  
+**Last Updated:** 2026-02-25
+
+---
+
+## 1. Purpose
+- Standardize creation of feature test-case documents in markdown.
+- Mirror Excel-style worksheet layout while keeping output reviewable in `.md`.
+- Keep QA artifacts reusable across projects, domains, and screen sets.
+
+---
+
+## 2. Output Contract
+- Primary output file:
+  - `docs/qa/[FEATURE_KEY]_TEST_CASE.md`
+- Optional language/project variant:
+  - `docs/en/qa/[FEATURE_KEY]_TEST_CASE.md` (only when the project explicitly uses `docs/en/**`).
+
+---
+
+## 3. Required Section Order
+1. Document metadata (`Document ID`, `Version`, `Date`, `Author`, `Status`)
+2. `Statistic Summary (Excel-aligned)`
+3. `Abbreviations`
+4. `1. Scope`
+5. `2. References`
+6. `3. Test Environment and Common Data`
+7. `4. Feature Coverage Matrix`
+8. `5. Screen-based Test Cases (Excel-aligned)`
+9. `6. Open Questions (for final freeze)`
+10. `7. STC/UAT Note`
+
+Do not reorder these sections unless the user requests a different contract.
+
+---
+
+## 4. Statistic Summary Rules
+- Place summary near the top of the file (before detailed UTC/ITC tables).
+- Keep columns aligned with Excel `Statistic` logic:
+  - `Num of Case`
+  - `OK`
+  - `NG`
+  - `Not tested yet`
+  - `Done (%)`
+  - `Updated Date`
+- Compute:
+  - `Not tested yet = Num of Case - (OK + NG)`
+  - `Done (%) = (OK + NG) / Num of Case`
+- Include subtotal rows at minimum:
+  - `Unit Test Total`
+  - `Integration Test Total`
+  - `Grand Total`
+
+---
+
+## 5. Screen-Based Worksheet Mapping
+- Use `screen-first` layout to mirror Excel tabs:
+  - Each functional screen has a section (for example `SCH01`, `SCH02`, ...).
+  - Each screen section can have 2 sub-sections:
+    - `UTC` worksheet (name pattern: `[SCREEN_KEY]_UTC`)
+    - `ITC` worksheet (name pattern: `[SCREEN_KEY]_ITC`)
+- Add one mapping table in section 5 that lists:
+  - `Screen`
+  - `Suggested Worksheet Pair`
+  - `UTC Cases`
+  - `ITC Cases`
+  - `Notes`
+
+---
+
+## 6. Test Case Table Schema (Mandatory)
+- Use one unified 18-column schema for both UTC and ITC tables:
+  - `No`
+  - `Test Type`
+  - `Test Perspective`
+  - `Test Item`
+  - `Precondition`
+  - `Test Steps`
+  - `Expected Result`
+  - `Browser`
+  - `Test Execution Result`
+  - `Remarks`
+  - `Reviewer`
+  - `Review Date`
+  - `OK/NG`
+  - `Cause`
+  - `Countermeasure`
+  - `Owner`
+  - `Completion Date`
+  - `Confirmation`
+
+---
+
+## 7. Numbering and ID Policy
+- Keep a stable `No` for each test case row.
+- Do not renumber existing case IDs only for visual regrouping.
+- In screen-split sections, case IDs may be non-contiguous; this is allowed.
+- Summary/helper tables must keep contiguous `No` (`1,2,3,...`).
+
+---
+
+## 8. Open Questions (OQ) Policy
+- If expected behavior is unclear, create `OQ-xx` entries in section 6.
+- Do not assume behavior for:
+  - permission matrix
+  - conflict dialog/action semantics
+  - API contract ambiguities
+  - release-scope boundary
+- When resolved, keep question row and set status/remarks to resolved.
+
+---
+
+## 9. Language and Encoding
+- Default artifact language: English.
+- Keep original JP/VI labels when required by business/UI traceability (for example button labels).
+- File encoding must be UTF-8.
+- Avoid mojibake and placeholder tokens (`??`, `?????`) in final output.
+
+---
+
+## 10. Final Validation Checklist
+- Required sections exist and are in correct order.
+- Statistic totals equal sum of UTC/ITC rows.
+- Screen mapping table case counts match actual UTC/ITC table counts.
+- Scope explicitly states in-scope and out-of-scope boundaries.
+- Open questions are explicitly tracked with owner and impact.

package/assets/toolkit/toolkit/skills/sdtk-test-case-spec/scripts/validate_test_case_spec.py

@@ -0,0 +1,97 @@
+#!/usr/bin/env python3
+"""
+Lightweight validator for [FEATURE_KEY]_TEST_CASE.md
+"""
+
+from __future__ import annotations
+
+import argparse
+import re
+import sys
+from pathlib import Path
+
+
+REQUIRED_HEADINGS = [
+    "## Statistic Summary (Excel-aligned)",
+    "## Abbreviations",
+    "## 1. Scope",
+    "## 2. References",
+    "## 3. Test Environment and Common Data",
+    "## 4. Feature Coverage Matrix",
+    "## 5. Screen-based Test Cases (Excel-aligned)",
+    "## 6. Open Questions (for final freeze)",
+    "## 7. STC/UAT Note",
+]
+
+
+def has_18_columns(header_line: str) -> bool:
+    # markdown table columns = number of "|" minus 1 (leading/trailing)
+    pipe_count = header_line.count("|")
+    return pipe_count - 1 == 18
+
+
+def main() -> int:
+    parser = argparse.ArgumentParser(
+        description="Validate QA test-case markdown format for SDTK."
+    )
+    parser.add_argument(
+        "--file",
+        required=True,
+        help="Path to [FEATURE_KEY]_TEST_CASE.md",
+    )
+    args = parser.parse_args()
+
+    path = Path(args.file)
+    if not path.exists():
+        print(f"[FAIL] file not found: {path}")
+        return 1
+
+    text = path.read_text(encoding="utf-8")
+    lines = text.splitlines()
+    ok = True
+
+    for heading in REQUIRED_HEADINGS:
+        if heading not in text:
+            print(f"[FAIL] missing heading: {heading}")
+            ok = False
+
+    utc_sections = re.findall(r"^#### .*UTC.*$", text, flags=re.MULTILINE)
+    itc_sections = re.findall(r"^#### .*ITC.*$", text, flags=re.MULTILINE)
+    if not utc_sections:
+        print("[FAIL] no UTC subsection found")
+        ok = False
+    if not itc_sections:
+        print("[FAIL] no ITC subsection found")
+        ok = False
+
+    # Validate 18-column table headers for UTC/ITC tables
+    expected_header = (
+        "| No | Test Type | Test Perspective | Test Item | Precondition | Test Steps | "
+        "Expected Result | Browser | Test Execution Result | Remarks | Reviewer | "
+        "Review Date | OK/NG | Cause | Countermeasure | Owner | Completion Date | Confirmation |"
+    )
+    if expected_header not in text:
+        print("[FAIL] missing canonical 18-column test-case table header")
+        ok = False
+    else:
+        if not has_18_columns(expected_header):
+            print("[FAIL] canonical test-case table header column count is not 18")
+            ok = False
+
+    placeholders = []
+    for i, line in enumerate(lines, start=1):
+        if "??" in line or "?????" in line:
+            placeholders.append(i)
+    if placeholders:
+        print(f"[FAIL] unresolved placeholder tokens at lines: {placeholders[:20]}")
+        ok = False
+
+    if ok:
+        print("[PASS] test-case markdown format validation passed")
+        return 0
+
+    return 1
+
+
+if __name__ == "__main__":
+    sys.exit(main())

package/assets/toolkit/toolkit/templates/QUALITY_CHECKLIST.md

@@ -0,0 +1,124 @@
+# QUALITY CHECKLIST
+## Multi-Agent Development Pipeline Gates (v2.1 - 6 Phases)
+
+**Current Feature:** `{{FEATURE_KEY}}`
+**Last Updated:** `{{DATETIME}}`
+**Overall Status:** `PHASE 1 PM INITIATION - IN PROGRESS`
+
+---
+
+## PHASE 1: PM INITIATION (Project Kickoff) CHECKLIST
+
+| # | Criteria | Status | Verified By | Notes/Issues | Date |
+|---|----------|--------|-------------|--------------|------|
+| 1 | Requirements received from stakeholders | [ ] Pending | PM Agent | | |
+| 2 | Business problem clearly defined | [ ] Pending | PM Agent | | |
+| 3 | High-level scope documented (REQ-xx) + Open Questions (OQ-xx) captured | [ ] Pending | PM Agent | | |
+| 4 | Stakeholders identified | [ ] Pending | PM Agent | | |
+| 5 | Success criteria defined (measurable) | [ ] Pending | PM Agent | | |
+| **6** | **PM INITIATION COMPLETE** | [ ] Pending | **PM Gate** | **@ba please analyze** | |
+
+---
+
+## PHASE 2: BA (Business Analysis) CHECKLIST
+
+| # | Criteria | Status | Verified By | Notes/Issues | Date |
+|---|----------|--------|-------------|--------------|------|
+| 1 | Domain Glossary complete (all terms defined) | [ ] Pending | BA Agent | | |
+| 2 | Business Rules numbered and complete (BR-01...) | [ ] Pending | BA Agent | | |
+| 3 | Use Cases cover 100% requirements (UC-01...) | [ ] Pending | BA Agent | | |
+| 4 | Data entities and relationships documented | [ ] Pending | BA Agent | | |
+| 5 | Risks identified + prioritized (High/Med/Low) | [ ] Pending | BA Agent | | |
+| 6 | Open questions listed for PM/ARCH review | [ ] Pending | BA Agent | | |
+| **7** | **BA SPECS READY** | [ ] Pending | **BA Gate** | **@pm specs ready for PRD** | |
+
+---
+
+## PHASE 2+: PM PLANNING CHECKLIST
+
+| # | Criteria | Status | Verified By | Notes/Issues | Date |
+|---|----------|--------|-------------|--------------|------|
+| 1 | PRD aligns 100% with BA specs + resolves/records OQ decisions | [ ] Pending | PM Agent | | |
+| 2 | Backlog 100% traceable to BA Use Cases | [ ] Pending | PM Agent | | |
+| 3 | Story priorities reflect business value (MoSCoW) | [ ] Pending | PM Agent | | |
+| 4 | Success metrics defined and measurable | [ ] Pending | PM Agent | | |
+| 5 | Release criteria objective and testable | [ ] Pending | PM Agent | | |
+| 6 | Effort estimates realistic (Fibonacci story points) | [ ] Pending | PM Agent | | |
+| **7** | **PM BACKLOG READY** | [ ] Pending | **PM Gate** | **@arch please design** | |
+
+---
+
+## PHASE 3: ARCH (Solution Architecture) CHECKLIST
+
+| # | Criteria | Status | Verified By | Notes/Issues | Date |
+|---|----------|--------|-------------|--------------|------|
+| 1 | System architecture diagram complete + assumptions/OQ documented | [ ] Pending | ARCH Agent | | |
+| 2 | Database schema/ERD documented + `DATABASE_SPEC_[FEATURE_KEY].md` updated (if DB scope) | [ ] Pending | ARCH Agent | | |
+| 3 | API contract docs updated (`[FeaturePascal]_API.yaml` + `[FEATURE_KEY]_ENDPOINTS.md`) when API scope exists | [ ] Pending | ARCH Agent | | |
+| 4 | API flow list updated (`[feature_snake]_api_flow_list.txt`) when API scope exists | [ ] Pending | ARCH Agent | | |
+| 5 | API design detail updated (`[FEATURE_KEY]_API_DESIGN_DETAIL.md`) when API scope exists and mode is `auto/on` | [ ] Pending | ARCH Agent | Mode from `sdtk.config.json` | |
+| 6 | Flow-action screen spec updated (`[FEATURE_KEY]_FLOW_ACTION_SPEC.md`) when UI flow scope exists | [ ] Pending | ARCH Agent | | |
+| 7 | Screen layout spec updated (`DESIGN_LAYOUT_[FEATURE_KEY].md`) when UI scope exists | [ ] Pending | ARCH Agent | | |
+| 8 | Security model documented (auth/authz) | [ ] Pending | ARCH Agent | | |
+| **9** | **ARCH DESIGN READY** | [ ] Pending | **ARCH Gate** | **@dev please implement** | |
+
+---
+
+## PHASE 4: DEV (Development + Code Review) CHECKLIST
+
+| # | Criteria | Status | Verified By | Notes/Issues | Date |
+|---|----------|--------|-------------|--------------|------|
+| 1 | Implementation Plan matches ARCH design + backlog + clarifications logged (OQ-xx) | [ ] Pending | Dev Agent | | |
+| 2 | **Unit Tests:** coverage meets target | [ ] Pending | Dev Agent | | |
+| 3 | **Integration Tests:** critical flows covered | [ ] Pending | Dev Agent | | |
+| 4 | Linting and type checking passes | [ ] Pending | Dev Agent | | |
+| 5 | Security checklist passed | [ ] Pending | Dev Agent | | |
+| 6 | Performance baseline established | [ ] Pending | Dev Agent | | |
+| 7 | **Self Code Review:** Complete | [ ] Pending | Dev Agent | MANDATORY | |
+| 8 | **Peer Code Review:** Requested | [ ] Pending | Dev Agent | MANDATORY (AI allowed) | |
+| 9 | **Peer Code Review:** All comments addressed | [ ] Pending | Dev Agent | MANDATORY (AI allowed) | |
+| **10** | **DEV READY FOR QA** | [ ] Pending | **Dev Gate** | **Code Review [x] then @qa please test** | |
+
+---
+
+## PHASE 5: QA (Quality Assurance) CHECKLIST
+
+| # | Criteria | Status | Verified By | Notes/Issues | Date |
+|---|----------|--------|-------------|--------------|------|
+| 1 | Test Strategy covers all critical paths + unclear ACs escalated (OQ-xx) | [ ] Pending | QA Agent | | |
+| 2 | Test case specification updated (`[FEATURE_KEY]_TEST_CASE.md`) when detailed test design scope exists | [ ] Pending | QA Agent | | |
+| 3 | **E2E Tests:** P1 flows PASS (if applicable) | [ ] Pending | QA Agent | | |
+| 4 | **Defects:** 0 HIGH severity open | [ ] Pending | QA Agent | | |
+| 5 | **Coverage:** meets targets | [ ] Pending | QA Agent | | |
+| 6 | No flaky tests | [ ] Pending | QA Agent | | |
+| 7 | Performance meets NFRs | [ ] Pending | QA Agent | | |
+| 8 | Release Report complete | [ ] Pending | QA Agent | | |
+| **9** | **QA APPROVED** | [ ] Pending | **QA Gate** | **@pm please finalize release status** | |
+
+---
+
+## PHASE 6: PM CLOSURE CHECKLIST
+
+| # | Criteria | Status | Verified By | Notes/Issues | Date |
+|---|----------|--------|-------------|--------------|------|
+| 1 | QA release decision reviewed by PM | [ ] Pending | PM Agent | | |
+| 2 | Outstanding OQ/risks triaged for next iteration | [ ] Pending | PM Agent | | |
+| 3 | Final status communicated to stakeholders | [ ] Pending | PM Agent | | |
+| **4** | **FEATURE CLOSURE COMPLETE** | [ ] Pending | **PM Gate** | **Pipeline complete** | |
+
+---
+
+## FINAL RELEASE GATE
+
+```
+RELEASE STATUS: IN PROGRESS
+
+FINAL VERIFICATION:
+[ ] Phase 1 PM Init
+[ ] Phase 2 BA
+[ ] Phase 2+ PM Plan
+[ ] Phase 3 ARCH
+[ ] Phase 4 DEV + Code Review
+[ ] Phase 5 QA
+[ ] Phase 6 PM Closure
+```