Home / Subagents / reviewer

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.

ScannedSubagentSource: GitHub
157 stars Updated 16d ago License MIT Context ≈57 tokens in your chat
61popular

Why this score

Stars9/35
Usage (forks)2/15
Upkeep25/25
License15/15
Owner history10/10

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.

NameScoreStarsContextUpdated
This onereviewer swingerman/engineer · Subagent 61★ 157≈57 in chat16d ago
proposal-reviewer flonat/flonat-research · Subagent Same job: proposals & decks 51★ 147≈22 in chat1mo ago
retro-reviewer Necmttn/ax · Subagent Same job: proposals & decks 66★ 118≈86 in chat28d ago
infrastructure-reviewer pavel-molyanov/molyanov-ai-dev · Subagent Both about reviewer and changed 63★ 298≈29 in chat18d ago
scicomm-reviewer dgilford/ai-science-toolkit · Subagent Both about reviewer 56★ 63≈21 in chat1mo ago
application-reviewer MadsLorentzen/ai-job-search · Subagent Both about reviewer and dispatched 95★ 57k≈80 in chat16d ago