Changeset 4d81de5 in Klonkt

Timestamp:

08/08/2026 05:30:29 AM (5 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

cf5073b

Parents:

34e63f0

git-author:

Robin <roboburr@…> (08/08/2026 05:28:46 AM)

git-committer:

Robin <roboburr@…> (08/08/2026 05:30:29 AM)

Message:

De feed vertelt ook WAT voor kanaal dit is

Funkwhale maakte van dev@… een kanaal en las de rss_url netjes uit
de Link-array op onze actor -- maar liet content_category leeg. Dat is de
aanwijzing: hij bouwt zo'n kanaal langs de RSS-kant op, en leest de categorie
daar. Niet uit category op de AP-actor.

Dus itunes-velden erbij op kanaalniveau: author, summary, image (de site-avatar,
absoluut gemaakt) en category. En per track de eigen artiest en hoes, naast de
duur die er al stond.

EN IK HEB MIJN EIGEN CATEGORIE-REGEL AANGESCHERPT. Die keek naar ELKE track,
ook een gated -- zodat een site met uitsluitend afgeschermde muziek toch "hier
is muziek" naar de hele fediverse riep. Dat botst met de regel die we overal
aanhouden: een gesloten track is AFWEZIG, niet stilletjes aanwezig. Nu tellen
alleen opengezette tracks, en zeggen de feed en de actor hetzelfde. Er staat
een test op die precies dat vastlegt: een gated track maakt er nog geen
muziekkanaal van.

Dat maakt de regel strenger dan hij vanmiddag was. Bewust: naar buiten toe is
een kanaal zonder publieke muziek geen muziekkanaal.

4 tests.

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

Files:

• src/routes/feed.js (modified) (3 diffs)
• src/services/ActivityPubService.js (modified) (2 diffs)
• test/channel-actor.test.js (modified) (1 diff)
• test/feed-itunes.test.js (added)

src/routes/feed.js

@@ -66 +66 @@
   const tracks = siteOpenTracks(site.id);
   const lastBuild = posts[0]?.published_at || new Date().toISOString();
+
+  // De itunes-velden waar een podcast-app een kanaal aan herkent. Funkwhale
+  // bouwde onze kanaalpagina langs de RSS-kant op en liet de categorie leeg,
+  // want die leest hij hier -- niet uit `category` op de AP-actor.
+  //
+  // De categorie volgt DEZELFDE regel als daar: alleen 'Music' als er ook
+  // werkelijk publieke muziek is. Een blog zonder open track is naar buiten
+  // toe geen muziekkanaal, en gated muziek telt niet mee -- afwezig is
+  // afwezig, ook in een categorie.
+  const abs = (u) => !u ? null : (/^https?:/i.test(u) ? u : `${base}${u.startsWith('/') ? '' : '/'}${u}`);
+  const kanaalKunst = abs(site.profile_photo || site.og_image_default || null);
+  const kanaalTags = [
+    `    <itunes:author>${escapeXml(site.author || site.title || '')}</itunes:author>`,
+    site.description || site.tagline
+      ? `    <itunes:summary>${escapeXml(site.description || site.tagline)}</itunes:summary>` : null,
+    kanaalKunst ? `    <itunes:image href="${escapeXml(kanaalKunst)}" />` : null,
+    tracks.length ? '    <itunes:category text="Music" />' : null,
+  ].filter(Boolean).join('\n');
 
   const wanneer = (d) => { const t = Date.parse(d); return Number.isNaN(t) ? 0 : t; };
@@ -88 +106 @@
       <description>${escapeXml(t.artist || '')}</description>
       <enclosure url="${escapeXml(`${base}/audio/stream/${encodeURIComponent(fn)}`)}" length="${Number(t.size) || 0}" type="${escapeXml(t.mime_type || 'audio/mpeg')}" />${t.duration ? `
-      <itunes:duration>${Math.round(t.duration)}</itunes:duration>` : ''}
+      <itunes:duration>${Math.round(t.duration)}</itunes:duration>` : ''}${t.artist ? `
+      <itunes:author>${escapeXml(t.artist)}</itunes:author>` : ''}${abs(t.cover_url) ? `
+      <itunes:image href="${escapeXml(abs(t.cover_url))}" />` : ''}
     </item>` };
     }),
