Changeset 19430fa in Klonkt for test

Timestamp:

08/14/2026 12:40:43 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

5a49eba

Parents:

919b82d

Message:

GUID's blijven altijd behouden bij een verhuizing

Robin vroeg of we bestaande GUID's hergebruiken. Het antwoord was zes vakjes:

zip ophaalknop

posts nieuw bij andere origin altijd nieuw
tracks OUD BEHOUDEN altijd nieuw
playlists OUD BEHOUDEN altijd nieuw

Vier keer nieuw, twee keer niet, en die twee uitzonderingen waren niet
besloten maar ontstaan: de zip schreef INSERT OR REPLACE met het id uit het
archief zonder dat daar ooit over nagedacht is. Die scheve tabel was precies de
oorzaak van de shortcode die Robin op TikTik zag: post uit de zip met
[[track:oud]], nummer uit de pull met een nieuw id.

Zijn besluit: altijd behouden. Nu is het één regel.

WAAROM DAT MAG. Het interne id is niet de AP-URI. ​https://nieuw/ap/notes/<id>
is een ander adres dan ​https://oud/ap/notes/<id>, dus je claimt niets van een
ander door het GUID te hergebruiken. Het oude argument in de code ("een id op
andermans domein publiceren is een vervalsingsoppervlak") haalde die twee door
elkaar. Wat je wint: elke interne verwijzing blijft kloppen, [[track:]],
[[playlist:]] en [[album:]] wijzen na de verhuizing nog naar het goede ding.

Wat NIET verandert is de AP-URI. Die is domeingebonden en hoort nieuw te zijn,
en daar is de migration-collectie voor. idsBehouden gaat voortaan alleen daar
nog over.

Gemeten door dezelfde inhoud via BEIDE routes over elkaar heen te halen:

posts oud 6 | nieuw 6 | zelfde id 6 | afwijkend 0
audio_tracks oud 3 | nieuw 3 | zelfde id 3 | afwijkend 0
playlists oud 1 | nieuw 1 | zelfde id 1 | afwijkend 0

Geen dubbele. Zip en ophaalknop zijn daarmee inwisselbaar geworden, en dat was
eerder de combinatie die stukging.

Changed files:
src/services/ArchiveImportService.js

• posts houden hun id, ongeacht de origin
• de waarschuwing zegt nu wat er echt verandert: het AP-adres, niet het id

src/services/MigrationService.js

• posts, tracks en playlists nemen het id van de bron over
• "staat hij hier al" is daarmee een blik in de tabel in plaats van een omweg via ap_migration; verwijderen en opnieuw ophalen werkt vanzelf
• de eerderPl/eerder-omwegen konden weg

test/archive-import.test.js

• de origin-test omgedraaid: het AP-adres verandert, het id blijft

test/fep1580-migration.test.js

• de shortcode-test toetst nu de UITKOMST (wijst naar een bestaand nummer) in plaats van de route ernaartoe
• nieuwe test voor het botsingsgeval, waar het bijtrekken wel nodig is

remarks: het bijtrekken van [[track:]] blijft bestaan als vangnet voor een
botsend id. In het normale geval doet het niets meer, en dat is de bedoeling.
Suite 991 groen.

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

Location:

test

Files:

• archive-import.test.js (modified) (2 diffs)
• fep1580-migration.test.js (modified) (2 diffs)

test/archive-import.test.js

@@ -126 +126 @@
 });
 
-test('een andere origin levert NIEUWE ids op, met een waarschuwing', () => {
-  // Oude ids houden op een ander domein zou objecten publiceren onder een id dat
-  // je niet beheert -- andere servers halen dat daar op, en het is bovendien een
-  // vervalsingsoppervlak.
+test('een andere origin geeft een nieuw AP-adres, maar het interne id BLIJFT', () => {
+  // Omgedraaid op 14-8 (Robin: "altijd behouden"). Het interne id is niet
+  // hetzelfde als de AP-URI: https://nieuw/ap/notes/<id> is een ander adres dan
+  // https://oud/ap/notes/<id>, dus je claimt niets van een ander door het GUID
+  // te hergebruiken. Wat je wint is dat elke INTERNE verwijzing blijft kloppen.
+  //
+  // Het oude gedrag brak precies dat: een post uit de zip hield
+  // [[track:<oud id>]] in zijn tekst terwijl het nummer een nieuw id kreeg, en
+  // dan zie je de shorthand als kale tekst in je bericht staan.
   leeg();
   const files = new Map(ARCHIEF.files);
@@ -136 +141 @@
   files.set('manifest.json', Buffer.from(JSON.stringify(m)));
   const r = AI.importArchive(files, { slug: 'me' });
-  assert.equal(r.idsBehouden, false);
+  assert.equal(r.idsBehouden, false, 'de AP-URI verandert wel, en dat blijft gemeld');
   assert.match(r.waarschuwingen.join(' '), /origin verschilt/);
-  assert.equal(db.prepare("SELECT COUNT(*) AS n FROM posts WHERE id = 'zwaar'").get().n, 0);
-  assert.equal(db.prepare('SELECT COUNT(*) AS n FROM posts').get().n, 1, 'wel geimporteerd, met een nieuw id');
+  assert.equal(db.prepare("SELECT COUNT(*) AS n FROM posts WHERE id = 'zwaar'").get().n, 1,
+    'het interne id komt ongewijzigd terug');
+  assert.equal(db.prepare('SELECT COUNT(*) AS n FROM posts').get().n, 1);
 });
 

test/fep1580-migration.test.js

@@ -481 +481 @@
 });
 
