代码巡检
将以下提示词粘贴到你的 AI 对话框中:
请根据 https://skillhub.cn/install/skillhub.md,安装 @user_71026492/code-patrol。
技能介绍
要解决什么问题
合并前或本地自测时,很多人只扫一眼 diff,容易漏掉权限校验、N+1、吞异常这类不会在编译期报错的问题。手工对照团队 checklist 又费时,且不同人关注点不一致,报告格式也难统一。
技能如何工作
本技能把变更审查收成固定流程,默认范围是 git diff,也可指定文件或目录。
- 收集上下文:汇总当前变更信息(若已注入则跳过)。
- 四维审查:对每个变更文件从四个维度逐项检查:
- 安全:注入、认证授权、敏感信息硬编码、依赖漏洞等
- 性能:算法与 N+1、资源泄漏、并发与阻塞、缓存设计
- 可维护性:过长函数、重复代码、命名与职责划分、测试可靠性
- 正确性:边界条件、状态与事务、错误处理、竞态与 TOCTOU
- 严重级别:按
[CRITICAL]/[WARNING]/[INFO]标注,区分必须修与可改。 - 结构化报告:每条问题带行号和具体修复建议(可含代码示例);涉及数据库、支付、用户数据等敏感变更会高亮。
核心原则:不纠结 Prettier/ESLint 能处理的风格问题;不要求 100% 完美,把精力放在必须修的风险上。
适用边界
- 审查质量依赖 diff 是否完整、上下文是否足够;大重构建议配合指定目录或补充说明。
- 输出是辅助判断,不能替代安全审计、压测或合规评审。
- 技能内可参考示例、反模式清单与各维度检查清单,但仍以当前变更的实际代码为准。
使用场景
- 功能分支提 PR 前,对相对主干的 `git diff` 做四维审查,优先揪出权限缺失与 SQL/命令注入
- 改动涉及数据库迁移、支付或用户资料字段时,要求报告高亮敏感域并按 [CRITICAL] 标出必须修项
- 重构公共模块后指定目录扫描,检查重复代码、过长函数以及是否吞异常或 catch 过宽
- 接口或批处理优化提交前,在变更文件里对照 N+1、连接/句柄未关闭与缓存穿透风险
适合人员
- 维护核心 API、每次合并前要在 diff 里自查认证授权与输入校验的后端工程师
- 需要把 PR 审查结论整理成统一格式(行号 + 修复建议 + 严重级别)以便团队跟进的 Tech Lead
- 经常改 ORM 与列表接口、担心 N+1 或资源泄漏混进上线的性能敏感服务开发者
- 在热修或发布值班分支上改登录/权限逻辑,希望结构化列出越权与硬编码密钥风险的工程师