Changeset abcf51a in Klonkt

Timestamp:

08/08/2026 04:55:00 PM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

4838760

Parents:

0261730

git-author:

Robin <roboburr@…> (08/08/2026 04:54:56 PM)

git-committer:

Robin <roboburr@…> (08/08/2026 04:55:00 PM)

Message:

Een track wijst naar de post die hem uitbrengt

Robins waarneming: Shaer toont sinds fb22f78 losse, niet-werkende
track-kaarten. Dat is mijn regressie -- ik zette Create(Audio) in de outbox
voor Funkwhale, en Shaer bouwt zijn HomeBase-feed uit precies die outbox.
noteFrom() daar kijkt niet naar type, dus een Audio wordt een C2SNote met lege
content, en onze url is een Link-array die as? String niet vangt.

Robins voorstel was beter dan wegfilteren: koppel ze aan de hostende post en
filter DAAROP. Die relatie bestond al, maar alleen in posts.content --
[[track:id]], [[playlist:id]], [[album:naam]] -- en nergens op de draad.

Nu wel, met twee velden die allebei al bestonden:

context AS2-KERN, "de context waarbinnen dit object bestaat". Wijst naar

de Note van de post. Funkwhale kijkt er niet naar, dus puur
additief.

url[0] de text/html-link naar de postpagina, vooraan zoals Funkwhale zijn

trackpagina zet. Dit veld lieten we bewust leeg omdat Klonkt geen
trackpagina heeft -- maar de post IS waar je het kunt horen.

Daarmee wordt de filterregel in Shaer principieel in plaats van bot: verberg
een Audio waarvan de context al in je feed staat. Een track die in GEEN post
zit houdt geen context en blijft zichtbaar -- dat is de enige plek waar je hem
zou zien.

Rangorde als een track op meerdere manieren is ingesloten: rechtstreeks boven
playlist boven albumnaam, en bij gelijke rang de nieuwste post. Specifieker
gaat dus voor recenter; daar staat een test op. Een concept telt niet als host.

EEN ZOEKOPDRACHT VOOR ALLES, niet per track: drie LIKE-scans maal
tweehonderd nummers is zeshonderd scans, en de collectie loopt over elke open
track. trackHostPosts levert een map, en die gaat als optie mee door de
collectie, de playlist en de outbox.

8 tests. context erbij in de AS2-allowlist -- het is een kernterm.

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

Files:

• src/services/ActivityPubService.js (modified) (8 diffs)
• test/activitypub-as2.test.js (modified) (1 diff)
• test/ap-track-context.test.js (added)

src/services/ActivityPubService.js

@@ -967 +967 @@
  * ophaalt niet denkt dat er iets nieuws is.
  */
