@linktr.ee/messaging-react 3.25.0-rc-1785443162 → 3.25.0

package/dist/index.cjs

@@ -1,2 +1,2 @@
-"use strict";Object.defineProperty(exports,Symbol.toStringTag,{value:"Module"});const e=require("./index-CWpDkA50.cjs");exports.ActionButton=e.ActionButton;exports.Avatar=e.Avatar;exports.ChannelEmptyState=e.ChannelEmptyState;exports.ChannelList=e.ChannelList;exports.ChannelView=e.ChannelView;exports.CustomMessageProvider=e.CustomMessageProvider;exports.FaqList=e.FaqList;exports.FaqListItem=e.FaqListItem;exports.LinkAttachment=e.LinkAttachment;exports.LockedAttachment=e.LockedAttachment;exports.MediaMessage=e.MediaMessage;exports.MessageAttachment=e.MessageAttachment;exports.MessageBubble=e.MessageBubble;exports.MessageVoteButtons=e.MessageVoteButtons;exports.MessagingProvider=e.MessagingProvider;exports.MessagingShell=e.MessagingShell;exports.buildCompactMetaLabel=e.buildCompactMetaLabel;exports.formatFileSize=e.formatFileSize;exports.formatRelativeTime=e.formatRelativeTime;exports.getFileExtensionLabel=e.getFileExtensionLabel;exports.getMessageDisplayText=e.getMessageDisplayText;exports.isLinkAttachment=e.isLinkAttachment;exports.isUuidLike=e.isUuidLike;exports.messageAttachmentGroupPositionFromStream=e.bubbleGroupPositionFromStream;exports.normalizeLanguageCode=e.normalizeLanguageCode;exports.optimizeMessagingAttachmentUrl=e.optimizeMessagingAttachmentUrl;exports.resolveConversationParticipant=e.resolveConversationParticipant;exports.resolveLinkAttachment=e.resolveLinkAttachment;exports.resolveMediaFromMessage=e.resolveMediaFromMessage;exports.resolveParticipantDisplayName=e.resolveParticipantDisplayName;exports.useComposerLocked=e.useComposerLocked;exports.useCustomMessage=e.useCustomMessage;exports.useMessageVote=e.useMessageVote;exports.useMessaging=e.useMessaging;
+"use strict";Object.defineProperty(exports,Symbol.toStringTag,{value:"Module"});const e=require("./index-QulHlER7.cjs");exports.ActionButton=e.ActionButton;exports.Avatar=e.Avatar;exports.ChannelEmptyState=e.ChannelEmptyState;exports.ChannelList=e.ChannelList;exports.ChannelView=e.ChannelView;exports.CustomMessageProvider=e.CustomMessageProvider;exports.FaqList=e.FaqList;exports.FaqListItem=e.FaqListItem;exports.LinkAttachment=e.LinkAttachment;exports.LockedAttachment=e.LockedAttachment;exports.MediaMessage=e.MediaMessage;exports.MessageAttachment=e.MessageAttachment;exports.MessageBubble=e.MessageBubble;exports.MessageVoteButtons=e.MessageVoteButtons;exports.MessagingProvider=e.MessagingProvider;exports.MessagingShell=e.MessagingShell;exports.buildCompactMetaLabel=e.buildCompactMetaLabel;exports.formatFileSize=e.formatFileSize;exports.formatRelativeTime=e.formatRelativeTime;exports.getFileExtensionLabel=e.getFileExtensionLabel;exports.getMessageDisplayText=e.getMessageDisplayText;exports.isLinkAttachment=e.isLinkAttachment;exports.isUuidLike=e.isUuidLike;exports.messageAttachmentGroupPositionFromStream=e.bubbleGroupPositionFromStream;exports.normalizeLanguageCode=e.normalizeLanguageCode;exports.optimizeMessagingAttachmentUrl=e.optimizeMessagingAttachmentUrl;exports.resolveConversationParticipant=e.resolveConversationParticipant;exports.resolveLinkAttachment=e.resolveLinkAttachment;exports.resolveMediaFromMessage=e.resolveMediaFromMessage;exports.resolveParticipantDisplayName=e.resolveParticipantDisplayName;exports.useComposerLocked=e.useComposerLocked;exports.useCustomMessage=e.useCustomMessage;exports.useMessageVote=e.useMessageVote;exports.useMessaging=e.useMessaging;
 //# sourceMappingURL=index.cjs.map

package/dist/index.js

