Changeset 2708282 in Klonkt for src/routes

Timestamp:

07/28/2026 10:45:26 AM (6 weeks ago)

Author:

Robin Genis <roboburr@…>

Branches:

main

Children:

d9ad6c5

Parents:

65abc85

Message:

De knop deed niets: AP.deliverTo bestond niet

Op Propose link previews drukken gaf geen zichtbaar resultaat. Oorzaak: de route
riep AP.deliverTo aan, maar die naam bestond alleen als sleutel in het
deps-object dat we aan de guardianship-module doorgeven, niet als export. Dus een
TypeError, een 500 uit Express, en de knop die zichzelf netjes weer inschakelde.
Van buiten af: er gebeurt niks.

Mijn fout zat in de aanname: ik zag deliverTo in een grep-treffer staan en nam
aan dat het een export was. Een grep-treffer is geen export. Er staat nu een test
die alle AP-helpers controleert die de guardian-route aanroept, zodat deze klasse
fout niet nog eens stil kan blijven.

Changed files:
src/services/ActivityPubService.js

• deliverToActor geexporteerd (stond alleen in de deps)

src/routes/guardian.js

• gebruikt AP.deliverToActor

test/gated-settings.test.js

• test dat elke AP-helper die de route aanroept ook echt bestaat
• test op de Offer-vorm (ward, feature, value, geadresseerd aan de ward)

remarks: 230 tests groen.

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

File:

• src/routes/guardian.js (modified) (1 diff)

src/routes/guardian.js

@@ -317 +317 @@
     return res.json({ ok: true, allow, state: r.state, need: r.need, of: r.of });
   }
-  AP.deliverTo(site, uri, offer).catch(() => { /* queued, best-effort */ });
+  AP.deliverToActor(site, uri, offer).catch(() => { /* queued, best-effort */ });
   res.json({ ok: true, allow, state: 'open', federated: true });
 });