关于 merge-risk
merge-risk 是用于评估代码合并请求风险的技能,面向审查变更的开发者。它按影响、验证、回滚成本等维度评分,并要求输出分类判断而不展示计算过程。
主要功能
- 01评估对齐度、影响和验证情况
- 02检查回滚成本与错误检测
- 03按加权规则生成风险等级
- 04采用指定格式列出分类判断
适用场景
- 01合并前审查 Pull Request
- 02检查敏感路径和变更影响
- 03评估测试证据及监控覆盖
安装
复制下面的命令,在终端中运行。
$
npx skills add https:/ / github.com/ warpdotdev/ common-skills --skill merge-risk安装后在 Claude Code 中输入/merge-risk确认已加载。
详细介绍
输出格式:
**Merge risk: <low-medium-high>**
**<category>:** <verdict>
<list gaps that increase risk. Use bullets, keep succinct. Include links to relevant files as needed>
<...list all categories>
以下是影响风险的类别和评定结果,以及“低风险”的含义:
- 一致性(差 ⚠️ / 尚可 👀 / 良好 ✅): 代码与描述及源问题(如有)保持一致。必须记录对行为的假设和顺手做的额外修改。
- 影响(大 ⚠️ / 中等 👀 / 小 ✅): 变更范围有限,且不涉及敏感代码路径。
- 考虑影响范围:此 PR 引入的变更可能造成的影响或涉及的范围。考虑所有可能性。例如,布局偏移、对使用方造成的问题、移动设备适配性等。
- 无论差异大小,敏感领域都存在较高风险:身份验证、权限、支付、PII、机密、数据迁移、依赖项升级,以及基础设施或配置变更。标出任何涉及这些领域的变更,并指出任何扩大访问范围、削弱验证或记录敏感数据的情况。
- 验证(差 ⚠️ / 尚可 👀 / 良好 ✅): 在适用情况下,已验证行为。
- 用户体验方面的变更需要提供端到端证据,例如截图或录屏。可能显著影响性能的变更需要进行性能分析。优先提供变更前后的证据,而非单独的证明。缺少这些验证覆盖时,应视为失败。
- 必须测试边缘情况;未测试时必须说明原因。仔细审查代码;如果只有文档说明,则应链接到相关代码。
- 回滚成本(高 ⚠️ / 中等 👀 / 低 ✅): 合并后代码是否可逆。考虑单向门与双向门。双向门可以走回头路,单向门则不行。容易回滚的 PR 风险较低。涉及破坏性操作或难以撤销的决策属于单向门。
- 错误检测(差 ⚠️ / 尚可 👀 / 良好 ✅ / 不适用 ✅): 如适用,应监控关键流程;如果此变更在生产环境中出问题,应在用户报告之前有人发现。留意被吞掉的错误或掩盖故障的回退机制。
为合并此 PR 生成一个低—中—高等级的风险评分。
评定结果:
- ⚠️ = 3
- 👀 = 2
- ✅ = 1
权重:
- 影响 = 2 倍
- 验证 = 2 倍
- 回滚成本 = 1.5 倍
- 错误检测 = 1 倍
- 一致性 = 1 倍
评分 = Σ(评分 × 权重)/ Σ(权重)。
- 低:< 1.5
- 中:1.5 – < 2.25
- 高:≥ 2.25
将数值评分及其计算过程保留在内部;不要在输出中包含。
评论
支持 Markdown