-export function buildTrackCreate(base, site, r) {
-  const audio = buildTrackAudio(base, site, r);
+export function buildTrackCreate(base, site, r, opts = {}) {
+  const audio = buildTrackAudio(base, site, r, opts);
   const me = actorId(base, site.slug);
   return {
@@ -1006 +1006 @@
   const items = [
     ...(posts || []).map((p) => buildCreate(base, site, p)),
-    ...(tracks || []).map((r) => buildTrackCreate(base, site, r)),
+    // Eén zoekopdracht voor alle tracks samen, niet per stuk.
+    ...(() => {
+      const posts = (tracks || []).length && site.id ? trackHostPosts(site.id) : null;
+      return (tracks || []).map((r) => buildTrackCreate(base, site, r, { hostPosts: posts }));
+    })(),
   ]
     .sort((a, b) => wanneer(b) - wanneer(a))
@@ -1130 +1134 @@
  * belooft.
  */
+/**
+ * Bij welke post hoort een track? (shaer-0nh)
+ *
+ * Een track staat nooit los in Klonkt: hij wordt getoond BINNEN een post, via
+ * een van drie insluitingen in posts.content. Die relatie stond alleen in die
+ * tekst en nergens op de draad -- waardoor Shaer, dat zijn feed uit de outbox
+ * bouwt, sinds fb22f78 losse Audio-kaarten kreeg zonder inhoud.
+ *
+ * ALLES IN EEN ZOEKOPDRACHT, niet per track. De collectie loopt over elke open
+ * track, en drie LIKE-scans per stuk wordt bij tweehonderd nummers zeshonderd
+ * scans. Nu is het er een, en de map gaat mee als optie.
+ *
+ * De rang bepaalt welke post wint als er meerdere zijn: rechtstreeks ingesloten
+ * is specifieker dan via een playlist, en die weer specifieker dan via een
+ * albumnaam. Bij gelijke rang de nieuwste post -- dat is waar iemand hem het
+ * laatst heeft uitgebracht.
+ */
+export function trackHostPosts(siteId) {
+  const rijen = db.prepare(`
+    SELECT tid, post_id, post_slug, rang, wanneer FROM (
+      SELECT t.id AS tid, p.id AS post_id, p.slug AS post_slug, 1 AS rang,
+             COALESCE(p.published_at, p.created_at) AS wanneer
+        FROM audio_tracks t
+        JOIN posts p ON p.site_id = t.site_id AND p.status = 'published'
+                    AND p.content LIKE '%[[track:' || t.id || ']]%'
+       WHERE t.site_id = ? AND t.fedi_open = 1
+      UNION ALL
+      SELECT t.id, p.id, p.slug, 2, COALESCE(p.published_at, p.created_at)
+        FROM playlist_tracks pt
+        JOIN audio_tracks t ON t.id = pt.track_id
+        JOIN posts p ON p.site_id = t.site_id AND p.status = 'published'
+                    AND p.content LIKE '%[[playlist:' || pt.playlist_id || ']]%'
+       WHERE t.site_id = ? AND t.fedi_open = 1
+      UNION ALL
+      SELECT t.id, p.id, p.slug, 3, COALESCE(p.published_at, p.created_at)
+        FROM audio_tracks t
+        JOIN posts p ON p.site_id = t.site_id AND p.status = 'published'
+                    AND p.content LIKE '%[[album:' || t.album || ']]%'
+       WHERE t.site_id = ? AND t.fedi_open = 1 AND t.album IS NOT NULL AND t.album <> ''
+    ) ORDER BY rang, wanneer DESC
+  `).all(siteId, siteId, siteId);
+  const uit = new Map();
+  for (const r of rijen) if (!uit.has(r.tid)) uit.set(r.tid, { id: r.post_id, slug: r.post_slug });
+  return uit;
+}
+
 export function buildTrackAudio(base, site, r, opts = {}) {
   const abs = (u) => !u ? null : (/^https?:/i.test(u) ? u : `${base}${u.startsWith('/') ? '' : '/'}${u}`);
@@ -1136 +1186 @@
   // representatie die zoveel bytes is en die bitrate heeft, niet het nummer.
   // Zo doet Funkwhale het ook.
+  // De post waar dit nummer in staat. Meegegeven door de collectie (een
+  // zoekopdracht voor alles), of hier opgezocht als deze track los wordt
+  // opgehaald. `hostPosts` mag expliciet null zijn: dan is er niets te zoeken.
+  const post = opts.hostPosts !== undefined
+    ? (opts.hostPosts && opts.hostPosts.get(r.id)) || null
+    : ((site.id && trackHostPosts(site.id).get(r.id)) || null);
+
   const bestand = { type: 'Link', href: `${base}/audio/stream/${encodeURIComponent(fn)}`, mediaType: r.mime_type || 'audio/mpeg' };
   if (Number(r.size)) bestand.size = Number(r.size);
@@ -1152 +1209 @@
     // track moet zelf kunnen zeggen dat hij openbaar is.
     to: [PUBLIC],
-    url: [bestand],
+    // De post die dit nummer uitbrengt staat VOORAAN als text/html, precies
+    // zoals Funkwhale zijn trackpagina zet. Wij hadden dat veld leeg gelaten
+    // omdat Klonkt geen trackpagina heeft -- maar de post IS waar je het kunt
+    // horen, en dat is wat zo'n link betekent.
+    url: [...(post ? [{ type: 'Link', href: `${base}/${post.slug}`, mediaType: 'text/html' }] : []), bestand],
   };
   if (r.artist) a.summary = r.artist;              // artiest als summary: kaal AS2, geen eigen vocab
+  // AS2-kern `context`: "de context waarbinnen dit object bestaat". Voor een
+  // track is dat de post die hem uitbrengt. Daarmee is de relatie die tot nu
+  // toe alleen in posts.content stond, op de draad te zien -- en kan een lezer
+  // die de post al heeft dit nummer overslaan in plaats van er een lege kaart
+  // van te maken.
+  if (post) a.context = noteId(base, post.id);
   if (r.duration) a.duration = `PT${Math.round(r.duration)}S`;
   if (r.created_at) a.published = new Date(r.created_at).toISOString();
@@ -1201 +1268 @@
     attributedTo: actorId(base, site.slug),
     totalItems: (rows || []).length,
-    orderedItems: (rows || []).map((r) => buildTrackAudio(base, site, r)),
+    // Eén zoekopdracht voor alle rijen samen; zie trackHostPosts.
+    orderedItems: (() => {
+      const posts = site.id ? trackHostPosts(site.id) : null;
+      return (rows || []).map((r) => buildTrackAudio(base, site, r, { hostPosts: posts }));
+    })(),
   };
 }
@@ -1263 +1334 @@
   // twee keer hetzelfde ding en niet twee dingen die toevallig gelijk klinken.
   // De hoes van de playlist dient als terugval voor een track zonder eigen hoes.
-  const items = (rows || []).map((r) => buildTrackAudio(base, site, r, { coverFallback: playlist.cover_url || null }));
+  const hostPosts = site.id ? trackHostPosts(site.id) : null;
+  const items = (rows || []).map((r) => buildTrackAudio(base, site, r, { coverFallback: playlist.cover_url || null, hostPosts }));
   const out = {
     '@context': AP_CONTEXT,
@@ -6189 +6261 @@
   buildActor, buildNote, buildCreate, buildOutbox, buildFollowers, buildFollowing, buildFeatured,
   channelUrls, channelCategory, timelineFields, guessMediaType,
-  siteOpenTracks, openTrack, buildTrackAudio, buildTrackCollection, buildTrackCreate,
+  siteOpenTracks, openTrack, buildTrackAudio, buildTrackCollection, buildTrackCreate, trackHostPosts,
   buildPlaylistCollection, playlistOpenTracks, listPlaylistsAP, playlistLinkTags,
   followerCount, deliver, fetchActor, verifyRequest, handleInbox, deliverCreate, deliverDelete, deliverUpdate, deliverActorUpdate, resyncFeaturedPins,

test/activitypub-as2.test.js

@@ -26 +26 @@
   'content', 'name', 'summary', 'url', 'href', 'mediaType',
   'published', 'updated', 'attributedTo', 'inReplyTo', 'replies',
+  // AS2-kern: de context waarbinnen een object bestaat. Een track wijst ermee
+  // naar de post die hem uitbrengt (shaer-0nh).
+  'context',
   'attachment', 'tag', 'icon', 'image', 'duration',
   'contentMap', 'nameMap', 'summaryMap', // AS2 @language-map counterparts of content/name/summary