stream: try to fix, this can have major consonances, really WIP

just want the CI to run

Author: rluvaton

lib/internal/streams/readable.js

@@ -108,7 +108,6 @@ function ReadableState(options, stream, isDuplex) {
   // linked list can remove elements from the beginning faster than
   // array.shift().
   this.buffer = new BufferList();
-  this.length = 0;
   this.pipes = [];
   this.flowing = null;
   this.ended = false;
@@ -191,6 +190,16 @@ function ReadableState(options, stream, isDuplex) {
 }
 
 
+ObjectDefineProperties(ReadableState.prototype, {
+  length: {
+    __proto__: null,
+    get() {
+      return this.buffer.length;
+    },
+  },
+});
+
+
 function Readable(options) {
   if (!(this instanceof Readable))
     return new Readable(options);
@@ -1108,7 +1117,7 @@ async function* createAsyncIterator(stream, options) {
   let callback = nop;
 
   function next(resolve) {
-    if (this === stream) {
+    if (this === stream || !resolve) {
       callback();
       callback = nop;
     } else {
@@ -1155,23 +1164,15 @@ async function* createAsyncIterator(stream, options) {
 }
 
 function staticUnref(stream) {
-  const unrefStream = new Readable({
-    objectMode: stream.readableObjectMode ?? stream.objectMode ?? true,
-    // highWaterMark 0 as unref is basically a proxy, so don't consume more data
-    // as we would lose it when continue consuming from the original stream
-    highWaterMark: 0,
-    // TODO - what about other options?
-    destroy(err, callback) {
-      // Not destroying the stream here as we unref it.
-      callback(err);
-    },
-  }).wrap(stream);
+  const unrefStream = Object.create(stream, {
+  });
 
-  unrefStream.once('error', (e) => {
-    if (e.name !== 'AbortError') {
-      destroyImpl.destroyer(stream, e);
+  unrefStream._destroy = (err) => {
+    if (err?.name !== 'AbortError') {
+      destroyImpl.destroyer(stream, err);
     }
-  });
+  };
+  unrefStream._readableState = Object.create(stream._readableState);
 
   return unrefStream;
 }