🚀 feat: Enhance PWA and asset caching strategy (danny-avila#3822)

* chore: Update VitePWA registerType to 'autoUpdate', add cache busting to static file outputs

* chore: disable windows frontend workflow for now

Author: danny-avila

.github/workflows/frontend-review.yml

@@ -33,24 +33,25 @@ jobs:
         run: npm run test:ci --verbose
         working-directory: client
 
-  tests_frontend_windows:
-    name: Run frontend unit tests on Windows
-    timeout-minutes: 60
-    runs-on: windows-latest
-    steps:
-      - uses: actions/checkout@v4
-      - name: Use Node.js 20.x
-        uses: actions/setup-node@v4
-        with:
-          node-version: 20
-          cache: 'npm'
-
-      - name: Install dependencies
-        run: npm ci
-
-      - name: Build Client
-        run: npm run frontend:ci
-
-      - name: Run unit tests
-        run: npm run test:ci --verbose
-        working-directory: client
+  # Windows job is commented out
+  # tests_frontend_windows:
+  #   name: Run frontend unit tests on Windows
+  #   timeout-minutes: 60
+  #   runs-on: windows-latest
+  #   steps:
+  #     - uses: actions/checkout@v4
+  #     - name: Use Node.js 20.x
+  #       uses: actions/setup-node@v4
+  #       with:
+  #         node-version: 20
+  #         cache: 'npm'
+
+  #     - name: Install dependencies
+  #       run: npm ci
+
+  #     - name: Build Client
+  #       run: npm run frontend:ci
+
+  #     - name: Run unit tests
+  #       run: npm run test:ci --verbose
+  #       working-directory: client

client/vite.config.ts

@@ -55,7 +55,7 @@ export default defineConfig({
     nodePolyfills(),
     VitePWA({
       injectRegister: 'auto', // 'auto' | 'manual' | 'disabled'
-      registerType: 'prompt', // 'prompt' | 'auto' | 'disabled'
+      registerType: 'autoUpdate', // 'prompt' | 'autoUpdate'
       devOptions: {
         enabled: false, // enable/disable registering SW in development mode
       },
@@ -118,6 +118,9 @@ export default defineConfig({
             return 'vendor';
           }
         },
+        entryFileNames: 'assets/[name].[hash].js',
+        chunkFileNames: 'assets/[name].[hash].js',
+        assetFileNames: 'assets/[name].[hash].[ext]',
       },
       /**
        * Ignore "use client" waning since we are not using SSR