Changeset a5d14c7 in Klonkt

Timestamp:

08/09/2026 08:20:24 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

0677bac

Parents:

b2adc20

Message:

De post is de uitgave: losse tracks worden een collectie (shaer-38y)

Losse tracks gingen tot nu toe los de deur uit -- Audio-objecten die een lezer
nergens kon plaatsen. Ze horen bij elkaar omdat ze in dezelfde post staan, en
dat is wat /ap/users/<slug>/posts/<id>/tracks nu zegt. De post wijst er langs
dezelfde weg naartoe als naar een playlist: een Link-tag in de Note, want zonder
die link bestaat de collectie wel maar vindt niemand hem.

PUNT 1, DE VOLGORDE. De tracks staan in de volgorde van de POST en niet die van
de tabel. Zoals iemand ze heeft neergezet is de volgorde waarin ze bedoeld zijn.

PUNT 3, DE METADATA. Een playlist heeft een titel en soms een hoes; een post
heeft een titel, een tekst, een hoes, tags en een datum. Voor audio-gebaseerde
inhoud is de post de uitgave -- dat is waar iemand hem heeft uitgebracht en waar
het verhaal erbij staat. Zowel de collectie van losse tracks als een
playlist-collectie leent nu van de post die haar uitbrengt.

De posttekst wordt content en niet summary: in AS2 is summary de korte
samenvatting en content het lijf. Artiest en jaar blijven dus in summary staan,
want dat IS een samenvatting en de posttekst is dat niet. De eigen naam van de
playlist gaat niet verloren maar verhuist naar alsoKnownAs.

WIE LEENT ER UIT: alleen een post die er EEN muzikale eenheid van maakt. Staan
er twee collecties in, dan is de post niet meer de drager van een identiteit en
houdt de playlist de zijne. Dezelfde regel als in de afleiding, hier alleen
toegepast.

Een test die eerder vastlegde dat losse tracks GEEN Link-tag kregen is
omgedraaid, met de reden erbij: dat klopte zolang alleen een playlist een
collectie had.

Punt 2 van de bead (type afleiden) stond al in postMusicType.

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

Files:

• src/routes/activitypub.js (modified) (1 diff)
• src/services/ActivityPubService.js (modified) (4 diffs)
• src/services/music/index.js (modified) (3 diffs)
• test/ap-playlist-discovery.test.js (modified) (1 diff)
• test/music-uitgave.test.js (added)

src/routes/activitypub.js

@@ -784 +784 @@
 });
 
+// De losse tracks van een post als EEN uitgave (shaer-38y). Ze gingen tot nu
+// toe los de deur uit -- Audio-objecten die een lezer nergens kon plaatsen. Ze
+// horen bij elkaar omdat ze in dezelfde post staan, en die post leent zijn
+// titel, tekst, hoes en tags uit. 404 als de post geen muzikale eenheid IS:
+// dan is er niets om naar te wijzen, en dat is geen lege collectie maar een
+// collectie die niet bestaat.
+router.get('/ap/users/:slug/posts/:id/tracks', (req, res) => {
+  const site = publicSite(req.params.slug);
+  if (!site) return res.status(404).end();
+  const post = db.prepare(
+    "SELECT id, slug, title, excerpt, content, cover_image_url, tags FROM posts WHERE id = ? AND site_id = ? AND status = 'published'"
+  ).get(req.params.id, site.id);
+  if (!post) return res.status(404).end();
+  const col = AP.buildPostTrackCollection(baseUrl(req), site, post);
+  if (!col) return res.status(404).end();
+  AP.sendAP(res, col);
+});
+
 router.get('/ap/users/:slug/playlists/:id', (req, res) => {
   const site = publicSite(req.params.slug);

src/services/ActivityPubService.js

@@ -38 +38 @@
   buildTrackAudio, buildTrackCollection, buildTrackCreate,
   buildPlaylistCollection, listPlaylistsAP, playlistLinkTags,
+  buildPostTrackCollection, uitgavePost,
   licentieUri, channelCategory,
 } from './music/index.js';
@@ -44 +45 @@
   buildTrackAudio, buildTrackCollection, buildTrackCreate,
   buildPlaylistCollection, listPlaylistsAP, playlistLinkTags, licentieUri,
+  buildPostTrackCollection, uitgavePost,
 };
 
