Addy Osmani 的 agent-skills 仓库里的代码审查技能：合并前（或审查别的代理、别人写的代码时）让代理按正确性、可读性、架构、安全、性能五个维度审查改动，先看测试再看实现，每条意见标严重级别（Critical / 必改 / Nit / Optional / FYI），最后给出批准或要求修改的结论。
skills.sh 安装量 79294（+975 / 24h）
安全审计：未审计
- 尚未扫描
理由
- 流程分 5 步：先弄清改动意图，再先审测试、逐文件按五个维度审实现、分级列问题、核对作者的验证说明；要求报告前先对照调用方、守卫和测试排除误报，零问题也是合法结论，不许硬凑
- 判断测试是否有效靠实验：把改动里的一个条件取反后跑测试，再从副本恢复文件；测试仍然通过就说明缺用例，要指出缺哪个
- 给了改动规模参考（约 100 行适合一次审完，约 1000 行应拆分）和四种拆分方式；依赖升级要读更新日志、一次只升一个包、审查锁文件差异
- 发现无用代码先列清单再问是否删除，不擅自删；提出结构问题时要同时给出具体的重构办法
- 仓库为 MIT 许可，同仓库另有 24 个技能；skills.sh 24 小时安装 975，累计 79294，近 8 周每周安装从 3503 升到 5359（只代表关注度）
用法
- 安装命令待审计通过后显示；说“审查这个 PR / diff”、功能写完准备合并或要评估别的代理写的代码时触发
- 不需要账号或外部服务；“取反条件跑测试”这一步需要本地能跑项目的测试
风险
- 本期未审计，固定 C 级、不显示安装命令；SKILL.md 主要是审查清单，唯一会动文件的是“临时取反一个条件跑测试、再从副本恢复”，与用途相符，但要确认文件已恢复；skills.sh 上 Gen Agent Trust Hub、Socket 审计为 Pass，Snyk 为 Warn
- 安全、性能部分引用同仓库的 security-and-hardening、performance-optimization 技能和仓库根目录的清单文件，只装这一个技能时这些链接可能读不到
- 批准标准偏宽：只要整体改善代码健康就批准；“一个工作日内回复”等团队规范对单人项目意义不大