Changeset 76bd53c in Klonkt

Timestamp:

09/30/2026 08:22:57 AM (3 hours ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

31680c3

Parents:

ca260d1

Message:

Outbox serves the same Note as /ap/notes/:id — no more column list

The hub could not find soundfabrics' hashtags. Fetched on its own
(/ap/notes/:id) a post carries #kawaii; read from the outbox, it does not.
outboxSlice loaded posts with a hand-kept column list, and that list fails
silently: a forgotten column is undefined, buildNote decides without it,
and nothing errors. tags was missing — and so were poll_json (a poll left
the outbox as a plain note), quote_uri/quote_actor (a quote post lost its
quote), cover_alt and language.

This was the fourth time. fan_only once sent a friends-only post out as
public; paid once put the full text of a paid post in the outbox. Each fix
added a name to the list and a warning above it. The note route has always
used SELECT *, so the outbox now does the same: the two paths can no longer
drift apart.

The new test compares the WHOLE object, outbox against the single Note,
rather than a handful of fields — a column added later that the outbox
doesn't carry fails it immediately. Verified it goes red on the old code.

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

Files:

• src/services/ActivityPubService.js (modified) (1 diff)
• test/outbox-gelijk-aan-note.test.js (added)

src/services/ActivityPubService.js

@@ -1159 +1159 @@
   const gaten = (n) => Array.from({ length: n }, () => '?').join(',');
   const posts = postIds.length ? db.prepare(
-    // fan_only en ap_visibility MOETEN mee. buildNote adresseert hierop, en
-    // zonder deze twee kolommen is post.fan_only altijd undefined: elke
-    // fan-only post ging dan de outbox uit met to: as:Public, terwijl hij
-    // alleen aan vrienden geserveerd wordt. Een volger kreeg dus een
-    // vrienden-post met een publiek etiket erop, en die mag hij dan publiek
-    // boosten. Gevonden tijdens de FEP-1580 end-to-end test (shaer-fuyo).
-    // EN paid + excerpt, om exact dezelfde reden (Barts melding, 15-8). Zonder
-    // `paid` is post.paid hier `undefined`, dan slaat buildNote zijn redactie
-    // over en gaat de VOLLEDIGE tekst van een betaalde post de outbox uit. Zo
-    // kwam een post via een hub-actor gewoon te lezen. `excerpt` moet mee omdat
-    // de teaser daaruit komt; zonder dat veld valt hij terug op de eerste
-    // alinea van precies de tekst die verborgen hoort te blijven.
+    // ALLE KOLOMMEN, zoals /ap/notes/:id ook doet. Hier stond een lijst, en
+    // die faalde stil: een vergeten kolom is `undefined` en geen fout, en
+    // buildNote besluit dan zonder dat veld. Het ging vier keer mis voordat
+    // deze regel er stond --
+    //   fan_only/ap_visibility: een vriendenpost ging de deur uit als publiek
+    //     (shaer-fuyo);
+    //   paid/excerpt: de VOLLEDIGE tekst van een betaalde post stond in de
+    //     outbox (Barts melding, 15-8);
+    //   tags, poll_json, quote_uri/quote_actor, cover_alt, language: geen
+    //     hashtags, een peiling als gewone post, een citaat zonder citaat
+    //     (Robin, 30-9 -- de hub vond de tags van soundfabrics niet, want die
+    //     staan niet in de tekst en kwamen alleen via de outbox binnen).
     //
-    // Dit is een KOLOMMENLIJST, en die faalt stil: een vergeten kolom is
-    // `undefined` en niet een fout. Wie hier een veld toevoegt waar buildNote
-    // op beslist, moet het HIER ook toevoegen.
-    `SELECT id, slug, title, excerpt, content, cover_image_url, cover_video_url, nsfw, content_warning,
-            c2s_attachments, quote_json, embed_json, published_at, created_at,
-            fan_only, ap_visibility, paid, paid_min_cents
-       FROM posts WHERE id IN (${gaten(postIds.length)})`).all(...postIds) : [];
+    // De outbox en de losse Note horen HETZELFDE object te zijn; wie een post
+    // op twee manieren ophaalt, hoort geen twee verschillende berichten te
+    // zien. test/outbox-gelijk-aan-note.test.js legt dat vast, voor elk veld,
+    // ook voor velden die er later bijkomen.
+    `SELECT * FROM posts WHERE id IN (${gaten(postIds.length)})`).all(...postIds) : [];
   const tracks = trackIds.length ? db.prepare(
     `SELECT ${TRACK_KOLOMMEN}