security-review

name	security-review
description	对代码变更、Pull Request、分支或工作区 diff 进行聚焦安全评审，关注认证、授权、输入校验、注入、密钥处理、敏感数据暴露、依赖风险、不安全传输和危险运维行为；当 Codex 被要求进行安全评审、威胁导向代码审查、认证/权限审查或安全风险评估时使用。

安全评审

快速开始

• 把这个技能当成聚焦型评审层，而不是正常正确性评审的替代品。
• 当仓库路径或变更文件路径可用时，运行 scripts/security_signal_scan.py <path>。
• 在输出发现前，先阅读 references/security-review-checklist.md。
• 遵循 ../_shared/references/skill-operating-rules.md 中共享的安全、验证和范围规则。
• 除非用户明确要求修复，否则保持只读。
• 未经明确许可，不要提交、推送、部署、轮换密钥、吊销凭证或执行破坏性命令。

评审流程

1. 明确范围：

• 评审目标：PR、分支、工作区变更或具体文件。
• 如果在评审 diff，确认比较基线。
• 该变更触及的信任边界。
• 被触及的安全敏感领域：认证、权限、密钥、支付、PII、文件上传、网络调用、命令执行、数据库访问、依赖升级、日志或遥测。

2. 运行信号扫描：

python3 scripts/security_signal_scan.py /path/to/repo-or-file

3. 手动追踪高风险链路：

• 外部输入到危险汇点：数据库查询、shell 命令、HTML/模板渲染、文件路径、网络请求、反序列化、日志或重定向。
• 身份到授权决策：认证来源、用户/会话查找、角色/权限检查、租户或归属边界。
• 密钥从创建到存储/使用/日志记录的路径。
• 敏感数据从存储到响应、日志、分析系统或第三方调用的路径。

4. 先报告发现项，并按严重级别排序。使用下面格式：

### [Severity] Finding title

- Location:
- Problem:
- Why it matters:
- Attack or failure scenario:
- Recommended fix:
- Severity: Critical / Major / Minor / Nit

5. 如果没有发现问题，要明确说明，同时列出剩余风险、跳过的检查和已执行的验证。

严重级别指引

• Critical：直接导致数据暴露、认证绕过、授权绕过、凭证泄漏、远程代码执行、破坏性数据修改或可利用注入。
• Major：缺少边界场景授权、存在可利用路径的不安全输入处理、敏感日志、不安全默认值、危险依赖升级或租户边界风险。
• Minor：纵深防御缺口、较弱的校验、不完整审计轨迹、含糊的安全行为或低概率暴露。
• Nit：没有实质性安全影响的命名、注释或结构问题。

与其他工作流协作

• 广泛代码评审用 code-review-agent-workflow.md；需要更深的安全视角时叠加本技能。
• 如果安全评审发现了缺陷，先报告发现项，再交接给缺陷修复工作流。
• 如果评审暴露出设计或策略决策缺口，交接给功能工作流，而不是自行发明安全策略。