Home / Subagents / qa-reviewer

qa-reviewer

first-fluke/oh-my-agentSubagent

OWASP security, performance, accessibility, code quality review agent

ScannedSubagentSource: GitHub
1,346 stars +7 this week Updated 16d ago License MIT Context ≈20 tokens in your chat
60popular

Why this score

Stars11/35
Usage (forks)3/15
Upkeep25/25
License15/15
Owner history6/10

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.

NameScoreStarsContextUpdated
This oneqa-reviewer first-fluke/oh-my-agent · Subagent 60★ 1.3k≈20 in chat16d ago
reviewer AVIDS2/memorix · Subagent Both about reviewer and quality 71★ 848≈16 in chat16d ago
qa-bach MaxMiksa/Autonomous-Company · Subagent Both about qa 69★ 3.1k≈17 in chat18d ago
team-qa fengshao1227/ccg-workflow · Subagent Both about qa 79★ 5.9k≈14 in chat17d ago
accessibility-reviewer code-yeongyu/lazycodex · Subagent Same job: accessibility 74★ 3.8k≈64 in chat16d ago
qa platformplatform/PlatformPlatform · Subagent Both about qa 59★ 442≈45 in chat1mo ago