From 250a20e6d0768b4b9cc5b616bd3eccff2a901fc3 Mon Sep 17 00:00:00 2001
From: Lukas <mail@lukasseiler.de>
Date: Fri, 9 Sep 2022 10:48:02 +0200
Subject: [PATCH 1/2] add connected event, fix connect future

---
 src/room/Room.ts   | 18 +++++++++++-------
 src/room/events.ts |  5 +++++
 src/room/utils.ts  | 29 ++++++++++++++++++-----------
 3 files changed, 34 insertions(+), 18 deletions(-)

diff --git a/src/room/Room.ts b/src/room/Room.ts
index 381d637d5..0dce2e5d3 100644
--- a/src/room/Room.ts
+++ b/src/room/Room.ts
@@ -171,7 +171,7 @@ class Room extends (EventEmitter as new () => TypedEmitter<RoomEventCallbacks>)
       })
       .on(EngineEvent.Resumed, () => {
         this.setAndEmitConnectionState(ConnectionState.Connected);
-        this.reconnectFuture?.resolve();
+        this.reconnectFuture?.resolve?.();
         this.reconnectFuture = undefined;
         this.emit(RoomEvent.Reconnected);
         this.updateSubscriptions();
@@ -218,7 +218,7 @@ class Room extends (EventEmitter as new () => TypedEmitter<RoomEventCallbacks>)
       this.connectFuture = this.reconnectFuture;
       return this.connectFuture.promise;
     }
-    const connectPromise = new Promise<void>(async (resolve, reject) => {
+    const connectFn = async (resolve: () => void, reject: (reason: any) => void) => {
       this.setAndEmitConnectionState(ConnectionState.Connecting);
       if (!this.abortController || this.abortController.signal.aborted) {
         this.abortController = new AbortController();
@@ -347,8 +347,11 @@ class Room extends (EventEmitter as new () => TypedEmitter<RoomEventCallbacks>)
         this.setAndEmitConnectionState(ConnectionState.Connected);
         resolve();
       });
-    }).finally(() => (this.connectFuture = undefined));
-    this.connectFuture = new Future(connectPromise);
+    };
+    this.connectFuture = new Future(connectFn, () => {
+      this.connectFuture = undefined;
+      this.emit(RoomEvent.Connected);
+    });
 
     return this.connectFuture.promise;
   };
@@ -363,7 +366,7 @@ class Room extends (EventEmitter as new () => TypedEmitter<RoomEventCallbacks>)
       log.warn('abort connection attempt');
       this.abortController?.abort();
       // in case the abort controller didn't manage to cancel the connection attempt, reject the connect promise explicitly
-      this.connectFuture?.reject(new ConnectionError('Client initiated disconnect'));
+      this.connectFuture?.reject?.(new ConnectionError('Client initiated disconnect'));
       this.connectFuture = undefined;
     }
     // send leave
@@ -625,7 +628,7 @@ class Room extends (EventEmitter as new () => TypedEmitter<RoomEventCallbacks>)
     });
     this.setAndEmitConnectionState(ConnectionState.Connected);
     this.emit(RoomEvent.Reconnected);
-    this.reconnectFuture?.resolve();
+    this.reconnectFuture?.resolve?.();
     this.reconnectFuture = undefined;
 
     // rehydrate participants
@@ -672,7 +675,7 @@ class Room extends (EventEmitter as new () => TypedEmitter<RoomEventCallbacks>)
     }
     // reject potentially ongoing reconnection attempt
     if (this.connectFuture === this.reconnectFuture) {
-      this.connectFuture?.reject(undefined);
+      this.connectFuture?.reject?.(undefined);
       this.connectFuture = undefined;
     }
     this.reconnectFuture = undefined;
@@ -1142,6 +1145,7 @@ class Room extends (EventEmitter as new () => TypedEmitter<RoomEventCallbacks>)
 export default Room;
 
 export type RoomEventCallbacks = {
+  connected: () => void;
   reconnecting: () => void;
   reconnected: () => void;
   disconnected: (reason?: DisconnectReason) => void;
diff --git a/src/room/events.ts b/src/room/events.ts
index 7a1187086..3864f4ebd 100644
--- a/src/room/events.ts
+++ b/src/room/events.ts
@@ -9,6 +9,11 @@
  */
 
 export enum RoomEvent {
+  /**
+   * When the connection to the server has been established
+   */
+  Connected = 'connected',
+
   /**
    * When the connection to the server has been interrupted and it's attempting
    * to reconnect.
diff --git a/src/room/utils.ts b/src/room/utils.ts
index 9dbe5b597..fb922ab42 100644
--- a/src/room/utils.ts
+++ b/src/room/utils.ts
@@ -182,16 +182,23 @@ export function getEmptyAudioStreamTrack() {
 export class Future<T> {
   promise: Promise<T>;
 
-  resolve!: (arg: T) => void;
-
-  reject!: (e: any) => void;
-
-  constructor(promise?: Promise<T>) {
-    this.promise =
-      promise ??
-      new Promise<T>((resolve, reject) => {
-        this.resolve = resolve;
-        this.reject = reject;
-      });
+  resolve?: (arg: T) => void;
+
+  reject?: (e: any) => void;
+
+  onFinally?: () => void;
+
+  constructor(
+    futureBase?: (resolve: (arg: T) => void, reject: (e: any) => void) => void,
+    onFinally?: () => void,
+  ) {
+    this.onFinally = onFinally;
+    this.promise = new Promise<T>(async (resolve, reject) => {
+      this.resolve = resolve;
+      this.reject = reject;
+      if (futureBase) {
+        await futureBase(resolve, reject);
+      }
+    }).finally(() => this.onFinally?.());
   }
 }

From 46ab7e4ba271fa57e9b8ffd7d26ff12d078574ba Mon Sep 17 00:00:00 2001
From: Lukas <mail@lukasseiler.de>
Date: Mon, 12 Sep 2022 11:45:06 +0200
Subject: [PATCH 2/2] changeset

---
 .changeset/stupid-rice-reflect.md | 5 +++++
 1 file changed, 5 insertions(+)
 create mode 100644 .changeset/stupid-rice-reflect.md

diff --git a/.changeset/stupid-rice-reflect.md b/.changeset/stupid-rice-reflect.md
new file mode 100644
index 000000000..037958e04
--- /dev/null
+++ b/.changeset/stupid-rice-reflect.md
@@ -0,0 +1,5 @@
+---
+'livekit-client': patch
+---
+
+Add RoomEvent.Connected, fix connectFuture rejection exception