Changeset 861c228 in Klonkt for test

Timestamp:

08/14/2026 01:43:59 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

dc802b4

Parents:

9f9b45f

Message:

Gebakken links wijzen voortaan naar hier, in beide routes

Robin: de gebakken tracklinks moeten de nieuwe links worden. Terecht, en ik had
ze te makkelijk afgedaan als "geen plaatje, dus niet mijn probleem". Het zijn
tijdbommen: buildNote zet ​https://oud/<slug>#track-<id> en ?fc=2 in de content,
en zodra dat domein opgezegd wordt staan er dode links in je eigen berichten.

Ze kunnen nu ook echt om, en dat is te danken aan "altijd behouden" van
vanmiddag: slug en track-id zijn hier hetzelfde, dus alleen de host hoeft eraf
en het fragment klopt vanzelf.

DE REGEL: alleen ombuigen voor een slug die hier ECHT bestaat. Wijst een link
naar iets dat we niet hebben, dan blijft hij naar de oude site wijzen. Een
verwijzing naar een pagina die er nog is, is beter dan een 404 op de nieuwe.

Pas aan het EIND van de ronde, want dan pas weten we welke slugs binnen zijn.
Bij een verse ingest zou de helft van de links nog niet te beoordelen zijn.

EN IN BEIDE ROUTES ("het moet wel gebeuren bij migratie direct ook"). Een
zip-import is net zo goed een verhuizing. Daar draait het binnen dezelfde
transactie: half bijgetrokken is erger dan niet.

Changed files:
src/services/MigrationService.js

• postLinksBijtrekken(), geexporteerd zodat de zip hem ook kan draaien
• aangeroepen zodra alle berichten binnen zijn

src/services/ArchiveImportService.js

• dezelfde stap aan het eind van de import, in de transactie
• rapportveld linksBijgetrokken

src/views/pages/admin-migrate.ejs

• in het import- en het ophaalverslag

src/services/i18n.js

• mig.r_links_fixed in nl/en/de

test/fep1580-migration.test.js

• fragment en querystring blijven heel, een onbekende slug blijft met rust

test/archive-audio.test.js

• dezelfde eis voor de zip-route

remarks: /media, /audio en /ap worden overgeslagen; die lopen via hun eigen weg.
Suite 1000 groen, tegenproef op allebei de routes.

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

Location:

test

Files:

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

test/archive-audio.test.js

@@ -234 +234 @@
 });
 
+test('ook een zip-import buigt links naar de bronpost om', () => {
+  // Robin: "het moet wel gebeuren bij migratie direct ook". Een zip-import is
+  // net zo goed een verhuizing, dus dezelfde regel.
+  leeg();
+  try { db.prepare('DELETE FROM posts').run(); } catch { /* leeg */ }
+  db.prepare(`INSERT INTO posts (id, site_id, author_id, slug, title, content, status, published_at)
+              VALUES ('pl1','s1','u1','tiktik','TikTik','<p><a href="https://oud.test/tiktik?fc=2">luister</a></p>','published','2026-01-01T10:00:00Z')`).run();
+  const uit = AX.buildArchive('me');
+
+  db.prepare('DELETE FROM posts').run();
+  // De import doet alsof het archief van oud.test komt; deze site is oud.test
+  // niet, dus de links moeten om.
+  const files = new Map(uit.files);
+  const man = JSON.parse(files.get('manifest.json').toString('utf8'));
+  man.origin = 'https://oud.test';
+  files.set('manifest.json', Buffer.from(JSON.stringify(man)));
+
+  const r = AI.importArchive(files, { slug: 'me', origin: 'https://nieuw.test' });
+  const c = db.prepare("SELECT content FROM posts WHERE slug = 'tiktik'").get().content;
+  assert.ok(c.includes('href="/tiktik?fc=2"'), `omgebogen naar hier, kreeg: ${c}`);
+  assert.ok(!c.includes('oud.test'), 'niets wijst meer naar de bron');
+  assert.equal(r.linksBijgetrokken, 1, 'en het verslag zegt het');
+});
+
 test.after(() => { try { fs.rmSync(TMP, { recursive: true, force: true }); } catch { /* niets */ } });

test/fep1580-migration.test.js

@@ -577 +577 @@
 });
 
+test('links naar de bronpost buigen om naar hier, in dezelfde ronde', async () => {
+  // De gebakken "luister op"-links: buildNote zet
+  // https://oud/<slug>#track-<id> en ?fc=2 in de content. Die bleven naar de
+  // oude site wijzen, en dat is een tijdbom: zodra dat domein weg is staan er
+  // dode links in je eigen berichten. Slugs en track-ids blijven behouden, dus
+  // de fragmenten kloppen vanzelf zodra de host eraf is.
+  const s = site({ aliases: [BRON] });
+  const OUD = 'https://oud.example';
+  const kaart = new Map([
+    [BRON, { id: BRON, type: 'Person', movedTo: IK, outbox: `${BRON}/outbox` }],
+    [`${BRON}/outbox`, { type: 'OrderedCollection', totalItems: 2, first: `${BRON}/outbox?page=1` }],
+    [`${BRON}/outbox?page=1`, { type: 'OrderedCollectionPage', orderedItems: [
+      note(`${BRON}/notes/tiktik`, { url: `${OUD}/tiktik`, content:
+        `<p>🎵 <a href="${OUD}/tiktik#track-t1">TikTik</a> — <a href="${OUD}/tiktik?fc=2">luister</a></p>` }),
+      // een link naar iets dat we NIET hebben blijft met rust
+      note(`${BRON}/notes/ander`, { url: `${OUD}/ander`, content:
+        `<p>zie <a href="${OUD}/bestaat-hier-niet">daar</a></p>` }),
+    ] }],
+  ]);
+  const r = await stil(() => Mig.ingestFromSource(s, { deps: {
+    getJson: async (_s, url) => kaart.get(url) || null,
+    noteId: (b, id) => `${b}/ap/notes/${id}`,
+    noteVisibility: AP.noteVisibility,
+  } }));
+  assert.equal(r.posts, 2);
+  const tik = db.prepare("SELECT content FROM posts WHERE slug = 'tiktik'").get().content;
+  assert.ok(tik.includes('href="/tiktik#track-t1"'), `fragment blijft heel, kreeg: ${tik}`);
+  assert.ok(tik.includes('href="/tiktik?fc=2"'), 'en de querystring ook');
+  assert.ok(!tik.includes('oud.example'), 'er wijst niets meer naar de oude site');
+  const ander = db.prepare("SELECT content FROM posts WHERE slug = 'ander'").get().content;
+  assert.ok(ander.includes(`${OUD}/bestaat-hier-niet`),
+    'een link naar iets dat hier niet bestaat blijft staan: beter de oude site dan een 404 op de nieuwe');
+  assert.equal(r.linksBijgetrokken, 1);
+});
+
 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."