@naturalcycles/nodejs-lib 12.48.1 → 12.51.0

package/dist/colors/colors.d.ts

@@ -1 +1,6 @@
-export declare function hasColors(): boolean;
+/**
+ * Based on: https://github.com/sindresorhus/yoctocolors/pull/5
+ *
+ * @experimental
+ */
+export declare const hasColors: boolean;

package/dist/colors/colors.js

@@ -1,10 +1,10 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.hasColors = void 0;
-const supporsColorLib = require('supports-color');
-function hasColors() {
-    if (process.env['NO_COLOR'])
-        return false; // https://no-color.org/
-    return !!supporsColorLib.stdout;
-}
-exports.hasColors = hasColors;
+const tty = require("tty");
+/**
+ * Based on: https://github.com/sindresorhus/yoctocolors/pull/5
+ *
+ * @experimental
+ */
+exports.hasColors = !process.env['NO_COLOR'] && tty.WriteStream.prototype.hasColors();

package/dist/security/secret.util.js

@@ -5,8 +5,6 @@ const fs = require("fs");
 const __1 = require("..");
 const crypto_util_1 = require("./crypto.util");
 let loaded = false;
-// it's wrapped to be able to pipe console.* to Stackdriver
-const getLog = () => (0, __1.Debug)('nc:nodejs-lib:secret');
 const secretMap = {};
 /**
  * Loads plaintext secrets from process.env, removes them, stores locally.
@@ -25,7 +23,7 @@ function loadSecretsFromEnv() {
         delete process.env[k];
     });
     loaded = true;
-    getLog()(`${Object.keys(secrets).length} secret(s) loaded from process.env: ${Object.keys(secrets).join(', ')}`);
+    console.log(`${Object.keys(secrets).length} secret(s) loaded from process.env: ${Object.keys(secrets).join(', ')}`);
 }
 exports.loadSecretsFromEnv = loadSecretsFromEnv;
 /**
@@ -58,7 +56,7 @@ function loadSecretsFromJsonFile(filePath, SECRET_ENCRYPTION_KEY) {
     }
     Object.entries(secrets).forEach(([k, v]) => (secretMap[k.toUpperCase()] = v));
     loaded = true;
-    getLog()(`${Object.keys(secrets).length} secret(s) loaded from ${filePath}: ${Object.keys(secrets)
+    console.log(`${Object.keys(secrets).length} secret(s) loaded from ${filePath}: ${Object.keys(secrets)
         .map(s => s.toUpperCase())
         .join(', ')}`);
 }
@@ -91,7 +89,7 @@ exports.getSecretMap = getSecretMap;
 function setSecretMap(map) {
     Object.keys(secretMap).forEach(k => delete secretMap[k]);
     Object.entries(map).forEach(([k, v]) => (secretMap[k.toUpperCase()] = v));
-    getLog()(`setSecretMap set ${Object.keys(secretMap).length} secret(s): ${Object.keys(map)
+    console.log(`setSecretMap set ${Object.keys(secretMap).length} secret(s): ${Object.keys(map)
         .map(s => s.toUpperCase())
         .join(', ')}`);
 }

package/dist/slack/slack.service.d.ts

@@ -19,7 +19,7 @@ export declare class SlackService<CTX = any> {
      * Allows to "log" many things at once, similar to `console.log(one, two, three).
      */
     log(...items: any[]): Promise<void>;
-    send(msg: SlackMessage<CTX> | string, ctx?: CTX): Promise<void>;
+    send(input: SlackMessage<CTX> | string, ctx?: CTX): Promise<void>;
     kvToFields(kv: StringMap<any>): SlackAttachmentField[];
 }
 export declare function slackDefaultMessagePrefixHook(msg: SlackMessage): string[];

package/dist/slack/slack.service.js

@@ -1,22 +1,22 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.slackDefaultMessagePrefixHook = exports.SlackService = void 0;
+const js_lib_1 = require("@naturalcycles/js-lib");
 const time_lib_1 = require("@naturalcycles/time-lib");
 const got_1 = require("got");
 const __1 = require("..");
 const GAE = !!process.env['GAE_INSTANCE'];
