메뉴
Review process overview, feedback tag definitions, and output format. Load when conducting or processing code reviews.
Codex 또는 Claude로 설치 이 Prompt를 복사해 Codex, Claude 또는 다른 어시스턴트에 붙여 넣으면 Skill 페이지를 검토하고 설치를 진행할 수 있습니다.
SOC 직업 분류 기준
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.
| 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.