fix: duplicate entries when running mise settings add idiomatic_version_file_enable_tools

mise.lock

@@ -103,7 +103,7 @@ backend = "aqua:BurntSushi/ripgrep"
 
 [tools.shellcheck]
 version = "0.10.0"
-backend = "aqua:koalaman/shellcheck"
+backend = "ubi:koalaman/shellcheck"
 
 [tools.shellcheck.checksums]
 "shellcheck-v0.10.0.darwin.aarch64.tar.xz" = "sha256:bbd2f14826328eee7679da7221f2bc3afb011f6a928b848c80c321f6046ddf81"

src/cli/settings/set.rs

@@ -27,7 +27,7 @@ impl SettingsSet {
 }
 
 pub fn set(mut key: &str, value: &str, add: bool, local: bool) -> Result<()> {
-    let value = if let Some(meta) = SETTINGS_META.get(key) {
+    let parsed_value = if let Some(meta) = SETTINGS_META.get(key) {
         match meta.type_ {
             SettingsType::Bool => parse_bool(value)?,
             SettingsType::Integer => parse_i64(value)?,
@@ -46,16 +46,19 @@ pub fn set(mut key: &str, value: &str, add: bool, local: bool) -> Result<()> {
         config::global_config_path()
     };
     file::create_dir_all(path.parent().unwrap())?;
+
     let raw = file::read_to_string(&path).unwrap_or_default();
     let mut config: DocumentMut = raw.parse()?;
+
     if !config.contains_key("settings") {
         config["settings"] = toml_edit::Item::Table(toml_edit::Table::new());
     }
+
     if let Some(mut settings) = config["settings"].as_table_mut() {
-        if let Some((parent_key, child_key)) = key.split_once('.') {
-            key = child_key;
+        if let Some((parent, child)) = key.split_once('.') {
+            key = child;
             settings = settings
-                .entry(parent_key)
+                .entry(parent)
                 .or_insert({
                     let mut t = toml_edit::Table::new();
                     t.set_implicit(true);
@@ -65,21 +68,30 @@ pub fn set(mut key: &str, value: &str, add: bool, local: bool) -> Result<()> {
                 .unwrap();
         }
 
-        let value = match settings.get(key).map(|c| c.as_array()) {
-            Some(Some(array)) if add => {
-                let mut array = array.clone();
-                array.extend(value.as_array().unwrap().iter().cloned());
-                array.into()
+        let new_value: toml_edit::Value = if add {
+            if let Some(existing_arr) = settings.get(key).and_then(|item| item.as_array()).cloned()
+            {
+                let mut arr = existing_arr;
+                // Dedupe by comparing to the original &str
+                if !arr.iter().any(|it| it.as_str() == Some(value)) {
+                    arr.push(value);
+                }
+                toml_edit::Value::Array(arr)
+            } else {
+                parsed_value.clone()
             }
-            _ => value,
+        } else {
+            parsed_value.clone()
         };
-        settings.insert(key, value.into());
+
+        settings.insert(key, new_value.into());
 
         // validate
         let _: SettingsFile = toml::from_str(&config.to_string())?;
 
-        file::write(path, config.to_string())?;
+        file::write(&path, config.to_string())?;
     }
+
     Ok(())
 }