Changeset f50a84b in Klonkt for src/services/MusicBrainzService.js

Timestamp:

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

Author:

Robin <roboburr@…>

Branches:

main

Children:

c3e736c

Parents:

013e16c

Message:

Jouw teksten, en de twee dingen die ze beloofden (shaer-mbz)

De titel is nu "MusicBrainz Koppeling" met jouw omschrijving eronder, in drie
talen. Maar die omschrijving belooft twee dingen die de code nog niet deed, en
een label dat liegt is erger dan een lelijk label. Dus allebei gebouwd.

"OF JE MUSICBRAINZ ID ALS JE DIE KENT". Plak je een UUID, dan wordt hij
rechtstreeks opgezocht in plaats van gezocht. Dat is geen luxe: een
zoekopdracht OP een UUID levert bij MusicBrainz niets op, dus zonder deze tak
gaf plakken juist het slechtste resultaat.

"MET TERUG-WEG VALIDATIE". Een koppeling van onze kant is een bewering --
iedereen kan een id in een veld typen. Pas als de MusicBrainz-pagina TERUGWIJST
naar dit domein is het een paar, en dan weet een lezer dat dezelfde persoon aan
allebei de kanten stond. Dezelfde gedachte als rel="me". Het paneel zegt
"wederzijds" of "nog eenzijdig", en bij dat laatste staat erbij WAT je moet doen:
het domein op je MusicBrainz-pagina zetten onder "social networking".

Wij zetten die terugwijzing niet zelf. Kan niet via hun API, en hoort ook niet.

ECHT GEMETEN, en dat gaf de test zijn vorm: robo-burr staat in MusicBrainz met
score 100 (24abe2be-c0bc-4c63-9642-d6f89ec6a00a, "AKA roboburr") en heeft
precies EEN url-relatie -- type "social network", naar sound-fabrics.com. Voor
dev.klonkt.com is de terug-weg dus false, en dat is precies de waarde van de
controle. Die vorm staat nu in een test.

Bij een storing komt er false EN een lege lijst uit: niet kunnen kijken is niet
hetzelfde als niet gevonden, en nooit stilletjes "wederzijds".

Een controle erbij die eist dat alle drie de talen dezelfde mb_-sleutels
hebben. Bij het herschrijven waren er zeven weggevallen die het scherm nog
gebruikt -- een gat in een vertaling geeft een leeg label en dat valt nooit
vanzelf op.

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

File:

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

src/services/MusicBrainzService.js

@@ -80 +80 @@
 }
 
+/**
+ * Een MBID rechtstreeks opzoeken. Wie zijn id al kent hoeft niet te zoeken --
+ * en een zoekopdracht op een UUID levert bij MusicBrainz niets op, dus zonder
+ * deze tak zou plakken juist het slechtste resultaat geven.
+ */
+export async function haalArtiest(mbid) {
+  if (!isMbid(mbid)) return null;
+  const url = `${BASIS}/artist/${encodeURIComponent(mbid)}?inc=url-rels&fmt=json`;
+  try {
+    await opDeBeurt();
+    const r = await safeFetch(url, { headers: { Accept: 'application/json', 'User-Agent': userAgent() } });
+    if (!r || !r.ok) return null;
+    return kandidaat(await r.json());
+  } catch {
+    return null;
+  }
+}
+
+/**
+ * DE TERUG-WEG. Noemt de MusicBrainz-pagina van deze artiest ons domein?
+ *
+ * Een koppeling van onze kant is een bewering: iedereen kan een MBID in een
+ * veld typen. Pas als de artiestenpagina TERUGWIJST is het een paar, en dan
+ * weet een lezer dat dezelfde persoon aan allebei de kanten stond. Dat is
+ * dezelfde gedachte als rel="me" bij Mastodon.
+ *
+ * Wij zetten die terugwijzing NIET zelf: via hun API kan het niet, en het hoort
+ * ook niet -- de artiest doet dat op musicbrainz.org onder "social networking".
+ * Wij kijken alleen of hij er staat.
+ *
+ * Geeft { verified, urls } -- bij een storing verified:false en een lege lijst,
+ * want niet kunnen kijken is niet hetzelfde als niet gevonden.
+ */
+export async function controleerTerugweg(mbid, domein) {
+  const leeg = { verified: false, urls: [] };
+  if (!isMbid(mbid) || !domein) return leeg;
+  let host;
+  try { host = new URL(domein).host.toLowerCase(); } catch { return leeg; }
+  const url = `${BASIS}/artist/${encodeURIComponent(mbid)}?inc=url-rels&fmt=json`;
+  try {
+    await opDeBeurt();
+    const r = await safeFetch(url, { headers: { Accept: 'application/json', 'User-Agent': userAgent() } });
+    if (!r || !r.ok) return leeg;
+    const doc = await r.json();
+    const urls = (doc.relations || [])
+      .map((rel) => rel && rel.url && rel.url.resource)
+      .filter((u) => typeof u === 'string');
+    const wijst = urls.some((u) => { try { return new URL(u).host.toLowerCase() === host; } catch { return false; } });
+    return { verified: wijst, urls };
+  } catch {
+    return leeg;
+  }
+}
+
 /** Een kandidaat, teruggebracht tot wat een mens nodig heeft om te kiezen. */
 function kandidaat(a) {
@@ -104 +158 @@
 // ze precies wonen.
 export { isMbid, artiestUrl };
-export default { zoekArtiesten, isMbid, artiestUrl };
+export default { zoekArtiesten, haalArtiest, controleerTerugweg, isMbid, artiestUrl };