fix: stale connectionId (#336)

Instead of assigning the connectionId once in the space's constructor, it should be taken directly from the ably client. This ensures that the connectionId stays in sync even after a disconnect e.g. when the machine hibernates.

Author: bastiankistner

src/Space.ts

@@ -87,10 +87,7 @@ class Space extends EventEmitter<SpaceEventMap> {
    */
   readonly client: RealtimeClient;
   private readonly channelName: string;
-  /**
-   * @internal
-   */
-  readonly connectionId: string | undefined;
+
   /**
    * The options passed to {@link default.get | `Spaces.get()`}.
    */
@@ -126,7 +123,6 @@ class Space extends EventEmitter<SpaceEventMap> {
 
     this.client = client;
     this.options = this.setOptions(options);
-    this.connectionId = this.client.connection.id;
     this.name = name;
     this.channelName = `${name}${SPACE_CHANNEL_TAG}`;
 
@@ -140,6 +136,13 @@ class Space extends EventEmitter<SpaceEventMap> {
     this.locks = new Locks(this, this.presenceUpdate);
   }
 
+  /**
+   * @internal
+   */
+  get connectionId(): string | undefined {
+    return this.client.connection.id;
+  }
+
   private presenceUpdate = ({ data, extras }: SpacePresenceData) => {
     if (!extras) {
       return this.channel.presence.update(data);