Changeset 383e897 in Klonkt

Timestamp:

08/10/2026 06:16:16 PM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

fc9ca9d

Parents:

e215090

Message:

De badge telt niet wat de poort verbergt

Gevonden bij het bijwerken van de bead, niet bij het bouwen: unreadPerConversation
telde alle inkomende berichten, ook die de app achter een dichte messages-poort
nooit toont. Dan vertelt het getal precies wat de poort verbergt.

Nu dezelfde regel als bij de serialisatie: dicht telt niet mee, behalve wat er
altijd door mag -- het guardian-kanaal en de boei.

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

Files:

• src/routes/activitypub.js (modified) (1 diff)
• src/services/ActivityPubService.js (modified) (2 diffs)
• test/unread.test.js (modified) (1 diff)

src/routes/activitypub.js

@@ -562 +562 @@
   // dezelfde categorie als shaer:liked. Niet in totalItems persen: dat betekent
   // 'hoeveel er zijn' en niet 'hoeveel jij nog niet zag'.
-  const ongelezen = AP.unreadPerConversation(auth.site.slug);
+  const ongelezen = AP.unreadPerConversation(auth.site.slug, {
+    messagesAllowed: P.messagesAllowed,
+    guardians: (() => { try { return new Set(Guardianship.listGuardians(auth.site.slug).map((g) => g.other_uri)); } catch { return new Set(); } })(),
+  });
   const gezien = new Set();
   for (const { head, item } of pairs) {

src/services/ActivityPubService.js

@@ -4410 +4410 @@
  * Eigen berichten tellen nooit mee: je hebt jezelf gelezen.
  */
-export function unreadPerConversation(slug) {
-  try {
+export function unreadPerConversation(slug, { messagesAllowed = true, guardians = new Set() } = {}) {
+  try {
+    // DE POORT TELT MEE. Staat messages dicht, dan toont de app die berichten
+    // niet -- en dan mag een badge ze ook niet aankondigen, want dat getal
+    // vertelt precies wat de poort verbergt. Wat er altijd door mag telt wel:
+    // het guardian-kanaal en de boei. Zelfde regel als bij de serialisatie.
     const rijen = db.prepare(`
       SELECT u.other AS other,
@@ -4421 +4425 @@
        WHERE u.direction = 'in'
          AND (r.cursor IS NULL OR (u.stamp || '|' || u.ref) > r.cursor)
-       GROUP BY u.other`).all({ slug });
+         AND (@open = 1 OR m.help_request = 1 OR u.other IN (SELECT value FROM json_each(@guardians)))
+       GROUP BY u.other`)
+      .all({ slug, open: messagesAllowed ? 1 : 0, guardians: JSON.stringify([...guardians]) });
     return new Map(rijen.map((r) => [r.other, { n: r.n, wave: !!r.wave }]));
   } catch { return new Map(); }

test/unread.test.js

@@ -71 +71 @@
   assert.equal(AP.markRead('kind', 'https://elders/n/bestaat-niet'), null);
 });
+
+test('een dichte messages-poort telt niet mee wat hij verbergt', () => {
+  // Anders vertelt het getal precies wat de poort verbergt: 'er zijn 3
+  // berichten' terwijl de app er geen laat zien.
+  const OPA = 'https://elders/u/opa';
+  arrived('https://elders/n/opa1', OPA, '2026-08-02T10:00:00Z');
+  arrived('https://elders/n/oom2', OOM, '2026-08-02T11:00:00Z');
+  db.prepare(`INSERT INTO ap_mentions (slug, object_uri, actor_uri, actor_name, content, published, help_request)
+              VALUES ('kind', 'https://elders/n/boei', ?, 'iemand', '<p>help</p>', '2026-08-02T12:00:00Z', 1)`).run(OPA);
+
+  const dicht = AP.unreadPerConversation('kind', { messagesAllowed: false, guardians: new Set([OOM]) });
+  assert.equal(dicht.get(OPA).n, 1, 'alleen de boei van opa telt');
+  assert.ok(dicht.get(OOM), 'het guardian-kanaal gaat altijd door');
+
+  const open = AP.unreadPerConversation('kind', { messagesAllowed: true });
+  assert.equal(open.get(OPA).n, 2, 'met de poort open tellen ze allebei');
+});