代码审查请求Skill RequestingCodeReview

这个技能用于在软件开发过程中请求代码审查,通过分派代码审查子代理来验证代码实现是否符合计划和需求,确保代码质量,防止问题积累。关键词:代码审查、软件开发、质量保证、审查流程、子代理、代码复审、代码质量。

测试 0 次安装 0 次浏览 更新于 3/17/2026

名称:请求代码审查 描述:在继续之前,分派代码审查子代理以根据计划或要求审查实现 使用时机:完成任务、实施主要功能或合并之前,以验证工作符合要求 版本:1.2.0 渐进披露: 级别:1 参考资料: - 路径:references/code-reviewer-template.md 标题:代码审查模板 描述:完整的子代理模板,包含占位符和审查清单 - 路径:references/review-examples.md 标题:审查示例和工作流 描述:好与坏的审查示例,严重性指南,完整的工作流示例

请求代码审查

分派代码审查子代理以在问题级联前捕获它们。

核心原则: 早审查,常审查。

何时请求审查

强制性:

  • 在子代理驱动开发中的每个任务后
  • 完成主要功能后
  • 合并到主分支前

可选但有价值:

  • 当卡住时(新视角)
  • 重构前(基线检查)
  • 修复复杂bug后

快速开始

1. 获取git SHAs:

BASE_SHA=$(git rev-parse HEAD~1)  # 或 origin/main
HEAD_SHA=$(git rev-parse HEAD)

2. 分派代码审查子代理:

使用代码审查类型的任务工具,填写模板在代码审查模板

必需的占位符:

  • {WHAT_WAS_IMPLEMENTED} - 您刚刚构建的内容
  • {PLAN_OR_REQUIREMENTS} - 它应该做什么
  • {BASE_SHA} - 起始提交
  • {HEAD_SHA} - 结束提交
  • {DESCRIPTION} - 简要摘要

3. 根据反馈行动:

严重性 行动
关键 立即修复,不要继续
重要 在下个主要任务前修复
次要 记录以备后用,可以继续

详情见严重性指南 对于细节。

与工作流集成

子代理驱动开发:

  • 在每个任务后审查
  • 在问题复合前捕获
  • 在移动到下个任务前修复

执行计划:

  • 每批(3个任务)后审查
  • 获取反馈,应用,继续

临时开发:

  • 合并前审查
  • 卡住时审查

对审查提出异议

如果审查员错误:

  • 用技术推理提出异议
  • 显示证明它有效的代码/测试
  • 参考计划要求
  • 请求澄清

提出异议示例 对于示例。

常见错误

绝不:

  • 因为“它简单”而跳过审查
  • 忽略关键问题
  • 继续未修复的重要问题
  • 没有技术理由的争论

总是:

  • 在审查请求中提供完整上下文
  • 立即修复关键问题
  • 记录您不同意的原因(如果有)
  • 在修复关键问题后重新审查

示例

需要示例?审查示例和工作流 对于:

  • 完整的审查输出示例
  • 好与坏的审查请求
  • 不同场景的审查工作流
  • 如何处理不同严重性级别
  • 何时以及如何提出异议

需要模板?代码审查模板 对于完整的子代理分派模板。