feat: add itemIndex and description to thrown errors to provide context

Author: pja-zerobounce

nodes/ZeroBounce/ZeroBounce.node.ts

@@ -121,7 +121,7 @@ export class ZeroBounce implements INodeType {
 			try {
 				const resource = this.getNodeParameter('resource', itemIndex);
 				const operation = this.getNodeParameter('operation', itemIndex);
-				const operationHandler = getHandler(this, handlers, resource);
+				const operationHandler = getHandler(this, itemIndex, handlers, resource);
 
 				const responseData = await operationHandler.handle(this, operation, itemIndex);
 

nodes/ZeroBounce/fields/activity-data.field.ts

@@ -7,7 +7,6 @@ export const ActivityData: INodeProperties = {
 	name: Fields.ActivityData,
 	type: 'boolean',
 	default: false,
-	placeholder: 'false',
 	noDataExpression: true,
 	description: 'Whether Activity Data information will be appended to the validation result',
 };

nodes/ZeroBounce/handlers/account.handler.ts

@@ -1,12 +1,12 @@
 import { IRequestParams, zbGetRequest, zbPostRequest } from '../utils/request.utils';
 import {
+	getDateParameter,
 	getNumberParameter,
 	IAltErrorResponse,
 	IErrorResponse,
 	IOperationHandler,
 	isAltErrorResponse,
 	isErrorResponse,
-	toDate,
 } from '../utils/handler.utils';
 import { IDataObject, IExecuteFunctions, INodeExecutionData, NodeOperationError } from 'n8n-workflow';
 import { BaseUrl, Endpoint, Operations } from '../enums';
@@ -81,7 +81,7 @@ interface IDeleteFilterResponse extends IDataObject {
  */
 async function getCredits(context: IExecuteFunctions, i: number): Promise<INodeExecutionData[]> {
 	const baseUrl = context.getNodeParameter(ApiEndpoint.name, i) as BaseUrl;
-	const creditsRequired = getNumberParameter(context, i, CreditsRequired.name, 0);
+	const creditsRequired = getNumberParameter(context, i, CreditsRequired, 0);
 
 	const fullResponse = await zbGetRequest(context, baseUrl, Endpoint.GetCredits);
 	const response = fullResponse.body as IGetCreditsResponse | IErrorResponse;
@@ -127,13 +127,11 @@ async function getCredits(context: IExecuteFunctions, i: number): Promise<INodeE
  * Dates are normalized to ISO yyyy-mm-dd.
  */
 async function getApiUsage(context: IExecuteFunctions, i: number): Promise<INodeExecutionData[]> {
-	const startDate = context.getNodeParameter(StartDate.name, i) as string;
-	const endDate = context.getNodeParameter(EndDate.name, i) as string;
 	const baseUrl = context.getNodeParameter(ApiEndpoint.name, i) as BaseUrl;
 
 	const request = {
-		start_date: toDate(context, startDate),
-		end_date: toDate(context, endDate),
+		start_date: getDateParameter(context, i, StartDate),
+		end_date: getDateParameter(context, i, EndDate),
 	} as IGetApiUsageRequest;
 
 	const fullResponse = await zbGetRequest(context, baseUrl, Endpoint.GetApiUsage, request);
@@ -225,7 +223,10 @@ class AccountHandler implements IOperationHandler {
 			case Operations.AccountDeleteFilter:
 				return addOrDeleteFilter(context, i, AddOrDelete.DELETE);
 			default:
-				throw new NodeOperationError(context.getNode(), `Operation ${operation} not supported`);
+				throw new NodeOperationError(context.getNode(), `Operation ${operation} not supported`, {
+					itemIndex: i,
+					description: 'Please select an operation from the list',
+				});
 		}
 	}
 }

nodes/ZeroBounce/handlers/domain-search.handler.ts

@@ -18,7 +18,10 @@ export class DomainSearchHandler implements IOperationHandler {
 			case Operations.BulkDomainSearchDeleteFile:
 				return deleteFile(context, i, Mode.DOMAIN_SEARCH);
 			default:
-				throw new NodeOperationError(context.getNode(), `Operation ${operation} not supported`);
+				throw new NodeOperationError(context.getNode(), `Operation ${operation} not supported`, {
+					itemIndex: i,
+					description: 'Please select an operation from the list',
+				});
 		}
 	}
 }

nodes/ZeroBounce/handlers/email-finder.handler.ts

@@ -18,7 +18,10 @@ export class EmailFinderHandler implements IOperationHandler {
 			case Operations.BulkEmailFinderDeleteFile:
 				return deleteFile(context, i, Mode.EMAIL_FINDER);
 			default:
-				throw new NodeOperationError(context.getNode(), `Operation ${operation} not supported`);
+				throw new NodeOperationError(context.getNode(), `Operation ${operation} not supported`, {
+					itemIndex: i,
+					description: 'Please select an operation from the list',
+				});
 		}
 	}
 }

