Skip to content

Docs: cross-refactoring 再々検証の結果を残す - #126

Merged
takemi-ohama merged 1 commit into
mainfrom
docs/cross-refactoring-re-retrial-report
Aug 18, 2026
Merged

Docs: cross-refactoring 再々検証の結果を残す#126
takemi-ohama merged 1 commit into
mainfrom
docs/cross-refactoring-re-retrial-report

Conversation

@takemi-ohama

Copy link
Copy Markdown
Contributor

概要

/ndf:cross-refactoring未到達だった 2 経路を実機で通した記録です。対象は Draft PR #125(検証後に閉じました)。

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

この 2 つは 1 回目(#118)・2 回目(#120)とも一度も実行されていませんでした。

結果

2 経路とも成立しました。 あわせて PR #121 で入れた 3 経路(公開の責務 / 生成物の同期 / 対象外への記録)も確認できています。

一方で新しく 4 件の不具合を見つけました。

#不具合影響
12生成物の同期コミットが必ず失敗する(_git_outstrip と固定幅パースの衝突)進行が止まる。生成物を持つリポジトリで毎回発生
13同期の後段で失敗すると差分が作業ツリーに残る次の実行が清浄性の検査で止まり、保留中の push を再試行できない
14実装担当がコミットを作れない条件が生じる修正フェーズが 0 コミットで終わる。迂回するかどうかがランタイム任せ
15見送りの後に読み取り用の作業ディレクトリを同期していない次ラウンドの提案が存在しないコードを対象にする

12 と 13 は進行を止めるため、優先して直す必要があります。

実行条件

指摘が出る確率を上げるため、修正の上限を 1 に下げ、1 ラウンドの採用上限を 8 へ上げました。

--max-outer-rounds 3 --max-fix-rounds 1 --max-items-per-round 8

変更内容

  • issues/issue-113-cross-refactoring-re-retrial.md を追加
  • 役目を終えた引継ぎメモ issues/issue-113-cross-refactoring-re-retrial-handoff.md を削除

コードの変更はありません。不具合 12〜15 の修正は別 PR で行います。

テスト

着手前のベースラインは 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 経路(指摘の修正と再レビュー / 上限到達時の項目単位の見送り)を
実機で通した。あわせて PR #121 で入れた 3 経路も確認し、新しく 4 件の不具合を
見つけた。うち 2 件は生成物を持つリポジトリで進行を止める。
役目を終えた引継ぎメモは、このレポートへ置き換えた。
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WxxF27RXyg3QMjayii5dp1
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@takemi-ohama