fix(formatter): is_long_curried_call cannot handle cases where the parent node is not a CallExpression

Author: Dunqing

crates/oxc_formatter/src/utils/member_chain/mod.rs

@@ -201,7 +201,7 @@ impl<'a> Format<'a> for MemberChain<'a, '_> {
         });
 
         if self.tail.len() <= 1 && !has_comments {
-            return if is_long_curried_call(self.root.parent) {
+            return if is_long_curried_call(self.root) {
                 write!(f, [format_one_line])
             } else if is_test_call_expression(self.root) && self.head.members().len() >= 2 {
                 write!(f, [self.head, soft_line_indent_or_space(&self.tail)])

crates/oxc_formatter/src/utils/mod.rs

@@ -8,7 +8,7 @@ use oxc_ast::{AstKind, ast::CallExpression};
 use crate::{
     Format, FormatResult, FormatTrailingCommas, format_args,
     formatter::{Formatter, prelude::soft_line_break_or_space},
-    generated::ast_nodes::AstNodes,
+    generated::ast_nodes::{AstNode, AstNodes},
 };
 
 /// This function is in charge to format the call arguments.
@@ -40,12 +40,10 @@ where
 /// ```javascript
 /// `connect(a, b, c)(d)`
 /// ```
-pub fn is_long_curried_call(parent: &AstNodes<'_>) -> bool {
-    if let AstNodes::CallExpression(call) = parent {
-        if let AstNodes::CallExpression(parent_call) = call.parent {
-            return call.arguments().len() > parent_call.arguments().len()
-                && !parent_call.arguments().is_empty();
-        }
+pub fn is_long_curried_call(call: &AstNode<'_, CallExpression<'_>>) -> bool {
+    if let AstNodes::CallExpression(parent_call) = call.parent {
+        return call.arguments().len() > parent_call.arguments().len()
+            && !parent_call.arguments().is_empty();
     }
 
     false

crates/oxc_formatter/src/write/call_arguments.rs

@@ -56,12 +56,14 @@ impl<'a> Format<'a> for AstNode<'a, ArenaVec<'a, Argument<'a>>> {
             );
         }
 
-        let (is_commonjs_or_amd_call, is_test_call) =
-            if let AstNodes::CallExpression(call) = self.parent {
-                (is_commonjs_or_amd_call(self, call), is_test_call_expression(call))
-            } else {
-                (false, false)
-            };
+        let call_expression =
+            if let AstNodes::CallExpression(call) = self.parent { Some(call) } else { None };
+
+        let (is_commonjs_or_amd_call, is_test_call) = if let Some(call) = call_expression {
+            (is_commonjs_or_amd_call(self, call), is_test_call_expression(call))
+        } else {
+            (false, false)
+        };
 
         let is_first_arg_string_literal_or_template = self.len() != 2
             || matches!(
@@ -105,7 +107,7 @@ impl<'a> Format<'a> for AstNode<'a, ArenaVec<'a, Argument<'a>>> {
 
         if let Some(group_layout) = arguments_grouped_layout(self, f) {
             write_grouped_arguments(self, group_layout, f)
-        } else if is_long_curried_call(self.parent) {
+        } else if call_expression.is_some_and(|call| is_long_curried_call(call)) {
             write!(
                 f,
                 [