Code Review
完成一块功能、合并前、或要给/收代码评审时用。做评审按「正确性→合需求→安全→简洁→风格」的优先级看；收评审先核实再改、不盲从也不表演式同意。
Install
$ agentstack add skill-chipfighter-coding-discipline-code-review ✓ scanned · ✓ verified — works with Claude Code, Cursor, and more.
Security review
✓ PassedNo issues found. Passed automated security review. · v0.1.0 How review works →
- ✓ Prompt-injection patterns
- ✓ Secret / credential exfiltration
- ✓ Dangerous shell & filesystem operations
- ✓ Untrusted network calls
- ✓ Known-malicious package signatures
What it can access
- ✓ Network access No
- ✓ Filesystem access No
- ✓ Shell / process execution No
- ✓ Environment & secrets No
- ✓ Dynamic code execution No
From automated source analysis of v0.1.0. “Used” means the capability is present in the source — more access means more to trust, not that it’s unsafe.
About
做评审（自己审或派子 agent 审）
派一个子 agent 当评审更好——给它精心裁过的上下文（改了啥、本该满足啥需求、diff 范围 base..head），别把整个 session 历史倒给它；这样它盯产物不盯你的思路，也省你自己的上下文。
按这个优先级看，高的没过就别纠结低的：
- 正确性：逻辑对不对、边界/错误/并发有没有漏、会不会崩。
- 合需求：逐条对着需求/方案勾，做的是不是要的、有没有缺。
- 安全：注入、越权、密钥泄露、未校验输入。
- 简洁：过度设计 / 重复 / 可删的死代码（YAGNI）。
- 风格：命名、一致性、可读性——最后看，别一上来挑这个。
分轻重：拦路的（崩/安全）立刻修，重要的往下走之前修，小的记下回头弄。
收评审
- 先核实再改：这条对本代码库成立吗？会弄坏现有功能吗？现在这么写是不是有原因？拿不准就说"我没法验证 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
- Source: chipfighter/coding-discipline
- License: MIT
Install and usage instructions live in the source repository linked above.
Reviews
No reviews yet — be the first.
Write a review
Versions
- v0.1.0 Imported from the upstream source.