fix(rich-editor): integration mentions are display-only; robust chip selection

- An integration mention's id is a block type (gmail_v2), not a routable resource —
  /integrations/[block] expects a slug and a type maps to zero-or-many credentials —
  so it no longer links to a 404. The chip only shows a pointer/navigates on kinds
  that resolve to a real page.
- Scope the block-leaf selection ring off the mention chip robustly (covers a
  node-view wrapper via :has), so a selected chip shows a subtle fill, not the outline.

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/rich-markdown-editor/mention/mention-node.tsx

@@ -131,27 +131,27 @@ function MentionChipView({ node, editor }: ReactNodeViewProps) {
   const router = useRouter()
   const params = useParams()
   const { kind, id, label } = node.attrs as MentionAttrs
-  const Icon = mentionIcon(kind, id) as StyleableIcon | undefined
-  const iconStyle = Icon ? getBareIconStyle(Icon) : undefined
+  const Icon = mentionIcon(kind, id) as StyleableIcon
+  const iconStyle = getBareIconStyle(Icon)
   const navigable = editor.storage.mention?.navigable === true
+  const workspaceId = typeof params.workspaceId === 'string' ? params.workspaceId : undefined
+  // Only show the pointer / route on a kind that actually resolves to a page (e.g. not an integration).
+  const path = navigable && workspaceId ? simLinkPath(workspaceId, kind, id) : null
 
   const handleClick = (event: MouseEvent) => {
-    if (!(event.metaKey || event.ctrlKey)) return
-    const workspaceId = typeof params.workspaceId === 'string' ? params.workspaceId : undefined
-    const path = workspaceId && simLinkPath(workspaceId, kind, id)
-    if (!path) return
+    if (!path || !(event.metaKey || event.ctrlKey)) return
     event.preventDefault()
     router.push(path)
   }
 
   return (
     <NodeViewWrapper
       as='span'
-      className={cn(CHIP_CLASS, navigable && 'cursor-pointer')}
-      onClick={navigable ? handleClick : undefined}
+      className={cn(CHIP_CLASS, path && 'cursor-pointer')}
+      onClick={path ? handleClick : undefined}
       title={label}
     >
-      {Icon && <Icon style={iconStyle} />}
+      <Icon style={iconStyle} />
       <span>{label}</span>
     </NodeViewWrapper>
   )

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

@@ -25,10 +25,11 @@ describe('simLinkPath', () => {
     expect(simLinkPath(ws, 'knowledge', 'k1')).toBe('/workspace/ws1/knowledge/k1')
     expect(simLinkPath(ws, 'workflow', 'w1')).toBe('/workspace/ws1/w/w1')
     expect(simLinkPath(ws, 'skill', 's1')).toBe('/workspace/ws1/skills?skillId=s1')
-    expect(simLinkPath(ws, 'integration', 'slack')).toBe('/workspace/ws1/integrations/slack')
   })
 
-  it('returns null for an unknown kind', () => {
+  it('returns null for kinds with no navigable resource (integration) and unknown kinds', () => {
+    // An integration mention's id is a block type, not a routable resource.
+    expect(simLinkPath(ws, 'integration', 'slack')).toBeNull()
     expect(simLinkPath(ws, 'mystery', 'x')).toBeNull()
   })
 })

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

@@ -19,9 +19,12 @@ export function parseSimHref(href: string): { kind: string; id: string } | null
 }
 
 /**
- * Resolves the in-app route for a clicked `sim:` mention link, or `null` for an unknown kind. Each
- * destination matches the entity's real route: files open the file detail view, folders/skills deep-link
- * the file browser / skills modal via their query params, the rest hit their `[id]` route.
+ * Resolves the in-app route for a clicked `sim:` mention, or `null` when the kind has no navigable
+ * destination. Each path matches the entity's real route: files open the file detail view,
+ * folders/skills deep-link the file browser / skills modal via their query params, the rest hit their
+ * `[id]` route. Integrations are intentionally non-navigable — a mention's id is a block *type*
+ * (`gmail_v2`), which isn't a routable resource (no per-type page; it maps to zero-or-many
+ * credentials), so the chip stays display-only.
  */
 export function simLinkPath(workspaceId: string, kind: string, id: string): string | null {
   const base = `/workspace/${workspaceId}`
@@ -38,8 +41,6 @@ export function simLinkPath(workspaceId: string, kind: string, id: string): stri
       return `${base}/w/${id}`
     case 'skill':
       return `${base}/skills?skillId=${id}`
-    case 'integration':
-      return `${base}/integrations/${id}`
     default:
       return null
   }

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

@@ -37,8 +37,10 @@
 }
 
 /* The inline mention chip is too small for the block-leaf ring above; a direct click selects it with a
-   subtle fill instead, so it never wears the heavy divider/image outline. */
-.rich-markdown-prose .mention-chip.ProseMirror-selectednode {
+   subtle fill instead, so it never wears the heavy divider/image outline. `:has` covers the case where
+   ProseMirror puts `selectednode` on a wrapper around the chip span rather than the span itself. */
+.rich-markdown-prose .mention-chip.ProseMirror-selectednode,
+.rich-markdown-prose .ProseMirror-selectednode:has(.mention-chip) {
   outline: none;
   border-radius: 3px;
   background: var(--surface-active);