审查当前分支改动中的 Bug、安全漏洞与代码质量：通过完整…

当被要求审查改动、查找 bug、做安全审查或审计当前分支代码时，用系统化清单发现并报告问题。

适用场景

解决思路

1. 阶段 1：完整收集输入
2. 1. 获取完整 diff：`git diff $(gh repo view --json defaultBranchRef --jq '.defaultBranchRef.name')...HEAD`
3. 2. 如果输出被截断，逐个读取每个改动文件，直到看过每一行改动。
4. 3. 在继续之前，列出本分支中修改的所有文件。
5. 阶段 2：攻击面映射
6. 对每个改动文件，识别并列出：
7. 所有用户输入（请求参数、请求头、请求体、URL 组件）。
8. 所有数据库查询。
9. 所有认证/授权检查。
10. 所有会话/状态操作。
11. 所有外部调用。
12. 所有加密操作。
13. 阶段 3：安全清单（对每个文件检查每一项）
14. 注入：SQL、命令、模板、请求头注入。
15. XSS：模板中的所有输出是否都正确转义？
16. 认证：所有受保护操作是否都有认证检查？
17. 授权/IDOR：是否已验证访问控制，而不仅仅是认证？
18. CSRF：状态变更操作是否受保护？
19. 竞态条件：任何 read-then-write 模式中是否存在 TOCTOU？
20. 会话：是否存在固定、过期、安全标志问题？
21. 密码学：是否使用安全随机、正确算法，且日志中没有秘密？
22. 信息泄露：错误消息、日志、时序攻击是否泄露信息？
23. DoS：是否存在无界操作、缺少速率限制、资源耗尽？
24. 业务逻辑：边界情况、状态机违规、数值溢出？
25. 阶段 4：验证
26. 对每个潜在问题：检查它是否已在改动代码的其他地方处理。
27. 对每个潜在问题：搜索覆盖该场景的现有测试。
28. 对每个潜在问题：阅读周边上下文，确认问题真实存在。
29. 阶段 5：结论前审计（最终结论前必须执行）
30. 列出你审查的每个文件，并确认你已完整阅读它。
31. 列出每个清单项，并注明你发现了问题或确认它是干净的。
32. 列出任何你无法完全验证的区域，并说明原因。
33. 只有完成上述步骤后，才提供最终 findings。
34. 输出格式
35. 优先级：安全漏洞 > bugs > 代码质量。
36. 跳过：样式/格式问题。
37. 对每个问题输出：File:Line - 简要描述。
38. 对每个问题输出：Severity - Critical/High/Medium/Low。
39. 对每个问题输出：Problem - 哪里错了。
40. 对每个问题输出：Evidence - 为什么这是真的（不是已修复、没有现有测试等）。
41. 对每个问题输出：Fix - 具体修复建议。
42. 对每个问题输出：References - 适用的 OWASP、RFC 或其他标准。
43. 如果找不到重要问题，就直说——不要发明问题。
44. 不要做修改——只报告 findings。由我来决定处理什么。

适用边界 · 注意事项

• 不适用于非 GitHub 仓库或未安装/无法使用 `gh` CLI 的环境，因为默认分支获取命令依赖 `gh repo view`。
• 不适用于审查未提交的工作区/暂存区改动；它针对当前分支相对默认分支的提交差异。
• 不适用于只做样式/格式审查或需要直接修改代码的任务；该技能只报告发现。