Changeset 4838760 in Klonkt

Timestamp:

08/08/2026 05:23:46 PM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

d1075a1

Parents:

abcf51a

Message:

De kaart op je eigen post, en de composer-preview (shaer-k3f)

Het bead-uitgangspunt "serverkant is klaar" bleek half waar: AP-quotes
kregen bij publiceren een quote_uri, maar eigen posts hadden geen kolom
voor een kaart-snapshot en de outbox serveerde geen shaer:quote of
shaer:embed -- de app kon een eigen post dus nooit als kaart tonen, ook
na publicatie niet.

Drie stukken, allemaal langs de pijplijn die er al was:

OPSLAAN deliverCreate resolvet de eerste externe link net als de

inbox dat voor binnenkomende posts doet: een fediverse-
object wordt een quote-snapshot (resolveQuoteByUri, de kern
van resolveQuote maar vanaf een kale URI), anders probeert
de link een externe kaart (resolveExternalEmbed). In
posts.quote_json/embed_json, VOOR de vroege return: ook een
post zonder volgers hoort zijn kaart te krijgen, de app
leest hem uit de outbox en niet uit een bezorging.

SERVEREN de friend-poot van de outbox draagt shaer:quote en -- alleen

voor de bearer en langs zijn eigen embeds-poort --
shaer:embed. Een remote vriend krijgt de embed niet: diens
server resolvet en gate zelf bij ontvangst.

PREVIEW GET /ap/users/:slug/card?url= (bearer-only): een URL langs

exact dezelfde pijplijn, zodat de preview in de composer
nooit iets belooft dat de post niet waarmaakt. Zelfde
gate-regels als de tijdlijn.

Drie tests met gestubde fetch: extern -> embed-snapshot, fediverse ->
quote-snapshot (en niet allebei), en de preview die voor dezelfde links
dezelfde kaarten geeft. Alle 723 groen.

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

Files:

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

src/config/database.js

@@ -718 +718 @@
   // time so buildNote (sync, also used by the outbox) needs no network.
   ensureColumn('posts', 'quote_uri', 'TEXT');     // the quoted object's id
+  // De kaart op je EIGEN post (shaer-k3f): dezelfde snapshots die ap_timeline
+  // voor binnenkomende posts draagt, maar dan voor wat je zelf publiceert --
+  // zonder deze twee kan de app een eigen post nooit als kaart tonen.
+  ensureColumn('posts', 'quote_json', 'TEXT');    // FEP-044f resolved quote snapshot
+  ensureColumn('posts', 'embed_json', 'TEXT');    // externe linkkaart (oEmbed/OG), thumbnail-only
   ensureColumn('posts', 'quote_actor', 'TEXT');   // its author, so we can address them
   ensureColumn('ap_timeline', 'embed_json', 'TEXT');       // resolved EXTERNAL embed (oEmbed/provider), thumbnail-only; gated per site (sites.external_embeds)

src/routes/activitypub.js

@@ -191 +191 @@
     // entries carry, so your own cards get a header too (avatar + name).
     const me = AP.selfAuthor(baseUrl(req), site);
+    // De kaart op je eigen post (shaer-k3f): dezelfde shaer:quote/shaer:embed
+    // die de tijdlijn voor andermans posts draagt, uit de snapshots die
+    // deliverCreate bij het publiceren opsloeg. Op note-id gekoppeld, want
+    // buildOutbox sorteert en mengt tracks erdoorheen. De embed alleen voor de
+    // BEARER en langs zijn eigen poort: een remote vriend krijgt hem niet
+    // (diens server resolvet en gate zelf bij ontvangst), en een ward zonder
+    // open embeds-poort krijgt hem hier net zo min als in de tijdlijn.
+    const byNote = new Map(posts.map((p) => [AP.noteId(baseUrl(req), p.id), p]));
+    const bearerEmbeds = bearer ? (() => {
+      const isWard = (() => { try { return Guardianship.listGuardians(bearer.site.slug).length > 0; } catch { return false; } })();
+      return Guardianship.externalEmbedsAllowed(bearer.site.external_embeds, isWard)
+        ? { playback: Guardianship.externalPlaybackAllowed(bearer.site.external_playback, isWard) } : null;
+    })() : null;
     for (const it of ob.orderedItems) {
-      if (it && it.object && typeof it.object === 'object') it.object['shaer:author'] = me;
+      if (it && it.object && typeof it.object === 'object') {
+        it.object['shaer:author'] = me;
+        const row = byNote.get(it.object.id);
+        if (row) {
+          it.object['shaer:quote'] = AP.timelineQuote(row.quote_json);
+          if (bearerEmbeds) it.object['shaer:embed'] = AP.timelineEmbed(row.embed_json, { playback: bearerEmbeds.playback });
+        }
+      }
     }
   }
@@ -813 +833 @@
 
 // ── Replies collection ── lets remote servers fetch a post's whole thread.