nodes/ZeroBounce/handlers/scoring.handler.ts

@@ -52,7 +52,10 @@ export class ScoringHandler implements IOperationHandler {
 			case Operations.BulkScoringDeleteFile:
 				return deleteFile(context, i, Mode.SCORING);
 			default:
-				throw new NodeOperationError(context.getNode(), `Operation ${operation} not supported`);
+				throw new NodeOperationError(context.getNode(), `Operation ${operation} not supported`, {
+					itemIndex: i,
+					description: 'Please select an operation from the list',
+				});
 		}
 	}
 }

nodes/ZeroBounce/handlers/validation.handler.ts

@@ -106,7 +106,7 @@ interface IValidateBatchResult extends IDataObject {
  * - Handles all request construction and response parsing internally.
  */
 async function validate(context: IExecuteFunctions, i: number): Promise<INodeExecutionData[]> {
-	const timeout = getNumberParameter(context, i, Timeout.name);
+	const timeout = getNumberParameter(context, i, Timeout);
 	const activityData = context.getNodeParameter(ActivityData.name, i, false, { ensureType: 'boolean' }) as boolean;
 	const verifyPlus = context.getNodeParameter(VerifyPlus.name, i, false, { ensureType: 'boolean' }) as boolean;
 	const email = context.getNodeParameter(Email.name, i) as string;
@@ -174,7 +174,7 @@ async function validate(context: IExecuteFunctions, i: number): Promise<INodeExe
  * - The function wraps the ZeroBounce response in an n8n `INodeExecutionData` structure for workflow compatibility.
  */
 async function batchValidate(context: IExecuteFunctions, i: number): Promise<INodeExecutionData[]> {
-	const timeout = getNumberParameter(context, i, Timeout.name);
+	const timeout = getNumberParameter(context, i, Timeout);
 	const activityData = context.getNodeParameter(ActivityData.name, i, false, { ensureType: 'boolean' }) as boolean;
 	const verifyPlus = context.getNodeParameter(VerifyPlus.name, i, false, { ensureType: 'boolean' }) as boolean;
 	const emailBatch = convertItemInput(context, i, Mode.VALIDATION) as IEmailEntry[];
@@ -225,7 +225,10 @@ export class ValidationHandler implements IOperationHandler {
 			case Operations.BulkValidationDeleteFile:
 				return deleteFile(context, i, Mode.VALIDATION);
 			default:
-				throw new NodeOperationError(context.getNode(), `Operation ${operation} not supported`);
+				throw new NodeOperationError(context.getNode(), `Operation ${operation} not supported`, {
+					itemIndex: i,
+					description: 'Please select an operation from the list',
+				});
 		}
 	}
 }

nodes/ZeroBounce/utils/finder.utils.ts

@@ -89,7 +89,14 @@ export async function find(context: IExecuteFunctions, i: number, mode: Mode): P
 		const lastName = context.getNodeParameter(LastName.name, i) as string | undefined;
 
 		if (isBlank(firstName) && isBlank(lastName)) {
-			throw new NodeOperationError(context.getNode(), 'Email Finder at least first name or last name to be set');
+			throw new NodeOperationError(
+				context.getNode(),
+				`Email Finder requires '${FirstName.displayName}' or '${LastName.displayName}'`,
+				{
+					itemIndex: i,
+					description: `Enter a value for '${FirstName.displayName}', '${LastName.displayName}' or both`,
+				},
+			);
 		}
 
 		request = {