Changeset 7d19465 in Klonkt

Timestamp:

07/17/2026 02:27:52 AM (8 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

f1a23b8

Parents:

92a2c46

git-author:

Robin <roboburr@…> (07/17/2026 02:27:34 AM)

git-committer:

Robin <roboburr@…> (07/17/2026 02:27:52 AM)

Message:

Fix: boosted video-only posts keep their media in the Circle (Loops)

Boosting a video-only post lost its media twice over: resolveRemoteNote only
surfaced image/* attachments (images list), so upsertBoostedNote stored
media_json=[] for a Loops video; and its refresh-UPDATE clobbered an existing
good media_json (cached via following) with that empty set. resolveRemoteNote
now also returns full typed media via mediaFromNote (images stays image-only
for the interact preview); upsertBoostedNote prefers the typed media and its
refresh keeps cached media when the resolve yielded none. Verified live: the
reported Loops file is 2.9MB with moov in reach, and a seeded boosted video row
now renders a grid tile with a real generated poster (200 image/webp, 13KB).
Covered by test/boost-media.test.js (63 tests green). Reported by Robin.

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

Files:

• CHANGELOG.de.md (modified) (1 diff)
• CHANGELOG.md (modified) (1 diff)
• CHANGELOG.nl.md (modified) (1 diff)
• src/services/ActivityPubService.js (modified) (3 diffs)
• test/boost-media.test.js (added)

CHANGELOG.de.md

@@ -21 +21 @@
 
 ### Behoben
+- **Ein geboosteter Videobeitrag behält sein Video im Zirkel.** Das Boosten
+  eines Nur-Video-Beitrags (Loops.video) speicherte ihn ohne Medien, sodass der
+  Zirkel eine nackte Textkachel statt eines Video-Thumbnails zeigte; erneutes
+  Boosten konnte das Video sogar aus einer bereits gecachten Kopie löschen.
+  Boosts tragen jetzt die vollen typisierten Medien, und ein Refresh löscht
+  gecachte Medien nicht mehr.
 - **Der Titel der Interaktionsseite folgt deiner Sprache.** "Interacteer via de
   fediverse" stand fest auf Niederländisch im Browser-Tab, auch auf einer

CHANGELOG.md

@@ -20 +20 @@
 
 ### Fixed
+- **A boosted video post keeps its video in the Circle.** Boosting a video-only
+  post (Loops.video) stored it without its media, so the Circle showed a bare
+  text tile instead of a video thumbnail; re-boosting could even wipe the video
+  from an already-cached copy. Boosts now carry the full typed media, and a
+  refresh never erases cached media.
 - **The interact page title follows your language.** "Interacteer via de
   fediverse" was hardcoded Dutch in the browser tab, even on an English site.

CHANGELOG.nl.md

@@ -20 +20 @@
 
 ### Opgelost
+- **Een geboostte videopost houdt zijn video in de Cirkel.** Een video-only post
+  (Loops.video) boosten sloeg hem op zonder media, waardoor de Cirkel een kale
+  teksttegel toonde in plaats van een videothumbnail; opnieuw boosten kon de
+  video zelfs wissen uit een al-gecachte kopie. Boosts dragen nu de volledige
+  getypeerde media mee, en een refresh wist gecachte media nooit meer.
 - **De titel van de interactiepagina volgt je taal.** "Interacteer via de
   fediverse" stond hardcoded in het Nederlands in het browsertabblad, ook op een

src/services/ActivityPubService.js

@@ -1828 +1828 @@
     cw: note.summary || '',
     images,
+    // Full typed media (incl. video/mp4) for the timeline cache. `images` above is
+    // image-only for the interact page preview; a boosted video-only post (Loops)
+    // lost its media entirely because upsertBoostedNote only saw `images`.
+    media: mediaFromNote(note),
     threadInboxes,                                          // every ancestor author's inbox
     localPostId: localTgt ? localTgt.post_id : '',          // our post this belongs to (if any)
@@ -2014 +2018 @@
   if (!slug || !note || !note.object_uri) return;
   const id = note.object_uri;
-  const media = JSON.stringify((note.images || []).map((u) => ({ url: u, type: 'image/jpeg' })));
+  // Prefer the full typed media (incl. video/mp4 — a Loops boost is video-only and
+  // rendered a bare text tile); fall back to the image-only list for older callers.
+  const media = (note.media && note.media !== '[]')
+    ? note.media
+    : JSON.stringify((note.images || []).map((u) => ({ url: u, type: 'image/jpeg' })));
   try {
     const r = tlStmts().ins.run(id, slug, note.actor_uri || '', note.actor_name || '', note.actor_handle || '',
@@ -2023 +2031 @@
       // resolved note. Without this a row cached without its cover (or with
       // stale content) stayed stale forever — even boosting again didn't heal it.
-      db.prepare('UPDATE ap_timeline SET content = ?, media_json = ?, nsfw = ?, cw = ?, url = COALESCE(?, url) WHERE slug = ? AND id = ?')
-        .run(note.content || '', media, note.sensitive ? 1 : 0, note.cw || null, note.url || null, slug, id);
+      // Keep the CACHED media when the resolve yielded none: an empty re-resolve
+      // used to clobber a good media_json (the followed copy had the video, the
+      // boost wiped it to []).
+      db.prepare(`UPDATE ap_timeline SET content = ?, media_json = CASE WHEN ? = '[]' THEN media_json ELSE ? END,
+                  nsfw = ?, cw = ?, url = COALESCE(?, url) WHERE slug = ? AND id = ?`)
+        .run(note.content || '', media, media, note.sensitive ? 1 : 0, note.cw || null, note.url || null, slug, id);
     }
   } catch { /* ignore */ }