Changeset 41de7cb in Klonkt

Timestamp:

08/10/2026 04:13:24 PM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

21ef238

Parents:

f0a33e1

Message:

Een kopje draagt ook het nieuwste bericht VAN de ander

Op het toestel gezien: de hemel noemde mensen opeens 'boiert' en
'hotelbreakfast' in plaats van hun naam, en tekende letters in plaats van
gezichten. Oorzaak: is het nieuwste bericht in een gesprek van MIJ, dan draagt
het kopje mijn byline, en de hemel zoekt naam en gezicht van de ander in een
bericht VAN de ander. Vond hij dat niet, dan viel hij terug op het staartje van
de actor-uri.

conversationHeads geeft nu twee rijen per persoon: het nieuwste bericht (dat
bepaalt waar iemand hangt) en het nieuwste bericht van hem. Vallen ze samen,
dan blijft het een rij. Geen clientwijziging nodig -- conversationsFrom maakt er
dezelfde Conversation van.

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

Files:

• src/services/ActivityPubService.js (modified) (1 diff)
• test/conversations.test.js (modified) (2 diffs)

src/services/ActivityPubService.js

@@ -4266 +4266 @@
 export function conversationHeads(slug) {
   try {
+    // Twee rijen per persoon, niet een: het nieuwste bericht (dat bepaalt waar
+    // iemand in de hemel hangt) EN het nieuwste bericht VAN HEM.
+    //
+    // Die tweede is er omdat het nieuwste bericht van jou kan zijn, en dan
+    // draagt het jouw byline. De hemel zoekt de naam en het gezicht van de
+    // ander in een bericht van de ander -- vond hij dat niet, dan viel hij
+    // terug op het staartje van de actor-uri en heette tante opeens
+    // 'hotelbreakfast'. Op het toestel gezien, 10-8.
+    //
+    // Valt het samen (het nieuwste is al van hem), dan is het een rij; dubbel
+    // sturen doen we niet.
     return db.prepare(`
       SELECT other, stamp, direction, ref FROM (
         SELECT *, ROW_NUMBER() OVER (PARTITION BY other ORDER BY stamp DESC, ref DESC) AS rn
           FROM (${CONVERSATION_UNION})
+      ) WHERE rn = 1
+      UNION
+      SELECT other, stamp, direction, ref FROM (
+        SELECT *, ROW_NUMBER() OVER (PARTITION BY other ORDER BY stamp DESC, ref DESC) AS rn
+          FROM (${CONVERSATION_UNION}) WHERE direction = 'in'
       ) WHERE rn = 1
       ORDER BY stamp DESC, ref DESC`).all({ slug });

test/conversations.test.js

@@ -50 +50 @@
 test('een rij per tegenpartij, hoe druk de drukste ook is', () => {
   const heads = AP.conversationHeads('kind');
-  assert.deepEqual(heads.map((k) => k.other), [OMA, TANTE], 'nieuwste gesprek eerst, allebei aanwezig');
+  assert.deepEqual([...new Set(heads.map((k) => k.other))], [OMA, TANTE], 'nieuwste gesprek eerst, allebei aanwezig');
+});
+
+test('is het nieuwste bericht van MIJ, dan gaat dat van hem ook mee', () => {
+  // Anders draagt het kopje mijn byline en zoekt de hemel de naam en het
+  // gezicht van de ander tevergeefs -- die viel dan terug op het staartje van
+  // de actor-uri. Bij tante is mijn antwoord het nieuwste.
+  const vanTante = AP.conversationHeads('kind').filter((k) => k.other === TANTE);
+  assert.equal(vanTante.length, 2);
+  assert.deepEqual(vanTante.map((k) => k.direction).sort(), ['in', 'out']);
+  // Bij oma is het nieuwste al van haar: dan is een rij genoeg.
+  assert.equal(AP.conversationHeads('kind').filter((k) => k.other === OMA).length, 1);
 });
 
@@ -110 +121 @@
   const list = await get(`${slug}/conversations`);
   assert.equal(list.status, 200);
-  assert.equal(list.body.totalItems, 2, 'oma en tante, en niet de publieke vreemde');
+  // Drie items voor twee mensen: bij tante gaat haar eigen bericht mee omdat
+  // het nieuwste van mij is (anders mist de hemel haar naam en gezicht).
+  assert.equal(list.body.totalItems, 3);
+  assert.ok(!JSON.stringify(list.body).includes(VREEMDE), 'de publieke vreemde is geen gesprek');
+  const vanTante = list.body.orderedItems.filter((i) => i.actor === TANTE);
+  assert.equal(vanTante.length, 1, 'en dat bericht van tante draagt haar byline');
+  assert.ok(vanTante[0].object.attributedTo.name);
   // De byline zit in de ingesloten actor, net als overal sinds shaer-nmw.
   assert.ok(list.body.orderedItems.every((i) => i.object.attributedTo));