code-review

A code-review procedure that checks changed code by running available tests and type checks, then examining correctness, security, performance, style, and maintainability.

In plain words
What is it for?
Use it to review a change set, verify its actual behavior, and produce a report with required, suggested, and optional fixes.
Why use it?
It helps find bugs and risky changes before they reach users, while making clear how serious each issue is.

Skill for Claude CodeCodex

Install

Getting it into your agent

One page per mod, every tool's command on it. A separate URL per tool would split the same page into five that compete with each other.

agentmods
npx agentmods add skills/today-hbw/claude-code-dev-workflow/code-review
Any agent
npx skills add Today-Hbw/claude-code-dev-workflow --skill code-review
Clone the repo
git clone --depth 1 https://github.com/Today-Hbw/claude-code-dev-workflow

Made for: Claude Code, Codex.

Per session 22 Skills are progressive disclosure: only the name and description are preloaded; the body loads when the skill is used.
When invoked 1,172 The whole file, excluding the scripts and references it only reads on demand.
Security scan A 0 findings. Scan, not verified.
Origin original No closer match found in the catalogue.
Token cost

What it costs to keep this loaded

Counted locally with the o200k_base tokenizer, which is exact for GPT models; Claude uses its own tokenizer and its counts differ. Treat this as one consistent yardstick across the catalogue rather than a bill. Prices are per million input tokens.

ModelPer sessionOnce invoked
Fable 5 $0.00022 $0.01172
Opus 5 $0.00011 $0.00586
Sonnet 5 $0.00004 $0.00234
Haiku 4.5 $0.00002 $0.00117

Measured 2d ago against content hash c22803b17d5a, method: parsed. Prices are Anthropic first-party input rates as of 2026-08-30, from the pricing page.

Security

Grade A, and why

code-review scanned grade A with 0 findings against 26 rules in 11 categories — prompt injection, anti-refusal, data exfiltration, privilege escalation, supply chain, agent snooping, system-prompt leakage, SSRF and excessive agency — measured 2d ago.

A static scan of the body, not an audit. Every finding is printed with the line that produced it so you can judge whether it matters here. A mod is markdown that instructs an agent; that is exactly why what it instructs is worth reading.

Nothing flagged

None of the 26 patterns this scan looks for appear in this file: no shell pipes, no recursive deletes, no credential paths, no hidden text, no instruction-override or anti-refusal phrasing, no agent-config snooping. That is not a guarantee, it is the absence of the things that are checkable.

plugins/dev-flow-lite/skills/code-review/SKILL.md · 142 lines

What it actually says

Code Review · 审查方法

你的任务

审查步骤 3 产出的所有代码变更,从多个维度检查质量。审查前先验证代码实际行为

审查流程

0. 审查前验证

在静态审查之前,先验证代码的实际运行行为:

  1. 检测测试配置

    • 用 Glob 检查项目根目录有无 package.json
    • 有则读取 scripts.test 字段,确认测试命令
    • 也检查有无 pytestgo testcargo test 等其他测试配置
  2. 运行测试(如有测试配置):

    • 执行测试命令,记录结果
    • 将测试通过/失败/跳过数量记入审查报告
    • 如有失败的测试,先修复再进入审查
  3. 运行类型检查(如有配置):

    • 检查有无 typecheck 脚本,或项目使用 tsc --noEmit
    • 记录类型检查结果
  4. 无测试配置时

    • 标注"项目无测试配置,仅做静态审查"
    • 不阻塞审查流程

1. 查看变更文件

git diff 或 Read 查看所有变更文件。

2. 逐文件审查

按以下 6 个维度检查每个文件。

3. 标注严重级别

对每个问题标注级别:

  • 🔴 必须修复:逻辑错误、安全问题
  • 🟡 建议修复:性能问题、规范问题
  • 🔵 可选:代码风格、小优化

4. 在对话中输出审查报告

审查维度

1. 正确性(最重要)

  • 逻辑是否正确(对照对话中确认的验收标准;验收标准里的视觉规格项——列归属/文案/合并/顺序/所属页签——同样逐条核对,见维度 6)
  • 边界条件是否处理
  • 空值/异常是否防御
  • 数据计算是否正确

2. 边界与安全

  • SQL 是否有注入风险
  • 用户输入是否校验
  • 权限控制是否正确
  • 敏感数据是否暴露

3. 性能

  • 是否有 N+1 查询
  • 大数据量场景是否分页
  • 是否有不必要的循环或重复计算

4. 规范

  • 命名是否符合项目规范
  • 代码风格是否一致
  • 注释是否充分
  • 文件结构是否合理

