fix: LC start should never block

packages/light-client/src/index.ts

@@ -160,22 +160,10 @@ export class Lightclient {
   }
 
   /**
-   * @returns a `Promise` that will resolve once `LightclientEvent.statusChange` with `RunStatusCode.started` value is emitted
+   * @returns a `Promise` that will resolve once `runStatus` equals `RunStatusCode.started`
    */
   start(): Promise<void> {
-    const startPromise = new Promise<void>((resolve) => {
-      const lightclientStarted = (status: RunStatusCode): void => {
-        if (status === RunStatusCode.started) {
-          this.emitter.off(LightclientEvent.statusChange, lightclientStarted);
-          resolve();
-        }
-      };
-      this.emitter.on(LightclientEvent.statusChange, lightclientStarted);
-    });
-
-    void this.runLoop();
-
-    return startPromise;
+    return this.runLoop();
   }
 
   stop(): void {
@@ -272,17 +260,14 @@ export class Lightclient {
       }
 
       // Wait for the next epoch
-      if (this.runStatus.code !== RunStatusCode.started) {
-        return;
-      } else {
-        try {
-          await sleep(timeUntilNextEpoch(this.config, this.genesisTime), this.runStatus.controller.signal);
-        } catch (e) {
-          if (isErrorAborted(e)) {
-            return;
-          }
-          throw e;
+      try {
+        const runStatus = this.runStatus as {code: RunStatusCode.started; controller: AbortController}; // At this point, client is started
+        await sleep(timeUntilNextEpoch(this.config, this.genesisTime), runStatus.controller.signal);
+      } catch (e) {
+        if (isErrorAborted(e)) {
+          return;
         }
+        throw e;
       }
     }
   }