refactor(chat): apply cleanup pass findings

emcn design: the table context menu built its Add-to-Chat label in the parent
and never pluralized it, so right-clicking a single row read 'Add rows to Chat'
directly above 'Delete row'. Derive it inside ContextMenu from selectedRowCount
like every sibling label, with an addToChatCellScoped boolean mirroring
workflowCellScoped. Also more correct: selectedRowCount accounts for a
select-all beyond the loaded page, which contextMenuRowIds.length does not.

callbacks: drop two useCallback wrappers whose identity nothing observes — both
handleAddSelectionToChat handlers feed unmemoized components (one through an
inline arrow). buildSelectionContext stays wrapped; it is a real dep of the
copy-bridge effect.

comments: fold a duplicated rationale paragraph in handleAddSelectionToChat
left by two commits stacking, drop a {@link MothershipHandoff} that resolves to
nothing (the type is not imported there), and trim a rowIds doc that restated
its own type.

Skipped, deliberately: the effects pass proposed moving buildContext out of
useSelectionCopyBridge's deps behind a latest-ref. buildSelectionContext is
already stable, so churn is near-zero, and it would leave two sibling
useCallbacks purposeless.

Author: waleedlatif1

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

@@ -1154,10 +1154,10 @@ export function LoadedRichMarkdownEditor({
     }
   }, [editor, file.id, file.name])
 
-  const handleAddSelectionToChat = useCallback(() => {
+  const handleAddSelectionToChat = () => {
     const context = buildSelectionContext()
     if (context) addToChat(context)
-  }, [addToChat, buildSelectionContext])
+  }
 
   useSelectionCopyBridge(containerRef, buildSelectionContext)
 

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

@@ -402,10 +402,10 @@ export const TextEditor = memo(function TextEditor({
     }
   }, [file.id, file.name])
 
-  const handleAddSelectionToChat = useCallback(() => {
+  const handleAddSelectionToChat = () => {
     const context = buildSelectionContext()
     if (context) addToChat(context)
-  }, [addToChat, buildSelectionContext])
+  }
 
   const {
     content,

apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/context-menu/context-menu.tsx

@@ -58,8 +58,12 @@ interface ContextMenuProps {
   disableDelete?: boolean
   /** Adds the selected rows / cell range to Chat as a reference. Omit to hide. */
   onAddToChat?: () => void
-  /** Label describing the current selection scope, e.g. "3 rows" or "cell range". */
-  addToChatLabel?: string
+  /**
+   * True when the selection is a spreadsheet-style cell range rather than whole
+   * rows, switching the label from row-scoped to cell-scoped. Mirrors
+   * {@link ContextMenuProps.workflowCellScoped}.
+   */
+  addToChatCellScoped?: boolean
 }
 
 export function ContextMenu({
@@ -85,7 +89,7 @@ export function ContextMenu({
   disableDuplicate = false,
   disableDelete = false,
   onAddToChat,
-  addToChatLabel = 'Add to Chat',
+  addToChatCellScoped = false,
 }: ContextMenuProps) {
   const count = selectedRowCount.toLocaleString()
   const deleteLabel = selectedRowCount > 1 ? `Delete ${count} rows` : 'Delete row'
@@ -107,6 +111,11 @@ export function ContextMenu({
     runningInSelectionCount === 1
       ? 'Stop running workflow'
       : `Stop ${runningInSelectionCount} running workflows`
+  const addToChatLabel = addToChatCellScoped
+    ? 'Add cell range to Chat'
+    : selectedRowCount > 1
+      ? `Add ${count} rows to Chat`
+      : 'Add row to Chat'
 
   return (
     <DropdownMenu

apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/table-grid/table-grid.tsx

@@ -3864,10 +3864,8 @@ export function TableGrid({
     // `contextMenuRowIds` reflects; drain up to the cap so the chip references as
     // many rows as it can carry (bounded by MAX_TABLE_SELECTION_ROWS) instead of a
     // silent loaded-only subset — mirroring how the copy path loads before writing.
-    // A gutter selection can extend past the loaded page, and `contextMenuRowIds`
-    // is the loaded intersection. The chip references ids the server re-fetches,
-    // so send the whole selection rather than whichever rows happen to be paged in.
     const gutterSelection = rowSelectionRef.current
+    // Prefer the whole gutter selection over the loaded intersection.
     let sourceRowIds =
       gutterSelection.kind === 'some' &&
       contextMenu.row &&
@@ -4631,7 +4629,7 @@ export function TableGrid({
         disableDuplicate={!canInsertFullRow}
         disableDelete={!canDeleteRow}
         onAddToChat={contextMenuRowIds.length > 0 ? handleAddSelectionToChat : undefined}
-        addToChatLabel={contextMenuColumnIds ? 'Add cell range to Chat' : 'Add rows to Chat'}
+        addToChatCellScoped={Boolean(contextMenuColumnIds)}
       />
 
       <ExpandedCellPopover

apps/sim/lib/mothership/events.ts

@@ -75,8 +75,8 @@ export interface MothershipAddContextDetail {
  * the same stale list and drop colliding chips.
  *
  * @returns `true` when a mounted input consumed it, `false` when none was
- * listening — callers fall back to persisting a chip-only
- * {@link MothershipHandoff} for the next chat mount.
+ * listening — callers fall back to persisting a chip-only handoff (see
+ * `MothershipHandoffStorage`) for the next chat mount.
  */
 export function addMothershipContexts(contexts: ChatContext[]): boolean {
   if (contexts.length === 0) return false

apps/sim/stores/panel/types.ts

@@ -34,7 +34,7 @@ export type ChatContext =
        * rewrite to keep chip tokens unique.
        */
       tableName: string
-      /** Ids of the selected rows. Always present (materialized from the grid selection). */
+      /** Materialized from the grid selection, including rows not yet paged in. */
       rowIds: string[]
       /**
        * Ids of the selected columns. Present only for a spreadsheet-style cell