安全视角代码评审 Skill 两种写法对比:五行提示词还是完整技能包
安全工程师与开发团队经常要做「安全视角代码评审」这类事:从安全角度审查合并请求。每次都从头写提示词,结果时好时坏;把它做成智能体 Skill,就能让智能体每次按同一套步骤执行,输出结构也更稳定。
很多人纠结一开始要不要把技能写得很完整。答案是:先极简,再按需升级。下面对比「安全视角代码评审」的两种写法。
极简版:五行就够
你负责安全视角代码评审。
输入:PR 差异。
步骤:关注认证、权限、输入、加密、日志;标出高风险改动;给出修复;建议补测试。
输出:安全意见。
信息不足先提问,不要编造。
适合个人临时使用、验证流程是否合理。缺点是输出格式不够稳定,边界情况处理不一致。
进阶版:完整技能包
.claude/skills/security-review-pr/
├── SKILL.md
├── scripts/security_review_pr.py
└── reference.md # 可选:团队规范、模板等长资料,按需加载
进阶版在极简版基础上增加:
- 精确的 description,确保自动触发
- 固定的输出结构与“需人工确认”清单
- 约束段落(权限、领域规矩、不可逆操作)
- 参考资料(团队规范、范例),按需加载
- 脚本,负责确定性的解析与校验
- tests/ 测试样例
对比
| 维度 | 极简版 | 进阶版 |
|---|---|---|
| 编写时间 | 几分钟 | 一到两小时 |
| 输出稳定性 | 一般 | 较好 |
| 边界处理 | 依赖模型自觉 | 有明确规则 |
| 团队共享 | 不方便 | 可评审、可版本化 |
| 适用 | 个人试用 | 日常高频、多人使用 |
什么时候该升级
- 每周使用超过几次
- 输出要给别人看或进入正式流程
- 已经发现两次以上同类错误
- 属于中等风险:结果会影响业务判断,建议在输出前设置人工审核点
升级时参考SKILL.md 写法,并用测试用例验证。
平台选择建议
同一个「安全视角代码评审」技能可以落在不同平台上,选择时主要看你在哪里工作、要不要执行代码、要不要接外部系统:
- 扣子(Coze)智能体与工作流:扣子(Coze)是字节跳动推出的智能体开发平台。详见扣子(Coze)智能体与工作流 版
- AGENTS.md:AGENTS.md 是放在仓库根目录、写给编码智能体看的说明文件。详见AGENTS.md 版
- MCP 服务:MCP(模型上下文协议)是一个开放协议。详见MCP 服务 版
这个技能需要运行本地脚本(解析文件、调用命令行工具等),要求运行环境允许执行代码,可据此排除不支持相应能力的平台。
智能体与人的分工
技能不是把工作全部交出去,而是把重复、机械的部分交给智能体,人把住判断和责任:
| 步骤 | 智能体负责 | 人负责 |
|---|---|---|
| 关注认证、权限、输入、加密、日志 | 按指令完成并给出依据 | 抽查关键处,确认无误 |
| 标出高风险改动 | 按指令完成并给出依据 | 抽查关键处,确认无误 |
| 给出修复 | 按指令完成并给出依据 | 抽查关键处,确认无误 |
| 建议补测试 | 按指令完成并给出依据 | 抽查关键处,确认无误 |
| 最终交付 | 整理安全意见 | 决定是否采用、是否对外 |
属于中等风险:结果会影响业务判断,建议在输出前设置人工审核点。
准备一份好范例
在技能的参考资料里放一份你认可的安全意见范例,往往比多写十条规则更有效。好范例应当:
- 来自真实工作,但已去除个人信息和敏感数据
- 体现团队习惯的结构、措辞和详略程度
- 包含一处“需人工确认”的标注示范,让智能体学会如何表达不确定
- 篇幅适中,不要比平时的产出长太多
范例不必多,一到两份即可;太多范例反而会让智能体机械模仿某一份的细节。
延伸阅读
- 安全视角代码评审 · Function Calling 版
- 安全视角代码评审 · 扣子(Coze)智能体与工作流 版
- 安全视角代码评审 · 场景实录
- 供应商安全问卷 · 极简与进阶
- 备份加密检查 · 极简与进阶
- 智能体Skills 分类:625 个技能 × 32 个角度
说明:文中的配置、代码与目录结构均为示意,用于说明思路;各平台的入口、字段名与能力会更新,落地前请以对应官方文档为准。示例中的数据与场景为虚构,不代表真实案例。