Changeset fb22f78 in Klonkt

Timestamp:

08/08/2026 04:57:37 AM (5 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

47ca083

Parents:

39a9d21

git-author:

Robin <roboburr@…> (08/08/2026 04:57:33 AM)

git-committer:

Robin <roboburr@…> (08/08/2026 04:57:37 AM)

Message:

De outbox vertelt de discografie mee (shaer-0nh, stap 4)

Een track verschijnt nu als Create(Audio) in de outbox, door de posts heen op
datum. Daarmee is een Klonkt-actor als kanaal uit te lezen: precies zoals ik
Funkwhales kanaal uitlas -- hun achttien tracks stonden in hun outbox.

WAAROM GEEN BEZORGING NAAR VOLGERS, terwijl ik stap 4 eerder zo beschreef.
Mastodon neemt Audio aan als STATUSTYPE. Publiceer je een post met een album,
dan zouden volgers de Note met spelers krijgen EN N losse Audio-statussen:
dezelfde muziek twee keer in hun tijdlijn. Dat is geen spiegel afmaken maar
andermans tijdlijn vervuilen. Een kanaal-lezer HAALT op; hij hoeft niet
geduwd te worden. De post is het bericht, de outbox is de discografie.

DE DEUR BLIJFT DICHT. buildOutbox krijgt de tracks als ARGUMENT en haalt ze
uitdrukkelijk niet zelf op. De route beslist wie wat ziet, en de blocked-tak
daar levert een outbox zonder posts EN zonder tracks. Een bouwer die stiekem
zijn eigen database bevraagt zou dwars door die dichte deur heen leveren --
dat had ik bijna gebouwd, en de tak in de route ving het pas bij nalezen.

Het id van de activiteit is stabiel (<track>#create), zodat twee keer ophalen
niet twee keer nieuws is.

5 tests.

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

Files:

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

src/routes/activitypub.js

@@ -133 +133 @@
      ORDER BY COALESCE(published_at, created_at) DESC LIMIT 20`
   ).all(site.id);
-  const ob = AP.buildOutbox(baseUrl(req), site, posts);
+  // De tracks gaan mee voor iedereen die de deur door mag; de blocked-tak
+  // hierboven levert bewust een outbox ZONDER posts en zonder tracks.
+  const ob = AP.buildOutbox(baseUrl(req), site, posts, AP.siteOpenTracks(site.id));
   if (audience === 'friend') {
     // The owner's app builds its feed from this leg, and every note here is

src/services/ActivityPubService.js

@@ -945 +945 @@
 }
 
-export function buildOutbox(base, site, posts) {
+/**
+ * Een track als publicatie: Create(Audio) (shaer-0nh, stap 4).
+ *
+ * Zelfde vorm als buildCreate voor een post, met een STABIEL id: dezelfde track
+ * levert altijd dezelfde activiteit, zodat een lezer die de outbox twee keer
+ * ophaalt niet denkt dat er iets nieuws is.
+ */
+export function buildTrackCreate(base, site, r) {
+  const audio = buildTrackAudio(base, site, r);
+  const me = actorId(base, site.slug);
+  return {
+    '@context': AP_CONTEXT,
+    id: `${audio.id}#create`,
+    type: 'Create',
+    actor: me,
+    published: audio.published,
+    to: [PUBLIC],
+    cc: [`${me}/followers`],
+    object: audio,
+  };
+}
+
+/**
+ * De outbox: wat deze actor heeft uitgebracht. Posts EN tracks (shaer-0nh,
+ * stap 4).
+ *
+ * WAAROM HIER EN NIET IN EEN BEZORGING. Een kanaal-lezer HAALT de outbox op --
+ * zo heb ik zelf Funkwhales kanaal uitgelezen. Een Create(Audio) ook naar de
+ * inboxen van volgers duwen zou schade doen: Mastodon neemt Audio aan als
+ * statustype, dus bij een album-post zou dezelfde muziek twee keer in hun
+ * tijdlijn komen -- een keer als bijlage bij de Note, en dan nog N keer los.
+ * De post is het bericht, de outbox is de discografie.
+ *
+ * Door elkaar op datum, nieuwste eerst, zodat de outbox één verhaal vertelt in
+ * plaats van twee lijstjes achter elkaar.
+ *
+ * De tracks komen als ARGUMENT binnen, net als de posts, en worden hier
+ * uitdrukkelijk NIET zelf opgehaald. De route beslist wie wat mag zien -- een
+ * geblokkeerde bezoeker krijgt daar een lege outbox, en een bouwer die stiekem
+ * zijn eigen database bevraagt zou dwars door die deur heen leveren.
+ */
+export function buildOutbox(base, site, posts, tracks = []) {
   const id = `${actorId(base, site.slug)}/outbox`;
-  const items = (posts || []).slice(0, MAX_OUTBOX).map((p) => buildCreate(base, site, p));
+  const wanneer = (x) => Date.parse(x && x.published ? x.published : 0) || 0;
+  const items = [
+    ...(posts || []).map((p) => buildCreate(base, site, p)),
+    ...(tracks || []).map((r) => buildTrackCreate(base, site, r)),
+  ]
+    .sort((a, b) => wanneer(b) - wanneer(a))
+    .slice(0, MAX_OUTBOX);
   return {
     '@context': AP_CONTEXT,
@@ -5969 +6016 @@
   buildActor, buildNote, buildCreate, buildOutbox, buildFollowers, buildFollowing, buildFeatured,
   channelUrls, timelineFields, guessMediaType,
-  siteOpenTracks, openTrack, buildTrackAudio, buildTrackCollection,
+  siteOpenTracks, openTrack, buildTrackAudio, buildTrackCollection, buildTrackCreate,
   buildPlaylistCollection, playlistOpenTracks, listPlaylistsAP, playlistLinkTags,
   followerCount, deliver, fetchActor, verifyRequest, handleInbox, deliverCreate, deliverDelete, deliverUpdate, deliverActorUpdate, resyncFeaturedPins,