Changeset df9da7d in Klonkt

Timestamp:

07/16/2026 12:20:51 PM (8 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

72f936a

Parents:

4424af7

git-author:

Robin <roboburr@…> (07/12/2026 11:35:56 PM)

git-committer:

Robin <roboburr@…> (07/16/2026 12:20:51 PM)

Message:

Fix: PWA no longer shows stale homepage on flaky cold start

The service worker is network-first for navigations (fresh when online), but
its offline fallback served the '/' snapshot cached once at install time, so a
cold PWA launch on a slow/offline connection showed old data. Now refresh the
cached '/' on every successful homepage load, so the fallback is always the
latest seen page. Bumped cache version v16 → v17 so clients adopt the new SW.

Co-Authored-By: Claude Opus 4.8 <noreply@…>

File:

• src/server.js (modified) (2 diffs)

src/server.js

@@ -470 +470 @@
   res.set('Cache-Control', 'no-cache');
   res.send(`
-const CACHE_VERSION = 'pcms-v16-' + new Date().toISOString().split('T')[0];
+const CACHE_VERSION = 'pcms-v17-' + new Date().toISOString().split('T')[0];
 self.addEventListener('install', e => {
   e.waitUntil(caches.open(CACHE_VERSION).then(c => c.addAll(['/'])));
@@ -495 +495 @@
   try { if (new URL(e.request.url).origin !== self.location.origin) return; } catch (err) { return; }
   e.respondWith(
-    fetch(e.request).catch(() => caches.match('/').then(r => r || Response.error()))
+    fetch(e.request).then(resp => {
+      // Network-first: always serve fresh when online. Also refresh the '/' offline
+      // fallback with the homepage we just served, so a later cold start on a flaky or
+      // offline connection no longer shows the stale install-time snapshot ("old data
+      // on first PWA load").
+      try {
+        if (resp && resp.ok && new URL(e.request.url).pathname === '/') {
+          const copy = resp.clone();
+          e.waitUntil(caches.open(CACHE_VERSION).then(c => c.put('/', copy)).catch(() => {}));
+        }
+      } catch (err) { /* ignore cache refresh failures */ }
+      return resp;
+    }).catch(() => caches.match('/').then(r => r || Response.error()))
   );
 });