@nxtedition/lib 23.9.12 → 23.9.14

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nxtedition/lib",
-  "version": "23.9.12",
+  "version": "23.9.14",
   "license": "MIT",
   "author": "Robert Nagy <robert.nagy@boffins.se>",
   "type": "module",

package/yield.js

@@ -1,16 +1,36 @@
+import { AbortError } from './errors.js'
+
 let yieldTime = performance.now()
 
-export function maybeYield(opts) {
+export function maybeYield(callback, opaque, opts) {
+  if (callback != null && typeof callback !== 'function') {
+    throw new TypeError('callback must be a function')
+  }
+
   const timeout = opts?.timeout ?? 50
+
+  if (!Number.isFinite(timeout) || timeout < 0) {
+    throw new TypeError('timeout must be a finite non-negative number')
+  }
+
   if (performance.now() - yieldTime > timeout) {
-    return doYield(opts)
+    doYield(opts, callback, opaque)
+  } else {
+    callback(opts?.signal?.aborted ? (opts.signal.reason ?? new AbortError()) : null, opaque)
   }
-  opts?.signal?.throwIfAborted()
 }
 
-export async function doYield(opts) {
-  await new Promise((resolve) => setImmediate(resolve))
-  resetYield(opts)
+export async function doYield(callback, opaque, opts) {
+  if (callback != null && typeof callback !== 'function') {
+    throw new TypeError('callback must be a function')
+  }
+
+  // TODO (fix): Use a yield queue?
+
+  setImmediate(() => {
+    resetYield(opts)
+    callback(opts?.signal?.aborted ? (opts.signal.reason ?? new AbortError()) : null, opaque)
+  })
 }
 
 function resetYield(opts) {
@@ -21,3 +41,12 @@ function resetYield(opts) {
 setInterval(() => {
   yieldTime = performance.now()
 }, 500).unref()
+
+export const promises = {
+  maybeYield(opts, opaque) {
+    return new Promise((resolve, reject) =>
+      maybeYield((err, val) => (err ? reject(err) : resolve(val)), opaque, opts),
+    )
+  },
+  resetYield,
+}