Changeset 383e897 in Klonkt for src

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@…>

Location:

src

Files:

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

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(); }