smithery/klaaay

superpowers-requesting-code-review

在完成任务、实施主要功能或合并前验证工作是否符合要求时使用

Installation

$ npx skills add smithery/klaaay --skill superpowers-requesting-code-review

Similar popular skills

Related neighbors and high-traction skills in the same topics — useful to compare before installing.

Also in this package

Other skills from smithery/klaaay.

npx skills add smithery/klaaay

Browse all from smithery/klaaay

More details

Agent compatibility

Declared targets from SKILL.md / docs. Unmarked agents are not listed — the skill may still install via the CLI.

Claude Code Not declared
Cursor Not declared
Codex Not declared
GitHub Copilot Not declared
Windsurf Not declared
Gemini CLI Not declared
Cline Not declared
OpenCode Not declared

Package contents

Files included with this skill beyond the listing page.

  • skill md SKILL.md 2,744 B
  • docs SUMMARY.md 129 B

History

  1. First recorded snapshot · 0 installs

SKILL.md

发起代码评审

分派 code reviewer 子代理,在问题级联扩散前尽早发现它们。Reviewer 拿到的是你精确构造的上下文,而不是你整段会话历史;这样既能让 review 聚焦于工作产物,也能保留你的主上下文用于继续推进。

核心原则: 尽早审查,经常审查。

何时请求 Review

必须:

  • subagent-driven development 的每个任务后
  • 完成主要功能后
  • 合并到主分支前

可选但有价值:

  • 卡住时,获得新视角
  • 重构前,先建立基准
  • 修复复杂 bug 后

如何请求

1. 获取 git SHA:

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

2. 分派 code reviewer 子代理:

分派 general-purpose 子代理,并填写 [code-reviewer.md](code-reviewer.md) 模板。

占位符:

  • {DESCRIPTION} - 简要说明刚完成的内容
  • {PLANORREQUIREMENTS} - 它应该满足什么要求
  • {BASE_SHA} - 起始提交
  • {HEAD_SHA} - 结束提交

3. 根据反馈行动:

  • 立即修复 Critical issues
  • 继续前修复 Important issues
  • 记录 Minor issues,稍后处理
  • 如果 reviewer 判断错误,用技术推理反驳

示例

[刚完成任务 2:添加验证函数]

你:继续前先请求一次 code review。

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

[分派 code reviewer 子代理]
  DESCRIPTION: 添加 verifyIndex() 和 repairIndex(),覆盖 4 种问题类型
  PLAN_OR_REQUIREMENTS: docs/superpowers/plans/deployment-plan.md 中的任务 2
  BASE_SHA: a7981ec
  HEAD_SHA: 3df7661

[子代理返回]:
  Strengths: 架构清晰,测试验证真实行为
  Issues:
    Important: 缺少进度指示器
    Minor: report interval 使用 magic number (100)
  Assessment: 可以继续

你:[修复进度指示器]
[继续任务 3]

与工作流的衔接

Subagent-Driven Development:

  • 每个任务结束后 review
  • 在问题累积前拦截
  • 修复后再进入下一个任务

Executing Plans:

  • 每个任务后或自然检查点做 review
  • 拿到反馈,修正后继续

临时开发:

  • 合并前 review
  • 卡住时也可以 review

红旗

绝不要:

  • 因为“很简单”跳过 review
  • 忽略 Critical issues
  • 在 Important issues 未修复时继续
  • 对有效技术反馈硬拗

如果 reviewer 错了:

  • 用技术推理反驳
  • 展示能证明实现有效的代码或测试
  • 请求澄清

模板见:[code-reviewer.md](code-reviewer.md)