fix: use strict object equality in tasks stealing code

jerome-benoit · jerome-benoit · commit f2fd2ff6d2a6 · 2025-08-26T16:09:28.000+02:00
Signed-off-by: Jérôme Benoit &lt;jerome.benoit@piment-noir.org&gt;
diff --git a/src/pools/abstract-pool.ts b/src/pools/abstract-pool.ts
@@ -2085,8 +2085,8 @@ export abstract class AbstractPool<
       return
     }
     const { workerId } = event.detail
-    const sourceWorkerNode =
-      this.workerNodes[this.getWorkerNodeKeyByWorkerId(workerId)]
+    const sourceWorkerNodeKey = this.getWorkerNodeKeyByWorkerId(workerId)
+    const sourceWorkerNode = this.workerNodes[sourceWorkerNodeKey]
     if (sourceWorkerNode == null) {
       return
     }
@@ -2101,7 +2101,7 @@ export abstract class AbstractPool<
         break
       }
       if (
-        workerNode.info.id !== workerId &&
+        workerNode !== sourceWorkerNode &&
         !workerNode.info.backPressureStealing &&
         workerNode.usage.tasks.queued <
           this.opts.tasksQueueOptions!.size! - sizeOffset

Original file line number	Diff line number	Diff line change
`@@ -2085,8 +2085,8 @@ export abstract class AbstractPool<`
`2085`	`2085`	`return`
`2086`	`2086`	`}`
`2087`	`2087`	`const { workerId } = event.detail`
`2088`		`- const sourceWorkerNode =`
`2089`		`- this.workerNodes[this.getWorkerNodeKeyByWorkerId(workerId)]`
	`2088`	`+ const sourceWorkerNodeKey = this.getWorkerNodeKeyByWorkerId(workerId)`
	`2089`	`+ const sourceWorkerNode = this.workerNodes[sourceWorkerNodeKey]`
`2090`	`2090`	`if (sourceWorkerNode == null) {`
`2091`	`2091`	`return`
`2092`	`2092`	`}`
`@@ -2101,7 +2101,7 @@ export abstract class AbstractPool<`
`2101`	`2101`	`break`
`2102`	`2102`	`}`
`2103`	`2103`	`if (`
`2104`		`- workerNode.info.id !== workerId &&`
	`2104`	`+ workerNode !== sourceWorkerNode &&`
`2105`	`2105`	`!workerNode.info.backPressureStealing &&`
`2106`	`2106`	`workerNode.usage.tasks.queued <`
`2107`	`2107`	`this.opts.tasksQueueOptions!.size! - sizeOffset`