receiving-code-review

概览

receiving-code-review 技能的作用

receiving-code-review 技能为代理在收到代码评审反馈（特别是 GitHub pull request 上的反馈）时，定义了一套清晰、可重复的响应模式。它强调的是技术评估，而不是社交表演。

这个技能不会让代理对评审者言听计从、立刻照做，而是训练它：

• 在做出任何反应之前，先完整阅读所有反馈
• 先复述或澄清对方要求的改动
• 将反馈对照真实代码库逐条核实
• 评估建议在当前仓库中是否技术上成立
• 用有依据的技术理由回应，而不是奉承
• 在充分理解和核实之后，再逐项实现改动

适用人群

在以下场景下，可以使用 receiving-code-review：

• 你在 GitHub PR 或代码评审讨论中使用 AI 助理
• 希望助理像一个深思熟虑的评审者或被评审者，而不是一味附和
• 需要帮助理解评审意见，并决定下一步如何处理
• 更在意改动的正确性和安全性，而不是“快但浅”的一致

它特别适合：

• 在功能分支和 pull request 上协作的开发者
• 希望团队统一处理评审反馈方式的技术负责人
• 正在尝试把 Claude 或其他 LLM 作为 PR 评审伙伴的团队

不适用的场景

这个技能不是为以下用途设计的：

• 从零开始生成新特性
• 没有评审上下文的大规模重构
• 客套寒暄、表扬或状态汇报

如果你主要需求是代码生成、文档草稿撰写或高层设计建议，可以搭配其他技能使用。将 receiving-code-review 专门用在代理接收并回应代码评审反馈的环节。

关键优势

安装 receiving-code-review 之后，你的代理将会：

• 避免诸如“You're absolutely right!”、“Great point!” 这类表演式回复
• 基于真实代码作出回应，而不是凭空猜测
• 在反馈不清楚时询问澄清，而不是瞎猜
• 在建议技术上有问题时，礼貌但明确地提出异议
• 降低误解评审意见、引入有害改动的风险

这有助于你在 code-review、git-workflows 和 pr-review 流程中更信任代理的表现。

使用方法

1. 安装

要从 obra/superpowers 仓库安装 receiving-code-review 技能，运行：

npx skills add https://github.com/obra/superpowers --skill receiving-code-review

这会将技能定义文件（包括 SKILL.md）拉取到你的代理技能环境中。安装前默认你已配置好 npx skills 工具；如果还没有，请先根据平台或代理宿主的说明完成该工具的安装和配置。

2. 安装后需要查看的文件

安装完成后，先检查此技能的核心文件：

• skills/receiving-code-review/SKILL.md – 这是在接收代码评审反馈时行为模式的权威说明。

在更大的 obra/superpowers 仓库中，你还会看到一些共享模式，例如：

• 仓库根目录下的 README.md、AGENTS.md 或 metadata.json – 介绍技能整体结构和使用方式的通用说明

这些文件有助于理解 receiving-code-review 在整个 Claude/代理规则集中的位置，但这个技能的运行核心仍然在 SKILL.md 中。

3. 核心响应流程

当代理收到代码评审反馈（例如 GitHub PR 评论线程）时，这个技能定义了一套固定的响应流程：

1. READ:     Consume all feedback before reacting
2. UNDERSTAND: Restate the requirement in its own words, or ask
3. VERIFY:   Check the feedback against the real codebase
4. EVALUATE: Decide if it is technically sound for THIS repo
5. RESPOND:  Give a technical acknowledgment or reasoned pushback
6. IMPLEMENT: Change one item at a time and test each

在实际操作中，这意味着：

• 代理不应该一上来就说要立刻实现建议。
• 它会先确认自己是否理解了评审者的要求。
• 然后检查相关文件/代码行或当前仓库状态。
• 之后才决定是采纳、调整还是拒绝这条建议。

这一流程特别适用于 GitHub pull request review 的场景，在这里背景信息和正确性比速度更重要。

4. 禁止和不鼓励的回应

这个技能明确禁止某些在 LLM 中常见、但在严肃代码评审中有害的回应方式：

禁止示例：

• "You're absolutely right!"（在更高层级的 CLAUDE 规则中被明确视为违规）
• "Great point!" / "Excellent feedback!" 等仅有表扬、不含实质内容的回复
• 当代理尚未核实建议时，回复 "Let me implement that now"

在使用 receiving-code-review 时，代理应当改为：

• 复述技术需求，例如：“You are asking to extract this logic into a separate function to avoid duplication.”
• 在有疑问时提出针对性问题
• 当认为建议不正确或不完整时，给出技术上的理由
• 向实际改动推进，但避免过度解释或泛泛而谈的表扬

这样对话始终聚焦在代码质量，而非客套话上。

5. 处理不清楚或不完整的反馈

这个技能对模糊反馈有严格规则：

IF any item in the feedback is unclear:
  STOP – do not implement anything yet
  ASK for clarification on the unclear items

原因在于：单条评审意见之间可能存在关联，如果你在部分不明的情况下，只实现自己“觉得理解了”的部分，可能会导致：

• 互相冲突的改动
• 工作流被破坏
• 实现行为与评审者意图偏离

