fix(core): Improve handling of wrapped errors (#8510)

packages/workflow/src/errors/abstract/node.error.ts

@@ -39,12 +39,15 @@ export abstract class NodeError extends ExecutionBaseError {
 		readonly node: INode,
 		error: Error | JsonObject,
 	) {
-		if (error instanceof NodeError) return error;
-
 		const isError = error instanceof Error;
 		const message = isError ? error.message : '';
 		const options = isError ? { cause: error } : { errorResponse: error };
 		super(message, options);
+
+		if (error instanceof NodeError) {
+			this.level = 'error';
+			this.message = `[RE-WRAPPED]: ${message}`;
+		}
 	}
 
 	/**

packages/workflow/test/errors/node.error.test.ts

@@ -6,11 +6,17 @@ import { NodeOperationError } from '@/errors/node-operation.error';
 describe('NodeError', () => {
 	const node = mock<INode>();
 
-	it('should prevent errors from being re-wrapped', () => {
+	it('should update re-wrapped error level and message', () => {
 		const apiError = new NodeApiError(node, mock({ message: 'Some error happened', code: 500 }));
-		const opsError = new NodeOperationError(node, mock());
+		const opsError = new NodeOperationError(node, mock(), { message: 'Some operation failed' });
+		const wrapped1 = new NodeOperationError(node, apiError);
+		const wrapped2 = new NodeOperationError(node, opsError);
 
-		expect(new NodeOperationError(node, apiError)).toEqual(apiError);
-		expect(new NodeOperationError(node, opsError)).toEqual(opsError);
+		expect(wrapped1.level).toEqual('error');
+		expect(wrapped1.message).toEqual(
+			'[RE-WRAPPED]: The service was not able to process your request',
+		);
+		expect(wrapped2.level).toEqual('error');
+		expect(wrapped2.message).toEqual('[RE-WRAPPED]: Some operation failed');
 	});
 });