qa-phase

name	qa-phase
classification	workflow
classification-reason	QA phase automation within PDCA cycle
deprecation-risk	none
effort	high
description	QA Phase execution — L1-L5 test planning, generation, execution, and reporting for a single feature. For sprint-level QA (7-Layer dataFlowIntegrity / S1 gate across multiple features) use /sprint qa <sprintId> which delegates to sprint-qa-flow agent (v2.1.13). Triggers: qa phase, QA test, qa run
argument-hint	[feature]
user-invocable	true
agents	{"lead":"bkit:qa-lead","planner":"bkit:qa-test-planner","generator":"bkit:qa-test-generator","debug":"bkit:qa-debug-analyst","monitor":"bkit:qa-monitor","default":"bkit:qa-lead"}
allowed-tools	["Read","Write","Edit","Glob","Grep","Bash","Task","AskUserQuestion"]
imports	["${PLUGIN_ROOT}/templates/qa-report.template.md","${PLUGIN_ROOT}/templates/qa-test-plan.template.md"]
next-skill	pdca
pdca-phase	qa
task-template	[QA] {feature}

QA Phase Skill

Execute QA phase of the PDCA cycle. Automatically runs L1-L5 tests with Chrome MCP integration.

Arguments

Argument	Description	Example
[feature]	Target feature to test	/qa-phase user-auth

Workflow

1. Context: Read design doc and Check phase analysis
2. Plan: Generate test plan (L1-L5 items with priorities)
3. Generate: Create test code files
4. Execute: Run L1-L5 tests (L3-L5 require Chrome MCP)
5. Report: Generate QA report to docs/05-qa/{feature}.qa-report.md

PRE-SCAN: Pre-Release Quality Check

Before running L1 tests, execute the automated quality scanners to catch structural issues early.

Steps

1. Run scripts/qa/pre-release-check.sh via Bash
2. Parse the output for CRITICAL / WARNING / INFO counts
3. If CRITICAL issues found:
   • Report all CRITICAL issues with file paths and suggested fixes
   • Recommend fixing CRITICAL issues before proceeding with L1-L5 tests
   • Use AskUserQuestion to ask whether to continue or abort the QA phase (e.g. options: "Fix CRITICAL first" / "Continue anyway" / "Abort QA"). This gate is issued directly here, in the main session context — qa-phase is deliberately not context: fork. AskUserQuestion is stripped at the fork sub-agent boundary (CC #34592 / #54892), so it must run in the main context and must not be delegated to a sub-agent (qa-lead, etc.).
4. If only WARNING/INFO issues (no CRITICAL):
   • Include scanner results in the QA report under "Pre-Release Scan" section
   • Continue to L1 test planning

Scanner Coverage

Scanner	Detects	Severity
dead-code	Stale require/import, unused exports	CRITICAL / WARNING
config-audit	Unreferenced config keys, hardcoded values, missing paths	CRITICAL / WARNING / INFO