Adversarially review one ready-for-review ticket task in a read-only Pi subagent using a model different from the worker, then return a structured markdown findings list and a verdict. Use only for the review stage; it never edits code or applies its own findings.
84%
Does it follow best practices?
Run evals on this skill
Adds up to 20 points to the overall score
Passed
No findings from the security scan
Determine whether the implementation solves the stated problem. Be adversarial about evidence. Stay inside the task's scope.
Remain read-only. Never edit code, tests, plans, or state.
If the model matches the work model, return blocked. If mutation tools are present, return blocked. The controller must start this session with a model different from the work model and read-only tools. Never edit or write.
If the packet or diff file is missing and cannot be read, return blocked and name the missing input.
Do not review your own prior implementation session.
Read references/reviewer-prompt.md and follow it.
After behavior and acceptance criteria, apply references/code-quality.md.
Return only the factory.review.v4 template in references/output.md. Do not emit a JSON object. Do not wrap the result in a Markdown fence. Do not add a recap after it. Never write a review artifact to the ticket directory. After the template, call subagent_done once and emit nothing else. If it replies that the session is shutting down, stop. Do not call it again, do not call caller_ping, and do not send more text. caller_ping is only for a question you cannot answer. Do not edit the controller database.
geut/factory-skills
6f119c3
If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.