@@ -103 +123 @@
     <lastBuildDate>${new Date(lastBuild).toUTCString()}</lastBuildDate>
     <atom:link href="${escapeXml(base + '/feed.xml')}" rel="self" type="application/rss+xml" />
+${kanaalTags}
 ${items}
   </channel>

src/services/ActivityPubService.js

@@ -233 +233 @@
  * de kast, niet enable_audio_player -- die staat standaard aan en zegt niets.
  */
-function channelCategory(site) {
-  try {
-    return db.prepare('SELECT 1 FROM audio_tracks WHERE site_id = ? LIMIT 1').get(site.id) ? 'music' : null;
+export function channelCategory(site) {
+  try {
+    // ALLEEN opengezette tracks tellen. Eerst keek dit naar elke track, ook een
+    // gated -- en dan roept een site met uitsluitend afgeschermde muziek toch
+    // "hier is muziek" naar de hele fediverse. Dat botst met de regel die we
+    // overal aanhouden: een gesloten track is AFWEZIG, niet stilletjes
+    // aanwezig. Naar buiten toe is een kanaal zonder publieke muziek geen
+    // muziekkanaal.
+    return db.prepare('SELECT 1 FROM audio_tracks WHERE site_id = ? AND fedi_open = 1 LIMIT 1').get(site.id) ? 'music' : null;
   } catch { return null; }
 }
@@ -6039 +6045 @@
   AP_CONTEXT, getOrCreateKeys, apWants, sendAP, actorId, noteId, stripLeadingMentions,
   buildActor, buildNote, buildCreate, buildOutbox, buildFollowers, buildFollowing, buildFeatured,
-  channelUrls, timelineFields, guessMediaType,
+  channelUrls, channelCategory, timelineFields, guessMediaType,
   siteOpenTracks, openTrack, buildTrackAudio, buildTrackCollection, buildTrackCreate,
   buildPlaylistCollection, playlistOpenTracks, listPlaylistsAP, playlistLinkTags,

test/channel-actor.test.js

@@ -36 +36 @@
 });
 
-test('met muziek is de category "music"', () => {
+test('een GATED track maakt er nog geen muziekkanaal van', () => {
+  // Aangescherpt: eerst telde elke track mee, ook een afgeschermde -- dan riep
+  // een site met uitsluitend gated muziek toch "hier is muziek" naar buiten.
+  // Afwezig is afwezig, ook in een categorie.
+  db.prepare('INSERT INTO media (id, site_id, filename, storage_path, mime_type, size) VALUES (?,?,?,?,?,1)')
+    .run('m0', 's1', 'dicht.mp3', 'audio/dicht.mp3', 'audio/mpeg');
+  db.prepare('INSERT INTO audio_tracks (id, site_id, title, media_id, fedi_open) VALUES (?,?,?,?,0)')
+    .run('t0', 's1', 'Achter de poort', 'm0');
+  assert.equal(AP.buildActor(BASE, site()).category, undefined);
+});
+
+test('met OPENGEZETTE muziek is de category "music"', () => {
   db.prepare('INSERT INTO media (id, site_id, filename, storage_path, mime_type, size) VALUES (?,?,?,?,?,1)')
     .run('m1', 's1', 'a.mp3', 'audio/a.mp3', 'audio/mpeg');
-  db.prepare('INSERT INTO audio_tracks (id, site_id, title, media_id) VALUES (?,?,?,?)')
+  db.prepare('INSERT INTO audio_tracks (id, site_id, title, media_id, fedi_open) VALUES (?,?,?,?,1)')
     .run('t1', 's1', 'Een nummer', 'm1');
   assert.equal(AP.buildActor(BASE, site()).category, 'music');