applied review changes to adapters.js

lib/internal/webstreams/adapters.js

@@ -451,6 +451,9 @@ function newReadableStreamFromStreamReadable(streamReadable, options = kEmptyObj
   const highWaterMark = streamReadable.readableHighWaterMark;
 
   const evaluateStrategyOrFallback = (strategy) => {
+    // If the stream is BYOB, we only use highWaterMark
+    if (isBYOB)
+      return { highWaterMark };
     // If there is a strategy available, use it
     if (strategy)
       return strategy;
@@ -519,7 +522,7 @@ function newReadableStreamFromStreamReadable(streamReadable, options = kEmptyObj
       wasCanceled = true;
       destroy(streamReadable, reason);
     },
-  }, isBYOB ? { highWaterMark: strategy.highWaterMark } : strategy);
+  }, strategy);
 }
 
 /**
@@ -622,6 +625,7 @@ function newStreamReadableFromReadableStream(readableStream, options = kEmptyObj
 /**
  * @param {Duplex} duplex
  * @param {{ type?: 'bytes' }} [options]
+ * @returns {ReadableWritablePair}
  */
 function newReadableWritablePairFromDuplex(duplex, options = kEmptyObject) {
   // Not using the internal/streams/utils isWritableNodeStream and
@@ -656,7 +660,7 @@ function newReadableWritablePairFromDuplex(duplex, options = kEmptyObject) {
   const readable =
     isReadable(duplex) ?
       newReadableStreamFromStreamReadable(duplex, options) :
-      new ReadableStream(options);
+      new ReadableStream({ type: options.type });
 
   if (!isReadable(duplex))
     readable.cancel();