-const DEFAULTS = () => ({
+const DEFAULTS = {
     username: 'bot',
     channel: '#log',
     icon_emoji: ':spider_web:',
     items: 'no text',
-});
+};
 const INSPECT_OPT = {
     colors: false,
     includeErrorData: true,
     includeErrorStack: true,
 };
-const log = (0, __1.Debug)('nc:nodejs-lib:slack');
 /**
  * Has 2 main methods:
  *
@@ -33,7 +33,12 @@ class SlackService {
     constructor(cfg) {
         this.cfg = {
             messagePrefixHook: slackDefaultMessagePrefixHook,
+            logger: console,
             ...cfg,
+            inspectOptions: {
+                ...INSPECT_OPT,
+                ...cfg.inspectOptions,
+            },
         };
     }
     /**
@@ -45,38 +50,29 @@ class SlackService {
             items: items.length === 1 ? items[0] : items,
         });
     }
-    async send(msg, ctx) {
-        const { webhookUrl, messagePrefixHook } = this.cfg;
+    async send(input, ctx) {
+        const { webhookUrl, messagePrefixHook, inspectOptions } = this.cfg;
         // If String is passed as first argument - just transform it to a full SlackMessage
-        if (typeof msg === 'string') {
-            msg = {
-                items: msg,
-            };
-        }
+        const msg = typeof input === 'string' ? { items: input } : input;
         if (ctx !== undefined) {
             Object.assign(msg, { ctx });
         }
-        if (!msg.noLog) {
-            log[msg.level || __1.DebugLogLevel.info](...[msg.items, msg.kv, msg.attachments, msg.mentions].filter(Boolean));
-        }
+        this.cfg.logger.log(...[msg.items, msg.kv, msg.attachments, msg.mentions].filter(Boolean));
         if (!webhookUrl)
             return;
         // Transform msg.kv into msg.attachments
         if (msg.kv) {
-            msg.attachments = [...(msg.attachments || []), { fields: this.kvToFields(msg.kv) }];
+            ;
+            (msg.attachments || (msg.attachments = [])).push({ fields: this.kvToFields(msg.kv) });
             delete msg.kv; // to not pass it all the way to Slack Api
         }
         let text;
-        const inspectOpt = {
-            ...INSPECT_OPT,
-            ...msg.inspectOptions,
-        };
         // Array has a special treatment here
         if (Array.isArray(msg.items)) {
-            text = msg.items.map(t => (0, __1.inspectAny)(t, inspectOpt)).join('\n');
+            text = msg.items.map(t => (0, __1.inspectAny)(t, inspectOptions)).join('\n');
         }
         else {
-            text = (0, __1.inspectAny)(msg.items, inspectOpt);
+            text = (0, __1.inspectAny)(msg.items, inspectOptions);
         }
         // Wrap in markdown-text-block if it's anything but plain String
         if (typeof msg.items !== 'string') {
@@ -88,22 +84,18 @@ class SlackService {
         const prefix = await messagePrefixHook(msg);
         if (prefix === null)
             return; // filtered out!
-        const json = {
-            ...DEFAULTS(),
+        const json = (0, js_lib_1._omit)({
+            ...DEFAULTS,
             ...this.cfg.defaults,
             ...msg,
             // Text with Prefix
             text: [prefix.join(': '), text].filter(Boolean).join('\n'),
-        };
-        // they're not needed in the json payload
-        delete json['items'];
-        delete json['ctx'];
-        delete json['noLog'];
-        json.channel = (this.cfg.channelByLevel || {})[msg.level] || json.channel;
+        }, ['items', 'ctx']);
         await got_1.default
             .post(webhookUrl, {
             json,
             responseType: 'text',
+            timeout: 90000,
         })
             .catch(err => {
             // ignore (unless throwOnError is set)

package/dist/slack/slack.service.model.d.ts

@@ -1,5 +1,5 @@
-import { StringMap } from '@naturalcycles/js-lib';
-import { DebugLogLevel, InspectAnyOptions } from '..';
+import { CommonLogger, StringMap } from '@naturalcycles/js-lib';
+import { InspectAnyOptions } from '..';
 /**
  * Properties that exists both in SlackApiBody (as per Slack API) and SlackMessage (our abstraction).
  */
