Changeset 742ba7e in Klonkt for src/routes

Timestamp:

07/28/2026 07:58:26 PM (6 weeks ago)

Author:

Robin Genis <roboburr@…>

Branches:

main

Children:

6c152a5

Parents:

70677e96

Message:

Een ward loslaten vraagt eerst, en zegt wat het doet

Loslaten zat achter een window.confirm met een lap tekst erin. Dat is precies
het soort dialoog dat mensen wegklikken om ervan af te zijn, terwijl dit de
zwaarste knop in de app is. Nu is het een stap in het paneel: eerst de gevolgen,
dan ja of nee, met nee links zodat de uitweg de makkelijke is om te raken.

De waarschuwing wordt opgebouwd uit wat er echt aan de hand is, niet uit een
vaste zin, want loslaten betekent twee verschillende dingen. Blijven er andere
guardians over, dan stap jij op en blijft het kind een ward (FEP-633c 3.3). Ben
je de laatste, dan is het emancipatie, en 3.4 zegt uitdrukkelijk dat geen enkele
guardian daar alleen over gaat: drie instemmende volwassenen, of een meerderheid
met twee getuigen. Een nieuwe route zoekt dat op het moment van drukken op, want
voor een ward die wij niet hosten moet daarvoor die server bevraagd worden en
dat hoort niet bij elke ververs-ronde.

En het derde ding dat er hoorde te staan: het Undo federeert nog niet (fase 4 in
relations.js), dus vandaag haalt de knop de band alleen hier weg terwijl de
server van het kind je gewoon als guardian blijft noemen. Zonder die regel drukt
een guardian erop in de overtuiging dat het kind losgelaten is.

Changed files:
src/routes/guardian.js

• GET /guardian/wards/release-check: telt de guardians van dit kind (lokaal uit de relaties, remote uit shaer:guardians van de actor) en weigert een ward die niet van jou is
• de nieuwe labels toegevoegd aan uiStrings

src/assets/js/guardian.js

• releaseStep: de waarschuwing plus ja/nee, in plaats van window.confirm
• de site gaat mee in de check, anders kijkt de server naar de verkeerde

src/assets/css/guardian.css

• --danger toegevoegd; die bestond niet, waardoor het kader en de knop onzichtbaar bleven
• opmaak voor het waarschuwingsblok

src/services/i18n.js

• 24 strings voor de waarschuwing in nl, en, de

test/guardian-panel.test.js

• loslaten gaat in twee stappen en niet via window.confirm
• elke CSS-variabele die de PWA gebruikt is er ook een die hij definieert

remarks: alle vier de gevallen in de browser nagelopen: laatste guardian,
meerdere guardians, onbereikbare server, en een ward die niet van je is. Nee
zet de knop terug zonder iets te doen, ja verwijdert alleen die ene relatie.

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

File:

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

src/routes/guardian.js

@@ -45 +45 @@
     'settings_title', 'panel_open', 'panel_close', 'panel_help', 'panel_help_empty',
     'panel_follow', 'panel_follow_empty', 'panel_posts', 'panel_posts_empty',
-    'panel_actions', 'badge_help', 'badge_follow', 'badge_follow_one', 'help_empty'];
+    'panel_actions', 'badge_help', 'badge_follow', 'badge_follow_one', 'help_empty',
+    // Releasing a ward: a deliberate two-step answer, never one click.
+    'release_title', 'release_effect', 'release_local', 'release_step_down',
+    'release_last', 'release_unknown', 'release_yes', 'release_no'];
   const s = Object.fromEntries(keys.map((k) => [k, i18nT(L, `guardian.${k}`)]));
   s.wave = i18nT(L, 'guardian.wave');
@@ -283 +286 @@
 
 // ── Manage: release a committed ward (local Undo; federation is Fase 4). ──
+/**
+ * What actually happens if this guardian releases this ward?
+ *
+ * Releasing is not one action but two very different ones, and the difference
+ * is the number of guardians the child has left (FEP-633c):
+ *   - more than one → §3.3, you step down and the child stays a ward;
+ *   - you are the last → §3.4, that is emancipation, and the FEP is explicit
+ *     that no single guardian decides it alone (three consenting adults, or a
+ *     majority plus two witnesses).
+ * On top of that, today's release is LOCAL: the Undo is not federated yet
+ * (relations.js, fase 4), so the ward's server keeps listing this guardian.
+ * A guardian pressing the button would otherwise believe the child is released.
+ *
+ * Answered on demand rather than in the dashboard state: for a ward we do not
+ * host this reaches out to that ward's server, and nobody should pay for that
+ * on every refresh.
+ */
+router.get('/wards/release-check', requireAuth, async (req, res) => {
+  const site = siteForUser(req);
+  if (!site) return res.status(404).json({ error: 'no_site' });
+  const uri = String(req.query.uri || '').trim();
+  if (!uri) return res.status(400).json({ error: 'empty_uri' });
+  if (!Guardianship.listWards(site.slug).some((w) => w.other_uri === uri)) {
+    return res.status(403).json({ error: 'not_my_ward' });
+  }
+  const base = (process.env.PUBLIC_BASE_URL || '').replace(/\/+$/, '');
+  const local = !!base && uri.startsWith(`${base}/`);
+  let guardians = null;   // null = we could not find out; say so rather than guess
+  if (local) {
+    const slug = uri.replace(/\/+$/, '').split('/').pop();
+    try { guardians = Guardianship.listGuardians(slug).length; } catch { /* stays null */ }
+  } else {
+    const doc = await AP.fetchActor(uri).catch(() => null);
+    const g = doc && doc['shaer:guardians'];
+    if (Array.isArray(g)) guardians = g.length;
+    else if (typeof g === 'string') guardians = 1;
+    else if (g && Array.isArray(g.items)) guardians = g.items.length;
+    else if (doc) guardians = 0;   // the actor answered and names no guardians
+  }
+  res.json({
+    guardians,
+    last: guardians === null ? null : guardians <= 1,
+    local,
+    federates: false,   // the Undo does not travel yet (§3.2, fase 4)
+  });
+});
+
 router.post('/wards/remove', requireAuth, express.json({ limit: '4kb' }), (req, res) => {
   const site = siteForUser(req);