stream: forward errors correctly for duplexPair endpoints

fix the duplexPair implementation so that when one side is
destroyed with an error, the other side also receives the error
or a close event as appropriate.

previous behavior caused sideA to never emit an 'error' or
'close' when sideB errored, which prevented users from
observing or handling the paired stream failure.

Fixes: #61015

Author: aelhor

lib/internal/streams/duplexpair.js

@@ -50,13 +50,26 @@ class DuplexSide extends Duplex {
     this.#otherSide.on('end', callback);
     this.#otherSide.push(null);
   }
+
+
+  _destroy(err, callback) {
+    if (err) {
+      // Error case: tell the other side to also destroy with that error.
+      this.#otherSide.destroy(err);
+    } else if (this.#otherSide && !this.#otherSide.destroyed) {
+      // Graceful close case (destroy() without error):
+      // send an EOF to the other side's readable end if it hasn't already closed.
+      this.#otherSide.push(null);
+    }
+    callback(err);
+  }
 }
 
 function duplexPair(options) {
   const side0 = new DuplexSide(options);
   const side1 = new DuplexSide(options);
   side0[kInitOtherSide](side1);
   side1[kInitOtherSide](side0);
-  return [ side0, side1 ];
+  return [side0, side1];
 }
 module.exports = duplexPair;

test/parallel/test-duplex-error.js

@@ -0,0 +1,20 @@
+'use strict';
+
+const common = require('../common');
+const assert = require('assert');
+const { duplexPair } = require('stream');
+
+const [sideA, sideB] = duplexPair();
+
+sideA.on('error', common.mustCall((err) => {
+  assert.strictEqual(err.message, 'Simulated error');
+}));
+
+sideB.on('error', common.mustCall((err) => {
+  assert.strictEqual(err.message, 'Simulated error');
+}));
+
+sideA.resume();
+sideB.resume();
+
+sideB.destroy(new Error('Simulated error'));