@naturalcycles/js-lib 14.176.0 → 14.177.1

package/dist/error/error.util.js

@@ -246,7 +246,9 @@ class AppError extends Error {
     }
     constructor(message, data = {}, opt = {}) {
         super(message);
-        const { name = 'AppError', cause } = opt;
+        // Here we default to `this.constructor.name` on Node, but to 'AppError' on the Frontend
+        // because Frontend tends to minify class names, so `constructor.name` is not reliable
+        const { name = (0, __1.isServerSide)() ? this.constructor.name : 'AppError', cause } = opt;
         Object.defineProperties(this, {
             name: {
                 value: name,

package/dist/http/fetcher.d.ts

@@ -15,7 +15,7 @@ export declare class Fetcher {
      *
      * Version is to be incremented every time a difference in behaviour (or a bugfix) is done.
      */
-    static readonly VERSION = 1;
+    static readonly VERSION = 2;
     static readonly userAgent: string | undefined;
     private constructor();
     /**

package/dist/http/fetcher.js

@@ -40,7 +40,7 @@ class Fetcher {
      *
      * Version is to be incremented every time a difference in behaviour (or a bugfix) is done.
      */
-    static { this.VERSION = 1; }
+    static { this.VERSION = 2; }
     static { this.userAgent = (0, env_1.isServerSide)() ? `fetcher${this.VERSION}` : undefined; }
     constructor(cfg = {}) {
         if (typeof globalThis.fetch !== 'function') {
@@ -224,7 +224,7 @@ class Fetcher {
             }
             res.statusFamily = this.getStatusFamily(res);
             res.statusCode = res.fetchResponse?.status;
-            if (res.fetchResponse?.ok) {
+            if (res.fetchResponse?.ok || !req.throwHttpErrors) {
                 try {
                     // We are applying a separate Timeout (as long as original Timeout for now) to "download and parse the body"
                     await (0, pTimeout_1.pTimeout)(async () => await this.onOkResponse(res), {
@@ -293,7 +293,7 @@ class Fetcher {
         }
         res.retryStatus.retryStopped = true;
         // res.err can happen on `failed to fetch` type of error, e.g JSON.parse, CORS, unexpected redirect
-        if (!res.err && this.cfg.logResponse) {
+        if ((!res.err || !req.throwHttpErrors) && this.cfg.logResponse) {
             const { retryAttempt } = res.retryStatus;
             const { logger } = this.cfg;
             logger.log([
@@ -531,6 +531,7 @@ class Fetcher {
                 redirect: cfg.redirect,
             },
             hooks: {},
+            throwHttpErrors: true,
         }, (0, object_util_1._omit)(cfg, ['method', 'credentials', 'headers', 'redirect', 'logger']));
         norm.init.headers = (0, object_util_1._mapKeys)(norm.init.headers, k => k.toLowerCase());
         return norm;
@@ -549,6 +550,7 @@ class Fetcher {
                 'logResponse',
                 'logResponseBody',
                 'debug',
+                'throwHttpErrors',
             ]),
             started: Date.now(),
             ...(0, object_util_1._omit)(opt, ['method', 'headers', 'credentials']),

package/dist/http/fetcher.model.d.ts

@@ -4,7 +4,7 @@ import type { CommonLogger } from '../log/commonLogger';
 import type { Promisable } from '../typeFest';
 import type { AnyObject, NumberOfMilliseconds, Reviver, UnixTimestampMillisNumber } from '../types';
 import type { HttpMethod, HttpStatusFamily } from './http.model';
-export interface FetcherNormalizedCfg extends Required<FetcherCfg>, Omit<FetcherRequest, 'started' | 'fullUrl' | 'logRequest' | 'logRequestBody' | 'logResponse' | 'logResponseBody' | 'debug' | 'redirect' | 'credentials'> {
+export interface FetcherNormalizedCfg extends Required<FetcherCfg>, Omit<FetcherRequest, 'started' | 'fullUrl' | 'logRequest' | 'logRequestBody' | 'logResponse' | 'logResponseBody' | 'debug' | 'redirect' | 'credentials' | 'throwHttpErrors'> {
     logger: CommonLogger;
     searchParams: Record<string, any>;
 }
@@ -67,6 +67,7 @@ export interface FetcherCfg {
      * Defaults to `console`.
      */
     logger?: CommonLogger;
+    throwHttpErrors?: boolean;
 }
 export interface FetcherRetryStatus {
     retryAttempt: number;
@@ -182,6 +183,11 @@ export interface FetcherOptions {
      * If provided - will be used instead of static `Fetcher.callNativeFetch`.
      */
     fetchFn?: FetchFunction;
+    /**
+     * Default to true.
+     * Set to false to not throw on `!Response.ok`, but simply return `Response.body` as-is (json parsed, etc).
+     */
+    throwHttpErrors?: boolean;
 }
 export type RequestInitNormalized = Omit<RequestInit, 'method' | 'headers'> & {
     method: HttpMethod;

package/dist-esm/error/error.util.js

@@ -1,4 +1,4 @@
-import { _jsonParseIfPossible, _stringifyAny, _truncate, _truncateMiddle } from '..';
+import { _jsonParseIfPossible, _stringifyAny, _truncate, _truncateMiddle, isServerSide } from '..';
 /**
  * Useful to ensure that error in `catch (err) { ... }`
  * is indeed an Error (and not e.g `string` or `undefined`).
@@ -228,7 +228,9 @@ export class AppError extends Error {
     }
     constructor(message, data = {}, opt = {}) {
         super(message);
-        const { name = 'AppError', cause } = opt;
+        // Here we default to `this.constructor.name` on Node, but to 'AppError' on the Frontend
+        // because Frontend tends to minify class names, so `constructor.name` is not reliable
+        const { name = isServerSide() ? this.constructor.name : 'AppError', cause } = opt;
         Object.defineProperties(this, {
             name: {
                 value: name,

package/dist-esm/http/fetcher.js

@@ -202,7 +202,7 @@ export class Fetcher {
             }
             res.statusFamily = this.getStatusFamily(res);
             res.statusCode = (_b = res.fetchResponse) === null || _b === void 0 ? void 0 : _b.status;
-            if ((_c = res.fetchResponse) === null || _c === void 0 ? void 0 : _c.ok) {
+            if (((_c = res.fetchResponse) === null || _c === void 0 ? void 0 : _c.ok) || !req.throwHttpErrors) {
                 try {
                     // We are applying a separate Timeout (as long as original Timeout for now) to "download and parse the body"
                     await pTimeout(async () => await this.onOkResponse(res), {
@@ -271,7 +271,7 @@ export class Fetcher {
         }
         res.retryStatus.retryStopped = true;
         // res.err can happen on `failed to fetch` type of error, e.g JSON.parse, CORS, unexpected redirect
-        if (!res.err && this.cfg.logResponse) {
+        if ((!res.err || !req.throwHttpErrors) && this.cfg.logResponse) {
             const { retryAttempt } = res.retryStatus;
             const { logger } = this.cfg;
             logger.log([
@@ -511,6 +511,7 @@ export class Fetcher {
                 redirect: cfg.redirect,
             },
             hooks: {},
+            throwHttpErrors: true,
         }, _omit(cfg, ['method', 'credentials', 'headers', 'redirect', 'logger']));
         norm.init.headers = _mapKeys(norm.init.headers, k => k.toLowerCase());
         return norm;
@@ -529,6 +530,7 @@ export class Fetcher {
             'logResponse',
             'logResponseBody',
             'debug',
+            'throwHttpErrors',
         ])), { started: Date.now() }), _omit(opt, ['method', 'headers', 'credentials'])), { inputUrl: opt.url || '', fullUrl: opt.url || '', retry: Object.assign(Object.assign({}, this.cfg.retry), _filterUndefinedValues(opt.retry || {})), init: _merge(Object.assign(Object.assign({}, this.cfg.init), { headers: Object.assign({}, this.cfg.init.headers), method: opt.method || this.cfg.init.method, credentials: opt.credentials || this.cfg.init.credentials, redirect: opt.redirect || this.cfg.init.redirect || 'follow' }), {
                 headers: _mapKeys(opt.headers || {}, k => k.toLowerCase()),
             }) });
@@ -582,7 +584,7 @@ _a = Fetcher;
  *
  * Version is to be incremented every time a difference in behaviour (or a bugfix) is done.
  */
-Fetcher.VERSION = 1;
+Fetcher.VERSION = 2;
 Fetcher.userAgent = isServerSide() ? `fetcher${_a.VERSION}` : undefined;
 export function getFetcher(cfg = {}) {
     return Fetcher.create(cfg);

package/package.json

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

package/src/error/error.util.ts

@@ -6,7 +6,7 @@ import type {
   HttpRequestErrorData,
   ErrorLike,
 } from '..'
-import { _jsonParseIfPossible, _stringifyAny, _truncate, _truncateMiddle } from '..'
+import { _jsonParseIfPossible, _stringifyAny, _truncate, _truncateMiddle, isServerSide } from '..'
 
 /**
  * Useful to ensure that error in `catch (err) { ... }`
@@ -325,7 +325,9 @@ export class AppError<DATA_TYPE extends ErrorData = ErrorData> extends Error {
 
   constructor(message: string, data = {} as DATA_TYPE, opt: AppErrorOptions = {}) {
     super(message)
-    const { name = 'AppError', cause } = opt
+    // Here we default to `this.constructor.name` on Node, but to 'AppError' on the Frontend
+    // because Frontend tends to minify class names, so `constructor.name` is not reliable
+    const { name = isServerSide() ? this.constructor.name : 'AppError', cause } = opt
 
     Object.defineProperties(this, {
       name: {

package/src/http/fetcher.model.ts

@@ -19,6 +19,7 @@ export interface FetcherNormalizedCfg
       | 'debug'
       | 'redirect'
       | 'credentials'
+      | 'throwHttpErrors'
     > {
   logger: CommonLogger
   searchParams: Record<string, any>
@@ -93,6 +94,8 @@ export interface FetcherCfg {
    * Defaults to `console`.
    */
   logger?: CommonLogger
+
+  throwHttpErrors?: boolean
 }
 
 export interface FetcherRetryStatus {
@@ -232,6 +235,12 @@ export interface FetcherOptions {
    * If provided - will be used instead of static `Fetcher.callNativeFetch`.
    */
   fetchFn?: FetchFunction
+
+  /**
+   * Default to true.
+   * Set to false to not throw on `!Response.ok`, but simply return `Response.body` as-is (json parsed, etc).
+   */
+  throwHttpErrors?: boolean
 }
 
 export type RequestInitNormalized = Omit<RequestInit, 'method' | 'headers'> & {

package/src/http/fetcher.ts

@@ -71,7 +71,7 @@ export class Fetcher {
    *
    * Version is to be incremented every time a difference in behaviour (or a bugfix) is done.
    */
-  static readonly VERSION = 1
+  static readonly VERSION = 2
   static readonly userAgent = isServerSide() ? `fetcher${this.VERSION}` : undefined
 
   private constructor(cfg: FetcherCfg & FetcherOptions = {}) {
@@ -308,7 +308,7 @@ export class Fetcher {
       res.statusFamily = this.getStatusFamily(res)
       res.statusCode = res.fetchResponse?.status
 
-      if (res.fetchResponse?.ok) {
+      if (res.fetchResponse?.ok || !req.throwHttpErrors) {
         try {
           // We are applying a separate Timeout (as long as original Timeout for now) to "download and parse the body"
           await pTimeout(
@@ -384,7 +384,7 @@ export class Fetcher {
     res.retryStatus.retryStopped = true
 
     // res.err can happen on `failed to fetch` type of error, e.g JSON.parse, CORS, unexpected redirect
-    if (!res.err && this.cfg.logResponse) {
+    if ((!res.err || !req.throwHttpErrors) && this.cfg.logResponse) {
       const { retryAttempt } = res.retryStatus
       const { logger } = this.cfg
       logger.log(
@@ -654,6 +654,7 @@ export class Fetcher {
           redirect: cfg.redirect,
         },
         hooks: {},
+        throwHttpErrors: true,
       },
       _omit(cfg, ['method', 'credentials', 'headers', 'redirect', 'logger']),
     )
@@ -677,6 +678,7 @@ export class Fetcher {
         'logResponse',
         'logResponseBody',
         'debug',
+        'throwHttpErrors',
       ]),
       started: Date.now(),
       ..._omit(opt, ['method', 'headers', 'credentials']),