+// ── De composer-preview (shaer-k3f): een URL wordt alvast een kaart ──
+//
+// Bearer-only, net als de thread: dit is de eigen app die tijdens het typen
+// vraagt wat een link gaat worden. Dezelfde pijplijn als publiceren, dus de
+// preview kan niet iets beloven dat de post niet waarmaakt. De embed gaat
+// langs de eigen poort van de lezer -- een ward zonder open embeds-poort
+// krijgt in de composer geen kaart die zijn feed hem ook niet zou tonen.
+router.get('/ap/users/:slug/card', async (req, res) => {
+  const auth = OAuth.verifyBearer(req.headers.authorization);
+  if (!auth || auth.site.slug !== req.params.slug) return res.status(403).end();
+  const uit = await AP.previewCard(String(req.query.url || ''));
+  const isWard = (() => { try { return Guardianship.listGuardians(auth.site.slug).length > 0; } catch { return false; } })();
+  const embedsAllowed = Guardianship.externalEmbedsAllowed(auth.site.external_embeds, isWard);
+  const playback = embedsAllowed && Guardianship.externalPlaybackAllowed(auth.site.external_playback, isWard);
+  AP.sendAP(res, {
+    '@context': AP.AP_CONTEXT,
+    'shaer:quote': AP.timelineQuote(uit.quoteJson),
+    'shaer:embed': embedsAllowed ? AP.timelineEmbed(uit.embedJson, { playback }) : undefined,
+  }, 'private, no-store');
+});
+
 // ── De thread onder een post (shaer-tqz): ophalen, niet bewaren ────
 //

src/services/ActivityPubService.js

@@ -2955 +2955 @@
       post2 = { ...post2, quote_uri: q.uri, quote_actor: q.actor || null };
     }
+  }
+  // De kaart op de eigen post (shaer-k3f), langs dezelfde pijplijn als een
+  // binnenkomende: een fediverse-quote wordt een quote-snapshot, anders
+  // probeert de link een externe kaart. VOOR de vroege return hieronder, want
+  // ook een post zonder volgers hoort zijn kaart te krijgen -- de app leest
+  // hem uit de outbox, niet uit een bezorging. Best-effort en eenmalig: wat
+  // hier niet lukt blijft een kale link, precies wat het was.
+  if (!post2.quote_json && !post2.embed_json) {
+    try {
+      if (post2.quote_uri) {
+        const qj = await resolveQuoteByUri(post2.quote_uri);
+        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 || '');
+        if (ej) { db.prepare('UPDATE posts SET embed_json = ? WHERE id = ?').run(ej, post.id); post2 = { ...post2, embed_json: ej }; }
+      }
+    } catch { /* een kaart is nooit een blokkade voor de post zelf */ }
   }
   if (post2.quote_actor) {
@@ -4996 +5013 @@
 }
 
+/**
+ * De composer-preview (shaer-k3f): één URL langs exact dezelfde pijplijn als
+ * publiceren, zodat wat de preview toont ook is wat de post krijgt. Twee
+ * uitkomsten, hoogstens een gevuld: een AP-object wordt een quote-snapshot,
+ * een externe link probeert een kaart. Beide als JSON-string, dezelfde vorm
+ * als de kolommen -- de route serveert ze door timelineQuote/timelineEmbed en
+ * de gate, net als de tijdlijn.
+ */
+export async function previewCard(url) {
+  if (!/^https?:\/\//i.test(String(url || ''))) return {};
+  const html = `<a href="${String(url).replace(/"/g, '&quot;')}">x</a>`;
+  const q = await resolveOwnQuote(html);
+  if (q && q.uri) {
+    const quoteJson = await resolveQuoteByUri(q.uri).catch(() => null);
+    if (quoteJson) return { quoteJson };
+  } else {
+    const embedJson = await resolveExternalEmbed(html).catch(() => null);
+    if (embedJson) return { embedJson };
+  }
+  return {};
+}
+
 /** The first http(s) link in sanitized note HTML that is not a mention/hashtag. */
 export function firstExternalUrl(html) {
@@ -5051 +5090 @@
   const url = quoteHrefOf(note);
   if (!url) return null;
+  return resolveQuoteByUri(url);
+}
+
+/** Hetzelfde snapshot, maar vanaf een kale URI: eigen posts en de
+ *  composer-preview (shaer-k3f) kennen alleen de link, niet de tag-vorm. */
+async function resolveQuoteByUri(url) {
   const q = await apGetJson(url);
   if (!q || typeof q !== 'object') return null;
@@ -6270 +6315 @@
   acceptGatedFollow, rejectGatedFollow, isWardGuardian, outboxAudience, sendFollowDecision,
   gateOutgoingFollow, performApprovedFollow,
-  parseOwnPoll, pollTally, ownPollView, deliverPollUpdate, maybeCrawlThread, sendReport, localMentionSlugs,
+  parseOwnPoll, pollTally, ownPollView, deliverPollUpdate, maybeCrawlThread, sendReport, localMentionSlugs, previewCard,
   autoBoostCount, boostedCount, setReaction, getReaction, getReactionsFor, canonicalReactionUri, migrateReactions, upsertBoostedNote, getCirkelPosts, getCirkelMembers, selfHealTimeline,
   getNotifications, listBlocks, isBlockedAny, blockTarget, unblock,