mcpField
P
peer-review-initiator

peer-review-initiator

github.com/qodex-ai测试与质量47131 次安装42 周前更新
代码仓库

关于 peer-review-initiator

peer-review-initiator 是用于在开发关键节点发起代码审查的技能,说明如何调用 superpowers:code-reviewer 子 Agent 并处理反馈。

主要功能

  1. 01列出必需和可选的审查时机
  2. 02提供获取 Git SHA 的步骤
  3. 03使用模板派发审查任务
  4. 04按优先级处理审查意见

适用场景

  1. 01逐项审查子 Agent 开发任务
  2. 02合并前检查主要功能
  3. 03遇到难题时寻求新的视角

安装

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

$npx skills add https://github.com/qodex-ai/ai-agent-skills --skill peer-review-initiator

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

附带文件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