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
重み:
- 影響度 = 2x
- 検証 = 2x
- ロールバックコスト = 1.5x
- エラー検出 = 1x
- 整合性 = 1x
スコア = Σ(スコア × 重み)/ Σ(重み)。
- 低:< 1.5
- 中:1.5 – < 2.25
- 高:≥ 2.25
数値スコアとその計算は内部に留め、出力には含めないでください。
コメント
Markdown 対応