Menu
Review process overview, feedback tag definitions, and output format. Load when conducting or processing code reviews.
Install with Codex or Claude Copy this prompt, paste it into Codex, Claude, or another assistant, and let it review the skill page and install it for you.
Architectural validation checklist for feature approval. Load when validating that features fit into the existing architecture.
Architecture Decision Record format, naming conventions, and when to create ADRs. Load when making or documenting architectural decisions.
Audit the agentic configuration for consistency, coherence, and conciseness. Load when modifying agent definitions, skills, or pipeline structure, or to verify cross-tool parity.
Build, test, format, and lint requirements that must pass before code review. Load when checking implementation completeness or running the quality gate.
Go code quality checklist based on Google Go Style Guide. Load when conducting code quality reviews.
Pre-commit safety checks for secrets, credentials, local settings, and sensitive file patterns. Load before committing or when reviewing staged changes for accidental exposure.
Based on SOC occupation classification
| name | review-checklist |
| description | Review process overview, feedback tag definitions, and output format. Load when conducting or processing code reviews. |
| compatibility | ["claude-code","opencode","github-copilot"] |
| metadata | {"version":"1.0","author":"team"} |
After the feature-implementer passes the quality gate, invoke all four reviewers in parallel:
| Reviewer | Output File | Focus |
|---|---|---|
| code-quality-reviewer | .scratch/reviews/code-quality.md | Readability, Go style guide |
| test-reviewer | .scratch/reviews/test-coverage.md | Test pyramid, coverage, edge cases |
| security-reviewer | .scratch/reviews/security.md | OWASP, vulnerabilities, supply chain |
| doc-reviewer | .scratch/reviews/doc-review.md | Documentation coherence, structure |
| Tag | Meaning | Action |
|---|---|---|
[AUTOFIX] | Clear fix, no decision needed | Route to artifact owner |
[BLOCKED] | Critical issue, must fix before merge | Route to artifact owner; escalate if unclear |
[ESCALATE] | Needs human decision | Write to .scratch/escalations.md |
[CLARIFY:prd] | Requirement unclear | Route to product-requirements-expert |
[CLARIFY:system-design] | Architecture question | Route to system-design-expert |
[CLARIFY:security-reviewer] | Security question | Route to security-reviewer |
[CLARIFY:doc-reviewer] | Documentation question | Route to doc-reviewer |
Review feedback targets the artifact, not a fixed agent. Route fixes to the owning agent:
| Artifact | Owner Agent |
|---|---|
docs/prd.md | product-requirements-expert |
docs/system-design.md, docs/adr/*.md | system-design-expert |
internal/**/*.go, cmd/**/*.go | feature-implementer |
internal/**/*_test.go | feature-implementer |
| Templates, static assets | feature-implementer |
Do not bundle doc fixes into a feature-implementer call. Do not send code fixes to doc agents.
Each reviewer writes to their output file using the template in .claude/templates/review.md.
| Checklist Category | Default Severity | Tag |
|---|---|---|
| Cross-document coherence | Critical | [BLOCKED] |
| PRD boundary violations (Go code, function signatures, internal references) | Critical | [BLOCKED] |
Security vulnerabilities (CRITICAL/HIGH per security-review skill) | Critical | [BLOCKED] |
| Structural issues (missing anchors, broken links) | Fixable | [AUTOFIX] |
| Writing standards | Fixable | [AUTOFIX] |
After all reviewers complete:
[AUTOFIX] items: fix immediately.[BLOCKED] items: fix immediately; escalate if fix is unclear.[ESCALATE] items: write to .scratch/escalations.md.[CLARIFY:agent] items: request clarification from specified agent..scratch/review-summary.md.