Fix to not coerce nested arrays to single level when type is any (#7787)

Author: jtran

rust/kcl-lib/src/execution/mod.rs

@@ -1452,7 +1452,12 @@ mod tests {
     use pretty_assertions::assert_eq;
 
     use super::*;
-    use crate::{ModuleId, errors::KclErrorDetails, execution::memory::Stack};
+    use crate::{
+        ModuleId,
+        errors::KclErrorDetails,
+        exec::NumericType,
+        execution::{memory::Stack, types::RuntimeType},
+    };
 
     /// Convenience function to get a JSON value from memory and unwrap.
     #[track_caller]
@@ -1943,6 +1948,40 @@ clone001 = map(extrudes, f = clone)
         parse_execute(ast).await.unwrap();
     }
 
+    #[tokio::test(flavor = "multi_thread")]
+    async fn test_array_reduce_nested_array() {
+        let code = r#"
+fn id(@el, accum)  { return accum }
+
+answer = reduce([], initial=[[[0,0]]], f=id)
+"#;
+        let result = parse_execute(code).await.unwrap();
+        assert_eq!(
+            mem_get_json(result.exec_state.stack(), result.mem_env, "answer"),
+            KclValue::HomArray {
+                value: vec![KclValue::HomArray {
+                    value: vec![KclValue::HomArray {
+                        value: vec![
+                            KclValue::Number {
+                                value: 0.0,
+                                ty: NumericType::default(),
+                                meta: vec![SourceRange::new(69, 70, Default::default()).into()],
+                            },
+                            KclValue::Number {
+                                value: 0.0,
+                                ty: NumericType::default(),
+                                meta: vec![SourceRange::new(71, 72, Default::default()).into()],
+                            }
+                        ],
+                        ty: RuntimeType::any(),
+                    }],
+                    ty: RuntimeType::any(),
+                }],
+                ty: RuntimeType::any(),
+            }
+        );
+    }
+
     #[tokio::test(flavor = "multi_thread")]
     async fn test_zero_param_fn() {
         let ast = r#"sigmaAllow = 35000 // psi

rust/kcl-lib/src/execution/types.rs

@@ -1090,12 +1090,18 @@ impl KclValue {
         exec_state: &mut ExecState,
     ) -> Result<KclValue, CoercionError> {
         match self {
-            KclValue::Tuple { value, .. } if value.len() == 1 && !matches!(ty, RuntimeType::Tuple(..)) => {
+            KclValue::Tuple { value, .. }
+                if value.len() == 1
+                    && !matches!(ty, RuntimeType::Primitive(PrimitiveType::Any) | RuntimeType::Tuple(..)) =>
+            {
                 if let Ok(coerced) = value[0].coerce(ty, convert_units, exec_state) {
                     return Ok(coerced);
                 }
             }
-            KclValue::HomArray { value, .. } if value.len() == 1 && !matches!(ty, RuntimeType::Array(..)) => {
+            KclValue::HomArray { value, .. }
+                if value.len() == 1
+                    && !matches!(ty, RuntimeType::Primitive(PrimitiveType::Any) | RuntimeType::Array(..)) =>
+            {
                 if let Ok(coerced) = value[0].coerce(ty, convert_units, exec_state) {
                     return Ok(coerced);
                 }