Changeset c4b24b3 in Klonkt

Timestamp:

08/08/2026 01:08:33 AM (5 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

9c9a3d9

Parents:

4f97b70

Message:

De thread van een vreemde post: ook de next-pagina lezen

Barts verduidelijking (8-8) wees de fout aan: het ging om reacties op een
VREEMDE post, niet een eigen. Mijn Waydroid-test draaide op een eigen
post, en die gaat door de lokale kortsluiting -- orderedItems meteen vol,
niets te wandelen. Vandaar dat de fout niet opviel.

Mastodon serveert de replies-collectie met een first die een INLINE
pagina is met lege items en een next waar de antwoorden echt staan.
Wie alleen de eerste pagina leest ziet op elke Mastodon-post een leeg
gesprek. getThread wandelt nu tot drie pagina's, met behoud van inline
objecten (die hoeven niet opnieuw opgehaald) -- dezelfde kneep die
collectReplyItems voor de eigen-post-crawler al kende.

Test erbij die de Mastodon-vorm exact nabootst; alle 624 groen.

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

Files:

• .beads/.auto-import-issues.jsonl (added)
• src/services/ActivityPubService.js (modified) (1 diff)
• test/thread.test.js (modified) (1 diff)

src/services/ActivityPubService.js

@@ -3407 +3407 @@
       : (repliesRef.id ? await signedGetJson(slug, repliesRef.id) : repliesRef);
   }
-  // ÉÉN pagina, met opzet: de eerste. Wie meer wil moet eerst kunnen zeggen
-  // waarom dertig directe antwoorden niet genoeg context is voor een gesprek.
-  let items = collectionItems(coll);
-  if (!items.length && coll && coll.first) {
-    const first = typeof coll.first === 'string' ? await signedGetJson(slug, coll.first) : coll.first;
-    items = collectionItems(first);
+  // De pagina-wandeling van collectReplyItems, maar met behoud van INLINE
+  // objecten (die niet opnieuw opgehaald hoeven). Niet "de eerste pagina":
+  // Mastodon serveert `first` als inline-pagina met LEGE items en een `next`
+  // waar de antwoorden echt staan -- wie alleen de eerste pagina leest, ziet
+  // op elke Mastodon-post een leeg gesprek. Dat was precies Barts melding
+  // (8-8, reacties op een vreemde post). Eigen posts maskeerden het: die
+  // gaan door de lokale kortsluiting en hebben orderedItems meteen vol.
+  let items = [];
+  let node = coll;
+  if (node && node.first && !collectionItems(node).length) {
+    node = typeof node.first === 'string' ? await signedGetJson(slug, node.first) : node.first;
+  }
+  let pages = 0;
+  while (node && pages++ < 3 && items.length < THREAD_VIEW_LIMIT) {
+    items.push(...collectionItems(node));
+    if (!node.next) break;
+    node = typeof node.next === 'string' ? await signedGetJson(slug, node.next) : node.next;
   }
   items = items.slice(0, THREAD_VIEW_LIMIT);

test/thread.test.js

@@ -91 +91 @@
 });
 
+test('de Mastodon-vorm: first is een lege inline-pagina, de antwoorden staan op next', async () => {
+  // Precies wat Mastodon serveert en wat Barts melding (8-8) verklaarde: wie
+  // alleen de eerste pagina leest, ziet op elke Mastodon-post een leeg gesprek.
+  const MPOST = 'https://203.0.113.50/notes/masto-1';
+  const vorige = globalThis.fetch;
+  globalThis.fetch = async (url) => {
+    const u = String(url);
+    const json = (o) => new Response(JSON.stringify(o), { status: 200, headers: { 'content-type': 'application/activity+json' } });
+    if (u === MPOST) return json({
+      id: MPOST, type: 'Note', attributedTo: TANTE, content: '<p>toot</p>',
+      replies: {
+        id: `${MPOST}/replies`, type: 'Collection',
+        first: { type: 'CollectionPage', items: [], next: `${MPOST}/replies?page=true` },
+      },
+    });
+    if (u === `${MPOST}/replies?page=true`) return json({
+      type: 'CollectionPage',
+      items: [{ id: `${MPOST}/r1`, type: 'Note', attributedTo: TANTE, inReplyTo: MPOST, content: '<p>eerste echte antwoord</p>', published: '2026-08-02T10:00:00Z' }],
+    });
+    return vorige(url);
+  };
+  const uit = await AP.getThread('kind', MPOST, { isWard: false });
+  globalThis.fetch = vorige;
+  assert.equal(uit.notes.length, 1, 'het antwoord op de next-pagina is gevonden');
+  assert.ok(uit.notes[0].content.includes('eerste echte antwoord'));
+});
+
 test('de tweede lezing komt uit het geheugen, niet van het netwerk', async () => {
   let calls = 0;