@@ -697 +699 @@
       ...((post.fan_only || post.ap_visibility === 'quiet') ? [] : [`${aId}/followers`]),
       ..._mentionCc])],
-    tag: [...buildHashtagList(base, post.tags, body), ..._mentionTags, ...playlistLinkTags(base, site, post.content)],
+    tag: [...buildHashtagList(base, post.tags, body), ..._mentionTags, ...playlistLinkTags(base, site, post.content, post)],
     replies: `${id}/replies`,
     // NSFW → Mastodon-style content warning: sensitive (blurs media) + a summary/spoiler
@@ -6108 +6110 @@
   siteOpenTracks, openTrack, buildTrackAudio, buildTrackCollection, buildTrackCreate, trackHostPosts,
   buildPlaylistCollection, playlistOpenTracks, listPlaylistsAP, playlistLinkTags,
+  buildPostTrackCollection, uitgavePost,
   followerCount, deliver, fetchActor, verifyRequest, handleInbox, deliverCreate, deliverDelete, deliverUpdate, deliverActorUpdate, resyncFeaturedPins,
   feedCursor, feedChangesSince, waitForFeedChange,

src/services/music/index.js

@@ -15 +15 @@
 import db from '../../config/database.js';
 import { AP_CONTEXT, PUBLIC, actorId, noteId, safeUrl, guessMediaType } from '../ap-core.js';
-import { afleidenUitInsluitingen } from '../../assets/js/shared/post-music-type.js';
+import { afleidenUitInsluitingen, ingeslotenPlaylists } from '../../assets/js/shared/post-music-type.js';
 
 // m.size hoort erbij voor de RSS-enclosure: die eist een lengte in bytes.
@@ -222 +222 @@
 // zonder site-check zou een post van site A naar de collectie van site B
 // kunnen wijzen.
