@nxtedition/scheduler 1.0.5 → 1.0.7

package/lib/index.d.ts

@@ -8,5 +8,5 @@ export declare class Scheduler {
     constructor(opts?: SharedArrayBuffer | {
         concurrency?: number;
     });
-    schedule(fn: (next: Function) => any, priority?: 1): 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,16 +44,24 @@ 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')
     }
 
     if (priority == null) {
       priority = Scheduler.NORMAL
-    }
-
-    if (!Number.isInteger(priority)) {
+    } else if (typeof priority === 'string') {
+      if (priority === 'low') {
+        priority = Scheduler.LOW
+      } else if (priority === 'normal') {
+        priority = Scheduler.NORMAL
+      } else if (priority === 'high') {
+        priority = Scheduler.HIGH
+      } else {
+        throw new Error('Invalid priority')
+      }
+    } else if (!Number.isInteger(priority)) {
       throw new Error('Invalid priority')
     }
 
@@ -79,32 +88,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.5",
+  "version": "1.0.7",
   "type": "module",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",
@@ -20,5 +20,5 @@
     "rimraf": "^6.1.2",
     "typescript": "^5.9.3"
   },
-  "gitHead": "8317f7c58c48ff5a16b3380f3b9f4c87c56e8500"
+  "gitHead": "8a7862ed029e6ec42bbccd925bf638e7b069cc5e"
 }