Improve frontend build: drop django-npm, restructure, manifest storage for cache busting

Author: inducer

.ci/run-tests-for-ci.sh

@@ -11,6 +11,24 @@ cp local_settings_example.py local_settings.py
 poetry run python manage.py makemessages --all
 poetry run python manage.py compilemessages
 
+staticfiles=(
+  bundle-base.js
+  bundle-base-with-markup.js
+  bundle-codemirror.js
+  bundle-datatables.js
+  bundle-fullcalendar.js
+  bundle-prosemirror.js
+  bundle-prosemirror.js
+  tex-svg.js
+)
+
+mkdir -p frontend-dist
+for i in "${staticfiles[@]}"; do
+    touch "frontend-dist/$i"
+done
+
+poetry run python manage.py collectstatic
+
 echo "Starts testing"
 export RELATE_LOCAL_TEST_SETTINGS="local_settings_example.py"
 

.github/workflows/ci.yml

@@ -67,7 +67,7 @@ jobs:
       run: npm install
 
     - name: ESLint
-      run: npx eslint relate/static/js/*.js
+      run: npx eslint frontend/js/*.js
 
     - name: Rollup build
       run: npm run build

collectstatic.sh

@@ -0,0 +1,4 @@
+#! /bin/bash
+
+find node_modules/mathjax/es5 -name '*.js' -exec sed -i /sourceMappingURL=/d '{}' \;
+poetry run python manage.py collectstatic

course/templates/course/course-base-with-markup.html

@@ -4,5 +4,5 @@
 {% block bundle_loads %}
   {# contains mathjax config #}
   <script src="{% static 'bundle-base-with-markup.js' %}"></script>
-  <script src="{% static "mathjax/es5/tex-svg.js" %}" id="MathJax-script" async></script>
+  <script src="{% static "tex-svg.js" %}" id="MathJax-script" async></script>
 {% endblock %}

doc/misc.rst

@@ -194,11 +194,14 @@ Additional Setup Steps for Deploying to Production
 
 *   Run::
 
-        python manage.py collectstatic
+        ./collectstatic.sh
 
     to assemble the required collection of static files to be served, as the
     production app server will not serve them (unlike the dev server).
 
+    (Do not run ``python manage.py collectstatic`` directly; it will fail
+    because it cannot resolve some source map URLs in mathjax.)
+
 Configuring uwsgi
 ^^^^^^^^^^^^^^^^^