@naturalcycles/js-lib 15.89.0 → 15.91.0

package/src/promise/pMap.ts

@@ -1,4 +1,5 @@
 import { ErrorMode } from '../error/errorMode.js'
+import { consoleLogger } from '../log/commonLogger.js'
 import type { CommonLogger } from '../log/commonLogger.js'
 import { END, SKIP } from '../types.js'
 import type { AbortableAsyncMapper } from '../types.js'
@@ -17,12 +18,12 @@ export interface PMapOptions {
    * When set to THROW_AGGREGATED, instead of stopping when a promise rejects, it will wait for all the promises to settle and then reject with an Aggregated error.
    *
    * When set to SUPPRESS - will ignore errors and return results from successful operations.
-   * When in SUPPRESS - errors are still logged via the `logger` (which defaults to console).
+   * When in SUPPRESS - errors are still logged via the `logger` (which defaults to consoleLogger).
    */
   errorMode?: ErrorMode
 
   /**
-   * Default to `console`.
+   * Default to `consoleLogger`.
    * Pass `null` to skip logging completely.
    */
   logger?: CommonLogger | null
@@ -71,7 +72,11 @@ export async function pMap<IN, OUT>(
   const itemsLength = items.length
   if (itemsLength === 0) return [] // short circuit
 
-  const { concurrency = Infinity, errorMode = ErrorMode.THROW_IMMEDIATELY, logger = console } = opt
+  const {
+    concurrency = Infinity,
+    errorMode = ErrorMode.THROW_IMMEDIATELY,
+    logger = consoleLogger,
+  } = opt
 
   // Special cases that are able to preserve async stack traces
   // Special case: serial execution
@@ -181,7 +186,7 @@ async function pMap1<IN, OUT>(
         errors.push(err as Error)
       } else {
         // otherwise, suppress (but still log via logger)
-        logger?.error(err)
+        logger?.error({ err })
       }
     }
   }
@@ -219,7 +224,7 @@ async function pMapAll<IN, OUT>(
       errors.push(r.reason)
     } else {
       // otherwise, suppress (but still log via logger)
-      logger?.error(r.reason)
+      logger?.error({ err: r.reason })
     }
   }
 

package/src/promise/pQueue.ts

@@ -18,7 +18,7 @@ export interface PQueueCfg {
   errorMode?: ErrorMode
 
   /**
-   * Default to `console`
+   * Default to `consoleLogger`
    */
   logger?: CommonLogger
 

package/src/promise/pRetry.ts

@@ -1,6 +1,7 @@
 import { _since } from '../datetime/time.util.js'
 import type { ErrorData } from '../error/error.model.js'
 import { _errorDataAppend } from '../error/error.util.js'
+import { consoleLogger } from '../log/commonLogger.js'
 import type { CommonLogger } from '../log/commonLogger.js'
 import type { AnyAsyncFunction, UnixTimestampMillis } from '../types.js'
 import { pDelay } from './pDelay.js'
@@ -84,7 +85,7 @@ export interface PRetryOptions {
   logNone?: boolean
 
   /**
-   * Default to `console`
+   * Default to `consoleLogger`
    */
   logger?: CommonLogger
 
@@ -113,7 +114,7 @@ export async function pRetry<T>(
     delay: initialDelay = 1000,
     delayMultiplier = 2,
     predicate,
-    logger = console,
+    logger = consoleLogger,
     name,
     timeout,
   } = opt
@@ -164,7 +165,7 @@ export async function pRetry<T>(
       if (logFailures) {
         // Logger at warn (not error) level, because it's not a fatal error, but a retriable one
         // Fatal one is not logged either, because it's been thrown instead
-        logger.warn(`${fname} attempt #${attempt} error in ${_since(started)}:`, err)
+        logger.warn({ msg: `${fname} attempt #${attempt} error in ${_since(started)}`, err })
       }
 
       if (attempt >= maxAttempts || (predicate && !predicate(err as Error, attempt, maxAttempts))) {