分支变更 Bug 与安全审查

要解决什么问题

合并前或 PR 审查时，diff 往往很长，人工容易漏看用户输入路径、数据库查询和「先读后写」这类攻击面。本技能把「看改动」变成可重复的流程：先收齐变更，再按文件做安全与逻辑核对，只输出发现，不改仓库。

技能如何工作

阶段 1：完整收集输入

• 用 git diff <默认分支>...HEAD 拿到本分支相对主干的完整 diff
• 若终端输出被截断，逐个打开被改文件，直到每一行变更都看过
• 继续前列出本分支所有被修改的文件

阶段 2～3：攻击面与安全清单

对每个改动文件，先列出：用户输入（参数、头、体、URL）、数据库查询、认证/授权、会话/状态、外部调用、加密操作。再逐项勾选清单，例如：

• 注入（SQL、命令、模板、头注入）
• XSS（模板输出是否转义）
• 认证 / 授权 / IDOR、CSRF、竞态（TOCTOU）
• 会话、加密、信息泄露、DoS、业务逻辑（边界、状态机、溢出）

阶段 4～5：验证与结论前审计

每个疑点要查：同文件其它位置是否已处理、是否有测试覆盖、上下文是否真构成问题。下结论前必须列出已审文件、每条清单项是「有问题」还是「已确认干净」、以及无法完全验证的区域及原因。

输出与适用边界

优先级：安全漏洞 > bug > 代码质量；跳过纯风格/格式。每条问题含 File:Line、Severity、Problem、Evidence、Fix、References（如 OWASP）。无重要问题则如实说明，不编造。

适合：被要求审查更改、查 bug、安全审计、审当前分支或 PR diff。依赖 git 与 gh 获取默认分支与 diff；未给出完整 diff 时审查不完整。