superpowers-requesting-code-review

Use when completing tasks, implementing major features, or before merging - dispatches code review subagent to catch issues before they cascade, adapted for OpenClaw sessions_spawn model

3,891 stars

Best use case

superpowers-requesting-code-review is best used when you need a repeatable AI agent workflow instead of a one-off prompt.

Use when completing tasks, implementing major features, or before merging - dispatches code review subagent to catch issues before they cascade, adapted for OpenClaw sessions_spawn model

Teams using superpowers-requesting-code-review should expect a more consistent output, faster repeated execution, less prompt rewriting.

When to use this skill

  • You want a reusable workflow that can be run more than once with consistent structure.

When not to use this skill

  • You only need a quick one-off answer and do not need a reusable workflow.
  • You cannot install or maintain the underlying files, dependencies, or repository context.

Installation

Claude Code / Cursor / Codex

$curl -o ~/.claude/skills/openclaw-requesting-code-review/SKILL.md --create-dirs "https://raw.githubusercontent.com/openclaw/skills/main/skills/axelhu/openclaw-requesting-code-review/SKILL.md"

Manual Installation

  1. Download SKILL.md from GitHub
  2. Place it in .claude/skills/openclaw-requesting-code-review/SKILL.md inside your project
  3. Restart your AI agent — it will auto-discover the skill

How superpowers-requesting-code-review Compares

Feature / Agentsuperpowers-requesting-code-reviewStandard Approach
Platform SupportNot specifiedLimited / Varies
Context Awareness High Baseline
Installation ComplexityUnknownN/A

Frequently Asked Questions

What does this skill do?

Use when completing tasks, implementing major features, or before merging - dispatches code review subagent to catch issues before they cascade, adapted for OpenClaw sessions_spawn model

Where can I find the source code?

You can find the source code on GitHub using the link provided at the top of the page.

Related Guides

SKILL.md Source

# Superpowers Requesting Code Review(OpenClaw 适配版)

## 概述

在问题级联前派发代码审查 subagent 捕获问题。审查者获得精确构造的上下文做评估——永远不是你 session 的历史。这保持审查者专注在工作产出上,而非你的思维过程。

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

## OpenClaw 适配

Superpowers 原版使用 `Task` 工具派发 code-reviewer subagent。OpenClaw 用 `sessions_spawn`:
- 派发前准备审查上下文(git diff、变更摘要、规格引用)
- 用 `sessions_spawn(mode="run")` 派发一次性审查 session
- 审查结果通过 session 历史或文件系统返回

## 何时请求审查

**强制:**
- subagent-dev 中每个任务完成后
- 完成重大功能后
- Merge 到 main 前

**可选但有价值:**
- 卡住时(新鲜视角)
- 重构前(基线检查)
- 修复复杂 bug 后

## 如何请求

### 1. 收集上下文

```bash
# 获取 git diff
git diff BASE_SHA HEAD > /tmp/review-diff.patch
git log --oneline BASE_SHA..HEAD

# 获取变更统计
git diff --stat BASE_SHA HEAD
```

### 2. 准备审查 prompt

审查 prompt 应包含:
- **实现了什么** — 刚刚构建的内容
- **计划/需求** — 应该做什么
- **基准 SHA** — 起始 commit
- **头部 SHA** — 结束 commit
- **变更描述** — 简要总结

### 3. 派发审查 subagent

```javascript
sessions_spawn({
  task: `代码审查请求

实现了什么:
[具体描述刚完成的工作]

规格/需求:
[任务来自哪个计划,相关规格要求]

变更:
[git diff 关键内容或指向 diff 文件的路径]

请审查:
1. 规格合规——实现是否满足需求?
2. 代码质量——DRY、命名、测试设计
3. 潜在问题——bug、边界情况、安全

返回:strengths、issues(按 severity 分类)、assessment`,
  runtime: "subagent",
  mode: "run",
  cwd: "/path/to/project"
})
```

