Update on "[compiler][patch] Fix jsx non-ascii regex pattern"

[ghstack-poisoned]

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

@@ -2206,7 +2206,7 @@ function codegenInstructionValue(
  * https://en.wikipedia.org/wiki/List_of_Unicode_characters#Control_codes
  */
 const STRING_REQUIRES_EXPR_CONTAINER_PATTERN =
-  /[\u{0000}-\u{001F}|\u{007F}|\u{0080}-\u{FFFF}]|"/u;
+  /[\u{0000}-\u{001F}\u{007F}\u{0080}-\u{FFFF}]|"/u;
 function codegenJsxAttribute(
   cx: Context,
   attribute: JsxAttribute

compiler/packages/babel-plugin-react-compiler/src/__tests__/fixtures/compiler/fbt/error.todo-fbt-param-with-unicode.expect.md

@@ -0,0 +1,30 @@
+
+## Input
+
+```javascript
+import fbt from "fbt";
+
+function Component(props) {
+  const element = (
+    <fbt desc={"Dialog to show to user"}>
+      Hello <fbt:param name="user name ☺">{props.name}</fbt:param>
+    </fbt>
+  );
+  return element.toString();
+}
+
+export const FIXTURE_ENTRYPOINT = {
+  fn: Component,
+  params: [{ name: "Jason" }],
+};
+
+```
+
+
+## Error
+
+```
+Property arguments[0] of CallExpression expected node to be of a type ["Expression","SpreadElement","JSXNamespacedName","ArgumentPlaceholder"] but instead got "JSXExpressionContainer"
+```
+          
+      

compiler/packages/babel-plugin-react-compiler/src/__tests__/fixtures/compiler/fbt/error.todo-fbt-param-with-unicode.js

@@ -0,0 +1,15 @@
+import fbt from "fbt";
+
+function Component(props) {
+  const element = (
+    <fbt desc={"Dialog to show to user"}>
+      Hello <fbt:param name="user name ☺">{props.name}</fbt:param>
+    </fbt>
+  );
+  return element.toString();
+}
+
+export const FIXTURE_ENTRYPOINT = {
+  fn: Component,
+  params: [{ name: "Jason" }],
+};

compiler/packages/babel-plugin-react-compiler/src/__tests__/fixtures/compiler/fbt/fbt-params.expect.md

@@ -6,9 +6,14 @@ import fbt from "fbt";
 
 function Component(props) {
   return (
-    <fbt desc={"Dialog to show to user"}>
-      Hello <fbt:param name="user name">{props.name}</fbt:param>
-    </fbt>
+    <div>
+      <fbt desc={"Dialog to show to user"}>
+        Hello <fbt:param name="user name">{props.name}</fbt:param>
+      </fbt>
+      <fbt desc={"Available actions|response"}>
+        <fbt:param name="actions|response">{props.actions}</fbt:param>
+      </fbt>
+    </div>
   );
 }
 
@@ -27,7 +32,7 @@ import { c as _c } from "react/compiler-runtime";
 import fbt from "fbt";
 
 function Component(props) {
-  const $ = _c(2);
+  const $ = _c(7);
   let t0;
   if ($[0] !== props.name) {
     t0 = fbt._("Hello {user name}", [fbt._param("user name", props.name)], {
@@ -38,7 +43,33 @@ function Component(props) {
   } else {
     t0 = $[1];
   }
-  return t0;
+  let t1;
+  if ($[2] !== props.actions) {
+    t1 = fbt._(
+      "{actions|response}",
+      [fbt._param("actions|response", props.actions)],
+      { hk: "1cjfbg" },
+    );
+    $[2] = props.actions;
+    $[3] = t1;
+  } else {
+    t1 = $[3];
+  }
+  let t2;
+  if ($[4] !== t0 || $[5] !== t1) {
+    t2 = (
+      <div>
+        {t0}
+        {t1}
+      </div>
+    );
+    $[4] = t0;
+    $[5] = t1;
+    $[6] = t2;
+  } else {
+    t2 = $[6];
+  }
+  return t2;
 }
 
 export const FIXTURE_ENTRYPOINT = {

compiler/packages/babel-plugin-react-compiler/src/__tests__/fixtures/compiler/fbt/fbt-params.js

@@ -2,9 +2,14 @@ import fbt from "fbt";
 
 function Component(props) {
   return (
-    <fbt desc={"Dialog to show to user"}>
-      Hello <fbt:param name="user name">{props.name}</fbt:param>
-    </fbt>
+    <div>
+      <fbt desc={"Dialog to show to user"}>
+        Hello <fbt:param name="user name">{props.name}</fbt:param>
+      </fbt>
+      <fbt desc={"Available actions|response"}>
+        <fbt:param name="actions|response">{props.actions}</fbt:param>
+      </fbt>
+    </div>
   );
 }