Changeset dc802b4 in Klonkt

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@…>

Files:

• src/routes/activitypub.js (modified) (1 diff)
• src/services/ActivityPubService.js (modified) (4 diffs)
• test/fep1580-migration.test.js (modified) (1 diff)

src/routes/activitypub.js

@@ -212 +212 @@
     limit: AP.PAGINA_GROOTTE,
   });
-  const ob = AP.buildOutbox(baseUrl(req), site, posts, tracks, { page: nr, totalItems: totaal, alGesneden: true });
+  // FEP-1580: de instantie waar dit account naartoe verhuisd is krijgt de
+  // RAUWE inhoud, met [[track:]] en [[playlist:]] er nog in. Zij is een
+  // Klonkt en rendert die zelf tot een speler. De gebakken variant komt
+  // daar aan als tekstlink, en is bovendien onherstelbaar afgeknot: het
+  // bakken plakt hooguit vier titels aan.
+  const rauweInhoud = verifiedActor ? AP.isMoveTarget(site.slug, verifiedActor) : false;
+  const ob = AP.buildOutbox(baseUrl(req), site, posts, tracks, { page: nr, totalItems: totaal, alGesneden: true, rauweInhoud });
   if (audience === 'friend') {
     // The owner's app builds its feed from this leg, and every note here is

src/services/ActivityPubService.js

@@ -706 +706 @@
     } catch { /* non-fatal */ }
   }
-  body = body.replace(/\[\[(track|album|playlist):[^\]]+\]\]/gi, '');
+  // ONVERTAALD voor een verhuizing (FEP-1580). De doelinstantie IS een Klonkt:
+  // die rendert [[track:]], [[album:]] en [[playlist:]] zelf en maakt er een
+  // speler van. Bakken we ze eerst om, dan komt er een tekstlink aan en is de
+  // speler weg. Onherstelbaar bovendien: het bakken STRIPT de shorthand en
+  // plakt achteraan hooguit VIER titels, dus een album van tien nummers
+  // overleeft het niet.
+  //
+  // Dezelfde regel als bij de outbox en de tracks: wie ondertekend vraagt
+  // namens de actor waar wij naartoe verhuisd zijn, krijgt onze eigen kijk.
+  if (!opts.rauweInhoud) body = body.replace(/\[\[(track|album|playlist):[^\]]+\]\]/gi, '');
   // External embeds ([[embed:url]]) → emit the bare URL as a link so Mastodon
   // renders its OWN preview/player card (YouTube/Spotify/SoundCloud/etc) instead
@@ -714 +723 @@
     return `<p><a href="${u}">${u}</a></p>`;
   });
-  if (hadAudio) {
+  if (hadAudio && !opts.rauweInhoud) {
     // Elke titel als eigen link naar zijn anker; een titel zonder id (een
     // albumnaam zonder tracks) blijft gewone tekst.
@@ -1039 +1048 @@
 }
 
-export function buildCreate(base, site, post) {
-  const note = buildNote(base, site, post);
+export function buildCreate(base, site, post, opts = {}) {
+  const note = buildNote(base, site, post, opts);
   return {
     '@context': AP_CONTEXT,
@@ -1123 +1132 @@
 }
 
-export function buildOutbox(base, site, posts, tracks = [], { page = false, totalItems, alGesneden = false } = {}) {
+export function buildOutbox(base, site, posts, tracks = [], { page = false, totalItems, alGesneden = false, rauweInhoud = false } = {}) {
   const id = `${actorId(base, site.slug)}/outbox`;
   const wanneer = (x) => Date.parse(x && x.published ? x.published : 0) || 0;
   const items = [
-    ...(posts || []).map((p) => buildCreate(base, site, p)),
+    ...(posts || []).map((p) => buildCreate(base, site, p, { rauweInhoud })),
     // Eén zoekopdracht voor alle tracks samen, niet per stuk.
     ...(() => {

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."