lib: use kEmptyObject and update JSDoc in webstreams

Use kEmptyObject as default value of strategy.
Plus, make reason and chunk as optional.
And refactor to use validateBuffer.
Refs: https://github.com/nodejs/node/blob/main/doc/api/webstreams.md#transformstreamdefaultcontrollerenqueuechunk
Refs: https://github.com/nodejs/node/blob/main/doc/api/webstreams.md#transformstreamdefaultcontrollererrorreason
Refs: https://github.com/nodejs/node/blob/main/doc/api/webstreams.md#writablestreamdefaultwriterabortreason
Refs: https://github.com/nodejs/node/blob/main/doc/api/webstreams.md#writablestreamdefaultwriterwritechunk
PR-URL: #46183
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Minwoo Jung <nodecorelab@gmail.com>

Author: deokjinkim

lib/internal/webstreams/readablestream.js

@@ -1132,16 +1132,7 @@ class ReadableByteStreamController {
 enqueue(chunk){
 if(!isReadableByteStreamController(this))
 thrownewERR_INVALID_THIS('ReadableByteStreamController');
-if(!isArrayBufferView(chunk)){
-thrownewERR_INVALID_ARG_TYPE(
-'chunk',
-[
-'Buffer',
-'TypedArray',
-'DataView',
-],
-chunk);
-}
+validateBuffer(chunk);
 constchunkByteLength=ArrayBufferViewGetByteLength(chunk);
 constchunkBuffer=ArrayBufferViewGetBuffer(chunk);
 constchunkBufferByteLength=ArrayBufferPrototypeGetByteLength(chunkBuffer);

lib/internal/webstreams/transformstream.js

@@ -26,6 +26,7 @@ const {
 const{
  createDeferredPromise,
 customInspectSymbol: kInspect,
+ kEmptyObject,
  kEnumerableProperty,
 }=require('internal/util');
 
@@ -117,8 +118,8 @@ class TransformStream {
  */
 constructor(
 transformer=null,
-writableStrategy={},
-readableStrategy={}){
+writableStrategy=kEmptyObject,
+readableStrategy=kEmptyObject){
 constreadableType=transformer?.readableType;
 constwritableType=transformer?.writableType;
 conststart=transformer?.start;
@@ -292,7 +293,7 @@ class TransformStreamDefaultController {
 }
 
 /**
- * @param {any} chunk
+ * @param {any} [chunk]
  */
 enqueue(chunk=undefined){
 if(!isTransformStreamDefaultController(this))
@@ -301,7 +302,7 @@ class TransformStreamDefaultController {
 }
 
 /**
- * @param {any} reason
+ * @param {any} [reason]
  */
 error(reason=undefined){
 if(!isTransformStreamDefaultController(this))

lib/internal/webstreams/writablestream.js

@@ -33,6 +33,7 @@ const {
 const{
  createDeferredPromise,
 customInspectSymbol: kInspect,
+ kEmptyObject,
  kEnumerableProperty,
  SideEffectFreeRegExpPrototypeSymbolReplace,
 }=require('internal/util');
@@ -148,7 +149,7 @@ class WritableStream {
  * @param {UnderlyingSink} [sink]
  * @param {QueuingStrategy} [strategy]
  */
-constructor(sink=null,strategy={}){
+constructor(sink=null,strategy=kEmptyObject){
 consttype=sink?.type;
 if(type!==undefined)
 thrownewERR_INVALID_ARG_VALUE.RangeError('type',type);
@@ -217,7 +218,7 @@ class WritableStream {
 }
 
 /**
- * @param {any} reason
+ * @param {any} [reason]
  * @returns {Promise<void>}
  */
 abort(reason=undefined){
@@ -475,7 +476,7 @@ class WritableStreamDefaultWriter {
 }
 
 /**
- * @param {any} chunk
+ * @param {any} [chunk]
  * @returns {Promise<void>}
  */
 write(chunk=undefined){