Stream finished does not result in error when is stream already destroyed

[misos1]

• Version: v16.1.0
• Platform: Darwin Kernel Version 20.3.0
• Subsystem: stream

What steps will reproduce the bug?

let { PassThrough, finished } = require("stream");

let stream = new PassThrough();
stream.destroy();
stream.on("close", function()
{
	finished(stream, err => console.log(err));
});

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

Always.

What is the expected behavior?

Maybe it should produce an error like ERR_STREAM_PREMATURE_CLOSE (or ERR_STREAM_DESTROYED). But I would strongly prefer ERR_STREAM_PREMATURE_CLOSE (in fact this is form of premature close) because this error is returned when the stream is destroyed after the call to finished and before finished calls its callback. This would be much more consistent than race between two different errors based on when exactly the stream was destroyed. It just complicates code path which tries to react to this situation and is unaware whether the stream is already destroyed or not (it would have to check for two error codes instead of only one). Same should be done for the pipeline #36674.

What do you see instead?

undefined

Promise version resolves.

Additional information

[misos1]

node/lib/internal/streams/end-of-stream.js

Lines 184 to 194 in 3f025cb

	if (closed) {
	// TODO(ronag): Re-throw error if errorEmitted?
	// TODO(ronag): Throw premature close as if finished was called?
	// before being closed? i.e. if closed but not errored, ended or finished.
	// TODO(ronag): Throw some kind of error? Does it make sense
	// to call finished() on a "finished" stream?
	// TODO(ronag): willEmitClose?
	process.nextTick(() => {
	callback();
	});
	}

[avivkeller]

let { PassThrough, finished } = require("stream");

let stream = new PassThrough();
stream.destroy();
stream.on("close", function()
{
	finished(stream, err => console.log(err));
});

Error [ERR_STREAM_PREMATURE_CLOSE]: Premature close
    at onclose (node:internal/streams/end-of-stream:153:30)
    at process.processTicksAndRejections (node:internal/process/task_queues:77:11) {
  code: 'ERR_STREAM_PREMATURE_CLOSE'
}

This is no longer reproducible, closing.