fix(ai-builder): Remove unnecessary list() call from conflict handler (no-changelog)

The conflict error already tells us the table exists — no need to
verify with list(). The extra call could fail and re-throw the
original error, defeating the fix.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: JoseBra

packages/@n8n/instance-ai/src/tools/data-tables/__tests__/create-data-table.tool.test.ts

@@ -169,7 +169,6 @@ describe('createCreateDataTableTool', () => {
 			(wrappedError as Error & { cause: Error }).cause = conflictError;
 
 			(context.dataTableService.create as jest.Mock).mockRejectedValue(wrappedError);
-			(context.dataTableService.list as jest.Mock).mockResolvedValue([mockTable]);
 
 			const tool = createCreateDataTableTool(context);
 

packages/@n8n/instance-ai/src/tools/data-tables/create-data-table.tool.ts

@@ -112,16 +112,12 @@ export function createCreateDataTableTool(context: InstanceAiContext) {
 				return { table };
 			} catch (error) {
 				// If table already exists, guide the agent to use the existing one
-				// rather than throwing — which would cause the agent to retry in a loop
+				// rather than throwing — which would cause the agent to waste iterations retrying
 				if (isNameConflictError(error)) {
-					const tables = await context.dataTableService.list({ projectId: input.projectId });
-					const existing = tables.find((t) => t.name === input.name);
-					if (existing) {
-						return {
-							denied: true,
-							reason: `Table "${input.name}" already exists. Use list-data-tables to find it and get-data-table-schema to check its columns.`,
-						};
-					}
+					return {
+						denied: true,
+						reason: `Table "${input.name}" already exists. Use list-data-tables to find it and get-data-table-schema to check its columns.`,
+					};
 				}
 				throw error;
 			}