Changeset 3f32994 in Klonkt

Timestamp:

07/28/2026 08:47:48 AM (6 weeks ago)

Author:

Robin Genis <roboburr@…>

Branches:

main

Children:

b1512e0

Parents:

c52dc82

Message:

Link-preview backfill hing achter een her-fetch die hij niet nodig had

Een YouTube-post van sound-fabrics bleef een kale link, terwijl de resolver op
precies die inhoud gewoon een kaart met titel en thumbnail teruggaf. De backfill
in de self-heal stond namelijk NA de her-fetch van de note, en als die bron even
niet antwoordt slaat de lus de hele rij over met continue. Daar had de preview
niets mee te maken: die wordt opgelost uit de content die we al in de database
hebben. Nu staat hij vooraan in de lus, los van de origine, met zijn eigen
try/catch zodat hij de heling ook nooit blokkeert.

Dit is dezelfde soort fout als de body-cap: een afhankelijkheid toegevoegd die er
inhoudelijk niet hoort, en die stilletjes precies de gevallen wegneemt waar het
om gaat.

Changed files:
src/services/ActivityPubService.js

• embed-backfill verplaatst naar de kop van de self-heal-lus, voor fetchNoteAP
• SELFHEAL_VERSION 17 -> 18

remarks: 214 tests groen. Resolver was al direct geverifieerd op deze exacte
note-content: oembed, titel In The Air Tonight (AI Citypop), met thumbnail.

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

File:

• src/services/ActivityPubService.js (modified) (3 diffs)

src/services/ActivityPubService.js

@@ -2849 +2849 @@
 // during a flux window, e.g. a fleet-wide update), and drops notes that are gone
 // (404/410). Bump SELFHEAL_VERSION only on a release that warrants a re-sync.
-const SELFHEAL_VERSION = 17; // v17: re-resolve link previews now that OpenGraph works and big pages are read instead of refused
+const SELFHEAL_VERSION = 18; // v18: link previews no longer hang behind the note re-fetch (an unreachable origin skipped the whole row)
 async function fetchNoteAP(url) {
   try {
@@ -3162 +3162 @@
     let healed = 0, failed = 0;
     for (const r of rows) {
+      // Link previews first, and deliberately BEFORE the note re-fetch. A
+      // preview is resolved from the content we already hold, so hanging it
+      // behind a remote fetch meant one unreachable origin skipped the whole
+      // row (`continue` below) and the card never appeared. It needs nothing
+      // from the origin, so it must not depend on it.
+      if (!r.quote_json && !r.embed_json) {
+        try {
+          const ej = await resolveExternalEmbed(r.content);
+          if (ej) db.prepare('UPDATE ap_timeline SET embed_json = ? WHERE id = ?').run(ej, r.id);
+        } catch { /* best-effort, never blocks the heal */ }
+      }
       try {
         const note = await fetchNoteAP(r.id);
@@ -3185 +3196 @@
           const ai = actorInfo(await fetchActor(r.author_uri), r.author_uri);
           if (ai.emojis) { try { db.prepare('UPDATE ap_timeline SET author_emoji_json = ? WHERE id = ?').run(JSON.stringify(ai.emojis), r.id); } catch { /* ignore */ } }
-        }
-        // v16: link previews. A post from before the embed pipeline has no
-        // card at all, which is why nothing showed. Only for rows that have no
-        // quote (a quote already IS the card) and no embed yet, so this costs
-        // one page fetch per candidate and never repeats.
-        if (!r.quote_json) {
-          const ej = await resolveExternalEmbed(html || r.content).catch(() => null);
-          if (ej) { try { db.prepare('UPDATE ap_timeline SET embed_json = ? WHERE id = ?').run(ej, r.id); } catch { /* ignore */ } }
         }
         // v14: same for the booster's display name ("X boosted"). The row stores