Update on "[compiler] Fix mode for generating scopes for reassignments"

We have an experimental mode where we generate scopes for simple phi values, even if they aren't subsequently mutated. This mode was incorrectly generating scope ranges, leaving the start at 0 which is invalid. The fix is to allow non-zero identifier ranges to overwrite the scope start (rather than taking the min) if the scope start is still zero.

[ghstack-poisoned]

Author: josephsavona

compiler/packages/babel-plugin-react-compiler/src/ReactiveScopes/InferReactiveScopeVariables.ts

@@ -114,12 +114,12 @@ export function inferReactiveScopeVariables(fn: HIRFunction): void {
       };
       scopes.set(groupIdentifier, scope);
     } else {
-      if (identifier.mutableRange.start !== 0 && scope.range.start !== 0) {
+      if (scope.range.start === 0) {
+        scope.range.start = identifier.mutableRange.start;
+      } else if (identifier.mutableRange.start !== 0) {
         scope.range.start = makeInstructionId(
           Math.min(scope.range.start, identifier.mutableRange.start)
         );
-      } else if (identifier.mutableRange.start !== 0) {
-        scope.range.start = identifier.mutableRange.start;
       }
       scope.range.end = makeInstructionId(
         Math.max(scope.range.end, identifier.mutableRange.end)