菜单
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.