Changeset 407353e in Klonkt for src/services/guardianship/delivery.js

Timestamp:

09/23/2026 03:13:00 PM (2 hours ago)

Author:

Bart <bart@…>

Branches:

main

Parents:

0ad37a0

Message:

Een foto zonder woorden is een bericht (502 direct_failed)

deliverDirectNote begon met !String(text).trim() -> return null, en de inname
maakte van dat null een 502 direct_failed. Sinds de inname een bijlage-only
note doorlaat waren die twee lagen het oneens over wat een bericht is: wie een
foto op een gesprek liet vallen en niets typte kwam precies daar vast te
zitten. Het viel niet op in de tests, want dit was de enige plek waar de regel
stond en hij stond midden in een functie die netwerk doet.

Nu is het een eigen, pure regel -- text, html of bijlage -- met tests eromheen,
zoals de rest van deze laag het doet. Leeg blijft geweigerd: geen tekst, geen
opmaak en geen bijlage is niets.

En String(text) werd String(text || ''): zonder tekst stond er anders
letterlijk "undefined" in de body zodra een aanroeper het veld wegliet.

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

File:

• src/services/guardianship/delivery.js (modified) (3 diffs)

src/services/guardianship/delivery.js

@@ -35 +35 @@
 }
 
+/**
+ * Heeft dit directe bericht iets te zeggen? Puur, zodat de regel te toetsen is
+ * zonder database, netwerk of scherm.
+ *
+ * EEN FOTO KAN HET HELE BERICHT ZIJN. Hier stond alleen `!text.trim()`, en dat
+ * betekende: een direct bericht zonder woorden gaf null terug, waar de inname
+ * "502 direct_failed" van maakte. Precies wat je kreeg als je een foto op een
+ * gesprek liet vallen en niets typte -- de inname liet een bijlage-only note
+ * door, en deze laag eronder weigerde hem alsnog. Twee lagen die het niet eens
+ * waren over wat een bericht is.
+ *
+ * Een leeg bericht blijft geweigerd: geen tekst, geen opmaak en geen bijlage is
+ * niets, en dat hoort niet de deur uit te gaan.
+ */
+export function directNoteHasContent({ text, html, attachments } = {}) {
+  if (String(text || '').trim()) return true;
+  if (String(html || '').trim()) return true;
+  return Array.isArray(attachments) && attachments.length > 0;
+}
+
 // A direct note: a NEW conversation (or a direct reply) addressed to specific
 // actors only. Stored in ap_outbox with visibility 'direct' + the recipient
@@ -53 +73 @@
   const base = (process.env.PUBLIC_BASE_URL || '').replace(/\/+$/, '');
   const list = [...new Set((recipients || []).filter((u) => /^https?:\/\//i.test(String(u || ''))))].slice(0, 8);
-  if (!base || !site || !site.slug || !list.length || !String(text || '').trim()) return null;
+  if (!base || !site || !site.slug || !list.length) return null;
+  if (!directNoteHasContent({ text, html, attachments })) return null;
   const me = actorId(base, site.slug);
   // Resolve every recipient for a mention anchor + a delivery inbox.
@@ -108 +129 @@
   const richClean = html ? deps.sanitizeHtml(String(html)) : '';
   const rich = richClean && deps.htmlToPlainText(richClean).trim() ? richClean : '';
-  const body = escHtml(String(text).trim()).replace(/\r?\n/g, '<br>');
+  const body = escHtml(String(text || '').trim()).replace(/\r?\n/g, '<br>');
   // De mention-anker blijft een eigen alinea vooraan: de ontvanger moet in het
   // bericht genoemd staan, ook als de rijke inhoud met een kop of lijst begint.