Menu
Use when receiving feedback from code-reviewer subagent or human reviewer
Instalar com Codex ou Claude Copie este prompt, cole no Codex, Claude ou outro assistente e deixe que ele revise a página da skill e instale para você.
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
Baseado na classificação ocupacional SOC
| 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 → 主流程