fix: improve channel options reattachment logic

This internal change makes the client more selective about which options
provided to `channels.get` will be interpreted as requiring
reattachment. More precisely, the channel params and modes will now be
shallow compared to existing params and modes, and only if they differ
will the call the `chnanels.get` throw an error (, provided of course
that the channel is already attached or attaching).

Author: owenpearson

src/common/lib/client/realtimechannel.ts

@@ -185,7 +185,20 @@ class RealtimeChannel extends Channel {
   }
 
   _shouldReattachToSetOptions(options?: API.Types.ChannelOptions) {
-    return (this.state === 'attached' || this.state === 'attaching') && (options?.params || options?.modes);
+    if (!(this.state === 'attached' || this.state === 'attaching')) {
+      return false;
+    }
+    if (options?.params) {
+      if (!this.params || !Utils.shallowEquals(this.params, options.params)) {
+        return true;
+      }
+    }
+    if (options?.modes) {
+      if (!this.modes || !Utils.arrEquals(options.modes, this.modes)) {
+        return true;
+      }
+    }
+    return false;
   }
 
   publish(...args: any[]): void | Promise<void> {