Changeset 6259305 in Klonkt

Timestamp:

08/09/2026 08:46:28 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

e62eee4

Parents:

5ae4d61

git-author:

Robin <roboburr@…> (08/09/2026 08:46:10 AM)

git-committer:

Robin <roboburr@…> (08/09/2026 08:46:28 AM)

Message:

Eén weg naar de hulpvragen: het paneel had een eigen afkap (shaer-6wt, staart)

Gevonden bij de limietinventarisatie na Barts 429-jacht. De fix van
vanochtend haalt de afkap van OPEN hulpvragen weg in helpItemsFor -- maar
de guardian-PWA had een eigen kopie van die query staan, mét LIMIT 50.
Precies de gebruiker uit die bead (de jeugdzorgmedewerker met een
caseload) zag in het PANEEL dus nog steeds open vragen wegvallen zodra er
meer dan vijftig waren.

Twee wegen naar dezelfde staat, en de fix nam er maar één -- hetzelfde
patroon dat eerder bij de reply-gate misging.

De reden dat die kopie bestond was echt: het paneel toont kaarten en had
note_url, media, quote en emoji nodig, die de queue-query niet selecteerde.
Dus draagt de bron ze nu; de queue leest ze simpelweg niet, en er is geen
aanleiding meer om opnieuw te splitsen.

Drie toetsen met een caseload van 80: alle open vragen komen door, de
bron draagt de kaartvelden, en de geschiedenis mag nog steeds afgekapt
worden. De testopstelling ving onderweg zijn eigen les: zonder echte
ward-relaties telt elke vraag als van een oud-ward en dus niet-open --
withWardship doet dat met opzet, mijn eerste opzet toetste per ongeluk
dat pad. Alle 785 groen.

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

Files:

• src/routes/guardian.js (modified) (1 diff)
• src/services/guardianship/queues.js (modified) (1 diff)
• test/help-one-source.test.js (added)

src/routes/guardian.js

@@ -82 +82 @@
   const base = (process.env.PUBLIC_BASE_URL || '').replace(/\/+$/, '');
   const me = AP.actorId(base, site.slug);
-  const help = db.prepare(
-    `SELECT object_uri, note_url, actor_uri, actor_name, actor_handle, actor_icon, content, published, created_at,
-            emoji_json, actor_emoji_json, media_json, quote_json, embed_json
-     FROM ap_mentions WHERE slug = ? AND help_request = 1 ORDER BY created_at DESC LIMIT 50`
-  ).all(site.slug);
-  // De gedeelde staat in EEN query (shaer-lgo): wie er al op af is en of het is
-  // afgesloten. Per kaart vragen zou hier een N+1 opleveren, en dit is precies
-  // het scherm dat een guardian in een haast openslaat.
-  const helpStaat = Guardianship.help.statusFor(help.map((h) => h.object_uri));
-  // Wie bewaak je NU nog? Een hulpvraag van een oud-ward is niet meer van jou en
-  // hoort niet in de lijst die om je aandacht vraagt te blijven staan.
-  const mijnWards = new Set(Guardianship.listWards(site.slug).map((w) => w.other_uri));
-  const helpItems = help.map((h) => ({
+  // EEN weg naar de hulpvragen (Barts 429-jacht, 9-8): dit scherm had een
+  // eigen kopie van de queue-query, met een afkap op 50 -- dus de fix die open
+  // vragen nooit meer afkapt (shaer-6wt) ging aan het paneel voorbij, en juist
+  // de guardian met een caseload zag oude open vragen wegvallen. Nu dezelfde
+  // bron als de apps: open vragen volledig, geschiedenis afgekapt.
+  const helpItems = Guardianship.queues.helpItemsFor(site.slug).map((h) => ({
     ...h,
-    // Bij twijfel OPEN. Een hulpvraag die er afgehandeld uitziet terwijl hij dat
-    // niet is, is de gevaarlijke fout -- niet andersom.
-    state: Guardianship.help.withWardship(
-      helpStaat.get(h.object_uri) || { open: true, pickedUpBy: [], handled: null, oldestPickupAt: null },
-      mijnWards.has(h.actor_uri),
-    ),
     // The dashboard is built in the browser, so it gets the body finished: the
     // same partial de Krant and Berichten use. A 🛟 often carries a screenshot

src/services/guardianship/queues.js

@@ -214 +214 @@
   let rijen = [];
   try {
+    // Alle velden die een kaart kan tonen, niet alleen die van de queue: de
+    // PWA had hierom een EIGEN kopie van deze query -- mét een afkap op 50,
+    // waardoor de fix hierboven aan het paneel voorbijging (Barts 429-jacht,
+    // 9-8). Een tweede weg naar dezelfde staat is precies wat er bij de
+    // reply-gate al misging; nu is dit de enige weg, en dan hoort hij ook te
+    // dragen wat een kaart nodig heeft. De extra kolommen kosten de queue
+    // niets: die leest ze gewoon niet.
     rijen = db.prepare(
-      `SELECT object_uri, actor_uri, actor_name, actor_handle, actor_icon, content, published, created_at
+      `SELECT object_uri, note_url, actor_uri, actor_name, actor_handle, actor_icon, content, published, created_at,
+              emoji_json, actor_emoji_json, media_json, quote_json, embed_json
        FROM ap_mentions WHERE slug = ? AND help_request = 1 ORDER BY created_at DESC`,
     ).all(slug);