Refactor: 収束ループの状態管理を段ごとに分ける（cross-refactoring / cross-review）

[takemi-ohama]

何のために

収束ループの状態管理は、1 つの関数に多段の処理が直列で同居している。取り込みの
可否を決める関数は 100〜170 行あり、途中の 1 段だけを読んで正しさを確かめられない。
段ごとに名前を付けて分け、振る舞いを変えずに読める単位にする。

何を変えたか

改善は 12 件で、対象は収束ループの状態管理（cross-refactoring/scripts/refactor.py）と
集計（cross-review/scripts/lib/metrics.py）である。

適用結果の取り込み

対象	変更内容
cmd_merge_apply	「取り込み済みの判定」「着手前テストの確認」「範囲の確定」「申告の整合性の検証」「項目ごとの検証」「公開」の 6 段を、段ごとの関数へ分けた
_validate_apply_commit_ownership	申告されたコミットの所有者の特定、重複と未割当の検出、失敗時の切り戻しを分けた

修正結果の取り込み

対象	変更内容
cmd_merge_fix	叩き直しの判定、解決スレッドの照合、範囲の検証、状態の反映を段ごとに分けた（2 段階で実施）
verify_fix_commit	適用側のコミット検証と同じ 4 手順を重複して持っていたため、共通の検証へ寄せた

修正コミットと適用コミットは同じ基準で見る決まりだが、手順が 2 か所に書かれていると
片方だけが緩む余地が残る。1 か所にまとめた。

レビューと提案の判定

対象	変更内容
cmd_judge_review	結果の集計と、判定から終了コードを決める分岐を分けた
judge	結果の存在確認、判定語彙、投稿の可否、指摘の構造という 4 種の検証を分けた
cmd_merge_proposals	各ランタイムの結果ファイルの読み込みと、統合結果から状態を更新する処理を分けた

初期化・取り消し・同期・集計

対象	変更内容
cmd_init	Pull Request のメタデータを取る 3 回の呼び出しを 1 つの関数へまとめた
_drop_items	取り消し対象の抽出、積み直しの計画、確認用の出力、実行を段ごとに分けた
_sync_generated	同期コマンドの実行と、生成された差分のコミットを分けた
aggregate（集計）	実装担当とレビュー担当という別々の集計軸が同じループで更新されていたため、軸ごとに分けた

変えていないこと

公開しているコマンド、終了コード、状態ファイルの構造は変わらない。

対象範囲のテストは着手前 500 件から 502 件になった。増えた 2 件は、取り消しの
振る舞いを固定するために追加したものである。

$ uv run --with pytest python -m pytest \
plugins/ndf-shared/skills/cross-refactoring/tests \
plugins/ndf-shared/skills/cross-review/tests -q
502 passed in 26.17s

配布物（ndf-claude / ndf-codex / ndf-kiro）は scripts/build-runtime-plugins.sh
で生成したもので、編集元と同じ内容である。

どう作られたか

/ndf:cross-refactoring を 3 ラウンド回した。提案は codex / gemini / kiro が並列で行い、
適用は輪番で 1 者、残る 2 者がレビューする。各コミットは 1 手 1 コミットで、コミット
ごとにテストの実行・差分予算・対象範囲の遵守を機械的に検証している。

採用した 14 件のうち 2 件は、実差分が見積を大きく超えたため項目単位で取り消した。
残る 12 件はレビュー担当 2 者の承認を得ている。収束後に Pull Request 全体を
/ndf:cross-review にかけ、codex と gemini の双方が承認した（指摘 0 件）。

試行そのものの記録は #137 にある。

Test plan

• 対象範囲のテストが 502 passed
• 各ラウンドの適用コミットで同じテストが通ること（進行側が実行済み）
• /ndf:cross-review による Pull Request 全体のレビューが承認で収束すること

🤖 Generated with Claude Code

https://claude.ai/code/session_01XpZXeYvfEGymigQeCLp1Eh

[takemi-ohama]

🤖 cross-refactoring | round 3 | Reviewer: codex / default | APPROVE

判定: APPROVE

対象コミット（R3-001 / R3-002 / R3-004 / R3-005）を確認しました。抽出・処理連鎖への分解・重複の共通化はいずれも宣言されたスメルと手法の範囲内で、公開インタフェース、例外、状態保存、push 前同期、修正ラウンド取り消しの副作用順序に差し戻しが必要な変更は見つかりませんでした。R3-003 は abandoned / reverted として今回の適用差分に残っていないことを確認しています。

検証:

• uv run --with pytest python -m pytest plugins/ndf-shared/skills/cross-refactoring/tests plugins/ndf-shared/skills/cross-review/tests -q
• 結果: 502 passed in 26.29s

[takemi-ohama]

🤖 cross-refactoring | round 3 | Reviewer: kiro / claude-sonnet-5 | APPROVE

対象コミット（R3-001, R3-002 ×2, R3-004 ×3, R3-005）を確認しました。すべて extract_method / split_into_pipeline / consolidate_duplication の単純な抽出・集約で、公開インタフェースの入出力・例外種別・副作用の順序は変わっていません。

• R3-001: _fetch_pr_context への抽出は cmd_init の呼び出し順を変えず、戻り値の型もそのまま。
• R3-002: _run_sync_command / _commit_sync_changes への分割後も _sync_generated の判定順（command → clean → run → produced → commit）は同一。
• R3-004: _revert_invalid_fix_round / _record_accepted_fix_commits / _mark_resolved_fix_findings の 3 段抽出は元の分岐構造をそのまま移しており、state の保存順序（pending_push → 取り消し → fix_base_sha 保存）も保持されています。
• R3-005: _verify_commit_basics への集約は、verify_fix_commit と verify_apply_item それぞれの「実体が無いときの理由文」を呼び出し側の引数として残しており、両者の出力差を保ったまま重複を解消しています。

差分予算（見積 × 抽出系倍率 3）に対しても全項目が範囲内です（R3-001: 37/120、R3-002: 76/180、R3-004: 91/255、R3-005: 52/75）。R3-003 は放棄・取り消しが確認でき、コード上に残骸はありません。

uv run --with pytest python -m pytest plugins/ndf-shared/skills/cross-refactoring/tests plugins/ndf-shared/skills/cross-review/tests -q を実行し、502 passed（exit=0）を確認しました。

[takemi-ohama]

🤖 cross-review | round 1 | codex | APPROVE

差し戻しが必要な指摘はありません。対象テストは uv run --with pytest python -m pytest plugins/ndf-shared/skills/cross-refactoring/tests plugins/ndf-shared/skills/cross-review/tests -q で 502 passed です。

[takemi-ohama]

🤖 cross-review | round 1 | gemini | APPROVE

全 3 ラウンドでの 12 件の構造改善を確認しました。抽出されたヘルパー関数は副作用の順序やバリデーションの責務を正確に維持しており、verify_apply_item と verify_fix_commit の共通化 (_verify_commit_basics) もそれぞれの固有要件を損なわずに実装されています。現状固定テストも変更意図通りに維持されており、ラウンドを跨いだ整合性に問題は見られませんでした。