fix: improve worker error handling for asset fetching

Author: sshrihar

src/worker.js

@@ -19,8 +19,20 @@ export default {
     // Handle root redirect to listRegistry.json
     if (url.pathname === '/') {
       const redirectUrl = new URL('/listRegistry.json', url.origin);
-      const response = await env.ASSETS?.fetch(new Request(redirectUrl, request)) || 
-                      await fetch(redirectUrl);
+      let response;
+      
+      try {
+        // Try to fetch from assets first
+        if (env.ASSETS) {
+          response = await env.ASSETS.fetch(new Request(redirectUrl, request));
+        } else {
+          response = await fetch(redirectUrl);
+        }
+      } catch (error) {
+        console.error('Error fetching listRegistry.json:', error);
+        // Fallback to direct fetch
+        response = await fetch(redirectUrl);
+      }
 
       return new Response(response.body, {
         status: response.status,
@@ -36,7 +48,17 @@ export default {
     }
 
     // For all other requests, fetch from static assets and add CORS headers
-    const response = await env.ASSETS?.fetch(request) || await fetch(request);
+    let response;
+    try {
+      if (env.ASSETS) {
+        response = await env.ASSETS.fetch(request);
+      } else {
+        response = await fetch(request);
+      }
+    } catch (error) {
+      console.error('Error fetching asset:', error);
+      response = await fetch(request);
+    }
 
     // Clone the response to add CORS headers
     const newResponse = new Response(response.body, {