Improve readability and usability

staticfiles/apps.py

@@ -3,8 +3,9 @@ from pathlib import Path
 from django.contrib.staticfiles.apps import StaticFilesConfig
 
 GENERATED_ROOT = Path(__file__).parent.resolve() / "generated"
-BUNDLED_ROOT = GENERATED_ROOT / "bundled"
-IGNORE_PATTERNS_BUNDLED = ["bundled/*"]
+BUNDLED_FOLDER_NAME = "bundled"
+BUNDLED_ROOT = GENERATED_ROOT / BUNDLED_FOLDER_NAME
+IGNORE_PATTERNS_BUNDLED = [f"{BUNDLED_FOLDER_NAME}/*"]
 IGNORE_PATTERNS_SCSS = ["*.scss"]
 IGNORE_PATTERNS_TYPESCRIPT = ["*.ts"]
 IGNORE_PATTERNS = [

staticfiles/processors.py

@@ -12,28 +12,64 @@ import sass
 from django.conf import settings
 
 from sith.urls import api
-from staticfiles.apps import BUNDLED_ROOT, GENERATED_ROOT
+from staticfiles.apps import BUNDLED_FOLDER_NAME, BUNDLED_ROOT, GENERATED_ROOT
 
 
 @dataclass
 class JsBundlerManifestEntry:
-    out: str
     src: str
+    out: str
 
     @classmethod
     def from_json_entry(cls, entry: dict[str, any]) -> list[Self]:
+        # We have two parts for a manifest entry
+        # The `src` element which is what the user asks django as a static
+        # The `out` element which is it's real name in the output static folder
+
+        # For the src part:
+        # The manifest file contains the path of the file relative to the project root
+        # We want the relative path of the file inside their respective static folder
+        # because that's what the user types when importing statics and that's what django gives us
+        # This is really similar to what we are doing in the bundler, it uses a similar algorithm
+        # Example:
+        # core/static/bundled/alpine-index.js -> bundled/alpine-index.js
+        # core/static/bundled/components/include-index.ts -> core/static/bundled/components/include-index.ts
+        def get_relative_src_name(name: str) -> str:
+            original_path = Path(name)
+            relative_path: list[str] = []
+            for directory in reversed(original_path.parts):
+                relative_path.append(directory)
+                # Contrary to the bundler algorithm, we do want to keep the bundled prefix
+                if directory == BUNDLED_FOLDER_NAME:
+                    break
+            return str(Path(*reversed(relative_path)))
+
+        # For the out part:
+        # The bundler is configured to output files in generated/bundled and considers this folders as it's root
+        # Thus, the output name doesn't contain the `bundled` prefix that we need, we add it ourselves
         ret = [
             cls(
-                out=str(Path("bundled") / entry["file"]),
-                src=str(Path(*Path(entry["src"]).parts[2:])),
+                src=get_relative_src_name(entry["src"]),
+                out=str(Path(BUNDLED_FOLDER_NAME) / entry["file"]),
             )
         ]
+
+        def remove_hash(path: Path) -> str:
+            # Hashes are configured to be surrounded by `.`
+            # Filenames are like this path/to/file.hash.ext
+            unhashed = ".".join(path.stem.split(".")[:-1])
+            return str(path.with_stem(unhashed))
+
+        # CSS files generated by entrypoints don't have their own entry in the manifest
+        # They are however listed as an attribute of the entry point that generates them
+        # Their listed name is the one that has been generated inside the generated/bundled folder
+        # We prefix it with `bundled` and then generate an `src` name by removing the hash
         for css in entry.get("css", []):
-            path = Path("bundled") / css
+            path = Path(BUNDLED_FOLDER_NAME) / css
             ret.append(
                 cls(
+                    src=remove_hash(path),
                     out=str(path),
-                    src=str(path.with_stem(entry["name"])),
                 )
             )
         return ret
@@ -77,7 +113,7 @@ class JSBundler:
     def is_in_bundle(name: str | None) -> bool:
         if name is None:
             return False
-        return name.startswith("bundled/")
+        return Path(name).parts[0] == BUNDLED_FOLDER_NAME
 
 
 class Scss:
@@ -120,7 +156,7 @@ class JS:
             p
             for p in settings.STATIC_ROOT.rglob("*.js")
             if ".min" not in p.suffixes
-            and (settings.STATIC_ROOT / "bundled") not in p.parents
+            and (settings.STATIC_ROOT / BUNDLED_FOLDER_NAME) not in p.parents
         ]
         for path in to_exec:
             p = path.resolve()

staticfiles/storage.py

@@ -16,9 +16,10 @@ class ManifestPostProcessingStorage(ManifestStaticFilesStorage):
         # This name swap has to be done here
         # Otherwise, the manifest isn't aware of the file and can't work properly
         if settings.DEBUG:
+            # In production, the bundler manifest is used at compile time, we don't need to convert anything
             try:
                 manifest = JSBundler.get_manifest()
-            except Exception as e:
+            except FileNotFoundError as e:
                 raise Exception(
                     "Error loading manifest file, the bundler seems to be busy"
                 ) from e
@@ -27,7 +28,6 @@ class ManifestPostProcessingStorage(ManifestStaticFilesStorage):
                 name = converted
 
         path = Path(name)
