@naturalcycles/js-lib 15.50.1 → 15.51.0

package/dist/error/error.model.d.ts

@@ -56,6 +56,15 @@ export interface ErrorData {
      * Sentry takes string[], but for convenience we allow to pass a singe string.
      */
     fingerprint?: string;
+    /**
+     * Similar to fingerprint, with one important difference.
+     *
+     * fingerprint means FINAL fingerprint that'll be used for error reporting as is.
+     *
+     * errorGroup is meant to be used as a COMPONENT to construct a fingerprint, but not
+     * an exhaustive fingerprint by itself.
+     */
+    errorGroup?: string;
     /**
      * Set when throwing an error from your backend code, to indicate desired http status code.
      * e.g throw new AppError('oj', { backendResponseStatusCode: 401 })
@@ -95,6 +104,11 @@ export interface HttpRequestErrorData extends ErrorData {
      * GET /some/url
      */
     requestSignature: string;
+    /**
+     * FetcherOptions.requestName, if it was provided.
+     * Allows to construct errorGroup/fingerprint.
+     */
+    requestName?: string;
     /**
      * Can be set to 0 if request "failed to start" or "failed to reach the server".
      */

package/dist/http/fetcher.js

@@ -440,6 +440,7 @@ export class Fetcher {
             requestBaseUrl: this.cfg.baseUrl || undefined,
             requestMethod: res.req.init.method,
             requestSignature: res.signature,
+            requestName: res.req.requestName,
             requestDuration: Date.now() - res.req.started,
         }), {
             cause,
@@ -580,11 +581,11 @@ export class Fetcher {
         return shortUrl;
     }
     normalizeCfg(cfg) {
+        const { debug = false, logger = console } = cfg;
         if (cfg.baseUrl?.endsWith('/')) {
-            console.warn(`Fetcher: baseUrl should not end with slash: ${cfg.baseUrl}`);
+            logger.warn(`Fetcher: baseUrl should not end with slash: ${cfg.baseUrl}`);
             cfg.baseUrl = cfg.baseUrl.slice(0, cfg.baseUrl.length - 1);
         }
-        const { debug = false } = cfg;
         const norm = _merge({
             baseUrl: '',
             inputUrl: '',
@@ -595,8 +596,7 @@ export class Fetcher {
             retry3xx: false,
             retry4xx: false,
             retry5xx: true,
-            // logger: console, Danger! doing this mutates console!
-            logger: cfg.logger || console,
+            logger,
             debug,
             logRequest: debug,
             logRequestBody: debug,
@@ -665,11 +665,12 @@ export class Fetcher {
         // setup url
         const baseUrl = opt.baseUrl || this.cfg.baseUrl;
         if (baseUrl) {
-            if (req.fullUrl.startsWith('/')) {
-                console.warn('Fetcher: url should not start with / when baseUrl is specified');
-                req.fullUrl = req.fullUrl.slice(1);
+            let { inputUrl } = req;
+            if (inputUrl.startsWith('/')) {
+                this.cfg.logger.warn('Fetcher: url should not start with / when baseUrl is specified');
+                inputUrl = inputUrl.slice(1);
             }
-            req.fullUrl = `${baseUrl}/${req.inputUrl}`;
+            req.fullUrl = `${baseUrl}/${inputUrl}`;
         }
         const searchParams = _filterUndefinedValues({
             ...this.cfg.searchParams,

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

@@ -263,6 +263,12 @@ export interface FetcherOptions {
      * Cannot cancel/prevent the error - AfterResponseHook can be used for that instead.
      */
     onError?: FetcherOnErrorHook;
+    /**
+     * If provided - will be passed further to HttpRequestError if error happens,
+     * allowing to construct an errorGroup/fingerprint to be able to group errors
+     * related to "this type of request".
+     */
+    requestName?: string;
 }
 export type RequestInitNormalized = Omit<RequestInit, 'method' | 'headers'> & {
     method: HttpMethod;

package/dist/string/stringify.d.ts

@@ -13,6 +13,7 @@ import type { Reviver } from '../types.js';
  * It's recommended that this function is circular-reference-safe.
  */
 export declare function setGlobalStringifyFunction(fn: JsonStringifyFunction): void;
+export declare function resetGlobalStringifyFunction(): void;
 export type JsonStringifyFunction = (obj: any, reviver?: Reviver, space?: number) => string;
 export interface StringifyOptions {
     /**

package/dist/string/stringify.js

@@ -20,6 +20,9 @@ let globalStringifyFunction = _safeJsonStringify;
 export function setGlobalStringifyFunction(fn) {
     globalStringifyFunction = fn;
 }
+export function resetGlobalStringifyFunction() {
+    globalStringifyFunction = _safeJsonStringify;
+}
 /**
  * Inspired by `_inspect` from nodejs-lib, which is based on util.inpect that is not available in the Browser.
  * Potentially can do this (with extra 2Kb gz size): https://github.com/deecewan/browser-util-inspect

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@naturalcycles/js-lib",
   "type": "module",
-  "version": "15.50.1",
+  "version": "15.51.0",
   "dependencies": {
     "tslib": "^2",
     "undici": "^7",
@@ -13,7 +13,7 @@
     "@types/semver": "^7",
     "crypto-js": "^4",
     "dayjs": "^1",
-    "@naturalcycles/dev-lib": "20.12.10"
+    "@naturalcycles/dev-lib": "20.12.12"
   },
   "exports": {
     ".": "./dist/index.js",

package/src/error/error.model.ts

@@ -65,6 +65,16 @@ export interface ErrorData {
    */
   fingerprint?: string
 
+  /**
+   * Similar to fingerprint, with one important difference.
+   *
+   * fingerprint means FINAL fingerprint that'll be used for error reporting as is.
+   *
+   * errorGroup is meant to be used as a COMPONENT to construct a fingerprint, but not
+   * an exhaustive fingerprint by itself.
+   */
+  errorGroup?: string
+
   /**
    * Set when throwing an error from your backend code, to indicate desired http status code.
    * e.g throw new AppError('oj', { backendResponseStatusCode: 401 })
@@ -110,6 +120,11 @@ export interface HttpRequestErrorData extends ErrorData {
    * GET /some/url
    */
   requestSignature: string
+  /**
+   * FetcherOptions.requestName, if it was provided.
+   * Allows to construct errorGroup/fingerprint.
+   */
+  requestName?: string
   /**
    * Can be set to 0 if request "failed to start" or "failed to reach the server".
    */

package/src/http/fetcher.model.ts

@@ -333,6 +333,13 @@ export interface FetcherOptions {
    * Cannot cancel/prevent the error - AfterResponseHook can be used for that instead.
    */
   onError?: FetcherOnErrorHook
+
+  /**
+   * If provided - will be passed further to HttpRequestError if error happens,
+   * allowing to construct an errorGroup/fingerprint to be able to group errors
+   * related to "this type of request".
+   */
+  requestName?: string
 }
 
 export type RequestInitNormalized = Omit<RequestInit, 'method' | 'headers'> & {

package/src/http/fetcher.ts

@@ -551,6 +551,7 @@ export class Fetcher {
         requestBaseUrl: this.cfg.baseUrl || undefined,
         requestMethod: res.req.init.method,
         requestSignature: res.signature,
+        requestName: res.req.requestName,
         requestDuration: Date.now() - res.req.started,
       }),
       {
@@ -712,11 +713,12 @@ export class Fetcher {
   }
 
   private normalizeCfg(cfg: FetcherCfg & FetcherOptions): FetcherNormalizedCfg {
+    const { debug = false, logger = console } = cfg
+
     if (cfg.baseUrl?.endsWith('/')) {
-      console.warn(`Fetcher: baseUrl should not end with slash: ${cfg.baseUrl}`)
+      logger.warn(`Fetcher: baseUrl should not end with slash: ${cfg.baseUrl}`)
       cfg.baseUrl = cfg.baseUrl.slice(0, cfg.baseUrl.length - 1)
     }
-    const { debug = false } = cfg
 
     const norm: FetcherNormalizedCfg = _merge(
       {
@@ -729,8 +731,7 @@ export class Fetcher {
         retry3xx: false,
         retry4xx: false,
         retry5xx: true,
-        // logger: console, Danger! doing this mutates console!
-        logger: cfg.logger || console,
+        logger,
         debug,
         logRequest: debug,
         logRequestBody: debug,
@@ -809,11 +810,12 @@ export class Fetcher {
     // setup url
     const baseUrl = opt.baseUrl || this.cfg.baseUrl
     if (baseUrl) {
-      if (req.fullUrl.startsWith('/')) {
-        console.warn('Fetcher: url should not start with / when baseUrl is specified')
-        req.fullUrl = req.fullUrl.slice(1)
+      let { inputUrl } = req
+      if (inputUrl.startsWith('/')) {
+        this.cfg.logger.warn('Fetcher: url should not start with / when baseUrl is specified')
+        inputUrl = inputUrl.slice(1)
       }
-      req.fullUrl = `${baseUrl}/${req.inputUrl}`
+      req.fullUrl = `${baseUrl}/${inputUrl}`
     }
 
     const searchParams = _filterUndefinedValues({

package/src/string/stringify.ts

@@ -26,6 +26,10 @@ export function setGlobalStringifyFunction(fn: JsonStringifyFunction): void {
   globalStringifyFunction = fn
 }
 
+export function resetGlobalStringifyFunction(): void {
+  globalStringifyFunction = _safeJsonStringify
+}
+
 export type JsonStringifyFunction = (obj: any, reviver?: Reviver, space?: number) => string
 
 export interface StringifyOptions {