From bf22f65f910c05b734de15439eb74dddaa8e7a60 Mon Sep 17 00:00:00 2001
From: Mathis <gameurpro12gg@gmail.com>
Date: Sun, 4 Oct 2026 20:45:05 +0200
Subject: [PATCH] =?UTF-8?q?fix=20:=20service=20worker=20qui=20causait=20un?=
 =?UTF-8?q?=20=C3=A9cran=20blanc=20apr=C3=A8s=20mise=20=C3=A0=20jour?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- SW v3 : network-first sur index.html (plus de cache périmé)
- Assets versionnés (hash) : cache-first
- index.html : auto-update du SW + rechargement quand une nouvelle version prend le contrôle
- Résout le 'Unable to open asset URL' (référence à un JS/CSS qui n'existe plus)
---
 web/index.html   | 20 ++++++++++++++++++-
 web/public/sw.js | 52 ++++++++++++++++++++++++++++++++++++------------
 2 files changed, 58 insertions(+), 14 deletions(-)

diff --git a/web/index.html b/web/index.html
index fb16f0f..9c39029 100644
--- a/web/index.html
+++ b/web/index.html
@@ -33,7 +33,25 @@
     <script>
       if ('serviceWorker' in navigator) {
         window.addEventListener('load', () => {
-          navigator.serviceWorker.register('/sw.js').catch(() => {})
+          navigator.serviceWorker.register('/sw.js').then((reg) => {
+            // Si une nouvelle version du SW est disponible, on l'active.
+            reg.addEventListener('updatefound', () => {
+              const nw = reg.installing
+              if (!nw) return
+              nw.addEventListener('statechange', () => {
+                if (nw.state === 'installed' && navigator.serviceWorker.controller) {
+                  nw.postMessage({ type: 'SKIP_WAITING' })
+                }
+              })
+            })
+            // Quand un nouveau SW prend le contrôle, on recharge pour servir la nouvelle version.
+            let refreshing = false
+            navigator.serviceWorker.addEventListener('controllerchange', () => {
+              if (refreshing) return
+              refreshing = true
+              window.location.reload()
+            })
+          }).catch(() => {})
         })
       }
     </script>
diff --git a/web/public/sw.js b/web/public/sw.js
index 78dc714..a2fc8e6 100644
--- a/web/public/sw.js
+++ b/web/public/sw.js
@@ -1,16 +1,15 @@
 /**
  * Service worker Météo Exam.
- * - Cache les assets statiques (app shell) pour un démarrage hors ligne.
- * - Les appels API passent en network-first : les données restent fraîches.
+ *
+ * Stratégie :
+ *  - index.html : network-first (toujours la dernière version, pas de cache périmé).
+ *  - Assets versionnés (hash dans le nom) : cache-first (jamais changés après build).
+ *  - API / SSE : jamais caché.
  */
 
-const CACHE_NAME = 'meteo-exam-v2'
-const STATIC_ASSETS = ['/', '/index.html', '/manifest.webmanifest']
+const CACHE_NAME = 'meteo-exam-v3'
 
-self.addEventListener('install', (event) => {
-  event.waitUntil(
-    caches.open(CACHE_NAME).then((cache) => cache.addAll(STATIC_ASSETS)),
-  )
+self.addEventListener('install', () => {
   self.skipWaiting()
 })
 
@@ -27,13 +26,40 @@ self.addEventListener('fetch', (event) => {
   const { request } = event
   const url = new URL(request.url)
 
-  // API et flux SSE : toujours réseau, pas de cache.
-  if (url.pathname.startsWith('/api/')) return
+  // API et flux SSE : réseau uniquement.
+  if (url.pathname.startsWith('/api/') || url.pathname.startsWith('/stream')) return
 
-  // Assets statiques : cache d'abord, réseau en secours.
-  if (request.method === 'GET' && (url.origin === self.location.origin || url.hostname === 'localhost')) {
+  // Seulement les requêtes same-origin.
+  if (url.origin !== self.location.origin) return
+
+  // index.html (et racine) : network-first pour éviter le cache périmé.
+  if (url.pathname === '/' || url.pathname === '/index.html') {
     event.respondWith(
-      caches.match(request).then((cached) => cached ?? fetch(request).catch(() => caches.match('/index.html'))),
+      fetch(request)
+        .then((response) => {
+          const clone = response.clone()
+          caches.open(CACHE_NAME).then((cache) => cache.put(request, clone))
+          return response
+        })
+        .catch(() => caches.match(request)),
+    )
+    return
+  }
+
+  // Assets versionnés (hash dans le nom) : cache-first.
+  if (request.method === 'GET') {
+    event.respondWith(
+      caches.match(request).then(
+        (cached) =>
+          cached ??
+          fetch(request).then((response) => {
+            if (response.ok) {
+              const clone = response.clone()
+              caches.open(CACHE_NAME).then((cache) => cache.put(request, clone))
+            }
+            return response
+          }),
+      ),
     )
   }
 })