比如，评审者说 “Fix 1–6”，而代理只理解了 1、2、3 和 6。此时 receiving-code-review 会引导代理：

• 暂停实现
• 针对第 4 与第 5 条提出具体澄清问题
• 在完整理解全部需求之后再开始实现

在自动化或半自动化的 git 工作流中，这种行为尤为关键，因为“半懂不懂”的实现很容易让分支进入损坏状态。

6. 集成到你的 GitHub / PR 工作流

要在真实项目中充分发挥 receiving-code-review 的价值，可以这样做：

1. 将技能挂载到你的代理上，用于以下任务：

   • 审查 pull request
   • 草拟对评审意见的回复
   • 帮助梳理或总结评审反馈

2. 确保代理有仓库访问权限，使其可以实际核实建议是否适用于：

   • 当前分支代码
   • 相关文件和模块

3. 与其他互补技能组合使用，效果更佳，例如：

   • 负责实际实现和重构的 coding/refactoring 技能
   • 帮助快速定位受影响代码的仓库导航或搜索技能

4. 向团队解释代理的行为，让大家有合理预期：

   • 代理会提问澄清，而不是乱猜
   • 在建议有误或有风险时，可能会提出异议
   • 会避免空泛的表扬，改为给出具体技术回应

以这种方式集成后，receiving-code-review 会成为一条“护栏”，让你的 AI 协作者在代码评审对话中保持纪律性和可信度。

7. 何时启用该技能

在以下情况中使用 receiving-code-review：

• 代理正在阅读人类或机器人对某个 pull request 的反馈
• 正在浏览 GitHub diff 页面上的行内评论
• 正在处理代码评审工具中的评审笔记

通常在以下情况中不需要它：

• 生成初始代码或功能的第一版实现
• 编写设计文档或 ADR
• 做依赖升级等与评审无关的任务

只在评审上下文中启用该技能，可以让代理行为更可预期、更聚焦。

FAQ

receiving-code-review 解决了什么问题？

receiving-code-review 技能针对的是 AI 对代码评审反馈“浅、演、快”的回应问题。它不会让代理一味同意并立刻修改代码，而是要求代理：

• 完整阅读所有反馈
• 将反馈与现有代码库对照核实
• 澄清模糊的请求
• 在必要时用技术理由提出异议

这能显著减少 GitHub PR 以及其他代码评审工具中的错误实现和沟通偏差。

如何安装 receiving-code-review？

从 obra/superpowers 仓库安装该技能：

npx skills add https://github.com/obra/superpowers --skill receiving-code-review

安装完成后，请查看 receiving-code-review 目录下的 SKILL.md，了解具体的行为规则。

这个技能会改变代码的写法吗？

会有间接影响。receiving-code-review 本身不会生成代码，但会通过以下方式强烈影响代码何时、如何被修改：

• 在实现前必须先核实
• 按项逐个修改并测试
• 避免基于误解或片面的修复

你可以将它与 coding 技能配合使用：在评审反馈被验证之后，由其他技能负责具体实现。

receiving-code-review 会对人类评审者“顶嘴”吗？

会，在合适的时候。这个技能明确允许并鼓励在以下情况下给出有理有据的技术反驳：

• 建议不适用于当前代码库
• 建议基于过时假设
• 采纳建议可能引入 bug 或回归

但前提是：反驳必须建立在仓库中的具体事实之上，而不是主观观点。

这个技能只适用于 GitHub 吗？

这个技能是围绕 GitHub 式的 PR review 工作流设计的，但同样适用于其他只要能让代理接收结构化代码评审反馈的环境，比如：

• 各类基于 Git 的代码评审工具
• 内部评审看板或平台
• 在聊天工具中进行、但引用了具体文件和行号的评审会话

只要你的工作流类似“PR 评论 + git 仓库”的组合，receiving-code-review 基本都适用。

它如何与 CLAUDE 或其他代理规则协同？

在 obra/superpowers 生态中，技能会与更高层级的规则叠加（通常写在 CLAUDE.md 这类文件中）。receiving-code-review 会参考这些期望，例如禁止 "You're absolutely right!" 这种违背整体规则精神的回复。

你可以将它与现有代理规则一起使用，以：

• 强化评审阶段的行为约束
• 避免过度追求“社交表现”
• 在不同项目和仓库间保持行为一致

如果我的团队偏好更礼貌的语气怎么办？

在保持专业语气的前提下，这个技能仍然优先强调清晰的技术沟通，而不是礼貌套话。如果你需要更柔和的表达方式，可以：

• 在其他技能中补充关于语气/风格的单独指引
• 继续用 receiving-code-review 作为核实与严谨性的“骨架”

这种分层方式可以让你调节表达风格，而不削弱核心的评审纪律。

我怎么判断这个技能是否正常生效？

当 receiving-code-review 已启用且运行良好时，你会看到：

• 代理不再用泛泛的表扬来回复评审意见
• 在动手之前，会先复述需求
• 在反馈不完整或含糊时，会主动提问
• 在接受或质疑建议时，会引用具体文件、函数或代码行

如果你仍然经常看到没有核实就回复 “I’ll implement that” 之类的答案，请检查技能配置，确保在评审场景下已启用该技能。