fix(rich-markdown-editor): unique widget key per duplicate embed URL

Key embed widgets by source + per-source occurrence index so two standalone
links to the same URL render as two distinct players instead of collapsing into
one, while keeping the key stable across unrelated edits (no iframe reload).

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/rich-markdown-editor/embed/link-embed.test.ts

@@ -34,6 +34,11 @@ describe('LinkEmbed', () => {
     expect(iframe?.getAttribute('src')).toBe('https://www.youtube.com/embed/dQw4w9WgXcQ')
   })
 
+  it('renders one player per link when the same URL appears twice', () => {
+    const view = editorWith(`${YOUTUBE_LINK}${YOUTUBE_LINK}`).view
+    expect(view.dom.querySelectorAll('iframe')).toHaveLength(2)
+  })
+
   it('keeps the underlying document a plain markdown link (lossless round-trip)', () => {
     const markdown = editorWith(YOUTUBE_LINK).getMarkdown()
     expect(markdown).toContain('https://www.youtube.com/watch?v=dQw4w9WgXcQ')

apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/rich-markdown-editor/embed/link-embed.ts

@@ -35,18 +35,24 @@ function getStandaloneLinkHref(node: ProseMirrorNode): string | null {
 
 function buildDecorations(doc: ProseMirrorNode): DecorationSet {
   const decorations: Decoration[] = []
+  /** Per-source occurrence count, so repeated embeds of the same URL get distinct, stable keys. */
+  const sourceCounts = new Map<string, number>()
   doc.descendants((node, pos) => {
     if (node.type.name !== 'paragraph') return undefined
     const href = getStandaloneLinkHref(node)
     if (href) {
       const embedInfo = getEmbedInfo(href)
       if (embedInfo) {
-        // Render the player just after the link paragraph, keyed by source so the iframe/video
-        // DOM is reused across edits instead of reloading on every keystroke.
+        // Key by source + occurrence index so the iframe/video DOM is reused across unrelated
+        // edits (no reload on keystroke) while two links to the same URL still render as two
+        // distinct widgets rather than collapsing into one.
+        const source = `embed:${embedInfo.type}:${embedInfo.url}`
+        const index = sourceCounts.get(source) ?? 0
+        sourceCounts.set(source, index + 1)
         decorations.push(
           Decoration.widget(pos + node.nodeSize, () => createEmbedDom(embedInfo), {
             side: 1,
-            key: `embed:${embedInfo.type}:${embedInfo.url}`,
+            key: `${source}:${index}`,
           })
         )
       }