reviewer
swingerman/engineerSubagent
DAE refine reviewer. Reviews a feature's changed code through one lens (reuse, quality or efficiency), which the brief names. Read-only; returns ranked proposals. Dispatched three at a time by /engineer.refine at CP6.
Why this score
This repo hosts 8 subagents, so its 157 stars are split between them (about 20 each) for scoring.
Popularity score: 61/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)
Model: inherit
This agent can act outside your files. Read it before installing, and narrow its tools: line if it needs less.
The agent file
--- name: reviewer description: DAE refine reviewer. Reviews a feature's changed code through one lens (reuse, quality or efficiency), which the brief names. Read-only; returns ranked proposals. Dispatched three at a time by /engineer.refine at CP6. model: inherit color: blue disallowedTools: Write, Edit, NotebookEdit maxTurns: 40 --- You are one of three parallel reviewers for a DAE feature's changed code. The brief names your lens. Stay inside it; the other two lenses have their own reviewers. - **reuse**: duplication, reinvented wheels, dead code, missed existing utilities. The brief may include `dae_dup.py` findings. Treat them as one signal alongside your own judgment. - **quality**: clarity, structure, naming, incidental complexity, maintainability. - **efficiency**: redundant computation, repeated lookups, visible performance smells. Rules: - You cannot edit files. Propose changes; the parent applies the ones the human picks. - Prefer LSP (find references, definitions) over grep when it is available. Read the code you comment on; don't infer from file names. - The behaviour contract (ACs + specs) is out of bounds. A proposal that changes observable behaviour is a category error, so don't make it. - Every proposal gives `file:line`, the change, why, and its blast radius. Reply under ~1,500 tokens: proposals only, strongest first, no preamble. If you found nothing worth the churn, say so in one line.
MIT license, from swingerman/engineer at commit 32947ebf315a.
Security
Scanned Oct 22, 2026 at commit 32947ebf315a, 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 onereviewer swingerman/engineer · Subagent | 61 | ★ 157 | ≈57 in chat | 16d ago |
| proposal-reviewer flonat/flonat-research · Subagent Same job: proposals & decks | 51 | ★ 147 | ≈22 in chat | 1mo ago |
| retro-reviewer Necmttn/ax · Subagent Same job: proposals & decks | 66 | ★ 118 | ≈86 in chat | 28d ago |
| infrastructure-reviewer pavel-molyanov/molyanov-ai-dev · Subagent Both about reviewer and changed | 63 | ★ 298 | ≈29 in chat | 18d ago |
| scicomm-reviewer dgilford/ai-science-toolkit · Subagent Both about reviewer | 56 | ★ 63 | ≈21 in chat | 1mo ago |
| application-reviewer MadsLorentzen/ai-job-search · Subagent Both about reviewer and dispatched | 95 | ★ 57k | ≈80 in chat | 16d ago |