diff --git a/api/sixtyfps-node/native/lib.rs b/api/sixtyfps-node/native/lib.rs
index b12d9cf33..65647711c 100644
--- a/api/sixtyfps-node/native/lib.rs
+++ b/api/sixtyfps-node/native/lib.rs
@@ -286,11 +286,7 @@ fn to_js_value<'cx>(
             &format!("#{:02x}{:02x}{:02x}{:02x}", c.red(), c.green(), c.blue(), c.alpha()),
         )
         .as_value(cx),
-        Value::Brush(_)
-        | Value::PathElements(_)
-        | Value::EasingCurve(_)
-        | Value::EnumerationValue(..)
-        | Value::Model(_) => todo!("converting {:?} to js has not been implemented", val),
+        _ => todo!("converting {:?} to js has not been implemented", val),
     })
 }
 
diff --git a/sixtyfps_runtime/interpreter/eval.rs b/sixtyfps_runtime/interpreter/eval.rs
index c9738af49..003254948 100644
--- a/sixtyfps_runtime/interpreter/eval.rs
+++ b/sixtyfps_runtime/interpreter/eval.rs
@@ -101,9 +101,10 @@ impl PartialEq for ModelPtr {
     }
 }
 
-#[derive(Clone, PartialEq, Debug)]
 /// This is a dynamically typed Value used in the interpreter, it need to be able
 /// to be converted from and to anything that can be stored in a Property
+#[derive(Clone, PartialEq, Debug)]
+#[non_exhaustive]
 pub enum Value {
     /// There is nothing in this value. That's the default.
     /// For example, a function that do not return a result would return a Value::Void