@naturalcycles/js-lib 14.219.0 → 14.219.2

package/dist/promise/pMap.d.ts

@@ -4,11 +4,7 @@ export interface PMapOptions {
     /**
      * Number of concurrently pending promises returned by `mapper`.
      *
-     * Defaults to 16.
-     *
-     * It previously (and originally) defaulted to Infinity, which was later changed,
-     * because it's somewhat dangerous to run "infinite number of parallel promises".
-     * You can still emulate the old behavior by passing `Infinity`.
+     * Defaults to Infitity.
      */
     concurrency?: number;
     /**

package/dist/promise/pMap.js

@@ -41,14 +41,14 @@ async function pMap(iterable, mapper, opt = {}) {
     const itemsLength = items.length;
     if (itemsLength === 0)
         return []; // short circuit
-    const { concurrency = 16, errorMode = __1.ErrorMode.THROW_IMMEDIATELY, logger = console } = opt;
+    const { concurrency = Infinity, errorMode = __1.ErrorMode.THROW_IMMEDIATELY, logger = console } = opt;
     // Special cases that are able to preserve async stack traces
     // Special case: serial execution
     if (concurrency === 1) {
         return await pMap1(items, mapper, errorMode, logger);
     }
-    // Special case: concurrency === Infinity or items.length <= concurrency
-    if (concurrency === Infinity || items.length <= concurrency) {
+    // Special case: items.length <= concurrency (including when concurrency === Infinity)
+    if (items.length <= concurrency) {
         return await pMapAll(items, mapper, errorMode, logger);
     }
     // General case: execution with throttled concurrency

package/dist/string/string.util.js

@@ -71,12 +71,12 @@ function _substringBeforeLast(s, delimiter) {
 exports._substringBeforeLast = _substringBeforeLast;
 function _substringAfter(s, delimiter) {
     const pos = s.indexOf(delimiter);
-    return pos !== -1 ? s.slice(pos + 1) : s;
+    return pos !== -1 ? s.slice(pos + delimiter.length) : s;
 }
 exports._substringAfter = _substringAfter;
 function _substringAfterLast(s, delimiter) {
     const pos = s.lastIndexOf(delimiter);
-    return pos !== -1 ? s.slice(pos + 1) : s;
+    return pos !== -1 ? s.slice(pos + delimiter.length) : s;
 }
 exports._substringAfterLast = _substringAfterLast;
 /**

package/dist-esm/promise/pMap.js

@@ -38,14 +38,14 @@ export async function pMap(iterable, mapper, opt = {}) {
     const itemsLength = items.length;
     if (itemsLength === 0)
         return []; // short circuit
-    const { concurrency = 16, errorMode = ErrorMode.THROW_IMMEDIATELY, logger = console } = opt;
+    const { concurrency = Infinity, errorMode = ErrorMode.THROW_IMMEDIATELY, logger = console } = opt;
     // Special cases that are able to preserve async stack traces
     // Special case: serial execution
     if (concurrency === 1) {
         return await pMap1(items, mapper, errorMode, logger);
     }
-    // Special case: concurrency === Infinity or items.length <= concurrency
-    if (concurrency === Infinity || items.length <= concurrency) {
+    // Special case: items.length <= concurrency (including when concurrency === Infinity)
+    if (items.length <= concurrency) {
         return await pMapAll(items, mapper, errorMode, logger);
     }
     // General case: execution with throttled concurrency

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

@@ -59,11 +59,11 @@ export function _substringBeforeLast(s, delimiter) {
 }
 export function _substringAfter(s, delimiter) {
     const pos = s.indexOf(delimiter);
-    return pos !== -1 ? s.slice(pos + 1) : s;
+    return pos !== -1 ? s.slice(pos + delimiter.length) : s;
 }
 export function _substringAfterLast(s, delimiter) {
     const pos = s.lastIndexOf(delimiter);
-    return pos !== -1 ? s.slice(pos + 1) : s;
+    return pos !== -1 ? s.slice(pos + delimiter.length) : s;
 }
 /**
  * Returns the substring between LAST `leftDelimiter` and then FIRST `rightDelimiter`.

package/package.json

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

package/src/promise/pMap.ts

@@ -5,11 +5,7 @@ export interface PMapOptions {
   /**
    * Number of concurrently pending promises returned by `mapper`.
    *
-   * Defaults to 16.
-   *
-   * It previously (and originally) defaulted to Infinity, which was later changed,
-   * because it's somewhat dangerous to run "infinite number of parallel promises".
-   * You can still emulate the old behavior by passing `Infinity`.
+   * Defaults to Infitity.
    */
   concurrency?: number
 
@@ -73,7 +69,7 @@ export async function pMap<IN, OUT>(
   const itemsLength = items.length
   if (itemsLength === 0) return [] // short circuit
 
-  const { concurrency = 16, errorMode = ErrorMode.THROW_IMMEDIATELY, logger = console } = opt
+  const { concurrency = Infinity, errorMode = ErrorMode.THROW_IMMEDIATELY, logger = console } = opt
 
   // Special cases that are able to preserve async stack traces
   // Special case: serial execution
@@ -81,8 +77,8 @@ export async function pMap<IN, OUT>(
     return await pMap1(items, mapper, errorMode, logger)
   }
 
-  // Special case: concurrency === Infinity or items.length <= concurrency
-  if (concurrency === Infinity || items.length <= concurrency) {
+  // Special case: items.length <= concurrency (including when concurrency === Infinity)
+  if (items.length <= concurrency) {
     return await pMapAll(items, mapper, errorMode, logger)
   }
 

package/src/string/string.util.ts

@@ -65,12 +65,12 @@ export function _substringBeforeLast(s: string, delimiter: string): string {
 
 export function _substringAfter(s: string, delimiter: string): string {
   const pos = s.indexOf(delimiter)
-  return pos !== -1 ? s.slice(pos + 1) : s
+  return pos !== -1 ? s.slice(pos + delimiter.length) : s
 }
 
 export function _substringAfterLast(s: string, delimiter: string): string {
   const pos = s.lastIndexOf(delimiter)
-  return pos !== -1 ? s.slice(pos + 1) : s
+  return pos !== -1 ? s.slice(pos + delimiter.length) : s
 }
 
 /**