feat(cli): implement MultiSelect

Values for the MultiSelect are parsed as CSV. For common cases, it's
simply separating values with commas, but it handles cases where
characters need to be escaped properly.

Author: Oppzippy

Cargo.lock

@@ -69,3 +69,4 @@ glob = "0.3"
 insta = "1"
 insta-cmd = "0.6"
 sysinfo = "0.36"
+csv = "1"

Cargo.toml

@@ -24,6 +24,7 @@ dirs = { workspace = true }
 strum = { workspace = true }
 tabled = { workspace = true }
 terminal_size = { workspace = true }
+csv = { workspace = true }
 
 [dev-dependencies]
 insta = { workspace = true }

cli/Cargo.toml

@@ -231,7 +231,21 @@ fn parse_setting_value(setting: &Setting, unparsed_value: String) -> anyhow::Res
                 Value::String(name.into())
             }
         }
-        Setting::MultiSelect { setting, values } => todo!(),
+        Setting::MultiSelect { setting: _, .. } => {
+            let mut reader = csv::ReaderBuilder::new()
+                .has_headers(false)
+                .from_reader(unparsed_value.as_bytes());
+            let maybe_row = reader.records().next().transpose()?;
+            let strings = maybe_row
+                .map(|row| {
+                    row.into_iter()
+                        .map(|entry| Cow::from(entry.to_string()))
+                        .collect::<Vec<_>>()
+                })
+                .unwrap_or_default();
+
+            Value::StringVec(strings)
+        }
         Setting::Equalizer { setting, .. } => {
             let values = unparsed_value
                 .split(",")

cli/src/cli/device.rs

@@ -60,7 +60,18 @@ impl std::fmt::Display for DisplayableValue {
             Value::I16Vec(items) => write!(f, "{items:?}"),
             Value::I32(value) => write!(f, "{value}"),
             Value::String(value) => write!(f, "{value}"),
-            Value::StringVec(values) => write!(f, "{values:?}"),
+            Value::StringVec(values) => {
+                let mut buffer = Vec::new();
+                let mut writer = csv::WriterBuilder::new()
+                    .has_headers(false)
+                    .from_writer(&mut buffer);
+                // write the fields individually rather than write_record so that we don't get a newline at the end
+                for value in values {
+                    writer.write_field(value.as_bytes()).unwrap();
+                }
+                std::mem::drop(writer);
+                write!(f, "{}", String::from_utf8(buffer).unwrap())
+            }
             Value::OptionalString(value) => {
                 if let Some(value) = value {
                     write!(f, "{value}")