fix: check global process at module level

dai-shi · Aug 7, 2024 · 9c850b3 · 9c850b3
1 parent 610367e
commit 9c850b3
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/src/createContainer.ts b/src/createContainer.ts
@@ -16,6 +16,8 @@ import type { Context } from 'use-context-selector';
 
 import { createTrackedSelector } from './createTrackedSelector.js';
 
+const hasGlobalProcess = typeof process === 'object';
+
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 type AnyFunction = (...args: any[]) => any;
 type Options<State, Update extends AnyFunction> = {
@@ -69,7 +71,7 @@ export const createContainer = <State, Update extends AnyFunction, Props>(
   };
 
   const useSelector = <Selected>(selector: (state: State) => Selected) => {
-    if (typeof process === 'object' && process.env.NODE_ENV !== 'production') {
+    if (hasGlobalProcess && process.env.NODE_ENV !== 'production') {
       const selectorOrig = selector;
       selector = (state: State) => {
         if (state === undefined) {
@@ -91,7 +93,7 @@ export const createContainer = <State, Update extends AnyFunction, Props>(
   const useUpdate = concurrentMode
     ? () => {
         if (
-          typeof process === 'object' &&
+          hasGlobalProcess &&
           process.env.NODE_ENV !== 'production' &&
           useContextOrig(UpdateContext) === undefined
         ) {

diff --git a/src/createTrackedSelector.ts b/src/createTrackedSelector.ts
@@ -3,6 +3,8 @@ import { createProxy, isChanged } from 'proxy-compare';
 
 import { useAffectedDebugValue } from './utils.js';
 
+const hasGlobalProcess = typeof process === 'object';
+
 export const createTrackedSelector = <State>(
   useSelector: <Selected>(selector: (state: State) => Selected) => Selected,
 ) => {
@@ -38,7 +40,7 @@ export const createTrackedSelector = <State>(
       [affected],
     );
     const state = useSelector(selector);
-    if (typeof process === 'object' && process.env.NODE_ENV !== 'production') {
+    if (hasGlobalProcess && process.env.NODE_ENV !== 'production') {
       // eslint-disable-next-line react-hooks/rules-of-hooks
       useAffectedDebugValue(state, affected);
     }