From e97f4cae9a6e755b6285d92e0f4af4fa6eb97d44 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E9=9B=A8=E9=9C=96=E9=93=83?= Date: Fri, 29 May 2026 20:15:54 +0800 Subject: [PATCH] refactor(RightClickRestorer): simplify setDomain logic Replace nested if/try/catch with a single ternary expression. isUnsupportedPage() already handles invalid URLs, so new URL() is safe here. Co-Authored-By: Claude Opus 4.8 --- src/pages/RightClickRestorer/useRightClickRestorer.ts | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/src/pages/RightClickRestorer/useRightClickRestorer.ts b/src/pages/RightClickRestorer/useRightClickRestorer.ts index f7abeb4..d1ec990 100644 --- a/src/pages/RightClickRestorer/useRightClickRestorer.ts +++ b/src/pages/RightClickRestorer/useRightClickRestorer.ts @@ -39,13 +39,7 @@ export function useRightClickRestorer(): UseRightClickRestorerReturn { const [tab] = await chrome.tabs.query({ active: true, currentWindow: true }); const url = tab?.url; - if (url) { - try { - setDomain(new URL(url).hostname); - } catch { - setDomain(''); - } - } + setDomain(url ? new URL(url).hostname : ''); if (isUnsupportedPage(url)) { setIsUnsupported(true);