improvement(tables): truncate over-limit CSV imports to the plan cap instead of rejecting

Author: TheodoreSpeaks

apps/sim/lib/copilot/tools/server/table/user-table.test.ts

@@ -268,17 +268,21 @@ describe('userTableServerTool.create_from_file', () => {
     expect(createArgs.maxTables).toBe(3)
   })
 
-  it('rejects a file exceeding the plan row limit without creating a table', async () => {
+  it('truncates to the plan row limit and reports dropped rows', async () => {
+    // File has 2 data rows (Alice, Bob); plan cap is 1.
     mockGetWorkspaceTableLimits.mockResolvedValueOnce({ maxRowsPerTable: 1, maxTables: 3 })
 
     const result = await userTableServerTool.execute(
       { operation: 'create_from_file', args: { fileId: 'file-1' } },
       { userId: 'user-1', workspaceId: 'workspace-1' }
     )
 
-    expect(result.success).toBe(false)
-    expect(result.message).toMatch(/exceeds this plan's limit/i)
-    expect(mockCreateTable).not.toHaveBeenCalled()
+    expect(result.success).toBe(true)
+    expect(mockCreateTable).toHaveBeenCalledTimes(1)
+    const insertCall = mockBatchInsertRows.mock.calls[0][0] as { rows: unknown[] }
+    expect(insertCall.rows).toHaveLength(1)
+    expect(result.data?.rowCount).toBe(1)
+    expect(result.message).toMatch(/dropped 1 row/i)
     expect(mockDeleteTable).not.toHaveBeenCalled()
   })
 

apps/sim/lib/copilot/tools/server/table/user-table.ts

@@ -767,12 +767,8 @@ export const userTableServerTool: BaseServerTool<UserTableArgs, UserTableResult>
           assertNotAborted()
           const planLimits = await getWorkspaceTableLimits(workspaceId)
 
-          if (rows.length > planLimits.maxRowsPerTable) {
-            return {
-              success: false,
-              message: `"${file.name}" has ${rows.length.toLocaleString()} rows, which exceeds this plan's limit of ${planLimits.maxRowsPerTable.toLocaleString()} rows per table.`,
-            }
-          }
+          const droppedRows = Math.max(0, rows.length - planLimits.maxRowsPerTable)
+          const rowsToImport = droppedRows > 0 ? rows.slice(0, planLimits.maxRowsPerTable) : rows
 
           const table = await createTable(
             {
@@ -787,7 +783,7 @@ export const userTableServerTool: BaseServerTool<UserTableArgs, UserTableResult>
             requestId
           )
 
-          const coerced = coerceRowsForTable(rows, { columns }, headerToColumn)
+          const coerced = coerceRowsForTable(rowsToImport, { columns }, headerToColumn)
           let inserted: number
           try {
             inserted = await batchInsertAll(table.id, coerced, table, workspaceId, context)
@@ -801,12 +797,19 @@ export const userTableServerTool: BaseServerTool<UserTableArgs, UserTableResult>
             fileName: file.name,
             columns: columns.length,
             rows: inserted,
+            droppedRows,
             userId: context.userId,
           })
 
+          const createdMessage = `Created table "${table.name}" with ${columns.length} columns and ${inserted.toLocaleString()} rows from "${file.name}"`
+          const message =
+            droppedRows > 0
+              ? `${createdMessage}. Dropped ${droppedRows.toLocaleString()} row(s) that exceed this plan's limit of ${planLimits.maxRowsPerTable.toLocaleString()} rows per table.`
+              : createdMessage
+
           return {
             success: true,
-            message: `Created table "${table.name}" with ${columns.length} columns and ${inserted} rows from "${file.name}"`,
+            message,
             data: {
               tableId: table.id,
               tableName: table.name,