Changeset 2923a95 in Klonkt

Timestamp:

06/25/2026 05:57:01 AM (3 months ago)

Author:

Robin Genis <roboburr@…>

Branches:

main

Children:

30271e6

Parents:

ca25f360

Message:

feat(fediverse): audio posts skip image attachment so Mastodon shows the player card

A media attachment and a link/player card are mutually exclusive on Mastodon, so
the cover image was winning over the player card. Audio posts now omit image
attachments (the cover still shows as the player card's og:image thumbnail), so
Mastodon renders the inline player. Non-audio posts keep their image attachments.

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

File:

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

src/services/ActivityPubService.js

@@ -116 +116 @@
     return ({ jpg: 'image/jpeg', jpeg: 'image/jpeg', png: 'image/png', gif: 'image/gif', webp: 'image/webp', avif: 'image/avif' })[(e || '').toLowerCase()] || 'image/jpeg';
   };
+  const hadAudio = /\[\[(track|album|playlist):/i.test(post.content || '');
   const urls = [];
-  if (post.cover_image_url) urls.push(abs(post.cover_image_url));
+  // Audio posts suppress image attachments so Mastodon renders the player CARD
+  // (twitter:player) instead of the cover — on Mastodon a media attachment and a
+  // link/player card are mutually exclusive, and the inline player is the point.
+  if (post.cover_image_url && !hadAudio) urls.push(abs(post.cover_image_url));
   let body = post.content || '';
-  for (const m of body.matchAll(/<img\b[^>]*\bsrc="([^"]+)"[^>]*>/gi)) urls.push(abs(m[1]));
+  if (!hadAudio) for (const m of body.matchAll(/<img\b[^>]*\bsrc="([^"]+)"[^>]*>/gi)) urls.push(abs(m[1]));
   body = body.replace(/<img\b[^>]*>/gi, '');
   // Audio shortcodes: do NOT federate the raw audio file — Klonkt deliberately
@@ -132 +136 @@
     for (const m of body.matchAll(/\[\[album:([^\]]+)\]\]/g)) audioLabels.push(m[1].trim());
   } catch { /* non-fatal */ }
-  const hadAudio = /\[\[(track|album|playlist):/i.test(body);
   body = body.replace(/\[\[(track|album|playlist):[^\]]+\]\]/gi, '');
   if (hadAudio) {