@nxtedition/timers 1.1.1 → 1.1.3

package/README.md

@@ -86,7 +86,7 @@ Cancel a pending timer. Works with both native and pooled handles. Passing `null
 
 ## How it works
 
-All pooled timers share a single `setTimeout` handle that fires every 500ms. On each tick, pending timers are checked and expired ones are fired. This means pooled timers have at most ~500ms of jitter but dramatically reduce the number of active OS timer handles when many long-lived timers are in use (e.g. connection idle timeouts, retry delays).
+All pooled timers share a single `setTimeout` handle that fires every 500ms. On each tick, pending timers are checked and expired ones are fired. A +250ms offset is applied to each deadline to center the quantisation error, so pooled timers fire within ±250ms of the requested delay. This dramatically reduces the number of active OS timer handles when many long-lived timers are in use (e.g. connection idle timeouts, retry delays).
 
 **Note:** Timers with the same delay are not guaranteed to fire in creation order. The execution order of same-delay timers is nondeterministic.
 

package/lib/index.js

@@ -12,11 +12,18 @@ function dispatch() {
   while (fastIndex < len) {
     const timer = fastTimers[fastIndex]
 
-    if (timer.state === 0) {
-      timer.state = fastNow + timer.delay
-    } else if (timer.state > 0 && fastNow >= timer.state) {
+    if (timer.state > 0 && fastNow >= timer.state) {
       timer.state = -1
-      timer.callback(timer.opaque)
+      try {
+        timer.callback(timer.opaque)
+      } catch (err) {
+        // Isolate the throw: rethrow asynchronously so dispatch continues.
+        // Otherwise fastIndex stays mid-loop and every prior timer gets
+        // delayed by a full tick on the next onTimeout().
+        queueMicrotask(() => {
+          throw err
+        })
+      }
     }
 
     if (timer.state === -1) {
@@ -40,6 +47,9 @@ function dispatch() {
 
   if (fastTimers.length > 0) {
     refreshTimeout()
+  } else if (fastNowTimeout) {
+    globalThis.clearTimeout(fastNowTimeout)
+    fastNowTimeout = null
   }
 }
 
@@ -64,7 +74,6 @@ class Timeout {
 
   //  -2 not in timer list
   //  -1 in timer list but inactive
-  //   0 in timer list waiting for time
   // > 0 in timer list waiting for time to expire
   state         = -2
 
@@ -83,7 +92,11 @@ class Timeout {
       }
     }
 
-    this.state = 0
+    // Offset by half the tick interval to center the timing error.
+    // fastNow only advances every 500ms so the actual fire time can
+    // deviate from the requested delay. Without this offset the error
+    // is up to -500ms (early); with it the worst case is ±250ms.
+    this.state = fastNow + this.delay + 250
   }
 
   clear()       {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nxtedition/timers",
-  "version": "1.1.1",
+  "version": "1.1.3",
   "type": "module",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",
@@ -22,7 +22,9 @@
     "benchmark": "node --experimental-strip-types src/index.bench.ts"
   },
   "devDependencies": {
+    "@sinonjs/fake-timers": "^15.3.2",
     "@types/node": "^25.5.0",
+    "@types/sinonjs__fake-timers": "^15.0.1",
     "amaroc": "^1.0.1",
     "mitata": "^1.0.34",
     "oxlint-tsgolint": "^0.17.0",
@@ -32,5 +34,5 @@
   "dependencies": {
     "@nxtedition/yield": "^1.0.16"
   },
-  "gitHead": "a95ef1b72677b853fd7943f7071c266f1789e134"
+  "gitHead": "472a4cdfd11ff9f1a7dcf8b81599bc1d9270bc90"
 }

package/lib/index.bench.d.ts

@@ -1 +0,0 @@
-export {};