n8n-nodes-prestashop8 2.5.0 → 2.6.1

package/dist/nodes/PrestaShop8/PrestaShop8.node.description.js

@@ -670,6 +670,38 @@ exports.PrestaShop8Description = {
                         },
                     ],
                 },
+                {
+                    displayName: 'Retry On Error',
+                    name: 'retry',
+                    type: 'collection',
+                    placeholder: 'Add Retry Option',
+                    default: {},
+                    options: [
+                        {
+                            displayName: 'Enabled',
+                            name: 'retryEnabled',
+                            type: 'boolean',
+                            default: false,
+                            description: 'Whether to retry a call that fails on a transient error. Retries on: network timeout, connection drop (ECONNRESET / ECONNREFUSED / socket hang up), 5xx server errors and 429 rate-limit. Never retries client errors (4xx — invalid API key, 404, invalid XML). The retry budget is reset for each failing call.',
+                        },
+                        {
+                            displayName: 'Max Retries',
+                            name: 'maxRetries',
+                            type: 'number',
+                            typeOptions: { minValue: 0 },
+                            default: 3,
+                            description: 'Maximum number of retry attempts per failing call. This budget is reset for each call, it is not a global limit for the whole node execution.',
+                        },
+                        {
+                            displayName: 'Retry Delay (ms)',
+                            name: 'retryDelay',
+                            type: 'number',
+                            typeOptions: { minValue: 0 },
+                            default: 1000,
+                            description: 'Pause in milliseconds between two successive retry attempts of the same call',
+                        },
+                    ],
+                },
                 {
                     displayName: 'Timeout (ms)',
                     name: 'timeout',

package/dist/nodes/PrestaShop8/PrestaShop8.node.js

@@ -100,6 +100,12 @@ class PrestaShop8 {
                 let requestDebugInfo = {};
                 const opts = (0, http_1.getOperationOptions)(this, i);
                 const { rawMode, timeout, neverError, includeResponseHeaders, showRequestInfo, showRequestUrl } = opts;
+                // Trace each retry attempt to the n8n server logs (Option A).
+                if (opts.retry.enabled) {
+                    opts.retry.onRetry = (attempt, error) => {
+                        this.logger.warn(`PrestaShop8: retry ${attempt}/${opts.retry.maxRetries} on ${resource}/${operation} after ${(0, http_1.describeError)(error)} — waiting ${opts.retry.retryDelay}ms`);
+                    };
+                }
                 // Throttle: pause before each PrestaShop call except the first.
                 if (i > 0 && opts.delayBetweenCalls > 0) {
                     await (0, http_1.sleep)(opts.delayBetweenCalls);
@@ -124,13 +130,13 @@ class PrestaShop8 {
                         }
                         requestUrl = (0, utils_1.buildUrlWithFilters)(`${credentials.baseUrl}/${resource}`, urlParams, rawMode);
                         if (rawMode) {
-                            const rawResult = await (0, http_1.executeRawModeRequest)(requestUrl, credentials, timeout, neverError, includeResponseHeaders, operation, resource);
+                            const rawResult = await (0, http_1.executeRawModeRequest)(requestUrl, credentials, timeout, neverError, includeResponseHeaders, operation, resource, opts.retry);
                             responseData = rawResult.responseData;
                             requestDebugInfo = rawResult.requestDebugInfo;
                         }
                         else {
                             const options = (0, http_1.buildHttpOptions)('GET', requestUrl, credentials, rawMode, timeout);
-                            const { response, debugInfo, url, responseHeaders, statusCode } = await (0, http_1.executeHttpRequest)(this.helpers, options, credentials, rawMode, operation, resource, neverError);
+                            const { response, debugInfo, url, responseHeaders, statusCode } = await (0, http_1.executeHttpRequest)(this.helpers, options, credentials, rawMode, operation, resource, neverError, undefined, opts.retry);
                             requestUrl = url;
                             requestDebugInfo = debugInfo;
                             const processedResponse = (0, utils_1.processResponseForMode)(response, resource);
@@ -153,7 +159,7 @@ class PrestaShop8 {
                         }
                         requestUrl = (0, utils_1.buildUrlWithFilters)(`${credentials.baseUrl}/${resource}/${id}`, urlParams, rawMode);
                         const options = (0, http_1.buildHttpOptions)('GET', requestUrl, credentials, rawMode, timeout);
-                        const { response, debugInfo, url, responseHeaders, statusCode } = await (0, http_1.executeHttpRequest)(this.helpers, options, credentials, rawMode, operation, resource, neverError);
+                        const { response, debugInfo, url, responseHeaders, statusCode } = await (0, http_1.executeHttpRequest)(this.helpers, options, credentials, rawMode, operation, resource, neverError, undefined, opts.retry);
                         requestUrl = url;
                         requestDebugInfo = debugInfo;
                         const processedResponse = rawMode ? { raw: response } : (0, utils_1.processResponseForMode)(response, resource);
@@ -190,7 +196,7 @@ class PrestaShop8 {
                         // Build XML using new format
                         body = (0, utils_1.buildCreateXml)(resource, fieldsToCreate);
                         const options = (0, http_1.buildHttpOptions)('POST', `${credentials.baseUrl}/${resource}`, credentials, rawMode, timeout, body);
-                        const { response, debugInfo, url, responseHeaders, statusCode } = await (0, http_1.executeHttpRequest)(this.helpers, options, credentials, rawMode, operation, resource, neverError, body);
+                        const { response, debugInfo, url, responseHeaders, statusCode } = await (0, http_1.executeHttpRequest)(this.helpers, options, credentials, rawMode, operation, resource, neverError, body, opts.retry);
                         requestUrl = url;
                         requestDebugInfo = debugInfo;
                         const processedResponse = rawMode ? { raw: response } : (0, utils_1.processResponseForMode)(response, resource);
@@ -223,7 +229,7 @@ class PrestaShop8 {
                         // Build XML using new format
                         body = (0, utils_1.buildUpdateXml)(resource, id, fieldsToUpdate);
                         const options = (0, http_1.buildHttpOptions)('PATCH', `${credentials.baseUrl}/${resource}/${id}`, credentials, rawMode, timeout, body);
-                        const { response, debugInfo, url, responseHeaders, statusCode } = await (0, http_1.executeHttpRequest)(this.helpers, options, credentials, rawMode, operation, resource, neverError, body);
+                        const { response, debugInfo, url, responseHeaders, statusCode } = await (0, http_1.executeHttpRequest)(this.helpers, options, credentials, rawMode, operation, resource, neverError, body, opts.retry);
                         requestUrl = url;
                         requestDebugInfo = debugInfo;
                         const processedResponse = rawMode ? { raw: response } : (0, utils_1.processResponseForMode)(response, resource);
@@ -295,13 +301,13 @@ class PrestaShop8 {
                         }
                         requestUrl = (0, utils_1.buildUrlWithFilters)(`${credentials.baseUrl}/${resource}`, urlParams, rawMode);
                         if (rawMode) {
-                            const rawResult = await (0, http_1.executeRawModeRequest)(requestUrl, credentials, timeout, neverError, includeResponseHeaders, operation, resource);
+                            const rawResult = await (0, http_1.executeRawModeRequest)(requestUrl, credentials, timeout, neverError, includeResponseHeaders, operation, resource, opts.retry);
                             responseData = rawResult.responseData;
                             requestDebugInfo = rawResult.requestDebugInfo;
                         }
                         else {
                             const options = (0, http_1.buildHttpOptions)('GET', requestUrl, credentials, rawMode, timeout);
-                            const { response, debugInfo, url, responseHeaders, statusCode } = await (0, http_1.executeHttpRequest)(this.helpers, options, credentials, rawMode, operation, resource, neverError);
+                            const { response, debugInfo, url, responseHeaders, statusCode } = await (0, http_1.executeHttpRequest)(this.helpers, options, credentials, rawMode, operation, resource, neverError, undefined, opts.retry);
                             requestUrl = url;
                             requestDebugInfo = debugInfo;
                             const processedResponse = (0, utils_1.processResponseForMode)(response, resource);
@@ -315,7 +321,7 @@ class PrestaShop8 {
                             throw new n8n_workflow_1.NodeOperationError(this.getNode(), 'ID required for this operation');
                         }
                         const options = (0, http_1.buildHttpOptions)('DELETE', `${credentials.baseUrl}/${resource}/${id}`, credentials, rawMode, timeout);
-                        const { debugInfo, url, responseHeaders, statusCode } = await (0, http_1.executeHttpRequest)(this.helpers, options, credentials, rawMode, operation, resource, neverError);
+                        const { debugInfo, url, responseHeaders, statusCode } = await (0, http_1.executeHttpRequest)(this.helpers, options, credentials, rawMode, operation, resource, neverError, undefined, opts.retry);
                         requestUrl = url;
                         requestDebugInfo = debugInfo;
                         const deleteResponse = {

package/dist/nodes/PrestaShop8/helpers/http.d.ts

@@ -19,11 +19,40 @@ export interface OperationOptions {
     showRequestInfo: boolean;
     showRequestUrl: boolean;
     delayBetweenCalls: number;
+    retry: RetryOptions;
+}
+/**
+ * Retry configuration applied to each individual HTTP call.
+ * The retry budget is per call: every failing call gets a fresh set of attempts.
+ */
+export interface RetryOptions {
+    enabled: boolean;
+    maxRetries: number;
+    retryDelay: number;
+    /** Optional hook called before each retry wait, with the upcoming retry number (1-based) and the error. */
+    onRetry?: (attempt: number, error: any) => void;
 }
 /**
  * Pause execution for the given number of milliseconds (used to throttle calls).
  */
 export declare function sleep(ms: number): Promise<void>;
+/**
+ * Determine whether an error is transient and worth retrying.
+ * Retries on network/timeout errors, 5xx server errors and 429 rate-limit.
+ * Never retries on 4xx (invalid key, not found, invalid XML, etc.).
+ */
+export declare function isRetryableError(error: any): boolean;
+/**
+ * Build a short human-readable reason from an error, for retry logging.
+ * Prefers the network code, then the HTTP status, then the message.
+ */
+export declare function describeError(error: any): string;
+/**
+ * Run an async HTTP operation with per-call retry on transient errors.
+ * Each call gets its own retry budget; on the last attempt the error is rethrown
+ * so existing neverError / continueOnFail handling applies unchanged.
+ */
+export declare function withRetry<T>(retry: RetryOptions, fn: () => Promise<T>): Promise<T>;
 /**
  * Extract common operation options from node parameters
  */
@@ -43,7 +72,7 @@ export declare function wrapResponse(processedResponse: any, includeHeaders: boo
 /**
  * Execute HTTP request with debug capture and response metadata
  */
-export declare function executeHttpRequest(helpers: any, options: IHttpRequestOptions, credentials: any, rawMode: boolean, operation: string, resource: string, neverError?: boolean, body?: string): Promise<{
+export declare function executeHttpRequest(helpers: any, options: IHttpRequestOptions, credentials: any, rawMode: boolean, operation: string, resource: string, neverError?: boolean, body?: string, retry?: RetryOptions): Promise<{
     response: any;
     debugInfo: any;
     url: string;
@@ -53,7 +82,7 @@ export declare function executeHttpRequest(helpers: any, options: IHttpRequestOp
 /**
  * Execute a raw mode request using axios directly (bypasses n8n parsing)
  */
-export declare function executeRawModeRequest(requestUrl: string, credentials: IPrestaShopCredentials, timeout: number, neverError: boolean, includeResponseHeaders: boolean, operation: string, resource: string): Promise<{
+export declare function executeRawModeRequest(requestUrl: string, credentials: IPrestaShopCredentials, timeout: number, neverError: boolean, includeResponseHeaders: boolean, operation: string, resource: string, retry?: RetryOptions): Promise<{
     responseData: any;
     requestDebugInfo: any;
     requestHeaders: any;

package/dist/nodes/PrestaShop8/helpers/http.js

@@ -2,6 +2,9 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.FILTER_OPERATOR_FORMATS = void 0;
 exports.sleep = sleep;
+exports.isRetryableError = isRetryableError;
+exports.describeError = describeError;
+exports.withRetry = withRetry;
 exports.getOperationOptions = getOperationOptions;
 exports.buildHttpOptions = buildHttpOptions;
 exports.captureRequestDebugInfo = captureRequestDebugInfo;
@@ -34,6 +37,80 @@ exports.FILTER_OPERATOR_FORMATS = {
 function sleep(ms) {
     return new Promise((resolve) => setTimeout(resolve, ms));
 }
+/**
+ * Determine whether an error is transient and worth retrying.
+ * Retries on network/timeout errors, 5xx server errors and 429 rate-limit.
+ * Never retries on 4xx (invalid key, not found, invalid XML, etc.).
+ */
+function isRetryableError(error) {
+    var _a, _b;
+    // Network / timeout errors expose a code on the error or its cause
+    const code = (error === null || error === void 0 ? void 0 : error.code) || ((_a = error === null || error === void 0 ? void 0 : error.cause) === null || _a === void 0 ? void 0 : _a.code);
+    const retryableCodes = [
+        'ETIMEDOUT',
+        'ECONNRESET',
+        'ECONNREFUSED',
+        'ECONNABORTED',
+        'ENOTFOUND',
+        'EAI_AGAIN',
+        'EPIPE',
+    ];
+    if (code && retryableCodes.includes(code)) {
+        return true;
+    }
+    // "socket hang up" and timeout messages have no stable code
+    const message = String((error === null || error === void 0 ? void 0 : error.message) || '').toLowerCase();
+    if (message.includes('socket hang up') || message.includes('timeout')) {
+        return true;
+    }
+    // HTTP status: retry on 429 (rate-limit) and 5xx (server errors) only
+    const status = (error === null || error === void 0 ? void 0 : error.httpCode) || ((_b = error === null || error === void 0 ? void 0 : error.response) === null || _b === void 0 ? void 0 : _b.status);
+    if (status === 429 || (status >= 500 && status <= 599)) {
+        return true;
+    }
+    return false;
+}
+/**
+ * Build a short human-readable reason from an error, for retry logging.
+ * Prefers the network code, then the HTTP status, then the message.
+ */
+function describeError(error) {
+    var _a, _b;
+    const code = (error === null || error === void 0 ? void 0 : error.code) || ((_a = error === null || error === void 0 ? void 0 : error.cause) === null || _a === void 0 ? void 0 : _a.code);
+    if (code) {
+        return code;
+    }
+    const status = (error === null || error === void 0 ? void 0 : error.httpCode) || ((_b = error === null || error === void 0 ? void 0 : error.response) === null || _b === void 0 ? void 0 : _b.status);
+    if (status) {
+        return `HTTP ${status}`;
+    }
+    const message = error === null || error === void 0 ? void 0 : error.message;
+    return message ? String(message) : 'unknown error';
+}
+/**
+ * Run an async HTTP operation with per-call retry on transient errors.
+ * Each call gets its own retry budget; on the last attempt the error is rethrown
+ * so existing neverError / continueOnFail handling applies unchanged.
+ */
+async function withRetry(retry, fn) {
+    var _a;
+    const maxRetries = retry.enabled ? Math.max(0, retry.maxRetries) : 0;
+    for (let attempt = 0;; attempt++) {
+        try {
+            return await fn();
+        }
+        catch (error) {
+            if (attempt >= maxRetries || !isRetryableError(error)) {
+                throw error;
+            }
+            // attempt is 0-based; report the upcoming retry number (1-based)
+            (_a = retry.onRetry) === null || _a === void 0 ? void 0 : _a.call(retry, attempt + 1, error);
+            if (retry.retryDelay > 0) {
+                await sleep(retry.retryDelay);
+            }
+        }
+    }
+}
 /**
  * Extract common operation options from node parameters
  */
@@ -46,6 +123,11 @@ function getOperationOptions(executeFunctions, itemIndex) {
         showRequestInfo: executeFunctions.getNodeParameter('options.request.showRequestInfo', itemIndex, false),
         showRequestUrl: executeFunctions.getNodeParameter('options.request.showRequestUrl', itemIndex, false),
         delayBetweenCalls: executeFunctions.getNodeParameter('options.delayBetweenCalls', itemIndex, 0),
+        retry: {
+            enabled: executeFunctions.getNodeParameter('options.retry.retryEnabled', itemIndex, false),
+            maxRetries: executeFunctions.getNodeParameter('options.retry.maxRetries', itemIndex, 3),
+            retryDelay: executeFunctions.getNodeParameter('options.retry.retryDelay', itemIndex, 1000),
+        },
     };
 }
 /**
@@ -115,12 +197,12 @@ function wrapResponse(processedResponse, includeHeaders, headers, statusCode) {
 /**
  * Execute HTTP request with debug capture and response metadata
  */
-async function executeHttpRequest(helpers, options, credentials, rawMode, operation, resource, neverError = false, body) {
+async function executeHttpRequest(helpers, options, credentials, rawMode, operation, resource, neverError = false, body, retry = { enabled: false, maxRetries: 0, retryDelay: 0 }) {
     var _a, _b, _c, _d;
     const requestUrl = options.url;
     const debugInfo = captureRequestDebugInfo(options, credentials, rawMode, operation, resource, body);
     try {
-        const response = await helpers.httpRequest(options);
+        const response = await withRetry(retry, () => helpers.httpRequest(options));
         return {
             response,
             debugInfo,
@@ -149,11 +231,11 @@ async function executeHttpRequest(helpers, options, credentials, rawMode, operat
 /**
  * Execute a raw mode request using axios directly (bypasses n8n parsing)
  */
-async function executeRawModeRequest(requestUrl, credentials, timeout, neverError, includeResponseHeaders, operation, resource) {
+async function executeRawModeRequest(requestUrl, credentials, timeout, neverError, includeResponseHeaders, operation, resource, retry = { enabled: false, maxRetries: 0, retryDelay: 0 }) {
     var _a, _b;
     let responseData;
     try {
-        const axiosResponse = await axios({
+        const axiosResponse = await withRetry(retry, () => axios({
             method: 'GET',
             url: requestUrl,
             auth: {
@@ -164,7 +246,7 @@ async function executeRawModeRequest(requestUrl, credentials, timeout, neverErro
             timeout: timeout || 30000,
             transformResponse: [(data) => data],
             validateStatus: neverError ? () => true : undefined,
-        });
+        }));
         const requestDebugInfo = captureRequestDebugInfo({
             method: 'GET',
             url: requestUrl,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "n8n-nodes-prestashop8",
-  "version": "2.5.0",
+  "version": "2.6.1",
   "description": "Nœud n8n personnalisé pour PrestaShop 8 avec support CRUD complet et conversion XML/JSON automatique",
   "keywords": [
     "n8n-community-node-package",