Avoid putting symbols in global to fix incompatibility with Temporal …

…Sandbox. (#4196)

.changeset/sweet-bears-help.md

@@ -0,0 +1,7 @@
+---
+"effect": patch
+---
+
+Avoid putting symbols in global to fix incompatibility with Temporal Sandbox.
+
+After speaking with James Watkins-Harvey we realized current Effect escapes the Temporal Worker sandbox that doesn't look for symbols when restoring global context in the isolate they create leading to memory leaks.

packages/effect/src/GlobalValue.ts

@@ -14,13 +14,9 @@
  */
 import * as version from "./internal/version.js"
 
-const globalStoreId = Symbol.for(`effect/GlobalValue/globalStoreId/${version.getCurrentVersion()}`)
+const globalStoreId = `effect/GlobalValue/globalStoreId/${version.getCurrentVersion()}`
 
-if (!(globalStoreId in globalThis)) {
-  ;(globalThis as any)[globalStoreId] = new Map()
-}
-
-const globalStore = (globalThis as any)[globalStoreId] as Map<unknown, any>
+let globalStore: Map<unknown, any>
 
 /**
  * Retrieves or computes a global value associated with the given `id`. If the value for this `id`
@@ -46,6 +42,12 @@ const globalStore = (globalThis as any)[globalStoreId] as Map<unknown, any>
  * @since 2.0.0
  */
 export const globalValue = <A>(id: unknown, compute: () => A): A => {
+  if (!globalStore) {
+    // @ts-expect-error
+    globalThis[globalStoreId] ??= new Map()
+    // @ts-expect-error
+    globalStore = globalThis[globalStoreId] as Map<unknown, any>
+  }
   if (!globalStore.has(id)) {
     globalStore.set(id, compute())
   }