cleanup, only build font if there is a change

monkeytypegame · Jun 4, 2024 · ad6c83a · ad6c83a
1 parent 4258054
commit ad6c83a
Showing 1 changed file with 19 additions and 35 deletions.
diff --git a/frontend/vite.config.js b/frontend/vite.config.js
@@ -12,6 +12,7 @@ import { fontawesomeSubset } from "fontawesome-subset";
 
 //read used fontawesome icons from sources
 import { getFontawesomeConfig } from "./scripts/fontawesome.mjs";
+let cachedFontConfig = "";
 
 function pad(numbers, maxLength, fillString) {
   return numbers.map((number) =>
@@ -50,30 +51,32 @@ const BASE_CONFIG = {
       apply: "serve",
       buildStart() {
         const fontawesomeClasses = getFontawesomeConfig();
+        cachedFontConfig = JSON.stringify(fontawesomeClasses);
         fontawesomeSubset(fontawesomeClasses, "src/webfonts-generated", {
           targetFormats: ["woff2"],
         });
       },
-      async handleHotUpdate({ server, file }) {
+      async handleHotUpdate({ file }) {
         if (
           file.endsWith(".html") ||
           file.endsWith(".css") ||
           file.endsWith(".js") ||
           file.endsWith(".ts")
         ) {
           const fontawesomeClasses = getFontawesomeConfig();
-          await fontawesomeSubset(
-            fontawesomeClasses,
-            "src/webfonts-generated",
-            {
-              targetFormats: ["woff2"],
-            }
-          );
+          const newFontConfig = JSON.stringify(fontawesomeClasses);
+          if (cachedFontConfig != newFontConfig) {
+            await fontawesomeSubset(
+              fontawesomeClasses,
+              "src/webfonts-generated",
+              {
+                targetFormats: ["woff2"],
+              }
+            );
 
-          console.log("full reload");
-          server.ws.send({
-            type: "full-reload",
-          });
+            cachedFontConfig = newFontConfig;
+            return [];
+          }
         }
       },
     },
@@ -194,33 +197,14 @@ const BASE_CONFIG = {
   },
 };
 
-/** @type {import("vite").UserConfig} */
-const DEV_CONFIG = {
-  plugins: [
-    {
-      name: "force-disable-cache",
-      configureServer(server) {
-        server.middlewares.use((req, res, next) => {
-          if (req.url.endsWith("woff2")) {
-            res.setHeader(
-              "Cache-Control",
-              "no-cache, no-store, must-revalidate, max-age=1"
-            );
-            res.setHeader("Pragma", "no-cache");
-            res.setHeader("Etag", `W/"${Math.random()}"`);
-          }
-          next();
-        });
-      },
-    },
-  ],
-};
-
 /** @type {import("vite").UserConfig} */
 const BUILD_CONFIG = {
   plugins: [
     splitVendorChunkPlugin(),
     VitePWA({
+      devOptions: {
+        enabled: true,
+      },
       registerType: "autoUpdate",
       manifest: {
         short_name: "Monkeytype",
@@ -290,7 +274,7 @@ export default defineConfig(({ command }) => {
     }
     return mergeConfig(BASE_CONFIG, BUILD_CONFIG);
   } else {
-    return mergeConfig(BASE_CONFIG, DEV_CONFIG);
+    return BASE_CONFIG;
   }
 });