@naturalcycles/js-lib 14.139.0 → 14.139.2

package/dist/http/fetcher.d.ts

@@ -3,8 +3,6 @@ import type { FetcherAfterResponseHook, FetcherBeforeRequestHook, FetcherBeforeR
 /**
  * Experimental wrapper around Fetch.
  * Works in both Browser and Node, using `globalThis.fetch`.
- *
- * @experimental
  */
 export declare class Fetcher {
     private constructor();

package/dist/http/fetcher.js

@@ -20,8 +20,6 @@ const defRetryOptions = {
 /**
  * Experimental wrapper around Fetch.
  * Works in both Browser and Node, using `globalThis.fetch`.
- *
- * @experimental
  */
 class Fetcher {
     constructor(cfg = {}) {
@@ -121,7 +119,6 @@ class Fetcher {
             },
             signature,
         };
-        /* eslint-disable no-await-in-loop */
         while (!res.retryStatus.retryStopped) {
             const started = Date.now();
             if (this.cfg.logRequest) {
@@ -141,6 +138,8 @@ class Fetcher {
                 // For example, CORS error would result in "TypeError: failed to fetch" here
                 res.err = err;
                 res.ok = false;
+                // important to set it to undefined, otherwise it can keep the previous value (from previous try)
+                res.fetchResponse = undefined;
             }
             res.statusFamily = this.getStatusFamily(res);
             if (res.fetchResponse?.ok) {
@@ -340,7 +339,8 @@ class Fetcher {
             retryPost: false,
             retry4xx: false,
             retry5xx: true,
-            logger: console,
+            // logger: console, Danger! doing this mutates console!
+            logger: cfg.logger || console,
             debug,
             logRequest: debug,
             logRequestBody: debug,
@@ -355,7 +355,7 @@ class Fetcher {
                 credentials: cfg.credentials,
             },
             hooks: {},
-        }, (0, object_util_1._omit)(cfg, ['method', 'credentials', 'headers']));
+        }, (0, object_util_1._omit)(cfg, ['method', 'credentials', 'headers', 'logger']));
         norm.init.headers = (0, object_util_1._mapKeys)(norm.init.headers, k => k.toLowerCase());
         return norm;
     }

package/dist/promise/pRetry.js

@@ -25,7 +25,7 @@ async function pRetry(fn, opt = {}) {
     const fname = name || fn.name || 'pRetry function';
     let delay = initialDelay;
     let attempt = 0;
-    /* eslint-disable no-await-in-loop, no-constant-condition */
+    /* eslint-disable no-constant-condition */
     while (true) {
         const started = Date.now();
         try {

package/dist/seq/seq.js

@@ -149,7 +149,6 @@ function _seq(initialValue, nextFn) {
     return Sequence.create(initialValue, nextFn);
 }
 exports._seq = _seq;
-/* eslint-disable no-await-in-loop */
 /**
  * Experimental.
  * Feasibility to be proven.

package/dist-esm/http/fetcher.js

@@ -18,8 +18,6 @@ const defRetryOptions = {
 /**
  * Experimental wrapper around Fetch.
  * Works in both Browser and Node, using `globalThis.fetch`.
- *
- * @experimental
  */
 export class Fetcher {
     constructor(cfg = {}) {
@@ -129,7 +127,6 @@ export class Fetcher {
             },
             signature,
         };
-        /* eslint-disable no-await-in-loop */
         while (!res.retryStatus.retryStopped) {
             const started = Date.now();
             if (this.cfg.logRequest) {
@@ -149,6 +146,8 @@ export class Fetcher {
                 // For example, CORS error would result in "TypeError: failed to fetch" here
                 res.err = err;
                 res.ok = false;
+                // important to set it to undefined, otherwise it can keep the previous value (from previous try)
+                res.fetchResponse = undefined;
             }
             res.statusFamily = this.getStatusFamily(res);
             if ((_g = res.fetchResponse) === null || _g === void 0 ? void 0 : _g.ok) {
@@ -382,7 +381,8 @@ export class Fetcher {
             retryPost: false,
             retry4xx: false,
             retry5xx: true,
-            logger: console,
+            // logger: console, Danger! doing this mutates console!
+            logger: cfg.logger || console,
             debug,
             logRequest: debug,
             logRequestBody: debug,
@@ -397,7 +397,7 @@ export class Fetcher {
                 credentials: cfg.credentials,
             },
             hooks: {},
-        }, _omit(cfg, ['method', 'credentials', 'headers']));
+        }, _omit(cfg, ['method', 'credentials', 'headers', 'logger']));
         norm.init.headers = _mapKeys(norm.init.headers, k => k.toLowerCase());
         return norm;
     }

package/dist-esm/promise/pRetry.js

@@ -21,7 +21,7 @@ export async function pRetry(fn, opt = {}) {
     const fname = name || fn.name || 'pRetry function';
     let delay = initialDelay;
     let attempt = 0;
-    /* eslint-disable no-await-in-loop, no-constant-condition */
+    /* eslint-disable no-constant-condition */
     while (true) {
         const started = Date.now();
         try {

package/dist-esm/seq/seq.js

@@ -144,7 +144,6 @@ export class Sequence {
 export function _seq(initialValue, nextFn) {
     return Sequence.create(initialValue, nextFn);
 }
-/* eslint-disable no-await-in-loop */
 /**
  * Experimental.
  * Feasibility to be proven.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@naturalcycles/js-lib",
-  "version": "14.139.0",
+  "version": "14.139.2",
   "scripts": {
     "prepare": "husky install",
     "build-prod": "build-prod-esm-cjs",

package/src/http/fetcher.ts

@@ -39,8 +39,6 @@ const defRetryOptions: FetcherRetryOptions = {
 /**
  * Experimental wrapper around Fetch.
  * Works in both Browser and Node, using `globalThis.fetch`.
- *
- * @experimental
  */
 export class Fetcher {
   private constructor(cfg: FetcherCfg & FetcherOptions = {}) {
@@ -184,7 +182,6 @@ export class Fetcher {
       signature,
     } as FetcherResponse<any>
 
-    /* eslint-disable no-await-in-loop */
     while (!res.retryStatus.retryStopped) {
       const started = Date.now()
 
@@ -207,6 +204,8 @@ export class Fetcher {
         // For example, CORS error would result in "TypeError: failed to fetch" here
         res.err = err as Error
         res.ok = false
+        // important to set it to undefined, otherwise it can keep the previous value (from previous try)
+        res.fetchResponse = undefined
       }
       res.statusFamily = this.getStatusFamily(res)
 
@@ -435,7 +434,8 @@ export class Fetcher {
         retryPost: false,
         retry4xx: false,
         retry5xx: true,
-        logger: console,
+        // logger: console, Danger! doing this mutates console!
+        logger: cfg.logger || console,
         debug,
         logRequest: debug,
         logRequestBody: debug,
@@ -451,7 +451,7 @@ export class Fetcher {
         },
         hooks: {},
       },
-      _omit(cfg, ['method', 'credentials', 'headers']),
+      _omit(cfg, ['method', 'credentials', 'headers', 'logger']),
     )
 
     norm.init.headers = _mapKeys(norm.init.headers, k => k.toLowerCase())

package/src/promise/pRetry.ts

@@ -128,7 +128,7 @@ export async function pRetry<T>(
   let delay = initialDelay
   let attempt = 0
 
-  /* eslint-disable no-await-in-loop, no-constant-condition */
+  /* eslint-disable no-constant-condition */
   while (true) {
     const started = Date.now()
 

package/src/seq/seq.ts

@@ -158,8 +158,6 @@ export function _seq<T>(initialValue: T | typeof END, nextFn: AbortableMapper<T,
   return Sequence.create(initialValue, nextFn)
 }
 
-/* eslint-disable no-await-in-loop */
-
 /**
  * Experimental.
  * Feasibility to be proven.