From 60ee0d4ce7844764b5d91c401b80a1363b0a67b1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E0=A4=95=E0=A4=BE=E0=A4=B0=E0=A4=A4=E0=A5=8B=E0=A4=AB?=
 =?UTF-8?q?=E0=A5=8D=E0=A4=AB=E0=A5=87=E0=A4=B2=E0=A4=B8=E0=A5=8D=E0=A4=95?=
 =?UTF-8?q?=E0=A5=8D=E0=A4=B0=E0=A4=BF=E0=A4=AA=E0=A5=8D=E0=A4=9F=E2=84=A2?=
 <netroy@users.noreply.github.com>
Date: Mon, 23 Sep 2024 09:19:37 +0200
Subject: [PATCH] refactor(editor): Correct some prop types (no-changelog)
 (#10907)

---
 .../AskAssistantAvatar/AssistantAvatar.vue           |  2 +-
 .../components/AskAssistantIcon/AssistantIcon.vue    |  4 ++--
 .../components/AskAssistantText/AssistantText.vue    |  2 +-
 .../src/components/CodeDiff/CodeDiff.vue             | 12 ++++++------
 .../InlineAskAssistantButton.vue                     |  6 +++---
 .../src/components/N8nAvatar/Avatar.vue              |  4 ++--
 .../src/components/N8nUserSelect/UserSelect.vue      |  2 +-
 packages/editor-ui/src/__tests__/data/canvas.ts      |  4 ++--
 packages/editor-ui/src/components/CopyInput.vue      |  4 ++--
 .../editor-ui/src/components/CredentialsSelect.vue   |  4 ++--
 .../components/Node/NodeCreator/Panel/SearchBar.vue  |  4 ++--
 .../Node/NodeCreator/Renderers/ItemsRenderer.vue     |  2 +-
 .../editor-ui/src/components/NodeDetailsView.vue     |  1 +
 packages/editor-ui/src/components/ParameterInput.vue |  2 +-
 .../src/components/ResourceMapper/ResourceMapper.vue |  2 +-
 .../editor-ui/src/components/RunDataAi/RunDataAi.vue |  2 +-
 packages/editor-ui/src/components/SaveButton.vue     |  8 ++++----
 .../src/components/TagsManager/TagsManager.vue       | 10 +++++-----
 .../WorkflowSelectorParameterInput.vue               |  2 +-
 .../elements/handles/CanvasHandleRenderer.spec.ts    |  2 +-
 .../executions/global/GlobalExecutionsList.vue       |  4 ++--
 .../workflow/WorkflowExecutionsInfoAccordion.vue     |  2 +-
 22 files changed, 43 insertions(+), 42 deletions(-)

diff --git a/packages/design-system/src/components/AskAssistantAvatar/AssistantAvatar.vue b/packages/design-system/src/components/AskAssistantAvatar/AssistantAvatar.vue
index 9df650ce504f7..f930e5a674b16 100644
--- a/packages/design-system/src/components/AskAssistantAvatar/AssistantAvatar.vue
+++ b/packages/design-system/src/components/AskAssistantAvatar/AssistantAvatar.vue
@@ -1,7 +1,7 @@
 <script setup lang="ts">
 import AssistantIcon from '../AskAssistantIcon/AssistantIcon.vue';
 
-withDefaults(defineProps<{ size: 'small' | 'mini' }>(), {
+withDefaults(defineProps<{ size?: 'small' | 'mini' }>(), {
 	size: 'small',
 });
 </script>
diff --git a/packages/design-system/src/components/AskAssistantIcon/AssistantIcon.vue b/packages/design-system/src/components/AskAssistantIcon/AssistantIcon.vue
index 6e79db0fd5cc7..103b15b8cd35c 100644
--- a/packages/design-system/src/components/AskAssistantIcon/AssistantIcon.vue
+++ b/packages/design-system/src/components/AskAssistantIcon/AssistantIcon.vue
@@ -3,8 +3,8 @@ import { computed } from 'vue';
 
 const props = withDefaults(
 	defineProps<{
-		size: 'mini' | 'small' | 'medium' | 'large';
-		theme: 'default' | 'blank' | 'disabled';
+		size?: 'mini' | 'small' | 'medium' | 'large';
+		theme?: 'default' | 'blank' | 'disabled';
 	}>(),
 	{
 		size: 'medium',
diff --git a/packages/design-system/src/components/AskAssistantText/AssistantText.vue b/packages/design-system/src/components/AskAssistantText/AssistantText.vue
index 4cb912552daba..df941ef00ea8a 100644
--- a/packages/design-system/src/components/AskAssistantText/AssistantText.vue
+++ b/packages/design-system/src/components/AskAssistantText/AssistantText.vue
@@ -1,5 +1,5 @@
 <script setup lang="ts">
-withDefaults(defineProps<{ text: string; size: 'small' | 'medium' | 'large' | 'xlarge' }>(), {
+withDefaults(defineProps<{ text: string; size?: 'small' | 'medium' | 'large' | 'xlarge' }>(), {
 	text: '',
 	size: 'medium',
 });
diff --git a/packages/design-system/src/components/CodeDiff/CodeDiff.vue b/packages/design-system/src/components/CodeDiff/CodeDiff.vue
index cb2016c97624e..70304385712a6 100644
--- a/packages/design-system/src/components/CodeDiff/CodeDiff.vue
+++ b/packages/design-system/src/components/CodeDiff/CodeDiff.vue
@@ -7,12 +7,12 @@ import { useI18n } from 'n8n-design-system/composables/useI18n';
 const MIN_LINES = 4;
 
 interface Props {
-	title: string;
-	content: string;
-	replacing: boolean;
-	replaced: boolean;
-	error: boolean;
-	streaming: boolean;
+	title?: string;
+	content?: string;
+	replacing?: boolean;
+	replaced?: boolean;
+	error?: boolean;
+	streaming?: boolean;
 }
 
 type Line =
diff --git a/packages/design-system/src/components/InlineAskAssistantButton/InlineAskAssistantButton.vue b/packages/design-system/src/components/InlineAskAssistantButton/InlineAskAssistantButton.vue
index 38918611d753f..a80de28d8c97b 100644
--- a/packages/design-system/src/components/InlineAskAssistantButton/InlineAskAssistantButton.vue
+++ b/packages/design-system/src/components/InlineAskAssistantButton/InlineAskAssistantButton.vue
@@ -8,9 +8,9 @@ import AssistantText from '../AskAssistantText/AssistantText.vue';
 const { t } = useI18n();
 
 interface Props {
-	size: 'small' | 'medium';
-	static: boolean;
-	asked: boolean;
+	size?: 'small' | 'medium';
+	static?: boolean;
+	asked?: boolean;
 }
 
 const props = withDefaults(defineProps<Props>(), {
diff --git a/packages/design-system/src/components/N8nAvatar/Avatar.vue b/packages/design-system/src/components/N8nAvatar/Avatar.vue
index 1957401f34e1f..675f81062e9f7 100644
--- a/packages/design-system/src/components/N8nAvatar/Avatar.vue
+++ b/packages/design-system/src/components/N8nAvatar/Avatar.vue
@@ -5,8 +5,8 @@ import Avatar from 'vue-boring-avatars';
 import { getInitials } from '../../utils/labelUtil';
 
 interface AvatarProps {
-	firstName: string;
-	lastName: string;
+	firstName?: string;
+	lastName?: string;
 	size?: 'xsmall' | 'small' | 'medium' | 'large';
 	colors?: string[];
 }
diff --git a/packages/design-system/src/components/N8nUserSelect/UserSelect.vue b/packages/design-system/src/components/N8nUserSelect/UserSelect.vue
index ca99214639520..5c2cd0eaca1a9 100644
--- a/packages/design-system/src/components/N8nUserSelect/UserSelect.vue
+++ b/packages/design-system/src/components/N8nUserSelect/UserSelect.vue
@@ -70,7 +70,7 @@ const sortedUsers = computed(() =>
 	}),
 );
 
-const setFilter = (value: string) => {
+const setFilter = (value: string = '') => {
 	filter.value = value;
 };
 
diff --git a/packages/editor-ui/src/__tests__/data/canvas.ts b/packages/editor-ui/src/__tests__/data/canvas.ts
index 639e1a8e82d3d..e0e748b89c6d8 100644
--- a/packages/editor-ui/src/__tests__/data/canvas.ts
+++ b/packages/editor-ui/src/__tests__/data/canvas.ts
@@ -105,7 +105,7 @@ export function createCanvasNodeProvide({
 } = {}) {
 	const props = createCanvasNodeProps({ id, label, selected, readOnly, data });
 	return {
-		[`${CanvasNodeKey}`]: {
+		[String(CanvasNodeKey)]: {
 			id: ref(props.id),
 			label: ref(props.label),
 			selected: ref(props.selected),
@@ -136,7 +136,7 @@ export function createCanvasHandleProvide({
 	isReadOnly?: boolean;
 } = {}) {
 	return {
-		[`${CanvasNodeHandleKey}`]: {
+		[String(CanvasNodeHandleKey)]: {
 			label: ref(label),
 			mode: ref(mode),
 			type: ref(type),
diff --git a/packages/editor-ui/src/components/CopyInput.vue b/packages/editor-ui/src/components/CopyInput.vue
index fb68984a57afd..c381077999064 100644
--- a/packages/editor-ui/src/components/CopyInput.vue
+++ b/packages/editor-ui/src/components/CopyInput.vue
@@ -7,13 +7,13 @@ type Props = {
 	label?: string;
 	hint?: string;
 	value?: string;
-	copyButtonText: string;
+	copyButtonText?: string;
 	toastTitle?: string;
 	toastMessage?: string;
 	size?: 'medium' | 'large';
 	collapse?: boolean;
 	redactValue?: boolean;
-	disableCopy: boolean;
+	disableCopy?: boolean;
 };
 
 const props = withDefaults(defineProps<Props>(), {
diff --git a/packages/editor-ui/src/components/CredentialsSelect.vue b/packages/editor-ui/src/components/CredentialsSelect.vue
index 2ce5458bfc160..41989fb690dae 100644
--- a/packages/editor-ui/src/components/CredentialsSelect.vue
+++ b/packages/editor-ui/src/components/CredentialsSelect.vue
@@ -1,5 +1,5 @@
 <script setup lang="ts">
-import type { ICredentialType, INodeProperties, NodeParameterValue } from 'n8n-workflow';
+import type { ICredentialType, INodeProperties } from 'n8n-workflow';
 import { computed, ref } from 'vue';
 import ScopesNotice from '@/components/ScopesNotice.vue';
 import NodeCredentials from '@/components/NodeCredentials.vue';
@@ -12,7 +12,7 @@ type Props = {
 	parameter: INodeProperties;
 	node?: INodeUi;
 	inputSize?: 'small' | 'large' | 'mini' | 'medium' | 'xlarge';
-	displayValue: NodeParameterValue;
+	displayValue: string;
 	isReadOnly: boolean;
 	displayTitle: string;
 };
diff --git a/packages/editor-ui/src/components/Node/NodeCreator/Panel/SearchBar.vue b/packages/editor-ui/src/components/Node/NodeCreator/Panel/SearchBar.vue
index d4866eddf23af..57b0184340e71 100644
--- a/packages/editor-ui/src/components/Node/NodeCreator/Panel/SearchBar.vue
+++ b/packages/editor-ui/src/components/Node/NodeCreator/Panel/SearchBar.vue
@@ -3,8 +3,8 @@ import { onMounted, reactive, toRefs, onBeforeUnmount } from 'vue';
 import { useExternalHooks } from '@/composables/useExternalHooks';
 
 export interface Props {
-	placeholder: string;
-	modelValue: string;
+	placeholder?: string;
+	modelValue?: string;
 }
 
 withDefaults(defineProps<Props>(), {
diff --git a/packages/editor-ui/src/components/Node/NodeCreator/Renderers/ItemsRenderer.vue b/packages/editor-ui/src/components/Node/NodeCreator/Renderers/ItemsRenderer.vue
index c2327dcfbc543..ab225854fd894 100644
--- a/packages/editor-ui/src/components/Node/NodeCreator/Renderers/ItemsRenderer.vue
+++ b/packages/editor-ui/src/components/Node/NodeCreator/Renderers/ItemsRenderer.vue
@@ -12,7 +12,7 @@ import LinkItem from '../ItemTypes/LinkItem.vue';
 import CategorizedItemsRenderer from './CategorizedItemsRenderer.vue';
 
 export interface Props {
-	elements: INodeCreateElement[];
+	elements?: INodeCreateElement[];
 	activeIndex?: number;
 	disabled?: boolean;
 	lazyRender?: boolean;
diff --git a/packages/editor-ui/src/components/NodeDetailsView.vue b/packages/editor-ui/src/components/NodeDetailsView.vue
index cb3d5bc302d55..dc3e323ac8621 100644
--- a/packages/editor-ui/src/components/NodeDetailsView.vue
+++ b/packages/editor-ui/src/components/NodeDetailsView.vue
@@ -55,6 +55,7 @@ const props = withDefaults(
 	}>(),
 	{
 		isProductionExecutionPreview: false,
+		readOnly: false,
 	},
 );
 
diff --git a/packages/editor-ui/src/components/ParameterInput.vue b/packages/editor-ui/src/components/ParameterInput.vue
index df63a8b3228f4..0345e5a74991f 100644
--- a/packages/editor-ui/src/components/ParameterInput.vue
+++ b/packages/editor-ui/src/components/ParameterInput.vue
@@ -172,7 +172,7 @@ const dateTimePickerOptions = ref({
 });
 const isFocused = ref(false);
 
-const displayValue = computed<string | number | boolean | null>(() => {
+const displayValue = computed(() => {
 	if (remoteParameterOptionsLoadingIssues.value) {
 		if (!nodeType.value || nodeType.value?.codex?.categories?.includes(CORE_NODES_CATEGORY)) {
 			return i18n.baseText('parameterInput.loadOptionsError');
diff --git a/packages/editor-ui/src/components/ResourceMapper/ResourceMapper.vue b/packages/editor-ui/src/components/ResourceMapper/ResourceMapper.vue
index 26b356580aea7..78fb0952359b1 100644
--- a/packages/editor-ui/src/components/ResourceMapper/ResourceMapper.vue
+++ b/packages/editor-ui/src/components/ResourceMapper/ResourceMapper.vue
@@ -27,7 +27,7 @@ type Props = {
 	path: string;
 	inputSize: 'small' | 'medium';
 	labelSize: 'small' | 'medium';
-	teleported: boolean;
+	teleported?: boolean;
 	dependentParametersValues?: string | null;
 	isReadOnly?: boolean;
 };
diff --git a/packages/editor-ui/src/components/RunDataAi/RunDataAi.vue b/packages/editor-ui/src/components/RunDataAi/RunDataAi.vue
index eb7de453c16dc..89257df47c660 100644
--- a/packages/editor-ui/src/components/RunDataAi/RunDataAi.vue
+++ b/packages/editor-ui/src/components/RunDataAi/RunDataAi.vue
@@ -25,7 +25,7 @@ interface TreeNode {
 }
 export interface Props {
 	node: INodeUi;
-	runIndex: number;
+	runIndex?: number;
 	hideTitle?: boolean;
 	slim?: boolean;
 }
diff --git a/packages/editor-ui/src/components/SaveButton.vue b/packages/editor-ui/src/components/SaveButton.vue
index acf896e136d83..10471b86a7114 100644
--- a/packages/editor-ui/src/components/SaveButton.vue
+++ b/packages/editor-ui/src/components/SaveButton.vue
@@ -6,10 +6,10 @@ import { computed } from 'vue';
 const props = withDefaults(
 	defineProps<{
 		saved: boolean;
-		isSaving: boolean;
-		disabled: boolean;
-		type: string;
-		withShortcut: boolean;
+		isSaving?: boolean;
+		disabled?: boolean;
+		type?: string;
+		withShortcut?: boolean;
 		shortcutTooltip?: string;
 		savingLabel?: string;
 	}>(),
diff --git a/packages/editor-ui/src/components/TagsManager/TagsManager.vue b/packages/editor-ui/src/components/TagsManager/TagsManager.vue
index 9362eff7ef660..6583c592cf98b 100644
--- a/packages/editor-ui/src/components/TagsManager/TagsManager.vue
+++ b/packages/editor-ui/src/components/TagsManager/TagsManager.vue
@@ -10,11 +10,11 @@ import type { BaseTextKey } from '@/plugins/i18n';
 
 interface TagsManagerProps {
 	modalKey: string;
-	usageLocaleKey: BaseTextKey;
-	usageColumnTitleLocaleKey: BaseTextKey;
-	titleLocaleKey: BaseTextKey;
-	noTagsTitleLocaleKey: BaseTextKey;
-	noTagsDescriptionLocaleKey: BaseTextKey;
+	usageLocaleKey?: BaseTextKey;
+	usageColumnTitleLocaleKey?: BaseTextKey;
+	titleLocaleKey?: BaseTextKey;
+	noTagsTitleLocaleKey?: BaseTextKey;
+	noTagsDescriptionLocaleKey?: BaseTextKey;
 	tags: ITag[];
 	isLoading: boolean;
 	onFetchTags: () => Promise<void>;
diff --git a/packages/editor-ui/src/components/WorkflowSelectorParameterInput/WorkflowSelectorParameterInput.vue b/packages/editor-ui/src/components/WorkflowSelectorParameterInput/WorkflowSelectorParameterInput.vue
index 5eea4ed716764..3a3817b58cf5f 100644
--- a/packages/editor-ui/src/components/WorkflowSelectorParameterInput/WorkflowSelectorParameterInput.vue
+++ b/packages/editor-ui/src/components/WorkflowSelectorParameterInput/WorkflowSelectorParameterInput.vue
@@ -22,7 +22,7 @@ import { useWorkflowResourcesLocator } from './useWorkflowResourcesLocator';
 interface Props {
 	modelValue: INodeParameterResourceLocator;
 	eventBus?: EventBus;
-	inputSize: 'small' | 'mini' | 'medium' | 'large' | 'xlarge';
+	inputSize?: 'small' | 'mini' | 'medium' | 'large' | 'xlarge';
 	isValueExpression?: boolean;
 	isReadOnly?: boolean;
 	path: string;
diff --git a/packages/editor-ui/src/components/canvas/elements/handles/CanvasHandleRenderer.spec.ts b/packages/editor-ui/src/components/canvas/elements/handles/CanvasHandleRenderer.spec.ts
index bbbdc008dcd00..bf2ec39ed7ba5 100644
--- a/packages/editor-ui/src/components/canvas/elements/handles/CanvasHandleRenderer.spec.ts
+++ b/packages/editor-ui/src/components/canvas/elements/handles/CanvasHandleRenderer.spec.ts
@@ -88,7 +88,7 @@ describe('CanvasHandleRenderer', () => {
 			},
 			global: {
 				provide: {
-					[`${CanvasNodeHandleKey}`]: { label: ref(label) },
+					[String(CanvasNodeHandleKey)]: { label: ref(label) },
 				},
 				stubs: {
 					Handle,
diff --git a/packages/editor-ui/src/components/executions/global/GlobalExecutionsList.vue b/packages/editor-ui/src/components/executions/global/GlobalExecutionsList.vue
index 321eb4753480b..a1bcc95f94cc5 100644
--- a/packages/editor-ui/src/components/executions/global/GlobalExecutionsList.vue
+++ b/packages/editor-ui/src/components/executions/global/GlobalExecutionsList.vue
@@ -18,8 +18,8 @@ const props = withDefaults(
 	defineProps<{
 		executions: ExecutionSummaryWithScopes[];
 		filters: ExecutionFilterType;
-		total: number;
-		estimated: boolean;
+		total?: number;
+		estimated?: boolean;
 	}>(),
 	{
 		total: 0,
diff --git a/packages/editor-ui/src/components/executions/workflow/WorkflowExecutionsInfoAccordion.vue b/packages/editor-ui/src/components/executions/workflow/WorkflowExecutionsInfoAccordion.vue
index 25c456d2ca6e7..d58815ad88dce 100644
--- a/packages/editor-ui/src/components/executions/workflow/WorkflowExecutionsInfoAccordion.vue
+++ b/packages/editor-ui/src/components/executions/workflow/WorkflowExecutionsInfoAccordion.vue
@@ -19,7 +19,7 @@ interface IWorkflowSaveSettings {
 
 const props = withDefaults(
 	defineProps<{
-		initiallyExpanded: boolean;
+		initiallyExpanded?: boolean;
 	}>(),
 	{
 		initiallyExpanded: false,