Agentic Coding Review Report

Review against the spec, the evals, and project constraints, not just a subjective "looks good".

Review scope

• PR / commit:
• Spec:
• Risk tier:
• Required review level:

Summary

• Decision: Approved / Changes requested / Blocked

Spec compliance

• [ ] Goal satisfied
• [ ] Scope respected
• [ ] Out-of-scope respected
• [ ] Expected behavior implemented

Notes:

Eval compliance

• [ ] Required evals exist
• [ ] Evals pass
• [ ] Edge cases covered
• [ ] Existing tests still pass

Notes:

Architecture review

• [ ] Existing boundaries respected
• [ ] Existing patterns followed
• [ ] No unnecessary abstractions introduced
• [ ] No unrelated refactoring

Notes:

Security and data review

• [ ] No secrets read or exposed
• [ ] No auth / permission changes unless scoped
• [ ] No production data touched
• [ ] Input validation considered where relevant

Notes:

Operational review

• [ ] CI impact understood
• [ ] Deployment impact understood
• [ ] Observability / logging considered if relevant
• [ ] Rollback impact considered if relevant

Notes:

Blockers

• ...

Suggestions

• ...

Final decision

• Approve
• Request changes
• Escalate to rigorous review
• Block