界面已本地化;技能介绍与说明文档按所选语言由 AI 自动翻译,代码、命令、路径和专业术语保留原文,以原作者文档为准。
探索技能
开发与工程 · SKILL.md

requesting-code-review

requesting-code-review

【开发与工程】代码变更完成后请求审查。派发规范审查+代码质量审查两个子代理,汇总结果。触发词:审查代码、review、code review、检查代码。

独立 GitHub 快照 · 自动分类 · 以原作者文档为准

技能说明

GitHub

name: requesting-code-review description: 代码变更完成后请求审查。派发规范审查+代码质量审查两个子代理,汇总结果。触发词:审查代码、review、code review、检查代码。 agent_created: true

请求代码审查

概述

代码变更完成后请求代码审查。使用 WorkBuddy Agent 工具派发审查子代理。

开始时声明: "我将用 requesting-code-review 技能审查代码变更。"

审查流程

代码变更 → 规范审查子代理 → 代码质量审查子代理 → 汇总报告 → 修复 → 重新审查

步骤 1:准备审查上下文

# 获取变更范围
git diff --stat <base>..<head>
git diff <base>..<head>

步骤 2:派发规范审查子代理

使用 Agent 工具派发 general-purpose 子代理:

你是做规范合规审查的子代理。只审查实现是否匹配计划中的规格要求。

计划文件: [路径]
GIT 范围: [base..head]

审查:
- 每个规格要求都有实现吗?
- 实现的行为和规格描述一致吗?
- 有规格没要求但实现了的功能吗?(YAGNI 违反)

报告:
- PASS 或 FAIL
- 任何规格遗漏或偏差
- 不要评论代码风格或质量

步骤 3:派发代码质量审查子代理

你是做代码质量审查的子代理。

GIT 范围: [base..head]
描述: [变更概述]

审查:
- DRY 违反(重复代码)
- 死代码(永远不会执行的代码)
- YAGNI 违反(过度工程)
- 命名问题
- 缺少错误处理
- 测试覆盖不足

严重程度: 严重 / 重要 / 次要
报告: 批准,或按严重程度列出问题。

步骤 4:汇总结果

将两个审查报告汇总给用户,标注严重程度和处理建议。

步骤 5:修复与重新审查

  • 严重/重要问题必须修复
  • 修复后重新审查变更部分
  • 全部通过后标记审查完成

审查严重程度

严重程度行动
严重阻塞合并,必须修复
重要应该修复,强烈建议
次要可以改进,不阻塞
我的收藏