Fix DiffTab and AIReviewTab Monaco theme handling, layout, and tab reuse

[Copilot]

• Fix DiffTab and AIReviewTab theme handling with defineAndSetMonacoThemes
• Fix DiffTab height issue - single file diff now uses full container height with height: 100%
• Increase multiple files diff height from 360px to 500px
• Remove as any type casts - use proper ThemeColors type from @/context/ThemeContext
• Fix DiffTab opening multiple tabs in same pane - improved shouldReuseTab to compare commit IDs and file paths
• Fix theme not being applied after tab switch - removed caching that was skipping setTheme calls
• Simplify: removed updateDiffTabContent and extended updateTabContent to handle both editor and diff tabs

Original prompt

コードエディターの方と違って、difftabが、テーマの設定ができてない。いつもライトになってる。より良くして。

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[Stasshe]

@copilot あと、なぜかdiffタブの下の方が、切れてる。下までちゃんと表示されてない。固定になってる。

[Stasshe]

@copilot あと、この問題が直ってない。

切れてる。下までちゃんと表示されてない。固定になってる。
diffが、1つのファイルの時。後、複数あるやつの方でも、もう少し高さを取るように。

[Copilot]

@copilot あと、なぜかdiffタブの下の方が、切れてる。下までちゃんと表示されてない。固定になってる。

修正しました (34aa4fb)。単一ファイルのDiffの場合、固定高さ(360px)ではなく、コンテナの全高さを使用するように変更しました。複数ファイルの場合は従来通りスクロール可能な固定高さです。

また、colors as anyの型キャストもThemeColors型を正しく使用するように修正しました。

[Stasshe]

@copilot difftabが同じペーンで、何個もひらけてしまう問題も、ついでに修正して。後、できれば、codeeditor同士のように、別ペーンで同じファイル編集したときに、diffタブでも、リアルタイムで変更を共有し合うようにしたい。

[Copilot]

@copilot あと、この問題が直ってない。

切れてる。下までちゃんと表示されてない。固定になってる。
diffが、1つのファイルの時。後、複数あるや...

修正しました (0f2a8e1)。

• 単一ファイルのDiff: height: 100%を追加してコンテナの全高さを使用
• 複数ファイルのDiff: 360px → 500px に高さを増加

[Copilot]

@copilot difftabが同じペーンで、何個もひらけてしまう問題も、ついでに修正して。後、できれば、codeeditor同士のように、別ペーンで同じファイル編��...

修正しました (5fb651f)。

1. 同じペーンで複数開けてしまう問題: shouldReuseTabを改善し、コミットIDとファイルパスで比較するようにしました。同じDiffを開こうとすると既存タブが再利用されます。

2. リアルタイム変更共有: fileRepository.addChangeListenerを使って、別ペーンで同じファイルが編集された際にDiffTabのコンテンツも自動更新されるようにしました。

[Stasshe]

@copilot リアルタイム同期について。そのコードは間違えています。デバウンス待機状態では、savefileされないので。メモリ上で管理する必要がある。page.tsxだったかな？

[Copilot]

@copilot リアルタイム同期について。そのコードは間違えています。デバウンス待機状態では、savefileされないので。メモリ上で管理する必要がある。...

修正しました (041a62e)。

fileRepository.addChangeListenerを削除し、メモリベースのupdateDiffTabContent関数をtabStoreに追加しました。これはCodeEditorのupdateTabContentと同じパターンで、デバウンス待機中でも全ペーンの同じパスを持つDiffTabのコンテンツがリアルタイムで同期されます。

[Stasshe]

@copilot この問題が解決してない。修正して。また、diffタブから、to or from, 切り替えた時、全てのコードエディタまでテーマ再設定される。意味わからん。重い。

[Copilot]

@copilot この問題が解決してない。修正して。また、diffタブから、to or from, 切り替えた時、全てのコードエディタまでテーマ再設定される。意味わか�...

修正しました (bb7ca19)。

1. テーマ再設定問題: defineAndSetMonacoThemesを最適化し、テーマが既に同じ場合はsetThemeを呼び出さないようにしました。これでDiffTab切り替え時に全エディタのテーマが再設定されなくなります。

2. リアルタイム同期:

   • updateTabContentがDiffTabも更新するように拡張
   • updateDiffTabContentがEditorTabも更新するように拡張
   • EditorTabで編集するとDiffTabに反映、DiffTabで編集するとEditorTabに反映されます

[Stasshe]

@copilot 2じゃダメ？updateTabContentを拡張してDiffTab対応にし、updateDiffTabContentを削除

[Copilot]

@copilot 2じゃダメ？updateTabContentを拡張してDiffTab対応にし、updateDiffTabContentを削除

修正しました (3141796)。

updateDiffTabContentを削除し、updateTabContentを拡張してeditorとdiff両方に対応するようにしました。これでコードがシンプルになりました。

[Stasshe]

@copilot
謎に重いのは治った。しかし、すべてのeditorでテーマ設定ガされない