Refactor: cross-refactoring 再々検証（未到達 2 経路の実機確認） by takemi-ohama · Pull Request #125 · devbasex/ai-plugins

takemi-ohama · 2026-08-18T06:05:05Z

目的

/ndf:cross-refactoring の未到達 2 経路を実機で通す。

指摘の修正と再レビューの繰り返し（merge-fix → 再 review → 再 judge-review）
上限到達時の項目単位の見送り（should-abandon → abandon-items）

この 2 つは 1 回目（#118）・2 回目（#120）とも一度も実行されていない。
2 回目はラウンド 1 で両レビュー担当とも指摘 0 件だったため到達できなかった。

実行条件

指摘が出る確率を上げるため、次の 2 つを併用する。

引数	値	狙い
`--max-fix-rounds`	`1`	指摘が 1 回の修正で解決しなければ即 `should-abandon` へ到達する
`--max-items-per-round`	`8`	採用件数を増やし、指摘が出る確率を上げる

範囲は cross-refactoring/scriptscross-refactoring/testscross-review/scripts/libcross-review/tests の 4 つ。
生成物の同期は --sync-command "bash scripts/build-runtime-plugins.sh" で進行側が行う。

あわせて確認すること

PR #121 で入れた 3 経路を実機で見る。

対象	確認内容
公開の責務（不具合 10）	実装担当が push せず、`merge-apply` の後に進行側が push する
生成物の同期	同期コミットが push の直前に積まれる
対象外への記録（不具合 11）	適用で失敗した項目が次ラウンドの提案で再採用されない

テスト

着手前のベースラインは 444 件が通ることを確認済み。

uv run --with pytest python -m pytest \
plugins/ndf-shared/skills/cross-refactoring/tests \
plugins/ndf-shared/skills/cross-review/tests -q

🤖 Generated with Claude Code

https://claude.ai/code/session_01WxxF27RXyg3QMjayii5dp1

未到達の 2 経路（指摘の修正と再レビュー / 上限到達時の項目単位の見送り）を実機で通すための種コミット。実行条件は本文に記載した。 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WxxF27RXyg3QMjayii5dp1

…g/scripts/refactor.py#cmd_merge_apply 適用範囲のコミット所有権検証を _verify_ownership に抽出。 Item-Id: R1-001 Round: 1 Impl-Runtime: codex Impl-Model: default

…toring/scripts/refactor.py#cmd_merge_apply 項目ごとの適用検証と分類を _verify_and_classify_items に抽出。 Item-Id: R1-001 Round: 1 Impl-Runtime: codex Impl-Model: default

…ipts/lib/monitor.py#monitor_agent pidfile 起動待ちと初期 AgentStatus 作成を read_started_agent に抽出。 Item-Id: R1-002 Round: 1 Impl-Runtime: codex Impl-Model: default

…ipts/lib/monitor.py#monitor_agent sentinel と result age fallback による lingering process 完了判定を抽出。 Item-Id: R1-002 Round: 1 Impl-Runtime: codex Impl-Model: default

…ipts/lib/monitor.py#monitor_agent early error の fatal/warn 判定を check_early_error に抽出。 Item-Id: R1-002 Round: 1 Impl-Runtime: codex Impl-Model: default

…ipts/lib/monitor.py#monitor_agent プロセス終了後の result.json 判定を finish_exited_process に抽出。 Item-Id: R1-002 Round: 1 Impl-Runtime: codex Impl-Model: default

…ipts/lib/monitor.py#monitor_agent ログ進捗更新と STALLED 判定を update_progress_and_check_stall に抽出。 Item-Id: R1-002 Round: 1 Impl-Runtime: codex Impl-Model: default

…efactoring/scripts/refactor.py#cmd_merge_fix 未検証範囲の pending_push 保存、revert、base sha 更新を _revert_unverified_range に共通化。 Item-Id: R1-003 Round: 1 Impl-Runtime: codex Impl-Model: default

