Changeset ea211b9 in Klonkt for src/services/guardianship/gated.js

Timestamp:

08/08/2026 08:42:09 PM (4 weeks ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

29eb745

Parents:

3a0ca0f

git-author:

Robin <roboburr@…> (08/08/2026 08:41:47 PM)

git-committer:

roboburr <roboburr@…> (08/08/2026 08:42:09 PM)

Message:

Je hoort te weten dat je de doorslag geeft (shaer-8vt)

De telling is een race naar de drempel: zodra het aantal gehaald is, is het
besluit gevallen. Bij 2 van 3 is de tweede ja meteen de beslissing -- en sinds
Barts meerderheidsbesluit van vandaag is bij een volgverzoek met twee guardians
de EERSTE ja dat al. Wie antwoordde wist dat niet, en het scherm zei het nergens.

Dat kon niet, en waarom niet was de vondst bij shaer-nf9: gatedProgress werkt op
een LOKALE slug en er zijn geen lokale wards. De telling loopt op de server van
het kind, en die stuurde hem niet mee. Nu wel: shaer:decisive reist mee met de
doorgestuurde Offer, voor gate-voorstellen en voor volgverzoeken in beide
richtingen.

EEN JA/NEE, GEEN TELLING, en dat is een besluit. Een getal ("1 van 2") reist mee,
veroudert onderweg en leest daarna als een feit; de beschikbare set schuift met
3.6 bovendien mee. En hoeveel guardians een kind heeft, en wie er al gestemd
heeft, is niet vanzelf iets dat elke mede-guardian hoort te zien. Een
waarschuwing veroudert ook, maar hij CLAIMT niets -- en dat scheelt.

BIJ TWIJFEL WAARSCHUWEN. Ontbreekt het veld (een oudere server), dan zeggen we
dat je beslist. De twee fouten zijn niet gelijk: zeggen dat je beslist terwijl
dat niet zo is maakt iemand voorzichtiger dan nodig; niets zeggen terwijl hij wel
beslist laat hem het onwetend doen. Daar staat een toets op, en de mutatie die
ertoe doet -- onbekend als "je beslist niets" lezen -- maakt hem rood.

Eerlijk over de mutatietest: mijn eerste poging (de Number.isFinite-guards
weghalen) gaf nul fouten, want die geven voor undefined dezelfde uitkomst. De
regel bijt wel tegen de realistische verkeerde versie.

Zes toetsen, drie talen. Suite 747/747.

File:

• src/services/guardianship/gated.js (modified) (4 diffs)

src/services/guardianship/gated.js

@@ -260 +260 @@
 }
 
+/**
+ * Zou het antwoord van deze guardian het besluit AFMAKEN (shaer-8vt)?
+ *
+ * De telling is een race naar de drempel: zodra het aantal gehaald is, is het
+ * gevallen. Bij 2 van 3 is de tweede ja dus meteen de beslissing, en bij een
+ * volgverzoek met twee guardians is de EERSTE ja dat al. Wie antwoordt weet dat
+ * niet, en het scherm zei het nergens.
+ *
+ * EEN JA/NEE, GEEN TELLING, en dat is een besluit. Een getal ("1 van 2") reist
+ * mee, veroudert onderweg en leest daarna als een feit; de beschikbare set
+ * schuift bovendien met 3.6 mee. En hoeveel guardians een kind heeft, en wie er
+ * al gestemd heeft, is niet vanzelf iets dat elke mede-guardian hoort te zien.
+ * Een waarschuwing veroudert ook, maar hij CLAIMT niets -- en dat scheelt.
+ *
+ * BIJ TWIJFEL WAARSCHUWEN. De twee fouten zijn niet gelijk: zeggen dat je
+ * beslist terwijl dat niet zo is maakt iemand voorzichtiger dan nodig; niets
+ * zeggen terwijl hij wel beslist laat hem het onwetend doen.
+ */
+export function isDecisive(votes, need) {
+  const v = Number.isFinite(votes) ? votes : 0;
+  const n = Number.isFinite(need) ? need : 1;
+  return (n - v) <= 1;
+}
+
 /** The open decision for a feature, for showing progress ("1 of 2"). */
 export function gatedProgress(slug, feature) {
@@ -313 +337 @@
   if (!_rs) {
     _rs = {
-      ins: db.prepare(`INSERT INTO ap_gated_reviews (id, guardian_slug, ward_uri, ward_inbox, proposer, feature, value)
-                       VALUES (?,?,?,?,?,?,?)
-                       ON CONFLICT(guardian_slug, id) DO UPDATE SET value = excluded.value, ward_inbox = excluded.ward_inbox`),
+      ins: db.prepare(`INSERT INTO ap_gated_reviews (id, guardian_slug, ward_uri, ward_inbox, proposer, feature, value, decisive)
+                       VALUES (?,?,?,?,?,?,?,?)
+                       ON CONFLICT(guardian_slug, id) DO UPDATE SET value = excluded.value, ward_inbox = excluded.ward_inbox, decisive = excluded.decisive`),
       get: db.prepare('SELECT * FROM ap_gated_reviews WHERE guardian_slug = ? AND id = ?'),
       bySlug: db.prepare('SELECT * FROM ap_gated_reviews WHERE guardian_slug = ? ORDER BY created_at DESC'),
@@ -326 +350 @@
 
 export function recordGatedReview(guardianSlug, r) {
-  rstmts().ins.run(r.id, guardianSlug, r.wardUri, r.wardInbox || null, r.proposer || null, r.feature, r.value ? 1 : 0);
+  // decisive ontbreekt bij een oudere server -> 1, want bij twijfel waarschuwen.
+  rstmts().ins.run(r.id, guardianSlug, r.wardUri, r.wardInbox || null, r.proposer || null, r.feature, r.value ? 1 : 0, r.decisive === false ? 0 : 1);
   return rstmts().get.get(guardianSlug, r.id);
 }
@@ -424 +449 @@
 export default {
   GATE_CATALOGUE, gateRows, knownSetting,
-  tallyGatedSetting, thresholdFor, featureColumn, recordGatedVote, gatedProgress, gateConsequence, GATED_WINDOW_MS,
+  tallyGatedSetting, thresholdFor, featureColumn, recordGatedVote, gatedProgress, gateConsequence, isDecisive, GATED_WINDOW_MS,
   parseGatedSetting, buildGatedOffer, rememberGatedOffer, recallGatedOffer,
   recordGatedReview, getGatedReview, listGatedReviews, removeGatedReview, clearGatedReviews,