mcpField
M
merge-risk

merge-risk

github.com/warpdotdev测试与质量636585 次安装1 周前更新
代码仓库

关于 merge-risk

merge-risk 是用于评估代码合并请求风险的技能,面向审查变更的开发者。它按影响、验证、回滚成本等维度评分,并要求输出分类判断而不展示计算过程。

主要功能

  1. 01评估对齐度、影响和验证情况
  2. 02检查回滚成本与错误检测
  3. 03按加权规则生成风险等级
  4. 04采用指定格式列出分类判断

适用场景

  1. 01合并前审查 Pull Request
  2. 02检查敏感路径和变更影响
  3. 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