5. 可维护性

  • 代码是否易读
  • 是否有重复代码
  • 是否过度设计
  • 是否容易测试

6. 视觉规格落地(仅当验收标准含视觉规格项)

对照 PRD 截图/原型逐条比对实现:

  • 字段列归属是否一致(该折进已有列的没做成独立列)
  • 按钮/菜单文案是否与 PRD 图逐字一致(不自拟、不增减字)
  • 入口所属页签/区域是否一致(不新建 PRD 图未要求的页签)
  • 筛选项、字段的位置与顺序是否与图一致

凡 PRD 图已给出的位置/文案视为硬性验收项,不符即 🔴。

输出格式

在对话中按以下格式输出审查报告:

## 代码审查报告

### 🧪 运行结果
- 测试:N 通过 / M 失败 / K 跳过
- 类型检查:通过/失败
- (或"项目无测试配置,仅做静态审查")

### 🔴 必须修复(N 项)

1. **<文件:行号>**:<问题描述>
   - 原因:<为什么是问题>
   - 建议:<修复方案>

### 🟡 建议修复(N 项)

1. **<文件:行号>**:<问题描述>
   - 建议:<改进方案>

### 🔵 可选优化(N 项)

1. **<文件:行号>**:<问题描述>

### 总结
- 必须修复:N 项
- 建议修复:N 项
- 可选优化:N 项
- 整体评价:<一句话总结>

注意事项

  • 如果有 🔴 必须修复项,回到步骤 3 修复后重新审查
  • 不要纠结于代码风格问题(那是 linter 的事);但 PRD 图已定死的界面位置/文案属必查项、非风格问题,实现与图不符即 🔴
  • 重点关注逻辑正确性和安全问题
  • 审查意见要具体到文件和行号,不要泛泛而谈
  • 运行结果是审查报告的第一部分
Changes

What this file has done since we first saw it

Hashed on every crawl. A supply-chain change to an agent config is a question of when, not whether, so the history is kept rather than the latest state alone.

  1. 2d ago First seen · 142 lines · 22 tokens per session scan A c22803b17d5a

Subscribe to this mod's changes

code-review is a skill published in the GitHub repository Today-Hbw/claude-code-dev-workflow (22 stars, last pushed 21d ago), licensed MIT. It adds 22 tokens to every session and 1,172 once invoked, about $0.0001 per session on Opus 5. A static security scan graded it A with 0 findings. No closer match exists in the catalogue, so it is treated as the original; first seen 2026-08-30.

Related

Other skills, from other repositories

metrics-instrumentation

Specification for instrumenting an opik-backend workflow with operational OpenTelemetry metrics — per-stage throughput/latency/error counters and native histograms, dimensioned per-customer (workspace). Use when a pipeline (scoring, ingestion, experiments, jobs) needs per-stage visibility. Covers metric emission only…

comet-ml/opik · 93 tokens

happiness-skill

当用户问「怎么才能更幸福/为什么得到了还不满足/怎么减少焦虑」时调用。 核心理念: 幸福是缺憾感清空的默认状态, 是可训练的技能; 欲望是与自己的契约(得到前不快乐), 同时只留一个重大欲望; 活在当下。 不适用于: 临床抑郁等需要专业治疗的场景(本书方法不能替代医疗)。 Triggers: 幸福/不快乐/欲望/焦虑/知足/活在当下/happiness/desire/anxiety.

kangarooking/cangjie-skill · 136 tokens

short-drama-storyboard

把剧本和视觉事实转成有镜头职责、空间连续性和可冻结起点的 剧集/ /分镜.md。 每镜使用二级标题 ## SHOT-...,同镜下用 ### 冻结关键帧提示词 写起始帧正文。.

zenstory-ai/drama-skills · 102 tokens

seedance-prompt

This skill should be used when the user asks to write, improve, translate, compress, or debug a Seedance 2.0 video prompt; mentions T2V, I2V, V2V, R2V, camera direction, prompt quality, or provides reference assets for a production-ready prompt.

Emily2040/seedance-2.0 · 68 tokens

setup-matt-pocock-skills

为本仓库配置工程技能——设置其 issue tracker、分诊标签词汇表和领域文档布局。首次使用其他工程技能前运行一次。.

devcxl/mattpocock-skills-zh · 43 tokens

deep-review

Parallel competency-based code review. Launches independent Agent reviewers per competency (security, performance, architecture, database, concurrency, error-handling, frontend, testing), each with a focused checklist and isolated context. Synthesizes findings into unified report with FIX/DEFER/ACCEPT triage. Use…

AnastasiyaW/codex-claude-code-config · 166 tokens