-export function playlistLinkTags(base, site, content) {
+export function playlistLinkTags(base, site, content, post = null) {
   const out = [];
-  const seen = new Set();
   try {
-    for (const m of (content || '').matchAll(/\[\[playlist:([A-Za-z0-9_-]+)\]\]/g)) {
-      if (seen.has(m[1])) continue;
-      seen.add(m[1]);
-      const pl = db.prepare('SELECT id, title FROM playlists WHERE id = ? AND site_id = ?').get(m[1], site.id);
+    // Zelfde patroon als de renderer en als de afleiding: wat niet insluit,
+    // krijgt ook geen link. Dit stond hier met een eigen patroon dat
+    // underscores accepteerde die nergens anders meetellen.
+    for (const id of ingeslotenPlaylists(content)) {
+      const pl = db.prepare('SELECT id, title FROM playlists WHERE id = ? AND site_id = ?').get(id, site.id);
       if (!pl) continue;
       out.push({ type: 'Link', href: `${actorId(base, site.slug)}/playlists/${pl.id}`, mediaType: 'application/activity+json', name: pl.title });
+    }
+    // Losse tracks in een post zijn ook een uitgave (shaer-38y): ze krijgen een
+    // eigen collectie, en de post wijst er langs dezelfde weg naar. Zonder deze
+    // link zou die collectie bestaan maar door niemand te vinden zijn.
+    if (post && post.id) {
+      const eenheid = postMusicType(content, site.id);
+      if (eenheid && !eenheid.collectie && eenheid.tracks?.length && losseTracksVanPost(site.id, eenheid.tracks).length) {
+        out.push({
+          type: 'Link',
+          href: postTracksId(base, site, post.id),
+          mediaType: 'application/activity+json',
+          name: post.title || 'Tracks',
+        });
+      }
     }
   } catch { /* niet-fataal: een tag minder, geen kapotte Note */ }
@@ -291 +305 @@
   const cover = abs(playlist.cover_url || null);
   if (cover) out.icon = { type: 'Image', mediaType: guessMediaType(cover), url: cover };
-  return out;
+  return leenVanPost(base, site, out, uitgavePost(site.id, playlist.id));
+}
+
+// ── De post als uitgave (shaer-38y) ───────────────────────────────────────
+
+/** Het AS2-id van de collectie losse tracks van een post. */
+function postTracksId(base, site, postId) {
+  return `${actorId(base, site.slug)}/posts/${encodeURIComponent(postId)}/tracks`;
+}
+
+/**
+ * Welke post brengt deze playlist uit, en mag die zijn gegevens uitlenen?
+ *
+ * Niet zomaar de eerste post die de playlist noemt: alleen een post die er EEN
+ * muzikale eenheid van maakt leent uit. Staan er twee collecties in, dan is de
+ * post niet meer de drager van een identiteit en houdt de playlist de zijne --
+ * dezelfde regel als in de afleiding, hier alleen toegepast.
+ *
+ * De nieuwste wint als er meerdere zijn: dat is waar hij het laatst is
+ * uitgebracht.
+ */
+export function uitgavePost(siteId, playlistId) {
+  if (!siteId || !playlistId) return null;
+  try {
+    const rijen = db.prepare(`
+      SELECT id, slug, title, excerpt, content, cover_image_url, tags
+      FROM posts
+      WHERE site_id = ? AND status = 'published'
+        AND content LIKE '%[[playlist:' || ? || ']]%'
+      ORDER BY COALESCE(published_at, created_at) DESC
+    `).all(siteId, playlistId);
+    for (const p of rijen) {
+      const r = postMusicType(p.content, siteId);
+      if (r && r.leentMetadata && r.collectie && r.collectie.id === playlistId) return p;
+    }
+  } catch { /* geen lening is geen fout */ }
+  return null;
+}
+
+/**
+ * De post leent zijn gegevens aan de uitgave (shaer-38y, punt 3).
+ *
+ * WAAROM DE POST WINT EN NIET DE PLAYLIST. Een playlist heeft een titel en soms
+ * een hoes; een post heeft een titel, een tekst, een hoes, tags EN een datum.
+ * Voor audio-gebaseerde inhoud is de post de uitgave -- dat is waar iemand hem
+ * heeft uitgebracht en waar het verhaal erbij staat. Een Funkwhale-achtige
+ * lezer vindt een collectie met alleen een naam te mager, en dat is precies wat
+ * hij nu krijgt.
+ *
+ * De naam van de playlist gaat niet verloren: die blijft als `alsoKnownAs`
+ * staan, zodat de eigen naam terug te vinden is als hij afwijkt.
+ */
+function leenVanPost(base, site, obj, post) {
+  if (!post) return obj;
+  const abs = (u) => !u ? null : (/^https?:/i.test(u) ? u : `${base}${u.startsWith('/') ? '' : '/'}${u}`);
+
+  if (post.title) {
+    if (obj.name && obj.name !== post.title) obj.alsoKnownAs = obj.name;
+    obj.name = post.title;
+  }
+  // De tekst als `content`, niet als `summary`: in AS2 is summary de korte
+  // samenvatting en content het lijf. Artiest en jaar blijven dus in summary
+  // staan -- dat is een samenvatting, en de posttekst is dat niet.
+  const tekst = (post.excerpt || '').trim();
+  if (tekst) obj.content = tekst;
+
+  const cover = abs(post.cover_image_url || null);
+  if (cover) {
+    obj.image = { type: 'Image', mediaType: guessMediaType(cover), url: cover };
+    if (!obj.icon) obj.icon = obj.image;      // geen eigen hoes? dan die van de post
+  }
+
+  const tags = hashtagsVanPost(base, post.tags);
+  if (tags.length) obj.tag = tags;
+
+  // Waar je hem kunt horen, en waar hij bij hoort. Zelfde paar als bij een
+  // losse track: url wijst een mens naar de post, context zegt waar dit object
+  // thuishoort.
+  obj.url = `${base}/${post.slug}`;
+  obj.context = noteId(base, post.id);
+  return obj;
+}
+
+/** De tags van een post als AS2 Hashtags. Zelfde vorm als buildHashtagList. */
+function hashtagsVanPost(base, tagsField) {
+  const ruw = Array.isArray(tagsField)
+    ? tagsField
+    : String(tagsField || '').split(',');
+  const uit = [], gezien = new Set();
+  for (const t of ruw) {
+    const label = String(t || '').trim().replace(/^#/, '');
+    if (!label) continue;
+    const slug = label.toLowerCase().replace(/\s+/g, '-');
+    if (gezien.has(slug)) continue;
+    gezien.add(slug);
+    uit.push({ type: 'Hashtag', href: `${base}/tag/${encodeURIComponent(slug)}`, name: '#' + label });
+  }
+  return uit;
+}
+
+/** De open tracks uit een lijst ids, in de volgorde van die lijst. */
+function losseTracksVanPost(siteId, ids) {
+  if (!siteId || !ids?.length) return [];
+  const gaten = ids.map(() => '?').join(',');
+  const rijen = db.prepare(
+    `SELECT ${TRACK_KOLOMMEN}
+     FROM audio_tracks t JOIN media m ON m.id = t.media_id
+     WHERE t.site_id = ? AND t.fedi_open = 1 AND t.id IN (${gaten})`
+  ).all(siteId, ...ids);
+  // De volgorde van de POST, niet die van de tabel (shaer-38y, punt 1): zoals
+  // iemand ze heeft neergezet is de volgorde waarin ze bedoeld zijn.
+  const opId = new Map(rijen.map((r) => [r.id, r]));
+  return ids.map((id) => opId.get(id)).filter(Boolean);
+}
+
+/**
+ * De losse tracks van een post als EEN uitgave (shaer-38y).
+ *
+ * Tot nu toe gingen die los de deur uit: losse Audio-objecten die een lezer
+ * nergens kon plaatsen. Ze horen bij elkaar omdat ze in dezelfde post staan, en
+ * dat is wat deze collectie zegt -- met de gegevens van de post erbij, want die
+ * heeft ze wel en de losse tracks niet.
+ *
+ * Geeft null als er niets te tonen is: geen post, geen losse tracks, of een
+ * post die geen enkele muzikale eenheid IS.
+ */
+export function buildPostTrackCollection(base, site, post) {
+  if (!post || !post.id) return null;
+  const eenheid = postMusicType(post.content, site.id);
+  if (!eenheid || eenheid.collectie || !eenheid.tracks?.length) return null;
+
+  const rows = losseTracksVanPost(site.id, eenheid.tracks);
+  if (!rows.length) return null;
+
+  const hostPosts = new Map(rows.map((r) => [r.id, { id: post.id, slug: post.slug }]));
+  const out = {
+    '@context': AP_CONTEXT,
+    id: postTracksId(base, site, post.id),
+    type: 'OrderedCollection',
+    attributedTo: actorId(base, site.slug),
+    totalItems: rows.length,
+    orderedItems: rows.map((r) => buildTrackAudio(base, site, r, { hostPosts })),
+  };
+  return leenVanPost(base, site, out, post);
 }
 

test/ap-playlist-discovery.test.js

@@ -108 +108 @@
 });
 
-test('een post zonder playlist heeft geen Link-tags', () => {
+test('losse tracks in een post krijgen sinds shaer-38y wel een Link-tag', () => {
+  // Dit testte eerder dat er GEEN Link-tag was. Dat klopte zolang alleen een
+  // playlist een collectie had; nu zijn losse tracks in een post ook een
+  // uitgave, met een eigen collectie. Zonder deze link bestaat die wel maar
+  // vindt niemand hem -- dus de verwachting is omgedraaid, niet het gedrag
+  // stukgegaan.
   const site = db.prepare("SELECT * FROM sites WHERE id = 's1'").get();
   const note = AP.buildNote('https://test.example', site, { id: 'p3', slug: 'p3', title: 'Los', content: 'Gewoon tekst met [[track:t1]]', status: 'published' });
+  const links = (note.tag || []).filter((t) => t.type === 'Link');
+  assert.equal(links.length, 1);
+  assert.equal(links[0].href, 'https://test.example/ap/users/band/posts/p3/tracks');
+});
+
+test('en een post met alleen tekst nog steeds geen enkele Link-tag', () => {
+  const site = db.prepare("SELECT * FROM sites WHERE id = 's1'").get();
+  const note = AP.buildNote('https://test.example', site, { id: 'p4', slug: 'p4', title: 'Niets', content: 'Gewoon tekst', status: 'published' });
+  assert.equal((note.tag || []).filter((t) => t.type === 'Link').length, 0);
+});
+
+test('een post met alleen een DICHTE track ook niet -- geen lege uitgave', () => {
+  const site = db.prepare("SELECT * FROM sites WHERE id = 's1'").get();
+  const note = AP.buildNote('https://test.example', site, { id: 'p5', slug: 'p5', title: 'Dicht', content: 'Alleen [[track:t2]]', status: 'published' });
   assert.equal((note.tag || []).filter((t) => t.type === 'Link').length, 0);
 });