Changeset 6e9bc70 in Klonkt for src/routes/activitypub.js

Timestamp:

10/12/2026 04:15:08 PM (2 days ago)

Author:

Bart <bart@…>

Branches:

main

Children:

61d52ed

Parents:

746d98a

Message:

Zachte groepen: een gesprek is de set deelnemers, niet één tegenpartij

Een direct bericht aan Anna en Ben bestond al als groep -- to_actors droeg de
hele set -- maar elke lezing sleutelde op één other. Een uitgaand bericht
aan twee mensen viel in twee gesprekken uiteen, en een binnenkomend bericht
van Anna aan jou en Ben kwam alleen bij Anna terecht: Ben viel eruit, want de
adressering van een binnenkomer werd nergens bewaard.

Nu is er een sleutel, één keer berekend bij het schrijven
(conversation-key.js): voor één ander zijn kale URI, precies zoals het altijd
was, dus bestaande gesprekken, leesmarkeringen en ?with=-URL's blijven geldig
zonder migratie; voor een groep de gesorteerde set, gescheiden door een
spatie. Koppen, geschiedenis, leesmarkeringen en ongelezen lezen hem allemaal
via dezelfde unie, en werken daarmee per gesprek zonder dat er in die queries
iets hoefde te veranderen. Oude rijen hebben geen sleutel en lezen zoals ze
altijd lazen.

Alleen directe berichten groeperen: een openbare post aan jou en Ben is geen
besloten gesprek met z'n drieën.

/messages neemt meer dan één with, en elk item draagt shaer:conversation en
shaer:participants. Uitgaande directe notes krijgen een FEP-7888 context --
die van de ouder als het een antwoord is, anders een nieuwe -- die bewaard en
uitgestuurd wordt, maar geen bubbels samenvoegt: een bubbel die Carla toont
terwijl je antwoord haar niet bereikt, zou liegen over wie meeleest.

shaer-mbji
Co-Authored-By: Claude Opus 5.5 <claude@…>

File:

• src/routes/activitypub.js (modified) (4 diffs)

src/routes/activitypub.js

@@ -20 +20 @@
 import OAuth from '../services/OAuthService.js';
 import * as Guardianship from '../services/guardianship/index.js';
+import * as ConversationKey from '../services/conversation-key.js';
 import * as Migration from '../services/MigrationService.js';
 import { getPrimarySite } from '../middleware/site.js';
@@ -626 +627 @@
     }
   }
+  // WELK GESPREK, EN MET WIE (shaer-mbji). De client kon dat afleiden uit de
+  // adressering van de note -- maar een binnenkomer draagt die niet mee in zijn
+  // kaart, dus een bericht van Anna aan jou en Ben las als iets van Anna alleen.
+  // De server weet het: de sleutel staat in de kop. Twee termen, zodat een
+  // client die alleen de deelnemers wil niet een sleutel hoeft te ontleden.
+  for (const { head, item } of pairs) {
+    const obj = item && item.object;
+    if (!obj || !head.other) continue;
+    obj['shaer:conversation'] = head.other;
+    obj['shaer:participants'] = ConversationKey.participantsOfKey(head.other);
+  }
   return pairs;
 }
@@ -673 +685 @@
   const auth = OAuth.verifyBearer(req.headers.authorization);
   if (!auth || auth.site.slug !== req.params.slug) return res.status(403).end();
-  const other = String(req.query.with || '');
-  if (!/^https?:\/\//i.test(other)) return res.status(400).json({ error: 'with must be an actor URI' });
+  // EÉN OF MEER `with` (shaer-mbji). Eén is het gesprek met die persoon, zoals
+  // altijd; meer is de zachte groep van precies die mensen. De sleutel komt
+  // uit dezelfde functie als bij het schrijven, dus {Anna, Ben} en {Ben, Anna}
+  // vragen hetzelfde gesprek op.
+  const withs = [].concat(req.query.with || []).map(String);
+  if (!withs.length || !withs.every((u) => /^https?:\/\//i.test(u))) {
+    return res.status(400).json({ error: 'with must be one or more actor URIs' });
+  }
+  const other = ConversationKey.conversationKey(withs);
+  if (!other) return res.status(400).json({ error: 'with must be one or more actor URIs' });
   const page = AP.conversationHistory(auth.site.slug, other, {
     before: req.query.before ? String(req.query.before) : null,
@@ -684 +704 @@
   // dan klopt het ritme van een 'load more' niet meer met wat de gebruiker ziet.
   const size = req.query.limit ? `&limit=${encodeURIComponent(String(req.query.limit))}` : '';
-  const self = `${baseUrl(req)}/ap/users/${encodeURIComponent(auth.site.slug)}/messages?with=${encodeURIComponent(other)}`;
+  // Elke deelnemer een eigen `with`, zodat de next-link hetzelfde gesprek
+  // opvraagt. De sleutel zelf er kaal in zetten zou bij een groep een spatie in
+  // de URL leggen, en wie die als één adres leest vraagt iets anders op.
+  const withQuery = ConversationKey.participantsOfKey(other).map((u) => `with=${encodeURIComponent(u)}`).join('&');
+  const self = `${baseUrl(req)}/ap/users/${encodeURIComponent(auth.site.slug)}/messages?${withQuery}`;
   AP.sendAP(res, {
     '@context': AP.AP_CONTEXT,