refactor(tables): take the table's address as props, not route params

Step 2 of the migration. A canonical unit may not read route context (R6, held
at 0 with no annotation escape), and both of these would fail it the moment the
tree moves.

- `TableGrid` had `workspaceId`/`tableId` as optional props with a
  `useParams()` fallback. Its only mount already passes both, so the fallback
  was dead code that nonetheless made the grid unmountable more than once per
  page. Props are now required and the fallback is gone.

- `RowModal` read `params.workspaceId` with no fallback at all — a hard route
  dependency feeding three mutations. It now takes `workspaceId` as a required
  prop from its two mount sites in `table.tsx`.

No behavior change: every call site already had the values, and the panel was
already passing them explicitly. What changes is that the panel's copy no longer
silently resolves the *host page's* params when a prop is missing.

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/row-modal/row-modal.tsx

@@ -16,7 +16,6 @@ import {
 } from '@sim/emcn'
 import { createLogger } from '@sim/logger'
 import { getErrorMessage } from '@sim/utils/errors'
-import { useParams } from 'next/navigation'
 import {
   cleanCellValue,
   dateValueToLocalParts,
@@ -35,6 +34,12 @@ const logger = createLogger('RowModal')
 
 export interface RowModalProps {
   mode: 'edit' | 'delete'
+  /**
+   * The workspace the table lives in. A prop rather than a route-param read:
+   * this modal is mounted inside a view that also renders in the chat panel,
+   * where the page's params address a different resource entirely.
+   */
+  workspaceId: string
   isOpen: boolean
   onClose: () => void
   table: TableInfo
@@ -70,9 +75,16 @@ function cleanRowData(
  * call-site ever keeps it mounted across target-row changes, it must supply a `key`
  * prop (e.g. the row id) so React remounts with the new row's values.
  */
-export function RowModal({ mode, isOpen, onClose, table, row, rowIds, onSuccess }: RowModalProps) {
-  const params = useParams()
-  const workspaceId = params.workspaceId as string
+export function RowModal({
+  mode,
+  workspaceId,
+  isOpen,
+  onClose,
+  table,
+  row,
+  rowIds,
+  onSuccess,
+}: RowModalProps) {
   const tableId = table.id
 
   const schema = table?.schema

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

@@ -8,7 +8,6 @@ import { createLogger } from '@sim/logger'
 import type { TableCellSelection } from '@sim/realtime-protocol/table-presence'
 import { getErrorMessage } from '@sim/utils/errors'
 import { useVirtualizer } from '@tanstack/react-virtual'
-import { useParams } from 'next/navigation'
 import { usePostHog } from 'posthog-js/react'
 import type { EditingCell, SaveReason } from '@/components/resources/table-view'
 import {
@@ -165,8 +164,13 @@ export interface SelectionSnapshot {
 }
 
 interface TableGridProps {
-  workspaceId?: string
-  tableId?: string
+  /**
+   * The table's address, supplied by the view. Required rather than derived: a
+   * route-param fallback meant this grid could only ever exist once per page,
+   * and it is mounted in a panel beside the page that owns those params.
+   */
+  workspaceId: string
+  tableId: string
   embedded?: boolean
   /** Remote collaborators' cell selections, rendered as presence overlays. */
   remoteSelections: RemoteTableSelection[]
@@ -419,8 +423,8 @@ async function chunkBatchUpdates(
 }
 
 export function TableGrid({
-  workspaceId: propWorkspaceId,
-  tableId: propTableId,
+  workspaceId,
+  tableId,
   embedded,
   remoteSelections,
   emitCellSelection,
@@ -456,9 +460,6 @@ export function TableGrid({
   confirmDeleteColumnsSinkRef,
   pushTableRenameUndoSinkRef,
 }: TableGridProps) {
-  const params = useParams()
-  const workspaceId = propWorkspaceId || (params.workspaceId as string)
-  const tableId = propTableId || (params.tableId as string)
   const posthog = usePostHog()
 
   useEffect(() => {

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

@@ -1592,6 +1592,7 @@ export function Table({
       {editingRow && tableData && (
         <RowModal
           mode='edit'
+          workspaceId={workspaceId}
           isOpen={true}
           onClose={() => setEditingRow(null)}
           table={tableData}
@@ -1602,6 +1603,7 @@ export function Table({
       {deletingRows.length > 0 && tableData && (
         <RowModal
           mode='delete'
+          workspaceId={workspaceId}
           isOpen={true}
           onClose={() => setDeletingRows([])}
           table={tableData}