Changeset e62eee4 in Klonkt

Timestamp:

08/09/2026 08:50:48 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

709dc6f

Parents:

6259305

Message:

Elke chip in het lijf krijgt zijn eigen URL (shaer-38y)

Een lezer kreeg een vetgedrukte opsomming van titels met een enkele "listen
on"-link naar de post als geheel. Vijf namen waar je niets mee kon. Elke track
heeft op de postpagina al een anker -- #track-<id>, of hij nu direct is
ingesloten, in een album zit of in een playlist -- dus elke naam kan naar
precies het nummer wijzen waar hij bij hoort.

De opsomming loopt nu in EEN scan over alle drie de vormen, in de volgorde van
de post. Dat repareert er meteen iets bij: een playlist-insluiting leverde
helemaal geen namen op, want alleen track en album werden gelezen. Een
albumnaam zonder tracks blijft gewone tekst -- geen link naar een anker dat er
niet is.

De listen-link naar de pagina als geheel blijft staan; die nodigt uit naar de
speler, en dat is iets anders dan naar een nummer springen.

DIT LOST ZICHZELF NIET OP VIA DE SELF-HEAL. Die heelt ap_timeline -- de cache
van notes die wij ONTVANGEN -- en niet wat wij versturen. Onze eigen Notes
worden bij elk verzoek vers uit posts.content gebouwd, dus wie hem nu ophaalt
krijgt meteen de nieuwe vorm. Alleen kopieen die elders al gecachet zijn blijven
oud tot er een Update langskomt, en daar reikt geen self-heal bij.

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

Files:

• src/services/ActivityPubService.js (modified) (2 diffs)
• test/ap-note-track-links.test.js (added)

src/services/ActivityPubService.js

@@ -595 +595 @@
   // a click-through to the protected player (discovery without leaking the file).
   const esc = (s) => String(s == null ? '' : s).replace(/[<>&]/g, (c) => ({ '<': '&lt;', '>': '&gt;', '&': '&amp;' }[c]));
+  // Elke titel met zijn track-id erbij, zodat hij hieronder een EIGEN link
+  // krijgt naar #track-<id> op de postpagina (shaer-38y). Zonder id was dit een
+  // vetgedrukte opsomming waar je niets mee kon: vijf namen en een enkele
+  // "listen on"-link naar de post als geheel. Elke track heeft daar al een
+  // anker -- direct ingesloten, in een album of in een playlist -- dus dit
+  // wijst naar precies het nummer waar de naam bij hoort.
   const audioLabels = [];
   try {
-    for (const m of body.matchAll(/\[\[track:([A-Za-z0-9_-]+)\]\]/g)) { const r = db.prepare('SELECT title FROM audio_tracks WHERE id = ?').get(m[1]); if (r && r.title) audioLabels.push(r.title); }
-    for (const m of body.matchAll(/\[\[album:([^\]]+)\]\]/g)) audioLabels.push(m[1].trim());
+    const zien = new Set();
+    const voegToe = (id, titel) => {
+      const t = String(titel || '').trim();
+      if (!t) return;
+      const sleutel = id || ('naam:' + t);
+      if (zien.has(sleutel)) return;
+      zien.add(sleutel);
+      audioLabels.push({ id: id || null, titel: t });
+    };
+    // In de volgorde van de POST: een enkele scan over alle drie de vormen,
+    // zodat de opsomming leest zoals de post is neergezet.
+    for (const m of body.matchAll(/\[\[(track|album|playlist):([^\]]+)\]\]/gi)) {
+      const soort = m[1].toLowerCase(), waarde = m[2].trim();
+      if (soort === 'track') {
+        const r = db.prepare('SELECT id, title FROM audio_tracks WHERE id = ?').get(waarde);
+        if (r) voegToe(r.id, r.title);
+      } else if (soort === 'album') {
+        const rs = db.prepare('SELECT id, title FROM audio_tracks WHERE site_id = ? AND album = ? ORDER BY rowid').all(site.id, waarde);
+        if (rs.length) for (const r of rs) voegToe(r.id, r.title);
+        else voegToe(null, waarde);            // album zonder tracks: dan maar de naam
+      } else {
+        for (const r of db.prepare('SELECT t.id, t.title FROM playlist_tracks pt JOIN audio_tracks t ON t.id = pt.track_id WHERE pt.playlist_id = ? ORDER BY pt.position').all(waarde)) voegToe(r.id, r.title);
+      }
+    }
   } catch { /* non-fatal */ }
   // fedi_open tracks → real AS2 Audio attachments (the actual file URL, served ungated) so
@@ -633 +661 @@
   });
   if (hadAudio) {
-    const lbl = audioLabels.length ? esc(audioLabels.slice(0, 4).join(', ')) : '';
+    // Elke titel als eigen link naar zijn anker; een titel zonder id (een
+    // albumnaam zonder tracks) blijft gewone tekst.
+    const lbl = audioLabels.slice(0, 4)
+      .map((a) => (a.id ? `<a href="${human}#track-${esc(a.id)}">${esc(a.titel)}</a>` : esc(a.titel)))
+      .join(', ');
     if (trackEmbedLinks.length) {
       // Link-only track(s): emit the external link(s). Mastodon cards the first (Spotify → its