fix: immutable global variables

hmerritt · Jan 28, 2024 · 6e712cb · 6e712cb
1 parent 67e3012
commit 6e712cb
Show file tree

Hide file tree

Showing 5 changed files with 24 additions and 13 deletions.
diff --git a/src/lib/global/devTools.ts b/src/lib/global/devTools.ts
@@ -1,4 +1,4 @@
-import { $global } from "./utils";
+import { setGlobalValue } from "./utils";
 
 // @TODO maybe remove these and add better dev tools.
 
@@ -41,6 +41,6 @@ export const getObjectOfEventListeners = () => {
 };
 
 export const injectDevTools = () => {
-	$global.getNumberOfEventListeners = getNumberOfEventListeners;
-	$global.getObjectOfEventListeners = getObjectOfEventListeners;
+	setGlobalValue("getNumberOfEventListeners", getNumberOfEventListeners);
+	setGlobalValue("getObjectOfEventListeners", getObjectOfEventListeners);
 };
diff --git a/src/lib/global/env.ts b/src/lib/global/env.ts
@@ -1,4 +1,4 @@
-import { $global } from "./utils";
+import { setGlobalValue } from "./utils";
 
 /**
  * Environment variables.
@@ -29,5 +29,5 @@ export type EnvObj = typeof env;
 export type EnvKeys = keyof EnvObj;
 
 export const injectEnv = () => {
-	$global.env = env;
+	setGlobalValue("env", env);
 };
diff --git a/src/lib/global/featureFlags.ts b/src/lib/global/featureFlags.ts
@@ -1,5 +1,5 @@
 import { type EnvKeys, env } from "./env";
-import { $global } from "./utils";
+import { setGlobalValue } from "./utils";
 
 /**
  * Returns `true` if the feature is enabled in `env` object.
@@ -32,7 +32,7 @@ export type FeatureOptions = {
 export type FeatureFlags = EnvKeys;
 
 export const injectFeature = () => {
-	$global.feature = feature;
+	setGlobalValue("feature", feature);
 };
 
 const isFalse = (value: unknown): value is false => {

diff --git a/src/lib/global/log.ts b/src/lib/global/log.ts
@@ -2,7 +2,7 @@ import dayjs from "dayjs";
 
 import { padChar } from "lib/strings";
 
-import { $global } from "./utils";
+import { $global, setGlobalValue } from "./utils";
 
 // @TODO:
 // export interface Logger {
@@ -141,9 +141,9 @@ export const debugn = (namespace: string, logLevel: any, ...args: any[]) => {
 };
 
 export const injectLog = () => {
-	$global.logStore = new LogStore();
-	$global.log = log;
-	$global.logn = logn;
-	$global.debug = debug;
-	$global.debugn = debugn;
+	setGlobalValue("logStore", new LogStore());
+	setGlobalValue("log", log);
+	setGlobalValue("logn", logn);
+	setGlobalValue("debug", debug);
+	setGlobalValue("debugn", debugn);
 };
diff --git a/src/lib/global/utils.ts b/src/lib/global/utils.ts
@@ -13,4 +13,15 @@ export const getGlobal = () => {
 	}
 };
 
+/**
+ * Set immutable global variable.
+ */
+export const setGlobalValue = (key: string, value: any) => {
+	Object.defineProperty(getGlobal(), key, {
+		value: value,
+		configurable: false,
+		writable: false
+	});
+};
+
 export const $global = getGlobal();