Changeset 19a9a1e in Klonkt for src/services

Timestamp:

08/15/2026 07:43:02 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

bec9001

Parents:

e1da6a1

Message:

Library: audience meesturen en de ongeordende collectievorm gebruiken

De 500 op /ap/users/dev/library kwam van open.audio, niet van ons, en de
oorzaak staat in hun bron (federation/serializers.py op develop):

LibrarySerializer.create() regel 1147

"privacy_level": privacy[validated_dataaudience]

audience staat als required=False in de serializer, dus als wij het
weglaten zit de sleutel niet in validated_data en is dat een KeyError --
een 500 op elke poging onze bibliotheek op te halen. Wij stuurden hem
niet.

Het veld is bovendien gewoon waar: alles in de bak is fedi_open. En bij
hen is het precies het verschil tussen privacy_level 'everyone' en 'me',
oftewel of onze nummers daar afspeelbaar zijn.

Daarnaast de vorm. Hun CollectionPageSerializer (regel 1157) accepteert
alleen CollectionPage met items; wij stuurden OrderedCollectionPage
met orderedItems, en dan komt er van de inhoud niets binnen. Dat is geen
dialect maar de andere helft van AS2: een platenkast heeft geen volgorde
die iets betekent, en hun LibrarySerializer accepteert als type ook alleen
as:Collection of fw:Library. De outbox is wél geordend en blijft zoals hij
was.

De eisen staan nu met bronverwijzing in test/ap-library.test.js, want ze
stonden nergens bij ons opgeschreven -- en het volgende dat gebeurt is
anders dat iemand dit 'opruimt' naar de gewone vorm.

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

Location:

src/services

Files:

• ap-core.js (modified) (3 diffs)
• music/index.js (modified) (1 diff)

src/services/ap-core.js

@@ -265 +265 @@
  * en zonder `next`. Hem naar de laatste pagina terugbuigen zou opnieuw een
  * antwoord zijn dat over zichzelf liegt.
- */
-export function pagedCollection(id, items, { totalItems, page = false, perPage = PAGINA_GROOTTE, alGesneden = false, extra = {} } = {}) {
+ *
+ * `ongeordend` maakt er de NIET-geordende vorm van: `Collection` met
+ * `CollectionPage` en `items`, in plaats van `OrderedCollection` met
+ * `OrderedCollectionPage` en `orderedItems`. Dat is geen dialect maar de andere
+ * helft van AS2 -- en de bibliotheek hoort daar: een platenkast heeft geen
+ * volgorde die iets betekent, en `Library` is bij Funkwhale expliciet een
+ * `Collection`. Onze outbox is wél geordend (chronologie is daar de inhoud) en
+ * blijft dus zoals hij was.
+ *
+ * De pagina draagt in die vorm ook `first` en `last`. AS2 staat dat toe --
+ * CollectionPage erft van Collection -- en een lezer die halverwege binnenkomt
+ * kan zo terug naar het begin zonder eerst de wortel op te halen.
+ */
+export function pagedCollection(id, items, { totalItems, page = false, perPage = PAGINA_GROOTTE, alGesneden = false, ongeordend = false, extra = {} } = {}) {
   const lijst = items || [];
   const telling = totalItems === undefined ? lijst.length : totalItems;
@@ -283 +295 @@
       '@context': AP_CONTEXT,
       id: url(n),
-      type: 'OrderedCollectionPage',
+      type: ongeordend ? 'CollectionPage' : 'OrderedCollectionPage',
       partOf: id,
       totalItems: telling,
+      ...(ongeordend ? { first: url(1), last: url(paginas) } : {}),
+      ...(extra.attributedTo ? { attributedTo: extra.attributedTo } : {}),
       ...(n > 1 ? { prev: url(n - 1) } : {}),
       ...(n < paginas ? { next: url(n + 1) } : {}),
-      orderedItems: deel,
+      ...(ongeordend ? { items: deel } : { orderedItems: deel }),
     };
   }
@@ -294 +308 @@
     '@context': AP_CONTEXT,
     id,
-    type: 'OrderedCollection',
+    type: ongeordend ? 'Collection' : 'OrderedCollection',
     ...extra,
     totalItems: telling,
     first: url(1),
     last: url(paginas),
-    orderedItems: lijst,
+    ...(ongeordend ? { items: lijst } : { orderedItems: lijst }),
   };
 }

src/services/music/index.js

@@ -253 +253 @@
   return pagedCollection(id, items, {
     page,
+    // Een platenkast is geen tijdlijn: `Collection`, niet `OrderedCollection`.
+    // Funkwhale's LibrarySerializer accepteert ook alleen die twee typen
+    // (as:Collection of fw:Library) en zijn CollectionPageSerializer alleen
+    // `CollectionPage` met `items`.
+    ongeordend: true,
     extra: {
       type: 'Library',
       name: site.title || site.slug,
       attributedTo: actorId(base, site.slug),
+      // WAAROM DIT VELD ER MOET STAAN. Funkwhale's LibrarySerializer noemt
+      // `audience` optioneel, maar zijn create() doet er meteen
+      // `privacy[validated_data["audience"]]` mee -- zonder de sleutel is dat
+      // een KeyError en geeft hun server een 500. Dat is wat open.audio op 15-8
+      // teruggaf toen Robin onze library-URI daar opzocht.
+      //
+      // Het is bovendien gewoon waar: alles hierin is fedi_open, dus openbaar.
+      // Bij hen is dit precies het verschil tussen privacy_level 'everyone' en
+      // 'me' -- oftewel of onze nummers daar afspeelbaar zijn.
+      audience: 'https://www.w3.org/ns/activitystreams#Public',
       // Vereist volgens hun docs. Openbaar, dus de telling is eerlijk en de
       // lijst blijft leeg -- wie ons volgt volgt de ACTOR, niet de bak.