| name | pr-security-review |
| description | Use this skill to review a PR or diff for security regressions. Do not use it for full-repository audits or legal contract review. |
pr-security-review
English
Purpose
Review the current PR, diff, or changed-files summary for security regressions.
When to use
- The user asks to review a PR.
- The user asks whether a diff introduces security risk.
- The user asks for blocking issues or missing tests.
When not to use
- Full-repository audits.
- Smart-contract protocol audits.
- Legal contract review.
- Requests to auto-fix every issue.
Workflow
- Identify changed files and affected call paths.
- Classify PR risk level L0-L3.
- Check authz, data access, secrets, dependencies, dangerous sinks, and missing tests.
- Output blocking issues and non-blocking issues.
- Do not modify files.
Output format
- Risk level
- Blocking issues
- Non-blocking issues
- Missing tests
- Security boundary changes
- Verdict: approve / approve with comments / request changes
Safety rules
Default audit-only. Do not commit, push, merge, or edit files.
Canonical finding format
id: F-001
severity: Critical | High | Medium | Low | Informational
confidence: High | Medium | Low
category:
affected_code:
root_cause:
exploit_path:
preconditions:
impact:
evidence:
minimal_fix:
regression_test:
auto_fix_suitability: Safe | Needs Human Review | Do Not Auto-Fix
notes:
v0.6 operational guardrails
- Keep the skill within its stated trigger conditions and the user's explicitly provided scope.
- Preserve project safety boundaries: audit-only by default; Do not execute exploits, Do not auto-merge, Do not upload private source code or secrets, and do not scan unrelated repositories without explicit user request.
- Ask for explicit human approval before patching high-risk auth, IAM, governance, funds, terminal, or agent-tooling behavior.
- Report validation performed, files changed, residual risk, and any skipped future-phase work when finished.
中文
目的
使用这个 skill 进行PR 安全审查。它应该帮助审查者把输入边界、风险证据、影响、修复建议和回归测试组织成可复核的安全输出。
触发条件
适用于 已有 PR、diff 或 changed-files summary，需要判断安全回归、blocking issue 和 missing tests。如果请求超出这些边界，先说明范围差异，并选择更合适的 prompt、skill 或人工 review 路径。
不适用场景
不要用于完整仓库审计、智能合约协议深审、法律合同 review 或自动修复全部问题。不要把这个 skill 当作自动扫描整个仓库、执行 exploit、上传私有源码或 secrets、自动提交、自动推送或 auto-merge 的许可。
操作流程
- 明确用户给出的目标、允许查看的材料和不能触碰的范围。
- 收集必要上下文，但只读取完成任务所需的文件、diff、workflow、fixture 或文档。
- 识别 trust boundary、privileged operation、sensitive data、preconditions 和 security impact。
- 只报告有 evidence 的 finding；缺少上下文时写 question 或 assumption。
- 为 confirmed issue 提出 minimal fix，并规划与 blocking issue 对应的最小回归测试和直接受影响调用路径测试。
- 完成后报告验证输出、残余风险和需要人工确认的事项。
安全规则
默认 audit-only。未经明确授权，不 patch、不 commit、不 push、不创建 PR、不 merge。不要执行 exploit，不要访问生产系统，不要打印 secrets。涉及 IAM、authz 模型、资金、治理、terminal 执行或 agent-tooling 权限的修复必须进入人工 review。
输出要求
使用 canonical finding format。每个 finding 都要包含 severity、confidence、category、affected_code、root_cause、exploit_path、preconditions、impact、evidence、minimal_fix、regression_test、auto_fix_suitability 和 notes。
