메뉴
Use when a feature, bugfix, or refactoring step is completed and needs review, or before merging to main, or when user says "review", "审查", "帮我看看代码"
Codex 또는 Claude로 설치 이 Prompt를 복사해 Codex, Claude 또는 다른 어시스턴트에 붙여 넣으면 Skill 페이지를 검토하고 설치를 진행할 수 있습니다.
SOC 직업 분류 기준
| name | sdk-code-review |
| description | Use when a feature, bugfix, or refactoring step is completed and needs review, or before merging to main, or when user says "review", "审查", "帮我看看代码" |
Type: Technique | Discipline: Rigid
dispatch 审查 subagent 对已完成的工作进行独立审查。审查者不继承当前会话历史——你负责构造它需要的全部上下文。
强制：
可选但推荐：
工作完成，需要审查?
│
▼
纯文档/注释/格式/typo 且 < 3 files 且无行为变更?
├─ YES → 可跳过（须向用户明示并得到默示/确认）
└─ NO
▼
有 plan 文件对应本次改动?
├─ YES → 模式 A：完整审查（spec-reviewer → code-reviewer）
└─ NO → 变更文件 < 3 且不涉及公开 API?
├─ YES → 模式 B：独立审查（仅 code-reviewer）
└─ NO → 回退补 plan（writing-plans），再走模式 A
跳过分支只适用于"显然无行为影响"的改动（如修 typo、更新注释、调整格式）。一旦触碰逻辑、接口、数据字段、构建脚本，立刻退回 Mode A/B。
spec-reviewer subagent → 通过后进入步骤 2code-reviewer subagent满足全部条件时，只 dispatch code-reviewer：
# 场景 A：review 整个功能分支
BASE_SHA=$(git merge-base HEAD master)
HEAD_SHA=$(git rev-parse HEAD)
# 场景 B：单个任务（subagent-driven-development 中）
# BASE_SHA 在 dispatch 实现者前记录，HEAD_SHA 在实现者提交后记录
# 查看变更文件
git diff --name-only $BASE_SHA..$HEAD_SHA
spec-reviewer： 变更内容、规格/规划全文、实现者报告、BASE_SHA、HEAD_SHA、变更文件列表
code-reviewer： 变更内容、plan 文件路径（参考）、BASE_SHA、HEAD_SHA、变更文件列表
若变更文件包含
.agents/或.claude/路径，显式注明："本次变更包含 skill/agent 配置文件，请额外执行维度 7（Skill/Agent 架构一致性）检查。"
审查者返回结果
├── "通过" → 继续后续工作
├── "需要修改"
│ ├── Critical → 立即修复，重新 dispatch 同审查者
│ ├── Important → 合并前修复，重新 dispatch
│ └── Suggestion → 记录，可后续处理
└── "需要讨论" → 与用户讨论后决定
| Excuse | Reality |
|---|---|
| "改动小，跳过审查吧" | 模式 B 已是最轻量路径，再跳 = 裸奔 |
| "我自己看过一遍没问题" | 自审替代不了独立审查 |
| "先做质量审查，规格审查之后补" | 顺序不可颠倒 |
| "改完了，直接声明通过" | Critical/Important 修复后必须重新 dispatch |
verification-before-completion 通过后进入本 skillspec-reviewer（模式 A 第一阶段）/ code-reviewer（模式 A 第二阶段 / 模式 B）finishing-a-development-branchreceiving-code-review（修复后先 verify 再重新 dispatch reviewer）Use when receiving an ambiguous feature request, when scope is unclear, or before writing any plan
Use after verification-before-completion passes and code review is clean, to close out the development branch
Use after writing a plan document in docs/plans/, before requesting user confirmation
Use when receiving feedback from code-reviewer subagent or human reviewer
Use when executing an implementation plan with 3+ independent tasks in the current session
Use when facing Xcode build errors, linker failures, XCTest failures, event pipeline bugs, network issues, SDK crashes, or any technical problem where a first attempt didn't work or the cause is unclear