Fix global access in browsers (#178)

global is only available on Node, use window on browsers instead.
wix-incubator · Sep 21, 2024 · 3236a31 · 3236a31
1 parent 5e3b5a7
commit 3236a31
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/packages/react-obsidian/src/graph/registry/GraphRegistry.ts b/packages/react-obsidian/src/graph/registry/GraphRegistry.ts
@@ -3,6 +3,7 @@ import { Graph } from '../Graph';
 import { Middleware } from './Middleware';
 import GraphMiddlewareChain from './GraphMiddlewareChain';
 import { ObtainLifecycleBoundGraphException } from './ObtainLifecycleBoundGraphException';
+import { getGlobal } from '../../utils/getGlobal';
 
 export class GraphRegistry {
   private readonly constructorToInstance = new Map<Constructable<Graph>, Set<Graph>>();
@@ -151,7 +152,6 @@ export class GraphRegistry {
   }
 }
 
-// @ts-ignore
-global.graphRegistry = global.graphRegistry || new GraphRegistry();
-// @ts-ignore
-export default global.graphRegistry as GraphRegistry;
+const globalObject = getGlobal();
+globalObject.graphRegistry = globalObject.graphRegistry || new GraphRegistry();
+export default globalObject.graphRegistry as GraphRegistry;
diff --git a/packages/react-obsidian/src/utils/getGlobal.ts b/packages/react-obsidian/src/utils/getGlobal.ts
@@ -0,0 +1,3 @@
+export function getGlobal(): any {
+  return typeof window !== 'undefined' ? window : global;
+}