Changeset 4d81de5 in Klonkt for src/services/ActivityPubService.js

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@…>

File:

• src/services/ActivityPubService.js (modified) (2 diffs)

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,