Changeset 7e53594 in Klonkt

Timestamp:

08/10/2026 03:53:26 AM (4 weeks ago)

Author:

Bart <bart@…>

Branches:

main

Children:

0db3c72

Parents:

f06ed56

Message:

shaer:log: het logboek staat naast de wachtrijen, niet erin

De apps konden er niet bij. Het logboek zat achter /guardian/api/events, een
PWA-route, dus een ward kon niet lezen waarom haar eigen aanbod geweigerd was --
en dat is precies de helft die §4.2 eist.

Eigen sleutel op de actor, GET /ap/users/:slug/log, dezelfde eigenaar-only
bearer als de wachtrijen. Bewust niet onder shaer:queues: alles daar wacht op
een antwoord en dit is wat er al besloten is. Geschiedenis onderbrengen bij een
woord dat "wachtend" betekent maakt van twee dingen één, en dat is de fout die
deze module vandaag drie keer heeft opgeruimd.

type is 'shaer:Event' en geen AS2-werkwoord. De meeste soorten zijn er geen: een
lapse-stem of een opgepakte hulpvraag is geen Accept, en het zo noemen zou
netter lezen dan het is.

Clients vinden hem via het actor-document, niet via een pad dat ze zelf
verzinnen (shaer-qa0). De AS2-test viel netjes om tot shaer:log was aangemeld --
tweede keer vandaag dat die het vangt.

De daemon volgt (shaer-6d9): zolang die geen shaer:log serveert, liegt een
client die tegen beide praat over een van de twee.

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

Files:

• src/routes/activitypub.js (modified) (1 diff)
• src/services/guardianship/index.js (modified) (1 diff)
• src/services/guardianship/queues.js (modified) (1 diff)
• src/services/guardianship/relations.js (modified) (1 diff)
• test/activitypub-as2.test.js (modified) (1 diff)
• test/guardian-event-log.test.js (modified) (1 diff)

src/routes/activitypub.js

@@ -344 +344 @@
 // guardians' real states here and nowhere else.
 queueRoute('guardians', (id, slug) => Guardianship.guardiansCollection(id, slug));
+
+// ── Het logboek (FEP-633c §4.2, shaer:log) ────────────────────────────
+// NAAST de wachtrijen en niet erin: alles onder shaer:queues wacht op een
+// antwoord, dit is wat er al besloten is. Eigen pad, dezelfde eigenaar-only
+// bearer. Het bestaat omdat een weigering anders alleen te merken viel doordat
+// er iets uit een lijst verdween, en "het is weg" is geen reden.
+router.get('/ap/users/:slug/log', (req, res) => {
+  const auth = OAuth.verifyBearer(req.headers.authorization);
+  if (!auth || auth.site.slug !== req.params.slug) return res.status(403).end();
+  const me = `${baseUrl(req)}/ap/users/${auth.site.slug}`;
+  AP.sendAP(res, {
+    '@context': AP.AP_CONTEXT,
+    ...Guardianship.logCollection(`${me}/log`, auth.site.slug, (s) => AP.listGuardianEvents(s, 50)),
+  }, 'private, no-store');
+});
 // De hulpvragen MET hun staat (5.2.1, shaer-lgo). De apps lazen ze uit de feed
 // en wisten dus niet of er al iemand op af was -- daarom bleef een afgehandeld

src/services/guardianship/index.js

@@ -19 +19 @@
 export { wireDelivery, c2sVisibility, deliverDirectNote } from './delivery.js';
 export { wireHandshake, handleOutbox as handleGuardianshipOutbox, handleInbox as handleGuardianshipInbox, parseRelationship, parseUndoRelationship, endGuardianship } from './handshake.js';
-export { offersCollection, followsCollection, outgoingFollowsCollection, wardsCollection, guardiansCollection, helpCollection } from './queues.js';
+export { offersCollection, followsCollection, outgoingFollowsCollection, logCollection, wardsCollection, guardiansCollection, helpCollection } from './queues.js';
 export * as availability from './availability.js';
 export { wireAvailability } from './availability.js';

src/services/guardianship/queues.js

@@ -118 +118 @@
 }
 
