feat: add SlintOptions to WorkspaceSettings and update UI handlers to properly save slint specific settings

Author: gramistella

src/core/workspace.rs

@@ -19,6 +19,8 @@ pub struct WorkspaceSettings {
     pub dirs_only: bool,
     #[serde(flatten)]
     pub rust: RustOptions,
+    #[serde(flatten)]
+    pub slint: SlintOptions,
 }
 
 #[derive(Debug, Clone, Serialize, Deserialize, Default, PartialEq, Eq)]
@@ -33,6 +35,14 @@ pub struct RustOptions {
     pub rust_signatures_only_filter: String,
 }
 
+#[derive(Debug, Clone, Serialize, Deserialize, Default, PartialEq, Eq)]
+pub struct SlintOptions {
+    #[serde(default)]
+    pub slint_remove_line_comments: bool,
+    #[serde(default)]
+    pub slint_remove_block_comments: bool,
+}
+
 #[derive(Debug, Clone, Serialize, Deserialize, Default)]
 pub struct LocalSettings {
     #[serde(default)]

src/ui/handlers.rs

@@ -13,7 +13,7 @@ use notify::{RecommendedWatcher, RecursiveMode, Watcher};
 use std::sync::mpsc;
 
 use stitch::core::{
-    Node, Profile, ProfileScope, RustFilterOptions, RustOptions, WorkspaceSettings,
+    Node, Profile, ProfileScope, RustFilterOptions, RustOptions, SlintOptions, WorkspaceSettings,
     apply_rust_filters, apply_slint_filters, clean_remove_regex, collapse_consecutive_blank_lines,
     collect_selected_paths, compile_remove_regex_opt, delete_profile, ensure_profiles_dirs,
     ensure_workspace_dir, gather_paths_set, is_ancestor_of, is_rust_file_path, is_slint_file_path,
@@ -209,6 +209,9 @@ pub fn on_select_folder(app: &AppWindow, state: &SharedState) {
             app.set_rust_remove_doc_comments(ws.rust.rust_remove_doc_comments);
             app.set_rust_function_signatures_only(ws.rust.rust_function_signatures_only);
             app.set_rust_signatures_only_filter(ws.rust.rust_signatures_only_filter.clone().into());
+            // Slint settings
+            app.set_slint_remove_line_comments(ws.slint.slint_remove_line_comments);
+            app.set_slint_remove_block_comments(ws.slint.slint_remove_block_comments);
 
             state.borrow_mut().workspace_baseline = Some(ws.clone());
         } else {
@@ -227,6 +230,10 @@ pub fn on_select_folder(app: &AppWindow, state: &SharedState) {
                     rust_function_signatures_only: app.get_rust_function_signatures_only(),
                     rust_signatures_only_filter: app.get_rust_signatures_only_filter().to_string(),
                 },
+                slint: SlintOptions {
+                    slint_remove_line_comments: app.get_slint_remove_line_comments(),
+                    slint_remove_block_comments: app.get_slint_remove_block_comments(),
+                },
             };
             let _ = save_workspace(&dir, &seed);
             state.borrow_mut().workspace_baseline = Some(seed);
@@ -1526,6 +1533,10 @@ fn capture_profile_from_ui(app: &AppWindow, state: &SharedState, name: &str) ->
             rust_function_signatures_only: app.get_rust_function_signatures_only(),
             rust_signatures_only_filter: app.get_rust_signatures_only_filter().to_string(),
         },
+        slint: SlintOptions {
+            slint_remove_line_comments: app.get_slint_remove_line_comments(),
+            slint_remove_block_comments: app.get_slint_remove_block_comments(),
+        },
     };
 
     // NOTE: Preserve root selection by storing an empty relative path ("")
@@ -1573,6 +1584,8 @@ fn apply_profile_to_ui(app: &AppWindow, state: &SharedState, profile: &Profile)
             .clone()
             .into(),
     );
+    app.set_slint_remove_line_comments(profile.settings.slint.slint_remove_line_comments);
+    app.set_slint_remove_block_comments(profile.settings.slint.slint_remove_block_comments);
 
     app.set_profile_name(profile.name.clone().into());
 
@@ -1683,6 +1696,10 @@ pub fn on_save_profile_current(app: &AppWindow, state: &SharedState) {
                 rust_function_signatures_only: app.get_rust_function_signatures_only(),
                 rust_signatures_only_filter: app.get_rust_signatures_only_filter().to_string(),
             },
+            slint: SlintOptions {
+                slint_remove_line_comments: app.get_slint_remove_line_comments(),
+                slint_remove_block_comments: app.get_slint_remove_block_comments(),
+            },
         };
 
         let _ = save_workspace(&project_root, &ws);
