fix(#675): stop MarkdownEditor re-render loop blocking worktree URL updates

[Kewton]

Summary

.md ファイル表示中にサイドバーから別ブランチを選択しても URL が更新されず画面遷移しないバグを修正。MarkdownEditor 経由で発生していた無限 re-render ループが router.push の App Router transition を starve させていた根本原因を A案 + B案 の多層防御で解消。

Closes #675

Root Cause

useFileTabs の return が毎レンダー新オブジェクト参照を返すため、WorktreeDetailRefactored の 4 つの useCallback (deps=[fileTabs]) が毎レンダー再生成される。これが FilePanelContent → MarkdownEditor へ不安定 prop として伝播し、useEffect([isDirty, onDirtyChange]) が毎レンダー発火 → dispatch({type:'SET_DIRTY'}) → reducer が同値でも新 state を返す → 上流再レンダー → ループ。高優先度 dispatch が積まれ続けるため router.push が起動した低優先度 transition が starve し URL が commit されない。.md 以外で再現しないのは MarkdownEditor だけが onDirtyChange useEffect を持つため。

Changes

Fixed

• A案: src/components/worktree/WorktreeDetailRefactored.tsx の handleLoadContent / handleLoadError / handleSetLoading / handleDirtyChange の useCallback deps を [fileTabs] から [fileTabs.dispatch] に変更。dispatch は useReducer 由来で identity stable なためコールバック identity が安定し、下流への不安定 prop 伝播が止まる。
• B案: src/hooks/useFileTabs.ts の SET_DIRTY reducer で対象 tab の isDirty が既に同値なら state を同一参照のまま返す no-op 判定を追加。useReducer は state 参照が同一なら再レンダーをスキップするため、将来別経路で deps が崩れても SET_DIRTY 経由のループは構造的に防がれる多層防御。

Added

• tests/unit/hooks/useFileTabs.test.ts: SET_DIRTY 同値 no-op 検証 3 件 (false→false / true→true / double-apply による state 参照安定性)。

Out of Scope

同ファイル (WorktreeDetailRefactored.tsx) 内の他 5 箇所 (L551/L570/L581/L875/L904) にも『fileTabs 全体を deps に置く』同種アンチパターンが残るが、Issue #675 の再現経路とは別系統のため別 Issue として起票予定。

Test Results

Unit Tests

npm run test:unit
Test Files  340 passed (340)
Tests       6384 passed | 7 skipped (6391)
Duration    14.27s

Lint & Type Check

• ESLint: 0 errors / 0 warnings
• TypeScript: 0 errors (tsc --noEmit exit 0)

Build

npm run build
Compiled successfully

Manual Verification (recommended before merge)

• Worktree A で .md を開いた状態でサイドバーから Worktree B を選択 → URL が /worktrees/B に変わる
• .yaml / .yml でも同様に遷移する (MarkdownEditor を経由する全拡張子で同じ構造のため)
• .png / .json / .pdf で回帰なし (元々再現しないので挙動維持のはず)

Checklist

• Unit tests pass
• Lint check passes
• Type check passes
• Build succeeds
• No console.log in production code
• Scope kept minimal (no unrelated refactoring)

---

Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com