Changeset cf5073b in Klonkt

Timestamp:

08/08/2026 05:47:57 AM (5 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

f3a2556

Parents:

4d81de5

git-author:

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

git-committer:

Robin <roboburr@…> (08/08/2026 05:47:57 AM)

Message:

Onze Audio inhoudelijk gelijk aan die van Funkwhale

Dezelfde track aan beide kanten naast elkaar gelegd -- 0METAL MARIO, bij hen op
the_ceeesg en bij ons op dev -- en de uitkomst was gunstiger dan verwacht.

OP ELK VELD WAAR AS2 IETS ZEGT ZIJN WIJ CONFORM EN ZIJ NIET, en dat is precies
een veld: duration. AS2 zegt MUST xsd:duration ("PT103S"), schema.org zegt
hetzelfde in ISO 8601, Funkwhale stuurt 103. Daar bewegen we dus niet.

Al hun EXTRA velden bleken geen eigen verzinsel maar termen die schema.org
gewoon heeft -- en schema.org stond al in onze context. Overgenomen zonder
concessie: de SLEUTELS zijn die van Funkwhale, want daar leest hij op, de
BETEKENIS komt van schema.org. Geen vreemd vocabulaire binnengehaald.

to op het OBJECT, niet alleen op de omhullende Create: een los

opgehaalde track moet zelf kunnen zeggen dat hij openbaar is

position het nummer op de plaat
license schema:license
size schema:contentSize, OP DE LINK -- het is die representatie die
bitrate schema:bitrate zoveel bytes is, niet het nummer
image naast icon; allebei AS2-kern, wij lazen icon, zij lezen image

BITRATE IS AFGELEID, niet gegokt: bytes maal acht gedeeld door seconden IS de
gemiddelde bitrate van een bestand. Alleen als we allebei de getallen hebben --
liever geen veld dan een verzonnen getal.

LICENTIE IS DE ENIGE MET EEN VERTAALSLAG. Bij ons is het vrije tekst uit een
keuzelijst ("CC BY 4.0"), bij hen een URI. Alleen de waarden die onze eigen
keuzelijst aanbiedt worden vertaald -- die kennen we exact. "Alle rechten
voorbehouden" heeft geen URI en levert dus niets op, en een al ingevulde URI
gaat ongemoeid door. Een zelfbedachte licentie-URI is erger dan geen: een lezer
gelooft hem.

Onderweg: t.position en t.license moesten alsnog in de trackquery, die
selecteerde ze niet.

7 tests.

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

Files:

• src/services/ActivityPubService.js (modified) (4 diffs)
• test/ap-track-fields.test.js (added)
• test/ap-tracks.test.js (modified) (1 diff)

src/services/ActivityPubService.js

@@ -52 +52 @@
     value: 'schema:value',
     embedUrl: { '@id': 'schema:embedUrl', '@type': '@id' },
+    // Wat een track beschrijft en AS2 niet kent (shaer-0nh). Funkwhale zet deze
+    // vier op zijn Audio; het bleken geen eigen verzinsels maar termen die
+    // schema.org gewoon heeft -- en schema.org stond hier al. De SLEUTELS zijn
+    // die van Funkwhale, want daar leest hij op; de BETEKENIS komt van
+    // schema.org, dus we hoeven geen vreemd vocabulaire binnen te halen.
+    license: { '@id': 'schema:license', '@type': '@id' },
+    position: 'schema:position',
+    bitrate: 'schema:bitrate',
+    size: 'schema:contentSize',
     // Poll (Question) extension: Question/oneOf/anyOf/endTime/closed are AS2 core, but the
     // per-poll unique-voter count is a Mastodon (toot) term — declare it so the emitted
@@ -1067 +1076 @@
 // m.size hoort erbij voor de RSS-enclosure: die eist een lengte in bytes.
 const TRACK_KOLOMMEN = `t.id, t.title, t.artist, t.duration, t.cover_url, t.created_at,
+     t.position, t.license,
      m.filename, m.storage_path, m.mime_type, m.size`;
 
@@ -1123 +1133 @@
   const abs = (u) => !u ? null : (/^https?:/i.test(u) ? u : `${base}${u.startsWith('/') ? '' : '/'}${u}`);
   const fn = r.filename || (r.storage_path || '').split('/').pop();
+  // De bestandsgegevens horen bij de LINK, niet bij het object: het is die ene
+  // representatie die zoveel bytes is en die bitrate heeft, niet het nummer.
+  // Zo doet Funkwhale het ook.
+  const bestand = { type: 'Link', href: `${base}/audio/stream/${encodeURIComponent(fn)}`, mediaType: r.mime_type || 'audio/mpeg' };
+  if (Number(r.size)) bestand.size = Number(r.size);
+  // Bitrate leiden we af uit bytes en seconden. Geen gok: voor een bestand IS
+  // dat de gemiddelde bitrate, en bij CBR ook de echte. Alleen als we allebei
+  // de getallen hebben -- liever geen veld dan een verzonnen getal.
+  if (Number(r.size) && Number(r.duration)) bestand.bitrate = Math.round((Number(r.size) * 8) / Number(r.duration));
+
   const a = {
     ...(opts.standalone ? { '@context': AP_CONTEXT } : {}),
@@ -1129 +1149 @@
     name: r.title || 'Audio',
     attributedTo: actorId(base, site.slug),
-    url: [{ type: 'Link', href: `${base}/audio/stream/${encodeURIComponent(fn)}`, mediaType: r.mime_type || 'audio/mpeg' }],
+    // Op het OBJECT, niet alleen op de omhullende Create: een los opgehaalde
+    // track moet zelf kunnen zeggen dat hij openbaar is.
+    to: [PUBLIC],
+    url: [bestand],
   };
   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`;
   if (r.created_at) a.published = new Date(r.created_at).toISOString();
+  if (Number(r.position)) a.position = Number(r.position);
+  const lic = licentieUri(r.license);
+  if (lic) a.license = lic;
   const art = abs(r.cover_url || opts.coverFallback || null);
-  if (art) a.icon = { type: 'Image', mediaType: guessMediaType(art), url: art };
+  // icon EN image: allebei AS2-kern. Wij gebruikten alleen icon; Funkwhale
+  // leest image. Dezelfde hoes, twee namen, niemand die iets misloopt.
+  if (art) {
+    const plaat = { type: 'Image', mediaType: guessMediaType(art), url: art };
+    a.icon = plaat;
+    a.image = plaat;
+  }
   return a;
+}
+
+/**
+ * Onze licentie is VRIJE TEKST uit een keuzelijst ("CC BY 4.0", "Alle rechten
+ * voorbehouden"); schema.org en Funkwhale willen een URI. Alleen de waarden die
+ * onze eigen keuzelijst aanbiedt worden vertaald -- die kennen we exact. Al het
+ * andere levert niets op: een zelfbedachte licentie-URI is erger dan geen, want
+ * een lezer gelooft hem.
+ */
+const LICENTIES = {
+  'cc0 1.0 (publiek domein)': 'http://creativecommons.org/publicdomain/zero/1.0/',
+  'cc by 4.0': 'http://creativecommons.org/licenses/by/4.0/',
+  'cc by-sa 4.0': 'http://creativecommons.org/licenses/by-sa/4.0/',
+  'cc by-nc 4.0': 'http://creativecommons.org/licenses/by-nc/4.0/',
+  'cc by-nc-sa 4.0': 'http://creativecommons.org/licenses/by-nc-sa/4.0/',
+  'cc by-nd 4.0': 'http://creativecommons.org/licenses/by-nd/4.0/',
+};
+export function licentieUri(waarde) {
+  const s = String(waarde || '').trim();
+  if (!s) return null;
+  if (/^https?:\/\//i.test(s)) return safeUrl(s);   // iemand vulde al een URI in
+  return LICENTIES[s.toLowerCase()] || null;        // "Alle rechten voorbehouden" heeft er geen
 }
 

test/ap-tracks.test.js

@@ -75 +75 @@
   assert.equal(a.duration, 'PT203S');
   assert.equal(a.summary, 'De Band');
-  assert.deepEqual(a.url, [{
-    type: 'Link', href: 'https://test.example/audio/stream/open.mp3', mediaType: 'audio/mpeg',
-  }], 'de mediaType hoort bij de link, niet bij het object');
+  // De Link draagt sinds de veldvergelijking met Funkwhale ook de
+  // bestandsgegevens: die horen bij DEZE representatie, niet bij het nummer.
+  assert.equal(a.url.length, 1);
+  assert.equal(a.url[0].type, 'Link');
+  assert.equal(a.url[0].href, 'https://test.example/audio/stream/open.mp3');
+  assert.equal(a.url[0].mediaType, 'audio/mpeg', 'de mediaType hoort bij de link, niet bij het object');
+  assert.equal(a.mediaType, undefined, 'en dus NIET op het object');
   assert.equal(a.icon.url, 'https://test.example/media/hoes.jpg');
 });