skills/qa/SKILL.md
Performs structured QA review of a feature, PR, or implementation: verifies acceptance criteria coverage, generates edge case test cases, and flags gaps before shipping. Use when asked to "QA this", "review for quality", "write QA test cases", "find edge cases", or before marking a feature done. Ported from https://github.com/mattpocock/skills (Matt Pocock).
npx skillsauth add michaelalber/ai-toolkit qaInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
Perform structured QA review.
1. Load acceptance criteria
From the PRD, issue, or conversation. If none exist:
"I need acceptance criteria before reviewing. Please provide them or run to-prd first."
2. Verify happy path — for each AC:
3. Generate edge case test matrix — see references/edge-case-taxonomy.md:
| Scenario | Input | Expected behavior | Priority | |----------|-------|------------------|----------| | Empty / null input | [describe] | [expected] | High | | Boundary values | [describe] | [expected] | High | | Invalid format | [describe] | [expected] | High | | Concurrent access | [describe] | [expected] | Medium | | Large volume | [describe] | [expected] | Medium | | Dependency failure | [describe] | [expected] | Medium |
4. Flag gaps:
5. Report:
## QA Report: [Feature / PR Name]
**AC coverage:** [N/N met] | [N/N tested]
**Gaps:**
- [ ] [gap + recommended fix or decision needed]
**Edge cases to add:**
- [ ] [test case description]
**Ready to ship:** Yes / No — [reason if No]
development
Federal / government security overlay applied ON TOP OF a base language security review (dotnet/python/php/rust/react). Language-agnostic: adds NIST SP 800-53 control mapping, FIPS 140-2/3 cryptographic compliance (with a per-language crypto table), CUI handling, EO 14028 supply-chain requirements, and DOE Order 205.1B, and emits POA&M-ready findings with FIPS 199 impact levels. Use for federal/DOE/DOD/national-laboratory systems. Triggers on "federal security review", "NIST compliance", "NIST 800-53", "FISMA", "CUI", "FIPS audit", "DOE security", "POA&M", "ATO review". Do NOT use alone — run the matching <lang>-security-review FIRST; this overlay maps and extends it.
tools
OWASP-based security review of React / TypeScript front-end applications. Detects the framework (Vite/CRA/Next), entry points, and data flows, scans against the OWASP Top 10 (2025) mapped to React client-side patterns (XSS via raw HTML, URL/protocol injection, secrets in the bundle, insecure token storage, dependency CVEs, missing CSP, open redirects), and produces a manager-friendly executive summary plus a graded technical findings table. Use to audit React code for vulnerabilities. Triggers on "react security review", "frontend security audit", "audit react for vulnerabilities", "owasp react", "react xss", "react security posture", "npm audit review". For federal / gov / DOE / NIST / FIPS / CUI context, run security-review-federal after this base review. Do NOT use to grade architecture/structure — use react-architecture-checklist.
tools
Analyzes legacy React codebases and produces actionable modernization plans. Primary migration paths include class components to function components + hooks, Create React App to Vite, React 16/17 to 18 to 19, JavaScript to TypeScript, Enzyme to React Testing Library, legacy Redux to Redux Toolkit / Zustand / Context, and deprecated lifecycle/API removal. Does NOT perform the migration — assesses, quantifies risk, and plans. Triggers on phrases like "modernize react", "class to hooks", "upgrade react", "migrate CRA to vite", "react legacy migration", "react 17 to 18", "react js to typescript", "react technical debt", "enzyme to RTL".
development
Scaffolds feature-based React / TypeScript architecture using feature folders, presentational + container components, custom hooks, a typed data layer, and structural CQRS (query hooks vs mutation hooks). React analog of dotnet-vertical-slice and python-feature-slice — no DI framework; uses props/context for dependency injection and a query cache for server state. Use when creating feature-based React projects, adding React features, organizing components by feature rather than by technical type, or scaffolding a feature's data layer. Triggers on phrases like "scaffold react feature", "create react slice", "react feature folder", "react vertical slice", "add react feature", "react feature architecture", "organize react by feature".