@@ -1,4 +1,4 @@
-import { a as e, b as t, C as i, c as o, d as n, e as m, F as r, f as g, L as l, h as M, M as c, i as u, j as L, k as h, l as p, m as d, n as v, p as C, q as A, s as k, t as F, u as b, v as P, w as f, x, o as y, y as z, z as B, B as S, D as q, E as D, G as E, H as V, I as w } from "./index-Hlj3mc6w.js";
+import { a as e, b as t, C as i, c as o, d as n, e as m, F as r, f as g, L as l, h as M, M as c, i as u, j as L, k as h, l as p, m as d, n as v, p as C, q as A, s as k, t as F, u as b, v as P, w as f, x, o as y, y as z, z as B, B as S, D as q, E as D, G as E, H as V, I as w } from "./index-CitR7B66.js";
 export {
   e as ActionButton,
   t as Avatar,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@linktr.ee/messaging-react",
-  "version": "3.25.0-rc-1785443162",
+  "version": "3.25.0",
   "description": "React messaging components built on messaging-core for web applications",
   "type": "module",
   "main": "dist/index.cjs",
@@ -50,7 +50,7 @@
   },
   "dependencies": {
     "@linktr.ee/component-library": "11.8.6",
-    "@linktr.ee/messaging-core": "2.3.0-rc-1785443162",
+    "@linktr.ee/messaging-core": "^2.2.0",
     "@linktr.ee/messaging-taxonomy": "^0.3.0",
     "@phosphor-icons/react": "^2.1.10",
     "culori": "^4.0.2"

package/src/components/LinkAttachment/LinkAttachment.test.tsx

@@ -234,39 +234,40 @@ describe('LinkAttachment accent chin (MES-1349, MES-1388)', () => {
     expect(container.querySelector('video')).not.toHaveAttribute('poster')
   })
 
-  it('derives audio artwork into the surface, and inks the flat fill when there is none', async () => {
-    // MES-1349 refused the accent here because the hero collapses to the native
-    // player and the artwork is never shown. The colour is derived now, so it
-    // cannot mislead about legibility, and artwork is what distinguishes one
-    // track from another — so it paints. Reachable through Stream, which maps
-    // any playable link attachment to `featured`.
+  it('leaves a playable audio card on its flat fill, and inks that fill dark', async () => {
+    // The accent stays out: the hero collapses to the native player, so artwork
+    // never appears and a colour taken from it would describe an image the
+    // viewer cannot see (MES-1349). Colouring audio from its artwork belongs to
+    // the audio tickets, not here.
+    //
+    // The ink does change. `AUDIO_BG_CLASS` is near-white and is applied to a
+    // *sent* card, whose variant ink is white — 1.13:1, invisible. Ink now
+    // follows the surface the card was given rather than the variant (MES-1388).
     const AUDIO = {
       mimeType: 'audio/mpeg',
       sourceUrl: 'https://cdn.example.com/clip.mp3',
     }
 
-    const withArtwork = renderWithProviders(
+    const { container, unmount } = renderWithProviders(
       <LinkAttachment.Sent {...CARD_PROPS} {...AUDIO} accentColor={ACCENT} />
     )
-    const painted = withArtwork.container.firstElementChild as HTMLElement
-    expect(painted.style.backgroundColor).not.toBe('')
+    const sent = container.firstElementChild as HTMLElement
+    expect(sent.style.backgroundColor).toBe('')
+    expect(sent.className).toContain(AUDIO_BG_CLASS)
+    expect(screen.getByText(CARD_PROPS.title)).toHaveStyle({ color: '#040403' })
     expect(screen.queryByRole('img')).toBeNull()
-    withArtwork.unmount()
+    unmount()
 
-    // No artwork: the flat audio fill stays, and this is the MES-1388 bug —
-    // `AUDIO_BG_CLASS` is near-white and lands on a *sent* card, whose variant
-    // ink is white. Ink must follow the fill it was given, not the variant.
-    const bare = renderWithProviders(
-      <LinkAttachment.Sent
+    renderWithProviders(
+      <LinkAttachment.Received
         {...CARD_PROPS}
         {...AUDIO}
-        thumbnailUrl={undefined}
+        accentColor={ACCENT}
       />
     )
-    const flat = bare.container.firstElementChild as HTMLElement
-    expect(flat.style.backgroundColor).toBe('')
-    expect(flat.className).toContain(AUDIO_BG_CLASS)
-    expect(screen.getByText(CARD_PROPS.title)).toHaveStyle({ color: '#040403' })
+    const received = await screen.findByTestId('link-attachment')
+    expect(received.style.backgroundColor).toBe('')
+    expect(received.className).toContain(AUDIO_BG_CLASS)
   })
 
   it('drops the accent when the featured hero has no image to show', async () => {

package/src/components/LinkAttachment/components/_shared/useChinPalette.ts

@@ -105,18 +105,17 @@ export function useChinPalette({
   // Truthiness rather than `!= null` so this tracks `CardThumbnail`'s own
   // `thumbnailUrl ?` branch, which renders the placeholder for `''` too.
   //
-  // Playable audio is included even though its hero collapses to the native
-  // `<audio>` player and the artwork is never displayed. MES-1349 excluded it
-  // on the grounds that an accent is a claim about an image the viewer can see;
-  // the colour is now derived rather than painted raw, so it can no longer
-  // mislead about legibility, and an audio attachment with artwork is exactly
-  // the case where colour distinguishes one track from another. The cost is
-  // that a 404 artwork still tints the card here: the `<img>` never mounts, so
-  // `onImageError` cannot fire to withdraw it.
-  const hasArtwork = layout === 'featured' && !!thumbnailUrl && !heroFailed
+  // Playable audio stays excluded, as MES-1349 decided: its hero collapses to
+  // the native `<audio>` player, so the artwork never appears and an accent
+  // taken from it would be a claim about an image the viewer cannot see. The
+  // `<img>` never mounts either, so `onImageError` could not withdraw the colour
+  // if the artwork 404'd. Colouring audio cards from their artwork is a change
+  // to the audio tickets (MES-1360 / MES-1361), not to this one.
   const isAudio = isPlayableAudio(mimeType, sourceUrl)
+  const hasHero =
+    layout === 'featured' && !!thumbnailUrl && !isAudio && !heroFailed
   const surface =
-    hasArtwork && status === 'ready'
+    hasHero && status === 'ready'
       ? accentSurface(accentColor, variant)
       : undefined