refactor(media-embed): tighten comments and drop a redundant guard

- Drop the redundant paragraph type-check in getStandaloneLinkHref (the caller
  already filters to paragraphs) and rename the param for clarity
- Remove an inline comment and a TSDoc sentence that restated logic documented
  elsewhere

Author: waleedlatif1

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

@@ -13,11 +13,11 @@ const LINK_EMBED_PLUGIN_KEY = new PluginKey('linkEmbed')
  * standalone links become media embeds — a link inline within a sentence stays a plain link,
  * matching how Notion and Linear auto-embed.
  */
-function getStandaloneLinkHref(node: ProseMirrorNode): string | null {
-  if (node.type.name !== 'paragraph' || node.childCount === 0) return null
+function getStandaloneLinkHref(paragraph: ProseMirrorNode): string | null {
+  if (paragraph.childCount === 0) return null
   let href: string | null = null
   let isStandalone = true
-  node.forEach((child) => {
+  paragraph.forEach((child) => {
     if (!isStandalone) return
     const linkMark = child.isText
       ? child.marks.find((mark) => mark.type.name === 'link')
@@ -43,9 +43,6 @@ function buildDecorations(doc: ProseMirrorNode): DecorationSet {
     if (href) {
       const embedInfo = getEmbedInfo(href)
       if (embedInfo) {
-        // 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)
@@ -57,7 +54,7 @@ function buildDecorations(doc: ProseMirrorNode): DecorationSet {
         )
       }
     }
-    // Paragraphs hold only inline content — never another embeddable paragraph.
+    // Paragraphs hold only inline content, so there is nothing more to descend into.
     return false
   })
   return DecorationSet.create(doc, decorations)

packages/utils/src/media-embed.ts

@@ -57,10 +57,9 @@ function toDropboxDirectVideoUrl(parsed: URL): string | null {
  * generic video/audio file extensions. Returns null when the URL is not a
  * recognized embeddable source.
  *
- * Each platform is gated on its parsed hostname via {@link hostMatches} before the
- * id-extracting regex runs, so a look-alike host can never produce a trusted-looking
- * embed. The generic file-extension fallbacks are intentionally host-agnostic — any
- * direct media file URL is embeddable.
+ * Each platform is gated on its parsed hostname via {@link hostMatches} before its
+ * id-extracting regex runs. The generic file-extension fallbacks are intentionally
+ * host-agnostic — any direct media file URL is embeddable.
  */
 export function getEmbedInfo(url: string): EmbedInfo | null {
   const parsed = parseUrl(url)