Changeset a117862 in Klonkt

Timestamp:

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

Author:

Robin <roboburr@…>

Branches:

main

Children:

5300652

Parents:

8455dac

git-author:

Robin <roboburr@…> (08/13/2026 12:06:21 PM)

git-committer:

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

Message:

Stap 1: een artiest kan zichzelf opzoeken in MusicBrainz (shaer-mbz)

Waarom dit geen dialect is en Funkwhale's Track/ArtistCredit wel: een MBID is
geen vocabulaire maar een REGISTER. Ernaar verwijzen is als een ISBN noemen --
je neemt niemands model over en je wijst naar iets dat al bestaat. En het is de
brug die Funkwhale zelf al kent, want hun Track draagt musicbrainzId.

WAT ER STAAT

• MusicBrainzService: zoeken op artiestennaam, kandidaten met hun disambiguatie, soort, land en jaren erbij. De naam alleen is niet genoeg om te kiezen -- er zijn drie bands die Nirvana heten, en dat is precies waar hun disambiguation-veld voor is.
• mb_artist_id en mb_artist_name op sites. De naam erbij zodat het scherm kan tonen WAT er gekoppeld is zonder ervoor te netwerken, en zodat een verkeerde koppeling opvalt.
• GET /admin/audio/api/musicbrainz, dat standaard zoekt op de artiestennaam die al in de site staat.

HUN TWEE HARDE REGELS, INGEBAKKEN EN GETEST. Overtreden leidt tot een BLOKKADE
en niet tot een foutmelding, dus "het werkte toen ik het probeerde" is er geen
bewijs voor:

• hoogstens EEN verzoek per seconde, over de hele applicatie. Daarom draait de zoekopdracht server-side: die regel geldt per applicatie en niet per bezoeker, dus twee tabbladen zouden hem samen overtreden. Een test doet twee zoekopdrachten tegelijk en eist een gat van een seconde.
• een echte User-Agent met contactgegevens. Een test leest hem uit het verzoek.

DE KEUZE BLIJFT VAN DE ARTIEST. We tonen kandidaten; we kiezen er niet zelf een,
ook niet als er maar een treffer is. Een verkeerd geraden MBID koppelt iemand aan
het werk van een ander, en dat is erger dan geen koppeling.

WAT ER NIET IN ZIT: schrijven naar MusicBrainz. Kan niet via hun API voor
artiesten -- alleen tags, ratings, ISRC's en barcodes -- en zou ook niet moeten.

Zonder netwerk getest: globalThis.fetch vervangen en musicbrainz.org in
AP_ALLOW_HOSTS, zodat er geen DNS aan te pas komt. Anders test het of deze
machine internet heeft.

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

Files:

• src/config/database.js (modified) (1 diff)
• src/routes/admin-audio.js (modified) (2 diffs)
• src/services/MusicBrainzService.js (added)
• test/musicbrainz.test.js (added)

src/config/database.js

@@ -134 +134 @@
   ensureColumn('ap_follow_reviews', 'target_handle', 'TEXT');
   ensureColumn('sites', 'profile_photo', 'TEXT');
+  // De MusicBrainz-koppeling van de artiest (shaer-mbz). Een MBID is een
+  // verwijzing naar hun register, geen kopie ervan -- de naam staat erbij zodat
+  // het beheerscherm kan tonen WAT er gekoppeld is zonder ervoor te moeten
+  // netwerken, en zodat een verkeerde koppeling opvalt.
+  ensureColumn('sites', 'mb_artist_id', 'TEXT');
+  ensureColumn('sites', 'mb_artist_name', 'TEXT');
   ensureColumn('audio_tracks', 'cover_url', 'TEXT');
   ensureColumn('audio_tracks', 'album', 'TEXT');

src/routes/admin-audio.js

@@ -23 +23 @@
 import { audioUrl } from '../services/AudioStreamService.js';
 import { mediaDir } from '../config/paths.js';
+import MusicBrainz from '../services/MusicBrainzService.js';
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
@@ -383 +384 @@
 
 /** GET /admin/audio/api/albums — distinct list of album names (for datalist) */
+/**
+ * "Ben jij dit?" -- kandidaten uit MusicBrainz (shaer-mbz, stap 1).
+ *
+ * De zoekopdracht draait HIER en niet in de browser: MusicBrainz staat een
+ * verzoek per seconde toe per APPLICATIE, en dat is alleen af te dwingen als
+ * alles langs een plek gaat. Bovendien eisen ze een User-Agent met contact, en
+ * die kan een browser niet zetten.
+ *
+ * De keuze blijft van de artiest. Wij tonen kandidaten met hun toelichting; we
+ * kiezen er niet zelf een, ook niet als er maar een treffer is -- een verkeerd
+ * geraden MBID koppelt iemand aan het werk van een ander.
+ */
+router.get('/api/musicbrainz', requireGod, async (req, res) => {
+  const site = res.locals.site;
+  if (!site) return res.status(404).json({ error: 'no_site' });
+  // Standaard de artiestennaam die al in de site staat: negen van de tien keer
+  // is dat precies waar iemand op zou zoeken.
+  const q = String(req.query.q || site.author || site.title || '').trim();
+  if (!q) return res.json({ ok: true, q: '', kandidaten: [] });
+  const kandidaten = await MusicBrainz.zoekArtiesten(q);
+  res.json({
+    ok: true,
+    q,
+    gekoppeld: site.mb_artist_id
+      ? { mbid: site.mb_artist_id, naam: site.mb_artist_name || '', url: MusicBrainz.artiestUrl(site.mb_artist_id) }
+      : null,
+    kandidaten,
+  });
+});
+
 router.get('/api/albums', requireGod, (req, res) => {
   const site = res.locals.site;