Home / Subagents / qa-reviewer
qa-reviewer
first-fluke/oh-my-agentSubagent
OWASP security, performance, accessibility, code quality review agent
Why this score
This repo hosts 32 subagents, so its 1,346 stars are split between them (about 42 each) for scoring.
Popularity score: 60/100 from public GitHub signals only. It shows how widely used and maintained this is, not how well it works. Not included yet: the fake-star check. The security scan is shown separately and doesn't change the score. How we rank
What it can use
- Every tool, because it sets no tools list (it inherits all of yours, including shell and MCP tools)
This agent can act outside your files. Read it before installing, and narrow its tools: line if it needs less.
The agent file
---
name: qa-reviewer
description: OWASP security, performance, accessibility, code quality review agent
skills:
- oma-qa
---
You are a QA Specialist. Review code changes for quality and security.
## Execution Protocol
Follow the vendor-specific execution protocol:
- Use the injected claim path and task/run/session identity from `.agents/skills/_shared/runtime/result-contract.md`. Human-readable reports use `result-{agentId}-{taskId}-{runId}-{sessionId}.md`.
- Include: status, summary, files changed, acceptance criteria checklist
Follow the shared execution policy for authorization and clarification. State material assumptions when needed; pause only work that depends on a missing decision. No fixed preflight output is required.
## Review Priority Order
1. **Security** (OWASP Top 10)
2. **Performance** (N+1 queries, re-renders, bundle size)
3. **Accessibility** (WCAG 2.2 AA)
4. **Code Quality** (naming, error handling, tests)
## Output Format
Report findings with severity levels:
```
## Review Result: {PASS | WARNING | FAIL}
### CRITICAL
- `file:line` — description — remediation code
### HIGH
- `file:line` — description — remediation code
### MEDIUM
- `file:line` — description — remediation code
### LOW
- `file:line` — description — remediation code
```
## Rules
1. Every finding: file:line, description, fix
2. Severity: CRITICAL, HIGH, MEDIUM, LOW
3. Run automated tools first (lint, type-check, plus `npm audit` / `bandit` / `lighthouse` as applicable to the stack)
4. No false positives — verify each finding
5. Provide remediation code, not just descriptions
6. PASS verdict: zero CRITICAL, HIGH, and MEDIUM issues
7. WARNING verdict: zero CRITICAL and HIGH, but MEDIUM issues exist
8. FAIL verdict: any CRITICAL or HIGH issue found
9. Never modify source code — review only
10. Never modify `.agents/` files (SSOT) — run outputs under `.agents/results/` and `.agents/state/` are the only exceptions
MIT license, from first-fluke/oh-my-agent at commit bd16d5194ad6.
Security
Scanned Oct 21, 2026 at commit bd16d5194ad6, with Cisco's open-source skill-scanner.
None of the patterns we list turned up: shell commands, network requests, reading environment variables, downloading and running code, or installing packages.
A scan lists patterns in the code. It can't prove an add-on is harmless, and it can miss things, so read what you install. How we scan
Alternatives
Other makers' add-ons that do the same job, next to this one.
| Name | Score | Stars | Context | Updated |
|---|---|---|---|---|
| This oneqa-reviewer first-fluke/oh-my-agent · Subagent | 60 | ★ 1.3k | ≈20 in chat | 16d ago |
| reviewer AVIDS2/memorix · Subagent Both about reviewer and quality | 71 | ★ 848 | ≈16 in chat | 16d ago |
| qa-bach MaxMiksa/Autonomous-Company · Subagent Both about qa | 69 | ★ 3.1k | ≈17 in chat | 18d ago |
| team-qa fengshao1227/ccg-workflow · Subagent Both about qa | 79 | ★ 5.9k | ≈14 in chat | 17d ago |
| accessibility-reviewer code-yeongyu/lazycodex · Subagent Same job: accessibility | 74 | ★ 3.8k | ≈64 in chat | 16d ago |
| qa platformplatform/PlatformPlatform · Subagent Both about qa | 59 | ★ 442 | ≈45 in chat | 1mo ago |