Sync skeleton controls and improve JSON save

Call _sync_skeleton_controls_from_model() to centralize enabling/disabling of skeleton UI controls (ensures controls are disabled when self._skeleton is None) instead of manually toggling the checkbox. Change save_skeleton to use model.model_dump_json() for .json output so Pydantic's JSON serialization is used (ensures Enums and other non-primitive types are serialized correctly); keep YAML branch using model_dump and safe_dump. Raises same error for unsupported suffixes.

Author: C-Achard

dlclivegui/gui/main_window.py

@@ -1834,8 +1834,8 @@ def _configure_skeleton_for_model(self, model_path: str) -> None:
 
         # Default: disable until we find a compatible skeleton
         if hasattr(self, "show_skeleton_checkbox"):
-            self.show_skeleton_checkbox.setEnabled(False)
-            # keep checked state but it won't be used unless enabled
+            # This will disable all skeleton controls when self._skeleton is None
+            self._sync_skeleton_controls_from_model()
 
         p = Path(model_path).expanduser()
 

dlclivegui/utils/skeleton.py

@@ -165,12 +165,13 @@ def load_skeleton(path: Path) -> Skeleton:
 
 
 def save_skeleton(path: Path, model: SkeletonModel) -> None:
-    data = model.model_dump()
-
     if path.suffix in {".yaml", ".yml"}:
+        data = model.model_dump()
         path.write_text(yaml.safe_dump(data, sort_keys=False))
     elif path.suffix == ".json":
-        path.write_text(json.dumps(data, indent=2))
+        # Use Pydantic's JSON serialization to ensure Enums and other types
+        # are converted to JSON-friendly values.
+        path.write_text(model.model_dump_json(indent=2))
     else:
         raise SkeletonLoadError(f"Unsupported skeleton file type: {path.suffix}")