spark-ssr 0.3.1 → 0.3.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "spark-ssr",
-  "version": "0.3.1",
+  "version": "0.3.3",
   "description": "Zero-config SSR for spark-html on Bun. The HTML template infers everything: filesystem routing, layouts, <spark-ssr> declarative data (SQL, URLs, globs, modules), auto CRUD with validation, guards, no-JS forms, schema + seeds, live updates, SEO. No build step.",
   "homepage": "https://wilkinnovo.github.io/spark-html",
   "type": "module",

package/src/hydrate.js

@@ -155,9 +155,20 @@ export function clientScript({ analysis, plan, table, cols, key, live }) {
   // live (§9): every write the server sees on this table pings the channel;
   // every open tab refetches through its own session. Realtime as one
   // attribute — no socket code, no pub/sub setup.
+  // Close on pagehide so the channel's HTTP/1.1 socket frees the instant we
+  // navigate away — a live EventSource that outlives its page eats one of the
+  // browser's ~6 per-host connections, and enough of them (rapid navigation
+  // across live pages) starve the next page's own request until the tab hangs.
+  // Reopen and refetch on a back/forward-cache restore.
   if (live && listVar) {
-    L.push(`const __live = new EventSource('/__spark/live');`);
-    L.push(`__live.onmessage = (e) => { if (e.data === '${table}') __refresh(); };`);
+    L.push(`let __live;`);
+    L.push(`function __openLive() {`);
+    L.push(`  __live = new EventSource('/__spark/live');`);
+    L.push(`  __live.onmessage = (e) => { if (e.data === '${table}') __refresh(); };`);
+    L.push(`}`);
+    L.push(`__openLive();`);
+    L.push(`addEventListener('pagehide', () => { if (__live) __live.close(); });`);
+    L.push(`addEventListener('pageshow', (e) => { if (e.persisted) { __openLive(); __refresh(); } });`);
   }
   return L.join('\n') + '\n';
 }

package/src/server.js

@@ -297,8 +297,30 @@ export async function serve(options = {}) {
   }
   // Reconnect-then-reload: after a server restart the EventSource reconnects,
   // and a fresh open following an error means "the server came back" — reload.
-  const RELOAD_CLIENT = '<script>(()=>{const e=new EventSource("/__spark/reload");let d=0;'
-    + 'e.onmessage=()=>location.reload();e.onerror=()=>{d=1};e.onopen=()=>{if(d)location.reload()}})()</script>';
+  // Close on pagehide: a live EventSource holds one of the browser's ~6
+  // per-host HTTP/1.1 sockets, and one that outlives its page starves the
+  // next navigation — rapid link-clicking (or a service worker that keeps a
+  // controlled client's sockets around) piles them up until the tab hangs
+  // loading. Freeing the socket the instant we leave keeps the pool clear;
+  // reopen if the page is restored from the back/forward cache.
+  // A service worker must never control a spark-ssr dev page: the dev server
+  // ships none, so a controller is always a leftover from a PREVIOUS project on
+  // this same localhost port. A stale caching worker serves old HTML (so fixes
+  // never appear), holds the per-host sockets live reload needs, and throws
+  // Cache.put() errors on aborted navigations — the tab hangs and no amount of
+  // rescaffolding helps, because deleting files never unregisters a worker.
+  // Unregister it, drop its caches, reload once (a session flag stops any loop).
+  const RELOAD_CLIENT = '<script>(()=>{'
+    + 'if(navigator.serviceWorker&&navigator.serviceWorker.controller&&!sessionStorage.getItem("__spark_sw")){'
+    + 'sessionStorage.setItem("__spark_sw","1");'
+    + 'console.warn("[spark-ssr] a stale service worker was controlling this dev page — unregistering it and clearing its caches");'
+    + 'navigator.serviceWorker.getRegistrations().then(r=>Promise.all(r.map(x=>x.unregister())))'
+    + '.then(()=>window.caches?caches.keys().then(k=>Promise.all(k.map(x=>caches.delete(x)))):0)'
+    + '.then(()=>location.reload());return}'
+    + 'let e,d=0;const open=()=>{e=new EventSource("/__spark/reload");'
+    + 'e.onmessage=()=>location.reload();e.onerror=()=>{d=1};e.onopen=()=>{if(d)location.reload()}};open();'
+    + 'addEventListener("pagehide",()=>{if(e)e.close()});'
+    + 'addEventListener("pageshow",v=>{if(v.persisted)open()})})()</script>';
 
   // Heartbeats keep every SSE socket outside Bun's idleTimeout (the default
   // would kill them at 10 s — and a killed reload socket reconnects, which