Fix callee execution order

Author: jtran

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

@@ -197,6 +197,17 @@ impl Node<CallExpressionKw> {
         let fn_name = &self.callee;
         let callsite: SourceRange = self.into();
 
+        // Clone the function so that we can use a mutable reference to
+        // exec_state.
+        let func = fn_name.get_result(exec_state, ctx).await?.clone();
+
+        let Some(fn_src) = func.as_function() else {
+            return Err(KclError::new_semantic(KclErrorDetails::new(
+                "cannot call this because it isn't a function".to_string(),
+                vec![callsite],
+            )));
+        };
+
         // Evaluate the unlabeled first param, if any exists.
         let unlabeled = if let Some(ref arg_expr) = self.unlabeled {
             let source_range = SourceRange::from(arg_expr.clone());
@@ -247,17 +258,6 @@ impl Node<CallExpressionKw> {
             exec_state.pipe_value().map(|v| Arg::new(v.clone(), callsite)),
         );
 
-        // Clone the function so that we can use a mutable reference to
-        // exec_state.
-        let func = fn_name.get_result(exec_state, ctx).await?.clone();
-
-        let Some(fn_src) = func.as_function() else {
-            return Err(KclError::new_semantic(KclErrorDetails::new(
-                "cannot call this because it isn't a function".to_string(),
-                vec![callsite],
-            )));
-        };
-
         let return_value = fn_src
             .call_kw(Some(fn_name.to_string()), exec_state, ctx, args, callsite)
             .await

rust/kcl-lib/tests/var_ref_in_own_def/input.kcl

@@ -1,3 +1,3 @@
 // This won't work, because `sketch001` is being referenced in its own definition.
 sketch001 = startSketchOn(XY)
-  |> startProfileAt([20, -20], sketch001)
+  |> startProfile(sketch001, at = [20, -20])