# Code Review

> 完成一块功能、合并前、或要给/收代码评审时用。做评审按「正确性→合需求→安全→简洁→风格」的优先级看；收评审先核实再改、不盲从也不表演式同意。

- **Type:** Skill
- **Install:** `agentstack add skill-chipfighter-coding-discipline-code-review`
- **Verified:** Yes — security-reviewed for prompt injection and unsafe behavior
- **Seller:** [chipfighter](https://agentstack.voostack.com/s/chipfighter)
- **Installs:** 0
- **Category:** [AI & ML](https://agentstack.voostack.com/c/ai-and-ml)
- **Latest version:** 0.1.0
- **License:** MIT
- **Upstream author:** [chipfighter](https://github.com/chipfighter)
- **Source:** https://github.com/chipfighter/coding-discipline/tree/main/plugins/coding-discipline/skills/code-review

## Install

```sh
agentstack add skill-chipfighter-coding-discipline-code-review
```

Requires the [AgentStack CLI](https://agentstack.voostack.com/docs/cli). Works with Claude Code, Cursor, and any MCP-compatible agent.

## About

## 做评审（自己审或派子 agent 审）
派一个子 agent 当评审更好——给它**精心裁过的上下文**（改了啥、本该满足啥需求、diff 范围 base..head），别把整个 session 历史倒给它；这样它盯产物不盯你的思路，也省你自己的上下文。

按这个**优先级**看，高的没过就别纠结低的：
1. **正确性**：逻辑对不对、边界/错误/并发有没有漏、会不会崩。
2. **合需求**：逐条对着需求/方案勾，做的是不是要的、有没有缺。
3. **安全**：注入、越权、密钥泄露、未校验输入。
4. **简洁**：过度设计 / 重复 / 可删的死代码（YAGNI）。
5. **风格**：命名、一致性、可读性——最后看，别一上来挑这个。

分轻重：拦路的（崩/安全）立刻修，重要的往下走之前修，小的记下回头弄。

## 收评审
- **先核实再改**：这条对**本**代码库成立吗？会弄坏现有功能吗？现在这么写是不是有原因？拿不准就说"我没法验证 X，要我查/问/还是先放着"。
- **不盲从**：建议错了、缺上下文、违反 YAGNI、和既定架构决定冲突——用**技术理由**顶回去，别因为是 reviewer 说的就改。涉及架构的冲突先停下找人。
- **不表演**：禁"你说得对！""好建议！""谢谢指出！"。改了就直说改了啥、在哪改；顶错了就一句"查了确实我错，因为 X，改"，别长篇道歉。
- 不清楚就**先把所有条目问清再动手**，别只挑懂的几条做（条目常相关，半懂=做错）。
- 一次改一条、各自测、确认没引入回归（见 verify-before-done）。

## Source & license

This open-source skill is cataloged on AgentStack and links to its original source — we do not rehost the code.

- **Author:** [chipfighter](https://github.com/chipfighter)
- **Source:** [chipfighter/coding-discipline](https://github.com/chipfighter/coding-discipline)
- **License:** MIT

Install and usage instructions live in the source repository linked above.

## Pricing

- **Free** — Free

## Security capabilities

Automated source analysis of v0.1.0 — what this tool can access:

- **Network access:** no
- **Filesystem access:** no
- **Shell / process execution:** no
- **Environment & secrets:** no
- **Dynamic code execution:** no

*"Yes" means the capability is present in the source — more access means more to trust, not that it is unsafe.*


## Versions

- **0.1.0** — security scan: passed — Imported from the upstream source.

## Links

- Listing page: https://agentstack.voostack.com/l/skill-chipfighter-coding-discipline-code-review
- Seller: https://agentstack.voostack.com/s/chipfighter
- Browse the marketplace: https://agentstack.voostack.com/browse

---
Listed on AgentStack — the marketplace for AI agent skills and MCP servers. Every listing is security-reviewed. Creators keep 70%.