From: Jérôme Benoit <jerome.benoit@piment-noir.org>
Date: Tue, 26 Aug 2025 14:31:18 +0000 (+0200)
Subject: fix: use strict object equality in tasks stealing code
X-Git-Url: https://git.piment-noir.org/?a=commitdiff_plain;h=eec09324ae02df9fe24c12c39cb2c7e802708c90;p=poolifier.git

fix: use strict object equality in tasks stealing code

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>
---

diff --git a/src/pools/abstract-pool.ts b/src/pools/abstract-pool.ts
index 3a1fb9fd9..6da9d1627 100644
--- a/src/pools/abstract-pool.ts
+++ b/src/pools/abstract-pool.ts
@@ -1877,8 +1877,8 @@ export abstract class AbstractPool<
       return
     }
     const { workerId } = eventDetail
-    const sourceWorkerNode =
-      this.workerNodes[this.getWorkerNodeKeyByWorkerId(workerId)]
+    const sourceWorkerNodeKey = this.getWorkerNodeKeyByWorkerId(workerId)
+    const sourceWorkerNode = this.workerNodes[sourceWorkerNodeKey]
     // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
     if (sourceWorkerNode == null) {
       return
@@ -1894,7 +1894,7 @@ export abstract class AbstractPool<
         break
       }
       if (
-        workerNode.info.id !== workerId &&
+        workerNode !== sourceWorkerNode &&
         !workerNode.info.backPressureStealing &&
         workerNode.usage.tasks.queued <
           // eslint-disable-next-line @typescript-eslint/no-non-null-assertion