{"id":"54e9aa1c-4148-4e05-9a1f-33f849b7effb","shortId":"LaXBFH","kind":"skill","title":"receiving-code-review","tagline":"收到代码审查反馈后、实施建议之前使用，尤其当反馈不明确或技术上有疑问时——需要技术严谨性和验证，而非敷衍附和或盲目执行","description":"# 接收代码审查\n\n## 概述\n\n代码审查需要的是技术评估，不是情绪表演。\n\n**核心原则：** 先验证再实施。先提问再假设。技术正确性优先于社交舒适度。\n\n## 响应模式\n\n```\n收到代码审查反馈时：\n\n1. 阅读：完整阅读反馈，不急于反应\n2. 理解：用自己的话复述需求（或提问）\n3. 验证：对照代码库的实际情况检查\n4. 评估：对这个代码库来说技术上合理吗？\n5. 回应：技术性确认或有理有据的反驳\n6. 实施：一次一项，逐个测试\n```\n\n## 禁止的回应\n\n**绝不要说：**\n- \"你说得太对了！\"（明确违反 CLAUDE.md 规定）\n- \"好观点！\"/\"反馈很棒！\"（敷衍表演）\n- \"让我立刻实施\"（在验证之前）\n\n**应该这样做：**\n- 复述技术需求\n- 提出澄清性问题\n- 如果审查意见有误，用技术理由反驳\n- 直接动手做（行动胜于言辞）\n\n## 处理不明确的反馈\n\n```\n如果有任何一项不明确：\n  停下来——先不要实施任何内容\n  就不明确的项目提出澄清\n\n为什么：各项之间可能有关联。部分理解 = 错误实施。\n```\n\n**示例：**\n```\n搭档：\"修复第 1-6 项\"\n你理解 1、2、3、6。对 4、5 不确定。\n\n❌ 错误做法：先实施 1、2、3、6，稍后再问 4、5\n✅ 正确做法：\"第 1、2、3、6 项我理解了。第 4 和第 5 项需要澄清后再动手。\"\n```\n\n## 按来源区别处理\n\n### 来自搭档的反馈\n- **可信赖** —— 理解后直接实施\n- **仍然要问** 如果范围不明确\n- **不要敷衍附和**\n- **直接行动** 或给出技术性确认\n\n### 来自外部审查者的反馈\n```\n实施之前：\n  1. 检查：对这个代码库来说技术上正确吗？\n  2. 检查：是否会破坏现有功能？\n  3. 检查：当前实现这样写是否有原因？\n  4. 检查：在所有平台/版本上都适用吗？\n  5. 检查：审查者了解完整上下文吗？\n\n如果建议似乎有误：\n  用技术理由反驳\n\n如果无法轻易验证：\n  说明情况：\"没有 [X] 我无法验证这一点。我应该 [调查/提问/先做]？\"\n\n如果与搭档之前的决策冲突：\n  先停下来和搭档讨论\n```\n\n**搭档的原则：** \"对外部反馈要持怀疑态度，但要仔细核实\"\n\n## YAGNI 检查——针对\"专业化\"功能建议\n\n```\n如果审查者建议\"正规地实现\"：\n  在代码库中 grep 实际使用情况\n\n  如果没人用：\"这个接口没有被调用。删掉它（YAGNI）？\"\n  如果有人用：那就正规实现\n```\n\n**搭档的原则：** \"你和审查者都对我负责。如果我们不需要这个功能，就不要加。\"\n\n## 实施顺序\n\n```\n对于包含多项的反馈：\n  1. 先澄清所有不明确的项\n  2. 然后按以下顺序实施：\n     - 阻塞性问题（崩溃、安全）\n     - 简单修复（拼写、导入）\n     - 复杂修复（重构、逻辑）\n  3. 逐个测试每项修复\n  4. 验证没有回归\n```\n\n## 何时反驳\n\n在以下情况反驳：\n- 建议会破坏现有功能\n- 审查者缺少完整上下文\n- 违反 YAGNI（功能没人用）\n- 对当前技术栈来说技术上不正确\n- 存在遗留/兼容性原因\n- 与搭档的架构决策冲突\n\n**如何反驳：**\n- 用技术理由，不要带防御情绪\n- 提出具体问题\n- 引用可正常工作的测试/代码\n- 如果涉及架构问题，让搭档参与\n\n**如果觉得不方便当众反驳，暗号是：** \"Strange things are afoot at the Circle K\"\n\n## 确认正确的反馈\n\n当反馈确实正确时：\n```\n✅ \"已修复。[简要说明改了什么]\"\n✅ \"发现得好——[具体问题]。已在 [位置] 修复。\"\n✅ [直接修复并在代码中体现]\n\n❌ \"你说得太对了！\"\n❌ \"好观点！\"\n❌ \"感谢你发现了这个！\"\n❌ \"感谢你 [任何内容]\"\n❌ 任何感谢的表达\n```\n\n**为什么不用感谢：** 行动说明一切。直接修复。代码本身就能表明你收到了反馈。\n\n**如果你发现自己要写\"感谢\"：** 删掉它。直接说明修复内容。\n\n## 优雅地纠正自己的反驳\n\n如果你反驳了但事后发现自己错了：\n```\n✅ \"你是对的——我检查了 [X]，确实 [Y]。正在实施。\"\n✅ \"验证后确认你是对的。我最初的理解有误，因为 [原因]。正在修复。\"\n\n❌ 长篇道歉\n❌ 为自己的反驳辩护\n❌ 过度解释\n```\n\n如实陈述纠正，然后继续。\n\n## 常见错误\n\n| 错误 | 修正 |\n|------|------|\n| 敷衍附和 | 复述需求或直接行动 |\n| 盲目实施 | 先对照代码库验证 |\n| 批量实施不测试 | 一次一项，逐个测试 |\n| 假设审查者一定对 | 检查是否会破坏现有功能 |\n| 回避反驳 | 技术正确性 > 社交舒适度 |\n| 部分理解就开始实施 | 先澄清所有项 |\n| 无法验证却继续推进 | 说明限制，请求指导 |\n\n## 真实案例\n\n**敷衍附和（反面例子）：**\n```\n审查者：\"删除遗留代码\"\n❌ \"你说得太对了！让我删掉它……\"\n```\n\n**技术验证（正面例子）：**\n```\n审查者：\"删除遗留代码\"\n✅ \"查了一下……构建目标是 10.15+，这个 API 需要 13+。向后兼容需要保留遗留代码。当前实现有错误的 bundle ID——修复它还是放弃 pre-13 支持？\"\n```\n\n**YAGNI（正面例子）：**\n```\n审查者：\"实现完善的指标追踪，包括数据库、日期过滤、CSV 导出\"\n✅ \"在代码库中 grep 了一下——没有任何地方调用这个接口。删掉它（YAGNI）？还是有我遗漏的调用？\"\n```\n\n**不明确的项（正面例子）：**\n```\n搭档：\"修复第 1-6 项\"\n你理解 1、2、3、6。对 4、5 不确定。\n✅ \"第 1、2、3、6 项我理解了。第 4 和第 5 项需要澄清后再动手。\"\n```\n\n## GitHub 评论回复\n\n在 GitHub 上回复行内审查评论时，在评论线程中回复（`gh api repos/{owner}/{repo}/pulls/{pr}/comments/{id}/replies`），不要发顶层 PR 评论。\n\n## 底线\n\n**外部反馈 = 待评估的建议，不是必须执行的命令。**\n\n验证。质疑。然后实施。\n\n不要敷衍附和。始终保持技术严谨。","tags":["receiving","code","review","superpowers","jnmetacode","agent-skills","agentic-coding","ai-coding","chinese","claude-code","code-review","cursor"],"capabilities":["skill","source-jnmetacode","skill-receiving-code-review","topic-agent-skills","topic-agentic-coding","topic-ai-coding","topic-chinese","topic-claude-code","topic-code-review","topic-cursor","topic-gemini-cli","topic-kiro","topic-mcp","topic-npm-package","topic-prompt-engineering"],"categories":["superpowers-zh"],"synonyms":[],"warnings":[],"endpointUrl":"https://skills.sh/jnMetaCode/superpowers-zh/receiving-code-review","protocol":"skill","transport":"skills-sh","auth":{"type":"none","details":{"cli":"npx skills add jnMetaCode/superpowers-zh","source_repo":"https://github.com/jnMetaCode/superpowers-zh","install_from":"skills.sh"}},"qualityScore":"0.700","qualityRationale":"deterministic score 0.70 from registry signals: · indexed on github topic:agent-skills · 1857 github stars · SKILL.md body (2,710 chars)","verified":false,"liveness":"unknown","lastLivenessCheck":null,"agentReviews":{"count":0,"score_avg":null,"cost_usd_avg":null,"success_rate":null,"latency_p50_ms":null,"narrative_summary":null,"summary_updated_at":null},"enrichmentModel":"deterministic:skill-github:v1","enrichmentVersion":1,"enrichedAt":"2026-05-03T00:52:45.958Z","embedding":null,"createdAt":"2026-04-18T21:55:58.108Z","updatedAt":"2026-05-03T00:52:45.958Z","lastSeenAt":"2026-05-03T00:52:45.958Z","tsv":"'-13':301 '-6':72,323 '/comments':358 '/pulls':356 '/replies':360 '1':20,71,75,85,94,115,169,322,326,335 '10.15':290 '13':294 '2':24,76,86,95,118,171,327,336 '3':28,77,87,96,121,182,328,337 '4':31,80,90,100,124,184,331,341 '5':34,81,91,102,128,332,343 '6':37,78,88,97,329,338 'afoot':210 'api':292,352 'bundl':297 'circl':213 'claude.md':45 'code':3 'csv':309 'gh':351 'github':345,348 'grep':155,312 'id':298,359 'k':214 'owner':354 'pr':357,362 'pre':300 'receiv':2 'receiving-code-review':1 'repo':353,355 'review':4 'skill' 'skill-receiving-code-review' 'source-jnmetacode' 'strang':207 'thing':208 'topic-agent-skills' 'topic-agentic-coding' 'topic-ai-coding' 'topic-chinese' 'topic-claude-code' 'topic-code-review' 'topic-cursor' 'topic-gemini-cli' 'topic-kiro' 'topic-mcp' 'topic-npm-package' 'topic-prompt-engineering' 'x':136,243 'y':245 'yagni':147,160,191,303,316 '一次一项':39,265 '上回复行内审查评论时':349 '不急于反应':23 '不明确的项':318 '不是必须执行的命令':367 '不是情绪表演':13 '不确定':82,333 '不要发顶层':361 '不要带防御情绪':199 '不要敷衍附和':110,371 '与搭档的架构决策冲突':196 '专业化':150 '为什么':64 '为什么不用感谢':231 '为自己的反驳辩护':253 '了一下':313 '仍然要问':108 '代码':202 '代码审查需要的是技术评估':12 '代码本身就能表明你收到了反馈':234 '任何内容':229 '任何感谢的表达':230 '优雅地纠正自己的反驳':239 '但要仔细核实':146 '位置':222 '何时反驳':186 '你和审查者都对我负责':164 '你是对的':241 '你理解':74,325 '你说得太对了':43,225,282 '修复':223 '修复它还是放弃':299 '修复第':70,321 '修正':259 '假设审查者一定对':267 '停下来':61 '先不要实施任何内容':62 '先做':141 '先停下来和搭档讨论':143 '先实施':84 '先对照代码库验证':263 '先提问再假设':16 '先澄清所有不明确的项':170 '先澄清所有项':273 '先验证再实施':15 '具体问题':220 '兼容性原因':195 '删掉它':159,237,315 '删除遗留代码':281,287 '功能建议':151 '功能没人用':192 '包括数据库':307 '原因':250 '反面例子':279 '反馈很棒':48 '发现得好':219 '可信赖':106 '各项之间可能有关联':65 '向后兼容需要保留遗留代码':295 '和第':101,342 '响应模式':18 '回应':35 '回避反驳':269 '因为':249 '在':347 '在代码库中':154,311 '在以下情况反驳':187 '在所有平台':126 '在评论线程中回复':350 '在验证之前':51 '处理不明确的反馈':59 '复杂修复':179 '复述技术需求':53 '复述需求或直接行动':261 '外部反馈':365 '好观点':47,226 '如何反驳':197 '如实陈述纠正':255 '如果与搭档之前的决策冲突':142 '如果你反驳了但事后发现自己错了':240 '如果你发现自己要写':235 '如果审查意见有误':55 '如果审查者建议':152 '如果建议似乎有误':131 '如果我们不需要这个功能':165 '如果无法轻易验证':133 '如果有人用':161 '如果有任何一项不明确':60 '如果没人用':157 '如果涉及架构问题':203 '如果范围不明确':109 '如果觉得不方便当众反驳':205 '始终保持技术严谨':372 '存在遗留':194 '安全':175 '完整阅读反馈':22 '实施':38 '实施之前':114 '实施建议之前使用':6 '实施顺序':167 '实现完善的指标追踪':306 '实际使用情况':156 '审查者':280,286,305 '审查者了解完整上下文吗':130 '审查者缺少完整上下文':189 '对':79,330 '对于包含多项的反馈':168 '对外部反馈要持怀疑态度':145 '对当前技术栈来说技术上不正确':193 '对照代码库的实际情况检查':30 '对这个代码库来说技术上合理吗':33 '对这个代码库来说技术上正确吗':117 '导入':178 '导出':310 '尤其当反馈不明确或技术上有疑问时':7 '就不明确的项目提出澄清':63 '就不要加':166 '崩溃':174 '已修复':217 '已在':221 '常见错误':257 '应该这样做':52 '底线':364 '建议会破坏现有功能':188 '引用可正常工作的测试':201 '当前实现有错误的':296 '当前实现这样写是否有原因':123 '当反馈确实正确时':216 '待评估的建议':366 '感谢':236 '感谢你':228 '感谢你发现了这个':227 '我应该':138 '我无法验证这一点':137 '我最初的理解有误':248 '我检查了':242 '或提问':27 '或给出技术性确认':112 '批量实施不测试':264 '技术性确认或有理有据的反驳':36 '技术正确性':270 '技术正确性优先于社交舒适度':17 '技术验证':284 '拼写':177 '按来源区别处理':104 '接收代码审查':10 '提出具体问题':200 '提出澄清性问题':54 '提问':140 '搭档':69,320 '搭档的原则':144,163 '支持':302 '收到代码审查反馈后':5 '收到代码审查反馈时':19 '敷衍表演':49 '敷衍附和':260,278 '无法验证却继续推进':274 '日期过滤':308 '明确违反':44 '是否会破坏现有功能':120 '暗号是':206 '来自外部审查者的反馈':113 '来自搭档的反馈':105 '构建目标是':289 '查了一下':288 '核心原则':14 '检查':116,119,122,125,129,148 '检查是否会破坏现有功能':268 '概述':11 '正在修复':251 '正在实施':246 '正确做法':92 '正规地实现':153 '正面例子':285,304,319 '没有':135 '没有任何地方调用这个接口':314 '然后实施':370 '然后按以下顺序实施':172 '然后继续':256 '版本上都适用吗':127 '理解':25 '理解后直接实施':107 '用技术理由':198 '用技术理由反驳':56,132 '用自己的话复述需求':26 '盲目实施':262 '直接修复':233 '直接修复并在代码中体现':224 '直接动手做':57 '直接行动':111 '直接说明修复内容':238 '真实案例':277 '确实':244 '确认正确的反馈':215 '示例':68 '社交舒适度':271 '禁止的回应':41 '稍后再问':89 '第':93,99,334,340 '简单修复':176 '简要说明改了什么':218 '绝不要说':42 '而非敷衍附和或盲目执行':9 '行动胜于言辞':58 '行动说明一切':232 '规定':46 '让我删掉它':283 '让我立刻实施':50 '让搭档参与':204 '评估':32 '评论':363 '评论回复':346 '说明情况':134 '说明限制':275 '请求指导':276 '调查':139 '质疑':369 '过度解释':254 '还是有我遗漏的调用':317 '这个':291 '这个接口没有被调用':158 '违反':190 '逐个测试':40,266 '逐个测试每项修复':183 '逻辑':181 '那就正规实现':162 '部分理解':66 '部分理解就开始实施':272 '重构':180 '针对':149 '错误':258 '错误做法':83 '错误实施':67 '长篇道歉':252 '阅读':21 '阻塞性问题':173 '需要':293 '需要技术严谨性和验证':8 '项':73,324 '项我理解了':98,339 '项需要澄清后再动手':103,344 '验证':29,368 '验证后确认你是对的':247 '验证没有回归':185","prices":[{"id":"d4fe3d2a-d947-4749-9295-705b200bb55b","listingId":"54e9aa1c-4148-4e05-9a1f-33f849b7effb","amountUsd":"0","unit":"free","nativeCurrency":null,"nativeAmount":null,"chain":null,"payTo":null,"paymentMethod":"skill-free","isPrimary":true,"details":{"org":"jnMetaCode","category":"superpowers-zh","install_from":"skills.sh"},"createdAt":"2026-04-18T21:55:58.108Z"}],"sources":[{"listingId":"54e9aa1c-4148-4e05-9a1f-33f849b7effb","source":"github","sourceId":"jnMetaCode/superpowers-zh/receiving-code-review","sourceUrl":"https://github.com/jnMetaCode/superpowers-zh/tree/main/skills/receiving-code-review","isPrimary":false,"firstSeenAt":"2026-04-18T21:55:58.108Z","lastSeenAt":"2026-05-03T00:52:45.958Z"}],"details":{"listingId":"54e9aa1c-4148-4e05-9a1f-33f849b7effb","quickStartSnippet":null,"exampleRequest":null,"exampleResponse":null,"schema":null,"openapiUrl":null,"agentsTxtUrl":null,"citations":[],"useCases":[],"bestFor":[],"notFor":[],"kindDetails":{"org":"jnMetaCode","slug":"receiving-code-review","github":{"repo":"jnMetaCode/superpowers-zh","stars":1857,"topics":["agent-skills","agentic-coding","ai-coding","chinese","claude-code","code-review","cursor","gemini-cli","kiro","mcp","npm-package","prompt-engineering","skills","superpowers","tdd","trae"],"license":"mit","html_url":"https://github.com/jnMetaCode/superpowers-zh","pushed_at":"2026-04-28T15:32:18Z","description":"🦸 AI 编程超能力 · 中文增强版 — superpowers（116k+ ⭐）完整汉化 + 6 个中国原创 skills，让 Claude Code / Copilot CLI / Hermes Agent / Cursor / Windsurf / Kiro / Gemini CLI 等 16 款 AI 编程工具真正会干活","skill_md_sha":"570ba81e994e80f51fe373a2c29d305764c2bf1c","skill_md_path":"skills/receiving-code-review/SKILL.md","default_branch":"main","skill_tree_url":"https://github.com/jnMetaCode/superpowers-zh/tree/main/skills/receiving-code-review"},"layout":"multi","source":"github","category":"superpowers-zh","frontmatter":{"name":"receiving-code-review","description":"收到代码审查反馈后、实施建议之前使用，尤其当反馈不明确或技术上有疑问时——需要技术严谨性和验证，而非敷衍附和或盲目执行"},"skills_sh_url":"https://skills.sh/jnMetaCode/superpowers-zh/receiving-code-review"},"updatedAt":"2026-05-03T00:52:45.958Z"}}