Changeset d1075a1 in Klonkt

Timestamp:

08/08/2026 05:41:58 PM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

3a0ca0f

Parents:

4838760

Message:

Eerst bakken, dan linken zoeken -- en de verloren kolomselectie terug

Twee vondsten van het toestel (shaer-k3f, vervolg op 4838760):

EERST BAKKEN. firstExternalUrl leest <a href>-ankers, en de web-editor
bakt die er bij het opslaan al in -- maar een post uit de APP is platte
tekst waarin de URL nog geen anker is. Het C2S-pad zag dus nooit een
link en een app-post kreeg nooit een kaart, terwijl de preview hem net
wel beloofd had. deliverCreate bakt de inhoud nu voor het zoeken; de
test dekt voortaan het app-geval (platte tekst) in plaats van het
editor-geval.

DE VERLOREN SELECTIE. De kolomtoevoeging aan de outbox-query is bij het
invlechten van parallel werk verloren gegaan: de friend-poot vroeg
row.embed_json aan rijen waar de query hem nooit in had gestopt. Een
diagnoseregel op dev wees het exact aan (rij gevonden, poort open,
embed: false). De query draagt quote_json en embed_json nu echt, en de
route is weer schoon.

Op het toestel bewezen: preview in de composer, posten, en de eigen
feedkaart draagt de kaart. Alle 723 groen.

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

Files:

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

src/routes/activitypub.js

@@ -179 +179 @@
   const fanClause = audience === 'friend' ? '' : "AND (fan_only IS NULL OR fan_only = 0)";
   const posts = db.prepare(
-    `SELECT id, slug, title, content, cover_image_url, cover_video_url, nsfw, content_warning, c2s_attachments, published_at, created_at
+    `SELECT id, slug, title, content, cover_image_url, cover_video_url, nsfw, content_warning, c2s_attachments, quote_json, embed_json, published_at, created_at
      FROM posts WHERE site_id = ? AND status = 'published' ${fanClause}
      ORDER BY COALESCE(published_at, created_at) DESC LIMIT 20`

src/services/ActivityPubService.js

@@ -2949 +2949 @@
   // IS the notification.
   const quoteInboxes = [];
+  // EERST BAKKEN, dan pas linken zoeken (shaer-k3f, gevonden op het toestel):
+  // firstExternalUrl leest <a href>-ankers, en de web-editor bakt die er bij
+  // het opslaan al in -- maar een post uit de APP is platte tekst waarin de
+  // URL nog geen anker is. Zonder deze bak zag het C2S-pad dus nooit een link
+  // en kreeg een app-post nooit een kaart, terwijl de preview hem net wel
+  // beloofd had.
+  const gebakken = bakePostContent(post2.content || '');
   if (post2.quote_uri === undefined || post2.quote_uri === null) {
-    const q = await resolveOwnQuote(post2.content || '');
+    const q = await resolveOwnQuote(gebakken);
     if (q) {
       try { db.prepare('UPDATE posts SET quote_uri = ?, quote_actor = ? WHERE id = ?').run(q.uri, q.actor || null, post.id); } catch { /* ignore */ }
@@ -2968 +2975 @@
         if (qj) { db.prepare('UPDATE posts SET quote_json = ? WHERE id = ?').run(qj, post.id); post2 = { ...post2, quote_json: qj }; }
       } else {
-        const ej = await resolveExternalEmbed(post2.content || '');
+        const ej = await resolveExternalEmbed(gebakken);
         if (ej) { db.prepare('UPDATE posts SET embed_json = ? WHERE id = ?').run(ej, post.id); post2 = { ...post2, embed_json: ej }; }
       }

test/own-card.test.js

@@ -46 +46 @@
 
 test('een externe link in een eigen post wordt een embed-snapshot op de post', async () => {
+  // PLATTE tekst, geen anker: zo komt een post uit de app binnen (het
+  // C2S-pad bakt niet voor het opslaan). Op het toestel bleef precies dit
+  // geval een kale link terwijl de web-editor wel een kaart kreeg.
   db.prepare("INSERT INTO posts (id, site_id, author_id, slug, title, content, status) VALUES ('p1','s1','u1','p1','', ?, 'published')")
-    .run(`<p>Kijk: <a href="${EXTERN}">${EXTERN}</a></p>`);
+    .run(`<p>Kijk: ${EXTERN}</p>`);
   const post = db.prepare("SELECT * FROM posts WHERE id = 'p1'").get();
   await AP.deliverCreate(site, post);