fix(ecmascript): skip array length evaluation if there are any spread elements

crates/oxc_ecmascript/src/constant_evaluation/mod.rs

@@ -495,15 +495,7 @@ impl<'a> ConstantEvaluation<'a> for StaticMemberExpression<'a> {
         _target_ty: Option<ValueType>,
     ) -> Option<ConstantValue<'a>> {
         match self.property.name.as_str() {
-            "length" => {
-                if let Some(ConstantValue::String(s)) = self.object.evaluate_value(ctx) {
-                    Some(ConstantValue::Number(s.encode_utf16().count().to_f64().unwrap()))
-                } else if let Expression::ArrayExpression(arr) = &self.object {
-                    Some(ConstantValue::Number(arr.elements.len().to_f64().unwrap()))
-                } else {
-                    None
-                }
-            }
+            "length" => evaluate_value_length(&self.object, ctx),
             _ => None,
         }
     }
@@ -517,19 +509,30 @@ impl<'a> ConstantEvaluation<'a> for ComputedMemberExpression<'a> {
     ) -> Option<ConstantValue<'a>> {
         match &self.expression {
             Expression::StringLiteral(s) if s.value == "length" => {
-                if let Some(ConstantValue::String(s)) = self.object.evaluate_value(ctx) {
-                    Some(ConstantValue::Number(s.encode_utf16().count().to_f64().unwrap()))
-                } else if let Expression::ArrayExpression(arr) = &self.object {
-                    Some(ConstantValue::Number(arr.elements.len().to_f64().unwrap()))
-                } else {
-                    None
-                }
+                evaluate_value_length(&self.object, ctx)
             }
             _ => None,
         }
     }
 }
 
+fn evaluate_value_length<'a>(
+    object: &Expression<'a>,
+    ctx: &impl ConstantEvaluationCtx<'a>,
+) -> Option<ConstantValue<'a>> {
+    if let Some(ConstantValue::String(s)) = object.evaluate_value(ctx) {
+        Some(ConstantValue::Number(s.encode_utf16().count().to_f64().unwrap()))
+    } else if let Expression::ArrayExpression(arr) = object {
+        if arr.elements.iter().any(|e| matches!(e, ArrayExpressionElement::SpreadElement(_))) {
+            None
+        } else {
+            Some(ConstantValue::Number(arr.elements.len().to_f64().unwrap()))
+        }
+    } else {
+        None
+    }
+}
+
 impl<'a> ConstantEvaluation<'a> for CallExpression<'a> {
     fn evaluate_value_to(
         &self,

crates/oxc_minifier/src/peephole/fold_constants.rs

@@ -1778,6 +1778,7 @@ mod test {
         // Cannot fold
         fold("x = [foo(), 0].length", "x = [foo(),0].length");
         fold_same("x = y.length");
+        fold_same("[...[1, 2, 3]].length");
     }
 
     #[test]