@@ -36,7 +36,6 @@ export interface SlackMessage<CTX = any> extends SlackMessageProps {
      * Optional "context object", to be used by `messagePrefixHook`.
      */
     ctx?: CTX;
-    level?: DebugLogLevel;
     /**
      * Keys-values will be rendered as MessageAttachment with Fields
      */
@@ -46,21 +45,11 @@ export interface SlackMessage<CTX = any> extends SlackMessageProps {
      */
     mentions?: string[];
     /**
-     * @default false
      * By default it ignores possible errors from slack
-     */
-    throwOnError?: boolean;
-    /**
+     *
      * @default false
-     * Skips logging message
-     */
-    noLog?: boolean;
-    /**
-     * Defaults to:
-     * includeErrorData: true
-     * includeErrorStack: true
      */
-    inspectOptions?: InspectAnyOptions;
+    throwOnError?: boolean;
 }
 export interface SlackAttachmentField {
     title: string;
@@ -96,15 +85,21 @@ export interface SlackServiceCfg<CTX = any> {
      */
     webhookUrl?: string;
     defaults?: Partial<SlackMessage>;
-    /**
-     * Override channel when msg.level is set.
-     * key: DebugLogLevel
-     * value: channel name to send message to
-     */
-    channelByLevel?: StringMap;
     /**
      * Function to return an array of "prefix tokens" (will be joined by ': ').
      * Allows to skip (filter out) the message by returning `null`.
      */
     messagePrefixHook: SlackMessagePrefixHook<CTX>;
+    /**
+     * By default SlackService logs every message to console.log
+     * Pass another logger if needed.
+     * Pass `noopLogger` to suppress logging completely.
+     */
+    logger: CommonLogger;
+    /**
+     * Defaults to:
+     * includeErrorData: true
+     * includeErrorStack: true
+     */
+    inspectOptions: InspectAnyOptions;
 }

package/dist/stream/transform/transformLogProgress.js

@@ -8,7 +8,7 @@ const time_lib_1 = require("@naturalcycles/time-lib");
 const colors_1 = require("../../colors");
 const colors_2 = require("../../colors/colors");
 const inspectOpt = {
-    colors: (0, colors_2.hasColors)(),
+    colors: colors_2.hasColors,
     breakLength: 300,
 };
 /**
@@ -46,14 +46,14 @@ function transformLogProgress(opt = {}) {
             return;
         const mem = process.memoryUsage();
         const now = Date.now();
+        const batchedProgress = progress * batchSize;
         const lastRPS = (processedLastSecond * batchSize) / ((now - lastSecondStarted) / 1000) || 0;
-        const rpsTotal = Math.round((progress * batchSize) / ((now - started) / 1000)) || 0;
+        const rpsTotal = Math.round(batchedProgress / ((now - started) / 1000)) || 0;
         lastSecondStarted = now;
         processedLastSecond = 0;
         const rps10 = Math.round(sma.push(lastRPS));
         if (mem.rss > peakRSS)
             peakRSS = mem.rss;
-        const batchedProgress = progress * batchSize;
         console.log((0, util_1.inspect)({
             [final ? `${metric}_final` : metric]: batchedProgress,
             ...(extra ? extra(chunk, progress) : {}),
@@ -72,14 +72,14 @@ function transformLogProgress(opt = {}) {
                 : {}),
         }, inspectOpt));
         if (tenx) {
-            let perHour = Math.round((progress * 1000 * 60 * 60) / (now - started)) || 0;
+            let perHour = Math.round((batchedProgress * 1000 * 60 * 60) / (now - started)) || 0;
             if (perHour > 900) {
                 perHour = Math.round(perHour / 1000) + 'K';
             }
-            console.log(`${(0, colors_1.dimGrey)((0, time_lib_1.dayjs)().toPretty())} ${(0, colors_1.white)(metric)} took ${(0, colors_1.yellow)((0, js_lib_1._since)(started))} so far to process ${(0, colors_1.yellow)(progress)} rows, ~${(0, colors_1.yellow)(perHour)}/hour`);
+            console.log(`${(0, colors_1.dimGrey)((0, time_lib_1.dayjs)().toPretty())} ${(0, colors_1.white)(metric)} took ${(0, colors_1.yellow)((0, js_lib_1._since)(started))} so far to process ${(0, colors_1.yellow)(batchedProgress)} rows, ~${(0, colors_1.yellow)(perHour)}/hour`);
         }
         else if (final) {
-            console.log(`${(0, colors_1.boldWhite)(metric)} took ${(0, colors_1.yellow)((0, js_lib_1._since)(started))} to process ${(0, colors_1.yellow)(progress)} rows with total RPS of ${(0, colors_1.yellow)(rpsTotal)}`);
+            console.log(`${(0, colors_1.boldWhite)(metric)} took ${(0, colors_1.yellow)((0, js_lib_1._since)(started))} to process ${(0, colors_1.yellow)(batchedProgress)} rows with total RPS of ${(0, colors_1.yellow)(rpsTotal)}`);
         }
     }
 }

package/dist/validation/ajv/ajvValidationError.js

@@ -5,20 +5,6 @@ const js_lib_1 = require("@naturalcycles/js-lib");
 class AjvValidationError extends js_lib_1.AppError {
     constructor(message, data) {
         super(message, data);
-        this.constructor = AjvValidationError;
-        this.__proto__ = AjvValidationError.prototype;
-        Object.defineProperty(this, 'name', {
-            value: this.constructor.name,
-            configurable: true,
-        });
-        if (Error.captureStackTrace) {
-            Error.captureStackTrace(this, this.constructor);
-        }
-        else {
-            Object.defineProperty(this, 'stack', {
-                value: new Error().stack, //  eslint-disable-line unicorn/error-message
-            });
-        }
     }
 }
 exports.AjvValidationError = AjvValidationError;

package/dist/validation/joi/joi.validation.error.js

@@ -5,20 +5,6 @@ const js_lib_1 = require("@naturalcycles/js-lib");
 class JoiValidationError extends js_lib_1.AppError {
     constructor(message, data) {
         super(message, data);
-        this.constructor = JoiValidationError;
-        this.__proto__ = JoiValidationError.prototype;
-        Object.defineProperty(this, 'name', {
-            value: this.constructor.name,
-            configurable: true,
-        });
-        if (Error.captureStackTrace) {
-            Error.captureStackTrace(this, this.constructor);
-        }
-        else {
-            Object.defineProperty(this, 'stack', {
-                value: new Error().stack, //  eslint-disable-line unicorn/error-message
-            });
-        }
     }
 }
 exports.JoiValidationError = JoiValidationError;

package/dist/validation/joi/joi.validation.util.js

@@ -130,6 +130,7 @@ function createError(value, err, objectName) {
     // Make annotation non-enumerable, to not get it automatically printed,
     // but still accessible
     Object.defineProperty(data, 'annotation', {
+        writable: true,
         configurable: true,
         enumerable: false,
         value: annotation,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@naturalcycles/nodejs-lib",
-  "version": "12.48.1",
+  "version": "12.51.0",
   "scripts": {
     "prepare": "husky install",
     "docs-serve": "vuepress dev docs",
@@ -37,7 +37,6 @@
     "move-file": "^2.0.0",
     "nanoid": "^3.0.0",
     "sanitize-html": "^2.5.2",
-    "supports-color": "^8.0.0",
     "through2-concurrent": "^2.0.0",
     "yargs": "^17.0.0"
   },

package/src/colors/colors.ts

@@ -1,6 +1,8 @@
-const supporsColorLib = require('supports-color')
+import * as tty from 'tty'
 
-export function hasColors(): boolean {
-  if (process.env['NO_COLOR']) return false // https://no-color.org/
-  return !!supporsColorLib.stdout
-}
+/**
+ * Based on: https://github.com/sindresorhus/yoctocolors/pull/5
+ *
+ * @experimental
+ */
+export const hasColors = !process.env['NO_COLOR'] && tty.WriteStream.prototype.hasColors()

package/src/security/secret.util.ts

@@ -1,13 +1,10 @@
 import * as fs from 'fs'
 import { StringMap } from '@naturalcycles/js-lib'
-import { base64ToString, Debug } from '..'
+import { base64ToString } from '..'
 import { decryptRandomIVBuffer } from './crypto.util'
 
 let loaded = false
 
-// it's wrapped to be able to pipe console.* to Stackdriver
-const getLog = () => Debug('nc:nodejs-lib:secret')
-
 const secretMap: StringMap = {}
 
 /**
@@ -29,7 +26,7 @@ export function loadSecretsFromEnv(): void {
     })
 
   loaded = true
-  getLog()(
+  console.log(
     `${Object.keys(secrets).length} secret(s) loaded from process.env: ${Object.keys(secrets).join(
       ', ',
     )}`,
@@ -69,7 +66,7 @@ export function loadSecretsFromJsonFile(filePath: string, SECRET_ENCRYPTION_KEY?
   Object.entries(secrets).forEach(([k, v]) => (secretMap[k.toUpperCase()] = v))
 
   loaded = true
-  getLog()(
+  console.log(
     `${Object.keys(secrets).length} secret(s) loaded from ${filePath}: ${Object.keys(secrets)
       .map(s => s.toUpperCase())
       .join(', ')}`,
@@ -105,7 +102,7 @@ export function getSecretMap(): StringMap {
 export function setSecretMap(map: StringMap): void {
   Object.keys(secretMap).forEach(k => delete secretMap[k])
   Object.entries(map).forEach(([k, v]) => (secretMap[k.toUpperCase()] = v))
-  getLog()(
+  console.log(
     `setSecretMap set ${Object.keys(secretMap).length} secret(s): ${Object.keys(map)
       .map(s => s.toUpperCase())
       .join(', ')}`,

package/src/slack/slack.service.model.ts

@@ -1,5 +1,5 @@
-import { StringMap } from '@naturalcycles/js-lib'
-import { DebugLogLevel, InspectAnyOptions } from '..'
+import { CommonLogger, StringMap } from '@naturalcycles/js-lib'
+import { InspectAnyOptions } from '..'
 
 /**
  * Properties that exists both in SlackApiBody (as per Slack API) and SlackMessage (our abstraction).
@@ -44,8 +44,6 @@ export interface SlackMessage<CTX = any> extends SlackMessageProps {
    */
   ctx?: CTX
 
-  level?: DebugLogLevel
-
   /**
    * Keys-values will be rendered as MessageAttachment with Fields
    */
@@ -57,23 +55,11 @@ export interface SlackMessage<CTX = any> extends SlackMessageProps {
   mentions?: string[]
 
   /**
-   * @default false
    * By default it ignores possible errors from slack
-   */
-  throwOnError?: boolean
-
-  /**
+   *
    * @default false
-   * Skips logging message
-   */
-  noLog?: boolean
-
-  /**
-   * Defaults to:
-   * includeErrorData: true
-   * includeErrorStack: true
    */
-  inspectOptions?: InspectAnyOptions
+  throwOnError?: boolean
 }
 
 export interface SlackAttachmentField {
@@ -120,16 +106,23 @@ export interface SlackServiceCfg<CTX = any> {
 
   defaults?: Partial<SlackMessage>
 
-  /**
-   * Override channel when msg.level is set.
-   * key: DebugLogLevel
-   * value: channel name to send message to
-   */
-  channelByLevel?: StringMap
-
   /**
    * Function to return an array of "prefix tokens" (will be joined by ': ').
    * Allows to skip (filter out) the message by returning `null`.
    */
   messagePrefixHook: SlackMessagePrefixHook<CTX>
+
+  /**
+   * By default SlackService logs every message to console.log
+   * Pass another logger if needed.
+   * Pass `noopLogger` to suppress logging completely.
+   */
+  logger: CommonLogger
+
+  /**
+   * Defaults to:
+   * includeErrorData: true
+   * includeErrorStack: true
+   */
+  inspectOptions: InspectAnyOptions
 }

package/src/slack/slack.service.ts

@@ -1,7 +1,7 @@
-import { StringMap } from '@naturalcycles/js-lib'
+import { _omit, StringMap } from '@naturalcycles/js-lib'
 import { dayjs } from '@naturalcycles/time-lib'
 import got from 'got'
-import { Debug, DebugLogLevel, inspectAny, InspectAnyOptions } from '..'
+import { inspectAny, InspectAnyOptions } from '..'
 import {
   SlackApiBody,
   SlackAttachmentField,
@@ -11,12 +11,12 @@ import {
 
 const GAE = !!process.env['GAE_INSTANCE']
 
-const DEFAULTS = (): SlackMessage => ({
+const DEFAULTS: SlackMessage = {
   username: 'bot',
   channel: '#log',
   icon_emoji: ':spider_web:',
   items: 'no text',
-})
+}
 
 const INSPECT_OPT: InspectAnyOptions = {
   colors: false,
@@ -24,8 +24,6 @@ const INSPECT_OPT: InspectAnyOptions = {
   includeErrorStack: true,
 }
 
-const log = Debug('nc:nodejs-lib:slack')
-
 /**
  * Has 2 main methods:
  *
@@ -42,7 +40,12 @@ export class SlackService<CTX = any> {
   constructor(cfg: Partial<SlackServiceCfg<CTX>>) {
     this.cfg = {
       messagePrefixHook: slackDefaultMessagePrefixHook,
+      logger: console,
       ...cfg,
+      inspectOptions: {
+        ...INSPECT_OPT,
+        ...cfg.inspectOptions,
+      },
     }
   }
 
@@ -58,47 +61,34 @@ export class SlackService<CTX = any> {
     })
   }
 
-  async send(msg: SlackMessage<CTX> | string, ctx?: CTX): Promise<void> {
-    const { webhookUrl, messagePrefixHook } = this.cfg
+  async send(input: SlackMessage<CTX> | string, ctx?: CTX): Promise<void> {
+    const { webhookUrl, messagePrefixHook, inspectOptions } = this.cfg
 
     // If String is passed as first argument - just transform it to a full SlackMessage
-    if (typeof msg === 'string') {
-      msg = {
-        items: msg,
-      }
-    }
+    const msg = typeof input === 'string' ? { items: input } : input
 
     if (ctx !== undefined) {
       Object.assign(msg, { ctx })
     }
 
-    if (!msg.noLog) {
-      log[msg.level || DebugLogLevel.info](
-        ...[msg.items, msg.kv, msg.attachments, msg.mentions].filter(Boolean),
-      )
-    }
+    this.cfg.logger.log(...[msg.items, msg.kv, msg.attachments, msg.mentions].filter(Boolean))
 
     if (!webhookUrl) return
 
     // Transform msg.kv into msg.attachments
     if (msg.kv) {
-      msg.attachments = [...(msg.attachments || []), { fields: this.kvToFields(msg.kv) }]
+      ;(msg.attachments ||= []).push({ fields: this.kvToFields(msg.kv) })
 
       delete msg.kv // to not pass it all the way to Slack Api
     }
 
     let text: string
 
-    const inspectOpt = {
-      ...INSPECT_OPT,
-      ...msg.inspectOptions,
-    }
-
     // Array has a special treatment here
     if (Array.isArray(msg.items)) {
-      text = msg.items.map(t => inspectAny(t, inspectOpt)).join('\n')
+      text = msg.items.map(t => inspectAny(t, inspectOptions)).join('\n')
     } else {
-      text = inspectAny(msg.items, inspectOpt)
+      text = inspectAny(msg.items, inspectOptions)
     }
 
     // Wrap in markdown-text-block if it's anything but plain String
@@ -113,29 +103,26 @@ export class SlackService<CTX = any> {
     const prefix = await messagePrefixHook(msg)
     if (prefix === null) return // filtered out!
 
-    const json: SlackApiBody = {
-      ...DEFAULTS(),
-      ...this.cfg.defaults,
-      ...msg,
-      // Text with Prefix
-      text: [prefix.join(': '), text].filter(Boolean).join('\n'),
-    }
-
-    // they're not needed in the json payload
-    delete json['items']
-    delete json['ctx']
-    delete json['noLog']
-
-    json.channel = (this.cfg.channelByLevel || {})[msg.level!] || json.channel
+    const json: SlackApiBody = _omit(
+      {
+        ...DEFAULTS,
+        ...this.cfg.defaults,
+        ...msg,
+        // Text with Prefix
+        text: [prefix.join(': '), text].filter(Boolean).join('\n'),
+      },
+      ['items', 'ctx'],
+    )
 
     await got
       .post(webhookUrl, {
         json,
         responseType: 'text',
+        timeout: 90_000,
       })
       .catch(err => {
         // ignore (unless throwOnError is set)
-        if ((msg as SlackMessage).throwOnError) throw err
+        if (msg.throwOnError) throw err
       })
   }
 

package/src/stream/transform/transformLogProgress.ts

@@ -104,7 +104,7 @@ export interface TransformLogProgressOptions<IN = any> extends TransformOptions
 }
 
 const inspectOpt: InspectOptions = {
-  colors: hasColors(),
+  colors: hasColors,
   breakLength: 300,
 }
 
@@ -163,16 +163,15 @@ export function transformLogProgress<IN = any>(
     const mem = process.memoryUsage()
 
     const now = Date.now()
+    const batchedProgress = progress * batchSize
     const lastRPS = (processedLastSecond * batchSize) / ((now - lastSecondStarted) / 1000) || 0
-    const rpsTotal = Math.round((progress * batchSize) / ((now - started) / 1000)) || 0
+    const rpsTotal = Math.round(batchedProgress / ((now - started) / 1000)) || 0
     lastSecondStarted = now
     processedLastSecond = 0
 
     const rps10 = Math.round(sma.push(lastRPS))
     if (mem.rss > peakRSS) peakRSS = mem.rss
 
-    const batchedProgress = progress * batchSize
-
     console.log(
       inspect(
         {
@@ -197,7 +196,8 @@ export function transformLogProgress<IN = any>(
     )
 
     if (tenx) {
-      let perHour: number | string = Math.round((progress * 1000 * 60 * 60) / (now - started)) || 0
+      let perHour: number | string =
+        Math.round((batchedProgress * 1000 * 60 * 60) / (now - started)) || 0
       if (perHour > 900) {
         perHour = Math.round(perHour / 1000) + 'K'
       }
@@ -205,12 +205,12 @@ export function transformLogProgress<IN = any>(
       console.log(
         `${dimGrey(dayjs().toPretty())} ${white(metric)} took ${yellow(
           _since(started),
-        )} so far to process ${yellow(progress)} rows, ~${yellow(perHour)}/hour`,
+        )} so far to process ${yellow(batchedProgress)} rows, ~${yellow(perHour)}/hour`,
       )
     } else if (final) {
       console.log(
         `${boldWhite(metric)} took ${yellow(_since(started))} to process ${yellow(
-          progress,
+          batchedProgress,
         )} rows with total RPS of ${yellow(rpsTotal)}`,
       )
     }

package/src/validation/ajv/ajvValidationError.ts

@@ -10,20 +10,5 @@ export interface AjvValidationErrorData extends ErrorData {
 export class AjvValidationError extends AppError<AjvValidationErrorData> {
   constructor(message: string, data: AjvValidationErrorData) {
     super(message, data)
-
-    this.constructor = AjvValidationError
-    ;(this as any).__proto__ = AjvValidationError.prototype
-    Object.defineProperty(this, 'name', {
-      value: this.constructor.name,
-      configurable: true,
-    })
-
-    if (Error.captureStackTrace) {
-      Error.captureStackTrace(this, this.constructor)
-    } else {
-      Object.defineProperty(this, 'stack', {
-        value: new Error().stack, //  eslint-disable-line unicorn/error-message
-      })
-    }
   }
 }

package/src/validation/joi/joi.validation.error.ts

@@ -28,20 +28,5 @@ export interface JoiValidationErrorData extends ErrorData {
 export class JoiValidationError extends AppError<JoiValidationErrorData> {
   constructor(message: string, data: JoiValidationErrorData) {
     super(message, data)
-
-    this.constructor = JoiValidationError
-    ;(this as any).__proto__ = JoiValidationError.prototype
-    Object.defineProperty(this, 'name', {
-      value: this.constructor.name,
-      configurable: true,
-    })
-
-    if (Error.captureStackTrace) {
-      Error.captureStackTrace(this, this.constructor)
-    } else {
-      Object.defineProperty(this, 'stack', {
-        value: new Error().stack, //  eslint-disable-line unicorn/error-message
-      })
-    }
   }
 }

package/src/validation/joi/joi.validation.util.ts

@@ -172,6 +172,7 @@ function createError(value: any, err: ValidationError, objectName?: string): Joi
   // Make annotation non-enumerable, to not get it automatically printed,
   // but still accessible
   Object.defineProperty(data, 'annotation', {
+    writable: true,
     configurable: true,
     enumerable: false,
     value: annotation,