Repository navigation
stream.isReadable and stream.isWritable return null instead of false #59006
Description
Activity
The docs say that the only valid parameters are
<Readable> | <Duplex> | <ReadableStream>. So I think it's reasonable that it returns null or throws when passed something else. Are you just looking for the docs to be updated?I'd suggest that the API as currently defined is buggy / incomplete. It really ought to return
booleanin every case.What should the parameter type be?
I'd say same list as
stream.isErrored:stream.isErrored(stream) stream <Readable> | <Writable> | <Duplex> | <WritableStream> | <ReadableStream> Returns: <boolean>Although I'd also replace
<Readable> | <Writable> | <Duplex>with just<Stream>(which should be no-op replacement, just a bit clearer).but this should be the same for the other ones in this file?
Those seem to be just some internal helpers? I don't see them on https://nodejs.org/api/stream.html
Reacted by Haroen Viaene- added a commit that references this issue
on Sep 7, 2025 - added a commit that references this issue
on Sep 9, 2025 - added a commit that references this issue
on Sep 20, 2025
Version
v24.2.0
Platform
Subsystem
No response
What steps will reproduce the bug?
Try these functions in REPL:
How often does it reproduce? Is there a required condition?
Always.
What is the expected behavior? Why is that the expected behavior?
It should always return a boolean as per docs.
What do you see instead?
trueandnullAdditional information
No response