Changeset 87696ce in Klonkt

Timestamp:

08/18/2026 06:47:58 PM (3 weeks ago)

Author:

Bart <bart@…>

Branches:

main

Children:

847775c

Parents:

85b51bf

git-author:

Bart <bart@…> (08/18/2026 04:37:05 AM)

git-committer:

Bart <bart@…> (08/18/2026 06:47:58 PM)

Message:

De mod/-modules krijgen een cache-buster: MOD_V

/assets staat buiten development op max-age=1y, en de dynamische import van
assets/js/mod/*.js had als enige geen ?v= -- style.css, audio.css en
audio-player.js hebben er al jaren een. Gevolg: wie de pagina eerder bezocht,
draaide een jaar lang de oude module. Een reparatie bereikt dan precies de
bezoekers die de fout al hebben.

Gemerkt bij read.js: de gerepareerde module stond goed op de server (curl
bevestigde het) maar de browser bleef de oude draaien, ook na herladen en na het
wissen van de service-worker-caches. En de hernoemde klasse maakte het zichtbaar:
oude JS zet read-chrome-weg, nieuwe CSS luistert naar read-chrome-hidden, dus de
balken schoven bij die bezoekers helemaal niet meer weg.

Eén nummer voor de hele map, met de hand te bumpen; de regel staat nu ook in
CLAUDE.md en AGENTS.md, want een discipline die nergens staat is er geen.

Co-Authored-By: Claude Opus 5 <claude@…>

Files:

• AGENTS.md (modified) (1 diff)
• CLAUDE.md (modified) (1 diff)
• src/views/shell.ejs (modified) (2 diffs)

AGENTS.md

@@ -36 +36 @@
 - `apt-get` - use `-y` flag
 - `brew` - use `HOMEBREW_NO_AUTO_UPDATE=1` env var
+
+## Conventions & Patterns
+
+- **Bump `MOD_V` whenever you change anything in `src/assets/js/mod/`.** It sits
+  at the top of the module loader in `src/views/shell.ejs` and is the
+  cache-buster for every page module. `/assets` is served `max-age=1y` outside
+  development, so without a bump a browser that visited before keeps running the
+  old module for a year — meaning a fix reaches everyone *except* the people who
+  already have the bug. Same discipline as `audio-player.js?v=N` a few hundred
+  lines up. One number for the whole directory: bumping too often costs one
+  download, bumping too rarely costs a bugfix that never arrives.
+
+- **Comments and commit messages in Dutch, identifiers in English.** The modules
+  in `assets/js/mod` read `setIcon`, `uploadOne`, `applyAccent`; the comments
+  around them are Dutch prose. Both halves matter — a Dutch identifier in an
+  English file is the same wrong note as an English comment in a Dutch one. This
+  extends to anything long-lived and outward-facing: URL paths and CSS class
+  names are English (`/read`, `.read-end`), never a Dutch verb form.
 
 <!-- BEGIN BEADS INTEGRATION v:1 profile:minimal hash:7510c1e2 -->

CLAUDE.md

@@ -72 +72 @@
 ## Conventions & Patterns
 
-_Add your project-specific conventions here_
+- **Bump `MOD_V` whenever you change anything in `src/assets/js/mod/`.** It sits
+  at the top of the module loader in `src/views/shell.ejs` and is the
+  cache-buster for every page module. `/assets` is served `max-age=1y` outside
+  development, so without a bump a browser that visited before keeps running the
+  old module for a year — meaning a fix reaches everyone *except* the people who
+  already have the bug. Same discipline as `audio-player.js?v=N` a few hundred
+  lines up. One number for the whole directory: bumping too often costs one
+  download, bumping too rarely costs a bugfix that never arrives.
+
+- **Comments and commit messages in Dutch, identifiers in English.** The modules
+  in `assets/js/mod` read `setIcon`, `uploadOne`, `applyAccent`; the comments
+  around them are Dutch prose. Both halves matter — a Dutch identifier in an
+  English file is the same wrong note as an English comment in a Dutch one. This
+  extends to anything long-lived and outward-facing: URL paths and CSS class
+  names are English (`/read`, `.read-end`), never a Dutch verb form.

src/views/shell.ejs

@@ -496 +496 @@
   window.__modBoot = true;
 
+  // ?v=N — cache-buster voor ALLE modules in assets/js/mod, net als bij
+  // audio-player.js hierboven. BUMP BIJ ELKE WIJZIGING IN DIE MAP. Zonder
+  // query staat /assets op max-age=1y, en dan blijft een browser die de
+  // pagina eerder bezocht een jaar lang de oude module draaien: een
+  // reparatie bereikt precies de bezoekers die hem al hebben.
+  // Eén nummer voor de hele map. Te vaak bumpen kost één download; te weinig
+  // bumpen kost een bugfix die nooit aankomt.
+  var MOD_V = 1;
+
   // name -> 1 (aan het laden) of de module-namespace (geladen). Een module
   // die `init` exporteert draait die bij ELKE paginawissel waarop hij actief
@@ -514 +523 @@
       if (loaded[name]) { start(name, loaded[name]); return; }
       loaded[name] = 1;
-      import('/assets/js/mod/' + name + '.js').then(function (m) {
+      import('/assets/js/mod/' + name + '.js?v=' + MOD_V).then(function (m) {
         loaded[name] = m;
         start(name, m);