Changeset 88838bc in Klonkt

Timestamp:

08/20/2026 02:37:12 AM (3 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

520c993

Parents:

a505f31

Message:

Opruimen rond Lenis: een kopie, de juiste map, en de licenties genoteerd

Robins vraag of er iets met de MIT-licentie moest, en het antwoord bleek
groter dan de vraag.

LENIS ZAT ER AL. src/assets/js/lenis.min.js (1.1.13) plus een
smooth-scroll.js die er een instantie mee maakte. Ik heb daar gisteren een
tweede kopie naast gezet (1.3.26, ESM) zonder te kijken -- twee versies van
dezelfde bibliotheek in twee mappen. Had ik moeten opzoeken voor ik ging
vendoren.

Het gevaar dat ik vreesde was er niet: smooth-scroll.js wordt door geen
enkel sjabloon ingeladen, dus er draaiden geen twee instanties. Het was dode
code. Wel leerzaam: die oude opzet stond op duration 1.0 met easeOutExpo,
precies de trage instelling die Robin gisteren als "te zweverig" herkende.

Opgeruimd:

• lenis.min.js (1.1.13) en smooth-scroll.js weg -- ongebruikt.
• Mijn ESM-kopie verhuisd van src/assets/js/vendor/ naar src/assets/vendor/, waar cropper en simplewebauthn al stonden. Een tweede vendor-map naast de bestaande was mijn eigen verzinsel.
• data-lenis-prevent in topnav.ejs blijft staan: dat werkt weer, want er draait nu daadwerkelijk een Lenis. Het hoorde bij de oude poging en is nu toevallig weer zinvol -- vandaar deze regel, zodat de volgende lezer niet denkt dat het een restant is.

LICENTIES. MIT eist dat de copyrightvermelding meereist met de code, en die
staat in src/assets/vendor/lenis-LICENSE.txt. Maar de repo heeft een
THIRD-PARTY-NOTICES.md waar de README naar verwijst, en daar ontbrak een
sectie voor gevendorde browser-bibliotheken -- niet alleen Lenis, ook
cropper, simplewebauthn en de losse htmx-kopie stonden er niet in. Die
sectie is er nu, met versie en licentie per bestand.

Alles nagekeken in de bestanden zelf en via npm view, niet uit het hoofd:
Lenis 1.3.26 MIT, Cropper.js 1.6.2 MIT, @simplewebauthn/browser 13.3.0 MIT,
htmx 0BSD. Klonkt is AGPL-3.0-or-later en MIT-code mag daarin.

MOD_V naar 20 (het importpad in read.js is verhuisd).

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

Files:

• THIRD-PARTY-NOTICES.md (modified) (1 diff)
• src/assets/js/lenis.min.js (deleted)
• src/assets/js/mod/read.js (modified) (1 diff)
• src/assets/js/smooth-scroll.js (deleted)
• src/assets/vendor/lenis-LICENSE.txt (moved) (moved from src/assets/js/vendor/lenis-LICENSE.txt )
• src/assets/vendor/lenis-snap.mjs (moved) (moved from src/assets/js/vendor/lenis-snap.mjs )
• src/assets/vendor/lenis.mjs (moved) (moved from src/assets/js/vendor/lenis.mjs )
• src/views/shell.ejs (modified) (1 diff)

THIRD-PARTY-NOTICES.md

@@ -17 +17 @@
 | node-webpmux | LGPL-3.0-or-later |
 | ffmpeg-static | GPL-3.0-or-later |
+
+## Vendored browser libraries
+
+Checked into `src/assets/` rather than pulled from a CDN: the Content-Security-Policy allows
+scripts from `'self'` only, so anything the browser runs has to ship with Klonkt. Each file keeps
+the licence header its author put there; Lenis ships its licence as a separate file because its
+build has no header.
+
+| Library | Version | Licence | Where |
+|---|---|---|---|
+| [Lenis](https://github.com/darkroomengineering/lenis) + `lenis/snap` | 1.3.26 | MIT (© darkroom.engineering) — text in `src/assets/vendor/lenis-LICENSE.txt` | `src/assets/vendor/lenis*.mjs` |
+| [Cropper.js](https://github.com/fengyuanchen/cropperjs) | 1.6.2 | MIT (© Chen Fengyuan) — header in the file | `src/assets/vendor/cropper.min.*` |
+| [@simplewebauthn/browser](https://github.com/MasterKale/SimpleWebAuthn) | 13.3.0 | MIT | `src/assets/vendor/simplewebauthn-browser.umd.min.js` |
+| [htmx](https://htmx.org) | — | 0BSD (also listed under npm above) | `src/assets/js/htmx.min.js` |
 
 ## Bundled binaries & native libraries

src/assets/js/mod/read.js

@@ -151 +151 @@
   if (lenis || !OP_DESKTOP.matches) return;
   const [L, S] = await Promise.all([
-    import(`/assets/js/vendor/lenis.mjs?v=${VENDOR_V}`),
-    import(`/assets/js/vendor/lenis-snap.mjs?v=${VENDOR_V}`),
+    import(`/assets/vendor/lenis.mjs?v=${VENDOR_V}`),
+    import(`/assets/vendor/lenis-snap.mjs?v=${VENDOR_V}`),
   ]);
   lenis = new L.default({

src/views/shell.ejs

@@ -505 +505 @@
   // 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 = 19;
+  var MOD_V = 20;
 
   // name -> 1 (aan het laden) of de module-namespace (geladen). Een module