テック技報最新号へ →
AIコーディング

PR を取り込む危険度を5項目で採点させる(Warp の merge-risk スキル)

PR のマージ前に、エージェントに「どれだけ危ないか」を low / medium / high で出させる。見るのは、狙いとの一致、影響範囲、動作の証拠、戻しやすさ、壊れたときに気づけるか、の5項目。Warp が公開しているスキルで、元は Matt Pocock の /pr スキル。

@sumika45379 · 投稿 2026-10-29
#レビュー#pull-request#skills#リスク評価

どんな時に使う?

  • AI が作った PR が多く、どれを丁寧に見るべきか順番を付けたい
  • スクリーンショットや計測なしの UI 変更・性能に関わる変更を、そのまま通してしまいがち

手順

  1. 入れる: npx skills add warpdotdev/common-skills --skill merge-risk
  2. 対象の PR やブランチで「この PR の merge risk を出して」と頼む
  3. 出力は **Merge risk: <low/medium/high>** に続けて、項目ごとの判定と、危険度を上げている穴の箇条書き(関係ファイルへのリンク付き)
  4. 5項目と判定の目安:
    • Alignment: 説明・元の Issue とコードが合っているか。ついでの変更や前提が書いてあるか
    • Impact: 影響範囲。認証・権限・決済・個人情報・秘密情報・データ移行・依存の更新・インフラや設定に触れていれば、差分が小さくても危険
    • Verification: UI の変更にはスクリーンショットや録画、性能に関わる変更には計測が要る。前後比較が望ましい。エッジケースはテストか、せめて記述
    • Rollback cost: 戻せない変更(破壊的な操作、後戻りできない決定)かどうか
    • Error detection: 本番で壊れたら利用者より先に気づけるか。エラーを握りつぶすフォールバックがないか
  5. 採点: ⚠️=3・👀=2・✅=1 に、Impact と Verification は2倍、Rollback cost は1.5倍、残りは1倍の重みを掛けて平均する。1.5 未満が low、2.25 以上が high。数値と計算は出力に出さない指示になっている

注意点

出典

X の元の投稿を開く