popup-studio-ai/bkit-claude-code/qa-phase

QA Phase Skill

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

Arguments

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 bash ${PLUGIN_ROOT}/scripts/qa/pre-release-check.sh via Bash.

The path must be absolute: the script ships inside the plugin, not in the user's project, so a relative scripts/qa/... resolves to nothing wherever this skill actually runs. The script scans $CLAUDE_PROJECT_DIR (falling back to the working directory) — pass --root DIR to point it elsewhere.

1. Parse the output for CRITICAL / WARNING / INFO counts
2. 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.).

1. 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

QA Report Integration

When scanner results are available, include them in the QA report:

## Pre-Release Scan Results
 
-**Scanner**: dead-code — 0 CRITICAL, 1 WARNING, 0 INFO
-**Scanner**: config-audit — 0 CRITICAL, 0 WARNING, 2 INFO
-**Scanner**: completeness — 0 CRITICAL, 0 WARNING, 1 INFO
-**Scanner**: shell-escape — 0 CRITICAL, 0 WARNING, 0 INFO
-**Scanner**: wiring — 0 CRITICAL, 1 WARNING, 0 INFO
 
**Overall**: PASS (0 CRITICAL issues)

Test Levels

Browser QA Depth (L3-L4)

qa-lead adds three things on top of the scripted L3-L4 scenarios. The full procedure is in agents/qa-lead.md:

• Diff-aware scope — changed files are mapped to the pages and endpoints

they serve, and those pages are tested first.

• Exploratory pass — each priority page is checked for forms, empty/error

states, navigation, and console output after every action.

• Health Score — a weighted 0-100 summary of the exploratory findings,

reported next to the gate metrics. It is informational; the QA gate (qaPassRate, qaCriticalCount, runtimeErrorCount) still decides the verdict.

On QA_FAIL, issues are handed to Act in fix order with reproduction steps. Act fixes one issue at a time, re-verifies it, and adds a regression test. It stops to ask when fixes start spreading to unrelated files or breaking passing tests.

Fallback

Chrome MCP unavailable:

• L1 + L2 only
• QA report notes "L3-L5 skipped"
• QA pass/fail based on L1+L2 results only