fix(js-client-sdk): better undefined handling

Author: joker23

packages/sdk/browser/src/BrowserApi.ts

@@ -50,13 +50,13 @@ export function addWindowEventListener(
   listener: (this: Document, ev: Event) => any,
   options?: boolean | AddEventListenerOptions,
 ): () => void {
-  if (isDocument()) {
+  if (isWindow()) {
     window.addEventListener(type, listener, options);
     return () => {
       window.removeEventListener(type, listener, options);
     };
   }
-  // No document, so no need to unregister anything.
+  // No window, so no need to unregister anything.
   return () => {};
 }
 

packages/sdk/browser/src/platform/randomUuidV4.ts

@@ -36,7 +36,7 @@ const nodes = {
 };
 
 function getRandom128bit(): number[] {
-  if (crypto && crypto.getRandomValues) {
+  if (typeof crypto !== 'undefined' && crypto.getRandomValues) {
     const typedArray = new Uint8Array(16);
     crypto.getRandomValues(typedArray);
     return [...typedArray.values()];

packages/telemetry/browser-telemetry/src/randomUuidV4.ts

@@ -39,7 +39,7 @@ const nodes = {
 };
 
 function getRandom128bit(): number[] {
-  if (crypto && crypto.getRandomValues) {
+  if (typeof crypto !== 'undefined' && crypto.getRandomValues) {
     const typedArray = new Uint8Array(16);
     crypto.getRandomValues(typedArray);
     return [...typedArray.values()];