stream: fix enqueue race condition on esm modules

stream: use nextTick on close
PR-URL: #40901
Reviewed-By: Robert Nagy <ronagy@icloud.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>

Author: RafaelGSS

lib/internal/webstreams/readablestream.js

@@ -1446,13 +1446,18 @@ function readableStreamTee(stream, cloneForBranch2) {
 });
 },
 [kClose](){
-reading=false;
-if(!canceled1)
-readableStreamDefaultControllerClose(branch1[kState].controller);
-if(!canceled2)
-readableStreamDefaultControllerClose(branch2[kState].controller);
-if(!canceled1||!canceled2)
-cancelPromise.resolve();
+// The `process.nextTick()` is not part of the spec.
+// This approach was needed to avoid a race condition working with esm
+// Further information, see: https://github.com/nodejs/node/issues/39758
+process.nextTick(()=>{
+reading=false;
+if(!canceled1)
+readableStreamDefaultControllerClose(branch1[kState].controller);
+if(!canceled2)
+readableStreamDefaultControllerClose(branch2[kState].controller);
+if(!canceled1||!canceled2)
+cancelPromise.resolve();
+});
 },
 [kError](){
 reading=false;

test/parallel/test-whatwg-readablestream.mjs

@@ -0,0 +1,36 @@
+import{mustCall}from'../common/index.mjs';
+import{ReadableStream}from'stream/web';
+importassertfrom'assert';
+
+{
+// Test tee() with close in the nextTick after enqueue
+asyncfunctionread(stream){
+constchunks=[];
+forawait(constchunkofstream)
+chunks.push(chunk);
+returnBuffer.concat(chunks).toString();
+}
+
+const[r1,r2]=newReadableStream({
+start(controller){
+process.nextTick(()=>{
+controller.enqueue(newUint8Array([102,111,111,98,97,114]));
+
+process.nextTick(()=>{
+controller.close();
+});
+});
+}
+}).tee();
+
+(async()=>{
+const[dataReader1,dataReader2]=awaitPromise.all([
+read(r1),
+read(r2),
+]);
+
+assert.strictEqual(dataReader1,dataReader2);
+assert.strictEqual(dataReader1,'foobar');
+assert.strictEqual(dataReader2,'foobar');
+})().then(mustCall());
+}