chore: commit all pending changes

static/sw.js

@@ -17,21 +17,51 @@ const STATIC_PREFIXES = [
     '/static/fonts/',
 ];
 
-const CACHE_EXACT = ['/manifest.json'];
-
-function isNetworkOnly(req) {
-    if (req.method !== 'GET') return true;
-    const accept = req.headers.get('Accept') || '';
-    if (accept.includes('text/event-stream')) return true;
-    const url = new URL(req.url);
+const CACHE_EXACT = ['/manifest.json'];
+
+function isHttpRequest(req) {
+    const url = new URL(req.url);
+    return url.protocol === 'http:' || url.protocol === 'https:';
+}
+
+function isNetworkOnly(req) {
+    if (req.method !== 'GET') return true;
+    const accept = req.headers.get('Accept') || '';
+    if (accept.includes('text/event-stream')) return true;
+    const url = new URL(req.url);
     return NETWORK_ONLY_PREFIXES.some(p => url.pathname.startsWith(p));
 }
 
-function isStaticAsset(req) {
-    const url = new URL(req.url);
-    if (CACHE_EXACT.includes(url.pathname)) return true;
-    return STATIC_PREFIXES.some(p => url.pathname.startsWith(p));
-}
+function isStaticAsset(req) {
+    const url = new URL(req.url);
+    if (CACHE_EXACT.includes(url.pathname)) return true;
+    return STATIC_PREFIXES.some(p => url.pathname.startsWith(p));
+}
+
+function fallbackResponse(req, status = 503) {
+    const accept = req.headers.get('Accept') || '';
+    if (accept.includes('application/json')) {
+        return new Response(
+            JSON.stringify({ status: 'error', message: 'Network unavailable' }),
+            {
+                status,
+                headers: { 'Content-Type': 'application/json' },
+            }
+        );
+    }
+
+    if (accept.includes('text/event-stream')) {
+        return new Response('', {
+            status,
+            headers: { 'Content-Type': 'text/event-stream' },
+        });
+    }
+
+    return new Response('Offline', {
+        status,
+        headers: { 'Content-Type': 'text/plain; charset=utf-8' },
+    });
+}
 
 self.addEventListener('install', (e) => {
     self.skipWaiting();
@@ -45,14 +75,21 @@ self.addEventListener('activate', (e) => {
     );
 });
 
-self.addEventListener('fetch', (e) => {
-    const req = e.request;
-
-    // Always bypass service worker for non-GET and streaming routes
-    if (isNetworkOnly(req)) {
-        e.respondWith(fetch(req));
-        return;
-    }
+self.addEventListener('fetch', (e) => {
+    const req = e.request;
+
+    // Ignore non-HTTP(S) requests so extensions/browser-internal URLs are untouched.
+    if (!isHttpRequest(req)) {
+        return;
+    }
+
+    // Always bypass service worker for non-GET and streaming routes
+    if (isNetworkOnly(req)) {
+        e.respondWith(
+            fetch(req).catch(() => fallbackResponse(req, 503))
+        );
+        return;
+    }
 
     // Cache-first for static assets
     if (isStaticAsset(req)) {
@@ -65,15 +102,15 @@ self.addEventListener('fetch', (e) => {
                             if (res && res.status === 200) cache.put(req, res.clone());
                         }).catch(() => {});
                         return cached;
-                    }
-                    return fetch(req).then(res => {
-                        if (res && res.status === 200) cache.put(req, res.clone());
-                        return res;
-                    });
-                })
-            )
-        );
-        return;
+                    }
+                    return fetch(req).then(res => {
+                        if (res && res.status === 200) cache.put(req, res.clone());
+                        return res;
+                    }).catch(() => fallbackResponse(req, 504));
+                })
+            )
+        );
+        return;
     }
 
     // Network-first for HTML pages