diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 0164937307a770..71ed8f5d9bfafa 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -29,7 +29,6 @@ # streams -/lib/_stream* @nodejs/streams /lib/internal/streams/* @nodejs/streams /lib/stream.js @nodejs/streams /lib/stream/* @nodejs/streams diff --git a/doc/api/deprecations.md b/doc/api/deprecations.md index 610bcc4f7302a1..31e2925a40398e 100644 --- a/doc/api/deprecations.md +++ b/doc/api/deprecations.md @@ -2786,12 +2786,15 @@ This property is a reference to the instance itself. -Type: Runtime +Type: End-of-Life The `node:_stream_wrap` module is deprecated. @@ -4163,6 +4166,9 @@ an internal nodejs implementation rather than a public facing API, use `node:tls -Type: Runtime +Type: End-of-Life 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 diff --git a/lib/_stream_duplex.js b/lib/_stream_duplex.js deleted file mode 100644 index 3dca7fee54e356..00000000000000 --- a/lib/_stream_duplex.js +++ /dev/null @@ -1,6 +0,0 @@ -'use strict'; - -module.exports = require('stream').Duplex; - -process.emitWarning('The _stream_duplex module is deprecated. Use `node:stream` instead.', - 'DeprecationWarning', 'DEP0193'); diff --git a/lib/_stream_passthrough.js b/lib/_stream_passthrough.js deleted file mode 100644 index 9255ec40e7c347..00000000000000 --- a/lib/_stream_passthrough.js +++ /dev/null @@ -1,6 +0,0 @@ -'use strict'; - -module.exports = require('stream').PassThrough; - -process.emitWarning('The _stream_passthrough module is deprecated. Use `node:stream` instead.', - 'DeprecationWarning', 'DEP0193'); diff --git a/lib/_stream_readable.js b/lib/_stream_readable.js deleted file mode 100644 index 8b7db04064001f..00000000000000 --- a/lib/_stream_readable.js +++ /dev/null @@ -1,6 +0,0 @@ -'use strict'; - -module.exports = require('stream').Readable; - -process.emitWarning('The _stream_readable module is deprecated. Use `node:stream` instead.', - 'DeprecationWarning', 'DEP0193'); diff --git a/lib/_stream_transform.js b/lib/_stream_transform.js deleted file mode 100644 index a62b4ae338cc3d..00000000000000 --- a/lib/_stream_transform.js +++ /dev/null @@ -1,6 +0,0 @@ -'use strict'; - -module.exports = require('stream').Transform; - -process.emitWarning('The _stream_transform module is deprecated. Use `node:stream` instead.', - 'DeprecationWarning', 'DEP0193'); diff --git a/lib/_stream_wrap.js b/lib/_stream_wrap.js deleted file mode 100644 index 710f3565c3e8b6..00000000000000 --- a/lib/_stream_wrap.js +++ /dev/null @@ -1,6 +0,0 @@ -'use strict'; - -module.exports = require('internal/js_stream_socket'); - -process.emitWarning('The _stream_wrap module is deprecated. Use `node:stream` instead.', - 'DeprecationWarning', 'DEP0125'); diff --git a/lib/_stream_writable.js b/lib/_stream_writable.js deleted file mode 100644 index 3cef06126e5f42..00000000000000 --- a/lib/_stream_writable.js +++ /dev/null @@ -1,6 +0,0 @@ -'use strict'; - -module.exports = require('stream').Writable; - -process.emitWarning('The _stream_writable module is deprecated. Use `node:stream` instead.', - 'DeprecationWarning', 'DEP0193'); diff --git a/lib/internal/main/mksnapshot.js b/lib/internal/main/mksnapshot.js index b06b1a70bf4939..3b17531baf0aaf 100644 --- a/lib/internal/main/mksnapshot.js +++ b/lib/internal/main/mksnapshot.js @@ -42,12 +42,6 @@ const supportedModules = new SafeSet(new SafeArrayIterator([ // '_http_incoming', // '_http_outgoing', // '_http_server', - '_stream_duplex', - '_stream_passthrough', - '_stream_readable', - '_stream_transform', - '_stream_wrap', - '_stream_writable', // '_tls_common', // '_tls_wrap', 'assert', diff --git a/test/parallel/test-internal-module-require.js b/test/parallel/test-internal-module-require.js index 213838150b96d9..7e22c94464424a 100644 --- a/test/parallel/test-internal-module-require.js +++ b/test/parallel/test-internal-module-require.js @@ -24,12 +24,6 @@ const expectedPublicModules = new Set([ '_http_incoming', '_http_outgoing', '_http_server', - '_stream_duplex', - '_stream_passthrough', - '_stream_readable', - '_stream_transform', - '_stream_wrap', - '_stream_writable', '_tls_common', '_tls_wrap', 'assert', diff --git a/test/parallel/test-stream-aliases-legacy.js b/test/parallel/test-stream-aliases-legacy.js deleted file mode 100644 index 2c87f0ad0fa4a4..00000000000000 --- a/test/parallel/test-stream-aliases-legacy.js +++ /dev/null @@ -1,14 +0,0 @@ -'use strict'; - -require('../common'); - -const assert = require('assert'); -const stream = require('stream'); - -// Verify that all individual aliases are left in place. - -assert.strictEqual(stream.Readable, require('_stream_readable')); -assert.strictEqual(stream.Writable, require('_stream_writable')); -assert.strictEqual(stream.Duplex, require('_stream_duplex')); -assert.strictEqual(stream.Transform, require('_stream_transform')); -assert.strictEqual(stream.PassThrough, require('_stream_passthrough')); diff --git a/test/parallel/test-warn-stream-duplex-deprecation.js b/test/parallel/test-warn-stream-duplex-deprecation.js deleted file mode 100644 index a52fb55d0b80a4..00000000000000 --- a/test/parallel/test-warn-stream-duplex-deprecation.js +++ /dev/null @@ -1,10 +0,0 @@ -'use strict'; - -const common = require('../common'); - -// _stream_duplex is deprecated. - -common.expectWarning('DeprecationWarning', - 'The _stream_duplex module is deprecated. Use `node:stream` instead.', 'DEP0193'); - -require('_stream_duplex'); diff --git a/test/parallel/test-warn-stream-passthrough-deprecation.js b/test/parallel/test-warn-stream-passthrough-deprecation.js deleted file mode 100644 index 6726aeb4d7346e..00000000000000 --- a/test/parallel/test-warn-stream-passthrough-deprecation.js +++ /dev/null @@ -1,10 +0,0 @@ -'use strict'; - -const common = require('../common'); - -// _stream_passthrough is deprecated. - -common.expectWarning('DeprecationWarning', - 'The _stream_passthrough module is deprecated. Use `node:stream` instead.', 'DEP0193'); - -require('_stream_passthrough'); diff --git a/test/parallel/test-warn-stream-readable-deprecation.js b/test/parallel/test-warn-stream-readable-deprecation.js deleted file mode 100644 index dbdc1702b59e60..00000000000000 --- a/test/parallel/test-warn-stream-readable-deprecation.js +++ /dev/null @@ -1,10 +0,0 @@ -'use strict'; - -const common = require('../common'); - -// _stream_readable is deprecated. - -common.expectWarning('DeprecationWarning', - 'The _stream_readable module is deprecated. Use `node:stream` instead.', 'DEP0193'); - -require('_stream_readable'); diff --git a/test/parallel/test-warn-stream-transform-deprecation.js b/test/parallel/test-warn-stream-transform-deprecation.js deleted file mode 100644 index 600cbcc1923b8a..00000000000000 --- a/test/parallel/test-warn-stream-transform-deprecation.js +++ /dev/null @@ -1,10 +0,0 @@ -'use strict'; - -const common = require('../common'); - -// _stream_transform is deprecated. - -common.expectWarning('DeprecationWarning', - 'The _stream_transform module is deprecated. Use `node:stream` instead.', 'DEP0193'); - -require('_stream_transform'); diff --git a/test/parallel/test-warn-stream-wrap-deprecation.js b/test/parallel/test-warn-stream-wrap-deprecation.js deleted file mode 100644 index 9146cbd6af1555..00000000000000 --- a/test/parallel/test-warn-stream-wrap-deprecation.js +++ /dev/null @@ -1,10 +0,0 @@ -'use strict'; - -const common = require('../common'); - -// _stream_wrap is deprecated. - -common.expectWarning('DeprecationWarning', - 'The _stream_wrap module is deprecated. Use `node:stream` instead.', 'DEP0125'); - -require('_stream_wrap'); diff --git a/test/parallel/test-warn-stream-writable-deprecation.js b/test/parallel/test-warn-stream-writable-deprecation.js deleted file mode 100644 index 3a8765ca93d525..00000000000000 --- a/test/parallel/test-warn-stream-writable-deprecation.js +++ /dev/null @@ -1,10 +0,0 @@ -'use strict'; - -const common = require('../common'); - -// _stream_writable is deprecated. - -common.expectWarning('DeprecationWarning', - 'The _stream_writable module is deprecated. Use `node:stream` instead.', 'DEP0193'); - -require('_stream_writable'); diff --git a/tsconfig.json b/tsconfig.json index 3f5a3067ced063..c584b09c5456b6 100644 --- a/tsconfig.json +++ b/tsconfig.json @@ -20,12 +20,6 @@ "_http_incoming": ["./lib/_http_incoming.js"], "_http_outgoing": ["./lib/_http_outgoing.js"], "_http_server": ["./lib/_http_server.js"], - "_stream_duplex": ["./lib/_stream_duplex.js"], - "_stream_passthrough": ["./lib/_stream_passthrough.js"], - "_stream_readable": ["./lib/_stream_readable.js"], - "_stream_transform": ["./lib/_stream_transform.js"], - "_stream_wrap": ["./lib/_stream_wrap.js"], - "_stream_writable": ["./lib/_stream_writable.js"], "_tls_common": ["./lib/_tls_common.js"], "_tls_wrap": ["./lib/_tls_wrap.js"], "assert": ["./lib/assert.js"],