Agent Skills
返回列表
💻

卓越代码审查

开发编程 更新于 2026.08.30

将以下提示词粘贴到你的 AI 对话框中：

请根据 https://skillhub.cn/install/skillhub.md，安装 @org-02qudk26/code-review-excellence-zh。

技能介绍

审查容易变成「挑刺」或「盖章」

很多团队的 PR 审查要么流于形式（只看能不能 merge），要么变成个人偏好之争，缺少对正确性、安全性、性能和可维护性的一致标准。审查者往往没有先对齐需求与测试信号，反馈也缺少严重程度与理由，作者难以判断该先改什么。

技能如何组织一次审查

使用本技能时，应先阅读上下文、需求与测试相关信号，再按维度做系统检查，并输出可操作的结论：

  • 发现摘要：用一段话概括变更风险与整体质量
  • 按严重程度分组：阻塞、重要、次要，每条附理由
  • 建议与疑问：意图不清时主动提问，而不是臆测
  • 测试与覆盖率：说明现有测试是否覆盖变更、还缺什么

需要逐项清单时，可参照技能内的 resources/implementation-playbook.md 中的审查模式与模板。

适用边界

  • 适合：审查拉取请求与代码变更、起草或统一审查标准、用反馈带新人、做正确性/安全/性能审计
  • 不适合：没有可审的代码 diff、纯架构/产品讨论且无实现、你的任务是直接改代码而不是给出审查意见

使用场景

  • 合并前审查同事提交的 PR，按阻塞/重要/次要给出带理由的修改清单
  • 为新项目起草团队统一的代码审查标准与合并门槛说明
  • 对涉及鉴权或数据处理的变更做正确性与安全性专项检查
  • 在审查意见中附带测试与覆盖率缺口，要求作者补测后再合并

适合人员

  • 负责合并策略的技术负责人，希望审查结论可比较、可复用
  • 经常审安全敏感模块的资深工程师，需要系统化检查清单而非凭感觉挑错
  • 刚接手小组的 Team Lead，想通过审查反馈统一团队对可维护性的预期
  • 开源或内部库维护者，要在大量小 PR 里快速识别性能与正确性风险