Antigravity のエージェントが開く PR の説明が「Update files」のままになる問題と、レビューできる要約を強制するゲート

PR の説明は、必ず以下の見出しをすべて埋めて Markdown で出力してください。
各見出しには最低1文を書き、該当しない場合も「該当なし」と理由を添えてください。
 
## What
変更したファイル群と、加えた変更の要点を箇条書きで。
 
## Why
この変更が解決する課題。関連 issue があれば番号を添える。
 
## Risk
壊れたときに影響が大きい箇所。課金・認証・データ移行・設定値の
いずれかに触れた場合は、必ずその旨と影響範囲を明記する。
 
## Test
正しさを確かめた方法。追加・変更したテストと、確認した境界条件。
 
## Review focus
レビュアーに特に確認してほしい箇所を1〜3点。

#!/usr/bin/env node
// validate-pr-description.mjs
// 使い方:
//   git diff --name-only origin/main...HEAD > /tmp/changed.txt
//   cat pr-body.md | node validate-pr-description.mjs --changed /tmp/changed.txt
 
import { readFileSync } from "node:fs";
 
// --- 1. 入力の取得 -------------------------------------------------
function readStdin() {
  try {
    return readFileSync(0, "utf8");
  } catch {
    return "";
  }
}
 
function getChangedFiles() {
  const idx = process.argv.indexOf("--changed");
  if (idx === -1) return [];
  const path = process.argv[idx + 1];
  return readFileSync(path, "utf8")
    .split("\n")
    .map((l) => l.trim())
    .filter(Boolean);
}
 
// --- 2. 必須見出しの定義 -------------------------------------------
const REQUIRED_SECTIONS = ["What", "Why", "Risk", "Test", "Review focus"];
 
// 「実質空欄」とみなす定型句。これだけしか書かれていなければ不合格。
const VACUOUS = [
  /^update files?\.?$/i,
  /^fix(es|ed)? issues?\.?$/i,
  /^apply changes\.?$/i,
  /^n\/?a\.?$/i,
  /^なし\.?$/,
  /^特になし\.?$/,
  /^該当なし\.?$/, // Risk/Test では理由なしの「該当なし」も不可（後述）
];
 
// --- 3. 高リスクなパスのパターン -----------------------------------
const HIGH_RISK = [
  { label: "課金", re: /(billing|payment|checkout|stripe|pricing|subscription)/i },
  { label: "認証", re: /(auth|login|session|token|oauth|credential)/i },
  { label: "データ移行", re: /(migration|migrate|schema|\.sql$)/i },
  { label: "設定値", re: /(config|\.env|settings|\.ya?ml$|\.toml$)/i },
  { label: "削除", re: null }, // 削除は別途、行数から判定
];
 
// --- 4. 説明文を見出しごとに分解 -----------------------------------
function parseSections(body) {
  const map = {};
  const re = /^##\s+(.+?)\s*$/gm;
  const heads = [...body.matchAll(re)];
  for (let i = 0; i < heads.length; i++) {
    const name = heads[i][1].trim();
    const start = heads[i].index + heads[i][0].length;
    const end = i + 1 < heads.length ? heads[i + 1].index : body.length;
    map[name.toLowerCase()] = body.slice(start, end).trim();
  }
  return map;
}
 
function isVacuous(text) {
  if (!text) return true;
  const firstLine = text.split("\n")[0].trim();
  return VACUOUS.some((re) => re.test(firstLine));
}
 
// 単語の重なりで「Why が What の言い換え」を検出する素朴な指標
function jaccard(a, b) {
  const norm = (s) =>
    new Set(
      s
        .toLowerCase()
        .replace(/[^\p{L}\p{N}\s]/gu, " ")
        .split(/\s+/)
        .filter((w) => w.length > 1)
    );
  const sa = norm(a);
  const sb = norm(b);
  if (sa.size === 0 || sb.size === 0) return 0;
  let inter = 0;
  for (const w of sa) if (sb.has(w)) inter++;
  return inter / (sa.size + sb.size - inter);
}
 
