القائمة
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 المهني
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
| 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）