babel-plugin-react-compiler 19.0.0-beta-decd7b8-20250118 → 19.0.0-beta-27714ef-20250124

package/dist/index.js

@@ -121306,8 +121306,16 @@ addObject(BUILTIN_SHAPES, BuiltInMixedReadonlyId, [
   [
     "map",
     addFunction(BUILTIN_SHAPES, [], {
+      /**
+       * Note `map`'s arguments are annotated as Effect.ConditionallyMutate as
+       * calling `<array>.map(fn)` might invoke `fn`, which means replaying its
+       * effects.
+       *
+       * (Note that Effect.Read / Effect.Capture on a function type means
+       * potential data dependency or aliasing respectively.)
+       */
       positionalParams: [],
-      restParam: "read" /* Read */,
+      restParam: "mutate?" /* ConditionallyMutate */,
       returnType: { kind: "Object", shapeId: BuiltInArrayId },
       calleeEffect: "mutate?" /* ConditionallyMutate */,
       returnValueKind: "mutable" /* Mutable */,
@@ -121318,7 +121326,7 @@ addObject(BUILTIN_SHAPES, BuiltInMixedReadonlyId, [
     "flatMap",
     addFunction(BUILTIN_SHAPES, [], {
       positionalParams: [],
-      restParam: "read" /* Read */,
+      restParam: "mutate?" /* ConditionallyMutate */,
       returnType: { kind: "Object", shapeId: BuiltInArrayId },
       calleeEffect: "mutate?" /* ConditionallyMutate */,
       returnValueKind: "mutable" /* Mutable */,
@@ -121329,7 +121337,7 @@ addObject(BUILTIN_SHAPES, BuiltInMixedReadonlyId, [
     "filter",
     addFunction(BUILTIN_SHAPES, [], {
       positionalParams: [],
-      restParam: "read" /* Read */,
+      restParam: "mutate?" /* ConditionallyMutate */,
       returnType: { kind: "Object", shapeId: BuiltInArrayId },
       calleeEffect: "mutate?" /* ConditionallyMutate */,
       returnValueKind: "mutable" /* Mutable */,
@@ -133744,7 +133752,7 @@ function codegenInstructionValue(cx, instrValue) {
   }
   return value;
 }
-var STRING_REQUIRES_EXPR_CONTAINER_PATTERN = /[\u{0000}-\u{001F}\u{007F}\u{0080}-\u{FFFF}]|"/u;
+var STRING_REQUIRES_EXPR_CONTAINER_PATTERN = /[\u{0000}-\u{001F}\u{007F}\u{0080}-\u{FFFF}]|"|\\/u;
 function codegenJsxAttribute(cx, attribute) {
   switch (attribute.kind) {
     case "JsxAttribute": {
@@ -133789,7 +133797,7 @@ function codegenJsxAttribute(cx, attribute) {
     }
   }
 }
-var JSX_TEXT_CHILD_REQUIRES_EXPR_CONTAINER_PATTERN = /[<>&]/;
+var JSX_TEXT_CHILD_REQUIRES_EXPR_CONTAINER_PATTERN = /[<>&{}]/;
 function codegenJsxElement(cx, place) {
   const value = codegenPlace(cx, place);
   switch (value.type) {
@@ -134985,11 +134993,15 @@ function inferOperandEffect(state, place) {
       if (isRefOrRefValue(place.identifier)) {
         break;
       } else if (value.kind === "context" /* Context */) {
+        CompilerError.invariant(value.context.size > 0, {
+          reason: "[InferFunctionEffects] Expected Context-kind value's capture list to be non-empty.",
+          loc: place.loc
+        });
         return {
           kind: "ContextMutation",
           loc: place.loc,
           effect: place.effect,
-          places: value.context.size === 0 ? /* @__PURE__ */ new Set([place]) : value.context
+          places: value.context
         };
       } else if (value.kind !== "mutable" /* Mutable */ && // We ignore mutations of primitives since this is not a React-specific problem
       value.kind !== "primitive" /* Primitive */) {
@@ -135744,10 +135756,11 @@ function inferBlock(env, state, block, functionEffects) {
         break;
       }
       case "ArrayExpression": {
-        const valueKind = hasContextRefOperand(state, instrValue) ? {
+        const contextRefOperands = getContextRefOperand(state, instrValue);
+        const valueKind = contextRefOperands.length > 0 ? {
           kind: "context" /* Context */,
           reason: /* @__PURE__ */ new Set(["other" /* Other */]),
-          context: /* @__PURE__ */ new Set()
+          context: new Set(contextRefOperands)
         } : {
           kind: "mutable" /* Mutable */,
           reason: /* @__PURE__ */ new Set(["other" /* Other */]),
@@ -135788,10 +135801,11 @@ function inferBlock(env, state, block, functionEffects) {
         break;
       }
       case "ObjectExpression": {
-        const valueKind = hasContextRefOperand(state, instrValue) ? {
+        const contextRefOperands = getContextRefOperand(state, instrValue);
+        const valueKind = contextRefOperands.length > 0 ? {
           kind: "context" /* Context */,
           reason: /* @__PURE__ */ new Set(["other" /* Other */]),
-          context: /* @__PURE__ */ new Set()
+          context: new Set(contextRefOperands)
         } : {
           kind: "mutable" /* Mutable */,
           reason: /* @__PURE__ */ new Set(["other" /* Other */]),
@@ -136381,11 +136395,17 @@ function inferBlock(env, state, block, functionEffects) {
       }
       case "LoadLocal": {
         const lvalue = instr.lvalue;
-        const effect = state.isDefined(lvalue) && state.kind(lvalue).kind === "context" /* Context */ ? "mutate?" /* ConditionallyMutate */ : "capture" /* Capture */;
+        CompilerError.invariant(
+          !(state.isDefined(lvalue) && state.kind(lvalue).kind === "context" /* Context */),
+          {
+            reason: "[InferReferenceEffects] Unexpected LoadLocal with context kind",
+            loc: lvalue.loc
+          }
+        );
         state.referenceAndRecordEffects(
           freezeActions,
           instrValue.place,
-          effect,
+          "capture" /* Capture */,
           "other" /* Other */
         );
         lvalue.effect = "mutate?" /* ConditionallyMutate */;
@@ -136639,13 +136659,14 @@ function inferBlock(env, state, block, functionEffects) {
     ({ values, reason }) => state.freezeValues(values, reason)
   );
 }
-function hasContextRefOperand(state, instrValue) {
+function getContextRefOperand(state, instrValue) {
+  const result = [];
   for (const place of eachInstructionValueOperand(instrValue)) {
     if (state.isDefined(place) && state.kind(place).kind === "context" /* Context */) {
-      return true;
+      result.push(place);
     }
   }
-  return false;
+  return result;
 }
 function getFunctionCallSignature(env, type) {
   if (type.kind !== "Function") {