| name | review-code |
| description | Quick one-shot code review for the aiagent project. The developer-reviewer checks DDD layer boundaries, Go idioms, error handling, storage duality, and test coverage. No iteration loop — immediate feedback. |
| user-invocable | true |
| argument-hint | review internal/impl/tools/new_tool.go |
| context | fork |
Code Review
Review the following:
Instructions
Use the Agent tool with subagent_type: "developer-reviewer" to perform a one-shot code review.
Pass it:
- The file(s) or description of what to review
- Any relevant context (what feature this is for, what the acceptance criteria are)
The reviewer will:
- Perform a mandatory duplication check first
- Run automated checks:
go vet ./... && go build . && go test ./... -race
- Score against the 6 pillars (Maintainability, Reliability, Scalability, Usability, Security, Quality)
- Produce a verdict: APPROVED or REVISE
Output Format
The reviewer produces:
- Duplication check results
- Pillar scores table
- Verdict: APPROVED / REVISE
- Critical Issues (blocking)
- Warnings (non-blocking)
- Strengths
Important
- This is a one-shot review with no iteration — use
/peer-review for the full implementation + review loop
- Critical issues must be addressed before merging
- Warnings are recommendations
