Structured Code Review Checklist Agent Skill

Performs structured, read-only code and diff reviews across correctness, edge cases, security, maintainability, and test coverage.

Context cost
700 tokensestimated from the bundle, loaded when it triggers
Bundle
1 file · 2.8 kBtext throughout, nothing executable
Licence
MITthe author’s own terms
Last change
—no release on file
Servers it uses
Noneruns standalone

What it does

对一段 diff 或改动做结构化审查,按正确性、边界情况、安全、可读性、测试覆盖分类给出具体发现,而不是笼统的"看起来不错"。当用户说"帮我审查一下这段代码"、"看看这个 PR 有没有问题"、"这段改动能合并吗"、"帮我 review 一下"时使用。只做只读审查,不直接修改代码;如果用户想要"审查完顺便把问题改了",先完成审查、列出发现,再单独确认是否要动手改。

What this skill changes about the agent has not been set out here as a list. Its SKILL.md, shown below, says it in the author’s own words.

code reviewcollaborationsecurity

The skill itself

A skill is instructions the model reads, so you can see what you are getting: below is the start of its SKILL.md. The front matter loads with every session, the body loads when the skill triggers, and the whole file comes with the install.

SKILL.md2.8 kB · preview
--- name: code-review-checklist ---
6# code-review-checklist
7
8## 目标
9
10产出具体、可核实的审查发现——每条发现都要能回答"在什么输入/场景下,会出什么错",而不是空泛的风格意见。找不到真实问题时如实说"没有发现明显问题",不要为了显得"审查得很认真"硬凑问题。
11
12## 审查步骤
13
141. **理解改动的意图**:先看清楚这段改动想解决什么问题、涉及哪些文件,再逐处检查,不要孤立地看单个文件而不管上下文。
152. **按以下五个维度过一遍,不是每个维度都会有发现,跳过没问题的维度**:
16 - **正确性**:逻辑是否符合意图?有没有明显的笔误、条件写反、off-by-one?
17 - **边界情况**:空输入、null/undefined、极大极小值、并发/竞态、网络/IO 失败时会怎样?
18 - **安全**:有没有引入注入、越权访问、敏感信息泄露、不受信输入未校验直接使用等问题?
19 - **可读性与维护性**:命名是否清楚?有没有重复到应该抽取的逻辑?复杂逻辑有没有必要的说明?(风格类的小事,比如空格、引号统一,除非项目有明确规范否则不用纠结)
20 - **测试覆盖**:新增/改动的行为有没有对应测试?测试是否真的验证了行为而不是形式上凑数?
213. **每条发现都要包含**:具体位置(文件名/行号或代码片段)、问题描述、触发条件(什么情况下会出问题)、以及可能的修复方向。
224. **分级**:区分"会导致错误行为的问题"和"值得改进但不影响正确性的建议",让用户知道哪些必须处理、哪些可以自行取舍。
23
24## 输出格式
This is a preview of SKILL.md. Download or install the skill to get the whole thing.
In the file
SKILL.md53 words
Files1
LicenceMIT
Why you can read it

Nothing in a skill executes. The client loads the text and the model follows it, so a skill can be audited the way a runbook is — by reading it.

What it costs in context

Skills are not billed by the call. They are paid for in context: every token the instructions occupy is a token your code, your diff and your conversation cannot use. Here is what this one takes and when it takes it.

≈60
always loaded
The name and description, so the model knows the skill exists and when to reach for it.
640
on trigger
The instruction body, read only when the skill fires.
0.35%
of a 200k window
Ten skills this size would take about 4% of the window before you open a file.
050k100k150k200k context window

700 tokens, estimated from the bundle at four bytes to the token, held for the rest of the session once it triggers. Small enough to keep loaded permanently without thinking about it.

Servers bill, skills cost

A server charges by the month. A skill charges once per session, in context, and then keeps charging it for as long as the session lives.

Before and after

The same question, put to the same model twice: once as it comes, and once with these instructions loaded.

No worked example has been published for this skill yet.

The procedure it runs

The procedure has not been set out here step by step. It is in the body of its SKILL.md: the start of it is shown above, and the whole file comes with the install.

Prose, not code

These steps are written for a model to follow, not executed by a runtime. It can still be told to skip one, and it will say so when it does.

Servers it uses

None. This skill calls no MCP servers at all.

Everything it needs is in the instructions, so it works in a project with nothing connected — the model reads the file and changes how it works with what it can already reach.

Its SKILL.md does not declare which tools it asks the agent for, so what it touches is whatever the instructions themselves ask for — read them before you install it.

What it asks for
Writes filesnot declared
Network accessnot declared

This SKILL.md carries no allowed-tools line, so neither is declared. A skill grants no permissions of its own — it can only ask for tools your client already has.

