refactor: use Promise.all for service worker unregistration

Co-authored-by: hyoban <38493346+hyoban@users.noreply.github.com>

web/app/components/provider/serwist.tsx

@@ -1,3 +1,36 @@
 'use client'
 
-export { SerwistProvider } from '@serwist/turbopack/react'
+import { SerwistProvider } from '@serwist/turbopack/react'
+import { useEffect } from 'react'
+import { IS_DEV } from '@/config'
+
+export function PWAProvider({ children }: { children: React.ReactNode }) {
+  useEffect(() => {
+    if (IS_DEV && 'serviceWorker' in navigator) {
+      navigator.serviceWorker.getRegistrations()
+        .then((registrations) => {
+          return Promise.all(
+            registrations.map(registration => registration.unregister()),
+          )
+        })
+        .catch((error) => {
+          // Silently fail if service worker unregistration fails
+          // This is a development-only optimization and shouldn't block the app
+          console.warn('Failed to unregister service workers:', error)
+        })
+    }
+  }, [])
+
+  if (IS_DEV) {
+    return <>{children}</>
+  }
+
+  const basePath = process.env.NEXT_PUBLIC_BASE_PATH || ''
+  const swUrl = `${basePath}/serwist/sw.js`
+
+  return (
+    <SerwistProvider swUrl={swUrl}>
+      {children}
+    </SerwistProvider>
+  )
+}

web/app/layout.tsx

@@ -12,7 +12,7 @@ import { ToastProvider } from './components/base/toast'
 import BrowserInitializer from './components/browser-initializer'
 import { ReactScanLoader } from './components/devtools/react-scan/loader'
 import { I18nServerProvider } from './components/provider/i18n-server'
-import { SerwistProvider } from './components/provider/serwist'
+import { PWAProvider } from './components/provider/serwist'
 import SentryInitializer from './components/sentry-initializer'
 import RoutePrefixHandle from './routePrefixHandle'
 import './styles/globals.css'
@@ -40,9 +40,6 @@ const LocaleLayout = async ({
 }) => {
   const locale = await getLocaleOnServer()
 
-  const basePath = process.env.NEXT_PUBLIC_BASE_PATH || ''
-  const swUrl = `${basePath}/serwist/sw.js`
-
   const datasetMap: Record<DatasetAttr, string | undefined> = {
     [DatasetAttr.DATA_API_PREFIX]: process.env.NEXT_PUBLIC_API_PREFIX,
     [DatasetAttr.DATA_PUBLIC_API_PREFIX]: process.env.NEXT_PUBLIC_PUBLIC_API_PREFIX,
@@ -96,7 +93,7 @@ const LocaleLayout = async ({
         className="color-scheme h-full select-auto"
         {...datasetMap}
       >
-        <SerwistProvider swUrl={swUrl}>
+        <PWAProvider>
           <ReactScanLoader />
           <JotaiProvider>
             <ThemeProvider
@@ -124,7 +121,7 @@ const LocaleLayout = async ({
             </ThemeProvider>
           </JotaiProvider>
           <RoutePrefixHandle />
-        </SerwistProvider>
+        </PWAProvider>
       </body>
     </html>
   )