refactor: Remove usless catch blocks, and add a linting rule to prevent them (no-changelog) (#12730)

packages/@n8n/nodes-langchain/nodes/vector_store/shared/createVectorStoreNode.ts

@@ -392,13 +392,9 @@ export const createVectorStoreNode = (args: VectorStoreNodeConstructorArgs) =>
 					);
 					resultData.push(...serializedDocuments);
 
-					try {
 					await args.populateVectorStore(this, embeddings, processedDocuments, itemIndex);
 
 					logAiEvent(this, 'ai-vector-store-populated');
-					} catch (error) {
-						throw error;
-					}
 				}
 
 				return [resultData];
@@ -443,16 +439,12 @@ export const createVectorStoreNode = (args: VectorStoreNodeConstructorArgs) =>
 
 					resultData.push(...serializedDocuments);
 
-					try {
 					// Use ids option to upsert instead of insert
 					await vectorStore.addDocuments(processedDocuments, {
 						ids: [documentId],
 					});
 
 					logAiEvent(this, 'ai-vector-store-updated');
-					} catch (error) {
-						throw error;
-					}
 				}
 
 				return [resultData];

packages/@n8n_io/eslint-config/base.js

@@ -369,6 +369,8 @@ const config = (module.exports = {
 
 		'n8n-local-rules/no-unused-param-in-catch-clause': 'error',
 
+		'n8n-local-rules/no-useless-catch-throw': 'error',
+
 		'n8n-local-rules/no-plain-errors': 'error',
 
 		// ******************************************************************

packages/@n8n_io/eslint-config/local-rules.js

@@ -172,6 +172,49 @@ module.exports = {
 		},
 	},
 
+	'no-useless-catch-throw': {
+		meta: {
+			type: 'problem',
+			docs: {
+				description: 'Disallow `try-catch` blocks where the `catch` only contains a `throw error`.',
+				recommended: 'error',
+			},
+			messages: {
+				noUselessCatchThrow: 'Remove useless `catch` block.',
+			},
+			fixable: 'code',
+		},
+		create(context) {
+			return {
+				CatchClause(node) {
+					if (
+						node.body.body.length === 1 &&
+						node.body.body[0].type === 'ThrowStatement' &&
+						node.body.body[0].argument.type === 'Identifier' &&
+						node.body.body[0].argument.name === node.param.name
+					) {
+						context.report({
+							node,
+							messageId: 'noUselessCatchThrow',
+							fix(fixer) {
+								const tryStatement = node.parent;
+								const tryBlock = tryStatement.block;
+								const sourceCode = context.getSourceCode();
+								const tryBlockText = sourceCode.getText(tryBlock);
+								const tryBlockTextWithoutBraces = tryBlockText.slice(1, -1).trim();
+								const indentedTryBlockText = tryBlockTextWithoutBraces
+									.split('\n')
+									.map((line) => line.replace(/\t/, ''))
+									.join('\n');
+								return fixer.replaceText(tryStatement, indentedTryBlockText);
+							},
+						});
+					}
+				},
+			};
+		},
+	},
+
 	'no-skipped-tests': {
 		meta: {
 			type: 'problem',

packages/@n8n_io/eslint-config/local-rules.test.js

@@ -51,3 +51,33 @@ ruleTester.run('no-json-parse-json-stringify', rules['no-json-parse-json-stringi
 		},
 	],
 });
+
+ruleTester.run('no-useless-catch-throw', rules['no-useless-catch-throw'], {
+	valid: [
+		{
+			code: 'try { foo(); } catch (e) { console.error(e); }',
+		},
+		{
+			code: 'try { foo(); } catch (e) { throw new Error("Custom error"); }',
+		},
+	],
+	invalid: [
+		{
+			code: `
+try {
+	// Some comment
+	if (foo) {
+		bar();
+	}
+} catch (e) {
+	throw e;
+}`,
+			errors: [{ messageId: 'noUselessCatchThrow' }],
+			output: `
+// Some comment
+if (foo) {
+	bar();
+}`,
+		},
+	],
+});

packages/editor-ui/src/components/ButtonParameter/utils.ts

@@ -161,7 +161,6 @@ export function reducePayloadSizeOrThrow(
 	error: Error,
 	averageTokenLength = 4,
 ) {
-	try {
 	let remainingTokensToReduce = calculateRemainingTokens(error);
 
 	const [remaining, parentNodesTokenCount] = trimParentNodesSchema(
@@ -180,9 +179,6 @@ export function reducePayloadSizeOrThrow(
 	);
 
 	if (remainingTokensToReduce > 0) throw error;
-	} catch (e) {
-		throw e;
-	}
 }
 
 export async function generateCodeForAiTransform(prompt: string, path: string, retries = 1) {

packages/editor-ui/src/components/CanvasChat/CanvasChat.vue

@@ -200,7 +200,6 @@ async function createExecutionPromise() {
 }
 
 async function onRunChatWorkflow(payload: RunWorkflowChatPayload) {
-	try {
 	const runWorkflowOptions: Parameters<typeof runWorkflow>[0] = {
 		triggerNode: payload.triggerNode,
 		nodeData: payload.nodeData,
@@ -220,9 +219,6 @@ async function onRunChatWorkflow(payload: RunWorkflowChatPayload) {
 		return response;
 	}
 	return;
-	} catch (error) {
-		throw error;
-	}
 }
 
 // Initialize chat config

packages/editor-ui/src/stores/communityNodes.store.ts

@@ -55,21 +55,13 @@ export const useCommunityNodesStore = defineStore(STORES.COMMUNITY_NODES, () =>
 	};
 
 	const installPackage = async (packageName: string): Promise<void> => {
-		try {
 		await communityNodesApi.installNewPackage(rootStore.restApiContext, packageName);
 		await fetchInstalledPackages();
-		} catch (error) {
-			throw error;
-		}
 	};
 
 	const uninstallPackage = async (packageName: string): Promise<void> => {
-		try {
 		await communityNodesApi.uninstallPackage(rootStore.restApiContext, packageName);
 		removePackageByName(packageName);
-		} catch (error) {
-			throw error;
-		}
 	};
 
 	const removePackageByName = (name: string): void => {
@@ -82,16 +74,12 @@ export const useCommunityNodesStore = defineStore(STORES.COMMUNITY_NODES, () =>
 	};
 
 	const updatePackage = async (packageName: string): Promise<void> => {
-		try {
 		const packageToUpdate = installedPackages.value[packageName];
 		const updatedPackage = await communityNodesApi.updatePackage(
 			rootStore.restApiContext,
 			packageToUpdate.packageName,
 		);
 		updatePackageObject(updatedPackage);
-		} catch (error) {
-			throw error;
-		}
 	};
 
 	return {

packages/editor-ui/src/stores/executions.store.ts

@@ -173,8 +173,6 @@ export const useExecutionsStore = defineStore('executions', () => {
 			executionsCount.value = data.count;
 			executionsCountEstimated.value = data.estimated;
 			return data;
-		} catch (e) {
-			throw e;
 		} finally {
 			loading.value = false;
 		}

packages/nodes-base/nodes/Disqus/Disqus.node.ts

@@ -601,16 +601,12 @@ export class Disqus implements INodeType {
 
 						Object.assign(qs, additionalFields);
 
-						try {
 						const responseData = await disqusApiRequest.call(this, requestMethod, qs, endpoint);
 						const executionData = this.helpers.constructExecutionMetaData(
 							this.helpers.returnJsonArray(responseData.response as IDataObject[]),
 							{ itemData: { item: i } },
 						);
 						returnData.push(...executionData);
-						} catch (error) {
-							throw error;
-						}
 					} else if (operation === 'getPosts') {
 						// ----------------------------------
 						//         getPosts
@@ -629,7 +625,6 @@ export class Disqus implements INodeType {
 						qs.forum = id;
 						qs.limit = 100;
 
-						try {
 						let responseData: IDataObject = {};
 						if (returnAll) {
 							responseData.response = await disqusApiRequestAllItems.call(
@@ -648,9 +643,6 @@ export class Disqus implements INodeType {
 							{ itemData: { item: i } },
 						);
 						returnData.push(...executionData);
-						} catch (error) {
-							throw error;
-						}
 					} else if (operation === 'getCategories') {
 						// ----------------------------------
 						//         getCategories
@@ -668,7 +660,6 @@ export class Disqus implements INodeType {
 						qs.forum = id;
 						qs.limit = 100;
 
-						try {
 						let responseData: IDataObject = {};
 
 						if (returnAll) {
@@ -693,9 +684,6 @@ export class Disqus implements INodeType {
 							{ itemData: { item: i } },
 						);
 						returnData.push(...executionData);
-						} catch (error) {
-							throw error;
-						}
 					} else if (operation === 'getThreads') {
 						// ----------------------------------
 						//         getThreads
@@ -715,7 +703,6 @@ export class Disqus implements INodeType {
 
 						Object.assign(qs, additionalFields);
 
-						try {
 						let responseData: IDataObject = {};
 						if (returnAll) {
 							responseData.response = await disqusApiRequestAllItems.call(
@@ -734,9 +721,6 @@ export class Disqus implements INodeType {
 							{ itemData: { item: i } },
 						);
 						returnData.push(...executionData);
-						} catch (error) {
-							throw error;
-						}
 					} else {
 						throw new NodeOperationError(
 							this.getNode(),

packages/nodes-base/nodes/Disqus/GenericFunctions.ts

@@ -66,14 +66,10 @@ export async function disqusApiRequestAllItems(
 
 	let responseData;
 
-	try {
 	do {
 		responseData = await disqusApiRequest.call(this, method, qs, uri, body, option);
 		qs.cursor = responseData.cursor.id;
 		returnData.push.apply(returnData, responseData.response as IDataObject[]);
 	} while (responseData.cursor.more === true && responseData.cursor.hasNext === true);
 	return returnData;
-	} catch (error) {
-		throw error;
-	}
 }

packages/nodes-base/nodes/FileMaker/GenericFunctions.ts

@@ -154,13 +154,8 @@ export async function getFields(this: ILoadOptionsFunctions): Promise<any> {
 		json: true,
 	};
 
-	try {
 	const responseData = await this.helpers.request(options);
 	return responseData.response.fieldMetaData;
-	} catch (error) {
-		// If that data does not exist for some reason return the actual error
-		throw error;
-	}
 }
 
 /**
@@ -185,13 +180,8 @@ export async function getPortals(this: ILoadOptionsFunctions): Promise<any> {
 		json: true,
 	};
 
-	try {
 	const responseData = await this.helpers.request(options);
 	return responseData.response.portalMetaData;
-	} catch (error) {
-		// If that data does not exist for some reason return the actual error
-		throw error;
-	}
 }
 
 function parseScriptsList(scripts: ScriptObject[]): INodePropertyOptions[] {
@@ -230,15 +220,10 @@ export async function getScripts(this: ILoadOptionsFunctions): Promise<any> {
 		json: true,
 	};
 
-	try {
 	const responseData = await this.helpers.request(options);
 	const items = parseScriptsList(responseData.response.scripts as ScriptObject[]);
 	items.sort((a, b) => (a.name > b.name ? 0 : 1));
 	return items;
-	} catch (error) {
-		// If that data does not exist for some reason return the actual error
-		throw error;
-	}
 }
 
 export async function logout(

packages/nodes-base/nodes/Flow/FlowTrigger.node.ts

@@ -107,12 +107,8 @@ export class FlowTrigger implements INodeType {
 				}
 				qs.organization_id = credentials.organizationId as number;
 				const endpoint = '/integration_webhooks';
-				try {
 				webhooks = await flowApiRequest.call(this, 'GET', endpoint, {}, qs);
 				webhooks = webhooks.integration_webhooks;
-				} catch (error) {
-					throw error;
-				}
 				for (const webhook of webhooks) {
 					// @ts-ignore
 					if (webhookData.webhookIds.includes(webhook.id)) {

packages/nodes-base/nodes/ItemLists/V3/actions/router.ts

@@ -16,7 +16,6 @@ export async function router(this: IExecuteFunctions): Promise<INodeExecutionDat
 		operation,
 	} as ItemListsType;
 
-	try {
 	switch (itemListsNodeData.resource) {
 		case 'itemList':
 			returnData = await itemList[itemListsNodeData.operation].execute.call(this, items);
@@ -27,9 +26,6 @@ export async function router(this: IExecuteFunctions): Promise<INodeExecutionDat
 				`The operation "${operation}" is not supported!`,
 			);
 	}
-	} catch (error) {
-		throw error;
-	}
 
 	return [returnData];
 }

packages/nodes-base/nodes/Mailchimp/MailchimpTrigger.node.ts

@@ -214,7 +214,6 @@ export class MailchimpTrigger implements INodeType {
 			},
 
 			async create(this: IHookFunctions): Promise<boolean> {
-				let webhook;
 				const webhookUrl = this.getNodeWebhookUrl('default');
 				const listId = this.getNodeParameter('list') as string;
 				const events = this.getNodeParameter('events', []) as string[];
@@ -233,11 +232,7 @@ export class MailchimpTrigger implements INodeType {
 					}, {}),
 				};
 				const endpoint = `/lists/${listId}/webhooks`;
-				try {
+				const webhook = await mailchimpApiRequest.call(this, endpoint, 'POST', body);
-					webhook = await mailchimpApiRequest.call(this, endpoint, 'POST', body);
-				} catch (error) {
-					throw error;
-				}
 				if (webhook.id === undefined) {
 					return false;
 				}

packages/nodes-base/nodes/PayPal/PayPalTrigger.node.ts

@@ -115,7 +115,6 @@ export class PayPalTrigger implements INodeType {
 			},
 
 			async create(this: IHookFunctions): Promise<boolean> {
-				let webhook;
 				const webhookUrl = this.getNodeWebhookUrl('default');
 				const events = this.getNodeParameter('events', []) as string[];
 				const body = {
@@ -125,11 +124,7 @@ export class PayPalTrigger implements INodeType {
 					}),
 				};
 				const endpoint = '/notifications/webhooks';
-				try {
+				const webhook = await payPalApiRequest.call(this, endpoint, 'POST', body);
-					webhook = await payPalApiRequest.call(this, endpoint, 'POST', body);
-				} catch (error) {
-					throw error;
-				}
 
 				if (webhook.id === undefined) {
 					return false;
@@ -156,7 +151,6 @@ export class PayPalTrigger implements INodeType {
 	};
 
 	async webhook(this: IWebhookFunctions): Promise<IWebhookResponseData> {
-		let webhook;
 		const webhookData = this.getWorkflowStaticData('node');
 		const bodyData = this.getBodyData();
 		const req = this.getRequestObject();
@@ -188,11 +182,7 @@ export class PayPalTrigger implements INodeType {
 				webhook_id: webhookData.webhookId,
 				webhook_event: bodyData,
 			};
-			try {
+			const webhook = await payPalApiRequest.call(this, endpoint, 'POST', body);
-				webhook = await payPalApiRequest.call(this, endpoint, 'POST', body);
-			} catch (error) {
-				throw error;
-			}
 			if (webhook.verification_status !== 'SUCCESS') {
 				return {};
 			}

packages/nodes-base/nodes/Stripe/StripeTrigger.node.ts

@@ -877,12 +877,7 @@ export class StripeTrigger implements INodeType {
 					enabled_events: events,
 				};
 
-				let responseData;
+				const responseData = await stripeApiRequest.call(this, 'POST', endpoint, body);
-				try {
-					responseData = await stripeApiRequest.call(this, 'POST', endpoint, body);
-				} catch (error) {
-					throw error;
-				}
 
 				if (
 					responseData.id === undefined ||

packages/nodes-base/nodes/Webflow/V2/actions/router.ts

@@ -16,7 +16,6 @@ export async function router(this: IExecuteFunctions): Promise<INodeExecutionDat
 		operation,
 	} as WebflowType;
 
-	try {
 	switch (webflowNodeData.resource) {
 		case 'item':
 			returnData = await item[webflowNodeData.operation].execute.call(this, items);
@@ -27,9 +26,6 @@ export async function router(this: IExecuteFunctions): Promise<INodeExecutionDat
 				`The operation "${operation}" is not supported!`,
 			);
 	}
-	} catch (error) {
-		throw error;
-	}
 
 	return [returnData];
 }