エージェントが直した .proto がバイナリ互換を壊す前に — buf breaking で wire 互換を門番する

# buf.yaml
version: v2
modules:
  - path: proto
breaking:
  use:
    - WIRE            # バイナリ互換のみを審査（命名・整形は許容する）
  ignore_unstable_packages: true
lint:
  use:
    - STANDARD

# 変更後の proto を main の proto と突き合わせる
buf breaking --against '.git#branch=main,subdir=.'

// Before: エージェントが「整理」した危険な形
message UserProfile {
  string display_name = 1;
  string avatar_url = 2;
  // legacy_score(3) を消して、新しい nickname を 3 に置いた
  string nickname = 3;
}

// After: 番号と旧名を予約して二度と再利用させない
message UserProfile {
  string display_name = 1;
  string avatar_url = 2;
  string nickname = 4;          // 空いていた次の番号を使う
 
  reserved 3;                   // legacy_score が使っていた番号を封印
  reserved "legacy_score";      // 旧フィールド名も予約しておく
}

#!/usr/bin/env bash
# .git/hooks/pre-push — proto の wire 互換を push 前に検証する
set -euo pipefail
 
# proto に変更が無ければ即通過（無関係な push を遅くしない）
if git diff --quiet origin/main -- 'proto/**/*.proto'; then
  echo "proto に変更なし — wire 互換チェックをスキップします"
  exit 0
fi
 
echo "proto の変更を検出 — buf breaking を実行します"
if ! buf breaking --against '.git#branch=main,subdir=.'; then
  echo ""
  echo "⛔ wire 互換を壊す変更が見つかりました。"
  echo "   フィールド番号の再利用・型変更がないか確認してください。"
  echo "   削除は 'reserved' への置き換えで行ってください。"
  exit 1
fi
 
echo "✅ wire 互換 OK"

# check_reserved.py — 削除された番号が reserved に登録されているか確認する
import re
import subprocess
import sys
 
def field_numbers(proto_text: str) -> set[int]:
    # "name = 12;" のようなフィールド定義から番号を集める
    nums = set()
    for line in proto_text.splitlines():
        m = re.search(r"=\s*(\d+)\s*;", line)
        if m and "reserved" not in line:
            nums.add(int(m.group(1)))
    return nums
 
def reserved_numbers(proto_text: str) -> set[int]:
    nums = set()
    for m in re.finditer(r"reserved\s+([0-9,\s]+);", proto_text):
        for token in m.group(1).split(","):
            token = token.strip()
            if token.isdigit():
                nums.add(int(token))
    return nums
 
def read_main(path: str) -> str:
    return subprocess.run(
        ["git", "show", f"origin/main:{path}"],
        capture_output=True, text=True,
    ).stdout
 
def main(paths: list[str]) -> int:
    failed = False
    for path in paths:
        old = read_main(path)
        if not old:
            continue  # 新規ファイルは比較対象なし
        with open(path, encoding="utf-8") as f:
            new = f.read()
        removed = field_numbers(old) - field_numbers(new)
        unreserved = removed - reserved_numbers(new)
        if unreserved:
            print(f"⛔ {path}: 削除された番号 {sorted(unreserved)} が reserved 未登録です")
            failed = True
    return 1 if failed else 0
 
if __name__ == "__main__":
    sys.exit(main(sys.argv[1:]))