Changeset dc802b4 in Klonkt for test/fep1580-migration.test.js

Timestamp:

08/14/2026 01:55:45 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

e9128bc

Parents:

861c228

Message:

Een verhuizing krijgt de rauwe inhoud, niet de gebakken

Robin: het is gelukt, maar posts laden de playlists, albums of audio niet. Het
bericht toonde "🎵 TikTik — listen on soundfabrics" als tekstlink, waar een
speler hoort te staan.

buildNote bakt de shorthand om voor de fediverse: [[track:]], [[album:]] en
[[playlist:]] worden GESTRIPT en er komt achteraan een blok met de titels. Dat
is precies goed voor Mastodon, en precies fout voor een verhuizing: de
doelinstantie is zelf een Klonkt en maakt van [[track:]] gewoon een speler.

Ik heb eerst overwogen om die link terug te vertalen naar de shorthand. Dat kan
niet zonder verlies:

• de shorthand wordt gestript, dus zijn PLAATS in de post is weg; het blok komt altijd achteraan;
• er gaan hooguit VIER titels in, dus een album van tien nummers is onherstelbaar;
• [[album:]] en [[playlist:]] worden opgelost naar losse tracks, dus je krijgt tien spelers in plaats van een plaat.

Dus niet terugvertalen maar niet bakken. buildNote kreeg een optie
rauweInhoud, de outbox-route zet hem aan voor precies de actor waar dit
account naartoe verhuisd is. Dezelfde regel als bij de outbox-audience, de
tracks en de playlists: wie ondertekend vraagt namens de doel-actor krijgt onze
eigen kijk.

Gemeten op twee instanties, dezelfde post twee keer opgehaald:

anoniem <p>🎵 <strong><a href=".../tiktik#track-t-a">Nummer A</a>...
verhuizing <p>hoor dit</p>[[track:t-a]]

En [[playlist:pl]] komt heel aan, wat met terugvertalen nooit was gelukt.

Changed files:
src/services/ActivityPubService.js

• buildNote kent rauweInhoud: niet strippen, geen listen-blok aanplakken
• buildCreate en buildOutbox geven de optie door

src/routes/activitypub.js

• de outbox zet hem aan voor de doel-actor van een Move

test/fep1580-migration.test.js

• dezelfde post gebakken en rauw, naast elkaar

remarks: dit raakt alleen wie ondertekend vraagt namens moved_to. Voor de rest
van de fediverse verandert er niets aan de outbox. Suite 1001 groen, tegenproef
gedaan.

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

File:

• test/fep1580-migration.test.js (modified) (1 diff)

test/fep1580-migration.test.js

@@ -612 +612 @@
 });
 
+test('de doelinstantie krijgt de RAUWE inhoud, met de shorthand er nog in', () => {
+  // Robins laatste klacht: de post kwam aan met "🎵 TikTik — listen on
+  // soundfabrics" en zonder speler. De bron BAKT de shorthand namelijk om voor
+  // de fediverse: [[track:]] wordt gestript en er komt achteraan een blok met
+  // hooguit VIER titels. Onherstelbaar dus: een album van tien nummers is weg.
+  //
+  // Voor een verhuizing hoort dat bakken niet te gebeuren. De doelinstantie is
+  // zelf een Klonkt en maakt van [[track:]] gewoon weer een speler.
+  const s = site();
+  db.prepare("INSERT INTO media (id, site_id, filename, mime_type, size, storage_path) VALUES ('m1','s1','a.mp3','audio/mpeg',1,'/x/a.mp3')").run();
+  db.prepare("INSERT INTO audio_tracks (id, site_id, title, media_id) VALUES ('t1','s1','Nummer A','m1')").run();
+  db.prepare(`INSERT INTO posts (id, site_id, author_id, slug, title, content, status, published_at)
+              VALUES ('pa','s1','u1','plaat','Plaat','<p>hoor dit</p>[[track:t1]]','published','2026-01-01T10:00:00Z')`).run();
+  const post = db.prepare("SELECT * FROM posts WHERE id = 'pa'").get();
+
+  const gebakken = AP.buildNote('https://oud.example', s, post);
+  assert.ok(!gebakken.content.includes('[[track:t1]]'), 'de gewone federatie bakt hem om');
+  assert.match(gebakken.content, /listen on/, 'en plakt er een tekstlink aan');
+
+  const rauw = AP.buildNote('https://oud.example', s, post, { rauweInhoud: true });
+  assert.ok(rauw.content.includes('[[track:t1]]'), 'voor een verhuizing blijft de shorthand staan');
+  assert.ok(!/listen on/.test(rauw.content), 'en er wordt geen tekstlink aangeplakt');
+});
+
 test('een bericht dat je zelf hebt verwijderd komt bij een tweede ronde terug', async () => {
   // Robin: "ik kan handmatig deze keer de posts verwijderen en opnieuw ophalen."