lib: deprecate _stream_* modules

PR-URL: #58337
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Mattias Buelens <[email protected]>
Reviewed-By: Juan José Arboleda <[email protected]>

Author: dario-piotrowicz

doc/api/deprecations.md

@@ -3920,6 +3920,21 @@ Type: Runtime
 The `node:_tls_common` and `node:_tls_wrap` modules are deprecated as they should be considered
 an internal nodejs implementation rather than a public facing API, use `node:tls` instead.
 
+### DEP0193: `require('node:_stream_*')`
+
+<!-- YAML
+changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/58337
+    description: Runtime deprecation.
+-->
+
+Type: Runtime
+
+The `node:_stream_duplex`, `node:_stream_passthrough`, `node:_stream_readable`, `node:_stream_transform`,
+`node:_stream_wrap` and `node:_stream_writable` modules are deprecated as they should be considered
+an internal nodejs implementation rather than a public facing API, use `node:stream` instead.
+
 [DEP0142]: #dep0142-repl_builtinlibs
 [NIST SP 800-38D]: https://nvlpubs.nist.gov/nistpubs/Legacy/SP/nistspecialpublication800-38d.pdf
 [RFC 6066]: https://tools.ietf.org/html/rfc6066#section-3

lib/_stream_duplex.js

@@ -1,5 +1,6 @@
 'use strict';
 
-// Keep this file as an alias for the full stream module.
-
 module.exports = require('stream').Duplex;
+
+process.emitWarning('The _stream_duplex module is deprecated.',
+                    'DeprecationWarning', 'DEP0193');

lib/_stream_passthrough.js

@@ -1,5 +1,6 @@
 'use strict';
 
-// Keep this file as an alias for the full stream module.
-
 module.exports = require('stream').PassThrough;
+
+process.emitWarning('The _stream_passthrough module is deprecated.',
+                    'DeprecationWarning', 'DEP0193');

lib/_stream_readable.js

@@ -1,5 +1,6 @@
 'use strict';
 
-// Keep this file as an alias for the full stream module.
-
 module.exports = require('stream').Readable;
+
+process.emitWarning('The _stream_readable module is deprecated.',
+                    'DeprecationWarning', 'DEP0193');

lib/_stream_transform.js

@@ -1,5 +1,6 @@
 'use strict';
 
-// Keep this file as an alias for the full stream module.
-
 module.exports = require('stream').Transform;
+
+process.emitWarning('The _stream_transform module is deprecated.',
+                    'DeprecationWarning', 'DEP0193');

lib/_stream_wrap.js

@@ -1,5 +1,6 @@
 'use strict';
 
 module.exports = require('internal/js_stream_socket');
+
 process.emitWarning('The _stream_wrap module is deprecated.',
                     'DeprecationWarning', 'DEP0125');

lib/_stream_writable.js

@@ -1,5 +1,6 @@
 'use strict';
 
-// Keep this file as an alias for the full stream module.
-
 module.exports = require('stream').Writable;
+
+process.emitWarning('The _stream_writable module is deprecated.',
+                    'DeprecationWarning', 'DEP0193');

test/parallel/test-warn-stream-duplex-deprecation.js

@@ -0,0 +1,10 @@
+'use strict';
+
+const common = require('../common');
+
+// _stream_duplex is deprecated.
+
+common.expectWarning('DeprecationWarning',
+                     'The _stream_duplex module is deprecated.', 'DEP0193');
+
+require('_stream_duplex');

test/parallel/test-warn-stream-passthrough-deprecation.js

@@ -0,0 +1,10 @@
+'use strict';
+
+const common = require('../common');
+
+// _stream_passthrough is deprecated.
+
+common.expectWarning('DeprecationWarning',
+                     'The _stream_passthrough module is deprecated.', 'DEP0193');
+
+require('_stream_passthrough');

test/parallel/test-warn-stream-readable-deprecation.js

@@ -0,0 +1,10 @@
+'use strict';
+
+const common = require('../common');
+
+// _stream_readable is deprecated.
+
+common.expectWarning('DeprecationWarning',
+                     'The _stream_readable module is deprecated.', 'DEP0193');
+
+require('_stream_readable');