القائمة
Use when receiving feedback from code-reviewer subagent or human reviewer
التثبيت باستخدام 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 a feature, bugfix, or refactoring step is completed and needs review, or before merging to main, or when user says "review", "审查", "帮我看看代码"
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 | receiving-code-review |
| description | Use when receiving feedback from code-reviewer subagent or human reviewer |
Type: Technique | Discipline: Rigid
收到 code-reviewer subagent 或人类 reviewer 的反馈后，按本技能规范处理。
核心原则： 验证先于实施，澄清先于假设，技术正确性高于社交舒适度。
收到审查反馈
↓
1. READ：完整读完所有反馈，不急于行动
2. UNDERSTAND：用自己的话复述每条要求（或提问）
3. VERIFY：对照代码库现状核实
4. EVALUATE：对本项目是否技术合理？
5. RESPOND：技术性确认或有理据的 push back
6. IMPLEMENT：逐条修复，每条单独测试
如果反馈中任何一项不明确：
→ 全部暂停，不实施任何一项
→ 先对不明确的项提问澄清
→ 澄清完毕后再开始实施
示例：
reviewer 返回 6 条问题，你理解 1、2、3、6，不确定 4、5。
❌ 错误：先改 1、2、3、6，回头再问 4、5
✅ 正确："第 1、2、3、6 条已理解。第 4 条和第 5 条需要澄清：
- 第 4 条提到 'session 边界处理'，是指 UIApplication 进入后台还是 App 终止？
- 第 5 条 '协议对齐'，具体是哪个字段？"
当 reviewer 建议"实现得更完善"时：
→ 先 grep 代码库确认是否有实际调用
如果没有调用："这个接口当前没有调用方，是否需要实现？（YAGNI）"
绝不说：
应该说：
file:line 修复。"1. 先澄清所有不明确项（STOP-ASK）
2. 按优先级实施：
a. Critical（崩溃、数据丢失、隐私泄露）
b. 简单修复（格式、导入）
c. 复杂修复（重构、逻辑变更）
3. 每条修复后单独验证
4. 全部完成后确认无回归
反馈项全部实施完成后：
verification-before-completion 重跑| Excuse | Reality |
|---|---|
| "reviewer 说的有理，全盘接受" | 表演式认同是失职，先验证 |
| "改完了不用重新 dispatch" | Critical/Important 修完必须重新 review |
| "先改 Critical，Important 合并前再说" | Important 也阻塞合并 |
| "部分听懂了就开始改" | STOP-ASK，不明确项必须澄清 |
| "我改得很小，不用重跑 verify" | 任何修改后 verify 都要重跑 |
sdk-code-review 的 reviewer 或人类 reviewer 返回反馈verification-before-completion → 主流程