…toring/scripts/refactor.py#cmd_merge_fix 修正結果の冪等キー生成と既処理判定を build_fix_merge_key / already_merged_fix に抽出。 Item-Id: R1-004 Round: 1 Impl-Runtime: codex Impl-Model: default

…toring/scripts/refactor.py#cmd_merge_fix resolved_thread_ids の型検査と GitHub 実解決との照合を resolve_claimed_threads に抽出。 Item-Id: R1-004 Round: 1 Impl-Runtime: codex Impl-Model: default

…toring/scripts/refactor.py#cmd_merge_fix 修正範囲の確定、申告外コミット検出、commit facts 収集を collect_fix_range_report に抽出。 Item-Id: R1-004 Round: 1 Impl-Runtime: codex Impl-Model: default

…toring/scripts/refactor.py#cmd_merge_fix commit facts から problems と accepted を作る処理を evaluate_fix_commits に抽出。 Item-Id: R1-004 Round: 1 Impl-Runtime: codex Impl-Model: default

…toring/scripts/refactor.py#cmd_merge_fix 不正な修正範囲の取り消しと起点保存を reject_fix_range に抽出。 Item-Id: R1-004 Round: 1 Impl-Runtime: codex Impl-Model: default

…g/scripts/refactor.py#cmd_judge_review レビュー結果から state に保存する record 構築を _build_review_record に抽出。 Item-Id: R1-005 Round: 1 Impl-Runtime: codex Impl-Model: default

…g/scripts/refactor.py#cmd_merge_proposals runtime ごとの提案結果ファイル収集を _collect_proposal_files に抽出。 Item-Id: R1-006 Round: 1 Impl-Runtime: codex Impl-Model: default

…s-review/scripts/lib/monitor.py#monitor_agent 監視設定を MonitorConfig にまとめ、CLI 呼び出しを parameter object 経由に変更。 Item-Id: R1-007 Round: 1 Impl-Runtime: codex Impl-Model: default

…eview/scripts/lib/monitor.py#_scan_patterns ログ末尾を安全に読む処理を _read_tail に共通化。 Item-Id: R1-008 Round: 1 Impl-Runtime: codex Impl-Model: default

…/cross-review/scripts/lib/monitor.py#_scan_patterns" This reverts commit c80bff4.

…lls/cross-review/scripts/lib/monitor.py#monitor_agent" This reverts commit e5185a3.

…factoring/scripts/refactor.py#cmd_merge_proposals" This reverts commit ad295d4.

…factoring/scripts/refactor.py#cmd_judge_review" This reverts commit e300048.

…ss-refactoring/scripts/refactor.py#cmd_merge_fix" This reverts commit bd3f0b7.

…ss-refactoring/scripts/refactor.py#cmd_merge_fix" This reverts commit 53706cf.

…ss-refactoring/scripts/refactor.py#cmd_merge_fix" This reverts commit 3685c22.

…ss-refactoring/scripts/refactor.py#cmd_merge_fix" This reverts commit b535305.

…ss-refactoring/scripts/refactor.py#cmd_merge_fix" This reverts commit 685774b.

…/cross-refactoring/scripts/refactor.py#cmd_merge_fix" This reverts commit b884ae8.

…view/scripts/lib/monitor.py#monitor_agent" This reverts commit 1ba3f77.

…view/scripts/lib/monitor.py#monitor_agent" This reverts commit f210919.

…oss-refactoring/scripts/refactor.py#cmd_merge_fix" This reverts commit 68012f9.

…oss-refactoring/scripts/refactor.py#cmd_merge_fix" This reverts commit d5c805f.

…oss-refactoring/scripts/refactor.py#cmd_merge_fix" This reverts commit a31cd0f.

…efactoring/scripts/refactor.py#cmd_judge_review" This reverts commit 6ec72a8.

…efactoring/scripts/refactor.py#cmd_merge_proposals" This reverts commit f78ae13.

…ills/cross-review/scripts/lib/monitor.py#monitor_agent" This reverts commit 93f014f.

…s/cross-review/scripts/lib/monitor.py#_scan_patterns" This reverts commit f13fb08.