Changeset 88d7c8f in Klonkt for src/routes/guardian.js

Timestamp:

07/29/2026 07:15:16 AM (6 weeks ago)

Author:

Robin Genis <roboburr@…>

Branches:

main

Children:

e27b8db

Parents:

439f095

Message:

Een gated voorstel bereikte de andere guardians nooit

Op de vloot nagekeken waarom YouTube-voorbeelden bij beta uit blijven: het
voorstel van sound-fabrics staat er, met een ja van een van de drie guardians,
en het is stil verlopen. Niet omdat iemand bezwaar had, maar omdat niemand
anders het ooit gezien heeft.

Het voorstel wordt geadresseerd aan de server van het kind, want die telt en
handhaaft (5.6). Maar daarmee bereikt het alleen de voorsteller en het kind. De
twee guardians op andere servers weten van niets, kunnen dus niet antwoorden, en
een drempel van twee is onhaalbaar. Elk voorstel verloopt na een dag.

De ontbrekende schakel is het doorsturen, precies wat 5.3 al doet voor een
gated follow: de server van het kind kent de gezaghebbende guardian-lijst, dus
die stuurt het voorstel door. Elke guardian bewaart een kopie die hij kan
beantwoorden, en het antwoord reist terug naar het kind, dat telt.

Changed files:
src/config/database.js

• tabel ap_gated_reviews, de guardian-kopie (zelfde vorm als ap_follow_reviews)

src/services/guardianship/gated.js

• de kopie-opslag: bewaren, lezen, beantwoorden, opruimen

src/services/guardianship/handshake.js

• ward-kant: doorsturen naar de andere guardians zodra het voorstel openstaat
• guardian-kant: de doorgestuurde kopie bewaren om te kunnen antwoorden

src/routes/guardian.js

• gatedReviews in de dashboardstaat; POST /guardian/api/gated/:id stuurt het antwoord naar de inbox van het kind

src/assets/js/guardian.js, src/assets/css/guardian.css

• de voorstel-kaart met Eens/Oneens bij de aanvragen

src/services/i18n.js

• de teksten in nl, en, de

test/gated-settings.test.js

• de hele keten: voorstellen, doorsturen naar allebei de anderen, de kopie opslaan, antwoorden, en pas bij twee van drie gaat de gate open

remarks: dit forceert niets; het maakt alleen mogelijk wat de spec al bedoelde.
Twee van de drie guardians moeten nog steeds akkoord gaan, en het venster is
24 uur.

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

File:

• src/routes/guardian.js (modified) (3 diffs)

src/routes/guardian.js

@@ -52 +52 @@
     'avail_available', 'avail_away', 'avail_dormant', 'panel_guards', 'panel_guards_remote',
     'lapse_propose', 'lapse_line', 'lapse_tally', 'lapse_note', 'lapse_agree', 'lapse_disagree', 'voted',
-    'away_title', 'away_sub', 'away_week', 'away_month', 'away_done'];
+    'away_title', 'away_sub', 'away_week', 'away_month', 'away_done',
+    // A gated-setting proposal from a fellow guardian (5.6).
+    'gated_title', 'gated_line_on', 'gated_line_off', 'gated_agree', 'gated_disagree'];
   const s = Object.fromEntries(keys.map((k) => [k, i18nT(L, `guardian.${k}`)]));
   s.wave = i18nT(L, 'guardian.wave');
@@ -93 +95 @@
     // Running lapses (3.6.3) this guardian or its local wards are party to.
     lapses: Guardianship.availability.lapseQueueItems(site.slug, me, Date.now()),
+    // Gated-setting proposals another guardian opened on a ward we share
+    // (5.6), forwarded here by the ward's server. Without answering these the
+    // threshold is never met and the proposal simply expires.
+    gatedReviews: Guardianship.gated.listGatedReviews(site.slug).map((r) => ({
+      id: r.id, ward: r.ward_uri, proposer: r.proposer, feature: r.feature, value: !!r.value,
+    })),
     help,
     strings: uiStrings(L),
@@ -345 +353 @@
   if (!r || r.status >= 400) return res.status(r?.status || 500).json({ error: r?.error || 'lapse_failed' });
   res.json({ ok: true, lapse: r.id });
+});
+
+// ── Answer a forwarded gated-setting proposal (FEP-633c 5.6) ─────────────
+// The decision belongs to the ward's server, so the answer travels there as an
+// Accept/Reject on the offer id, exactly like a gated follow's decision.
+router.post('/api/gated/:id', requireAuth, express.json({ limit: '2kb' }), async (req, res) => {
+  const site = siteForUser(req);
+  if (!site) return res.status(404).json({ error: 'no_site' });
+  const review = Guardianship.gated.getGatedReview(site.slug, req.params.id);
+  if (!review) return res.status(404).json({ error: 'gone' });
+  const agree = req.body?.answer !== 'reject';
+  const base = (process.env.PUBLIC_BASE_URL || '').replace(/\/+$/, '');
+  const me = AP.actorId(base, site.slug);
+  const activity = {
+    id: `${me}#gated-${Date.now().toString(36)}`,
+    type: agree ? 'Accept' : 'Reject', actor: me, to: [review.ward_uri], object: review.id,
+  };
+  try { await AP.deliverToActor(site, review.ward_uri, activity); }
+  catch { return res.status(502).json({ error: 'delivery' }); }
+  Guardianship.gated.removeGatedReview(site.slug, review.id);
+  res.json({ ok: true, answer: agree ? 'accept' : 'reject' });
 });