fix: only close stream if it is open (#2823)

Adds a guard to the `.close` operation similar to the one on
`.abort` that ensures we only close the stream if it is open.

The individual `.closeRead`/`.closeWrite` operations already guard
on the read/write status of the stream so there's no functional
change, we just avoid a bit more async work as those methods
return promises.

Author: achingbrain

packages/integration-tests/test/circuit-relay.node.ts

@@ -519,8 +519,8 @@ describe('circuit-relay', () => {
       await deferred.promise
 
       // should have closed connections to remote and to relay
-      expect(events[0].detail.remotePeer.toString()).to.equal(relay1.peerId.toString())
-      expect(events[1].detail.remotePeer.toString()).to.equal(remote.peerId.toString())
+      expect(events[0].detail.remotePeer.toString()).to.equal(remote.peerId.toString())
+      expect(events[1].detail.remotePeer.toString()).to.equal(relay1.peerId.toString())
     })
 
     it('should mark an outgoing relayed connection as limited', async () => {

packages/utils/src/abstract-stream.ts

@@ -284,6 +284,10 @@ export abstract class AbstractStream implements Stream {
 
   // Close for both Reading and Writing
   async close (options?: AbortOptions): Promise<void> {
+    if (this.status !== 'open') {
+      return
+    }
+
     this.log.trace('closing gracefully')
 
     this.status = 'closing'