Changeset 75f2897 in Klonkt for test

Timestamp:

08/10/2026 07:33:52 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

aae5881

Parents:

0db3c72

Message:

Het dialect eruit waar de standaard het al draagt

shaer-nmw, tweede helft. shaer:author, shaer:booster, shaer:quote en
shaer:embed gaan van alle C2S-legs af: outbox-vriendenleg, tijdlijn,
berichten, antwoorden, eigen verzonden notes, de composer-kaart en de thread.
De standaardvormen uit de vorige commit dragen ze nu alleen.

De thread-leg had de standaardvorm nog niet en kreeg hem hier: de byline van
een antwoord zit in de ingesloten attributedTo, en de emoji-poort knipt in de
tag van die actor in plaats van in een eigen emoji-kaart ernaast.

Twee dingen die stil hadden kunnen breken:

• filterThreadToCircle vergeleek n.attributedTo als string. Nu dat een object is zou hij ALLES wegfilteren: een ward met een lege thread en nergens een foutmelding. Loopt nu door actorUriOf.
• de composer-kaart las op beide clients alleen de oude velden; die had ik in ronde een gemist. De preview zou leeg zijn gebleven.

Wat blijft: shaer:liked en shaer:boosted (per-lezer-interactiestatus, AS2 heeft
er niets voor) en de FEP-633c-termen, die bewust van ons zijn.

Nieuwe wachten: de C2S-leg door de echte route op afwezig dialect -- op de
SLEUTEL, want 'shaer:quote' zit ook in shaer:quoteCards en dat is een poort die
blijft -- en de thread-route, die tot nu toe geen route-dekking had.

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

Location:

test

Files:

• c2s-messages.test.js (modified) (1 diff)
• own-card.test.js (modified) (1 diff)
• standard-shapes.test.js (modified) (1 diff)
• thread.test.js (modified) (4 diffs)

test/c2s-messages.test.js

@@ -94 +94 @@
   assert.ok((wave.tag || []).some((x) => x.type === 'Mention' && x.href === 'https://test.example/ap/users/kid'),
     'with a Mention the client recognises itself in');
-  assert.equal(wave['shaer:author'].name, 'Oma', 'and a byline to show');
+  assert.equal(wave.attributedTo.name, 'Oma', 'and a byline to show');
   assert.ok(!/@kid@test\.example/.test(wave.content), 'the leading @mention is stripped, like Berichten on the web');
 

test/own-card.test.js

@@ -120 +120 @@
   globalThis.fetch = vorige;
   assert.equal(uit.notes.length, 1);
-  const auteur = uit.notes[0]['shaer:author'];
+  const auteur = uit.notes[0].attributedTo;
   assert.equal(auteur.url, `${HOST}/`, 'de html-Link, niet de array gestringd');
   assert.ok(!String(auteur.url).includes('object'), 'nergens [object Object]');

test/standard-shapes.test.js

@@ -83 +83 @@
   assert.equal(AP.quoteObject('rommel'), undefined);
 });
+
+// ── En dan het weghalen (shaer-nmw, tweede helft) ────────────────────
+//
+// Bewijzen dat de standaardvorm ERBIJ staat is de helft; de andere helft is dat
+// het dialect er niet meer NAAST staat. Zonder deze wacht sluipt een
+// shaer:author zo weer een nieuwe leg in en spreken we stilletjes weer onze
+// eigen taal. Daarom door de echte route, want daar wordt de vorm beslist.
+test('de C2S-leg spreekt geen dialect meer waar de standaard het regelt', async (t) => {
+  const crypto = await import('crypto');
+  const express = (await import('express')).default;
+  const db = (await import('../src/config/database.js')).default;
+  const routes = (await import('../src/routes/activitypub.js')).default;
+
+  db.prepare('INSERT INTO users (id, username, email, password_hash, role) VALUES (?,?,?,?,?)')
+    .run('u1', 'u1', 'u1@t', 'x', 'god');
+  db.prepare('INSERT INTO sites (id, slug, title, owner_id, is_primary) VALUES (?,?,?,?,1)')
+    .run('s1', 'kind', 'kind', 'u1');
+  db.prepare(`INSERT INTO ap_timeline (id, slug, author_uri, author_name, author_handle, author_icon,
+              reblog_name, reblog_handle, content)
+              VALUES ('https://elders/n1', 'kind', 'https://elders/u/til', 'Tante Til', '@til@elders',
+                      'https://elders/til.png', 'Esmee', '@es@elders', '<p>hoi</p>')`).run();
+
+  const bearer = 'test-token-' + 'b'.repeat(24);
+  db.prepare('INSERT INTO oauth_tokens (token_hash, client_id, user_id, site_slug, scope) VALUES (?,?,?,?,?)')
+    .run(crypto.createHash('sha256').update(bearer).digest('base64url'), 'c', 'u1', 'kind', 'read write');
+
+  const app = express();
+  app.use(routes);
+  const server = app.listen(0);
+  t.after(() => server.close());
+  await new Promise((r) => server.once('listening', r));
+  const doc = await (await fetch(`http://127.0.0.1:${server.address().port}/ap/users/kind/inbox`,
+    { headers: { Authorization: `Bearer ${bearer}` } })).json();
+
+  const tekst = JSON.stringify(doc);
+  // Op de SLEUTEL, niet op een stuk tekst: 'shaer:quote' zit ook in
+  // shaer:quoteCards, en dat is een poortnaam die blijft. Een te grove zeef
+  // zou hier rood staan om iets dat klopt -- of erger, groen om iets dat niet
+  // klopt.
+  for (const weg of ['shaer:author', 'shaer:booster', 'shaer:quote', 'shaer:embed']) {
+    assert.ok(!tekst.includes(`"${weg}":`), `${weg} hoort weg te zijn`);
+  }
+  // Wat WEL mag blijven, en waarom: per-lezer-interactiestatus heeft in AS2
+  // geen tegenhanger (Mastodon lost het in zijn eigen REST-API op), en de
+  // FEP-633c-termen zijn bewust van ons en in standaardisatie.
+  assert.ok(tekst.includes('shaer:liked'), 'interactiestatus blijft, die heeft geen standaard');
+
+  const item = doc.orderedItems.find((i) => i.object.id === 'https://elders/n1');
+  assert.equal(item.type, 'Announce', 'een boost is de wrapper, niet een zijkanaal');
+  assert.equal(item.object.attributedTo.name, 'Tante Til', 'en de byline zit in attributedTo');
+  assert.equal(item.object.attributedTo.preferredUsername, 'til');
+});