-test('de [[track:]]-verwijzing in een bericht wordt bijgetrokken', async () => {
+test('de [[track:]]-verwijzing in een bericht blijft naar een bestaand nummer wijzen', async () => {
   // Wat Robin op TikTik zag: het bericht toonde de shorthand zelf in plaats van
   // een speler. Zijn posts kwamen uit de ZIP (die bewaart posts.content
@@ -515 +515 @@
   }));
 
+  // DE EIS, ongeacht hoe: na een verhuizing wijst de shorthand naar een nummer
+  // dat hier bestaat. Sinds "altijd behouden" (14-8) klopt dat meestal vanzelf,
+  // want het id verandert niet meer. Botst het id wel, dan trekt de ingest de
+  // tekst bij. Deze test toetst de UITKOMST en niet de route ernaartoe.
   const inhoud = db.prepare("SELECT content FROM posts WHERE id = 'pz'").get().content;
   const m = /\[\[track:([^\]]+)\]\]/.exec(inhoud);
-  assert.ok(m, 'de shorthand blijft staan, alleen het id verandert');
-  assert.notEqual(m[1], 't-oud', 'het oude id wijst hier nergens heen');
+  assert.ok(m, 'de shorthand blijft staan');
   const bestaat = db.prepare('SELECT 1 FROM audio_tracks WHERE id = ? AND site_id = ?').get(m[1], 's1');
-  assert.ok(bestaat, 'en het nieuwe id hoort bij een nummer dat er echt is');
+  assert.ok(bestaat, `[[track:${m[1]}]] hoort bij een nummer dat er echt is`);
+  assert.equal(m[1], 't-oud', 'en omdat het id behouden blijft, hoefde er niets herschreven');
+});
+
+test('botst het track-id wel, dan wordt de tekst bijgetrokken', async () => {
+  // Het vangnet. "Altijd behouden" kan niet als er hier al iets anders met dat
+  // id staat; dan krijgt het nummer een ander id en moet de shorthand mee.
+  const s2 = site({ aliases: [BRON] });
+  db.prepare(`INSERT INTO posts (id, site_id, author_id, slug, title, content, status, published_at)
+              VALUES ('pb','s1','u1','botsing','Botsing','<p>[[track:t-bots]]</p>','published','2026-08-13T10:00:00Z')`).run();
+  // Een nummer dat hier AL bestaat onder datzelfde id, van iets anders.
+  db.prepare("INSERT INTO media (id, site_id, filename, mime_type, size, storage_path) VALUES ('mx','s1','x.mp3','audio/mpeg',1,'/x')").run();
+  db.prepare("INSERT INTO audio_tracks (id, site_id, title, media_id) VALUES ('t-bots','s1','Al van mij','mx')").run();
+
+  const kaart2 = new Map([
+    [BRON, { id: BRON, type: 'Person', movedTo: IK, outbox: `${BRON}/outbox`, streams: [`${BRON}/tracks`] }],
+    [`${BRON}/outbox`, { type: 'OrderedCollection', orderedItems: [] }],
+    [`${BRON}/tracks`, {
+      type: 'OrderedCollection',
+      orderedItems: [{
+        id: `${BRON}/tracks/t-bots`, type: 'Audio', name: 'Van de bron',
+        url: [{ type: 'Link', href: `${BRON}/audio/y.mp3`, mediaType: 'audio/mpeg' }],
+      }],
+    }],
+  ]);
+  await stil(() => Mig.ingestFromSource(s2, {
+    deps: {
+      getJson: async (_slug, url) => kaart2.get(url) || null,
+      noteId: (b, id) => `${b}/ap/notes/${id}`,
+      noteVisibility: AP.noteVisibility,
+      audioRoot: '/nep/audio', mediaRoot: '/nep/media',
+      signHeaders: () => ({ Signature: 'nep' }),
+      safeFetch: async () => ({ ok: true, arrayBuffer: async () => Buffer.from('y'), headers: { get: () => 'audio/mpeg' } }),
+      fs: { mkdirSync() {}, writeFileSync() {} },
+      path,
+    },
+  }));
+  // Het bestaande nummer blijft van jou; de tekst wijst naar iets dat bestaat.
+  const inhoud = db.prepare("SELECT content FROM posts WHERE id = 'pb'").get().content;
+  const m = /\[\[track:([^\]]+)\]\]/.exec(inhoud);
+  assert.ok(m);
+  assert.ok(db.prepare('SELECT 1 FROM audio_tracks WHERE id = ? AND site_id = ?').get(m[1], 's1'),
+    'wat er ook gebeurde met het id, de verwijzing wijst naar een bestaand nummer');
 });