fix: move node check to init (#1088)

Author: millcek

packages/web/src/check-browser.ts

@@ -16,7 +16,6 @@
  *
  */
 
-import './check-browser'
 import './polyfill-safari10'
 import { registerInstrumentations } from '@opentelemetry/instrumentation'
 import {
@@ -288,6 +287,13 @@ export const SplunkRum: SplunkOtelWebType = {
 
 	init: function (options) {
 		userTrackingMode = options.user?.trackingMode ?? 'noTracking'
+
+		if (typeof window !== 'object') {
+			throw Error(
+				'SplunkRum Error: This library is intended to run in a browser environment. Please ensure the code is evaluated within a browser context.',
+			)
+		}
+
 		// "env" based config still a bad idea for web
 		if (!('OTEL_TRACES_EXPORTER' in _globalThis)) {
 			_globalThis.OTEL_TRACES_EXPORTER = 'none'

packages/web/src/index.ts

@@ -20,7 +20,7 @@ import { isIframe } from '../utils'
 import { throttle } from '../utils/throttle'
 import { Store } from './store'
 
-const isSafari = /^((?!chrome|android).)*safari/i.test(navigator.userAgent)
+const isSafari = () => /^((?!chrome|android).)*safari/i.test(navigator.userAgent)
 
 export class CookieStore<T> implements Store<T> {
 	private cachedValue: T | undefined
@@ -92,7 +92,7 @@ export class CookieStore<T> implements Store<T> {
 		if (isIframe()) {
 			// Safari does not set cookie when the SameSite attribute is set to None and Secure is set to true in an iframe
 			// It fails also in our unit tests since they are running in iframe and on localhost.
-			if (['localhost', '127.0.0.1'].includes(window.location.hostname) && isSafari) {
+			if (['localhost', '127.0.0.1'].includes(window.location.hostname) && isSafari()) {
 				rawCookie += ';SameSite=None'
 			} else {
 				rawCookie += ';SameSite=None; Secure'