refactor(chat): tighten table copy fallback and helper placement

- writeLoadedRowsWithChip now requires a chip to carry; with none it falls
  through to the canonical paged path (preserving its row loading and
  truncation notice) instead of doing a bare synchronous write.
- Reuse selectedColumnIds in the context-menu memo.
- Restore resolveTableSelectionResource's TSDoc, orphaned when renderTableCell
  was extracted between the doc and its function.

Author: waleedlatif1

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

@@ -360,10 +360,13 @@ function buildTableSelectionContext(opts: {
 
 /**
  * Copies `rows` synchronously on the copy event so a chat-selection chip can
- * ride alongside the tab-separated text. Only viable when every selected row is
- * already loaded and the set is within the chip cap — otherwise the caller falls
- * back to the paged `writeSelectionToClipboard`, whose async Clipboard API write
- * replaces the whole clipboard and so cannot carry a custom MIME type.
+ * ride alongside the tab-separated text. The paged `writeSelectionToClipboard`
+ * cannot do this: its async Clipboard API write replaces the whole clipboard and
+ * so cannot carry a custom MIME type.
+ *
+ * Taking this path requires a chip to carry AND every selected row already
+ * loaded and within the chip cap; otherwise the canonical paged path handles the
+ * copy, including its row loading and truncation notice.
  *
  * @returns True when it handled the copy, false to fall through to the paged path.
  */
@@ -374,13 +377,15 @@ function writeLoadedRowsWithChip(opts: {
   buildCells: (row: TableRowType) => string[]
   context: ChatContext | null
 }): boolean {
-  const { rows } = opts
-  if (!opts.allLoaded || rows.length === 0 || rows.length > MAX_TABLE_SELECTION_ROWS) return false
+  const { rows, context } = opts
+  if (!context || !opts.allLoaded || rows.length === 0 || rows.length > MAX_TABLE_SELECTION_ROWS) {
+    return false
+  }
   opts.clipboardData?.setData(
     'text/plain',
     rows.map((row) => opts.buildCells(row).join('\t')).join('\n')
   )
-  if (opts.context) attachSelectionContextToClipboard(opts.clipboardData, opts.context)
+  attachSelectionContextToClipboard(opts.clipboardData, context)
   toast.success(`Copied ${rows.length} ${rows.length === 1 ? 'row' : 'rows'}`)
   return true
 }
@@ -3836,11 +3841,9 @@ export function TableGrid({
     if (!sel) return undefined
     const contextRowArrayIndex = rows.findIndex((r) => r.id === contextMenu.row!.id)
     if (contextRowArrayIndex < sel.startRow || contextRowArrayIndex > sel.endRow) return undefined
-    const ids: string[] = []
-    for (let c = sel.startCol; c <= sel.endCol; c++) {
-      const col = displayColumns[c]
-      if (col) ids.push(getColumnId(col))
-    }
+    // Collapsed here too (not only in buildTableSelectionContext) because this
+    // also decides whether the menu item reads "cell range" or "rows".
+    const ids = selectedColumnIds(displayColumns, sel)
     return ids.length > 0 && ids.length < displayColumns.length ? ids : undefined
   }, [contextMenu.isOpen, contextMenu.row, rowSelection, normalizedSelection, rows, displayColumns])
 

apps/sim/lib/copilot/chat/process-contents.ts

@@ -930,19 +930,20 @@ async function resolveFileSelectionResource(
   }
 }
 
-/**
- * Resolves a table selection into an inline markdown table. Rows are re-fetched
- * by id from the DB (never trusting client-sent cell values); when `columnIds`
- * is present the projection is narrowed to that cell range, otherwise every
- * column is included.
- */
 /** Renders one cell for a markdown table row, escaping the delimiters. */
 function renderTableCell(value: unknown): string {
   if (value === null || value === undefined) return ''
   const cell = typeof value === 'string' ? value : JSON.stringify(value)
   return cell.replace(/\|/g, '\\|').replace(/\n/g, ' ')
 }
 
+/**
+ * Resolves a table selection into an inline markdown table. Rows are re-fetched
+ * by id from the DB (never trusting client-sent cell values); when `columnIds`
+ * is present the projection is narrowed to that cell range, otherwise every
+ * column is included. Output is bounded by
+ * {@link MAX_TABLE_SELECTION_CONTENT_LENGTH}, not just the row and column caps.
+ */
 async function resolveTableSelectionResource(
   tableId: string,
   workspaceId: string,