Fix schema.json being auto deleted and remove formating and linting of generated openapi client

made country flags apply to windows in chrome browsers

core/static/bundled/country-flags-index.ts

@@ -0,0 +1,3 @@
+import { polyfillCountryFlagEmojis } from "country-flag-emoji-polyfill";
+
+polyfillCountryFlagEmojis();

core/static/core/header.scss

@@ -106,6 +106,7 @@ $hovered-red-text-color: #ff4d4d;
         color: $text-color;
         font-weight: normal;
         line-height: 1.3em;
+        font-family: "Twemoji Country Flags", sans-serif;
 
         &:hover {
           background-color: $background-color-hovered;

core/templates/core/base.jinja

@@ -23,6 +23,7 @@
       <script type="module" src={{ static("bundled/core/components/include-index.ts") }}></script>
       <script type="module" src="{{ static('bundled/alpine-index.js') }}"></script>
       <script type="module" src="{{ static('bundled/htmx-index.js') }}"></script>
+      <script type="module" src="{{ static('bundled/country-flags-index.ts') }}"></script>
 
       <!-- Jquery declared here to be accessible in every django widgets -->
       <script src="{{ static('bundled/vendored/jquery.min.js') }}"></script>

openapi-ts.config.ts

@@ -6,9 +6,7 @@ import { defineConfig } from "@hey-api/openapi-ts";
 export default defineConfig({
   input: resolve(__dirname, "./staticfiles/generated/openapi/schema.json"),
   output: {
-    lint: "biome",
-    format: "biome",
-    path: resolve(__dirname, "./staticfiles/generated/openapi"),
+    path: resolve(__dirname, "./staticfiles/generated/openapi/client"),
   },
   plugins: [
     {

package-lock.json

@@ -22,6 +22,7 @@
         "3d-force-graph": "^1.73.4",
         "alpinejs": "^3.14.7",
         "chart.js": "^4.4.4",
+        "country-flag-emoji-polyfill": "^0.1.8",
         "cytoscape": "^3.30.2",
         "cytoscape-cxtmenu": "^3.5.0",
         "cytoscape-klay": "^3.1.4",
@@ -3379,6 +3380,12 @@
         "url": "https://opencollective.com/core-js"
       }
     },
+    "node_modules/country-flag-emoji-polyfill": {
+      "version": "0.1.8",
+      "resolved": "https://registry.npmjs.org/country-flag-emoji-polyfill/-/country-flag-emoji-polyfill-0.1.8.tgz",
+      "integrity": "sha512-Mbah52sADS3gshUYhK5142gtUuJpHYOXlXtLFI3Ly4RqgkmPMvhX9kMZSTqDM8P7UqtSW99eHKFphhQSGXA3Cg==",
+      "license": "MIT"
+    },
     "node_modules/cross-spawn": {
       "version": "7.0.6",
       "resolved": "https://registry.npmjs.org/cross-spawn/-/cross-spawn-7.0.6.tgz",

package.json

@@ -7,6 +7,7 @@
     "compile": "vite build --mode production",
     "compile-dev": "vite build --mode development",
     "serve": "vite build --mode development --watch --minify false",
+    "openapi": "openapi-ts",
     "analyse-dev": "vite-bundle-visualizer --mode development",
     "analyse-prod": "vite-bundle-visualizer --mode production",
     "check": "biome check --write"
@@ -16,7 +17,7 @@
   "license": "GPL-3.0-only",
   "sideEffects": [".css"],
   "imports": {
-    "#openapi": "./staticfiles/generated/openapi/index.ts",
+    "#openapi": "./staticfiles/generated/openapi/client/index.ts",
     "#core:*": "./core/static/bundled/*",
     "#pedagogy:*": "./pedagogy/static/bundled/*",
     "#counter:*": "./counter/static/bundled/*",
@@ -48,6 +49,7 @@
     "3d-force-graph": "^1.73.4",
     "alpinejs": "^3.14.7",
     "chart.js": "^4.4.4",
+    "country-flag-emoji-polyfill": "^0.1.8",
     "cytoscape": "^3.30.2",
     "cytoscape-cxtmenu": "^3.5.0",
     "cytoscape-klay": "^3.1.4",

staticfiles/management/commands/collectstatic.py

@@ -50,7 +50,13 @@ class Command(CollectStatic):
             return Path(location)
 
         Scss.compile(self.collect_scss())
-        OpenApi.compile()  # This needs to be prior to javascript bundling
+        openapi = OpenApi.compile()  # This needs to be prior to javascript bundling
+        if openapi is not None:
+            _ = openapi.wait()
+            if openapi.returncode:
+                raise RuntimeError(
+                    f"Openapi generation failed with returncode {openapi.returncode}"
+                )
         JSBundler.compile()
 
         collected = super().collect()

staticfiles/management/commands/runserver.py

@@ -15,11 +15,18 @@ class Command(Runserver):
     """Light wrapper around default runserver that integrates javascirpt auto bundling."""
 
     def run(self, **options):
-        OpenApi.compile()
-        if (
-            os.environ.get(DJANGO_AUTORELOAD_ENV) is None
-            and settings.PROCFILE_STATIC is not None
-        ):
+        is_django_reload = os.environ.get(DJANGO_AUTORELOAD_ENV) is not None
+
+        proc = OpenApi.compile()
+        # Ensure that the first runserver launch creates openapi files
+        # before the bundler starts so that it detects them
+        # When django is reloaded, we can keep this process in background
+        # to reduce reload time
+        if proc is not None and not is_django_reload:
+            _ = proc.wait()
+
+        if not is_django_reload and settings.PROCFILE_STATIC is not None:
             start_composer(settings.PROCFILE_STATIC)
             _ = atexit.register(stop_composer, procfile=settings.PROCFILE_STATIC)
+
         super().run(**options)

staticfiles/processors.py

@@ -95,7 +95,7 @@ class JSBundler:
     def compile():
         """Bundle js files with the javascript bundler for production."""
         process = subprocess.Popen(["npm", "run", "compile"])
-        process.wait()
+        _ = process.wait()
         if process.returncode:
             raise RuntimeError(f"Bundler failed with returncode {process.returncode}")
 
@@ -163,7 +163,7 @@ class OpenApi:
     OPENAPI_DIR = GENERATED_ROOT / "openapi"
 
     @classmethod
-    def compile(cls):
+    def compile(cls) -> subprocess.Popen[bytes] | None:
         """Compile a TS client for the sith API. Only generates it if it changed."""
         logging.getLogger("django").info("Compiling open api typescript client")
         out = cls.OPENAPI_DIR / "schema.json"
@@ -191,4 +191,4 @@ class OpenApi:
         with open(out, "w") as f:
             _ = f.write(schema)
 
-        subprocess.run(["npx", "openapi-ts"], check=True)
+        return subprocess.Popen(["npm", "run", "openapi"])

tsconfig.json

@@ -14,7 +14,7 @@
     "types": ["jquery", "alpinejs"],
     "lib": ["es7"],
     "paths": {
-      "#openapi": ["./staticfiles/generated/openapi/index.ts"],
+      "#openapi": ["./staticfiles/generated/openapi/client/index.ts"],
       "#core:*": ["./core/static/bundled/*"],
       "#pedagogy:*": ["./pedagogy/static/bundled/*"],
       "#counter:*": ["./counter/static/bundled/*"],