Changeset 75f2897 in Klonkt for test/thread.test.js

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

File:

• test/thread.test.js (modified) (4 diffs)

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');
+});