[ujiro99]

概要

設定画面を開く操作で、すでに設定画面のタブが開いていれば重複して開かず、既存のタブを再利用するようにしました。

変更内容

• src/services/chrome.ts に openOptionPage(path?) を追加
  • 全ウィンドウから設定画面のタブを探し、見つかればアクティブにしてウィンドウも前面に出す
  • クエリ付きのパス（?addPageRule=... など）が渡された場合は、既存タブをそのURLへ遷移させる
  • 見つからなければ新規タブで開く
• 以下の箇所を openOptionPage() に置き換え
  • 拡張機能アイコンのクリック（chrome.action.onClicked）
  • ServiceWorkerCommand.openOption
  • ページルールの追加・編集（ServiceWorkerCommand.addPageRule）
  • Hub からのコマンド編集（handleEditCommand）
• handleEditCommand では、既存タブを再利用した場合は編集完了後もそのタブを閉じない（新規に作成したタブのみ閉じる）

補足

• クエリ付きで既存タブを開き直すとページが再読み込みされるため、設定画面に保存前の入力があると失われる可能性があります

確認

• tsc --noEmit が通ること
• Extension のユニットテスト（全件）が通ること
• ブラウザ上で、設定画面を開いた状態で以下を行い、既存タブが再利用されること
  • 拡張機能アイコンのクリック
  • ページルールの追加・編集
  • Hub からのコマンド編集（編集後に既存タブが閉じないこと）

🤖 Generated with Claude Code

[claude]

レビュー結果

設定画面タブの再利用という目的に対して、実装は素直で読みやすく、テストも追加されています。全体として良い PR です。以下、指摘と提案です。

指摘(軽微〜中)

1. packages/extension/src/services/chrome.ts の openOptionPage(startsWith(optionPageUrl) 判定)
   前方一致なので、将来 options_page.html.bak のような別パスができた場合に誤検知します。new URL(...) で origin + pathname を比較する方が堅牢です。

2. ハッシュのみが異なる場合の挙動(chrome.ts の props.url 設定部)
   ハッシュだけが異なる URL への tabs.update は再読み込みなしのフラグメント遷移になります。現状は ?editCommand=... などクエリが付くため問題ありませんが、判定が path !== OPTION_PAGE_PATH なので「クエリの有無」という意図とずれています。コメントや条件を揃えると明確です。

3. 保存前の入力が失われる点(PR 説明にも記載あり)
   ページルール追加・Hub 編集で既存タブを再読み込みするため、未保存の編集内容が警告なく消えます。将来的には再読み込みせず runtime.sendMessage でパラメータを渡す方式も検討に値します。

4. packages/extension/src/services/hub/serviceWorker.ts の handleEditCommand
   再利用時は editTabId を設定しないため、編集完了後にタブが閉じない設計は妥当です。再利用タブがユーザー操作で閉じられた場合も、EDIT_CONNECT_TIMEOUT_MS でセッションが回収されることを確認してください。

5. packages/extension/src/service_worker.ts の Object.assign 整形変更
   機能と無関係なフォーマット変更が混ざっています。害はありませんが、分離が望ましいです。

良い点

• excludeTabId でキャンセル済みセッションのタブを再利用しない配慮(EC-05)。
• pendingUrl で読み込み中のタブも検出(COP-05)。
• すべての呼び出し元で .catch によりエラー処理。
• 最後にフォーカスされたウィンドウを優先する挙動もテスト済み。

テスト

• openOptionPage の主要分岐は網羅されています。
• 追加提案: windows.update が失敗した場合や tabs.update が undefined を返す場合(tab ?? existing)のテスト。

セキュリティ上の懸念は見当たりません(対象は拡張機能自身の URL のみ)。