Changeset bfadc15 in Klonkt for test

Timestamp:

08/14/2026 03:19:09 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

a5515b4

Parents:

7e474f7

Message:

De teksten op de migrate-pagina in mensentaal

Robin: er zijn miljoenen Klonkt-gebruikers, dus het moet gebruiksvriendelijk
worden gemaakt. Ik had die opmerking eerst gelezen als een schaalwaarschuwing
en zat prestaties te repareren. Dat leverde wel een echte fout op (7e474f7),
maar het was zijn vraag niet.

Wat er stond was voor mij geschreven, niet voor wie verhuist:

claimen, alias, actor-adres, instantie, AP-ids, fediverse,
"een bericht met dezelfde webadres-naam maar een ander id"

Achttien teksten herschreven in nl/en/de. "Je oude account claimen" werd "je
vorige account koppelen". De overschrijf-waarschuwing gaat niet meer over
unieke sleutels maar over wat je kwijtraakt. De foutmelding bij een ontbrekende
koppeling verwijst nu naar stap 1 op het scherm in plaats van naar een begrip.

DE VOORBEELDEN TONEN HANDLES. Er stond ​https://oude-klonkt.nl/ap/users/jij als
placeholder. Niemand kent zijn eigen actor-URL; mensen kennen
@robo@….

EN HET VELD GEEFT TERUG WAT JE GAF. Je typte een handle, wij slaan de actor-URL
op omdat het protocol daarop draait, en dan kreeg je een adres terug dat je
nooit had ingetypt. Nu tonen we de handle, en alleen als er echt een uit te
halen valt: een verkeerde handle is erger dan een lelijke URL. Opslaan leest
beide vormen, dus daar verandert niets.

Changed files:
src/services/i18n.js

• 19 teksten herschreven in drie talen, jargon eruit

src/views/pages/admin-migrate.ejs

• placeholders tonen @handle in plaats van een actor-URL

src/routes/admin-migrate.js

• aliasTekst() toont handles, met de URL als terugval

test/migrate-page.test.js

• een jargonwacht over 23 sleutels maal drie talen
• de handle-heenweg en -terugweg
• de stap-5-test toetst nu op BETEKENIS; hij brak omdat het woord "alias" verdween, en dat was precies de bedoeling

remarks: de indeling blijft zoals hij was (Robins keuze). Het grotere idee, een
vraag bovenaan of je hier binnenkomt of weggaat en dan alleen die helft tonen,
is niet gedaan. Suite 1016 groen.

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

File:

• test/migrate-page.test.js (modified) (2 diffs)

test/migrate-page.test.js

@@ -12 +12 @@
 import assert from 'node:assert/strict';
 import fs from 'fs';
+const i18nMod = await import('../src/services/i18n.js');
 
 process.env.DATABASE_PATH = ':memory:';
@@ -107 +108 @@
   // beschrijft die niet bestaat is erger dan geen stappenplan.
   const i18n = await import('../src/services/i18n.js');
+  // Op BETEKENIS toetsen, niet op een woord. Deze test brak toen "alias" uit de
+  // teksten verdween, en dat is precies de verkeerde reden om rood te worden:
+  // het jargon weghalen was de bedoeling.
   const nl = i18n.t('nl', 'mig.plan_5_why');
-  assert.match(nl, /verwijder/i, 'de Delete-ronde');
-  assert.match(nl, /alias/i, 'en de alias weghalen');
+  assert.match(nl, /verwijder/i, 'de berichten opruimen');
+  assert.match(nl, /stap 1/i, 'en de koppeling met je vorige account weghalen');
   assert.match(nl, /nog geen knop/i, 'en eerlijk dat het handwerk is');
 });
+
+test('de teksten praten geen jargon', () => {
+  // Robin: er zijn miljoenen Klonkt-gebruikers, dus het moet gebruiksvriendelijk.
+  // Woorden als alias, actor, instantie en fediverse zeggen een ontwikkelaar
+  // alles en een gebruiker niets. Deze test bewaakt dat ze niet terugsluipen.
+  const jargon = /\b(alias(sen)?|claimen|actor|instantie|AP-id|origin|slug|URI)\b/i;
+  const sleutels = [
+    'lead', 'alias_title', 'alias_hint', 'alias_label', 'alias_note', 'alias_btn',
+    'import_hint', 'file_label', 'overwrite_label', 'overwrite_hint', 'r_new_ids',
+    'pull_hint', 'pull_source', 'pull_source_hint', 'move_hint', 'move_warn',
+    'plan_hint', 'plan_1', 'plan_1_why', 'plan_5_why',
+    'e_no_backreference', 'e_not_moved_here', 'e_no_source',
+  ];
+  const fout = [];
+  for (const l of ['nl', 'en', 'de']) {
+    for (const k of sleutels) {
+      const t = i18nMod.t(l, `mig.${k}`);
+      const m = jargon.exec(t);
+      if (m) fout.push(`${l}/${k}: "${m[0]}"`);
+    }
+  }
+  assert.deepEqual(fout, [], `jargon teruggeslopen: ${fout.join(', ')}`);
+});
+
+test('een gekoppeld account komt terug als handle, niet als URL', async () => {
+  // Je typt @jij@mastodon.social en wij slaan de actor-URL op, want daar draait
+  // het protocol op. Zonder deze vertaling terug kreeg je een adres te zien dat
+  // je nooit hebt ingetypt en niet herkent.
+  const AP = await import('../src/services/ActivityPubService.js');
+  assert.equal(AP.deriveHandle('https://oud.example/ap/users/robo'), '@robo@oud.example');
+  // En als er geen fatsoenlijke handle uit te halen valt blijft de URL staan:
+  // een verkeerde handle is erger dan een lelijke URL.
+  const raar = AP.deriveHandle('https://oud.example/');
+  assert.ok(!/^@[^@\s]+@[^@\s]+$/.test(raar) || raar === '@oud.example@oud.example',
+    `onverwachte handle uit een pad zonder naam: ${raar}`);
+});