chore(user-input): drop redundant inline comments from the mention-dismiss fix

Trims the inline // explanations added in the previous commit down to
the two declaration-level doc comments that carry real information —
matching the repo's no-inline-comments convention.

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/home/components/user-input/components/prompt-editor/use-prompt-editor.test.tsx

@@ -103,9 +103,6 @@ describe('usePromptEditor mention menu dismissal', () => {
 
     act(() => {
       typeInto(textarea, '@f')
-      // `onChange` in the real component is wired to `handleInputChange`; the
-      // hook only exposes it through the returned object, so invoke it the
-      // same way `<textarea onChange={editor.handleInputChange} />` would.
       result().handleInputChange({
         target: textarea,
         currentTarget: textarea,
@@ -131,24 +128,18 @@ describe('usePromptEditor mention menu dismissal', () => {
     })
     expect(result().mentionQuery).toBe('f')
 
-    // Radix's DismissableLayer calls this on outside pointerdown / Escape —
-    // never on a programmatic `plusMenuRef.current.close()`.
     act(() => {
       result().handlePlusMenuClose()
     })
     expect(result().mentionQuery).toBeNull()
 
-    // The same click that dismissed the popover also moves the caret via
-    // native textarea behavior; the caret lands back at the end of the text,
-    // i.e. still inside the unterminated "@f" token. Simulate the resulting
-    // onSelect/onMouseUp without any further keystroke.
     act(() => {
       textarea.setSelectionRange(2, 2)
       result().handleSelectAdjust()
     })
 
     expect(result().mentionQuery).toBeNull()
-    expect(openMenu.open).toHaveBeenCalledTimes(1) // only the original open — no reopen
+    expect(openMenu.open).toHaveBeenCalledTimes(1)
 
     unmount()
   })
@@ -173,7 +164,6 @@ describe('usePromptEditor mention menu dismissal', () => {
     })
     expect(openMenu.open).toHaveBeenCalledTimes(1)
 
-    // User keeps editing the same token — this must reopen it.
     act(() => {
       typeInto(textarea, '@fo', 3)
       result().handleInputChange({
@@ -208,8 +198,6 @@ describe('usePromptEditor mention menu dismissal', () => {
     })
     expect(openMenu.open).toHaveBeenCalledTimes(1)
 
-    // Clear the field and start a brand new mention elsewhere in the text —
-    // a stale dismissal must never leak onto an unrelated token.
     act(() => {
       typeInto(textarea, '@f done. @g', 11)
       result().handleInputChange({

apps/sim/app/workspace/[workspaceId]/home/components/user-input/components/prompt-editor/use-prompt-editor.ts

@@ -180,13 +180,10 @@ export function usePromptEditor({
   const [slashQuery, setSlashQuery] = useState<string | null>(null)
 
   /**
-   * Start offset of a mention/slash token the user just dismissed via outside
-   * click or Escape. A dismiss alone doesn't move the caret, so the very next
-   * `selectionchange`/click still lands inside the still-open token range and
-   * would otherwise reopen the menu it was just closed — the "can't click
-   * away" bug. Suppresses exactly one reopen per token; cleared the moment
-   * the user types again (`handleInputChange`) so editing the token still
-   * works normally.
+   * Start offset of a mention/slash token most recently dismissed by the user
+   * (outside click or Escape) without a following keystroke — suppresses a
+   * single reopen of the menu for that exact token when the caret's own
+   * selection-change handler runs immediately after.
    */
   const dismissedMentionStartRef = useRef<number | null>(null)
   const dismissedSlashStartRef = useRef<number | null>(null)
@@ -447,18 +444,23 @@ export function usePromptEditor({
     [textareaRef, addContextNotified]
   )
 
+  /**
+   * Only reachable via Radix's own dismiss detection (outside click /
+   * Escape) — programmatic closes (`skillsMenuRef.current?.close()`) bypass
+   * `onOpenChange` and never call this.
+   */
   const handleSkillsMenuClose = useCallback(() => {
-    // See `handlePlusMenuClose` — only reachable via a real Radix dismiss.
     dismissedSlashStartRef.current = slashRangeRef.current?.start ?? null
     slashRangeRef.current = null
     setSlashQuery(null)
   }, [])
 
+  /**
+   * Only reachable via Radix's own dismiss detection (outside click /
+   * Escape) — programmatic closes (`plusMenuRef.current?.close()`) bypass
+   * `onOpenChange` and never call this.
+   */
   const handlePlusMenuClose = useCallback(() => {
-    // Only reachable via Radix's own dismiss detection (outside click / Escape) —
-    // programmatic closes (`plusMenuRef.current?.close()`) bypass `onOpenChange`
-    // and never call this. Remember the token so the caret's own selection-change
-    // handler (fired by the same click) doesn't immediately reopen it.
     dismissedMentionStartRef.current = mentionRangeRef.current?.start ?? null
     atInsertPosRef.current = null
     mentionRangeRef.current = null
@@ -490,8 +492,6 @@ export function usePromptEditor({
         return
       }
 
-      // The user just dismissed the menu for this exact token (outside click /
-      // Escape) and hasn't typed since — a caret move alone must not reopen it.
       if (active.start === dismissedMentionStartRef.current) {
         if (mentionRangeRef.current !== null) {
           mentionRangeRef.current = null
@@ -529,7 +529,6 @@ export function usePromptEditor({
         return
       }
 
-      // See the mirrored check in `syncMentionState`.
       if (active.start === dismissedSlashStartRef.current) {
         if (slashRangeRef.current !== null) {
           slashRangeRef.current = null
@@ -628,8 +627,6 @@ export function usePromptEditor({
       const caret = e.target.selectionStart ?? finalValue.length
       valueRef.current = finalValue
       setValueState(finalValue)
-      // A keystroke is an active edit — always let it reopen a just-dismissed
-      // menu, even for the same token the user clicked away from.
       dismissedMentionStartRef.current = null
       dismissedSlashStartRef.current = null
       syncMentionState(e.target, finalValue, caret)