fix(parser): fix failed to parse JSXChild after JSXEmptyExpression (#2726)

fixes #2723

crates/oxc_parser/Cargo.toml

@@ -34,7 +34,7 @@ seq-macro        = { workspace = true }
 memchr = "2.7.1"
 
 [dev-dependencies]
-oxc_ast    = { workspace = true }
+oxc_ast    = { workspace = true, features = ["serialize"] }
 miette     = { workspace = true }
 serde_json = { workspace = true }
 ouroboros  = "0.18.3"             # for `multi-thread` example

crates/oxc_parser/src/jsx/mod.rs

@@ -215,7 +215,7 @@ impl<'a> ParserImpl<'a> {
             }
             // {expr}
             Kind::LCurly => self
-                .parse_jsx_expression_container(true)
+                .parse_jsx_expression_container(/* is_jsx_child */ true)
                 .map(JSXChild::ExpressionContainer)
                 .map(Some),
             // text
@@ -232,13 +232,17 @@ impl<'a> ParserImpl<'a> {
         let span = self.start_span();
         self.bump_any(); // bump `{`
 
-        let expr = if self.eat(Kind::RCurly) {
+        let expr = if self.at(Kind::RCurly) {
+            if in_jsx_child {
+                self.expect_jsx_child(Kind::RCurly)
+            } else {
+                self.expect(Kind::RCurly)
+            }?;
+            let span = self.end_span(span);
             // Handle comment between curly braces (ex. `{/* comment */}`)
             //                                            ^^^^^^^^^^^^^ span
-            let span = self.end_span(span);
+            let expr = self.ast.jsx_empty_expression(Span::new(span.start + 1, span.end - 1));
-            JSXExpression::EmptyExpression(
+            JSXExpression::EmptyExpression(expr)
-                self.ast.jsx_empty_expression(Span::new(span.start + 1, span.end - 1)),
-            )
         } else {
             let expr = self.parse_jsx_assignment_expression().map(JSXExpression::Expression)?;
             if in_jsx_child {
@@ -339,7 +343,7 @@ impl<'a> ParserImpl<'a> {
         match self.cur_kind() {
             Kind::Str => self.parse_literal_string().map(JSXAttributeValue::StringLiteral),
             Kind::LCurly => {
-                let expr = self.parse_jsx_expression_container(false)?;
+                let expr = self.parse_jsx_expression_container(/* is_jsx_child */ false)?;
                 Ok(JSXAttributeValue::ExpressionContainer(expr))
             }
             Kind::LAngle => {

tasks/coverage/codegen_misc.snap

@@ -1,3 +1,3 @@
 codegen_misc Summary:
-AST Parsed     : 14/14 (100.00%)
+AST Parsed     : 15/15 (100.00%)
-Positive Passed: 14/14 (100.00%)
+Positive Passed: 15/15 (100.00%)

tasks/coverage/misc/pass/oxc-2723.jsx

@@ -0,0 +1,13 @@
+const A = (
+  <div>
+    {/* comment */}
+    AAAA
+  </div>
+);
+
+const B = (
+  <div>
+    BBBB
+    {/* comment */}
+  </div>
+);

tasks/coverage/parser_misc.snap

@@ -1,6 +1,6 @@
 parser_misc Summary:
-AST Parsed     : 14/14 (100.00%)
+AST Parsed     : 15/15 (100.00%)
-Positive Passed: 14/14 (100.00%)
+Positive Passed: 15/15 (100.00%)
 Negative Passed: 8/8 (100.00%)
 
   × Unexpected token

tasks/coverage/prettier_misc.snap

@@ -1,9 +1,10 @@
 prettier_misc Summary:
-AST Parsed     : 14/14 (100.00%)
+AST Parsed     : 15/15 (100.00%)
-Positive Passed: 8/14 (57.14%)
+Positive Passed: 8/15 (53.33%)
 Expect to Parse: "pass/oxc-1740.tsx"
 Expect to Parse: "pass/oxc-2087.ts"
 Expect to Parse: "pass/oxc-2394.ts"
 Expect to Parse: "pass/oxc-2674.tsx"
+Expect to Parse: "pass/oxc-2723.jsx"
 Expect to Parse: "pass/swc-1627.js"
 Expect to Parse: "pass/swc-8243.tsx"