test/thread.test.js

@@ -52 +52 @@
 ];
 
+// De echte fetch, voordat de stub hem overneemt: de route-test onderaan doet
+// een ECHT verzoek aan een testserver op localhost, en dat mag de stub niet
+// beantwoorden met zijn 404.
+const echteFetch = globalThis.fetch;
+
 globalThis.fetch = async (url) => {
   const u = String(url);
@@ -69 +74 @@
   assert.equal(uit.notes.length, 2);
   // Oudste eerst: een gesprek lees je van boven naar beneden.
-  assert.equal(uit.notes[0]['shaer:author'].name, 'tante');
-  assert.equal(uit.notes[1]['shaer:author'].name, 'vreemde');
+  assert.equal(uit.notes[0].attributedTo.name, 'tante');
+  assert.equal(uit.notes[1].attributedTo.name, 'vreemde');
   // De sanitizer heeft het script eruit gehaald, de tekst mag blijven.
   assert.ok(!uit.notes[1].content.includes('<script'), 'script weggeschoond');
@@ -96 +101 @@
   const kring = AP.filterThreadToCircle('pupil', uit.notes);
   assert.equal(kring.notes.length, 1);
-  assert.equal(kring.notes[0]['shaer:author'].name, 'tante');
+  // Ook de zeef zelf leest de ingesloten actor: een kale vergelijking op
+  // attributedTo zou hier stil ALLES wegfilteren.
+  assert.equal(kring.notes[0].attributedTo.name, 'tante');
   // De vreemde is er, en dat mag gezegd: geteld, niet stil weggelaten.
   assert.equal(kring.hidden, 1);
@@ -143 +150 @@
   globalThis.fetch = vorige;
 });
+
+// ── Door de route heen, want daar wordt de vorm beslist ──────────────
+//
+// De byline van een antwoord verhuisde naar de ingesloten attributedTo
+// (shaer-nmw). Deze leg had geen route-dekking, en juist hier zit de
+// emoji-poort die IN die byline knipt -- knipt hij in het oude veld, dan
+// gebeurt er stil niets meer.
+test('de thread-route geeft de byline in attributedTo, en de emoji-poort knipt daarin', async (t) => {
+  const crypto = await import('crypto');
+  const express = (await import('express')).default;
+  const routes = (await import('../src/routes/activitypub.js')).default;
+
+  const bearer = 'test-token-' + 'c'.repeat(24);
+  db.prepare('INSERT INTO oauth_tokens (token_hash, client_id, user_id, site_slug, scope) VALUES (?,?,?,?,?)')
+    .run(crypto.createHash('sha256').update(bearer).digest('base64url'), 'c', 'u1', 'kind', 'read write');
+
+  const app = express();
+  app.use(routes);
+  const server = app.listen(0);
+  t.after(() => server.close());
+  await new Promise((r) => server.once('listening', r));
+  const haal = async () => (await (await echteFetch(
+    `http://127.0.0.1:${server.address().port}/ap/users/kind/thread?object=${encodeURIComponent(BRON)}`,
+    { headers: { Authorization: `Bearer ${bearer}` } })).json());
+
+  const doc = await haal();
+  assert.ok(!JSON.stringify(doc).includes('"shaer:author":'), 'geen dialect meer op deze leg');
+  const vanTante = doc.orderedItems.find((n) => n.id.endsWith('/replies/2'));
+  assert.equal(vanTante.attributedTo.name, 'tante', 'de byline zit in attributedTo');
+  assert.equal(vanTante.attributedTo.preferredUsername, 'tante');
+  assert.ok((vanTante.tag || []).some((x) => x.type === 'Emoji'), 'open poort: de emoji van het antwoord blijft');
+
+  // Poort dicht: FEP-9098 zit in de tag van de ingesloten actor, dus daar
+  // hoort geknipt te worden -- en het antwoord zelf houdt zijn eigen emoji
+  // niet meer over.
+  db.prepare("UPDATE sites SET gate_custom_emoji = 0 WHERE slug = 'kind'").run();
+  db.prepare("INSERT INTO ap_guardianships (slug, other_uri, role, status) VALUES ('kind', ?, 'guardian', 'accepted')")
+    .run('https://203.0.113.20/users/tante');
+  const dicht = await haal();
+  const na = dicht.orderedItems.find((n) => n.id.endsWith('/replies/2'));
+  assert.ok(!(na.tag || []).some((x) => x.type === 'Emoji'), 'dichte poort: geen emoji meer');
+  assert.ok(!na.attributedTo.tag, 'ook niet in de byline');
+});