mcpField
R
requesting-code-review

requesting-code-review

github.com/makerjackie测试与质量139 次安装36 周前更新
代码仓库

关于 requesting-code-review

面向开发 Agent 的代码审查工作流技能，帮助在问题累积前发现缺陷。它规定审查时机、如何调用 `superpowers:code-reviewer` 子 Agent，以及如何处理审查意见。

主要功能

  1. 01覆盖开发各阶段的审查节点
  2. 02调用 `superpowers:code-reviewer` 子 Agent
  3. 03提供提交 SHA 和实现背景
  4. 04确定反馈处理的优先级

适用场景

  1. 01审查子 Agent 开发中的每项任务
  2. 02合并前检查主要功能
  3. 03遇到阻碍时获取新视角
  4. 04重构前建立基线检查

安装

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

$npx skills add https://github.com/makerjackie/jackie-skills-starter --skill requesting-code-review

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

附带文件code-reviewer.md

详细介绍

（中文）请求代码审查

派遣 superpowers:code-reviewer 子 Agent，在问题连锁扩大前发现隐患。

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

何时请求审查

必须：

  • 在子 Agent 驱动开发中的每项任务完成后
  • 在完成主要功能后
  • 在合并到 main 之前

可选但很有价值：

  • 遇到阻碍时（获取新的视角）
  • 重构之前（检查基线）
  • 修复复杂 Bug 后

如何请求

1. 获取 git SHA：

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

2. 派遣 code-reviewer 子 Agent：

使用 Task 工具，类型设为 superpowers: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:code-reviewer subagent]
  WHAT_WAS_IMPLEMENTED: Verification and repair functions for conversation index
  PLAN_OR_REQUIREMENTS: Task 2 from docs/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]

与工作流的集成

子 Agent 驱动开发：

  • 在每项任务后进行审查
  • 在问题累积扩大前发现隐患
  • 在进入下一项任务前修复问题

执行计划：

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

临时开发：

  • 合并前进行审查
  • 遇到阻碍时进行审查

警示信号

切勿：

  • 因为“很简单”就跳过审查
  • 忽略严重问题
  • 带着未修复的重要问题继续
  • 与有效的技术反馈争辩

如果审查者有误：

  • 给出技术理由并提出异议
  • 提供能够证明其有效的代码/测试
  • 请求澄清

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

评论

支持 Markdown