fix: query for circuit relays after start (#2309)

This should be done in the `afterStart` method otherwise the DHT
query manager may not have been started yet.

Author: achingbrain

packages/transport-circuit-relay-v2/src/transport/discovery.ts

@@ -59,12 +59,14 @@ export class RelayDiscovery extends TypedEventEmitter<RelayDiscoveryEvents> impl
       }
     })
 
+    this.started = true
+  }
+
+  afterStart (): void {
     void this.discover()
       .catch(err => {
-        this.log.error('error listening on relays', err)
+        this.log.error('error discovering relays', err)
       })
-
-    this.started = true
   }
 
   stop (): void {

packages/transport-circuit-relay-v2/src/transport/reservation-store.ts

@@ -115,11 +115,11 @@ export class ReservationStore extends TypedEventEmitter<ReservationStoreEvents>
     return this.started
   }
 
-  async start (): Promise<void> {
+  start (): void {
     this.started = true
   }
 
-  async stop (): Promise<void> {
+  stop (): void {
     this.reserveQueue.clear()
     this.reservations.forEach(({ timeout }) => {
       clearTimeout(timeout)

packages/transport-circuit-relay-v2/src/transport/transport.ts

@@ -103,8 +103,7 @@ export class CircuitRelayTransport implements Transport {
   }
 
   async start (): Promise<void> {
-    await this.reservationStore.start()
-    await this.discovery?.start()
+    this.reservationStore.start()
 
     await this.registrar.handle(RELAY_V2_STOP_CODEC, (data) => {
       void this.onStop(data).catch(err => {
@@ -117,12 +116,18 @@ export class CircuitRelayTransport implements Transport {
       runOnTransientConnection: true
     })
 
+    await this.discovery?.start()
+
     this.started = true
   }
 
+  afterStart (): void {
+    this.discovery?.afterStart()
+  }
+
   async stop (): Promise<void> {
     this.discovery?.stop()
-    await this.reservationStore.stop()
+    this.reservationStore.stop()
     await this.registrar.unhandle(RELAY_V2_STOP_CODEC)
 
     this.started = false