@linktr.ee/messaging-react 4.4.4 → 4.4.5-rc-1788226184

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@linktr.ee/messaging-react",
-  "version": "4.4.4",
+  "version": "4.4.5-rc-1788226184",
   "description": "React messaging components built on messaging-core for web applications",
   "type": "module",
   "main": "dist/index.cjs",
@@ -50,8 +50,8 @@
     "chromatic:local": "yarn storybook:build && chromatic"
   },
   "dependencies": {
-    "@linktr.ee/messaging-core": "^2.6.0",
-    "@linktr.ee/messaging-taxonomy": "^0.11.0",
+    "@linktr.ee/messaging-core": "2.6.0-rc-1788226184",
+    "@linktr.ee/messaging-taxonomy": "0.11.0-rc-1788226184",
     "@phosphor-icons/react": "^2.1.10"
   },
   "devDependencies": {

package/src/components/CustomMessage/StreamAttachmentMessage.test.tsx

@@ -266,6 +266,37 @@ describe('StreamAttachmentMessage', () => {
     )
   })
 
+  it('renders a scraped page typed audio by its og:audio preview as a link card', () => {
+    // Spotify episode: enrichment types it `audio` and puts the 30s clip in
+    // `asset_url`, but the shared thing is still a link (MES-1359).
+    renderWithProviders(
+      <StreamAttachmentMessage
+        groupPosition="single"
+        isMyMessage
+        message={message([
+          {
+            type: 'audio',
+            og_scrape_url:
+              'https://open.spotify.com/episode/61syKMXdpkEnkHr43ZsrG2',
+            asset_url: 'https://p.scdn.co/mp3-preview/abc123',
+            image_url: 'https://i.scdn.co/image/artwork.jpg',
+            title: 'The Lazarus Heist S1: Trailer',
+            text: 'Cyber Hack · Episode',
+          },
+        ])}
+      />
+    )
+
+    expect(
+      screen.getByRole('link', { name: /lazarus heist/i })
+    ).toHaveAttribute(
+      'href',
+      'https://open.spotify.com/episode/61syKMXdpkEnkHr43ZsrG2'
+    )
+    expect(screen.getByRole('img', { name: /lazarus heist/i })).toBeVisible()
+    expect(screen.queryByTestId('audio-attachment')).toBeNull()
+  })
+
   it('renders the accompanying message text next to a media attachment', () => {
     renderWithProviders(
       <StreamAttachmentMessage

package/src/components/MediaMessage/index.tsx

@@ -2,9 +2,11 @@ import type { Attachment } from 'stream-chat'
 
 /**
  * True when an attachment is a link preview (OG card) rather than media:
- * either Stream's native `link` type, or a non-empty (trimmed) `og_scrape_url`
- * with no `asset_url`. A whitespace-only `og_scrape_url` is not a link.
+ * either Stream's native `link` type, or a non-empty (trimmed) `og_scrape_url`.
+ * A whitespace-only `og_scrape_url` is not a link. Only enrichment sets
+ * `og_scrape_url`, so it stays authoritative even when the scrape also filled
+ * `asset_url` with a preview clip (Spotify).
  */
 export function isLinkAttachment(a: Attachment): boolean {
-  return a.type === 'link' || (!!a.og_scrape_url?.trim() && !a.asset_url)
+  return a.type === 'link' || !!a.og_scrape_url?.trim()
 }

package/src/components/MediaMessage/isLinkAttachment.test.ts

@@ -18,10 +18,22 @@ describe('isLinkAttachment', () => {
     expect(isLinkAttachment({ og_scrape_url: '   ' } as Attachment)).toBe(false)
   })
 
-  it('is false when og_scrape_url accompanies an asset_url (media, not a link)', () => {
+  it('is true for a scraped page whose og:audio preview typed it as audio', () => {
+    // Spotify: the clip lands in `asset_url`, but it's still a link (MES-1359).
     expect(
       isLinkAttachment({
-        og_scrape_url: 'https://example.com',
+        type: 'audio',
+        og_scrape_url:
+          'https://open.spotify.com/episode/61syKMXdpkEnkHr43ZsrG2',
+        asset_url: 'https://p.scdn.co/mp3-preview/abc123',
+      } as Attachment)
+    ).toBe(true)
+  })
+
+  it('is false for uploaded media, which never carries an og_scrape_url', () => {
+    expect(
+      isLinkAttachment({
+        type: 'video',
         asset_url: 'https://cdn.example.com/file.mp4',
       } as Attachment)
     ).toBe(false)