Changeset 26b1848 in Klonkt

Timestamp:

08/11/2026 05:36:21 PM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

fc41e04

Parents:

abbb72a

Message:

Drie spellingen van dezelfde naam, en "door wie" was de hele vraag (shaer-lgo)

Zichtbaar geworden in de echte data van vandaag: naast een markering van
boiert.eu met handle @boiert@… stond die van dev met handle NULL. Op het
scherm valt dat terug op de kale URI.

Onze eigen markering werd op twee plekken geboekt, en allebei anders:

routes/guardian.js (het paneel) null
ActivityPubService (de app, C2S) @${site.slug} -- zonder host
inkomend van een ander de echte handle

Nu alle drie deriveHandle(me). Die stond module-privé in ActivityPubService en
is geexporteerd in plaats van in de route nagebouwd -- een tweede afleiding van
dezelfde naam is precies de naad die vandaag al twee keer beet.

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

Files:

• src/routes/guardian.js (modified) (1 diff)
• src/services/ActivityPubService.js (modified) (2 diffs)
• test/help-c2s.test.js (modified) (1 diff)

src/routes/guardian.js

@@ -431 +431 @@
   // Onze eigen kopie meteen, zonder op bezorging te wachten: het scherm van
   // degene die klikt hoort niet te liegen omdat een andere server traag is.
-  Guardianship.help.record(noteUri, me, kind, null);
+  // MET onze eigen handle. Die stond hier op null, en "door wie" was juist de
+  // hele vraag van deze bead: een binnengekomen markering draagt de handle van
+  // de afzender wel, dus onze EIGEN rij was de enige zonder naam. Op het scherm
+  // viel dat terug op de kale URI.
+  Guardianship.help.record(noteUri, me, kind, AP.deriveHandle(me));
 
   // DE MEDE-GUARDIANS, en dit ging mis (shaer-lgo, gevonden 11-8 met @mee).

src/services/ActivityPubService.js

@@ -1338 +1338 @@
   return m ? decodeURIComponent(m[1]) : null;
 }
-function deriveHandle(actorUri) {
+export function deriveHandle(actorUri) {
   try { const u = new URL(actorUri); const seg = u.pathname.split('/').filter(Boolean).pop() || ''; return `@${seg}@${u.host}`; } catch { return String(actorUri || ''); }
 }
@@ -3204 +3204 @@
           if (mark) {
             const base2 = (process.env.PUBLIC_BASE_URL || '').replace(/\/+$/, '');
-            Guardianship.help.record(mark.noteUri, actorId(base2, site.slug), mark.kind, `@${site.slug}`);
+            // Met de VOLLEDIGE handle. Hier stond `@${site.slug}` -- zonder host,
+            // dus een derde vorm naast de kale URI van de PWA-route en de echte
+            // handle die een binnengekomen markering draagt. Drie spellingen van
+            // dezelfde naam, en "door wie" was de hele vraag van shaer-lgo.
+            const mij = actorId(base2, site.slug);
+            Guardianship.help.record(mark.noteUri, mij, mark.kind, deriveHandle(mij));
           }
           const r = await deliverDirectNote(site, { recipients, text: plain, language: object.language || null, inReplyTo: typeof object.inReplyTo === 'string' ? object.inReplyTo : null, attachments: atts, helpRequest: help, awayUntil, gateRequest: gateReq && gateReq.feature, helpMark: mark });

test/help-c2s.test.js

@@ -58 +58 @@
   // markering.
   assert.equal(h.state.open, true);
+});
+
+test('onze EIGEN markering draagt onze eigen handle', () => {
+  // "Door wie" was de hele vraag van deze bead. Een binnengekomen markering
+  // draagt de handle van de afzender wel; onze eigen rij stond op null en viel
+  // op het scherm terug op de kale URI. Gezien op 11-8 in de echte data naast
+  // een rij van boiert.eu, die zijn handle wel had.
+  const coll = queues.helpCollection('https://x/queues/help', 'oma');
+  const item = coll.orderedItems.find((i) => i.id === BOEI);
+  assert.deepEqual(item['shaer:pickedUpBy'], ['@oma@oma.test'],
+    'een handle en geen URI');
 });