fix(lazer-sdk-js): swapped to the correct WS closing function and removed an unnecessary isBrowser conditional check

Author: benduran

lazer/sdk/js/src/socket/resilient-websocket.ts

@@ -155,7 +155,7 @@ export class ResilientWebSocket {
 
     this.heartbeatTimeout = setTimeout(() => {
       this.logger.warn("Connection timed out. Reconnecting...");
-      this.wsClient?.terminate();
+      this.wsClient?.close();
       this.handleReconnect();
     }, this.heartbeatTimeoutDurationMs);
   }
@@ -184,8 +184,8 @@ export class ResilientWebSocket {
     if (this.shouldLogRetry()) {
       this.logger.error(
         "Connection closed unexpectedly or because of timeout. Reconnecting after " +
-          String(this.retryDelayMs()) +
-          "ms.",
+        String(this.retryDelayMs()) +
+        "ms.",
       );
     }
 

lazer/sdk/js/src/util.ts

@@ -23,8 +23,8 @@ export function envIsWorker() {
 export function envIsBrowser() {
   // disabling this rule for this line, as the rule is reporting a false positive (it's a bug)
   // for the typeof check for the existence of the querySelector function
-  // eslint-disable-next-line @typescript-eslint/no-deprecated
-  return typeof window !== 'undefined' && Boolean(window.navigator.userAgent) && typeof window.document.querySelector === 'function';
+
+  return typeof window !== 'undefined' && Boolean(window.navigator.userAgent);
 }
 
 /**