SEA binary not dying when calling process.exit if a fd is open

[Llorx]

Version

22.14 and 23.10

Platform

Windows 11 and Debian 11

What steps will reproduce the bug?

Create 2 files:

// parent.js
var spawn = require('child_process').spawn;
var stdio = ["inherit", "inherit", "inherit", "pipe"];
const proc = spawn("child", [], {
    env: process.env,
    stdio: stdio
});
proc.on("error", (e) => {
    console.log("error", e);
});
proc.on("exit", (code) => {
    console.log("exit", code);
});

// child.js
const Fs = require("fs");

console.log("OK!");
const rx = Fs.createReadStream("", {
    fd: 3
});

const onData = () => {};
const onError = (e) => console.error(e);
const onClose = () => {console.log("closed")};

rx.addListener("data", onData);
rx.addListener("error", onError);
rx.addListener("close", onClose);

setTimeout(() => {
    console.log("END!");
    rx.destroy();
    Fs.close(3, console.log);
    process.exit();
}, 2000);

Create a SEA binary from child.js:

{ "main": "child.js", "output": "sea-prep.child.blob" }

node --experimental-sea-config sea-config.child.json
node -e "require('fs').copyFileSync(process.execPath, 'child.exe')"
call npx postject child.exe NODE_SEA_BLOB sea-prep.child.blob --sentinel-fuse NODE_SEA_FUSE_fce680ab2cc467b6e072b8b5df1996b2

Now run parent.js

node parent.js

You will notice that the child never dies. It will hang forever in the task manager. If instead of SEA you use plain node scripts (and modify parent.js to spawn node child.js) it works as expected.

How often does it reproduce? Is there a required condition?

Always

What is the expected behavior? Why is that the expected behavior?

The child process to exit

What do you see instead?

It hangs forever

Additional information

No response

[Llorx]

PS: I have this workaround that works, which follows the nodejs internalities when creating the stdin, but still createReadStream is the way to go I guess as this workaround is not a public API. Still feels weird that createReadStream in non-sea works and sea doesn't.

const rx = new net.Socket({
    fd: 3,
    readable: true,
    writable: false,
    manualStart: true
});
rx._writableState.ended = true;

[github-actions]

This issue has been marked as stale due to 210 days of inactivity.
It will be automatically closed in 30 days if no further activity occurs. If this is still relevant, please leave a comment or update it to keep it open.