refactor: workspace.ts に getActiveTab helper を抽出 (#257)

[ymnao]

概要

src/stores/workspace.ts 内で state.tabs.find((t) => t.id === state.activeTabId) パターンが 3 箇所で重複していたため、module-local helper getActiveTab(state: WorkspaceState): Tab | undefined として抽出。

関連 Issue

closes #257

移行 Stage

(該当なし — store 内部の小規模 refactor)

• Stage 0: 雛形 + フロント表示確認
• Stage 1: ファイル I/O
• Stage 2: ワークスペース・ファイルツリー・ファイル監視
• Stage 3: 全文検索（ripgrep sidecar）
• Stage 4: Git Sync
• Stage 5: OGP / PDF / アップデート
• Stage 6: 仕上げ・配布・切り替え

変更内容

• src/stores/workspace.ts に module-local helper を追加: function getActiveTab(state: WorkspaceState): Tab | undefined
• 以下 3 箇所で state.tabs.find((t) => t.id === state.activeTabId) を getActiveTab(state) に置換:
  • L62 navigateHistory 内
  • L86 selectNavigation selector
  • L201 navigateInTab 内
• findIndex 系 (L77 activateTabByOffset / L285 closeTabsByPrefix) と別配列 find (L291) は対象配列または用途が異なるため scope 外
• Issue 起票時の Option A 採用 (Option B = helper 化せず現状維持 は不採用、3 箇所同一パターンの集約による intent clarification 効果を優先)

動作確認

• biome check src/stores/workspace.ts — clean
• tsc --noEmit (web / node / e2e 3 config) — clean
• vitest run — 2230 pass / 2 skipped (98 file)
• /code-review low — (none)
• /code-review medium — [] (8 angle 全 (none)、Reuse 2 件 = L77/L285 findIndex 非対称は scope 外として skip)
• /simplify pass 1 — 4 reviewer 全 (none)

スクリーンショット

(該当なし — store 内部 refactor、UI 変更なし)