More warnings

Signed-off-by: Nick Cameron <nrc@ncameron.org>

Author: nrc

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

@@ -12,7 +12,7 @@ use crate::{
 };
 
 /// Annotations which should cause re-execution if they change.
-pub(super) const SIGNIFICANT_ATTRS: [&str; 2] = [SETTINGS, NO_PRELUDE];
+pub(super) const SIGNIFICANT_ATTRS: [&str; 3] = [SETTINGS, NO_PRELUDE, WARNINGS];
 
 pub(crate) const SETTINGS: &str = "settings";
 pub(crate) const SETTINGS_UNIT_LENGTH: &str = "defaultLengthUnit";

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

@@ -604,8 +604,9 @@ impl ExecutorContext {
                     settings: &self.settings,
                 };
 
+                let cache_result = cache::get_changed_program(old, new).await;
                 // Get the program that actually changed from the old and new information.
-                let (clear_scene, program, import_check_info) = match cache::get_changed_program(old, new).await {
+                let (clear_scene, program, import_check_info) = match cache_result {
                     CacheResult::ReExecute {
                         clear_scene,
                         reapply_settings,

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

@@ -760,7 +760,7 @@ impl NumericType {
         }
     }
 
-    fn coerce(&self, val: &KclValue) -> Result<KclValue, CoercionError> {
+    fn coerce(&self, val: &KclValue, exec_state: &mut ExecState) -> Result<KclValue, CoercionError> {
         let KclValue::Number { value, ty, meta } = val else {
             return Err(val.into());
         };
@@ -832,6 +832,14 @@ impl NumericType {
             }
 
             (Default { angle: a1, .. }, Known(UnitType::Angle(a2))) => {
+                let mut source_ranges = Into::<Vec<SourceRange>>::into(val);
+                // A single source range means it's not via a function or something.
+                if source_ranges.len() == 1 {
+                    exec_state.warn(
+                        CompilationError::err(source_ranges.pop().unwrap(), "Prefer to use explicit units for angles"),
+                        annotations::WARN_ANGLE_UNITS,
+                    );
+                }
                 let (value, ty) = a1.adjust_to(*value, *a2);
                 Ok(KclValue::Number {
                     value,
@@ -1156,7 +1164,7 @@ impl KclValue {
             PrimitiveType::Any => Ok(self.clone()),
             PrimitiveType::Number(ty) => {
                 if convert_units {
-                    return ty.coerce(self);
+                    return ty.coerce(self, exec_state);
                 }
 
                 // Instead of converting units, reinterpret the number as having
@@ -1172,10 +1180,10 @@ impl KclValue {
                             value: *n,
                             meta: meta.clone(),
                         };
-                        return ty.coerce(&value);
+                        return ty.coerce(&value, exec_state);
                     }
                 }
-                ty.coerce(self)
+                ty.coerce(self, exec_state)
             }
             PrimitiveType::String => match self {
                 KclValue::String { .. } => Ok(self.clone()),