Changeset 76290bf in Klonkt for src

Timestamp:

08/26/2026 04:49:33 AM (2 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

0d6d266

Parents:

78b8e58

git-author:

Robin <roboburr@…> (08/26/2026 04:49:17 AM)

git-committer:

Robin <roboburr@…> (08/26/2026 04:49:33 AM)

Message:

Inbox: een binnengekomen antwoord houdt zijn inReplyTo

Robins melding, 26-8: de C2S-lezing serveerde antwoorden zonder ouder.

De oorzaak lag een laag dieper dan de serialisatie: ap_mentions had geen
kolom voor inReplyTo, dus de ouder viel al bij het OPSLAAN op de grond en
messageItem had niets te serveren. Een client kan een gesprek alleen
teruglopen langs inReplyTo, dus elk antwoord kwam aan als het begin van
een gesprek -- de ketenlezing in de app kon nooit verder dan een.

Vier plekken, want het is de hele weg: de kolom (met ensureColumn voor
bestaande databases), het schrijven in de inbox, MESSAGE_COLUMNS -- de
ene leesplek die zowel de inbox-lijst als de gesprekken voedt -- en
messageItem.

De ouder gaat door dezelfde poort als de note-url: alleen http(s), en zowel de
string- als de objectvorm die AS2 toestaat. Alleen de string erkennen
zou hetzelfde gat laten voor wie de objectvorm stuurt.

Nagegaan wat WEL goed ging, zodat de reparatie niet breder wordt dan de
kwaal: replyItem droeg hem al uit ap_interactions.parent_uri, sentItem
via buildNote uit ap_outbox.in_reply_to, en de tijdlijn kan hem per
definitie niet missen -- belongsInTimeline weigert alles met inReplyTo.
Deze leg was de enige.

Bestaande rijen blijven leeg: die ouder is niet meer te achterhalen
zonder hem opnieuw op te halen, en een verzonnen ouder is erger dan
geen. Twee toetsen over de hele keten, tegenbewijs gedraaid: allebei
vallen ze tegen de code van hiervoor. Volle suite 1231 groen.

Location:

src

Files:

• config/database.js (modified) (2 diffs)
• routes/activitypub.js (modified) (1 diff)
• services/ap-inbox.js (modified) (1 diff)
• services/ap-timeline.js (modified) (1 diff)

src/config/database.js

@@ -834 +834 @@
       content TEXT,                 -- sanitized HTML snippet of the mentioning note
       published TEXT,
+      in_reply_to TEXT,             -- de note waarop dit een antwoord is (AS2 inReplyTo)
       created_at DATETIME DEFAULT CURRENT_TIMESTAMP,
       UNIQUE(slug, object_uri)
@@ -987 +988 @@
   ensureColumn('ap_mentions', 'quote_json', 'TEXT');        // FEP-044f quoted post
   ensureColumn('ap_mentions', 'embed_json', 'TEXT');        // external link preview
+  // AS2 inReplyTo op een vermelding/bericht. Bestond hier niet, en daarmee ging
+  // de ouder bij het opslaan verloren: elk ANTWOORD in een gesprek kwam bij de
+  // client aan alsof het een gesprek begon (Robins melding, 26-8). Bestaande
+  // rijen blijven leeg -- die ouder is niet meer te achterhalen zonder hem
+  // opnieuw op te halen, en een verzonnen ouder is erger dan geen.
+  ensureColumn('ap_mentions', 'in_reply_to', 'TEXT');
   ensureColumn('ap_interactions', 'media_json', 'TEXT');
   ensureColumn('ap_interactions', 'quote_json', 'TEXT');

src/routes/activitypub.js

@@ -543 +543 @@
       content: AP.stripLeadingMentions(m.content),
       url: m.note_url || undefined,
+      // Waar dit een antwoord op is (Robins melding, 26-8). Zonder dit veld
+      // kwam elk antwoord binnen als het begin van een gesprek: de client kan
+      // een keten alleen teruglopen langs inReplyTo. De andere twee legs
+      // (replyItem uit ap_interactions, sentItem via buildNote) droegen hem
+      // al -- deze was de enige die hem niet eens opsloeg.
+      inReplyTo: m.in_reply_to || undefined,
       published: AP.isoStamp(m.published || m.created_at),
       // Addressed to us and to nobody we know of: the other recipients of a

src/services/ap-inbox.js

@@ -691 +691 @@
         for (const slug of slugs) {
           try {
-            const r = db.prepare(`INSERT OR IGNORE INTO ap_mentions (slug, object_uri, note_url, actor_uri, actor_name, actor_handle, actor_icon, actor_url, content, published, help_request, wave, has_guardians, emoji_json, actor_emoji_json, media_json, created_at)
-                                  VALUES (?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,CURRENT_TIMESTAMP)`)
-              .run(slug, o.id, safeUrl(o.url) || null, actorUri, ai.name, ai.handle, ai.icon, ai.url, html, o.published || null, help ? 1 : 0, wave ? 1 : 0, hasG ? 1 : 0,
+            // De OUDER gaat mee (Robins melding, 26-8). Hij stond nergens in
+            // deze rij, dus een antwoord binnen een gesprek kwam bij de client
+            // aan alsof het een gesprek begon: de app kan een keten alleen
+            // teruglopen langs inReplyTo, en die was leeg.
+            //
+            // Alleen een http(s)-adres, langs dezelfde poort als `url`: een
+            // inReplyTo komt van een vreemde en mag geen ander schema
+            // binnensmokkelen. AS2 staat een string of een object toe, dus
+            // allebei uitpakken -- alleen de string erkennen zou hetzelfde gat
+            // laten voor iedereen die de objectvorm stuurt.
+            const ouder = safeUrl(typeof o.inReplyTo === 'string' ? o.inReplyTo : (o.inReplyTo && o.inReplyTo.id)) || null;
+            const r = db.prepare(`INSERT OR IGNORE INTO ap_mentions (slug, object_uri, note_url, actor_uri, actor_name, actor_handle, actor_icon, actor_url, content, published, in_reply_to, help_request, wave, has_guardians, emoji_json, actor_emoji_json, media_json, created_at)
+                                  VALUES (?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,CURRENT_TIMESTAMP)`)
+              .run(slug, o.id, safeUrl(o.url) || null, actorUri, ai.name, ai.handle, ai.icon, ai.url, html, o.published || null, ouder, help ? 1 : 0, wave ? 1 : 0, hasG ? 1 : 0,
                 extractEmojiTags(o.tag), emojiJsonOf(ai.emojis), mediaFromNote(o));
             if (r.changes) {

src/services/ap-timeline.js

@@ -334 +334 @@
 const MESSAGE_COLUMNS = `
       m.object_uri, m.note_url, m.actor_uri, m.actor_name, m.actor_handle, m.actor_icon, m.actor_url,
-      m.content, m.published, m.created_at, m.wave, m.help_request,
+      m.content, m.published, m.created_at, m.wave, m.help_request, m.in_reply_to,
       m.emoji_json, m.actor_emoji_json, m.media_json, m.quote_json, m.embed_json`;