fix(tables): keep insert body base omittable for v1 contract

The afterRowId/beforeRowId mutual-exclusion .refine() turned the schema into a
ZodEffects, which Zod forbids .omit() on — v1's insertTableRowBodySchema.omit({
position }) threw at module load (runtime-only; tsc misses it). Split the plain
object base out, apply the shared refine on top, and have v1 omit from the base
then re-apply it.

Author: TheodoreSpeaks

apps/sim/lib/api/contracts/tables.ts

@@ -147,19 +147,28 @@ export const rowDataSchema = domainObjectSchema<RowData>()
 export const tableDefinitionSchema = domainObjectSchema<TableDefinition>()
 export const tableRowSchema = domainObjectSchema<TableRow>()
 
-export const insertTableRowBodySchema = z
-  .object({
-    workspaceId: z.string().min(1, 'Workspace ID is required'),
-    data: rowDataSchema,
-    position: z.number().int().min(0).optional(),
-    /** Fractional ordering: insert directly after this row id. Takes precedence over `position`. */
-    afterRowId: z.string().min(1).optional(),
-    /** Fractional ordering: insert directly before this row id. Takes precedence over `position`. */
-    beforeRowId: z.string().min(1).optional(),
-  })
-  .refine((data) => !data.afterRowId || !data.beforeRowId, {
-    message: 'afterRowId and beforeRowId are mutually exclusive',
-  })
+/**
+ * Plain-object base for the single-row insert body. Kept un-refined so callers
+ * (e.g. the v1 public contract) can `.omit()` fields before applying
+ * {@link rowAnchorMutexRefine} — Zod forbids `.omit()` on a refined schema.
+ */
+export const insertTableRowBodyBaseSchema = z.object({
+  workspaceId: z.string().min(1, 'Workspace ID is required'),
+  data: rowDataSchema,
+  position: z.number().int().min(0).optional(),
+  /** Fractional ordering: insert directly after this row id. Takes precedence over `position`. */
+  afterRowId: z.string().min(1).optional(),
+  /** Fractional ordering: insert directly before this row id. Takes precedence over `position`. */
+  beforeRowId: z.string().min(1).optional(),
+})
+
+/** `afterRowId` and `beforeRowId` are mutually exclusive insert anchors. */
+export const rowAnchorMutexRefine = [
+  (data: { afterRowId?: string; beforeRowId?: string }) => !data.afterRowId || !data.beforeRowId,
+  { message: 'afterRowId and beforeRowId are mutually exclusive' },
+] as const
+
+export const insertTableRowBodySchema = insertTableRowBodyBaseSchema.refine(...rowAnchorMutexRefine)
 
 /**
  * POST `/api/table/[tableId]/rows/upsert` body — insert-or-update keyed by a

apps/sim/lib/api/contracts/v1/tables/index.ts

@@ -4,7 +4,8 @@ import {
   createTableColumnBodySchema,
   deleteTableColumnBodySchema,
   deleteTableRowsBodySchema,
-  insertTableRowBodySchema,
+  insertTableRowBodyBaseSchema,
+  rowAnchorMutexRefine,
   rowDataSchema,
   tableIdParamsSchema,
   tableRowParamsSchema,
@@ -60,7 +61,9 @@ export const v1CreateTableBodySchema = createTableBodySchema.omit({
  * Public API insert row body — no caller-controlled `position`. Server places
  * new rows at the tail; ordering by index is an in-app affordance only.
  */
-export const v1InsertTableRowBodySchema = insertTableRowBodySchema.omit({ position: true })
+export const v1InsertTableRowBodySchema = insertTableRowBodyBaseSchema
+  .omit({ position: true })
+  .refine(...rowAnchorMutexRefine)
 
 /**
  * Public API batch insert body — no `positions`. Same rationale as above.