Merge branch 'main' into mike/mbd-api-updates

Author: mlfarrell

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "kittycad-modeling-cmds"
-version = "0.2.138"
+version = "0.2.139"
 edition = "2021"
 authors = ["KittyCAD, Inc."]
 description = "Commands in the KittyCAD Modeling API"

modeling-cmds/Cargo.toml

@@ -101,8 +101,8 @@ define_modeling_cmd_enum! {
             /// This segment will implicitly begin at the current "pen" location.
             pub segment: PathSegment,
             /// Optional label to associate with the new path segment.
-            #[serde(default, skip_serializing_if = "String::is_empty")]
-            pub label: String,
+            #[serde(default, skip_serializing_if = "Option::is_none")]
+            pub label: Option<String>,
         }
 
         /// Command for extruding a solid 2d.

modeling-cmds/src/def_enum.rs

@@ -37,8 +37,6 @@ fn print_duplicate_stub_names() {
         .map(|(n, c)| (n.to_string(), c))
         .collect();
 
-    eprintln!("duplicate classes: {:?}", dup_classes);
-    eprintln!("duplicate enums: {:?}", dup_enums);
-
-    // This is informational; do not fail test. If we decide to enforce, we can assert emptiness.
+    assert!(dup_classes.is_empty(), "Duplicate class names: {dup_classes:?}");
+    assert!(dup_enums.is_empty(), "Duplicate enum names: {dup_enums:?}");
 }