Move NODE_ENV check up so debug checks are eliminated from prod

reduxjs · Oct 28, 2023 · 9d7290f · 9d7290f
1 parent 9f133f9
commit 9d7290f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/src/createSelectorCreator.ts b/src/createSelectorCreator.ts
@@ -386,7 +386,10 @@ export function createSelectorCreator<
         arguments
       )
 
-      if (shouldRunInputStabilityCheck(inputStabilityCheck, firstRun)) {
+      if (
+        process.env.NODE_ENV !== 'production' &&
+        shouldRunInputStabilityCheck(inputStabilityCheck, firstRun)
+      ) {
         // make a second copy of the params, to check if we got the same results
         const inputSelectorResultsCopy = collectInputSelectorResults(
           dependencies,

diff --git a/src/utils.ts b/src/utils.ts
@@ -144,9 +144,7 @@ export const shouldRunInputStabilityCheck = (
   firstRun: boolean
 ) => {
   return (
-    process.env.NODE_ENV !== 'production' &&
-    (inputStabilityCheck === 'always' ||
-      (inputStabilityCheck === 'once' && firstRun))
+    inputStabilityCheck === 'always' ||
+    (inputStabilityCheck === 'once' && firstRun)
   )
 }
-