// --- 5. 検証本体 ---------------------------------------------------
function validate(body, changed) {
  const errors = [];
  const sections = parseSections(body);
 
  // 5-1. 必須見出しの存在と非空
  for (const name of REQUIRED_SECTIONS) {
    const text = sections[name.toLowerCase()];
    if (text === undefined) {
      errors.push(`見出し「## ${name}」がありません`);
    } else if (isVacuous(text)) {
      errors.push(`「## ${name}」が実質空欄です: "${text.split("\n")[0]}"`);
    }
  }
 
  // 5-2. Why が What の言い換えになっていないか
  const what = sections["what"] ?? "";
  const why = sections["why"] ?? "";
  if (what && why && jaccard(what, why) > 0.8) {
    errors.push("Why が What とほぼ同一です。動機（なぜ）を書いてください");
  }
 
  // 5-3. 高リスクなパスに触れたら Risk への言及を必須にする
  const hitLabels = new Set();
  for (const f of changed) {
    for (const { label, re } of HIGH_RISK) {
      if (re && re.test(f)) hitLabels.add(label);
    }
  }
  // 高リスク領域に触れたのに Risk が実質空欄なら不合格。
  // ラベル名そのものの言及までは求めない（過剰検出を避け、非空であることだけ保証する）。
  if (hitLabels.size > 0 && isVacuous(sections["risk"] ?? "")) {
    const labels = [...hitLabels].join("・");
    errors.push(`${labels}に関わるファイルを変更していますが、Risk が空欄です`);
  }
 
  return errors;
}
 
// --- 6. 実行 -------------------------------------------------------
const body = readStdin();
const changed = getChangedFiles();
 
if (!body.trim()) {
  console.error("PR 説明が空です。テンプレートを埋めてください。");
  process.exit(1);
}
 
const errors = validate(body, changed);
if (errors.length > 0) {
  console.error("PR 説明がレビュー要件を満たしていません:\n");
  for (const e of errors) console.error("  - " + e);
  console.error("\nテンプレートの見出しをすべて埋め、Risk と Review focus を具体的に書いてください。");
  process.exit(1);
}
 
console.log("PR 説明はレビュー要件を満たしています。");

// risk-score.mjs — git diff の数値とパスからリスクの目安を出す
import { execSync } from "node:child_process";
 
const base = process.argv[2] ?? "origin/main";
const stat = execSync(`git diff --numstat ${base}...HEAD`, { encoding: "utf8" })
  .split("\n")
  .filter(Boolean)
  .map((l) => {
    const [add, del, file] = l.split("\t");
    return { add: Number(add) || 0, del: Number(del) || 0, file };
  });
 
const PATTERNS = [
  { label: "課金", re: /(billing|payment|checkout|stripe|pricing)/i, weight: 5 },
  { label: "認証", re: /(auth|session|token|credential)/i, weight: 4 },
  { label: "データ移行", re: /(migration|schema|\.sql$)/i, weight: 5 },
  { label: "設定値", re: /(config|\.env|\.ya?ml$|\.toml$)/i, weight: 3 },
];
 
let score = 0;
const reasons = [];
let totalDel = 0;
 
for (const { add, del, file } of stat) {
  totalDel += del;
  for (const { label, re, weight } of PATTERNS) {
    if (re.test(file)) {
      score += weight;
      reasons.push(`${label}: ${file} (+${add}/-${del})`);
    }
  }
}
 
// 大量削除は単独で加点（消すことは足すより壊しやすい）
if (totalDel > 200) {
  score += 3;
  reasons.push(`大量削除: 計 -${totalDel} 行`);
}
 
const level = score >= 8 ? "高" : score >= 4 ? "中" : "低";
console.log(`リスク目安: ${level} (score=${score})`);
for (const r of reasons) console.log("  - " + r);

#!/usr/bin/env bash
# open-pr.sh — 検証を通った PR だけを開く
set -euo pipefail
 
BASE="origin/main"
BODY_FILE="pr-body.md"   # エージェントが書いた説明
 
git diff --name-only "${BASE}...HEAD" > /tmp/changed.txt
 
# 1. 説明の検証ゲート（落ちたらここで停止）
if ! cat "${BODY_FILE}" | node validate-pr-description.mjs --changed /tmp/changed.txt; then
  echo "説明がレビュー要件未達のため PR を開きません。エージェントに書き直させます。"
  exit 1
fi
 
# 2. リスク目安を説明の末尾に追記（人間向けの注意喚起）
{
  echo ""
  echo "---"
  echo "### 自動リスク見積もり"
  echo '```'
  node risk-score.mjs "${BASE}"
  echo '```'
} >> "${BODY_FILE}"
 
# 3. ここで初めて PR を開く
gh pr create --base main --title "$(git log -1 --pretty=%s)" --body-file "${BODY_FILE}"