Requesting code review

Skill yinqd3/workbuddy-skills/superpowers/requesting-code-review

A curated collection of WorkBuddy agent skills — academic pipeline, frontend slides, PPTX generator, tool-call repair, and more.

Install

npx -y skills add yinqd3/workbuddy-skills --skill requesting-code-review

Assembled from the repository path, not quoted from the project. Check it against their README if it does not work.

One thing to look at

4 stars4 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.

What its author says it does

Copied from the file, not written here

代码变更完成后请求审查。派发规范审查+代码质量审查两个子代理，汇总结果。触发词：审查代码、review、code review、检查代码。

SKILL.md

2.0 KB, as published. Nobody here has run it

请求代码审查

概述

代码变更完成后请求代码审查。使用 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：修复与重新审查

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

审查严重程度

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