@@ -1839,6 +1856,8 @@ fn profiles_equal(a: &Profile, b: &Profile) -> bool {
         || sa.rust.rust_remove_doc_comments != sb.rust.rust_remove_doc_comments
         || sa.rust.rust_function_signatures_only != sb.rust.rust_function_signatures_only
         || sa.rust.rust_signatures_only_filter != sb.rust.rust_signatures_only_filter
+        || sa.slint.slint_remove_line_comments != sb.slint.slint_remove_line_comments
+        || sa.slint.slint_remove_block_comments != sb.slint.slint_remove_block_comments
     {
         return false;
     }
@@ -1881,6 +1900,10 @@ fn update_save_button_state(app: &AppWindow, state: &SharedState) {
                 rust_function_signatures_only: app.get_rust_function_signatures_only(),
                 rust_signatures_only_filter: app.get_rust_signatures_only_filter().to_string(),
             },
+            slint: SlintOptions {
+                slint_remove_line_comments: app.get_slint_remove_line_comments(),
+                slint_remove_block_comments: app.get_slint_remove_block_comments(),
+            },
         };
 
         let baseline_opt = { state.borrow().workspace_baseline.clone() };
@@ -2043,6 +2066,7 @@ fn workspace_settings_equal(a: &WorkspaceSettings, b: &WorkspaceSettings) -> boo
         && a.hierarchy_only == b.hierarchy_only
         && a.dirs_only == b.dirs_only
         && a.rust == b.rust
+        && a.slint == b.slint
     // Note: we intentionally ignore `current_profile` here for dirtiness comparison
 }
 

tests/profile_persistence.rs

@@ -1,6 +1,7 @@
 use stitch::core::{
-    Profile, ProfileScope, ProfileSelection, RustOptions, WorkspaceSettings, delete_profile,
-    ensure_profiles_dirs, ensure_workspace_dir, list_profiles, load_profile, save_profile,
+    Profile, ProfileScope, ProfileSelection, RustOptions, SlintOptions, WorkspaceSettings,
+    delete_profile, ensure_profiles_dirs, ensure_workspace_dir, list_profiles, load_profile,
+    save_profile,
 };
 use tempfile::TempDir;
 
@@ -36,6 +37,10 @@ fn sample_ws() -> WorkspaceSettings {
             rust_function_signatures_only: false,
             rust_signatures_only_filter: String::new(),
         },
+        slint: SlintOptions {
+            slint_remove_line_comments: false,
+            slint_remove_block_comments: false,
+        },
     }
 }
 

tests/workspace_current_profile.rs

@@ -1,5 +1,5 @@
 use stitch::core::{
-    LocalSettings, Profile, ProfileScope, RustOptions, WorkspaceSettings,
+    LocalSettings, Profile, ProfileScope, RustOptions, SlintOptions, WorkspaceSettings,
     clear_stale_current_profile, load_local_settings, save_local_settings, save_profile,
 };
 use tempfile::TempDir;
@@ -54,6 +54,10 @@ fn no_clear_when_shared_profile_exists() {
                 rust_function_signatures_only: false,
                 rust_signatures_only_filter: String::new(),
             },
+            slint: SlintOptions {
+                slint_remove_line_comments: false,
+                slint_remove_block_comments: false,
+            },
         },
         explicit: vec![],
     };
@@ -93,6 +97,10 @@ fn no_clear_when_local_profile_exists() {
                 rust_function_signatures_only: false,
                 rust_signatures_only_filter: String::new(),
             },
+            slint: SlintOptions {
+                slint_remove_line_comments: false,
+                slint_remove_block_comments: false,
+            },
         },
         explicit: vec![],
     };

tests/workspace_persistence.rs

@@ -1,6 +1,6 @@
 use stitch::core::{
-    RustOptions, WorkspaceSettings, ensure_workspace_dir, load_workspace, save_workspace,
-    workspace_dir, workspace_file,
+    RustOptions, SlintOptions, WorkspaceSettings, ensure_workspace_dir, load_workspace,
+    save_workspace, workspace_dir, workspace_file,
 };
 use tempfile::TempDir;
 
@@ -41,6 +41,10 @@ fn save_then_load_roundtrip_and_overwrite_is_atomic() {
             rust_function_signatures_only: false,
             rust_signatures_only_filter: String::new(),
         },
+        slint: SlintOptions {
+            slint_remove_line_comments: true,
+            slint_remove_block_comments: false,
+        },
     };
     save_workspace(root, &s1).expect("save v1");
 
@@ -68,17 +72,27 @@ fn save_then_load_roundtrip_and_overwrite_is_atomic() {
         loaded1.rust.rust_function_signatures_only,
         s1.rust.rust_function_signatures_only
     );
+    assert_eq!(
+        loaded1.slint.slint_remove_line_comments,
+        s1.slint.slint_remove_line_comments
+    );
+    assert_eq!(
+        loaded1.slint.slint_remove_block_comments,
+        s1.slint.slint_remove_block_comments
+    );
 
     let mut s2 = loaded1;
     s2.ext_filter = ".rs".into();
     s2.hierarchy_only = true;
     s2.rust.rust_signatures_only_filter = "src/*,tests/*".into();
+    s2.slint.slint_remove_block_comments = true;
     save_workspace(root, &s2).expect("save v2 overwrite");
 
     let loaded2 = load_workspace(root).expect("load s2");
     assert_eq!(loaded2.ext_filter, ".rs");
     assert!(loaded2.hierarchy_only);
     assert_eq!(loaded2.rust.rust_signatures_only_filter, "src/*,tests/*");
+    assert!(loaded2.slint.slint_remove_block_comments);
 
     let tmp_path = wf.with_extension("json.tmp");
     assert!(