Changeset c628db10 in Klonkt

Timestamp:

06/26/2026 09:49:44 AM (2 months ago)

Author:

Robin Genis <roboburr@…>

Branches:

main

Children:

14e2ce1

Parents:

5ceed61

Message:

fix(fediverse): expose cover via Note.image for player-card posts (Cirkel covers)

Hosted-audio posts suppress the image attachment so Mastodon shows the player
card; that left the Cirkel/timeline cards coverless. buildNote now also sets the
Note's AS2 image (Mastodon ignores it), and handleInbox uses it as a cover
fallback. scripts/backfill-cirkel-covers.mjs re-fetches existing coverless
timeline notes to fill them in.

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

Files:

• scripts/backfill-cirkel-covers.mjs (added)
• src/services/ActivityPubService.js (modified) (2 diffs)

src/services/ActivityPubService.js

@@ -241 +241 @@
   };
   if (attachment.length) note.attachment = attachment;
+  // Playable-audio posts suppress the cover attachment (player card). Still expose
+  // the cover via AS2 `image` so card/grid consumers (the Klonkt Cirkel) can show
+  // it — Mastodon ignores a Note's `image`, so the player card is unaffected.
+  if (post.cover_image_url && playable) {
+    const cov = abs(post.cover_image_url);
+    if (cov) note.image = { type: 'Image', mediaType: mediaType(cov), url: cov };
+  }
   return note;
 }
@@ -646 +653 @@
         const ai = actorInfo(await resolveActor(actorUri), actorUri);
         const html = HtmlSanitizerService.sanitize(o.content || '');
-        const media = JSON.stringify((Array.isArray(o.attachment) ? o.attachment : []).map((a) => ({ url: safeUrl(a && a.url), type: (a && a.mediaType) || '' })).filter((m) => m.url));
+        const _atts = (Array.isArray(o.attachment) ? o.attachment : []).map((a) => ({ url: safeUrl(a && a.url), type: (a && a.mediaType) || '' })).filter((m) => m.url);
+        // Fallback cover: a Note's `image` (set when the attachment was suppressed
+        // for a player-card post, e.g. hosted-audio posts).
+        if (!_atts.some((m) => !m.type || /image/i.test(m.type)) && o.image) {
+          const _im = Array.isArray(o.image) ? o.image[0] : o.image;
+          const _iu = safeUrl(typeof _im === 'string' ? _im : (_im && _im.url));
+          if (_iu) _atts.push({ url: _iu, type: (_im && _im.mediaType) || 'image/jpeg' });
+        }
+        const media = JSON.stringify(_atts);
         for (const s of subs) {
           tlStmts().ins.run(o.id, s.slug, actorUri, ai.name, ai.handle, ai.icon, ai.url, html, o.url || null, o.published || null, media);