Changeset c25eff7 in Klonkt

Timestamp:

08/07/2026 06:08:37 PM (5 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

c1df3e9

Parents:

ed5e7ab

git-author:

Robin <roboburr@…> (08/07/2026 06:08:25 PM)

git-committer:

Robin <roboburr@…> (08/07/2026 06:08:37 PM)

Message:

Een playlist federeert als AP-collectie (shaer-ayc, stap 1)

De eerste stap van het Funkwhale-spoor, en playlists gaan voor omdat zij als
enige al een id hebben -- een album is nog een tekstveld en kan nergens aan
hangen.

GET /ap/users/:slug/playlists/:id geeft een OrderedCollection van Audio-
objecten, dezelfde rijvorm die een post als attachment meestuurt, zodat elke
client die post-audio speelt dit ook speelt. Kaal AS2, geen eigen vocab:
artiest is summary, duur is PT..S, hoes is icon en een track zonder eigen
hoes erft die van de playlist.

De poortregel verandert niet: alleen fedi_open-tracks staan erin, met echte
bestands-URL. Een gated track is geen lege rij maar afwezig, en totalItems
telt alleen het open deel -- wie leest kan niet aftellen hoeveel er achter
de poort staat, en ziet titel noch bestandsnaam.

Tests: bestaan/vorm, poort (inclusief niet-lekken van titel en bestandsnaam),
volgorde (de gated track staat er middenin, dus meetellen zou ook de volgorde
breken), tenancy (playlist van site A op actor B is 404).

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

Files:

• src/routes/activitypub.js (modified) (1 diff)
• src/services/ActivityPubService.js (modified) (2 diffs)
• test/ap-playlist.test.js (added)

src/routes/activitypub.js

@@ -630 +630 @@
 });
 
+// ── Playlist als dereferenceerbare AP-collectie (shaer-ayc) ───────
+// De eerste stap van het Funkwhale-spoor: een playlist heeft een id, dus een
+// stabiele URI. Alleen het fedi_open-deel staat erin (de poort is per bestand
+// en eenrichtings; zie setAudioFediOpen in routes/posts.js) — een collectie
+// zonder open tracks bestaat wel maar is leeg, want de playlist zelf is niet
+// geheim, alleen de bestanden erachter.
+router.get('/ap/users/:slug/playlists/:id', (req, res) => {
+  const site = publicSite(req.params.slug);
+  if (!site) return res.status(404).end();
+  const pl = db.prepare('SELECT id, title, artist, year, cover_url, kind FROM playlists WHERE id = ? AND site_id = ?')
+    .get(req.params.id, site.id);
+  if (!pl) return res.status(404).end();
+  AP.sendAP(res, AP.buildPlaylistCollection(baseUrl(req), site, pl, AP.playlistOpenTracks(pl.id)));
+});
+
 // ── Note ──────────────────────────────────────────────────────────
 router.get('/ap/notes/:id', async (req, res) => {

src/services/ActivityPubService.js

@@ -868 +868 @@
     orderedItems: items,
   };
+}
+
+// ── Playlist als AP-collectie (shaer-ayc, stap 1 van het Funkwhale-spoor) ──
+// Een playlist heeft, anders dan een album-als-tekstveld, een id — dus kan hij
+// een stabiele URI dragen en federeren. De vorm is bewust kaal AS2: een
+// OrderedCollection van Audio-objecten, dezelfde rijvorm die een post als
+// attachment meestuurt, zodat elke client die post-audio al speelt dit ook
+// speelt.
+//
+// De poortregel verandert hier NIET: alleen fedi_open-tracks staan erin, met
+// echte bestands-URL. Een gated track is niet "een rij zonder url" maar
+// afwezig — wie de collectie leest ziet het open deel en kan niet aftellen
+// hoeveel er achter de poort staat. totalItems telt daarom ook alleen het
+// open deel: een eerlijke telling over wat er werkelijk in de collectie staat,
+// niet over wat wij thuis in de kast hebben.
+export function playlistOpenTracks(playlistId) {
+  return db.prepare(
+    `SELECT t.title, t.artist, t.duration, t.cover_url, m.filename, m.storage_path, m.mime_type
+     FROM playlist_tracks pt
+     JOIN audio_tracks t ON t.id = pt.track_id
+     JOIN media m ON m.id = t.media_id
+     WHERE pt.playlist_id = ? AND t.fedi_open = 1
+     ORDER BY pt.position`
+  ).all(playlistId);
+}
+
+export function buildPlaylistCollection(base, site, playlist, rows) {
+  const abs = (u) => !u ? null : (/^https?:/i.test(u) ? u : `${base}${u.startsWith('/') ? '' : '/'}${u}`);
+  // Zelfde afleiding als in buildNote; die daar is functie-lokaal.
+  const mediaType = (u) => {
+    const e = ((u || '').split('?')[0].match(/\.(\w+)$/) || [])[1];
+    return ({ jpg: 'image/jpeg', jpeg: 'image/jpeg', png: 'image/png', gif: 'image/gif', webp: 'image/webp', avif: 'image/avif' })[(e || '').toLowerCase()] || 'image/jpeg';
+  };
+  const items = (rows || []).map((r) => {
+    const fn = r.filename || (r.storage_path || '').split('/').pop();
+    const a = {
+      type: 'Audio',
+      mediaType: r.mime_type || 'audio/mpeg',
+      url: `${base}/audio/stream/${encodeURIComponent(fn)}`,
+      name: r.title || 'Audio',
+    };
+    if (r.artist) a.summary = r.artist; // artiest als summary: kaal AS2, geen eigen vocab
+    if (r.duration) a.duration = `PT${Math.round(r.duration)}S`;
+    const art = abs(r.cover_url || playlist.cover_url || null);
+    if (art) a.icon = { type: 'Image', mediaType: mediaType(art), url: art };
+    return a;
+  });
+  const out = {
+    '@context': AP_CONTEXT,
+    id: `${actorId(base, site.slug)}/playlists/${playlist.id}`,
+    type: 'OrderedCollection',
+    name: playlist.title,
+    attributedTo: actorId(base, site.slug),
+    totalItems: items.length,
+    orderedItems: items,
+  };
+  // Album of playlist is presentatie; op de draad is het één samenvattingsveld.
+  const parts = [];
+  if (playlist.artist) parts.push(playlist.artist);
+  if (playlist.year) parts.push(String(playlist.year));
+  if (parts.length) out.summary = parts.join(' · ');
+  const cover = abs(playlist.cover_url || null);
+  if (cover) out.icon = { type: 'Image', mediaType: mediaType(cover), url: cover };
+  return out;
 }
 
@@ -5418 +5482 @@
   AP_CONTEXT, getOrCreateKeys, apWants, sendAP, actorId, noteId, stripLeadingMentions,
   buildActor, buildNote, buildCreate, buildOutbox, buildFollowers, buildFollowing, buildFeatured,
+  buildPlaylistCollection, playlistOpenTracks,
   followerCount, deliver, fetchActor, verifyRequest, handleInbox, deliverCreate, deliverDelete, deliverUpdate, deliverActorUpdate, resyncFeaturedPins,
   feedCursor, feedChangesSince, waitForFeedChange,