Skip to content

Docs: cross-refactoring 4 回目の実機試行の記録を残す - #129

Merged
takemi-ohama merged 3 commits into
mainfrom
docs/cross-refactoring-4th-trial-report
Aug 21, 2026
Merged

Docs: cross-refactoring 4 回目の実機試行の記録を残す#129
takemi-ohama merged 3 commits into
mainfrom
docs/cross-refactoring-4th-trial-report

Conversation

@takemi-ohama

Copy link
Copy Markdown
Contributor

目的

/ndf:cross-refactoring の 4 回目の実機試行(対象: Draft PR #128)の計画と結果を記録する。
リファクタリング自体は収束しなかったため成果物は取り込まず、記録だけを残す

背景

PR #127(NDF v8.5.0)で直した 5 件が実機で成立するかを確かめた。過去 3 回と同じ枠組みで、
--sync-command を渡す構成で回している。

結果

v8.5.0 の修正は 5 件とも成立した。

#直したこと判定確認方法
12生成物の同期コミット成立実機 + 旧版との A/B 再現
13同期の後段の失敗成立旧版との A/B 再現(実機では未到達)
14実装担当のコミット成立実機
15見送り後の読み取り同期成立実機
cross-review の投稿数の突き合わせ成立単体テスト + PR #127 のクロスレビュー

あわせて新しい不具合を 2 件見つけた。どちらも v8.5.0 で入ったものではなく、
v8.3.0 にも同じ構造がある。

#内容影響
16レビュー結果が欠け続けると進行が終わらない進行が止まらない。 差し戻し上限の出口が fix_base_sha を記録しないため、fix_rounds が増えず見送りへ到達しない
17実装担当に直せない指摘が渡る修正フェーズが空回りする

修正は本 PR の範囲外とし、別途対応する。

変更内容

ファイル内容
issues/issue-113-cross-refactoring-4th-trial.md試行計画
issues/issue-113-cross-refactoring-4th-trial-report.md試行結果

ドキュメントのみの変更で、コードと配布物は触っていない。

テスト

着手前・試行ブランチとも 463 passed。

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

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