@profplum700/etsy-v3-api-client 2.5.2 → 2.5.4

package/dist/index.cjs

@@ -677,6 +677,11 @@ class FileTokenStorage {
     }
 }
 
+const ETSY_RATE_LIMITS = {
+    MAX_REQUESTS_PER_DAY: 5000,
+    MAX_REQUESTS_PER_SECOND: 5,
+    MIN_REQUEST_INTERVAL: 200,
+};
 class EtsyRateLimiter {
     constructor(config) {
         this.requestCount = 0;
@@ -685,9 +690,9 @@ class EtsyRateLimiter {
         this.isHeaderBasedLimiting = false;
         this.currentRetryCount = 0;
         this.config = {
-            maxRequestsPerDay: 10000,
-            maxRequestsPerSecond: 10,
-            minRequestInterval: 100,
+            maxRequestsPerDay: ETSY_RATE_LIMITS.MAX_REQUESTS_PER_DAY,
+            maxRequestsPerSecond: ETSY_RATE_LIMITS.MAX_REQUESTS_PER_SECOND,
+            minRequestInterval: ETSY_RATE_LIMITS.MIN_REQUEST_INTERVAL,
             maxRetries: 3,
             baseDelayMs: 1000,
             maxDelayMs: 30000,
@@ -742,7 +747,7 @@ class EtsyRateLimiter {
             if (value === null)
                 return undefined;
             const num = parseInt(value, 10);
-            return isNaN(num) ? undefined : num;
+            return Number.isNaN(num) ? undefined : num;
         };
         return {
             limitPerSecond: parseNumber(getHeader('x-limit-per-second')),
@@ -1083,7 +1088,7 @@ class FieldValidator {
             const value = data[this.field];
             if (value === undefined || value === null)
                 return null;
-            if (typeof value !== 'number' || isNaN(value)) {
+            if (typeof value !== 'number' || Number.isNaN(value)) {
                 return {
                     field: this.field,
                     message: options.message || `${this.field} must be a number`,
@@ -1292,9 +1297,14 @@ function combineValidators(...validators) {
 
 class DefaultLogger {
     debug(message, ...args) {
-        const isDevelopment = isNode$1
-            ? process.env.NODE_ENV === 'development'
-            : window.location?.hostname === 'localhost' || window.location?.hostname === '127.0.0.1';
+        let isDevelopment;
+        if (isNode$1) {
+            isDevelopment = process.env.NODE_ENV === 'development';
+        }
+        else {
+            const host = window.location?.hostname;
+            isDevelopment = host === 'localhost' || host === '127.0.0.1';
+        }
         if (isDevelopment) {
             console.log(`[DEBUG] ${message}`, ...args);
         }
@@ -1343,9 +1353,9 @@ class EtsyClient {
         this.sharedSecret = config.sharedSecret;
         if (config.rateLimiting?.enabled !== false) {
             this.rateLimiter = new EtsyRateLimiter({
-                maxRequestsPerDay: config.rateLimiting?.maxRequestsPerDay || 10000,
-                maxRequestsPerSecond: config.rateLimiting?.maxRequestsPerSecond || 10,
-                minRequestInterval: config.rateLimiting?.minRequestInterval ?? 100,
+                maxRequestsPerDay: config.rateLimiting?.maxRequestsPerDay || ETSY_RATE_LIMITS.MAX_REQUESTS_PER_DAY,
+                maxRequestsPerSecond: config.rateLimiting?.maxRequestsPerSecond || ETSY_RATE_LIMITS.MAX_REQUESTS_PER_SECOND,
+                minRequestInterval: config.rateLimiting?.minRequestInterval ?? ETSY_RATE_LIMITS.MIN_REQUEST_INTERVAL,
                 maxRetries: config.rateLimiting?.maxRetries,
                 baseDelayMs: config.rateLimiting?.baseDelayMs,
                 maxDelayMs: config.rateLimiting?.maxDelayMs,
@@ -1376,7 +1386,12 @@ class EtsyClient {
         if (useCache && this.cache && requestOptions.method === 'GET') {
             const cached = await this.cache.get(cacheKey);
             if (cached) {
-                return JSON.parse(cached);
+                try {
+                    return JSON.parse(cached);
+                }
+                catch {
+                    await this.cache.delete(cacheKey);
+                }
             }
         }
         await this.rateLimiter.waitForRateLimit();
@@ -1441,11 +1456,7 @@ class EtsyClient {
             if (value === undefined || value === null)
                 continue;
             if (Array.isArray(value)) {
-                for (const item of value) {
-                    if (item === undefined || item === null)
-                        continue;
-                    body.append(key, String(item));
-                }
+                body.append(key, value.filter(item => item !== undefined && item !== null).map(String).join(','));
             }
             else {
                 body.append(key, String(value));
@@ -2610,9 +2621,9 @@ class GlobalRequestQueue {
         this.requestCount = 0;
         this.dailyReset = new Date();
         this.lastRequestTime = 0;
-        this.maxRequestsPerDay = 10000;
-        this.maxRequestsPerSecond = 10;
-        this.minRequestInterval = 100;
+        this.maxRequestsPerDay = ETSY_RATE_LIMITS.MAX_REQUESTS_PER_DAY;
+        this.maxRequestsPerSecond = ETSY_RATE_LIMITS.MAX_REQUESTS_PER_SECOND;
+        this.minRequestInterval = ETSY_RATE_LIMITS.MIN_REQUEST_INTERVAL;
         this.setNextDailyReset();
     }
     static getInstance() {
@@ -2697,11 +2708,12 @@ class GlobalRequestQueue {
                 try {
                     let result;
                     if (item.timeout) {
-                        const remainingTimeout = item.timeout - elapsed;
+                        const timeout = item.timeout;
+                        const remainingTimeout = timeout - elapsed;
                         let timeoutId;
                         const timeoutPromise = new Promise((_, reject) => {
                             timeoutId = setTimeout(() => {
-                                reject(new Error(`Request timeout after ${item.timeout}ms (exceeded during execution)`));
+                                reject(new Error(`Request timeout after ${timeout}ms (exceeded during execution)`));
                             }, remainingTimeout);
                             if (timeoutId && typeof timeoutId.unref === 'function') {
                                 timeoutId.unref();
@@ -3018,7 +3030,18 @@ class EtsyWebhookHandler {
         }
     }
     parseEvent(payload) {
-        const data = typeof payload === 'string' ? JSON.parse(payload) : payload;
+        let data;
+        if (typeof payload === 'string') {
+            try {
+                data = JSON.parse(payload);
+            }
+            catch {
+                throw new Error('Invalid webhook payload: malformed JSON');
+            }
+        }
+        else {
+            data = payload;
+        }
         if (!data.type || !data.data) {
             throw new Error('Invalid webhook event format');
         }
@@ -3408,7 +3431,6 @@ function createCacheStorage(config = {}) {
         case 'lfu':
             return new LFUCache(config);
         case 'ttl':
-            return new LRUCache(config);
         default:
             return new LRUCache(config);
     }
@@ -4223,7 +4245,7 @@ function createCachingPlugin(config = {}) {
     };
 }
 function createRateLimitPlugin(config = {}) {
-    const maxRequestsPerSecond = config.maxRequestsPerSecond || 10;
+    const maxRequestsPerSecond = config.maxRequestsPerSecond || ETSY_RATE_LIMITS.MAX_REQUESTS_PER_SECOND;
     const requests = [];
     return {
         name: 'rateLimit',
@@ -4284,6 +4306,7 @@ exports.COMMON_SCOPE_COMBINATIONS = COMMON_SCOPE_COMBINATIONS;
 exports.CacheWithInvalidation = CacheWithInvalidation;
 exports.CreateListingSchema = CreateListingSchema;
 exports.DEFAULT_RETRY_CONFIG = DEFAULT_RETRY_CONFIG;
+exports.ETSY_RATE_LIMITS = ETSY_RATE_LIMITS;
 exports.ETSY_SCOPES = ETSY_SCOPES;
 exports.EncryptedFileTokenStorage = EncryptedFileTokenStorage;
 exports.EtsyApiError = EtsyApiError;