Changeset c34ef26 in Klonkt

Timestamp:

08/10/2026 01:26:03 PM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

f549cf5

Parents:

59d963f

Message:

De ouderdom van een oppak haalde de apps niet (shaer-lgo)

Barts besluit was dat "opgepikt" niet vervalt maar zichtbaar VEROUDERT -- het
verschil tussen "er is iemand mee bezig" en "er was ooit iemand mee bezig". Het
guardian-paneel toont dat ook, met een regel zodra het oudste oppakken langer
dan een uur geleden is.

Maar helpCollection gooide het veld weg. helpStatus() rekent oldestPickupAt
netjes uit; de C2S-wachtrij serialiseerde alleen open, handledBy, handledAt,
pickedUpBy en formerWard. In Shaer zag een oppak van vijf minuten er dus uit als
een van vijf dagen -- en de faalstand is hier nou juist "iedereen denkt dat het
geregeld is".

Als TIJDSTIP en niet als leeftijd, om dezelfde reden die in guardian.js staat:
een leeftijd maakt elk antwoord anders en dan kan de ETag nooit gelijk zijn. De
client rekent zelf terug.

En weg zodra hij is afgehandeld: hoe lang er iemand naar keek is dan geen
informatie meer, en zou als "er wacht nog iets" lezen.

Gecontroleerd dat de test bijt: veld eruit -> 7 groen 1 rood, terug -> 8 groen.

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

Files:

• src/services/guardianship/queues.js (modified) (1 diff)
• test/help-c2s.test.js (modified) (2 diffs)

src/services/guardianship/queues.js

@@ -294 +294 @@
     'shaer:handledAt': h.state.handled ? h.state.handled.at : undefined,
     'shaer:pickedUpBy': h.state.pickedUpBy.map((p) => p.handle || p.uri),
+    // HOE OUD het oudste oppakken is (shaer-lgo). Barts besluit was dat
+    // "opgepikt" niet vervalt maar zichtbaar VEROUDERT -- het verschil tussen
+    // "er is iemand mee bezig" en "er was ooit iemand mee bezig". Het paneel
+    // toonde dat al; de apps konden het niet, want dit veld bleef hier liggen.
+    // Een oppak van vijf minuten zag er daar uit als een van vijf dagen, en de
+    // faalstand is hier nou juist "iedereen denkt dat het geregeld is".
+    //
+    // Als TIJDSTIP en niet als leeftijd: een leeftijd maakt elk antwoord anders
+    // en dan kan de ETag nooit gelijk zijn. De client rekent zelf terug, precies
+    // zoals guardian.js het doet.
+    'shaer:oldestPickupAt': h.state.oldestPickupAt || undefined,
     'shaer:formerWard': h.state.formerWard || undefined,
   }));

test/help-c2s.test.js

@@ -60 +60 @@
 });
 
+test('en de apps krijgen te horen HOE OUD dat oppakken is', () => {
+  // Barts besluit: opgepikt vervalt niet maar veroudert zichtbaar -- het
+  // verschil tussen "er is iemand mee bezig" en "er was ooit iemand mee bezig".
+  // Het paneel toonde dat al uit dezelfde staat; de apps kregen het veld niet,
+  // dus daar zag een oppak van vijf minuten eruit als een van vijf dagen.
+  const coll = queues.helpCollection('https://x/queues/help', 'oma');
+  const item = coll.orderedItems.find((i) => i.id === BOEI);
+  assert.ok(item['shaer:oldestPickupAt'], 'het veld staat er');
+  assert.ok(!Number.isNaN(Date.parse(item['shaer:oldestPickupAt'])), 'en is een leesbaar tijdstip');
+  // Een TIJDSTIP en geen leeftijd: een leeftijd maakt elk antwoord anders en
+  // dan kan de ETag nooit gelijk zijn. De client rekent zelf terug.
+  assert.equal(typeof item['shaer:oldestPickupAt'], 'string');
+});
+
 test('afsluiten vanuit de app sluit hem ook echt', async () => {
   await AP.ingestOutboxActivity(site(), user, marker('handled'));
@@ -75 +89 @@
   assert.ok(item['shaer:handledBy']);
   assert.equal(item['shaer:helpRequest'], true);
+  // En de ouderdom is WEG zodra hij is afgehandeld: hoe lang er iemand naar
+  // keek is dan geen informatie meer, en zou als "er wacht nog iets" lezen.
+  assert.equal(item['shaer:oldestPickupAt'], undefined);
 });