remove determineUrlChanges as it is not needed

Author: mmcallister-cll

src/transports/websocket.ts

@@ -373,7 +373,10 @@ export class WebSocketTransport<
     // This is because some providers handle subscriptions on the URLs and not through messages.
     // Subclasses may also implement alternate URL handling logic,
     // eg: toggling through multiple possible URLs in case of failure.
-    const { urlChanged, url } = await this.determineUrlChange(context, subscriptions)
+    const url = await this.config.url(context, subscriptions.desired, {
+      streamHandlerInvocationsWithNoConnection: this.streamHandlerInvocationsWithNoConnection,
+    })
+    const urlChanged = this.currentUrl !== url
 
     // Check if we should close the current connection
     if (!connectionClosed && (urlChanged || connectionUnresponsive)) {
@@ -473,17 +476,6 @@ export class WebSocketTransport<
     return
   }
 
-  async determineUrlChange(
-    context: EndpointContext<T>,
-    subscriptions: SubscriptionDeltas<TypeFromDefinition<T['Parameters']>>,
-  ): Promise<{ urlChanged: boolean; url: string }> {
-    const url = await this.config.url(context, subscriptions.desired, {
-      streamHandlerInvocationsWithNoConnection: this.streamHandlerInvocationsWithNoConnection,
-    })
-    const urlChanged = this.currentUrl !== url
-    return { urlChanged, url }
-  }
-
   private rejectionHandler<E>(
     rejectionFn: (reason?: unknown) => void,
     handler: (event: E) => Promise<void>,