[compiler] _exp version of ValidateNoDerivedComputationsInEffects take precedence over stable version when enabled (facebook#35099)

jorge-cab · web-flow · commit 5e94655cbbea · 2025-11-11T10:16:20.000-08:00
Summary: We should only run one version of the validation. I think it makes sense that if the exp version is enable it takes precedence over the stable one --- [//]: # (BEGIN SAPLING FOOTER) Stack created with [Sapling](https://sapling-scm.com). Best reviewed with [ReviewStack](https://reviewstack.dev/facebook/react/pull/35099). * __->__ facebook#35099 * facebook#35100
diff --git a/compiler/packages/babel-plugin-react-compiler/src/Entrypoint/Pipeline.ts b/compiler/packages/babel-plugin-react-compiler/src/Entrypoint/Pipeline.ts
@@ -272,12 +272,10 @@ function runWithEnvironment(
       validateNoSetStateInRender(hir).unwrap();
     }
 
-    if (env.config.validateNoDerivedComputationsInEffects) {
-      validateNoDerivedComputationsInEffects(hir);
-    }
-
     if (env.config.validateNoDerivedComputationsInEffects_exp) {
       env.logErrors(validateNoDerivedComputationsInEffects_exp(hir));
+    } else if (env.config.validateNoDerivedComputationsInEffects) {
+      validateNoDerivedComputationsInEffects(hir);
     }
 
     if (env.config.validateNoSetStateInEffects) {

Original file line number	Diff line number	Diff line change
`@@ -272,12 +272,10 @@ function runWithEnvironment(`
`272`	`272`	`validateNoSetStateInRender(hir).unwrap();`
`273`	`273`	`}`
`274`	`274`
`275`		`- if (env.config.validateNoDerivedComputationsInEffects) {`
`276`		`- validateNoDerivedComputationsInEffects(hir);`
`277`		`- }`
`278`		`-`
`279`	`275`	`if (env.config.validateNoDerivedComputationsInEffects_exp) {`
`280`	`276`	`env.logErrors(validateNoDerivedComputationsInEffects_exp(hir));`
	`277`	`+ } else if (env.config.validateNoDerivedComputationsInEffects) {`
	`278`	`+ validateNoDerivedComputationsInEffects(hir);`
`281`	`279`	`}`
`282`	`280`
`283`	`281`	`if (env.config.validateNoSetStateInEffects) {`