refactor(tables): give the editing shell the host axis and an explicit address

Both surfaces that mount the table keep every capability they had. The mothership
panel still mounts the full editing shell — editing a table from chat is a
product decision, not something to drop as a side effect of a refactor — it just
says `host='panel'` instead of `embedded`.

Three prop smells go with it:

- `embedded?: boolean` becomes `host: 'page' | 'panel'`, the same vocabulary the
  canonical views use. `'public'` is deliberately unrepresentable: the shell
  holds a write path, and an anonymous surface mounts `TableView`.
- `workspaceId` / `tableId` become required instead of optional-with-a-
  `useParams()`-fallback. That fallback was documented as "page mode reads from
  useParams", which is another way of saying the component could only ever exist
  once per page.
- `page.tsx` now names `tableId` in its params type. It always had it — the type
  just did not say so.

The ids stay plain rather than becoming a `ResourceSource`: `page.tsx` is a
Server Component and a source carries functions, so it cannot cross the RSC
boundary. That is the same reason the public interface page hands over a plain
seed and lets the client mint the source, and the prop doc now says so.

Behaviour is identical by construction — `embedded` is simply derived as
`host !== 'page'`, and the permission context the panel reads is untouched.

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/resource-content/resource-content.tsx

@@ -264,9 +264,9 @@ export const ResourceContent = memo(function ResourceContent({
       return (
         <Table
           key={resource.id}
+          host='panel'
           workspaceId={workspaceId}
           tableId={resource.id}
-          embedded
           viewsEnabled={tableViewsEnabled}
         />
       )

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

@@ -11,7 +11,7 @@ export const metadata: Metadata = {
 }
 
 interface TablePageProps {
-  params: Promise<{ workspaceId: string }>
+  params: Promise<{ workspaceId: string; tableId: string }>
 }
 
 /**
@@ -28,7 +28,7 @@ interface TablePageProps {
  * the two flag lookups share them.
  */
 export default async function TablePage({ params }: TablePageProps) {
-  const [{ workspaceId }, session] = await Promise.all([params, getSession()])
+  const [{ workspaceId, tableId }, session] = await Promise.all([params, getSession()])
   const userId = session?.user?.id
   const host = userId ? await getWorkspaceHostContextForViewer(workspaceId, userId) : null
   const orgId = host?.hostOrganizationId ?? undefined
@@ -39,7 +39,13 @@ export default async function TablePage({ params }: TablePageProps) {
 
   return (
     <Suspense fallback={<TableLoading />}>
-      <Table tableLocksEnabled={tableLocksEnabled} viewsEnabled={viewsEnabled} />
+      <Table
+        host='page'
+        workspaceId={workspaceId}
+        tableId={tableId}
+        tableLocksEnabled={tableLocksEnabled}
+        viewsEnabled={viewsEnabled}
+      />
     </Suspense>
   )
 }

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

@@ -5,7 +5,7 @@ import { Chip, ChipConfirmModal, toast } from '@sim/emcn'
 import { Download, Lock, Pencil, Table as TableIcon, Trash, Upload } from '@sim/emcn/icons'
 import { createLogger } from '@sim/logger'
 import { getErrorMessage } from '@sim/utils/errors'
-import { useParams, useRouter } from 'next/navigation'
+import { useRouter } from 'next/navigation'
 import { useQueryStates } from 'nuqs'
 import { usePostHog } from 'posthog-js/react'
 import {
@@ -55,6 +55,7 @@ import {
 } from '@/hooks/queries/tables'
 import { useInlineRename } from '@/hooks/use-inline-rename'
 import { useSettingsNavigation } from '@/hooks/use-settings-navigation'
+import type { ResourceHost } from '@/resources'
 import { useLogDetailsUIStore } from '@/stores/logs/store'
 import type { DeletedRowSnapshot } from '@/stores/table/types'
 import {
@@ -92,12 +93,30 @@ const logger = createLogger('Table')
 const BLOCKED_TOAST_MS = 8000
 
 interface TableProps {
-  /** When set, the table renders without its page header / breadcrumbs / page-level
-   *  options bar. Used by the mothership chat panel to embed a table inline. */
-  embedded?: boolean
-  /** Identifiers — only set in embedded mode. Page mode reads from `useParams()`. */
-  workspaceId?: string
-  tableId?: string
+  /**
+   * Which surface this table is mounted on. `'page'` renders the full route
+   * chrome — header, breadcrumbs, page-level options bar; every other host
+   * renders the grid alone, as the mothership chat panel does.
+   *
+   * This replaces an `embedded` boolean so the shell speaks the same vocabulary
+   * as the canonical views it sits beside. `'public'` is not reachable here:
+   * the editing shell holds a write path, and an anonymous surface mounts
+   * `TableView` instead.
+   */
+  host: Extract<ResourceHost, 'page' | 'panel'>
+  /**
+   * The table's address. Required rather than derived: both mounts know it
+   * (`page.tsx` from its route params, the panel from the open resource), and a
+   * `useParams()` fallback meant this component could only ever exist once per
+   * page.
+   *
+   * Plain ids rather than a `ResourceSource` because `page.tsx` is a Server
+   * Component and a source carries functions, which cannot cross the RSC
+   * boundary — the same reason the public interface page hands over a plain
+   * seed and lets the client mint the source.
+   */
+  workspaceId: string
+  tableId: string
   /**
    * Whether an admin may CHANGE locks, resolved server-side by the page (the
    * flag's gating lives in AppConfig and has no client counterpart). Defaults
@@ -208,16 +227,16 @@ function isSameViewConfig(a: TableViewConfig, b: TableViewConfig): boolean {
  * Embedded mode skips the page header but otherwise renders the same surface.
  */
 export function Table({
-  embedded,
-  workspaceId: propWorkspaceId,
-  tableId: propTableId,
+  host,
+  workspaceId,
+  tableId,
   tableLocksEnabled = false,
   viewsEnabled = false,
-}: TableProps = {}) {
-  const params = useParams()
+}: TableProps) {
   const router = useRouter()
-  const workspaceId = propWorkspaceId || (params.workspaceId as string)
-  const tableId = propTableId || (params.tableId as string)
+
+  /** `host === 'page'` is the only surface that owns the route chrome. */
+  const embedded = host !== 'page'
 
   const posthog = usePostHog()
   const posthogRef = useRef(posthog)