mcpField
R
requesting-code-review

requesting-code-review

github.com/lucianghinda测试与质量43043 次安装今天更新
代码仓库

关于 requesting-code-review

一项指导开发者调用 superpowers-ruby:code-reviewer 子 Agent 的技能，提供聚焦于代码成果的说明和提交范围。它列出请求审查的时机，以及如何处理问题后再继续开发或合并。

主要功能

  1. 01任务完成、功能完成及合并前审查
  2. 02提供实现说明和提交 SHA
  3. 03按优先级处理审查意见
  4. 04在调试或重构时请求审查

适用场景

  1. 01逐项审查子 Agent 驱动开发的任务
  2. 02合并到 main 前检查功能
  3. 03为复杂错误寻求新视角

安装

复制下面的命令，在终端中运行。

$npx skills add https://github.com/lucianghinda/superpowers-ruby --skill requesting-code-review

安装后在 Claude Code 中输入/requesting-code-review确认已加载。

附带文件code-reviewer.md

详细介绍

请求代码审查

派遣 superpowers-ruby:code-reviewer 子代理，在问题扩散前发现它们。审查者会收到精心编写的评估上下文，而不会看到你的会话历史。这样审查者就能专注于工作成果，而非你的思考过程，也能为你保留上下文，以便继续工作。

核心原则： 尽早审查，频繁审查。

何时请求审查

必须：

  • 在子代理驱动开发中的每项任务完成后
  • 完成重大功能后
  • 合并到 main 之前

可选但很有价值：

  • 陷入困境时（获取新的视角）
  • 重构之前（检查基线）
  • 修复复杂 Bug 后

如何请求

1. 获取 git SHA：

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

2. 派遣 code-reviewer 子代理：

使用 Task 工具，选择 superpowers-ruby:code-reviewer 类型，并填写 code-reviewer.md 中的模板

占位符：

  • {WHAT_WAS_IMPLEMENTED} - 刚刚构建的内容
  • {PLAN_OR_REQUIREMENTS} - 应实现的功能
  • {BASE_SHA} - 起始提交
  • {HEAD_SHA} - 结束提交
  • {DESCRIPTION} - 简要摘要

3. 处理反馈：

  • 立即修复严重问题
  • 继续之前修复重要问题
  • 将次要问题记录下来，稍后处理
  • 如果审查者有误，请提出异议（并说明理由）

示例

[Just completed Task 2: Add verification function]

You: Let me request code review before proceeding.

BASE_SHA=$(git log --oneline | grep "Task 1" | head -1 | awk '{print $1}')
HEAD_SHA=$(git rev-parse HEAD)

[Dispatch superpowers-ruby:code-reviewer subagent]
  WHAT_WAS_IMPLEMENTED: Verification and repair functions for conversation index
  PLAN_OR_REQUIREMENTS: Task 2 from docs/superpowers/plans/deployment-plan.md
  BASE_SHA: a7981ec
  HEAD_SHA: 3df7661
  DESCRIPTION: Added verifyIndex() and repairIndex() with 4 issue types

[Subagent returns]:
  Strengths: Clean architecture, real tests
  Issues:
    Important: Missing progress indicators
    Minor: Magic number (100) for reporting interval
  Assessment: Ready to proceed

You: [Fix progress indicators]
[Continue to Task 3]

与工作流集成

子代理驱动开发：

  • 每项任务完成后都进行审查
  • 在问题累积前发现它们
  • 进入下一项任务前先修复问题

执行计划：

  • 每批任务（3 项）完成后进行审查
  • 获取反馈、采纳修改，然后继续

临时开发：

  • 合并前进行审查
  • 陷入困境时进行审查

危险信号

切勿：

  • 因为“很简单”就跳过审查
  • 忽视严重问题
  • 带着未修复的重要问题继续
  • 反驳合理的技术反馈

如果审查者有误：

  • 根据技术理由提出异议
  • 展示能够证明其有效的代码或测试
  • 请求澄清

参见模板：requesting-code-review/code-reviewer.md

评论

支持 Markdown