stateの整理

[HayatoYagi]

• VisualizerState.kt の肥大化
• Ready stateの分割
  • ready直下に derived propertyがたくさんあり、パフォーマンス上の懸念。 (related: Scope filter, navigation, and color state to Ready snapshot #135 (comment))
  • GitHubSnapshot, FilterState といった責務分割の仕方を見直すことで、derived state をそれぞれの子stateに委譲することができるのではないか？
• io.github.hayatoyagi.prvisualizer.ui.shared.computeConflictedDirs の存在
  • 複数のUIコンポーネントから呼び出されるなら、計算結果を state として保持しておいた方が良いのではないか？

[HayatoYagi]

補足です。

---

computeConflictedDirs について — 既に remember で保護済み

両方の呼び出し元でキャッシュされています：

• ExplorerPane.kt → remember(root, fileOverlayByPath, directoryOverlayByPath, expandedPaths) の中
• TreemapCanvas.kt → remember(fileOverlayByPath) の中

ただし根本問題は残ります。fileOverlayByPath が Ready の val として毎回再生成されるため、updateReady {} のたびに remember のキーが変わって無効化されます。computeConflictedDirs の結果を state に持つ案も、fileOverlayByPath が頻繁に変わる限り効果は同じです。

---

パフォーマンスの最重要ホットパス

最もコストが高いのはチェックボックス操作の経路です：

togglePr（PR チェックボックス1つ）
  → updateReady { copy(filterState = ...) }
  → Ready 再生成
  → filteredPrs 再計算       ← prSelection が変わっただけなのに不要
  → fileOverlayByPath 再計算  ← O(PR数 × ファイル数)、最重量
  → directoryOverlayByPath 再計算

チェックボックスを1つ変えても filteredPrs は変わらないはずなのに再計算される点が、委譲分割の最大の動機です。

---

委譲案の具体的な方向性

「FilterState 側に委譲」という方向は筋が良いと思います：

• filteredPrs → FilterState（allPrs + showDrafts + onlyMine + currentUser に依存）に持たせる
• selectedPrIds / selectedPrs → prSelection に依存する部分として分離
• fileOverlayByPath / directoryOverlayByPath → selectedPrs が変わったときだけ再計算

こうすると togglePr（= prSelection のみ変更）では filteredPrs が再計算されなくなります。computeConflictedDirs の state 化より、この分割の方が効果が大きいはずです。

[HayatoYagi]

追記です。

先ほどのコメントで「filteredPrs を FilterState に委譲」と書きましたが、そうすると FilterState が allPrs や viewerLogin を持つ必要が生じて責務が崩れます。訂正します。

---

子 state に切り出せるのは、その子 state 内で完結する属性だけ

ポイントは「どの操作で何が変わるか」を整理することです：

操作	変わるもの	変わらないはずのもの
togglePr	prSelection	filteredPrs（フィルター条件は同じ）
updateShowDrafts	filterState	—
selectDirectory	navigationState	filteredPrs, overlays
shufflePrColors	colorState	filteredPrs, overlays

togglePr がホットパスなのに filteredPrs まで毎回再計算されるのが問題の本質です。

---

FilteredView を切り出す案

filteredPrs を filterState（と snapshot）にだけ依存するネストオブジェクトにまとめると、togglePr の経路で使い回せます：

data class FilteredView(   // filterState + snapshot にだけ依存
    val filteredPrs: List<PullRequest>,
    val filteredPrIds: Set<String>,
)

data class Ready(
    val snapshot: GitHubSnapshot,
    val filteredView: FilteredView,          // togglePr では同じインスタンスを使い回せる
    val prSelection: PrSelection = PrSelection.allVisible(),
    val navigationState: NavigationState = NavigationState(),
    val colorState: ColorState = ColorState(),
) : SnapshotFetchState {
    val selectedPrIds = ...   // prSelection + filteredView にだけ依存
    val selectedPrs = ...
    val fileOverlayByPath = ...
    val directoryOverlayByPath = ...
}

togglePr で copy(prSelection = ...) すると filteredView は同じオブジェクトのまま渡されるため、filteredPrs の再計算が起きません。updateShowDrafts のときだけ FilteredView を再生成する、という形になります。