@nxtedition/scheduler 1.0.6 → 1.0.8

package/lib/index.d.ts

@@ -8,5 +8,5 @@ export declare class Scheduler {
     constructor(opts?: SharedArrayBuffer | {
         concurrency?: number;
     });
-    schedule(fn: (next: Function) => any, priority?: 0 | 1 | 2 | "low" | "normal" | "high"): any;
+    schedule(fn: (next: Function) => any, priority?: 0 | 1 | 2 | 'low' | 'normal' | 'high'): any;
 }

package/lib/index.js

@@ -15,6 +15,7 @@ export class Scheduler {
   #running = 0
   #pending = 0
   #counter = 0
+  #actived = false
 
   #lowQueue = new FixedQueue()
   #normalQueue = new FixedQueue()
@@ -43,7 +44,10 @@ export class Scheduler {
     }
   }
 
-  schedule(fn                         , priority                                        = Scheduler.NORMAL) {
+  schedule(
+    fn                         ,
+    priority                                        = Scheduler.NORMAL,
+  ) {
     if (typeof fn !== 'function') {
       throw new TypeError('First argument must be a function')
     }
@@ -87,32 +91,41 @@ export class Scheduler {
   }
 
   #next = () => {
-    let running = this.#stateView
-      ? Atomics.sub(this.#stateView, Scheduler.#RUNNING_INDEX, 1) - 1
-      : this.#running - 1
-    this.#running -= 1
-
-    while (this.#pending > 0 && (this.#running < 1 || running < this.#concurrency)) {
-      const counter = this.#stateView
-        ? Atomics.add(this.#stateView, Scheduler.#COUNTER_INDEX, 1)
-        : this.#counter++
-      const fn =
-        ((counter & 63) === 0 && this.#lowQueue.shift()) ??
-        ((counter & 15) === 0 && this.#normalQueue.shift()) ??
-        this.#highQueue.shift() ??
-        this.#normalQueue.shift() ??
-        this.#lowQueue.shift()
-
-      if (!fn) {
-        break
+    try {
+      let running = this.#stateView
+        ? Atomics.sub(this.#stateView, Scheduler.#RUNNING_INDEX, 1) - 1
+        : this.#running - 1
+      this.#running -= 1
+
+      if (this.#actived) {
+        return
       }
 
-      running = this.#stateView
-        ? Atomics.add(this.#stateView, Scheduler.#RUNNING_INDEX, 1) + 1
-        : this.#running + 1
-      this.#running += 1
-      this.#pending -= 1
-      fn(this.#next)
+      this.#actived = true
+      while (this.#pending > 0 && (this.#running < 1 || running < this.#concurrency)) {
+        const counter = this.#stateView
+          ? Atomics.add(this.#stateView, Scheduler.#COUNTER_INDEX, 1)
+          : this.#counter++
+        const fn =
+          ((counter & 63) === 0 && this.#lowQueue.shift()) ||
+          ((counter & 15) === 0 && this.#normalQueue.shift()) ||
+          this.#highQueue.shift() ||
+          this.#normalQueue.shift() ||
+          this.#lowQueue.shift()
+
+        running = this.#stateView
+          ? Atomics.add(this.#stateView, Scheduler.#RUNNING_INDEX, 1) + 1
+          : this.#running + 1
+        this.#running += 1
+        this.#pending -= 1
+        fn(this.#next)
+      }
+      this.#actived = false
+    } catch (err) {
+      // Throwing here is undefined behavior...
+      queueMicrotask(() => {
+        throw new Error('Scheduler task error', { cause: err })
+      })
     }
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nxtedition/scheduler",
-  "version": "1.0.6",
+  "version": "1.0.8",
   "type": "module",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",
@@ -20,5 +20,5 @@
     "rimraf": "^6.1.2",
     "typescript": "^5.9.3"
   },
-  "gitHead": "1c0ae17fb7bb015b6f352befd2c629bc5ad0ea64"
+  "gitHead": "ba7f0417c8dbf2a8d6e38f5848f5e8ec162f4eb7"
 }