Changeset 7090fd2 in Klonkt for scripts/funkwhale-gat.mjs

Timestamp:

08/16/2026 02:12:14 PM (3 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

9026de5

Parents:

aebde2b

Message:

Album: geen epoch-datum, en een naam die overal hetzelfde is

Gevonden door de twee representaties van hetzelfde id naast elkaar te
leggen, met een echte album-playlist op dev (Robin, 16-8).

EEN ID, TWEE NAMEN. Het ingesloten Album heette "Cartoon Epic" en zijn
eigen URI "Geen koffie, wel thee!". buildPlaylistCollection laat
leenVanPost de titel van de uitgavepost overnemen -- de post IS de
uitgave -- en buildAlbumObject deed dat niet. Nu leent hij dezelfde
titel, met de eigen titel als alsoKnownAs, precies zoals leenVanPost.
De uitgavepost wordt EEN keer per album opgezocht en niet per track:
uitgavePost() leest de typering van de post, en een site heeft veel meer
nummers dan platen.

PUBLISHED 1970-01-01. buildAlbumObject viel terug op epoch als
created_at ontbrak, en dat gebeurde: de AP-route selecteerde id, title,
artist, year, cover_url en kind -- en de rest niet. Daarmee ontbraken
released en musicbrainzId daar trouwens ook.

Twee dingen gerepareerd, en de terugval zelf is het derde: er komt nu
GEEN published meer als er geen datum is. Het veld is bij hen verplicht,
dus dat kost ons een item -- maar 1970 is een antwoord en geen ontbrekend
veld, en een lezer kan een gat opmerken en een leugen niet. Ontbreekt het
nog, dan is er een leespad dat de kolom laat vallen, en dat willen we
zien.

Ook PlaylistService.get() geeft created_at nu terug.

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

File:

• scripts/funkwhale-gat.mjs (modified) (1 diff)

scripts/funkwhale-gat.mjs

@@ -52 +52 @@
 const items = pagina.items || pagina.orderedItems || [];
 if (!items.length) { console.log('geen items in de bibliotheek'); process.exit(1); }
-const a = items[0];
 
-console.log(`bron: ${BRON}\nitem: ${a.name}\n`);
+// ALLE items, niet het eerste. De eerste versie keek naar items[0] en gaf
+// daarover een oordeel alsof het over de bibliotheek ging -- op 16-8 meldde hij
+// zo `track.album` als GAT terwijl drie van de vier tracks hem wel hadden: het
+// ene nummer dat hij toevallig pakte stond niet op een plaat. Een steekproef
+// van een is geen meting, en een meting die je vertrouwt moet zeggen hoeveel.
+console.log(`bron: ${BRON}\nitems: ${items.length}\n`);
+
 let gaten = 0;
-const toon = (lijst, waarde) => {
+const toon = (kop, lijst, kies) => {
+  console.log(kop);
   for (const [naam, test] of lijst) {
-    let ok = false;
-    try { ok = !!test(waarde); } catch { ok = false; }
-    if (!ok) gaten++;
-    console.log(`  ${ok ? 'OK ' : 'GAT'}  ${naam}`);
+    let ok = 0;
+    for (const a of items) { try { if (test(kies(a))) ok++; } catch { /* telt als niet ok */ } }
+    const staat = ok === items.length ? 'OK ' : (ok === 0 ? 'GAT' : 'DEELS');
+    if (ok !== items.length) gaten++;
+    console.log(`  ${staat.padEnd(5)} ${naam}${ok !== items.length ? `  (${ok}/${items.length})` : ''}`);
   }
 };
-console.log('UploadSerializer (regel 1741):');
-toon(UPLOAD, a);
-console.log('\nTrackSerializer (regel 1569 + MusicEntity 1278):');
-toon(TRACK, a.track);
+toon('UploadSerializer (regel 1741):', UPLOAD, (a) => a);
+console.log();
+toon('TrackSerializer (regel 1569 + MusicEntity 1278):', TRACK, (a) => a.track);
 
-console.log(`\n${gaten === 0 ? 'niets meer nodig voor hun ingest' : gaten + ' veld(en) te gaan'}`);
+console.log(`\n${gaten === 0 ? 'niets meer nodig voor hun ingest' : gaten + ' veld(en) nog niet op ELK item'}`);