chore(files): drop platform references and non-essential inline comments

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/components/resource/components/resource-header/resource-header.tsx

@@ -371,8 +371,13 @@ interface BreadcrumbLocationPopoverProps {
   veilBoundaryRef: React.RefObject<HTMLDivElement | null>
 }
 
-/** How long the reopen latch is held after a click-to-navigate, covering the route swap. */
-const NAVIGATE_LATCH_MS = 250
+/**
+ * Grace period before a hover-out dismisses the path popover. Covers the gap
+ * the pointer crosses between the trigger and the popover content (and brief
+ * jitter at their edges); re-entering either within this window cancels the
+ * close. Standard hover-intent close delay — not tied to any navigation timing.
+ */
+const POPOVER_CLOSE_DELAY_MS = 120
 
 function BreadcrumbLocationPopover({
   icon: Icon,
@@ -382,61 +387,51 @@ function BreadcrumbLocationPopover({
 }: BreadcrumbLocationPopoverProps) {
   const [open, setOpen] = useState(false)
   const closeTimeoutRef = useRef<ReturnType<typeof setTimeout> | null>(null)
-  /**
-   * Suppresses reopen for the brief window between a click-to-navigate and the
-   * route swap. Navigating away tears this popover down (the list and detail
-   * views render different subtrees), so if `open` were still true the dimming
-   * veil and popover content would snap away instead of fading — a visible
-   * flash. {@link navigateAndClose} closes the popover before running the
-   * crumb's handler and latches this so the pointer still resting on the
-   * trigger can't re-fire `openPopover` mid-navigation. The latch is held by a
-   * short timer (not cleared on the next pointer exit, which fires immediately
-   * and would let the popover flash back open before the route swaps); the
-   * timer releases it so the popover keeps working when the handler does not
-   * actually navigate (e.g. an unsaved-changes guard that opens a modal).
-   */
-  const navigatingRef = useRef(false)
-  const navigateLatchRef = useRef<ReturnType<typeof setTimeout> | null>(null)
   const rootBreadcrumb = breadcrumbs[0]
 
-  const clearCloseTimeout = () => {
+  const cancelScheduledClose = () => {
     if (closeTimeoutRef.current) {
       clearTimeout(closeTimeoutRef.current)
       closeTimeoutRef.current = null
     }
   }
 
+  /**
+   * Hover-intent open. Driven only by pointer-/keyboard-enter — never by
+   * pointer movement. This is what makes the popover dismiss cleanly on a
+   * click-to-navigate: a stationary click fires no enter event, so once
+   * {@link navigateAndClose} sets `open` false nothing re-opens it before the
+   * route swaps. (A move-driven open would re-fire under the resting cursor and
+   * flash the popover/veil back in mid-navigation.)
+   */
   const openPopover = () => {
-    if (navigatingRef.current) return
-    clearCloseTimeout()
+    cancelScheduledClose()
     setOpen(true)
   }
 
   const scheduleClose = () => {
-    clearCloseTimeout()
+    cancelScheduledClose()
     closeTimeoutRef.current = setTimeout(() => {
       setOpen(false)
       closeTimeoutRef.current = null
-    }, 120)
+    }, POPOVER_CLOSE_DELAY_MS)
   }
 
+  /**
+   * Closes the popover up front, then runs the crumb's handler. Closing first
+   * lets the veil fade and the popover play its exit animation instead of
+   * snapping away when navigation unmounts the header.
+   */
   const navigateAndClose = (onClick?: () => void) => {
     if (!onClick) return
-    navigatingRef.current = true
-    clearCloseTimeout()
+    cancelScheduledClose()
     setOpen(false)
     onClick()
-    if (navigateLatchRef.current) clearTimeout(navigateLatchRef.current)
-    navigateLatchRef.current = setTimeout(() => {
-      navigatingRef.current = false
-      navigateLatchRef.current = null
-    }, NAVIGATE_LATCH_MS)
   }
 
   useEffect(() => {
     return () => {
       if (closeTimeoutRef.current) clearTimeout(closeTimeoutRef.current)
-      if (navigateLatchRef.current) clearTimeout(navigateLatchRef.current)
     }
   }, [])
 
@@ -453,10 +448,6 @@ function BreadcrumbLocationPopover({
             onBlur={scheduleClose}
             onMouseEnter={openPopover}
             onMouseLeave={scheduleClose}
-            onMouseMove={openPopover}
-            onPointerEnter={openPopover}
-            onPointerLeave={scheduleClose}
-            onPointerMove={openPopover}
             className={cn(
               chipVariants({ flush: true }),
               'max-w-none gap-1.5 px-2 transition-colors',
@@ -492,10 +483,6 @@ function BreadcrumbLocationPopover({
           )}
           onMouseEnter={openPopover}
           onMouseLeave={scheduleClose}
-          onMouseMove={openPopover}
-          onPointerEnter={openPopover}
-          onPointerLeave={scheduleClose}
-          onPointerMove={openPopover}
         >
           <PopoverSection className='px-1.5 py-0.5 text-[var(--text-muted)] text-xs'>
             <span className='inline-flex items-center gap-1'>

apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/file-viewer.tsx

@@ -120,8 +120,6 @@ export function FileViewer({
       return <CsvTablePreview key={file.id} file={file} workspaceId={workspaceId} />
     }
 
-    // Markdown renders in the inline rich editor when idle. During agent streaming we keep
-    // the raw/preview editor, which already handles incremental token reconciliation.
     if (isMarkdownFile(file) && streamingContent === undefined) {
       return (
         <MarkdownFileEditor

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

@@ -149,9 +149,9 @@ export const MarkdownCodeBlock = CodeBlock.extend({
 })
 
 /**
- * Code block with hover-revealed controls (language picker, line-wrap toggle, copy), in the
- * style of Linear's editor. The `language` attribute drives {@link CodeBlockHighlight}'s Prism
- * highlighting and serializes to the ```lang fence on save; wrap is a view-only preference.
+ * Code block with hover-revealed controls (language picker, line-wrap toggle, copy). The
+ * `language` attribute drives {@link CodeBlockHighlight}'s Prism highlighting and serializes to
+ * the ```lang fence on save; wrap is a view-only preference.
  */
 export const CodeBlockWithLanguage = MarkdownCodeBlock.extend({
   addNodeView() {

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

@@ -17,8 +17,8 @@ function escapeAttr(value: string): string {
 
 /**
  * Serialize an image to markdown when it has no explicit size, and to an HTML `<img>` tag when
- * it does — standard markdown has no width syntax, so a resized image must round-trip as HTML
- * (the same convention GitHub uses). Unsized images stay clean `![alt](src)`.
+ * it does — standard markdown has no width syntax, so a resized image must round-trip as HTML to
+ * preserve its dimensions. Unsized images stay clean `![alt](src)`.
  */
 function imageMarkdown(node: JSONContent): string {
   const attrs = node.attrs ?? {}

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

@@ -30,7 +30,7 @@ function selectAdjacentLeaf(editor: Editor, direction: 'up' | 'down'): boolean {
  *   Backspace below a divider is a confusing no-op.
  * - **Mod-A** inside a code block selects only that block's contents; pressing it again (when the
  *   block is already fully selected) falls through to the default whole-document select-all, the
- *   same scoped behavior as Linear and code editors.
+ *   same scoped behavior as a code editor.
  * - **ArrowUp/ArrowDown** select an adjacent divider or image (see {@link selectAdjacentLeaf}).
  */
 export const EditorKeymap = Extension.create({

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

@@ -54,8 +54,6 @@ export function MarkdownFileEditor({
     return <PreviewLoadingFrame className='flex flex-1 flex-col' />
   }
 
-  // On a failed fetch, fall back to the raw editor rather than mount the rich editor over empty
-  // content; a later retry-success resolves `data` and the gate decides normally.
   if (decisionRef.current === null && error) {
     return (
       <TextEditor

apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/rich-markdown-editor/menus/bubble-menu.tsx

@@ -66,7 +66,7 @@ interface EditorBubbleMenuProps {
 }
 
 /**
- * Floating formatting toolbar shown on text selection (Linear-style). Marks and the common
+ * Floating formatting toolbar shown on text selection. Marks and the common
  * block types; the link button swaps the bar into an inline URL editor. Richer block inserts
  * live in the `/` slash menu. Active states are read through {@link useEditorState} so the bar
  * stays correct without re-rendering the editor on every transaction.

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

@@ -36,8 +36,8 @@ interface RichMarkdownEditorProps {
 
 /**
  * Inline WYSIWYG markdown editor (TipTap/ProseMirror) for markdown files. Renders a
- * single editing surface — markdown is transformed inline as you type, Linear-style —
- * with no raw/preview split. Content loading and autosave are delegated to
+ * single editing surface — markdown is transformed inline as you type — with no raw/preview
+ * split. Content loading and autosave are delegated to
  * {@link useEditableFileContent}; this component only renders the editor and bridges
  * markdown in and out of it.
  */
@@ -98,7 +98,6 @@ export const RichMarkdownEditor = memo(function RichMarkdownEditor({
         .catch(() => null)
       const editor = editorInstanceRef.current
       if (!result || !editor) continue
-      // The doc may have shrunk during the await; clamp so the insert can't land out of bounds.
       const safePosition = Math.min(position, editor.state.doc.content.size)
       try {
         editor