-export default { offersCollection, followsCollection, outgoingFollowsCollection, wardsCollection, guardiansCollection, helpCollection, helpItemsFor, wardGates, wardGuardianStatuses };
+/**
+ * Het logboek (§4.2): wat er is besloten, en waarom.
+ *
+ * Geen wachtrij, en daarom een eigen sleutel op de actor. Elk item draagt zijn
+ * soort en, als die er was, de REDEN -- want zonder die reden merkte een ward
+ * een weigering alleen doordat er iets uit een lijst verdween.
+ *
+ * `type` is geen AS2-werkwoord: de meeste soorten zijn er geen. Een lapse-stem
+ * of een opgepakte hulpvraag is geen Accept, en het zo noemen zou netter lezen
+ * dan het is.
+ *
+ * De lezer levert `listEvents` aan; deze module kent ActivityPubService niet en
+ * houdt dat zo (zie de kop van delivery.js).
+ */
+export function logCollection(id, slug, listEvents) {
+  const items = (typeof listEvents === 'function' ? listEvents(slug) : []).map((e) => {
+    const { id: n, kind, created, ...rest } = e;
+    const item = { id: `${id}/${n}`, type: 'shaer:Event', 'shaer:kind': kind, published: created };
+    for (const [k, v] of Object.entries(rest)) {
+      if (v === undefined || v === null) continue;
+      item[k.startsWith('shaer:') ? k : `shaer:${k}`] = v;
+    }
+    return item;
+  });
+  return collection(id, items);
+}
+
+export default { offersCollection, followsCollection, outgoingFollowsCollection, wardsCollection, guardiansCollection, helpCollection, helpItemsFor, wardGates, wardGuardianStatuses, logCollection };
 
 // ── Wat er voor een ward gated is (shaer-ahy.1) ─────────────────────────

src/services/guardianship/relations.js

@@ -82 +82 @@
       help: `${id}/queues/help`,
     },
+    // NAAST de wachtrijen, niet erin. Alles onder shaer:queues wacht op een
+    // antwoord; dit is wat er al besloten is, met de reden erbij (§4.2).
+    // Geschiedenis onderbrengen bij een woord dat "wachtend" betekent maakt van
+    // twee dingen één, en dat is precies de fout die de rest van deze module
+    // net heeft opgeruimd.
+    'shaer:log': `${id}/log`,
   };
   const guardians = listGuardians(slug).map((r) => r.other_uri);

test/activitypub-as2.test.js

@@ -45 +45 @@
   // `help` is de vragenlijst met haar STAAT (5.2.1): de apps lazen hulpvragen uit
   // de feed en wisten niet of er al iemand op af was.
-  'shaer:queues', 'offers', 'follows', 'outgoingFollows', 'wards', 'guardians', 'help',
+  'shaer:queues', 'offers', 'follows', 'outgoingFollows', 'wards', 'guardians',
+  // §4.2: het logboek staat NAAST de wachtrijen, want het wacht nergens op.
+  'shaer:log', 'help',
   // Elke OPEN hulpvraag zit in de collectie; daarom mag een app uit afwezigheid
   // concluderen dat iets niet open is (shaer-6wt).

test/guardian-event-log.test.js

@@ -76 +76 @@
   assert.doesNotThrow(() => AP.onGuardianshipEvent(null, { kind: 'setting' }));
 });
+
+// ── De collectie die de apps lezen (shaer:log) ────────────────────────────
+// Het logboek staat NAAST shaer:queues. Alles daarin wacht op een antwoord; dit
+// is wat er al besloten is. Een aparte sleutel dus, en een aparte vorm.
+
+test('de actor wijst het logboek aan, los van de wachtrijen', async () => {
+  const G = await import('../src/services/guardianship/index.js');
+  const props = G.guardianshipActorProps('https://test.example/ap/users/kid', 'kid');
+  assert.equal(props['shaer:log'], 'https://test.example/ap/users/kid/log');
+  assert.equal(props['shaer:queues'].log, undefined,
+    'geen geschiedenis onder een woord dat "wachtend" betekent');
+});
+
+test('elk item draagt zijn soort en zijn reden mee', async () => {
+  const G = await import('../src/services/guardianship/index.js');
+  const c = G.logCollection('https://test.example/ap/users/kid/log', 'kid',
+    (s) => AP.listGuardianEvents(s, 50));
+  assert.equal(c.type, 'OrderedCollection');
+  const weigering = c.orderedItems.find((i) => i['shaer:kind'] === 'offer_rejected');
+  assert.ok(weigering, 'de weigering staat erin');
+  assert.equal(weigering['shaer:reason'], 'not_a_teapot',
+    'MET de reden: daarvoor bestaat dit ding');
+  assert.equal(weigering['shaer:candidate'], 'https://elders.example/users/tess');
+  assert.ok(weigering.published, 'en wanneer');
+  // Geen AS2-werkwoord: een lapse-stem of een opgepakte hulpvraag is geen
+  // Accept, en het zo noemen zou netter lezen dan het is.
+  assert.equal(weigering.type, 'shaer:Event');
+});
+
+test('een leeg logboek is een lege collectie, geen fout', async () => {
+  const G = await import('../src/services/guardianship/index.js');
+  const c = G.logCollection('https://test.example/ap/users/niemand/log', 'niemand',
+    (s) => AP.listGuardianEvents(s, 50));
+  assert.equal(c.totalItems, 0);
+  assert.deepEqual(c.orderedItems, []);
+});