fix: improve manifest reading and caching logic; invalidate cache on manifest changes

Author: Yukaii

lib/vite-helpers.js

@@ -2,6 +2,7 @@ const fs = require('fs')
 const path = require('path')
 
 let manifest = null
+const assetsCache = {}
 
 // Configuration for manual CSS ordering
 const CSS_ORDER_CONFIG = {
@@ -26,18 +27,22 @@ function readManifest () {
   if (process.env.NODE_ENV !== 'production') {
     return {} // No manifest needed in dev
   }
-  if (manifest) {
-    return manifest
-  }
+
   try {
     const manifestPath = path.resolve(__dirname, '../public/build/.vite/manifest.json')
-    if (fs.existsSync(manifestPath)) {
-      manifest = JSON.parse(fs.readFileSync(manifestPath, 'utf-8'))
-      return manifest
-    } else {
+    if (!fs.existsSync(manifestPath)) {
       console.error('Vite manifest.json not found! Run `npm run build`.')
       return {}
     }
+
+    // Check if manifest has changed
+    const currentManifest = JSON.parse(fs.readFileSync(manifestPath, 'utf-8'))
+    if (!manifest || JSON.stringify(manifest) !== JSON.stringify(currentManifest)) {
+      manifest = currentManifest
+      // Invalidate cache when manifest changes
+      Object.keys(assetsCache).forEach(key => delete assetsCache[key])
+    }
+    return manifest
   } catch (err) {
     console.error('Error reading Vite manifest.json:', err)
     return {}
@@ -46,6 +51,11 @@ function readManifest () {
 
 // Helper function to get asset paths for an entry point
 function getViteAssets(entryName) {
+  // Check cache first
+  if (assetsCache[entryName]) {
+    return assetsCache[entryName];
+  }
+
   const manifestData = readManifest();
   const assets = { js: [], css: [] };
   const isProduction = process.env.NODE_ENV === 'production';
@@ -84,6 +94,9 @@ function getViteAssets(entryName) {
     assets.css = orderedCss;
   }
 
+  // Cache the result
+  assetsCache[entryName] = assets;
+
   return assets;
 }