Changeset f704237 in Klonkt for src/services/MigrationService.js

Timestamp:

08/14/2026 02:35:06 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

47f1183

Parents:

fc664ce

Message:

Een track zonder bestand is niet hetzelfde als een kapotte track

Robin, over het ene nummer dat niet meekwam: "ah, dat was een no audio 'audio'
voor alleen spotify en youtube links". Dat is een bestaand Klonkt-type, geen
ongelukje: geen gehost bestand, wel externe links, en buildNote maakt er een
embed-kaart van (trackEmbedLinks). Mijn regel "geen bestand, geen track" was te
grof en gooide hem weg.

"Geen bestand" en "niets om te tonen" zijn niet hetzelfde. De regel blijft
staan waar hij hoort: een track zonder bestand EN zonder links wordt nog steeds
geweigerd, want die kan niets.

Het archiefformaat heeft nu drie staten in plaats van twee: included, missing,
en linkOnly. Een link-only track telt dus ook niet meer als "ontbrekend", want
er ontbreekt niets.

EN EEN TWEEDE FOUT DIE HIERONDER LAG. De pull viste de audio-URL uit de
url-lijst met element NUL. buildTrackAudio zet daar een text/html-link naar de
post voor het bestand: wie blind de eerste pakt downloadt een HTML-pagina en
schrijft die weg als mp3. Nu wordt de link met mediaType audio/* gezocht, met
/audio/stream/ als terugval. Dat het bij Robin niet misging is geluk: zijn
tracks hadden geen host-post in die lijst.

Changed files:
src/services/ArchiveExportService.js

• shaer:availability kent linkOnly; die telt niet als audioMissing

src/services/ArchiveImportService.js

• link-only tracks komen mee, met media_id NULL zoals op de bron
• de schrijftak kan een track zonder bestand aan

src/services/MigrationService.js

• de pull vist de audio-link uit de lijst in plaats van element nul
• een track met alleen externe links wordt gewoon aangemaakt

test/archive-audio.test.js

• link-only reist mee; zonder bestand en zonder links blijft geweigerd

test/fep1580-migration.test.js

• hetzelfde voor de ophaalknop, plus: de audio-link wordt gevist

remarks: Robins Youngstown-nummer komt hiermee alsnog over. Suite 1009 groen;
tegenproef op beide, en de eerste poging daarvan kwam niet aan (te slordige
sabotage), dus opnieuw gedaan met twee rode tests als bewijs.

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

File:

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

src/services/MigrationService.js

@@ -512 +512 @@
   const rapport = {
     bron: bronActor.id, posts: 0, overgeslagen: 0, opnieuw: 0, postsBijgewerkt: 0, media: 0, mediaMislukt: 0,
-    blocks: 0, tracksBinnen: 0, tracksMislukt: 0, tracksBijgewerkt: 0, overgeslagenTracks: 0,
+    blocks: 0, tracksBinnen: 0, tracksMislukt: 0, tracksBijgewerkt: 0, tracksLinks: 0, overgeslagenTracks: 0,
     playlistsBinnen: 0, playlistsMislukt: 0, waarschuwingen: [],
   };
@@ -769 +769 @@
           }
         }
-        const bron = a.url && (typeof a.url === 'string' ? a.url : (Array.isArray(a.url) ? (a.url[0] && (a.url[0].href || a.url[0])) : a.url.href));
-        if (!bron || !/^https?:\/\//i.test(String(bron))) { rapport.tracksMislukt++; continue; }
+        // Uit de url-lijst de LINK NAAR HET BESTAND vissen, niet zomaar de eerste:
+        // buildTrackAudio zet er ook een text/html-link naar de post voor. En een
+        // LINK-ONLY track (alleen Spotify of YouTube, nooit een gehost bestand)
+        // heeft er helemaal geen. Die hoort gewoon mee, met media_id NULL.
+        // Weggooien kostte Robin een nummer dat het op de oude site prima deed.
+        const urls = [].concat(a.url || []).map((u) => (typeof u === 'string' ? { href: u } : u)).filter((u) => u && u.href);
+        const bestandLink = urls.find((u) => /^audio\//i.test(String(u.mediaType || '')))
+          || urls.find((u) => /\/audio\/stream\//.test(String(u.href)));
+        const externe = urls.map((u) => String(u.href)).filter((h) => /spotify|youtube|youtu\.be|soundcloud|bandcamp/i.test(h));
+        const bron = bestandLink && bestandLink.href;
+        if (!bron || !/^https?:\/\//i.test(String(bron))) {
+          if (externe.length) {
+            const trackIdL = ruwId(a.id) || crypto.randomUUID();
+            try {
+              db.prepare(`INSERT OR REPLACE INTO audio_tracks
+                  (id, site_id, title, artist, media_id, link_spotify, link_youtube, link_soundcloud, fedi_open)
+                VALUES (?,?,?,?,NULL,?,?,?,0)`)
+                .run(trackIdL, site.id, a.name || 'zonder titel', a.summary || a.artist || null,
+                  externe.find((u) => /spotify/i.test(u)) || null,
+                  externe.find((u) => /youtube|youtu\.be/i.test(u)) || null,
+                  externe.find((u) => /soundcloud/i.test(u)) || null);
+              recordMigrated(site.slug, { origin: a.id, target: `${me}/ap/tracks/${trackIdL}`, sourceActor: bronActor.id, isPublic: false });
+              trackKaart.set(String(a.id), trackIdL);
+              ruwKaart.set(ruwId(a.id), trackIdL);
+              rapport.tracksLinks++;
+              rapport.tracksBinnen++;
+            } catch (e) {
+              rapport.tracksMislukt++;
+              rapport.waarschuwingen.push(`nummer niet opgeslagen: ${a.name || a.id} (${e && e.message})`);
+            }
+            continue;
+          }
+          rapport.tracksMislukt++;
+          continue;
+        }
         const g = await haalBijlage(String(bron), {
           safeFetch, mediaRoot: audioRoot, fs, path, maxBytes, submap: '',