### 4. 处理反馈

- 立即修复 Critical 问题
- 继续前修复 Important 问题
- 记录 Minor 问题供以后
- 如果审查者错了——用技术理由反驳

## 审查关注点

### 规格合规
- 每个计划需求有对应实现吗?
- 没有做多余的东西?
- 边界情况覆盖了吗?

### 代码质量
- DRY(不要重复自己)?
- 命名清晰有意义?
- 测试覆盖好?
- 没有明显的性能问题?

### 问题严重度

| 级别 | 含义 | 行动 |
|------|------|------|
| **Critical** | 破坏功能或安全 | 立即修复 |
| **Important** | 重要但非破坏 | 继续前修复 |
| **Minor** | 风格/改进建议 | 记录可选做 |

## Red Flags

**永远不要:**
- 因为"简单"就跳过审查
- 忽略 Critical 问题
- 有未修复 Important 问题时继续
- 用合理技术反驳有效反馈

**如果审查者错了:**
- 用技术理由反驳
- 展示证明它工作的代码/测试
- 请求澄清

## 与工作流集成

**Subagent 驱动开发:**
- 每个任务**后**审查
- 在问题级联前捕获
- 继续前修复

**顺序执行:**
- 每个批次(3 个任务)后审查
- 获得反馈,应用,继续

**临时开发:**
- Merge 前审查
- 卡住时审查

Related Skills

Post-Mortem & Incident Review Framework

3891
from openclaw/skills

Run structured post-mortems that actually prevent repeat failures. Blameless analysis, root cause identification, and action tracking.

DevOps & Infrastructure

Pitch Deck Reviewer

3891
from openclaw/skills

Reviews pitch decks and provides investor-ready feedback with scoring

Business Strategy & Growth

Performance Review Engine

3891
from openclaw/skills

> Your AI-powered performance management system. Write reviews that develop people, not just evaluate them. From self-assessments to 360° feedback to calibration — complete frameworks for every review cycle.

Workflow & Productivity

Deal Desk — Structured Deal Review & Approval

3891
from openclaw/skills

Run every non-standard deal through a repeatable review process. Catch margin leaks, enforce discount guardrails, and close faster with pre-approved terms.

Contract Review Assistant

3891
from openclaw/skills

Analyze business contracts for risks, unfavorable terms, and missing clauses. Get a plain-English summary of what you're signing.

Legal & Finance

afrexai-code-reviewer

3891
from openclaw/skills

Enterprise-grade code review agent. Reviews PRs, diffs, or code files for security vulnerabilities, performance issues, error handling gaps, architecture smells, and test coverage. Works with any language, any repo, no dependencies required.

Coding & Development

performance-review-cn

3891
from openclaw/skills

绩效面谈报告、OKR对齐度检测、校准辅助

Workflow & Productivity

clawdtm-review

3891
from openclaw/skills

Review and rate OpenClaw skills on ClawdTM. See what humans and AI agents recommend.

General Utilities

cyber-owasp-review

3891
from openclaw/skills

Map application security findings to OWASP Top 10 categories and generate remediation checklists. Use for normalized AppSec review outputs and category-level prioritization.

Security

Contract Reviewer - AI Legal Document Risk Scanner

3891
from openclaw/skills

Upload any contract or legal document and get a structured risk analysis with flagged clauses, plain-language explanations, and negotiation suggestions.

serde-code-review

3891
from openclaw/skills

Reviews serde serialization code for derive patterns, enum representations, custom implementations, and common serialization bugs. Use when reviewing Rust code that uses serde, serde_json, toml, or any serde-based serialization format. Covers attribute macros, field renaming, and format-specific pitfalls.

rust-testing-code-review

3891
from openclaw/skills

Reviews Rust test code for unit test patterns, integration test structure, async testing, mocking approaches, and property-based testing. Use when reviewing _test.rs files,