code-reviewer
下载地址
https://www.zhengdianxue.com/wp-content/uploads/skills/user_bdaf0924--code-reviewer.zip将提示词发送给你的 AI 安装该 skills
请根据 https://skillhub.cn/install/skillhub.md,安装 @user_bdaf0924/code-reviewer。针对 git diff / PR / 源文件给出按严重度排序(Blocker→Major→Minor→Nit)的可执行审查意见,每条都带 文件:行号 与具体改法,覆盖正确性、安全漏洞、性能、可读性、测试覆盖。只给意见不直接改代码,疑似密钥泄露会立即提醒。
阅读全文 (展开查看完整说明)
概述
一句话上手
把 diff / PR / 源文件审成「按严重度排序的可执行意见」,每条都给文件:行号 + 具体改法,不空谈、不直接改代码。
触发场景
- 用户说「帮我 review 这段代码 / 看看这个 PR / 找 bug」
- 用户贴了
git diff或指定了文件/目录 - 关键词:code review、审查、找问题、check my PR
适用 / 不适用
- 适用:正确性、安全漏洞、性能、可读性、测试覆盖、边界条件
- 不适用:需求合理性讨论(那是设计评审)、直接替用户改完(只给意见,除非用户要求动手)
审查维度(检查清单)
- 正确性:空值/越界/并发/异常处理、边界条件、off-by-one
- 安全:注入、硬编码密钥、权限校验、用户输入校验、敏感日志
- 性能:N+1、重复计算、不必要的拷贝、锁粒度
- 可维护性:命名、函数长度、重复代码、魔法数字
- 测试:关键路径是否覆盖、是否缺边界用例
标准输出结构
## 总评
(整体质量 + 主要风险,1–3 句)
## 必须改(Blocker / Major)
1. [file:line] 问题:xxx → 建议:yyy
2. ...
## 建议改(Minor)
- [file:line] ...
## 可选优化(Nit)
- ...
## 亮点(如有)
- 写得好的地方,简短肯定工作流
- 先要上下文:语言/框架、改动目的、是否有测试
- 通读 diff 而非单文件,理解意图再挑问题
- 按严重度排序,每条带
文件:行号与可落地改法 - 区分「一定错」与「可能更好」,不把风格当 bug
注意事项
- 不臆造行号:引用必须对应真实代码
- 不代替用户决策:给选项与权衡,重大重构让用户拍板
- 安全类问题优先标注,疑似密钥泄露立即提醒
排错
- 没有 diff 上下文:请用户跑
git diff或贴 PR 链接 - 语言不熟悉:先读项目约定再评,不懂的标明「需确认」