Changeset f59a46c in Klonkt

Timestamp:

08/13/2026 01:09:24 PM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

013e16c

Parents:

6401681

git-author:

Robin <roboburr@…> (08/13/2026 12:52:57 PM)

git-committer:

Robin <roboburr@…> (08/13/2026 01:09:24 PM)

Message:

Stap 3: de koppeling gaat de draad op als schema:sameAs (shaer-mbz)

Alleen als de artiest ZELF gekozen heeft. Staat er niets, dan staat er niets --
een lege of geraden verwijzing is erger dan geen.

WAAROM sameAs EN NIET alsoKnownAs, en dat is geen smaakkwestie. alsoKnownAs is
in AS2 gereserveerd voor vroegere IDENTITEITEN van dezelfde actor, en FEP-7628
leunt erop bij een verhuizing: het oude adres controleert of het nieuwe hem daar
noemt voordat het volgers meegeeft. Een MBID daartussen zetten zou een
verhuizing kunnen laten mislukken. schema:sameAs zegt precies het goede: dit
ding is ook bekend onder die URI.

Geen nieuwe namespace: schema stond al in onze context, naast license, position
en bitrate. Dit is er een termregel bij.

EEN KRINGLOOP VERMEDEN. MusicBrainzService leent safeFetch uit
ActivityPubService, dus die er andersom uit laten importeren zou een cirkel
maken. isMbid en artiestUrl zijn pure vormcontroles zonder afhankelijkheden en
wonen nu in ap-core -- dezelfde plek en dezelfde reden als de tagparsers van
gisteren. MusicBrainzService exporteert ze her, zodat een aanroeper er niet over
hoeft na te denken.

DE ACTOR IS DE LAATSTE DEUR. Het scherm zeeft al op een echte MBID, maar
artiestUrl doet het nog een keer: wat hier langskomt gaat naar iedereen, en een
half adres is erger dan geen. Een test zet met opzet rommel in de kolom en eist
dat er niets naar buiten gaat.

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

Files:

• src/services/ActivityPubService.js (modified) (2 diffs)
• src/services/MusicBrainzService.js (modified) (2 diffs)
• src/services/ap-core.js (modified) (2 diffs)
• test/musicbrainz.test.js (modified) (1 diff)

src/services/ActivityPubService.js

@@ -29 +29 @@
 import Blocklist from './BlocklistService.js';
 import * as Guardianship from './guardianship/index.js';
-import { PUBLIC, AP_CONTEXT, safeUrl, actorId, noteId, guessMediaType, normalizeTags, tagParts, hashtagTags, buildHashtagList, pagedCollection, PAGINA_GROOTTE } from './ap-core.js';
+import { PUBLIC, AP_CONTEXT, safeUrl, actorId, noteId, guessMediaType, normalizeTags, tagParts, hashtagTags, buildHashtagList, pagedCollection, PAGINA_GROOTTE, artiestUrl } from './ap-core.js';
 // Doorgeven wat hier altijd vandaan kwam, zodat elke bestaande aanroep blijft werken.
 export { AP_CONTEXT, actorId, noteId, guessMediaType };
@@ -394 +394 @@
   // and publishers should stop delivering here.
   if (site.moved_to && /^https?:\/\//i.test(String(site.moved_to))) actor.movedTo = String(site.moved_to);
+  // De MusicBrainz-koppeling van de artiest (shaer-mbz). Alleen als hij ZELF
+  // gekozen heeft -- er staat niets als er niets gekoppeld is, want een lege
+  // of geraden verwijzing is erger dan geen.
+  //
+  // schema:sameAs en niet alsoKnownAs: dat laatste is in AS2 voor vroegere
+  // identiteiten van dezelfde actor, en FEP-7628 leunt erop bij een verhuizing.
+  // Een MBID hier neerzetten zou een verhuizing kunnen laten mislukken.
+  const mbUrl = artiestUrl(site.mb_artist_id);
+  if (mbUrl) actor.sameAs = mbUrl;
   // Profile links → PropertyValue rows: Mastodon/PeerTube/WordPress-ActivityPub render these as
   // profile metadata (rel=me enables link-back verification). Additive; ignored by simpler receivers.

src/services/MusicBrainzService.js

@@ -20 +20 @@
  */
 import { safeFetch } from './ActivityPubService.js';
+// De twee pure vormcontroles wonen in ap-core: ActivityPubService heeft ze ook
+// nodig voor de actor, en zonder die verhuizing zou dat een KRINGLOOP zijn --
+// deze module leent immers safeFetch dáár.
+import { isMbid, artiestUrl } from './ap-core.js';
 
 const BASIS = 'https://musicbrainz.org/ws/2';
@@ -97 +101 @@
 }
 
