fix: use global constants (#290)

* fix: use global constants

* PR review

change/@griffel-core-cb8022d0-7662-4724-a582-c6536e4413fc.json

@@ -0,0 +1,7 @@
+{
+  "type": "patch",
+  "comment": "fix: use global constants",
+  "packageName": "@griffel/core",
+  "email": "[email protected]",
+  "dependentChangeType": "patch"
+}

packages/core/src/constants.test.ts

@@ -0,0 +1,14 @@
+it('hosts global constants', async () => {
+  const importA = await import('./constants');
+
+  // We are reloading a module, so instances that are created inside a module will be different
+  // https://jestjs.io/docs/jest-object#jestresetmodules
+  jest.resetModules();
+  const importB = await import('./constants');
+
+  expect(importA.DEFINITION_LOOKUP_TABLE).toBe(importA.DEFINITION_LOOKUP_TABLE);
+  expect(importA.DEBUG_RESET_CLASSES).toBe(importA.DEBUG_RESET_CLASSES);
+
+  // This constant is not global, expected to be different
+  expect(importA.UNSUPPORTED_CSS_PROPERTIES).not.toBe(importB.UNSUPPORTED_CSS_PROPERTIES);
+});

packages/core/src/constants.ts

@@ -1,5 +1,31 @@
 import { GriffelStylesUnsupportedCSSProperties, LookupItem, SequenceHash } from './types';
 
+// ----
+
+// Heads up!
+// These constants are global and will be shared between Griffel instances.
+// Any change in them should happen only in a MAJOR version. If it happens,
+// please change "__NAMESPACE_PREFIX__" to include a version.
+
+const __GLOBAL__: any = typeof window === 'undefined' ? global : window;
+const __NAMESPACE_PREFIX__ = '@griffel/';
+
+function getGlobalVar<T>(name: string, defaultValue: T): T {
+  if (!__GLOBAL__[Symbol.for(__NAMESPACE_PREFIX__ + name)]) {
+    __GLOBAL__[Symbol.for(__NAMESPACE_PREFIX__ + name)] = defaultValue;
+  }
+
+  return __GLOBAL__[Symbol.for(__NAMESPACE_PREFIX__ + name)];
+}
+
+/** @internal */
+export const DEBUG_RESET_CLASSES = getGlobalVar<Record<string, 1>>('DEBUG_RESET_CLASSES', {});
+
+/** @internal */
+export const DEFINITION_LOOKUP_TABLE = getGlobalVar<Record<SequenceHash, LookupItem>>('DEFINITION_LOOKUP_TABLE', {});
+
+// ----
+
 /** @internal */
 export const DATA_BUCKET_ATTR = 'data-make-styles-bucket';
 
@@ -24,12 +50,6 @@ export const SEQUENCE_SIZE =
     ? SEQUENCE_PREFIX.length + SEQUENCE_HASH_LENGTH
     : SEQUENCE_PREFIX.length + SEQUENCE_HASH_LENGTH + DEBUG_SEQUENCE_SEPARATOR.length + SEQUENCE_HASH_LENGTH;
 
-/** @internal */
-export const DEBUG_RESET_CLASSES: Record<string, 1> = {};
-
-/** @internal */
-export const DEFINITION_LOOKUP_TABLE: Record<SequenceHash, LookupItem> = {};
-
 // indexes for values in LookupItem tuple
 
 /** @internal */