-        # Call bundler manifest
         if path.suffix == ".scss":
             # Compile scss files automatically in debug mode
             if settings.DEBUG:

vite.config.mts

@@ -2,7 +2,7 @@
 import { parse, resolve } from "node:path";
 import inject from "@rollup/plugin-inject";
 import { glob } from "glob";
-import type { AliasOptions, UserConfig } from "vite";
+import { type AliasOptions, type UserConfig, defineConfig } from "vite";
 import type { Rollup } from "vite";
 import { viteStaticCopy } from "vite-plugin-static-copy";
 import tsconfig from "./tsconfig.json";
@@ -44,65 +44,67 @@ function getRelativeAssetPath(path: string): string {
 }
 
 // biome-ignore lint/style/noDefaultExport: this is recommended by documentation
-export default {
-  base: "/static/bundled/",
-  appType: "custom",
-  build: {
-    outDir: outDir,
-    manifest: true, // goes into .vite/manifest.json in the build folder
-    modulePreload: false, // would require `import 'vite/modulepreload-polyfill'` to always be injected
-    emptyOutDir: true,
-    rollupOptions: {
-      input: collectedFiles,
-      output: {
-        // Mirror architecture of static folders in generated .js and .css
-        entryFileNames: (chunkInfo: Rollup.PreRenderedChunk) => {
-          if (chunkInfo.facadeModuleId !== null) {
-            return `${getRelativeAssetPath(chunkInfo.facadeModuleId)}.[hash].js`;
-          }
-          return "[name].[hash].js";
+export default defineConfig((config: UserConfig) => {
+  return {
+    base: "/static/bundled/",
+    appType: "custom",
+    build: {
+      outDir: outDir,
+      manifest: true, // goes into .vite/manifest.json in the build folder
+      modulePreload: false, // would require `import 'vite/modulepreload-polyfill'` to always be injected
+      emptyOutDir: config.mode === "production", // Avoid rebuilding everything in dev mode
+      rollupOptions: {
+        input: collectedFiles,
+        output: {
+          // Mirror architecture of static folders in generated .js and .css
+          entryFileNames: (chunkInfo: Rollup.PreRenderedChunk) => {
+            if (chunkInfo.facadeModuleId !== null) {
+              return `${getRelativeAssetPath(chunkInfo.facadeModuleId)}.[hash].js`;
+            }
+            return "[name].[hash].js";
+          },
+          assetFileNames: (chunkInfo: Rollup.PreRenderedAsset) => {
+            if (
+              chunkInfo.names?.length === 1 &&
+              chunkInfo.originalFileNames?.length === 1 &&
+              collectedFiles.includes(chunkInfo.originalFileNames[0])
+            ) {
+              return `${getRelativeAssetPath(chunkInfo.originalFileNames[0])}.[hash][extname]`;
+            }
+            return "[name].[hash][extname]";
+          },
+          chunkFileNames: "[name].[hash].js",
         },
-        assetFileNames: (chunkInfo: Rollup.PreRenderedAsset) => {
-          if (
-            chunkInfo.names?.length === 1 &&
-            chunkInfo.originalFileNames?.length === 1 &&
-            collectedFiles.includes(chunkInfo.originalFileNames[0])
-          ) {
-            return `${getRelativeAssetPath(chunkInfo.originalFileNames[0])}.[hash][extname]`;
-          }
-          return "[name].[hash][extname]";
-        },
-        chunkFileNames: "[name].[hash].js",
       },
     },
-  },
-  resolve: {
-    alias: getAliases(),
-  },
+    resolve: {
+      alias: getAliases(),
+    },
 
-  plugins: [
-    inject({
-      // biome-ignore lint/style/useNamingConvention: that's how it's called
-      Alpine: "alpinejs",
-    }),
-    viteStaticCopy({
-      targets: [
-        {
-          src: resolve(nodeModules, "jquery/dist/jquery.min.js"),
-          dest: vendored,
-        },
-        {
-          src: resolve(nodeModules, "jquery-ui/dist/jquery-ui.min.js"),
-          dest: vendored,
-        },
-        {
-          src: resolve(nodeModules, "jquery.shorten/src/jquery.shorten.min.js"),
-          dest: vendored,
-        },
-      ],
-    }),
-  ],
-  optimizeDeps: {
-    include: ["jquery"],
-  },
-} satisfies UserConfig;
+    plugins: [
+      inject({
+        // biome-ignore lint/style/useNamingConvention: that's how it's called
+        Alpine: "alpinejs",
+      }),
+      viteStaticCopy({
+        targets: [
+          {
+            src: resolve(nodeModules, "jquery/dist/jquery.min.js"),
+            dest: vendored,
+          },
+          {
+            src: resolve(nodeModules, "jquery-ui/dist/jquery-ui.min.js"),
+            dest: vendored,
+          },
+          {
+            src: resolve(nodeModules, "jquery.shorten/src/jquery.shorten.min.js"),
+            dest: vendored,
+          },
+        ],
+      }),
+    ],
+    optimizeDeps: {
+      include: ["jquery"],
+    },
+  } satisfies UserConfig;
+});