-/** Is dit een MBID? Een UUID, en niets anders. */
-export function isMbid(s) {
-  return /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i.test(String(s || '').trim());
-}
-
-/** De publieke pagina van een artiest, of null als het geen MBID is. */
-export function artiestUrl(mbid) {
-  return isMbid(mbid) ? `https://musicbrainz.org/artist/${String(mbid).trim().toLowerCase()}` : null;
-}
-
+// Her-geexporteerd zodat een aanroeper er niet over hoeft na te denken waar
+// ze precies wonen.
+export { isMbid, artiestUrl };
 export default { zoekArtiesten, isMbid, artiestUrl };

src/services/ap-core.js

@@ -50 +50 @@
     // schema.org, dus we hoeven geen vreemd vocabulaire binnen te halen.
     license: { '@id': 'schema:license', '@type': '@id' },
+    // "Dit ding is ook bekend onder die URI" -- voor de MusicBrainz-koppeling
+    // van een artiest (shaer-mbz). Bewust NIET alsoKnownAs: dat is in AS2
+    // gereserveerd voor vroegere IDENTITEITEN van dezelfde actor, en een
+    // verhuizing leunt erop (FEP-7628). Een verwijzing naar een register is
+    // iets anders dan een oud account van jezelf, en die twee door elkaar halen
+    // zou een verhuizing kunnen laten mislukken.
+    sameAs: { '@id': 'schema:sameAs', '@type': '@id' },
     position: 'schema:position',
     bitrate: 'schema:bitrate',
@@ -268 +275 @@
   };
 }
+
+/** Is dit een MBID? Een UUID, en niets anders. */
+export function isMbid(s) {
+  return /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i.test(String(s || '').trim());
+}
+
+/** De publieke pagina van een artiest, of null als het geen MBID is. */
+export function artiestUrl(mbid) {
+  return isMbid(mbid) ? `https://musicbrainz.org/artist/${String(mbid).trim().toLowerCase()}` : null;
+}

test/musicbrainz.test.js

@@ -124 +124 @@
   assert.equal(s.mb_artist_name, null, 'anders blijft er een naam staan zonder koppeling');
 });
+
+// ── Stap 3: op de draad ───────────────────────────────────────────────────
+
+test('de actor draagt sameAs zodra er gekoppeld is -- en anders niets', async () => {
+  const db = dbMod.default;
+  const AP = await import('../src/services/ActivityPubService.js');
+  const BASE = 'https://ons.test';
+  const kaal = db.prepare("SELECT * FROM sites WHERE id = 's1'").get();
+  assert.equal(AP.buildActor(BASE, kaal).sameAs, undefined,
+    'zonder koppeling staat er niets -- een lege verwijzing is erger dan geen');
+
+  db.prepare('UPDATE sites SET mb_artist_id = ? WHERE id = ?')
+    .run('8be31978-1884-4773-beae-f73df35b92aa', 's1');
+  const gekoppeld = db.prepare("SELECT * FROM sites WHERE id = 's1'").get();
+  const actor = AP.buildActor(BASE, gekoppeld);
+  assert.equal(actor.sameAs, 'https://musicbrainz.org/artist/8be31978-1884-4773-beae-f73df35b92aa');
+});
+
+test('rommel in de kolom komt NIET op de draad', () => {
+  // Het scherm zeeft al, maar de actor is de laatste deur. Wat hier langskomt
+  // gaat naar iedereen, en een half adres is erger dan geen.
+  const db = dbMod.default;
+  db.prepare('UPDATE sites SET mb_artist_id = ? WHERE id = ?').run('nirvana', 's1');
+  const s = db.prepare("SELECT * FROM sites WHERE id = 's1'").get();
+  assert.equal(MB.artiestUrl(s.mb_artist_id), null);
+});
+
+test('sameAs is een eigen term en NIET alsoKnownAs', async () => {
+  // alsoKnownAs is in AS2 voor vroegere IDENTITEITEN van dezelfde actor, en
+  // FEP-7628 leunt erop bij een verhuizing: het oude adres controleert of het
+  // nieuwe hem daar noemt. Een MBID daar neerzetten zou een verhuizing kunnen
+  // laten mislukken.
+  const core = await import('../src/services/ap-core.js');
+  const term = core.AP_CONTEXT.find((x) => x && typeof x === 'object' && x.sameAs);
+  assert.ok(term, 'de term is gedeclareerd, anders laat een strikte lezer hem vallen');
+  assert.equal(term.sameAs['@id'], 'schema:sameAs');
+  assert.equal(term.sameAs['@type'], '@id', 'het is een URI en geen tekst');
+});