Changeset 0448e34 in Klonkt for test/messages-conversations.test.js

Timestamp:

08/06/2026 02:04:30 AM (5 weeks ago)

Author:

Claude (agent) <aiclaude@…>

Branches:

main

Children:

a78a0ef

Parents:

7875a80

git-author:

Robin <roboburr@…> (08/06/2026 02:04:29 AM)

git-committer:

Claude (agent) <aiclaude@…> (08/06/2026 02:04:30 AM)

Message:

Gesprekken verborg regels die geen draad werden

Robins melding: een draad die je zelf begonnen bent viel weg onder Gesprekken.
Oorzaak was een halve opruiming van mijn kant. De chips gingen van zes naar
vier, maar msg-item bleef 'sent' en 'msgs' als data-kind uitdelen aan regels die
om wat voor reden dan ook geen draad werden. Die twee chips bestaan niet meer,
dus zo'n regel matchte geen enkel filter en was alleen nog onder Alles te zien.

Geraakt werden: een verzonden bericht zonder post en zonder herkenbare
ontvanger, een mention zonder afzender, en een losse privé-reply. Alles wat een
gesprek is krijgt nu data-kind 'conv'; alleen Activiteit en Moderatie houden een
eigen emmer.

Daarnaast de oorzaak een stap eerder aangepakt: een direct bericht kan zonder
to_handle in ap_outbox staan, en dan vond threadKey geen tegenpartij. De eerste
uit to_actors telt nu ook, met to_handle als winnaar zodat dezelfde persoon niet
twee draden oplevert. Juist een gesprek dat JIJ begon heeft vaak niets anders om
aan te hangen, dus daar deed het ontbreken van die terugval het meeste pijn.

Getest tegen dev: de drie gevallen renderden aantoonbaar als data-kind sent/msgs
en doen dat nu als conv, terwijl like en report op act en mod blijven. Suite
410/410, met twee nieuwe tests op de to_actors-terugval.

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

File:

• test/messages-conversations.test.js (modified) (1 diff)

test/messages-conversations.test.js

@@ -112 +112 @@
   assert.equal(out[0].type, 'sent');
 });
+
+test('een direct bericht zonder to_handle valt terug op to_actors', () => {
+  // Precies het geval waarin het het onlogischst is dat er geen draad ontstaat:
+  // een gesprek dat JIJ begon. Zonder deze terugval bleef het een losse regel.
+  const key = threadKey({
+    type: 'sent',
+    to_actors: JSON.stringify(['https://a.test/users/anna', 'https://b.test/users/bo']),
+    created_at: ts(1),
+  });
+  assert.equal(key, 'actor:anna@a.test');
+  // Rommel in de kolom mag niets omgooien.
+  assert.equal(threadKey({ type: 'sent', to_actors: 'geen json' }), null);
+  assert.equal(threadKey({ type: 'sent', to_actors: '[]' }), null);
+});
+
+test('to_handle wint van to_actors, zodat een draad niet splitst', () => {
+  const a = threadKey({ type: 'sent', to_handle: '@anna@a.test' });
+  const b = threadKey({ type: 'sent', to_handle: '@anna@a.test', to_actors: JSON.stringify(['https://b.test/users/bo']) });
+  assert.equal(a, b, 'dezelfde tegenpartij hoort dezelfde sleutel te geven');
+});