feat: improve caching

- if addAll failed, show helpful log
- only cache if response.ok
- use more async by removing unnecessary awaits

src/consts.ts

@@ -6,4 +6,4 @@ export const SITE_AUTHOR_MASTODON = 'https://social.ayco.io/@ayo';
 export const SITE_PROJECT_REPO = 'https://github.com/ayoayco/Cozy';
 export const SITE_DESCRIPTION = 'The Web is Yours.';
 
-export const VERSION = 'Singing-Sirens';
+export const VERSION = 'Tired-Tourists';

src/sw.mjs

@@ -1,4 +1,4 @@
-import { logInfo } from './utils/logger'
+import { logError, logInfo } from './utils/logger'
 /**
  * Note: @ayco/astro-sw integration injects variables `__prefix`, `__version`, & `__assets`
  * -- find usage in `astro.config.mjs` integrations
@@ -21,20 +21,31 @@ const cleanOldCaches = async () => {
 const addResourcesToCache = async (resources) => {
     const cache = await caches.open(cacheName);
     logInfo('adding resources to cache...', { force: !!forceLogging, context: 'cozy-sw', data: resources })
-    await cache.addAll(resources);
+    try {
+        await cache.addAll(resources);
+    } catch(error) {
+        logError('failed to add resources to cache; make sure requests exists and that there are no duplicates', {force: !!forceLogging, context: 'cozy-sw', data: {
+            resources,
+            error
+        }})
+    }
 };
 
 const putInCache = async (request, response) => {
     const cache = await caches.open(cacheName);
-    logInfo('adding one response to cache...', { force: forceLogging, context: 'cozy-sw', data: request.url })
-    // if exists, replace
 
-    const keys = await cache.keys();
-    if (keys.includes(request)) {
-        cache.delete(request);
+    if (response.ok) {
+        logInfo('adding one response to cache...', { force: forceLogging, context: 'cozy-sw', data: request.url })
+
+        // if exists, replace
+        cache.keys().then( keys => {
+            if (keys.includes(request)) {
+                cache.delete(request);
+            }
+        });
+
+        cache.put(request, response);
     }
-
-    await cache.put(request, response);
 };