@nxtedition/scheduler 3.0.6 → 3.0.9

package/lib/index.d.ts

@@ -1,14 +1,14 @@
 export type Priority = -3 | -2 | -1 | 0 | 1 | 2 | 3 | 'lowest' | 'lower' | 'low' | 'normal' | 'high' | 'higher' | 'highest';
-export declare function parsePriority(p: string | number): number;
+export declare function parsePriority(p: string | number): -3 | -2 | -1 | 0 | 1 | 2 | 3;
 export declare class Scheduler {
     #private;
-    static LOWEST: Priority;
-    static LOWER: Priority;
-    static LOW: Priority;
-    static NORMAL: Priority;
-    static HIGH: Priority;
-    static HIGHER: Priority;
-    static HIGHEST: Priority;
+    static readonly LOWEST = -3;
+    static readonly LOWER = -2;
+    static readonly LOW = -1;
+    static readonly NORMAL = 0;
+    static readonly HIGH = 1;
+    static readonly HIGHER = 2;
+    static readonly HIGHEST = 3;
     get concurrency(): number;
     get stats(): {
         deferred: number;

package/lib/index.js

@@ -25,7 +25,7 @@ class FastQueue {
             
              
 
-export function parsePriority(p                 )         {
+export function parsePriority(p                 )                               {
   if (typeof p === 'number') {
     // Do nothing...
   } else if (p === 'lowest') {
@@ -53,18 +53,18 @@ export function parsePriority(p                 )         {
   } else if (p > 3) {
     return 3
   } else {
-    return Math.trunc(p)
+    return Math.trunc(p)                                
   }
 }
 
 export class Scheduler {
-  static LOWEST           = -3
-  static LOWER           = -2
-  static LOW           = -1
-  static NORMAL           = 0
-  static HIGH           = 1
-  static HIGHER           = 2
-  static HIGHEST           = 3
+  static          LOWEST = -3
+  static          LOWER = -2
+  static          LOW = -1
+  static          NORMAL = 0
+  static          HIGH = 1
+  static          HIGHER = 2
+  static          HIGHEST = 3
 
   #concurrency
   #stateView
@@ -104,14 +104,17 @@ export class Scheduler {
     }
     const stateBuffer = new SharedArrayBuffer(64)
     const stateView = new Int32Array(stateBuffer)
-    Atomics.store(stateView, CONCURRENCY_INDEX, concurrency ?? 0)
+    Atomics.store(stateView, CONCURRENCY_INDEX, concurrency ?? -1)
     return stateBuffer
   }
 
   constructor(opts                                              ) {
     if (opts instanceof SharedArrayBuffer) {
       this.#stateView = new Int32Array(opts)
-      this.#concurrency = Atomics.load(this.#stateView, CONCURRENCY_INDEX) || Infinity
+      this.#concurrency = Atomics.load(this.#stateView, CONCURRENCY_INDEX)
+      if (this.#concurrency === -1) {
+        this.#concurrency = Infinity
+      }
     } else {
       this.#concurrency = opts?.concurrency ?? Infinity
     }
@@ -119,7 +122,7 @@ export class Scheduler {
 
   run      (
     fn                                ,
-    priority = Scheduler.NORMAL,
+    priority           = Scheduler.NORMAL,
     opaque    ,
   )             {
     return new Promise   ((resolve, reject) => {
@@ -153,9 +156,9 @@ export class Scheduler {
         return
       }
 
-      if (Atomics.add(this.#stateView, RUNNING_INDEX, 1) >= this.#concurrency) {
-        Atomics.sub(this.#stateView, RUNNING_INDEX, 1)
-      } else {
+      // We use non atomic access here as an optimization and treat the concurrency limit as a soft limit.
+      if (this.#stateView[RUNNING_INDEX] < this.#concurrency) {
+        Atomics.add(this.#stateView, RUNNING_INDEX, 1)
         this.#running += 1
         fn(opaque)
         return

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nxtedition/scheduler",
-  "version": "3.0.6",
+  "version": "3.0.9",
   "type": "module",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",
@@ -28,5 +28,5 @@
     "rimraf": "^6.1.2",
     "typescript": "^5.9.3"
   },
-  "gitHead": "80ace3245dc851f38a288b0c3b167ca0024f0469"
+  "gitHead": "64c1bd97cf9a785ca4a750b5380418878cbed2d2"
 }