fix(tables): cap the Add-to-Chat label at the rows a chip can carry

Cursor Bugbot: last round's fix for the label undercounting rows introduced the
opposite error. addToChatRowCount passed the raw selection size, but
buildTableSelectionContext caps rowIds at MAX_TABLE_SELECTION_ROWS, so a
2,000-row selection advertised 2,000 while the chip referenced 500 — breaking
the same invariant the fix claimed to establish.

Routes the count through a chipRowCount helper next to the builder that applies
the cap, so the label cannot drift from the payload again.

utils.test.ts now asserts the invariant directly rather than the formula:
across 1, 42, 500, 750 and 50,000 requested rows, chipRowCount equals the
rowIds length the context actually carries. Verified to fail if the cap is
dropped.

Author: waleedlatif1

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

@@ -69,6 +69,7 @@ import {
   type CellCoord,
   canWriteRowsWithChip,
   checkboxColLayout,
+  chipRowCount,
   classifyExecStatusMix,
   collectRowSnapshots,
   computeNormalizedSelection,
@@ -4592,7 +4593,9 @@ export function TableGrid({
         disableDelete={!canDeleteRow}
         onAddToChat={addToChatRowIds.length > 0 ? handleAddSelectionToChat : undefined}
         addToChatCellScoped={Boolean(contextMenuColumnIds)}
-        addToChatRowCount={contextMenuIsSelectAll ? selectedRowCount : addToChatRowIds.length}
+        addToChatRowCount={chipRowCount(
+          contextMenuIsSelectAll ? selectedRowCount : addToChatRowIds.length
+        )}
       />
 
       <ExpandedCellPopover

apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/table-grid/utils.test.ts

@@ -8,7 +8,12 @@ import {
 } from '@/lib/copilot/chat/selection-context'
 import { TABLE_LIMITS } from '@/lib/table/constants'
 import type { DisplayColumn } from './types'
-import { buildTableSelectionContext, canWriteRowsWithChip, selectedColumnIds } from './utils'
+import {
+  buildTableSelectionContext,
+  canWriteRowsWithChip,
+  chipRowCount,
+  selectedColumnIds,
+} from './utils'
 
 function columns(count: number): DisplayColumn[] {
   return Array.from({ length: count }, (_, i) => ({
@@ -107,3 +112,22 @@ describe('canWriteRowsWithChip', () => {
     expect(canWriteRowsWithChip({ ...ok, rowCount: TABLE_LIMITS.MAX_COPY_ROWS + 1 })).toBe(false)
   })
 })
+
+describe('chipRowCount', () => {
+  it.each([1, 42, MAX_TABLE_SELECTION_ROWS, MAX_TABLE_SELECTION_ROWS + 250, 50_000])(
+    'agrees with what a context carries for %i requested rows',
+    (requested) => {
+      // The invariant that keeps breaking: a label derived from the raw
+      // selection size over-promises once the context caps its rowIds, and one
+      // derived from the loaded page under-promises. Both must equal this.
+      const context = buildTableSelectionContext({
+        tableId: 't1',
+        tableName: 'Sales',
+        rowIds: rowIds(requested),
+      })
+
+      if (context?.kind !== 'table_selection') throw new Error('expected a table_selection')
+      expect(chipRowCount(requested)).toBe(context.rowIds.length)
+    }
+  )
+})

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

@@ -404,6 +404,15 @@ export function buildTableSelectionContext(opts: {
   }
 }
 
+/**
+ * Rows a chip will actually reference for a selection of `requested` rows —
+ * {@link buildTableSelectionContext} caps its `rowIds`, so any count shown to
+ * the user must pass through here or the UI promises more than it sends.
+ */
+export function chipRowCount(requested: number): number {
+  return Math.min(requested, MAX_TABLE_SELECTION_ROWS)
+}
+
 /**
  * Whether a copy can be written synchronously on the event — the only way a
  * chat-selection chip survives, since the paged path's async Clipboard API write