Uh oh!
There was an error while loading. Please reload this page.
Refactor: cross-refactoring スクリプト群のクロスリファクタリング試行 - #118
Merged
Conversation
…tion Codex の sentinel と result.json による完了判定を名前付き関数へ抽出する。 Item-Id: R1-001 Round: 1 Impl-Runtime: codex Impl-Model: default
…letion result.json の経過時間による完了判定を名前付き関数へ抽出する。 Item-Id: R1-001 Round: 1 Impl-Runtime: codex Impl-Model: default
fatal と warn の early error スキャンおよび判定を名前付き関数へ抽出する。 Item-Id: R1-001 Round: 1 Impl-Runtime: codex Impl-Model: default
…cation 適用範囲のコミット所有者と重複割当の検証を名前付き関数へ抽出する。 Item-Id: R1-002 Round: 1 Impl-Runtime: codex Impl-Model: default
割当不正時の項目取消、範囲 revert、起点更新を名前付き関数へ抽出する。 Item-Id: R1-002 Round: 1 Impl-Runtime: codex Impl-Model: default
項目ごとの検証、失敗時の取消、成功時の記録を名前付き関数へ抽出する。 Item-Id: R1-002 Round: 1 Impl-Runtime: codex Impl-Model: default
申告 SHA を完全 SHA に正規化して未割当コミットを検出する処理を apply と fix で共通化する。 Item-Id: R1-003 Round: 1 Impl-Runtime: codex Impl-Model: default
解決スレッド ID の型検証と GitHub 実状態との照合を名前付き関数へ抽出する。 Item-Id: R1-004 Round: 1 Impl-Runtime: codex Impl-Model: default
修正範囲、申告 SHA、git facts、未割当コミットの構築を名前付き関数へ抽出する。 Item-Id: R1-004 Round: 1 Impl-Runtime: codex Impl-Model: default
修正コミット facts を accepted と problems に分類する検証を名前付き関数へ抽出する。 Item-Id: R1-004 Round: 1 Impl-Runtime: codex Impl-Model: default
検証失敗時の修正範囲取消と成功時の項目コミット記録を名前付き関数へ抽出する。 Item-Id: R1-004 Round: 1 Impl-Runtime: codex Impl-Model: default
実装担当 bucket のモデル警告と成果指標加算を名前付き関数へ抽出する。 Item-Id: R1-005 Round: 1 Impl-Runtime: codex Impl-Model: default
初回レビュー承認率の加算を実装担当集計関数へ含める。 Item-Id: R1-005 Round: 1 Impl-Runtime: codex Impl-Model: default
レビュー担当ごとのモデル警告、時間、指摘、合意率の集計を名前付き関数へ抽出する。 Item-Id: R1-005 Round: 1 Impl-Runtime: codex Impl-Model: default
…rror" This reverts commit 3a27437.
…age completion" This reverts commit 5034910.
…l completion" This reverts commit 0a81b26.
merge-apply の検証で R1-001 が差分予算超過（実差分 668 行 / 予算 220 行）で 失敗し、続く R1-002 の取り消しが後続項目のコミットと競合して失敗した。 R1-003 以降は判定に到達していない。状態ファイルには適用結果が一切記録されて おらず、どのコミットが検証を通ったか確定できないため、ツリーを着手前 （e6d7222）へ戻す。 検証結果は issues/ の実機検証レポートに記録する。
収束ループを Pull Request #118 で実行し、提案の統合までは設計どおり動いたが、 適用結果の検証で失敗した項目を取り消す経路が破綻して進行を続行できなくなった。 発見した 9 件の不具合と、検証に至らなかった範囲を記録する。
編集対象・着手順・先に決めること・再検証の手順を記録する。 不具合の内容とエビデンスは実機検証レポートを参照する。
takemi-ohama
marked this pull request as ready for review
August 16, 2026 02:39
takemi-ohama added a commit
that referenced
this pull request
Aug 16, 2026
実機検証（PR #118）で収束ループが適用フェーズから先へ進めなくなった原因を直す。 - 取り消しが他項目のコミットと競合する問題を、範囲の巻き戻しと積み直しで解消 （分離できない位置関係のときはラウンド全件へ退避する） - 取り消し失敗を中断（終了コード 4）として扱い、「全件失敗」（2）と区別 - 項目ごとの判定を都度保存し、中断しても到達点を状態から読めるようにする - 取り消しへ着手する前に pending_drop / pending_push を立て、未検証の差分を残さない - --scope を適用・修正の検証にも反映し、生成物の同期を進行側の責務へ分離 - 提案の結果ファイル名にラウンド番号を追加し、次ラウンドでの上書きを防ぐ - gemini の作業ディレクトリへ読み取り除外を無効にする設定を配置 - 提案プロンプトへ語彙の許容値を機械的に列挙 - init が参加 CLI の認証状態を確認し、未認証なら中断 cross-review を 6 ラウンド回し、codex / gemini の両者が APPROVE で収束した。 テストは 430 件成功（着手前 387 件）。詳細は issues/issue-113-cross-refactoring-defect-fixes.md を参照。
This was referenced Aug 16, 2026
takemi-ohama added a commit
that referenced
this pull request
Aug 16, 2026
不具合 9 件の修正（#119 / v8.2.0）を実機で確かめた記録を残す。 修正した 9 件はすべて実機で成立し、前回（#118）は到達できなかったレビュー・判定・ 実装担当の輪番・集計まで通った。特に本丸だった取り消しの巻き戻しと積み直しは、 前回破綻したのと同じ条件（採用項目の多くが同一ファイルを触る）で成立している。 あわせて新しい不具合を 2 件見つけた（#121 で修正済み）。 - 10: pre-push の同期検査と範囲ルールが両立せず、あらゆる push が落ちる - 11: 適用で失敗した項目が「対象外」に入らず、同じ提案が再採用される 構造改善の成果 4 項目は取り込まない。#121 が同じ refactor.py を大きく変えており、 手でコンフリクトを解消すると「2 者のレビューを通った内容」という性質が失われる。 指摘の修正と再レビューの繰り返し、上限到達時の項目単位の見送りは依然として未到達。
This was referenced Aug 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
目的
/ndf:cross-refactoringを自分自身のスクリプト群へ適用し、収束ループが実機で成立するかを検証する。結果
提案の統合までは設計どおり動作したが、適用結果の検証で失敗した項目を取り消す経路が破綻し、進行を続行できなくなった。
検証レポート: issues/issue-113-cross-refactoring-trial-report.md
発見した不具合は 9 件で、うち 6 件は収束ループの続行を妨げる。詳細と修正の方向はレポートを参照。
この Pull Request の内容
検証を通っていない変更を残さないため、対象範囲は着手前の内容へ戻してある。差分は検証レポートと実行条件の記録のみ。
git diff e6d7222 HEAD -- plugins/が空scripts/build-runtime-plugins.sh --check: 通過未検証の範囲
レビューフェーズ、修正と再レビューの繰り返し、実装担当の輪番、収束判定、最終確認との連携、集計値の出力は実行に至っていない。
🤖 Generated with Claude Code