| name | qa |
| description | Review test coverage against spec for Formula Boss. Usage: /qa <pr-number-or-issue> |
| allowed-tools | Bash, Read, Glob, Grep |
QA — Formula Boss
Assess test coverage and quality for a PR or issue.
Instructions
- Parse the argument — PR number or issue number
- Read the linked issue and find the spec (if one exists)
- Read the PR diff:
gh pr diff <number> -R TagloGit/taglo-formula-boss
- Find test files related to the changed code (check
formula-boss.Tests/)
- Assess:
- Do tests exist for the new/changed code?
- Do tests cover the spec's acceptance criteria?
- Are edge cases tested (empty input, boundary values, errors)?
- Are tests well-written (clear names, good assertions)?
Output Format
QA Report: #N — [title]
Spec Coverage: [X of Y acceptance criteria have tests]
Edge Cases: [Covered / Gaps identified]
Test Quality: [observations]
Gaps:
- [specific gap]
Recommendation: [Pass / Needs more tests / Needs discussion]
Repo-Specific Testing
- Framework: xUnit (.NET 6)
- Test project:
formula-boss.Tests/
- Run:
dotnet test formula-boss/formula-boss.slnx
- Key areas to check: DSL parsing edge cases, ExcelDNA reflection patterns, COM lifecycle
Behaviour Notes
- Do NOT write code or merge PRs
- Flag missing tests for error paths and boundary conditions
Self-Improvement
When you notice a recurring problem or workflow gap:
- Create a
process issue on TagloGit/taglo-pm describing the observation
- Continue your current work
