Fix: cross-review fix結果 resolved_threads が int だと merge-fix が TypeError で落ちる問題の堅牢化 (#25)

[takemi-ohama]

Summary

• plan: issues/PLAN25_cross-review-merge-fix-count-normalize.md
• 関連 issue: cross-review: fix結果の resolved_threads が int だと state.py merge-fix が TypeError で落ちる（スキーマ不整合 + 受理が非寛容） #25
• /ndf:cross-review の修正フェーズで fix サブエージェントが fix-pr<PR>-result.json の
  resolved_threads を int(件数) で書くと state.py merge-fix が len() を int に適用して
  TypeError でクラッシュしループが停止する問題を、両側で堅牢化する。

変更内容

1. state.py（後方互換で受理を寛容化）

• _count() 正規化ヘルパを追加（int / list / None いずれも件数 int に正規化、bool は除外）
• cmd_merge_fix の deferred / rejected / resolved_threads を _count() 経由に変更
• deferred ループに isinstance(list) ガードを追加（int 混入時の 'int' object is not iterable を防止）

2. ドキュメント整合（そもそも int を書きにくくする）

• docs/02-fix-and-rotation.md: 戻り値スキーマに「resolved_threads/deferred/rejected は
  必ず list、件数(int) を書かない」注意書きを追加
• docs/01-state-and-review.md L75: fix結果側は list・state.json 側は len()(int) と区別して記述

Test plan

• _count() を int / list / tuple / None / bool / 空list で正規化検証（全て期待値一致）
• python3 -c ast.parse 構文チェック / claude plugin validate 通過（警告は既存・無関係）
• (レビュー観点) int を書いた fix-pr<PR>-result.json で merge-fix がクラッシュしないこと
• (レビュー観点) list の場合に従来どおり件数が state.json に保存されること（回帰なし）

[takemi-ohama]

🤖 cross-review | round 1 | gemini | APPROVE

PLAN25 の仕様に沿って、int や list の混在に対する堅牢化とドキュメントの明確化が適切に実装されていることを確認しました。

[takemi-ohama]

🤖 cross-review | round 1 | codex | REQUEST_CHANGES

バグ修正対象の int 入力を固定する回帰テストを追加してください。

[takemi-ohama]

fix round 1/8 対応サマリ

codex review (REQUEST_CHANGES) の指摘 1 件に対応しました。

重要度	件数
critical	0
major	0
minor	1
nit	0

• 対応: 1 件 / deferred: 0 件 / rejected: 0 件
• 対応内容: _count() の int/list 正規化に対する回帰テストを tests/test_state_merge_fix.py に追加。_count() 単体テスト (int/list/tuple/None/空list/bool=False/想定外型) + cmd_merge_fix 結合テスト (resolved_threads/deferred/rejected が int でも TypeError を出さず件数保存・deferred ループ非クラッシュ・list 時の deferred_nits 展開) を検証。
• テスト結果: pytest 32 passed
• commit: 1322177

resolve 済み thread: PRRT_kwDOSII3uc6IL6Pt

[takemi-ohama]

🤖 cross-review | round 2 | gemini | REQUEST_CHANGES

PLAN25 の目的である TypeError: object of type 'int' has no len() クラッシュの修正として、正規化ヘルパー _count の導入と型ガードは非常に適切です。
一方で、LLM の出力揺らぎ（スキーマ違反）に対する堅牢性をさらに高めるため、以下のエッジケースへの対応を追加することを提案します。

• deferred の要素が辞書でない（文字列など）場合の展開時 TypeError の回避
• 数値が文字列（"3"）として出力された場合の考慮

[takemi-ohama]

🤖 cross-review | round 2 | codex | APPROVE

修正必須の指摘はありません。

[takemi-ohama]

fix round2 対応サマリ

gemini round2 の指摘2件に対応しました (commit cd95be0)。

重要度	内容	対応
major / 堅牢性	cmd_merge_fix の deferred 展開ループに dict ガード追加 (文字列リスト等で TypeError: 'str' object is not a mapping を防止)	修正 + 回帰テスト追加
minor / 堅牢性	_count() で数値文字列 ("3" 等) を許容 (isdigit() で非負整数のみ)	修正 + 回帰テスト追加

• 対応件数: major=1 / minor=1 (deferred=0, rejected=0)
• 追加テスト: test_merge_fix_deferred_non_dict_elements_are_skipped、_count parametrize に数値文字列ケース追加
• pytest: tests/test_state_merge_fix.py 41 passed / cross-review 全体 93 passed
• 対応した2 thread は Resolve 済み

@Reviewer 再レビューをお願いします。

[takemi-ohama]

🤖 cross-review | round 3 | gemini | APPROVE

差分を確認しました。修正すべき点はありません。

[takemi-ohama]

🤖 cross-review | round 3 | codex | REQUEST_CHANGES

deferred の不正要素をスキップする場合、state に保存する件数も実際に保持した deferred nit と一致させてください。

[takemi-ohama]

fix round3 対応サマリ

codex round3 (REQUEST_CHANGES) の指摘 1 件に対応しました (commit 04c9d58)。

重要度	件数
critical	0
major	1
minor	0
nit	0

• 対応: 1 件 / deferred: 0 件 / rejected: 0 件
• 対応内容 (major / 正確性): cmd_merge_fix で deferred を dict 要素のみへ正規化 (_deferred_nits) してから fix dict を組み立て、rounds[].fix["deferred"] に正規化後件数 (len(_deferred_nits)) を保存。後段の deferred_nits 展開ループも同じ _deferred_nits を走査するため、保存件数と実展開件数が一致するようになりました。resolved_threads / rejected は後段ループが無いため _count() を維持。
• テスト: 回帰テスト test_merge_fix_deferred_count_matches_expanded_nits 追加 + 既存テストの期待値を整合 (保存件数=dict件数) に更新。pytest tests/test_state_merge_fix.py 42 passed / cross-review 全体 94 passed。
• 対応 thread (PRRT_kwDOSII3uc6IMLGS) は Resolve 済み。

@Reviewer 再レビューをお願いします。

[takemi-ohama]

🤖 cross-review | round 4 | codex | REQUEST_CHANGES

deferred の int 後方互換が state の保存件数で失われているため、ラウンドサマリと fix 結果の整合を保つ修正が必要です。

[takemi-ohama]

🤖 cross-review | round 4 | gemini | APPROVE

LLM出力の揺らぎ（空白の混入やリストの欠落）に対するいくつかのエッジケースのカバーをインラインで提案します。

[takemi-ohama]

/ndf:fix round 4/8 対応サマリ

commit: e35b9d4be65933fe3ccc4696470672c78e439912

対応件数: 4 件 (major=1 / minor=3 / critical=0 / nit=0)

統一設計 (round3/round4 の対立を両立)

rounds[].fix["deferred"] の保存件数を単一整合ルールに統一しました:

• deferred が list / dict (構造化データ) → deferred_nits へ展開する dict 要素の件数 (len(_deferred_nits))
• deferred が int / 数値文字列 (per-item を失った劣化表現) → _count() の値 (件数を失わない / 展開不能なので nits は空)
• それ以外 → 0

→ mixed list は dict 件数 (round3 懸念解消)、int は件数保持 (round4 懸念解消) を両立。

変更内容

• codex major (#3381649905): deferred が int でも _count() 件数を保存
• gemini minor Sync: takemi-ohama/ai-plugins から最新状態を反映 (e71f099) #1 (#3381675778): _count() に単一 dict → 1 を追加
• gemini minor Docs: AGENTS.md のリポジトリURLを devbasex/ai-plugins に更新 #2 (#3381675786): _count() を strip().isdigit() で空白許容
• gemini minor Fix: cross-review monitor.py の EARLY_ERROR 誤検知を修正 (ndf v4.7.2) #3 (#3381675793): 単一 dict deferred のフォールバック追加

_count() 判定順序: bool→0 / int→そのまま / dict→1 / list,tuple→len / str(strip後isdigit)→int / その他→0

テスト

tests/test_state_merge_fix.py 更新 — pytest 48 passed (cross-review 全体 100 passed)。

[takemi-ohama]

🤖 cross-review | round 5 | gemini | APPROVE

_count() の単一 dict フォールバックや前後空白の strip 処理、および deferred の件数保存・展開の統一ルールが正しく実装されており、網羅的なテストも追加されていることを確認しました。エッジケースへの対応として十分に堅牢です。

[takemi-ohama]

🤖 cross-review | round 5 | codex | APPROVE

修正必須の重複しない指摘はありません。