babel-plugin-react-compiler 0.0.0-experimental-3ab621d-20241203 → 0.0.0-experimental-37ed2a7-20241204

package/dist/index.js

@@ -151537,6 +151537,7 @@ function inferEffectDependencies(fn) {
   }
   const autodepFnLoads = new Map();
   const scopeInfos = new Map();
+  const loadGlobals = new Set();
   const reactiveIds = inferReactiveIdentifiers(fn);
   for (const [, block] of fn.body.blocks) {
     if (
@@ -151558,26 +151559,22 @@ function inferEffectDependencies(fn) {
       const {value: value, lvalue: lvalue} = instr;
       if (value.kind === 'FunctionExpression') {
         fnExpressions.set(lvalue.identifier.id, instr);
-      } else if (
-        value.kind === 'LoadGlobal' &&
-        value.binding.kind === 'ImportSpecifier'
-      ) {
-        const moduleTargets = autodepFnConfigs.get(value.binding.module);
-        if (moduleTargets != null) {
-          const numRequiredArgs = moduleTargets.get(value.binding.imported);
-          if (numRequiredArgs != null) {
-            autodepFnLoads.set(lvalue.identifier.id, numRequiredArgs);
-          }
-        }
-      } else if (
-        value.kind === 'LoadGlobal' &&
-        value.binding.kind === 'ImportDefault'
-      ) {
-        const moduleTargets = autodepFnConfigs.get(value.binding.module);
-        if (moduleTargets != null) {
-          const numRequiredArgs = moduleTargets.get(DEFAULT_EXPORT);
-          if (numRequiredArgs != null) {
-            autodepFnLoads.set(lvalue.identifier.id, numRequiredArgs);
+      } else if (value.kind === 'LoadGlobal') {
+        loadGlobals.add(lvalue.identifier.id);
+        if (
+          value.binding.kind === 'ImportSpecifier' ||
+          value.binding.kind === 'ImportDefault'
+        ) {
+          const moduleTargets = autodepFnConfigs.get(value.binding.module);
+          if (moduleTargets != null) {
+            const importSpecifierName =
+              value.binding.kind === 'ImportSpecifier'
+                ? value.binding.imported
+                : DEFAULT_EXPORT;
+            const numRequiredArgs = moduleTargets.get(importSpecifierName);
+            if (numRequiredArgs != null) {
+              autodepFnLoads.set(lvalue.identifier.id, numRequiredArgs);
+            }
           }
         }
       } else if (
@@ -151585,6 +151582,15 @@ function inferEffectDependencies(fn) {
         autodepFnLoads.get(value.callee.identifier.id) === value.args.length &&
         value.args[0].kind === 'Identifier'
       ) {
+        const effectDeps = [];
+        const newInstructions = [];
+        const deps = {
+          kind: 'ArrayExpression',
+          elements: effectDeps,
+          loc: GeneratedSource,
+        };
+        const depsPlace = createTemporaryPlace(fn.env, GeneratedSource);
+        depsPlace.effect = exports.Effect.Read;
         const fnExpr = fnExpressions.get(value.args[0].identifier.id);
         if (fnExpr != null) {
           const scopeInfo =
@@ -151602,8 +151608,6 @@ function inferEffectDependencies(fn) {
               loc: fnExpr.loc,
             });
           }
-          const effectDeps = [];
-          const newInstructions = [];
           for (const dep of scopeInfo.deps) {
             const {place: place, instructions: instructions} =
               writeDependencyToInstructions(
@@ -151615,13 +151619,21 @@ function inferEffectDependencies(fn) {
             newInstructions.push(...instructions);
             effectDeps.push(place);
           }
-          const deps = {
-            kind: 'ArrayExpression',
-            elements: effectDeps,
+          newInstructions.push({
+            id: makeInstructionId(0),
             loc: GeneratedSource,
-          };
-          const depsPlace = createTemporaryPlace(fn.env, GeneratedSource);
-          depsPlace.effect = exports.Effect.Read;
+            lvalue: Object.assign(Object.assign({}, depsPlace), {
+              effect: exports.Effect.Mutate,
+            }),
+            value: deps,
+          });
+          value.args.push(
+            Object.assign(Object.assign({}, depsPlace), {
+              effect: exports.Effect.Freeze,
+            })
+          );
+          rewriteInstrs.set(instr.id, newInstructions);
+        } else if (loadGlobals.has(value.args[0].identifier.id)) {
           newInstructions.push({
             id: makeInstructionId(0),
             loc: GeneratedSource,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "babel-plugin-react-compiler",
-  "version": "0.0.0-experimental-3ab621d-20241203",
+  "version": "0.0.0-experimental-37ed2a7-20241204",
   "description": "Babel plugin for React Compiler.",
   "main": "dist/index.js",
   "license": "MIT",