From c0be43bdbed99c6bcceba662866667b8162dcf91 Mon Sep 17 00:00:00 2001
From: Alex Grozav <alex@grozav.com>
Date: Mon, 26 Feb 2024 16:05:12 +0200
Subject: [PATCH] fix(editor): Fix various typecheck issues (no-changelog)
 (#8739)

---
 packages/editor-ui/src/Interface.ts           | 21 +++++++-----
 .../src/components/ExecutionsList.vue         | 34 +++++++++----------
 .../ExecutionsView/ExecutionCard.vue          |  4 +--
 .../ExecutionsView/ExecutionsList.vue         | 18 +++++-----
 .../ExecutionsView/ExecutionsSidebar.vue      |  6 ++--
 .../__tests__/ExecutionPreview.test.ts        |  6 ++--
 .../src/components/MainHeader/MainHeader.vue  |  6 ++--
 packages/editor-ui/src/components/Node.vue    |  4 +--
 .../__tests__/ExecutionsList.test.ts          |  6 ++--
 .../__tests__/WorkflowPreview.test.ts         |  4 +--
 .../editor-ui/src/composables/useToast.ts     | 22 ++++++++----
 .../src/composables/useWorkflowHelpers.ts     |  3 +-
 .../editor-ui/src/mixins/executionsHelpers.ts |  8 ++---
 packages/editor-ui/src/stores/ui.store.ts     |  5 ++-
 .../editor-ui/src/stores/workflows.store.ts   | 14 ++++----
 packages/editor-ui/src/views/NodeView.vue     |  4 +--
 16 files changed, 91 insertions(+), 74 deletions(-)

diff --git a/packages/editor-ui/src/Interface.ts b/packages/editor-ui/src/Interface.ts
index 5f295a8d1..870e4d3df 100644
--- a/packages/editor-ui/src/Interface.ts
+++ b/packages/editor-ui/src/Interface.ts
@@ -34,7 +34,7 @@ import {
 	type INodeListSearchItems,
 	type NodeParameterValueType,
 	type IDisplayOptions,
-	type IExecutionsSummary,
+	type ExecutionSummary,
 	type FeatureFlags,
 	type ExecutionStatus,
 	type ITelemetryTrackProperties,
@@ -53,6 +53,7 @@ import type { BulkCommand, Undoable } from '@/models/history';
 import type { PartialBy, TupleToUnion } from '@/utils/typeHelpers';
 import type { Component } from 'vue';
 import type { Scope } from '@n8n/permissions';
+import type { NotificationOptions as ElementNotificationOptions } from 'element-plus';
 
 export * from 'n8n-design-system/types';
 
@@ -395,7 +396,7 @@ export interface IExecutionShortResponse {
 
 export interface IExecutionsListResponse {
 	count: number;
-	results: IExecutionsSummary[];
+	results: ExecutionSummary[];
 	estimated: boolean;
 }
 
@@ -1061,8 +1062,8 @@ export interface IUsedCredential {
 export interface WorkflowsState {
 	activeExecutions: IExecutionsCurrentSummaryExtended[];
 	activeWorkflows: string[];
-	activeWorkflowExecution: IExecutionsSummary | null;
-	currentWorkflowExecutions: IExecutionsSummary[];
+	activeWorkflowExecution: ExecutionSummary | null;
+	currentWorkflowExecutions: ExecutionSummary[];
 	activeExecutionId: string | null;
 	executingNode: string[];
 	executionWaitingForWebhook: boolean;
@@ -1189,9 +1190,9 @@ export type ModalState = {
 	httpNodeParameters?: string;
 };
 
-export type NewCredentialsModal = ModalState & {
+export interface NewCredentialsModal extends ModalState {
 	showAuthSelector?: boolean;
-};
+}
 
 export type IRunDataDisplayMode = 'table' | 'json' | 'binary' | 'schema' | 'html' | 'ai';
 export type NodePanelType = 'input' | 'output';
@@ -1240,6 +1241,10 @@ export interface NDVState {
 	isMappingOnboarded: boolean;
 }
 
+export interface NotificationOptions extends Partial<ElementNotificationOptions> {
+	message: string | ElementNotificationOptions['message'];
+}
+
 export interface UIState {
 	activeActions: string[];
 	activeCredentialType: string | null;
@@ -1398,8 +1403,8 @@ export interface IUsersState {
 }
 
 export interface IWorkflowsState {
-	currentWorkflowExecutions: IExecutionsSummary[];
-	activeWorkflowExecution: IExecutionsSummary | null;
+	currentWorkflowExecutions: ExecutionSummary[];
+	activeWorkflowExecution: ExecutionSummary | null;
 	finishedExecutionsCount: number;
 }
 export interface IWorkflowsMap {
diff --git a/packages/editor-ui/src/components/ExecutionsList.vue b/packages/editor-ui/src/components/ExecutionsList.vue
index b9e6cb633..189e879e1 100644
--- a/packages/editor-ui/src/components/ExecutionsList.vue
+++ b/packages/editor-ui/src/components/ExecutionsList.vue
@@ -301,7 +301,7 @@ import type {
 	ExecutionFilterType,
 	ExecutionsQueryFilter,
 } from '@/Interface';
-import type { IExecutionsSummary, ExecutionStatus } from 'n8n-workflow';
+import type { ExecutionSummary, ExecutionStatus } from 'n8n-workflow';
 import { range as _range } from 'lodash-es';
 import { useUIStore } from '@/stores/ui.store';
 import { useWorkflowsStore } from '@/stores/workflows.store';
@@ -342,7 +342,7 @@ export default defineComponent({
 	data() {
 		return {
 			isMounting: true,
-			finishedExecutions: [] as IExecutionsSummary[],
+			finishedExecutions: [] as ExecutionSummary[],
 			finishedExecutionsCount: 0,
 			finishedExecutionsCountEstimated: false,
 
@@ -388,8 +388,8 @@ export default defineComponent({
 		activeExecutions(): IExecutionsCurrentSummaryExtended[] {
 			return this.workflowsStore.activeExecutions;
 		},
-		combinedExecutions(): IExecutionsSummary[] {
-			const returnData: IExecutionsSummary[] = [];
+		combinedExecutions(): ExecutionSummary[] {
+			const returnData: ExecutionSummary[] = [];
 
 			if (['all', 'running'].includes(this.filter.status)) {
 				returnData.push(...this.activeExecutions);
@@ -428,7 +428,7 @@ export default defineComponent({
 		closeDialog() {
 			this.$emit('closeModal');
 		},
-		displayExecution(execution: IExecutionsSummary) {
+		displayExecution(execution: ExecutionSummary) {
 			const route = this.$router.resolve({
 				name: VIEWS.EXECUTION_PREVIEW,
 				params: { name: execution.workflowId, executionId: execution.id },
@@ -529,7 +529,7 @@ export default defineComponent({
 			this.handleClearSelection();
 			this.isMounting = false;
 		},
-		async handleActionItemClick(commandData: { command: string; execution: IExecutionsSummary }) {
+		async handleActionItemClick(commandData: { command: string; execution: ExecutionSummary }) {
 			if (['currentlySaved', 'original'].includes(commandData.command)) {
 				let loadWorkflow = false;
 				if (commandData.command === 'currentlySaved') {
@@ -747,7 +747,7 @@ export default defineComponent({
 				this.showError(error, this.i18n.baseText('executionsList.showError.loadWorkflows.title'));
 			}
 		},
-		async retryExecution(execution: IExecutionsSummary, loadWorkflow?: boolean) {
+		async retryExecution(execution: ExecutionSummary, loadWorkflow?: boolean) {
 			this.isDataLoading = true;
 
 			try {
@@ -786,7 +786,7 @@ export default defineComponent({
 
 			this.isDataLoading = false;
 		},
-		getStatus(execution: IExecutionsSummary): ExecutionStatus {
+		getStatus(execution: ExecutionSummary): ExecutionStatus {
 			if (execution.status) {
 				return execution.status;
 			} else {
@@ -806,10 +806,10 @@ export default defineComponent({
 				return status;
 			}
 		},
-		getRowClass(execution: IExecutionsSummary): string {
+		getRowClass(execution: ExecutionSummary): string {
 			return [this.$style.execRow, this.$style[this.getStatus(execution)]].join(' ');
 		},
-		getStatusText(entry: IExecutionsSummary): string {
+		getStatusText(entry: ExecutionSummary): string {
 			const status = this.getStatus(entry);
 			let text = '';
 
@@ -833,7 +833,7 @@ export default defineComponent({
 
 			return text;
 		},
-		getStatusTextTranslationPath(entry: IExecutionsSummary): string {
+		getStatusTextTranslationPath(entry: ExecutionSummary): string {
 			const status = this.getStatus(entry);
 			let path = '';
 
@@ -857,7 +857,7 @@ export default defineComponent({
 
 			return path;
 		},
-		getStatusTooltipText(entry: IExecutionsSummary): string {
+		getStatusTooltipText(entry: ExecutionSummary): string {
 			const status = this.getStatus(entry);
 			let text = '';
 
@@ -894,7 +894,7 @@ export default defineComponent({
 				this.showError(error, this.i18n.baseText('executionsList.showError.stopExecution.title'));
 			}
 		},
-		isExecutionRetriable(execution: IExecutionsSummary): boolean {
+		isExecutionRetriable(execution: ExecutionSummary): boolean {
 			return (
 				execution.stoppedAt !== undefined &&
 				!execution.finished &&
@@ -903,7 +903,7 @@ export default defineComponent({
 				!execution.waitTill
 			);
 		},
-		async deleteExecution(execution: IExecutionsSummary) {
+		async deleteExecution(execution: ExecutionSummary) {
 			this.isDataLoading = true;
 			try {
 				await this.workflowsStore.deleteExecutions({ ids: [execution.id] });
@@ -921,17 +921,17 @@ export default defineComponent({
 			}
 			this.isDataLoading = true;
 		},
-		isWaitTillIndefinite(execution: IExecutionsSummary): boolean {
+		isWaitTillIndefinite(execution: ExecutionSummary): boolean {
 			if (!execution.waitTill) {
 				return false;
 			}
 			return new Date(execution.waitTill).toISOString() === WAIT_TIME_UNLIMITED;
 		},
-		isRunning(execution: IExecutionsSummary): boolean {
+		isRunning(execution: ExecutionSummary): boolean {
 			return this.getStatus(execution) === 'running';
 		},
 		selectAllVisibleExecutions() {
-			this.combinedExecutions.forEach((execution: IExecutionsSummary) => {
+			this.combinedExecutions.forEach((execution: ExecutionSummary) => {
 				this.selectedItems = { ...this.selectedItems, [execution.id]: true };
 			});
 		},
diff --git a/packages/editor-ui/src/components/ExecutionsView/ExecutionCard.vue b/packages/editor-ui/src/components/ExecutionsView/ExecutionCard.vue
index 8d6923956..cccdd3320 100644
--- a/packages/editor-ui/src/components/ExecutionsView/ExecutionCard.vue
+++ b/packages/editor-ui/src/components/ExecutionsView/ExecutionCard.vue
@@ -83,7 +83,7 @@
 
 <script lang="ts">
 import { defineComponent } from 'vue';
-import type { IExecutionsSummary } from '@/Interface';
+import type { ExecutionSummary } from '@/Interface';
 import type { IExecutionUIData } from '@/mixins/executionsHelpers';
 import { executionHelpers } from '@/mixins/executionsHelpers';
 import { VIEWS } from '@/constants';
@@ -97,7 +97,7 @@ export default defineComponent({
 	mixins: [executionHelpers],
 	props: {
 		execution: {
-			type: Object as () => IExecutionsSummary,
+			type: Object as () => ExecutionSummary,
 			required: true,
 		},
 		highlight: {
diff --git a/packages/editor-ui/src/components/ExecutionsView/ExecutionsList.vue b/packages/editor-ui/src/components/ExecutionsView/ExecutionsList.vue
index 4a20a89a0..e8c4d15bb 100644
--- a/packages/editor-ui/src/components/ExecutionsView/ExecutionsList.vue
+++ b/packages/editor-ui/src/components/ExecutionsView/ExecutionsList.vue
@@ -44,7 +44,7 @@ import type {
 	IWorkflowDb,
 } from '@/Interface';
 import type {
-	IExecutionsSummary,
+	ExecutionSummary,
 	IConnection,
 	IConnections,
 	IDataObject,
@@ -100,7 +100,7 @@ export default defineComponent({
 			loading: false,
 			loadingMore: false,
 			filter: {} as ExecutionFilterType,
-			temporaryExecution: null as IExecutionsSummary | null,
+			temporaryExecution: null as ExecutionSummary | null,
 			autoRefresh: false,
 			autoRefreshTimeout: undefined as undefined | NodeJS.Timer,
 		};
@@ -284,7 +284,7 @@ export default defineComponent({
 			this.loading = true;
 			try {
 				const executionIndex = this.executions.findIndex(
-					(execution: IExecutionsSummary) => execution.id === this.$route.params.executionId,
+					(execution: ExecutionSummary) => execution.id === this.$route.params.executionId,
 				);
 				const nextExecution =
 					this.executions[executionIndex + 1] ||
@@ -388,8 +388,8 @@ export default defineComponent({
 		},
 		async loadAutoRefresh(): Promise<void> {
 			// Most of the auto-refresh logic is taken from the `ExecutionsList` component
-			const fetchedExecutions: IExecutionsSummary[] = await this.loadExecutions();
-			let existingExecutions: IExecutionsSummary[] = [...this.executions];
+			const fetchedExecutions: ExecutionSummary[] = await this.loadExecutions();
+			let existingExecutions: ExecutionSummary[] = [...this.executions];
 			const alreadyPresentExecutionIds = existingExecutions.map((exec) => parseInt(exec.id, 10));
 			let lastId = 0;
 			const gaps = [] as number[];
@@ -465,7 +465,7 @@ export default defineComponent({
 				}
 			}
 		},
-		async loadExecutions(): Promise<IExecutionsSummary[]> {
+		async loadExecutions(): Promise<ExecutionSummary[]> {
 			if (!this.currentWorkflow) {
 				return [];
 			}
@@ -536,7 +536,7 @@ export default defineComponent({
 					);
 					return;
 				} else {
-					this.temporaryExecution = existingExecution as IExecutionsSummary;
+					this.temporaryExecution = existingExecution as ExecutionSummary;
 				}
 			}
 			// stop if the execution wasn't found in the first 1000 lookups
@@ -716,7 +716,7 @@ export default defineComponent({
 		async loadActiveWorkflows(): Promise<void> {
 			await this.workflowsStore.fetchActiveWorkflows();
 		},
-		async onRetryExecution(payload: { execution: IExecutionsSummary; command: string }) {
+		async onRetryExecution(payload: { execution: ExecutionSummary; command: string }) {
 			const loadWorkflow = payload.command === 'current-workflow';
 
 			this.showMessage({
@@ -733,7 +733,7 @@ export default defineComponent({
 				retry_type: loadWorkflow ? 'current' : 'original',
 			});
 		},
-		async retryExecution(execution: IExecutionsSummary, loadWorkflow?: boolean) {
+		async retryExecution(execution: ExecutionSummary, loadWorkflow?: boolean) {
 			try {
 				const retrySuccessful = await this.workflowsStore.retryExecution(
 					execution.id,
diff --git a/packages/editor-ui/src/components/ExecutionsView/ExecutionsSidebar.vue b/packages/editor-ui/src/components/ExecutionsView/ExecutionsSidebar.vue
index 3778d91b3..2ac984ee4 100644
--- a/packages/editor-ui/src/components/ExecutionsView/ExecutionsSidebar.vue
+++ b/packages/editor-ui/src/components/ExecutionsView/ExecutionsSidebar.vue
@@ -64,7 +64,7 @@ import ExecutionCard from '@/components/ExecutionsView/ExecutionCard.vue';
 import ExecutionsInfoAccordion from '@/components/ExecutionsView/ExecutionsInfoAccordion.vue';
 import ExecutionFilter from '@/components/ExecutionFilter.vue';
 import { VIEWS } from '@/constants';
-import type { IExecutionsSummary } from 'n8n-workflow';
+import type { ExecutionSummary } from 'n8n-workflow';
 import type { Route } from 'vue-router';
 import { defineComponent } from 'vue';
 import type { PropType } from 'vue';
@@ -88,7 +88,7 @@ export default defineComponent({
 			default: false,
 		},
 		executions: {
-			type: Array as PropType<IExecutionsSummary[]>,
+			type: Array as PropType<ExecutionSummary[]>,
 			required: true,
 		},
 		loading: {
@@ -100,7 +100,7 @@ export default defineComponent({
 			default: false,
 		},
 		temporaryExecution: {
-			type: Object as PropType<IExecutionsSummary>,
+			type: Object as PropType<ExecutionSummary>,
 			default: null,
 		},
 	},
diff --git a/packages/editor-ui/src/components/ExecutionsView/__tests__/ExecutionPreview.test.ts b/packages/editor-ui/src/components/ExecutionsView/__tests__/ExecutionPreview.test.ts
index a1715e7a2..089a09414 100644
--- a/packages/editor-ui/src/components/ExecutionsView/__tests__/ExecutionPreview.test.ts
+++ b/packages/editor-ui/src/components/ExecutionsView/__tests__/ExecutionPreview.test.ts
@@ -4,7 +4,7 @@ import userEvent from '@testing-library/user-event';
 import { faker } from '@faker-js/faker';
 import { createRouter, createWebHistory } from 'vue-router';
 import { createPinia, PiniaVuePlugin, setActivePinia } from 'pinia';
-import type { IExecutionsSummary } from 'n8n-workflow';
+import type { ExecutionSummary } from 'n8n-workflow';
 import { useSettingsStore } from '@/stores/settings.store';
 import { useWorkflowsStore } from '@/stores/workflows.store';
 import ExecutionPreview from '@/components/ExecutionsView/ExecutionPreview.vue';
@@ -48,7 +48,7 @@ const generateUndefinedNullOrString = () => {
 	}
 };
 
-const executionDataFactory = (): IExecutionsSummary => ({
+const executionDataFactory = (): ExecutionSummary => ({
 	id: faker.string.uuid(),
 	finished: faker.datatype.boolean(),
 	mode: faker.helpers.arrayElement(['manual', 'trigger']),
@@ -65,7 +65,7 @@ const executionDataFactory = (): IExecutionsSummary => ({
 describe('ExecutionPreview.vue', () => {
 	let workflowsStore: ReturnType<typeof useWorkflowsStore>;
 	let settingsStore: ReturnType<typeof useSettingsStore>;
-	const executionData: IExecutionsSummary = executionDataFactory();
+	const executionData: ExecutionSummary = executionDataFactory();
 
 	beforeEach(() => {
 		pinia = createPinia();
diff --git a/packages/editor-ui/src/components/MainHeader/MainHeader.vue b/packages/editor-ui/src/components/MainHeader/MainHeader.vue
index 08c549784..a84f0a829 100644
--- a/packages/editor-ui/src/components/MainHeader/MainHeader.vue
+++ b/packages/editor-ui/src/components/MainHeader/MainHeader.vue
@@ -18,7 +18,7 @@
 import { defineComponent } from 'vue';
 import type { Route, RouteLocationRaw } from 'vue-router';
 import { mapStores } from 'pinia';
-import type { IExecutionsSummary } from 'n8n-workflow';
+import type { ExecutionSummary } from 'n8n-workflow';
 import { pushConnection } from '@/mixins/pushConnection';
 import WorkflowDetails from '@/components/MainHeader/WorkflowDetails.vue';
 import TabBar from '@/components/MainHeader/TabBar.vue';
@@ -79,8 +79,8 @@ export default defineComponent({
 				(this.$route.meta.nodeView || this.$route.meta.keepWorkflowAlive === true)
 			);
 		},
-		activeExecution(): IExecutionsSummary {
-			return this.workflowsStore.activeWorkflowExecution as IExecutionsSummary;
+		activeExecution(): ExecutionSummary {
+			return this.workflowsStore.activeWorkflowExecution as ExecutionSummary;
 		},
 		readOnly(): boolean {
 			return this.sourceControlStore.preferences.branchReadOnly;
diff --git a/packages/editor-ui/src/components/Node.vue b/packages/editor-ui/src/components/Node.vue
index daf97fe81..a4e1b56e0 100644
--- a/packages/editor-ui/src/components/Node.vue
+++ b/packages/editor-ui/src/components/Node.vue
@@ -191,7 +191,7 @@ import {
 import { nodeBase } from '@/mixins/nodeBase';
 import type {
 	ConnectionTypes,
-	IExecutionsSummary,
+	ExecutionSummary,
 	INodeInputConfiguration,
 	INodeOutputConfiguration,
 	INodeTypeDescription,
@@ -494,7 +494,7 @@ export default defineComponent({
 			return this.data.name;
 		},
 		waiting(): string | undefined {
-			const workflowExecution = this.workflowsStore.getWorkflowExecution as IExecutionsSummary;
+			const workflowExecution = this.workflowsStore.getWorkflowExecution as ExecutionSummary;
 
 			if (workflowExecution?.waitTill) {
 				const lastNodeExecuted = get(workflowExecution, 'data.resultData.lastNodeExecuted');
diff --git a/packages/editor-ui/src/components/__tests__/ExecutionsList.test.ts b/packages/editor-ui/src/components/__tests__/ExecutionsList.test.ts
index 65c6ce2e8..c3b24d4ed 100644
--- a/packages/editor-ui/src/components/__tests__/ExecutionsList.test.ts
+++ b/packages/editor-ui/src/components/__tests__/ExecutionsList.test.ts
@@ -6,7 +6,7 @@ import { faker } from '@faker-js/faker';
 import { STORES, VIEWS } from '@/constants';
 import ExecutionsList from '@/components/ExecutionsList.vue';
 import type { IWorkflowDb } from '@/Interface';
-import type { IExecutionsSummary } from 'n8n-workflow';
+import type { ExecutionSummary } from 'n8n-workflow';
 import { retry, SETTINGS_STORE_DEFAULT_STATE, waitAllPromises } from '@/__tests__/utils';
 import { useWorkflowsStore } from '@/stores/workflows.store';
 import type { RenderOptions } from '@/__tests__/render';
@@ -48,7 +48,7 @@ const workflowDataFactory = (): IWorkflowDb => ({
 	versionId: faker.number.int().toString(),
 });
 
-const executionDataFactory = (): IExecutionsSummary => ({
+const executionDataFactory = (): ExecutionSummary => ({
 	id: faker.string.uuid(),
 	finished: faker.datatype.boolean(),
 	mode: faker.helpers.arrayElement(['manual', 'trigger']),
@@ -89,7 +89,7 @@ describe('ExecutionsList.vue', () => {
 	let workflowsData: IWorkflowDb[];
 	let executionsData: Array<{
 		count: number;
-		results: IExecutionsSummary[];
+		results: ExecutionSummary[];
 		estimated: boolean;
 	}>;
 
diff --git a/packages/editor-ui/src/components/__tests__/WorkflowPreview.test.ts b/packages/editor-ui/src/components/__tests__/WorkflowPreview.test.ts
index fe9c76162..dcda41cea 100644
--- a/packages/editor-ui/src/components/__tests__/WorkflowPreview.test.ts
+++ b/packages/editor-ui/src/components/__tests__/WorkflowPreview.test.ts
@@ -1,7 +1,7 @@
 import { vi } from 'vitest';
 import { createPinia, setActivePinia } from 'pinia';
 import { waitFor } from '@testing-library/vue';
-import type { IExecutionsSummary } from 'n8n-workflow';
+import type { ExecutionSummary } from 'n8n-workflow';
 import { createComponentRenderer } from '@/__tests__/render';
 import type { INodeUi, IWorkflowDb } from '@/Interface';
 import WorkflowPreview from '@/components/WorkflowPreview.vue';
@@ -152,7 +152,7 @@ describe('WorkflowPreview', () => {
 	it('should call also iframe postMessage with "setActiveExecution" if active execution is set', async () => {
 		vi.spyOn(workflowsStore, 'activeWorkflowExecution', 'get').mockReturnValue({
 			id: 'abc',
-		} as IExecutionsSummary);
+		} as ExecutionSummary);
 
 		const executionId = '123';
 		renderComponent({
diff --git a/packages/editor-ui/src/composables/useToast.ts b/packages/editor-ui/src/composables/useToast.ts
index fb5f12a61..f32d1bf6a 100644
--- a/packages/editor-ui/src/composables/useToast.ts
+++ b/packages/editor-ui/src/composables/useToast.ts
@@ -1,5 +1,6 @@
 import { ElNotification as Notification } from 'element-plus';
-import type { NotificationInstance, NotificationOptions, MessageBoxState } from 'element-plus';
+import type { NotificationHandle, MessageBoxState } from 'element-plus';
+import type { NotificationOptions } from '@/Interface';
 import { sanitizeHtml } from '@/utils/htmlUtils';
 import { useTelemetry } from '@/composables/useTelemetry';
 import { useWorkflowsStore } from '@/stores/workflows.store';
@@ -8,12 +9,19 @@ import { useI18n } from './useI18n';
 import { useExternalHooks } from './useExternalHooks';
 import { VIEWS } from '@/constants';
 
+export interface NotificationErrorWithNodeAndDescription extends Error {
+	node: {
+		name: string;
+	};
+	description: string;
+}
+
 const messageDefaults: Partial<Omit<NotificationOptions, 'message'>> = {
 	dangerouslyUseHTMLString: true,
 	position: 'bottom-right',
 };
 
-const stickyNotificationQueue: NotificationInstance[] = [];
+const stickyNotificationQueue: NotificationHandle[] = [];
 
 export function useToast() {
 	const telemetry = useTelemetry();
@@ -39,7 +47,7 @@ export function useToast() {
 			telemetry.track('Instance FE emitted error', {
 				error_title: messageData.title,
 				error_message: messageData.message,
-				caused_by_credential: causedByCredential(messageData.message),
+				caused_by_credential: causedByCredential(messageData.message as string),
 				workflow_id: workflowsStore.workflowId,
 			});
 		}
@@ -59,7 +67,7 @@ export function useToast() {
 		dangerouslyUseHTMLString?: boolean;
 	}) {
 		// eslint-disable-next-line prefer-const
-		let notification: NotificationInstance;
+		let notification: NotificationHandle;
 		if (config.closeOnClick) {
 			const cb = config.onClick;
 			config.onClick = () => {
@@ -87,7 +95,7 @@ export function useToast() {
 		return notification;
 	}
 
-	function collapsableDetails({ description, node }: Error) {
+	function collapsableDetails({ description, node }: NotificationErrorWithNodeAndDescription) {
 		if (!description) return '';
 
 		const errorDescription =
@@ -116,7 +124,7 @@ export function useToast() {
 				message: `
 					${messageLine}
 					<i>${error.message}</i>
-					${collapsableDetails(error)}`,
+					${collapsableDetails(error as NotificationErrorWithNodeAndDescription)}`,
 				type: 'error',
 				duration: 0,
 			},
@@ -138,7 +146,7 @@ export function useToast() {
 		});
 	}
 
-	function showAlert(config: NotificationOptions): NotificationInstance {
+	function showAlert(config: NotificationOptions): NotificationHandle {
 		return Notification(config);
 	}
 
diff --git a/packages/editor-ui/src/composables/useWorkflowHelpers.ts b/packages/editor-ui/src/composables/useWorkflowHelpers.ts
index a3d2d5b88..27e24e336 100644
--- a/packages/editor-ui/src/composables/useWorkflowHelpers.ts
+++ b/packages/editor-ui/src/composables/useWorkflowHelpers.ts
@@ -37,6 +37,7 @@ import type {
 	INodeTypesMaxCount,
 	INodeUi,
 	ITag,
+	IUpdateInformation,
 	IWorkflowData,
 	IWorkflowDataUpdate,
 	IWorkflowDb,
@@ -1021,7 +1022,7 @@ export function useWorkflowHelpers(router: Router) {
 					key: 'webhookId',
 					value: changedNodes[nodeName],
 					name: nodeName,
-				};
+				} as IUpdateInformation;
 				workflowsStore.setNodeValue(changes);
 			});
 
diff --git a/packages/editor-ui/src/mixins/executionsHelpers.ts b/packages/editor-ui/src/mixins/executionsHelpers.ts
index 116b64b14..c429bc139 100644
--- a/packages/editor-ui/src/mixins/executionsHelpers.ts
+++ b/packages/editor-ui/src/mixins/executionsHelpers.ts
@@ -2,7 +2,7 @@ import { defineComponent } from 'vue';
 import { mapStores } from 'pinia';
 import { useWorkflowsStore } from '@/stores/workflows.store';
 import { i18n as locale } from '@/plugins/i18n';
-import type { IExecutionsSummary } from 'n8n-workflow';
+import type { ExecutionSummary } from 'n8n-workflow';
 import { convertToDisplayDate } from '@/utils/formatters/dateFormatter';
 
 export interface IExecutionUIData {
@@ -24,15 +24,15 @@ export const executionHelpers = defineComponent({
 		currentWorkflow(): string {
 			return this.$route.params.name || this.workflowsStore.workflowId;
 		},
-		executions(): IExecutionsSummary[] {
+		executions(): ExecutionSummary[] {
 			return this.workflowsStore.currentWorkflowExecutions;
 		},
-		activeExecution(): IExecutionsSummary | null {
+		activeExecution(): ExecutionSummary | null {
 			return this.workflowsStore.activeWorkflowExecution;
 		},
 	},
 	methods: {
-		getExecutionUIDetails(execution: IExecutionsSummary): IExecutionUIData {
+		getExecutionUIDetails(execution: ExecutionSummary): IExecutionUIData {
 			const status = {
 				name: 'unknown',
 				startTime: this.formatDate(execution.startedAt),
diff --git a/packages/editor-ui/src/stores/ui.store.ts b/packages/editor-ui/src/stores/ui.store.ts
index ff2444ef7..2b98ac50d 100644
--- a/packages/editor-ui/src/stores/ui.store.ts
+++ b/packages/editor-ui/src/stores/ui.store.ts
@@ -55,6 +55,8 @@ import type {
 	ThemeOption,
 	AppliedThemeOption,
 	SuggestedTemplates,
+	NotificationOptions,
+	ModalState,
 } from '@/Interface';
 import { defineStore } from 'pinia';
 import { useRootStore } from '@/stores/n8nRoot.store';
@@ -144,7 +146,7 @@ export const useUIStore = defineStore(STORES.UI, {
 				mode: '',
 				activeId: null,
 				showAuthSelector: false,
-			},
+			} as ModalState,
 		},
 		modalStack: [],
 		sidebarMenuCollapsed: true,
@@ -172,6 +174,7 @@ export const useUIStore = defineStore(STORES.UI, {
 		stateIsDirty: false,
 		lastSelectedNode: null,
 		lastSelectedNodeOutputIndex: null,
+		lastSelectedNodeEndpointUuid: null,
 		nodeViewOffsetPosition: [0, 0],
 		nodeViewMoveInProgress: false,
 		selectedNodes: [],
diff --git a/packages/editor-ui/src/stores/workflows.store.ts b/packages/editor-ui/src/stores/workflows.store.ts
index 9489b9f95..074c14916 100644
--- a/packages/editor-ui/src/stores/workflows.store.ts
+++ b/packages/editor-ui/src/stores/workflows.store.ts
@@ -40,7 +40,7 @@ import type {
 	IConnection,
 	IConnections,
 	IDataObject,
-	IExecutionsSummary,
+	ExecutionSummary,
 	INode,
 	INodeConnections,
 	INodeCredentials,
@@ -263,11 +263,11 @@ export const useWorkflowsStore = defineStore(STORES.WORKFLOWS, {
 			return (nodeName: string) => this.executingNode.includes(nodeName);
 		},
 		// Executions getters
-		getExecutionDataById(): (id: string) => IExecutionsSummary | undefined {
-			return (id: string): IExecutionsSummary | undefined =>
+		getExecutionDataById(): (id: string) => ExecutionSummary | undefined {
+			return (id: string): ExecutionSummary | undefined =>
 				this.currentWorkflowExecutions.find((execution) => execution.id === id);
 		},
-		getAllLoadedFinishedExecutions(): IExecutionsSummary[] {
+		getAllLoadedFinishedExecutions(): ExecutionSummary[] {
 			return this.currentWorkflowExecutions.filter(
 				(ex) => ex.finished === true || ex.stoppedAt !== undefined,
 			);
@@ -1366,7 +1366,7 @@ export const useWorkflowsStore = defineStore(STORES.WORKFLOWS, {
 
 		async loadCurrentWorkflowExecutions(
 			requestFilter: ExecutionsQueryFilter,
-		): Promise<IExecutionsSummary[]> {
+		): Promise<ExecutionSummary[]> {
 			let activeExecutions = [];
 
 			if (!requestFilter.workflowId) {
@@ -1392,11 +1392,11 @@ export const useWorkflowsStore = defineStore(STORES.WORKFLOWS, {
 			return await getExecutionData(rootStore.getRestApiContext, executionId);
 		},
 
-		deleteExecution(execution: IExecutionsSummary): void {
+		deleteExecution(execution: ExecutionSummary): void {
 			this.currentWorkflowExecutions.splice(this.currentWorkflowExecutions.indexOf(execution), 1);
 		},
 
-		addToCurrentExecutions(executions: IExecutionsSummary[]): void {
+		addToCurrentExecutions(executions: ExecutionSummary[]): void {
 			executions.forEach((execution) => {
 				const exists = this.currentWorkflowExecutions.find((ex) => ex.id === execution.id);
 				if (!exists && execution.workflowId === this.workflowId) {
diff --git a/packages/editor-ui/src/views/NodeView.vue b/packages/editor-ui/src/views/NodeView.vue
index 24bc312eb..8c7691a8f 100644
--- a/packages/editor-ui/src/views/NodeView.vue
+++ b/packages/editor-ui/src/views/NodeView.vue
@@ -272,7 +272,7 @@ import type {
 	IConnection,
 	IConnections,
 	IDataObject,
-	IExecutionsSummary,
+	ExecutionSummary,
 	INode,
 	INodeConnections,
 	INodeCredentialsDetails,
@@ -1280,7 +1280,7 @@ export default defineComponent({
 					});
 				}
 			}
-			if ((data as IExecutionsSummary).waitTill) {
+			if ((data as ExecutionSummary).waitTill) {
 				this.showMessage({
 					title: this.$locale.baseText('nodeView.thisExecutionHasntFinishedYet'),
 					message: `<a data-action="reload">${this.$locale.baseText(