What it will not do

Every skill is narrow, and the useful ones say where they stop. These are the jobs this one is the wrong tool for.

What this skill is not for has not been published here. Nothing is implied by that: it is a section the author has not filled in.

What is in the bundle

1 file, 2.8 kB on disk. A bundle is text throughout: the instructions the model reads, plus the templates it fills in.

  • SKILL.md2.8 kB
What is not in it

No dependencies and nothing executable: a skill is text the agent reads, so the bundle is 1 file you can review in full before installing. The MIT licence covers the templates and examples as well as the instructions.

Install

Installing copies the bundle into your project. Nothing runs at install time — the files sit on disk until the model reads them.

# Structured Code Review Checklist · 700 tokens when loaded · macOS, Linux mkdir -p .claude/skills && t=$(mktemp -d) && curl -fsSL -o "$t/bundle.tar.gz" https://mcprush.com/api/skills/structured-code-review-checklist/bundle.tar.gz && tar -xzf "$t/bundle.tar.gz" -C "$t" && [ -d "$t/structured-code-review-checklist" ] && { [ ! -e .claude/skills/structured-code-review-checklist ] || { mv .claude/skills/structured-code-review-checklist "$t/.previous" && echo "Previous copy moved to $t/.previous"; }; } && mv "$t/structured-code-review-checklist" .claude/skills/ && echo "Installed .claude/skills/structured-code-review-checklist"
On Windows, or any OS with Node 20+
npx mcprush@latest skill add fengyan3141/structured-code-review-checklist

Windows PowerShell says 'running scripts is disabled on this system'? Run Set-ExecutionPolicy -Scope CurrentUser RemoteSigned once, or type npx.cmd instead of npx (codex.cmd, gemini.cmd, grok.cmd likewise). PowerShell 7 needs neither. Writes to .claude/skills/structured-code-review-checklist/ in the current project. For every project instead, run mkdir -p ~/.claude/skills && t=$(mktemp -d) && curl -fsSL -o "$t/bundle.tar.gz" https://mcprush.com/api/skills/structured-code-review-checklist/bundle.tar.gz && tar -xzf "$t/bundle.tar.gz" -C "$t" && [ -d "$t/structured-code-review-checklist" ] && { [ ! -e ~/.claude/skills/structured-code-review-checklist ] || { mv ~/.claude/skills/structured-code-review-checklist "$t/.previous" && echo "Previous copy moved to $t/.previous"; }; } && mv "$t/structured-code-review-checklist" ~/.claude/skills/ && echo "Installed ~/.claude/skills/structured-code-review-checklist" (macOS, Linux) — or add --global to the npx line. The curl line replaces .claude/skills/structured-code-review-checklist/ only after the download succeeds, so files left there by another skill of the same name, or by an older release, do not mix in. A copy already there is moved aside, not deleted, and the line prints where. It appears as code-review-checklist in Gemini CLI, Codex, Copilot and Grok (its own name); Claude Code lists it by the folder name, structured-code-review-checklist.

Which clients pick it up on their own

A skill is a folder of text. Most of these read a skills folder and pick it up without being told; the rest take the same folder another way — an upload in the app, a parameter on an API call — and then choose it by its description just the same.

Claude Code.claude/skills/
Claude DesktopCustomize › Skills
ChatGPTSkills › Create › Upload
Cursor.cursor/skills/
VS Code.github/skills/
Codex CLI.agents/skills/
Gemini CLI.gemini/skills/
Grok Build.grok/skills/
Microsoft CopilotSkills › Upload a skill
PerplexityComputer › Skills › Upload a skill
DeepSeek Harness.dsh/skills/
Zed.agents/skills/
Devin Desktop.devin/skills/
Windsurf (legacy).windsurf/skills/
OpenAI Agents SDKInstructions or sandbox Skills
HTTP / APIa request parameter
This release
Versionnot versioned
Publishedno release date on file
PriceFree
Referencefengyan3141/structured-code-review-checklist

Versions

Its author publishes no version number, so there is nothing here to pin to: what you install is the folder as it stands today. Instructions change more often than APIs do — a skill can be rewritten entirely without anything it depends on moving.

Pinning

Nothing to pin to: this skill carries no version number of its own. What you install is what the folder holds on the day you install it.

Reviews

—no reviews yet

Nobody has reviewed this skill yet. The rating is the mean of the reviews written here, so there is none until somebody writes the first.

Sign in to review it

A review is tied to an account so it can be answered, edited and taken down by the person who wrote it.

Sign in
Who can post

Anybody with an account, one review each. It carries the version that was current when you wrote it, and the publisher may reply once.

Publisher
Servers0
Claim this skill