From 15b7e8d1192370ea9e25fd1edb61197d0c5c442c Mon Sep 17 00:00:00 2001
From: Alex Grozav <alex@grozav.com>
Date: Fri, 17 May 2024 10:58:35 +0300
Subject: [PATCH 1/5] fix: fix type errors in external secrets files

---
 packages/editor-ui/src/Interface.ts           | 11 ++---
 .../editor-ui/src/api/externalSecrets.ee.ts   |  8 +---
 .../ExternalSecretsProviderCard.ee.vue        | 30 +++++++------
 .../ExternalSecretsProviderModal.ee.vue       | 43 +++++++++++--------
 .../composables/useExternalSecretsProvider.ts | 43 +++++++++++--------
 packages/editor-ui/src/shims.d.ts             | 10 +++++
 packages/editor-ui/src/utils/typeGuards.ts    |  6 +++
 7 files changed, 91 insertions(+), 60 deletions(-)

diff --git a/packages/editor-ui/src/Interface.ts b/packages/editor-ui/src/Interface.ts
index 0eddbadbce50c..a634cf7ba852a 100644
--- a/packages/editor-ui/src/Interface.ts
+++ b/packages/editor-ui/src/Interface.ts
@@ -1800,18 +1800,19 @@ export interface ExternalSecretsProviderSecret {
 
 export type ExternalSecretsProviderData = Record<string, IUpdateInformation['value']>;
 
+export type ExternalSecretsProviderProperty = INodeProperties;
+
+export type ExternalSecretsProviderState = 'connected' | 'tested' | 'initializing' | 'error';
+
 export interface ExternalSecretsProvider {
 	icon: string;
 	name: string;
 	displayName: string;
 	connected: boolean;
 	connectedAt: string | false;
-	state: 'connected' | 'tested' | 'initializing' | 'error';
+	state: ExternalSecretsProviderState;
 	data?: ExternalSecretsProviderData;
-}
-
-export interface ExternalSecretsProviderWithProperties extends ExternalSecretsProvider {
-	properties: INodeProperties[];
+	properties?: ExternalSecretsProviderProperty[];
 }
 
 export type CloudUpdateLinkSourceType =
diff --git a/packages/editor-ui/src/api/externalSecrets.ee.ts b/packages/editor-ui/src/api/externalSecrets.ee.ts
index 69c48f457cc3e..fb8cac9d77529 100644
--- a/packages/editor-ui/src/api/externalSecrets.ee.ts
+++ b/packages/editor-ui/src/api/externalSecrets.ee.ts
@@ -1,8 +1,4 @@
-import type {
-	IRestApiContext,
-	ExternalSecretsProvider,
-	ExternalSecretsProviderWithProperties,
-} from '@/Interface';
+import type { IRestApiContext, ExternalSecretsProvider } from '@/Interface';
 import { makeRestApiRequest } from '@/utils/apiUtils';
 
 export const getExternalSecrets = async (
@@ -20,7 +16,7 @@ export const getExternalSecretsProviders = async (
 export const getExternalSecretsProvider = async (
 	context: IRestApiContext,
 	id: string,
-): Promise<ExternalSecretsProviderWithProperties> => {
+): Promise<ExternalSecretsProvider> => {
 	return await makeRestApiRequest(context, 'GET', `/external-secrets/providers/${id}`);
 };
 
diff --git a/packages/editor-ui/src/components/ExternalSecretsProviderCard.ee.vue b/packages/editor-ui/src/components/ExternalSecretsProviderCard.ee.vue
index eb20c7d90e2f8..729c5d9e9f5e4 100644
--- a/packages/editor-ui/src/components/ExternalSecretsProviderCard.ee.vue
+++ b/packages/editor-ui/src/components/ExternalSecretsProviderCard.ee.vue
@@ -1,5 +1,5 @@
 <script lang="ts" setup>
-import type { PropType, Ref } from 'vue';
+import type { PropType } from 'vue';
 import type { ExternalSecretsProvider } from '@/Interface';
 import ExternalSecretsProviderImage from '@/components/ExternalSecretsProviderImage.ee.vue';
 import ExternalSecretsProviderConnectionSwitch from '@/components/ExternalSecretsProviderConnectionSwitch.ee.vue';
@@ -10,7 +10,8 @@ import { useI18n } from '@/composables/useI18n';
 import { useExternalSecretsProvider } from '@/composables/useExternalSecretsProvider';
 import { EXTERNAL_SECRETS_PROVIDER_MODAL_KEY } from '@/constants';
 import { DateTime } from 'luxon';
-import { computed, nextTick, onMounted, toRefs } from 'vue';
+import { computed, nextTick, onMounted, toRef } from 'vue';
+import { isDateObject } from '@/utils/typeGuards';
 
 const props = defineProps({
 	provider: {
@@ -24,15 +25,12 @@ const i18n = useI18n();
 const uiStore = useUIStore();
 const toast = useToast();
 
-const { provider } = toRefs(props) as Ref<ExternalSecretsProvider>;
-const providerData = computed(() => provider.value.data);
-const {
-	connectionState,
-	initialConnectionState,
-	normalizedProviderData,
-	testConnection,
-	setConnectionState,
-} = useExternalSecretsProvider(provider, providerData);
+const provider = toRef(props, 'provider');
+const providerData = computed(() => provider.value.data ?? {});
+const { connectionState, testConnection, setConnectionState } = useExternalSecretsProvider(
+	provider,
+	providerData,
+);
 
 const actionDropdownOptions = computed(() => [
 	{
@@ -50,11 +48,15 @@ const actionDropdownOptions = computed(() => [
 ]);
 
 const canConnect = computed(() => {
-	return props.provider.connected || Object.keys(props.provider.data).length > 0;
+	return props.provider.connected || Object.keys(providerData.value).length > 0;
 });
 
-const formattedDate = computed((provider: ExternalSecretsProvider) => {
-	return DateTime.fromISO(props.provider.connectedAt ?? new Date()).toFormat('dd LLL yyyy');
+const formattedDate = computed(() => {
+	return DateTime.fromISO(
+		isDateObject(provider.value.connectedAt)
+			? provider.value.connectedAt.toISOString()
+			: provider.value.connectedAt || new Date().toISOString(),
+	).toFormat('dd LLL yyyy');
 });
 
 onMounted(() => {
diff --git a/packages/editor-ui/src/components/ExternalSecretsProviderModal.ee.vue b/packages/editor-ui/src/components/ExternalSecretsProviderModal.ee.vue
index c44c4f8ee7569..84c3745a9f4c6 100644
--- a/packages/editor-ui/src/components/ExternalSecretsProviderModal.ee.vue
+++ b/packages/editor-ui/src/components/ExternalSecretsProviderModal.ee.vue
@@ -10,7 +10,6 @@ import { useMessage } from '@/composables/useMessage';
 import { useToast } from '@/composables/useToast';
 import { useExternalSecretsStore } from '@/stores/externalSecrets.ee.store';
 import { useUIStore } from '@/stores/ui.store';
-import { useRoute } from 'vue-router';
 import ParameterInputExpanded from '@/components/ParameterInputExpanded.vue';
 import type {
 	IUpdateInformation,
@@ -29,7 +28,7 @@ const props = defineProps({
 	},
 });
 
-const defaultProviderData = {
+const defaultProviderData: Record<string, Partial<ExternalSecretsProviderData>> = {
 	infisical: {
 		siteURL: 'https://app.infisical.com',
 	},
@@ -39,7 +38,6 @@ const externalSecretsStore = useExternalSecretsStore();
 const uiStore = useUIStore();
 const toast = useToast();
 const i18n = useI18n();
-const route = useRoute();
 const { confirm } = useMessage();
 
 const saving = ref(false);
@@ -50,7 +48,7 @@ const labelSize: IParameterLabel = { size: 'medium' };
 
 const provider = computed<ExternalSecretsProvider | undefined>(() =>
 	externalSecretsStore.providers.find((p) => p.name === props.data.name),
-) as Ref<ExternalSecretsProvider>;
+);
 const providerData = ref<ExternalSecretsProviderData>({});
 const {
 	connectionState,
@@ -64,7 +62,7 @@ const {
 const providerDataUpdated = computed(() => {
 	return Object.keys(providerData.value).find((key) => {
 		const value = providerData.value[key];
-		const originalValue = provider.value.data[key];
+		const originalValue = provider.value?.data?.[key];
 
 		return value !== originalValue;
 	});
@@ -72,7 +70,7 @@ const providerDataUpdated = computed(() => {
 
 const canSave = computed(
 	() =>
-		provider.value.properties
+		provider.value?.properties
 			?.filter((property) => property.required && shouldDisplayProperty(property))
 			.every((property) => {
 				const value = providerData.value[property.name];
@@ -82,21 +80,22 @@ const canSave = computed(
 
 onMounted(async () => {
 	try {
-		const provider = await externalSecretsStore.getProvider(props.data.name);
+		const fetchedProvider = await externalSecretsStore.getProvider(props.data.name);
+
 		providerData.value = {
 			...(defaultProviderData[props.data.name] || {}),
-			...provider.data,
+			...fetchedProvider.data,
 		};
 
-		setConnectionState(provider.state);
+		setConnectionState(fetchedProvider.state);
 
-		if (provider.connected) {
-			initialConnectionState.value = provider.state;
-		} else if (Object.keys(provider.data).length) {
+		if (fetchedProvider.connected) {
+			initialConnectionState.value = fetchedProvider.state;
+		} else if (Object.keys(fetchedProvider.data ?? {}).length) {
 			await testConnection();
 		}
 
-		if (provider.state === 'connected') {
+		if (fetchedProvider.state === 'connected') {
 			void externalSecretsStore.reloadProvider(props.data.name);
 		}
 	} catch (error) {
@@ -116,6 +115,10 @@ function onValueChange(updateInformation: IUpdateInformation) {
 }
 
 async function save() {
+	if (!provider.value) {
+		return;
+	}
+
 	try {
 		saving.value = true;
 		await externalSecretsStore.updateProvider(provider.value.name, {
@@ -143,7 +146,7 @@ async function onBeforeClose() {
 		const confirmModal = await confirm(
 			i18n.baseText('settings.externalSecrets.provider.closeWithoutSaving.description', {
 				interpolate: {
-					provider: provider.value.displayName,
+					provider: provider.value?.displayName ?? '',
 				},
 			}),
 			{
@@ -162,19 +165,23 @@ async function onBeforeClose() {
 
 	return true;
 }
+
+async function onConnectionStateChange() {
+	await testConnection();
+}
 </script>
 
 <template>
 	<Modal
 		id="external-secrets-provider-modal"
 		width="812px"
-		:title="provider.displayName"
+		:title="provider?.displayName"
 		:event-bus="data.eventBus"
 		:name="EXTERNAL_SECRETS_PROVIDER_MODAL_KEY"
 		:before-close="onBeforeClose"
 	>
 		<template #header>
-			<div :class="$style.header">
+			<div v-if="provider" :class="$style.header">
 				<div :class="$style.providerTitle">
 					<ExternalSecretsProviderImage :provider="provider" class="mr-xs" />
 					<span>{{ provider.displayName }}</span>
@@ -188,7 +195,7 @@ async function onBeforeClose() {
 						"
 						:event-bus="eventBus"
 						:provider="provider"
-						@change="testConnection"
+						@change="onConnectionStateChange"
 					/>
 					<n8n-button
 						type="primary"
@@ -207,7 +214,7 @@ async function onBeforeClose() {
 		</template>
 
 		<template #content>
-			<div :class="$style.container">
+			<div v-if="provider" :class="$style.container">
 				<hr class="mb-l" />
 				<div v-if="connectionState !== 'initializing'" class="mb-l">
 					<n8n-callout
diff --git a/packages/editor-ui/src/composables/useExternalSecretsProvider.ts b/packages/editor-ui/src/composables/useExternalSecretsProvider.ts
index 45ee1a1164a86..0b55a9693a2d6 100644
--- a/packages/editor-ui/src/composables/useExternalSecretsProvider.ts
+++ b/packages/editor-ui/src/composables/useExternalSecretsProvider.ts
@@ -1,36 +1,41 @@
 import type {
-	ExternalSecretsProviderWithProperties,
 	ExternalSecretsProvider,
 	IUpdateInformation,
 	ExternalSecretsProviderData,
+	ExternalSecretsProviderProperty,
+	ExternalSecretsProviderState,
 } from '@/Interface';
-import type { Ref } from 'vue';
+import type { ComputedRef, Ref } from 'vue';
 import { computed, ref } from 'vue';
 import { useExternalSecretsStore } from '@/stores/externalSecrets.ee.store';
 import { useToast } from '@/composables/useToast';
 
 export function useExternalSecretsProvider(
-	provider: Ref<ExternalSecretsProvider>,
+	provider:
+		| Ref<ExternalSecretsProvider | undefined>
+		| ComputedRef<ExternalSecretsProvider | undefined>,
 	providerData: Ref<ExternalSecretsProviderData>,
 ) {
 	const toast = useToast();
 	const externalSecretsStore = useExternalSecretsStore();
 
-	const initialConnectionState = ref<ExternalSecretsProviderWithProperties['state'] | undefined>(
-		'initializing',
-	);
+	const initialConnectionState = ref<ExternalSecretsProvider['state'] | undefined>('initializing');
 	const connectionState = computed(
-		() => externalSecretsStore.connectionState[provider.value?.name],
+		() => externalSecretsStore.connectionState[provider.value?.name ?? ''],
 	);
-	const setConnectionState = (state: ExternalSecretsProviderWithProperties['state']) => {
-		externalSecretsStore.setConnectionState(provider.value?.name, state);
+	const setConnectionState = (state: ExternalSecretsProvider['state']) => {
+		if (!provider.value) {
+			return;
+		}
+
+		externalSecretsStore.setConnectionState(provider.value.name, state);
 	};
 
 	const normalizedProviderData = computed(() => {
 		return Object.entries(providerData.value).reduce(
 			(acc, [key, value]) => {
-				const property = provider.value?.properties?.find((property) => property.name === key);
-				if (shouldDisplayProperty(property)) {
+				const property = provider.value?.properties?.find((p) => p.name === key);
+				if (property && shouldDisplayProperty(property)) {
 					acc[key] = value;
 				}
 
@@ -40,16 +45,14 @@ export function useExternalSecretsProvider(
 		);
 	});
 
-	function shouldDisplayProperty(
-		property: ExternalSecretsProviderWithProperties['properties'][0],
-	): boolean {
+	function shouldDisplayProperty(property: ExternalSecretsProviderProperty): boolean {
 		let visible = true;
 
 		if (property.displayOptions?.show) {
 			visible =
 				visible &&
 				Object.entries(property.displayOptions.show).every(([key, value]) => {
-					return value?.includes(providerData.value[key]);
+					return value?.includes(providerData.value[key] as string);
 				});
 		}
 
@@ -57,14 +60,20 @@ export function useExternalSecretsProvider(
 			visible =
 				visible &&
 				!Object.entries(property.displayOptions.hide).every(([key, value]) => {
-					return value?.includes(providerData.value[key]);
+					return value?.includes(providerData.value[key] as string);
 				});
 		}
 
 		return visible;
 	}
 
-	async function testConnection(options: { showError?: boolean } = { showError: true }) {
+	async function testConnection(
+		options: { showError?: boolean } = { showError: true },
+	): Promise<ExternalSecretsProviderState> {
+		if (!provider.value) {
+			return 'initializing';
+		}
+
 		try {
 			const { testState } = await externalSecretsStore.testProviderConnection(
 				provider.value.name,
diff --git a/packages/editor-ui/src/shims.d.ts b/packages/editor-ui/src/shims.d.ts
index b4da29d5927aa..3c637184d89c3 100644
--- a/packages/editor-ui/src/shims.d.ts
+++ b/packages/editor-ui/src/shims.d.ts
@@ -35,3 +35,13 @@ declare global {
 		findLast(predicate: (value: T, index: number, obj: T[]) => unknown, thisArg?: any): T;
 	}
 }
+
+declare module '*.webp' {
+	declare const path: string;
+	export default path;
+}
+
+declare module '*.svg' {
+	declare const path: string;
+	export default path;
+}
diff --git a/packages/editor-ui/src/utils/typeGuards.ts b/packages/editor-ui/src/utils/typeGuards.ts
index 73fd54ab19041..00e10a560eebb 100644
--- a/packages/editor-ui/src/utils/typeGuards.ts
+++ b/packages/editor-ui/src/utils/typeGuards.ts
@@ -43,3 +43,9 @@ export const isResourceMapperValue = (value: unknown): value is string | number
 export const isJSPlumbEndpointElement = (element: Node): element is HTMLElement => {
 	return 'jtk' in element && 'endpoint' in (element.jtk as object);
 };
+
+export function isDateObject(date: unknown): date is Date {
+	return (
+		!!date && Object.prototype.toString.call(date) === '[object Date]' && !isNaN(date as number)
+	);
+}

From ba9a03e413c0bb44e6c5a2040858e99a2fb2ee1d Mon Sep 17 00:00:00 2001
From: Alex Grozav <alex@grozav.com>
Date: Fri, 17 May 2024 11:24:23 +0300
Subject: [PATCH 2/5] fix: fix image imports

---
 .../src/components/ExternalSecretsProviderImage.ee.vue |  6 ++++++
 packages/editor-ui/src/shims.d.ts                      | 10 ----------
 packages/editor-ui/tsconfig.json                       |  2 +-
 3 files changed, 7 insertions(+), 11 deletions(-)

diff --git a/packages/editor-ui/src/components/ExternalSecretsProviderImage.ee.vue b/packages/editor-ui/src/components/ExternalSecretsProviderImage.ee.vue
index 4f72dc791811a..ae952cfb0c7db 100644
--- a/packages/editor-ui/src/components/ExternalSecretsProviderImage.ee.vue
+++ b/packages/editor-ui/src/components/ExternalSecretsProviderImage.ee.vue
@@ -1,10 +1,16 @@
 <script lang="ts" setup>
+/* eslint-disable @typescript-eslint/ban-ts-comment */
 import type { PropType } from 'vue';
 import type { ExternalSecretsProvider } from '@/Interface';
 import { computed } from 'vue';
+
+// @ts-expect-error
 import infisical from '../assets/images/infisical.webp';
+// @ts-expect-error
 import doppler from '../assets/images/doppler.webp';
+// @ts-expect-error
 import vault from '../assets/images/hashicorp.webp';
+// @ts-expect-error
 import awsSecretsManager from '../assets/images/aws-secrets-manager.svg';
 
 const props = defineProps({
diff --git a/packages/editor-ui/src/shims.d.ts b/packages/editor-ui/src/shims.d.ts
index 3c637184d89c3..b4da29d5927aa 100644
--- a/packages/editor-ui/src/shims.d.ts
+++ b/packages/editor-ui/src/shims.d.ts
@@ -35,13 +35,3 @@ declare global {
 		findLast(predicate: (value: T, index: number, obj: T[]) => unknown, thisArg?: any): T;
 	}
 }
-
-declare module '*.webp' {
-	declare const path: string;
-	export default path;
-}
-
-declare module '*.svg' {
-	declare const path: string;
-	export default path;
-}
diff --git a/packages/editor-ui/tsconfig.json b/packages/editor-ui/tsconfig.json
index 39cf47b586e5c..795872cf7f634 100644
--- a/packages/editor-ui/tsconfig.json
+++ b/packages/editor-ui/tsconfig.json
@@ -23,5 +23,5 @@
 		"useUnknownInCatchVariables": false,
 		"experimentalDecorators": true
 	},
-	"include": ["src/**/*.ts", "src/**/*.vue"]
+	"include": ["src/*.d.ts", "src/**/*.ts", "src/**/*.vue"]
 }

From 1c241c2847a0c05bd683c0f888914170b60e07b8 Mon Sep 17 00:00:00 2001
From: Alex Grozav <alex@grozav.com>
Date: Fri, 17 May 2024 11:27:48 +0300
Subject: [PATCH 3/5] fix: fix settings external secrets

---
 packages/editor-ui/src/Interface.ts | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/packages/editor-ui/src/Interface.ts b/packages/editor-ui/src/Interface.ts
index a634cf7ba852a..974f46538359a 100644
--- a/packages/editor-ui/src/Interface.ts
+++ b/packages/editor-ui/src/Interface.ts
@@ -1832,7 +1832,8 @@ export type CloudUpdateLinkSourceType =
 	| 'variables'
 	| 'community-nodes'
 	| 'workflow-history'
-	| 'worker-view';
+	| 'worker-view'
+	| 'external-secrets';
 
 export type UTMCampaign =
 	| 'upgrade-custom-data-filter'
@@ -1851,7 +1852,8 @@ export type UTMCampaign =
 	| 'upgrade-community-nodes'
 	| 'upgrade-workflow-history'
 	| 'upgrade-advanced-permissions'
-	| 'upgrade-worker-view';
+	| 'upgrade-worker-view'
+	| 'upgrade-external-secrets';
 
 export type N8nBanners = {
 	[key in BannerName]: {

From 5f02329c07d4c13e7dc59cb688f930609e00e839 Mon Sep 17 00:00:00 2001
From: Alex Grozav <alex@grozav.com>
Date: Fri, 17 May 2024 11:59:37 +0300
Subject: [PATCH 4/5] fix: fix image imports

---
 .../src/components/ExternalSecretsProviderImage.ee.vue   | 5 -----
 packages/editor-ui/src/shims.d.ts                        | 7 +++++++
 packages/editor-ui/tsconfig.json                         | 9 +++++++--
 3 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/packages/editor-ui/src/components/ExternalSecretsProviderImage.ee.vue b/packages/editor-ui/src/components/ExternalSecretsProviderImage.ee.vue
index ae952cfb0c7db..95c362820cb6a 100644
--- a/packages/editor-ui/src/components/ExternalSecretsProviderImage.ee.vue
+++ b/packages/editor-ui/src/components/ExternalSecretsProviderImage.ee.vue
@@ -1,16 +1,11 @@
 <script lang="ts" setup>
-/* eslint-disable @typescript-eslint/ban-ts-comment */
 import type { PropType } from 'vue';
 import type { ExternalSecretsProvider } from '@/Interface';
 import { computed } from 'vue';
 
-// @ts-expect-error
 import infisical from '../assets/images/infisical.webp';
-// @ts-expect-error
 import doppler from '../assets/images/doppler.webp';
-// @ts-expect-error
 import vault from '../assets/images/hashicorp.webp';
-// @ts-expect-error
 import awsSecretsManager from '../assets/images/aws-secrets-manager.svg';
 
 const props = defineProps({
diff --git a/packages/editor-ui/src/shims.d.ts b/packages/editor-ui/src/shims.d.ts
index b4da29d5927aa..112969867ba4d 100644
--- a/packages/editor-ui/src/shims.d.ts
+++ b/packages/editor-ui/src/shims.d.ts
@@ -35,3 +35,10 @@ declare global {
 		findLast(predicate: (value: T, index: number, obj: T[]) => unknown, thisArg?: any): T;
 	}
 }
+
+declare module '*.svg';
+declare module '*.png';
+declare module '*.jpg';
+declare module '*.jpeg';
+declare module '*.gif';
+declare module '*.webp';
diff --git a/packages/editor-ui/tsconfig.json b/packages/editor-ui/tsconfig.json
index 795872cf7f634..e0ffbaeacedea 100644
--- a/packages/editor-ui/tsconfig.json
+++ b/packages/editor-ui/tsconfig.json
@@ -11,7 +11,12 @@
 		"allowSyntheticDefaultImports": true,
 		"resolveJsonModule": true,
 		"baseUrl": ".",
-		"types": ["vitest/globals"],
+		"types": [
+			"vitest/globals",
+			"src/shims.d.ts",
+			"src/shims-vue.d.ts",
+			"src/v3-infinite-loading.d.ts"
+		],
 		"paths": {
 			"@/*": ["src/*"],
 			"n8n-design-system/*": ["../design-system/src/*"],
@@ -23,5 +28,5 @@
 		"useUnknownInCatchVariables": false,
 		"experimentalDecorators": true
 	},
-	"include": ["src/*.d.ts", "src/**/*.ts", "src/**/*.vue"]
+	"include": [ "src/**/*.ts", "src/**/*.d.ts", "src/**/*.vue"]
 }

From 23a71bad6686024c09a8f6a994edbbf2880779ee Mon Sep 17 00:00:00 2001
From: Alex Grozav <alex@grozav.com>
Date: Fri, 17 May 2024 12:00:52 +0300
Subject: [PATCH 5/5] chore: fix linting issue

---
 .../src/components/ExternalSecretsProviderModal.ee.vue          | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/editor-ui/src/components/ExternalSecretsProviderModal.ee.vue b/packages/editor-ui/src/components/ExternalSecretsProviderModal.ee.vue
index 84c3745a9f4c6..51b5efdc889a5 100644
--- a/packages/editor-ui/src/components/ExternalSecretsProviderModal.ee.vue
+++ b/packages/editor-ui/src/components/ExternalSecretsProviderModal.ee.vue
@@ -2,7 +2,7 @@
 import Modal from './Modal.vue';
 import { EXTERNAL_SECRETS_PROVIDER_MODAL_KEY, MODAL_CONFIRM } from '@/constants';
 import { computed, onMounted, ref } from 'vue';
-import type { PropType, Ref } from 'vue';
+import type { PropType } from 'vue';
 import type { EventBus } from 'n8n-design-system/utils';
 import { useExternalSecretsProvider } from '@/composables/useExternalSecretsProvider';
 import { useI18n } from '@/composables/useI18n';