Fix

Author: Sg312

apps/sim/lib/copilot/generated/tool-catalog-v1.ts

@@ -2969,7 +2969,8 @@ export const UserTable: ToolCatalogEntry = {
           },
           rowIds: {
             type: 'array',
-            description: 'Array of row IDs to delete (for batch_delete_rows)',
+            description:
+              'Array of row IDs. Used by batch_delete_rows (rows to delete) and run_column (optional row scope — when omitted, runs across the whole table; when provided, only these rows are candidates and the server eligibility predicate still applies).',
             items: { type: 'string' },
           },
           rows: {

apps/sim/lib/copilot/generated/tool-schemas-v1.ts

@@ -2799,7 +2799,8 @@ export const TOOL_RUNTIME_SCHEMAS: Record<string, ToolRuntimeSchemaEntry> = {
             },
             rowIds: {
               type: 'array',
-              description: 'Array of row IDs to delete (for batch_delete_rows)',
+              description:
+                'Array of row IDs. Used by batch_delete_rows (rows to delete) and run_column (optional row scope — when omitted, runs across the whole table; when provided, only these rows are candidates and the server eligibility predicate still applies).',
               items: {
                 type: 'string',
               },

apps/sim/lib/copilot/tool-executor/executor.test.ts

@@ -113,6 +113,31 @@ describe('copilot tool executor fallback', () => {
     )
   })
 
+  it('defaults copilot function_execute timeout to 10 seconds when invalid', async () => {
+    isKnownTool.mockReturnValue(false)
+    isSimExecuted.mockReturnValue(false)
+    executeAppTool.mockResolvedValue({ success: true, output: { result: 'ok' } })
+
+    await executeTool(
+      'function_execute',
+      { code: 'return 1', timeout: 0 },
+      {
+        userId: 'user-1',
+        workflowId: 'workflow-1',
+        workspaceId: 'ws-1',
+        copilotToolExecution: true,
+      }
+    )
+
+    expect(executeAppTool).toHaveBeenCalledWith(
+      'function_execute',
+      expect.objectContaining({
+        timeout: 10_000,
+      }),
+      false
+    )
+  })
+
   it('does not let copilot function_execute timeout exceed the default execution limit', async () => {
     isKnownTool.mockReturnValue(false)
     isSimExecuted.mockReturnValue(false)

apps/sim/lib/copilot/tool-executor/executor.ts

@@ -106,16 +106,18 @@ function buildAppToolParams(
   const result = { ...params }
 
   if (toolId === FUNCTION_EXECUTE_TOOL_ID && context.copilotToolExecution) {
-    const timeoutSeconds =
+    const rawTimeoutSeconds =
       result.timeout === undefined || result.timeout === null
         ? DEFAULT_FUNCTION_EXECUTE_TIMEOUT_SECONDS
         : Number(result.timeout)
-    if (Number.isFinite(timeoutSeconds) && timeoutSeconds > 0) {
-      result.timeout = Math.min(
-        Math.ceil(timeoutSeconds * MILLISECONDS_PER_SECOND),
-        DEFAULT_EXECUTION_TIMEOUT_MS
-      )
-    }
+    const timeoutSeconds =
+      Number.isFinite(rawTimeoutSeconds) && rawTimeoutSeconds > 0
+        ? rawTimeoutSeconds
+        : DEFAULT_FUNCTION_EXECUTE_TIMEOUT_SECONDS
+    result.timeout = Math.min(
+      Math.ceil(timeoutSeconds * MILLISECONDS_PER_SECOND),
+      DEFAULT_EXECUTION_TIMEOUT_MS
+    )
   }
 
   if (result.credentialId && !result.credential && !result.oauthCredential) {