| name | security-review |
| description | 对代码变更、Pull Request、分支或工作区 diff 进行聚焦安全评审，关注认证、授权、输入校验、注入、密钥处理、敏感数据暴露、依赖风险、不安全传输和危险运维行为；当 Codex 被要求进行安全评审、威胁导向代码审查、认证/权限审查或安全风险评估时使用。 |
安全评审
快速开始
评审流程
- 明确范围：
- 评审目标：PR、分支、工作区变更或具体文件。
- 如果在评审 diff，确认比较基线。
- 该变更触及的信任边界。
- 被触及的安全敏感领域：认证、权限、密钥、支付、PII、文件上传、网络调用、命令执行、数据库访问、依赖升级、日志或遥测。
- 运行信号扫描：
python3 scripts/security_signal_scan.py /path/to/repo-or-file
- 手动追踪高风险链路：
- 外部输入到危险汇点：数据库查询、shell 命令、HTML/模板渲染、文件路径、网络请求、反序列化、日志或重定向。
- 身份到授权决策：认证来源、用户/会话查找、角色/权限检查、租户或归属边界。
- 密钥从创建到存储/使用/日志记录的路径。
- 敏感数据从存储到响应、日志、分析系统或第三方调用的路径。
- 先报告发现项，并按严重级别排序。使用下面格式：
### [Severity] Finding title
- Location:
- Problem:
- Why it matters:
- Attack or failure scenario:
- Recommended fix:
- Severity: Critical / Major / Minor / Nit
- 如果没有发现问题，要明确说明，同时列出剩余风险、跳过的检查和已执行的验证。
严重级别指引
- Critical：直接导致数据暴露、认证绕过、授权绕过、凭证泄漏、远程代码执行、破坏性数据修改或可利用注入。
- Major：缺少边界场景授权、存在可利用路径的不安全输入处理、敏感日志、不安全默认值、危险依赖升级或租户边界风险。
- Minor：纵深防御缺口、较弱的校验、不完整审计轨迹、含糊的安全行为或低概率暴露。
- Nit：没有实质性安全影响的命名、注释或结构问题。
与其他工作流协作
- 广泛代码评审用
code-review-agent-workflow.md；需要更深的安全视角时叠加本技能。
- 如果安全评审发现了缺陷，先报告发现项，再交接给缺陷修复工作流。
- 如果评审暴露出设计或策略决策缺口，交接给功能工作流，而不是自行发明安全策略。
