Changeset bac4bf3 in Klonkt for src/services/ActivityPubService.js

Timestamp:

08/13/2026 11:55:04 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

783b9ff

Parents:

7e9d0ea

git-author:

Robin <roboburr@…> (08/13/2026 11:54:36 AM)

git-committer:

Robin <roboburr@…> (08/13/2026 11:55:04 AM)

Message:

Paginering die echt pagineert (shaer-sk4)

Robin zag dat elke ?page= dezelfde inhoud gaf. Dat klopte, en het was geen halve
implementatie maar een omhulsel: pagedCollection veranderde alleen de VORM en
sneed nooit. first en last wezen allebei naar ?page=1, en de routes lazen
!!req.query.page -- of de parameter er STAAT, niet welke. Live gaf ?page=2 en
?page=99 dezelfde acht items, en noemden zichzelf pagina 1.

Nu: echt snijden, met next en prev, en een pagina die zijn eigen nummer
draagt. Een pagina voorbij het einde is LEEG en zegt dat ook -- hem naar de
laatste terugbuigen zou opnieuw een antwoord zijn dat over zichzelf liegt.

DE WORTEL HOUDT ZIJN ITEMS INLINE, en dat is geen slordigheid maar de reden dat
dit veilig is. Shaer leest een document en volgt next niet (KlonktClient.swift
orderedItems). Werd de wortel nu leeg, dan kreeg elke draaiende app nul items en
geen foutmelding -- dezelfde stille val waar ik op 10 augustus bij Funkwhale zelf
in trapte. Eerst de clients leren pagineren, dan pas de wortel afslanken.

WAT WEL EN NIET GEPAGINEERD IS. Volgers en following pagineren nu volledig: die
lijsten zitten al in het geheugen, dus dat kost niets extra. De outbox krijgt de
juiste VORM maar niet meer diepte: de route kapt al op twintig rijen in SQL.
Echt doorbladeren vraagt daar een LIMIT/OFFSET, en dat is lastiger dan bij
volgers omdat posts en tracks op datum door elkaar gevlochten worden en uit twee
tabellen komen -- een UNION met een offset erover, geen tweede slice. Dat staat
als naad in de code en op de bead, niet weggemoffeld.

Zeven tests, waaronder de klacht zelf: pagina 2 geeft ANDERE items dan pagina 1.

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

File:

• src/services/ActivityPubService.js (modified) (2 diffs)

src/services/ActivityPubService.js

@@ -1032 +1032 @@
     .sort((a, b) => wanneer(b) - wanneer(a))
     .slice(0, MAX_OUTBOX);
+  // WAT HIER NOG NIET GEPAGINEERD IS, en dat hoort genoemd (shaer-sk4): deze
+  // lijst is al door de route op twintig rijen afgekapt, dus pagina 2 is leeg.
+  // Echt doorbladeren vraagt een LIMIT/OFFSET in SQL -- en dat is hier lastiger
+  // dan bij volgers, want posts en tracks worden op DATUM door elkaar gevlochten
+  // en komen uit twee tabellen. Dat vraagt een UNION met een offset erover, geen
+  // tweede slice. De vorm klopt nu wel: pagina 2 zegt eerlijk dat hij leeg is en
+  // biedt geen `next` aan, in plaats van pagina 1 nog eens te geven.
   // GEPAGINEERD, ook al past alles op een pagina (Funkwhale, 11-8).
   //
@@ -1049 +1056 @@
 // account owner (a C2S bearer scoped to this site) gets the real actor URIs via
 // `items`, so their own client can build a friends list.
-export function buildFollowers(base, site, count, items = null) {
+export function buildFollowers(base, site, count, items = null, { page = false } = {}) {
   const id = `${actorId(base, site.slug)}/followers`;
   // count-only for the public; full for the owner
-  return pagedCollection(id, items || [], { totalItems: items ? items.length : (count || 0) });
+  return pagedCollection(id, items || [], { totalItems: items ? items.length : (count || 0), page });
 }
 
 // The accounts this site follows — count only, mirroring buildFollowers. The spec lists
 // `following` as a standard actor property; Hubzilla/Friendica + crawlers expect it.
-export function buildFollowing(base, site, count, items = null) {
+export function buildFollowing(base, site, count, items = null, { page = false } = {}) {
   const id = `${actorId(base, site.slug)}/following`;
   // count-only for the public; full for the owner
-  return pagedCollection(id, items || [], { totalItems: items ? items.length : (count || 0) });
+  return pagedCollection(id, items || [], { totalItems: items ? items.length : (count || 0), page });
 }