From f677ca5699df8899e1ae64bcc6324cd00b6169e8 Mon Sep 17 00:00:00 2001
From: Charlie Kolb <charlie@n8n.io>
Date: Fri, 8 Nov 2024 14:20:02 +0100
Subject: [PATCH] self review

---
 .../nodes/agents/Agent/agents/ToolsAgent/execute.ts |  1 -
 packages/editor-ui/src/components/OutputPanel.vue   |  5 -----
 packages/editor-ui/src/components/RunData.vue       |  3 ---
 packages/editor-ui/src/components/RunDataTable.vue  | 13 +++++--------
 4 files changed, 5 insertions(+), 17 deletions(-)

diff --git a/packages/@n8n/nodes-langchain/nodes/agents/Agent/agents/ToolsAgent/execute.ts b/packages/@n8n/nodes-langchain/nodes/agents/Agent/agents/ToolsAgent/execute.ts
index cd2b67b720..25e6e78984 100644
--- a/packages/@n8n/nodes-langchain/nodes/agents/Agent/agents/ToolsAgent/execute.ts
+++ b/packages/@n8n/nodes-langchain/nodes/agents/Agent/agents/ToolsAgent/execute.ts
@@ -115,7 +115,6 @@ export async function toolsAgentExecute(this: IExecuteFunctions): Promise<INodeE
 		| undefined;
 
 	const tools = (await getConnectedTools(this, true, false)) as Array<DynamicStructuredTool | Tool>;
-	console.log('tools', tools);
 	const outputParser = (await getOptionalOutputParsers(this))?.[0];
 	let structuredOutputParserTool: DynamicStructuredTool | undefined;
 	/**
diff --git a/packages/editor-ui/src/components/OutputPanel.vue b/packages/editor-ui/src/components/OutputPanel.vue
index 627a505d30..558eb0d919 100644
--- a/packages/editor-ui/src/components/OutputPanel.vue
+++ b/packages/editor-ui/src/components/OutputPanel.vue
@@ -389,11 +389,6 @@ const shouldShowWarning = computed(() => {
 
 		<template v-if="shouldShowWarning" #table-user-info>
 			<div :class="$style.noToolsUsedAlert">
-				<!-- <N8nAlert
-					:title="i18n.baseText('ndv.output.noToolUsedInfo.title')"
-					:description="i18n.baseText('ndv.output.noToolUsedInfo.description')"
-					type="info"
-				/> -->
 				<N8nCallout theme="secondary">
 					{{
 						i18n.baseText('ndv.output.noToolUsedInfo.title') +
diff --git a/packages/editor-ui/src/components/RunData.vue b/packages/editor-ui/src/components/RunData.vue
index 64d2fd43fd..6239ecd8bf 100644
--- a/packages/editor-ui/src/components/RunData.vue
+++ b/packages/editor-ui/src/components/RunData.vue
@@ -1594,9 +1594,6 @@ defineExpose({ enterEditMode });
 					@active-row-changed="onItemHover"
 					@display-mode-change="onDisplayModeChange"
 				>
-					<!-- <template #table-user-info>
-						<slot name="table-user-info"></slot>
-					</template> -->
 				</LazyRunDataTable>
 			</Suspense>
 
diff --git a/packages/editor-ui/src/components/RunDataTable.vue b/packages/editor-ui/src/components/RunDataTable.vue
index da46a55037..f27bd74129 100644
--- a/packages/editor-ui/src/components/RunDataTable.vue
+++ b/packages/editor-ui/src/components/RunDataTable.vue
@@ -6,12 +6,7 @@ import { useWorkflowsStore } from '@/stores/workflows.store';
 import { getMappedExpression } from '@/utils/mappingUtils';
 import { getPairedItemId } from '@/utils/pairedItemUtils';
 import { shorten } from '@/utils/typesUtils';
-import {
-	NodeHelpers,
-	type GenericValue,
-	type IDataObject,
-	type INodeExecutionData,
-} from 'n8n-workflow';
+import type { GenericValue, IDataObject, INodeExecutionData } from 'n8n-workflow';
 import { computed, onMounted, ref, watch } from 'vue';
 import Draggable from './Draggable.vue';
 import MappingPill from './MappingPill.vue';
@@ -392,11 +387,11 @@ watch(focusedMappableInput, (curr) => {
 
 <template>
 	<div :class="[$style.dataDisplay, { [$style.highlight]: highlight }]">
-		<slot name="table-user-info"></slot>
 		<table v-if="tableData.columns && tableData.columns.length === 0" :class="$style.table">
 			<thead>
 				<tr>
 					<th :class="$style.emptyCell"></th>
+					<th :class="$style.tableRightMargin"></th>
 				</tr>
 			</thead>
 			<tbody>
@@ -413,6 +408,7 @@ watch(focusedMappableInput, (curr) => {
 					>
 						<N8nInfoTip>{{ i18n.baseText('runData.emptyItemHint') }}</N8nInfoTip>
 					</td>
+					<td :class="$style.tableRightMargin"></td>
 				</tr>
 			</tbody>
 		</table>
@@ -482,6 +478,7 @@ watch(focusedMappableInput, (curr) => {
 							</span>
 						</N8nTooltip>
 					</th>
+					<th :class="$style.tableRightMargin"></th>
 				</tr>
 			</thead>
 			<Draggable
@@ -549,6 +546,7 @@ watch(focusedMappableInput, (curr) => {
 						</N8nTree>
 					</td>
 					<td v-if="columnLimitExceeded"></td>
+					<td :class="$style.tableRightMargin"></td>
 				</tr>
 			</Draggable>
 		</table>
@@ -567,7 +565,6 @@ watch(focusedMappableInput, (curr) => {
 	word-break: normal;
 	height: 100%;
 	padding-bottom: var(--spacing-3xl);
-	padding-inline: var(--spacing-l);
 }
 
 .table {