use device name instead of node id to get audio device

src/bin/cli.rs

@@ -72,7 +72,7 @@ enum SetCommands {
     /// Playback position
     Position { position: f32 },
     /// Input
-    Input { id: u32 },
+    Input { name: String },
 }
 
 #[tokio::main]
@@ -102,7 +102,7 @@ async fn main() -> Result<(), Box<dyn Error>> {
         Commands::Set { parameter } => match parameter {
             SetCommands::Volume { volume } => Request::set_volume(volume),
             SetCommands::Position { position } => Request::seek(position),
-            SetCommands::Input { id } => Request::set_input(id),
+            SetCommands::Input { name } => Request::set_input(&name),
         },
     };
 

src/gui/draw.rs

@@ -287,7 +287,7 @@ impl SoundpadGui {
         ui.add_space(5.0);
         ui.horizontal_top(|ui| {
             // ---------- Microphone selection ----------
-            let mut mics: Vec<(&u32, &String)> =
+            let mut mics: Vec<(&String, &String)> =
                 self.audio_player_state.all_inputs.iter().collect();
             mics.sort_by_key(|(k, _)| *k);
 
@@ -301,8 +301,8 @@ impl SoundpadGui {
                         .unwrap_or(&String::new()),
                 )
                 .show_ui(ui, |ui| {
-                    for (index, device) in mics {
-                        ui.selectable_value(&mut selected_input, index.to_owned(), device);
+                    for (name, nick) in mics {
+                        ui.selectable_value(&mut selected_input, name.to_owned(), nick);
                     }
                 });
 

src/gui/mod.rs

@@ -92,12 +92,12 @@ impl SoundpadGui {
         make_request_sync(Request::play(path.to_str().unwrap())).ok();
     }
 
-    pub fn set_input(&mut self, id: u32) {
-        make_request_sync(Request::set_input(id)).ok();
+    pub fn set_input(&mut self, name: String) {
+        make_request_sync(Request::set_input(&name)).ok();
 
         if self.config.save_input {
             let mut daemon_config = get_daemon_config();
-            daemon_config.default_input_id = Some(id);
+            daemon_config.default_input_name = Some(name);
             daemon_config.save_to_file().ok();
         }
     }

src/types/audio_player.rs

@@ -40,8 +40,8 @@ impl AudioPlayer {
         let daemon_config = get_daemon_config();
         let default_volume = daemon_config.default_volume.unwrap_or(1.0);
         let mut default_input_device: Option<AudioDevice> = None;
-        if let Some(id) = daemon_config.default_input_id
-            && let Ok(device) = get_device(id).await
+        if let Some(name) = daemon_config.default_input_name
+            && let Ok(device) = get_device(&name).await
             && device.device_type == DeviceType::Input
         {
             default_input_device = Some(device);
@@ -224,8 +224,8 @@ impl AudioPlayer {
         &self.current_file_path
     }
 
-    pub async fn set_current_input_device(&mut self, id: u32) -> Result<(), Box<dyn Error>> {
-        let input_device = get_device(id).await?;
+    pub async fn set_current_input_device(&mut self, name: &str) -> Result<(), Box<dyn Error>> {
+        let input_device = get_device(name).await?;
 
         if input_device.device_type != DeviceType::Input {
             return Err("Selected device is not an input device".into());

src/types/commands.rs

@@ -47,7 +47,7 @@ pub struct GetCurrentInputCommand {}
 pub struct GetAllInputsCommand {}
 
 pub struct SetCurrentInputCommand {
-    pub id: Option<u32>,
+    pub name: Option<String>,
 }
 
 #[async_trait]
@@ -192,7 +192,10 @@ impl Executable for GetCurrentInputCommand {
     async fn execute(&self) -> Response {
         let audio_player = get_audio_player().await.lock().await;
         if let Some(input_device) = &audio_player.current_input_device {
-            Response::new(true, format!("{} - {}", input_device.id, input_device.nick))
+            Response::new(
+                true,
+                format!("{} - {}", input_device.name, input_device.nick),
+            )
         } else {
             Response::new(false, "No input device selected")
         }
@@ -209,7 +212,7 @@ impl Executable for GetAllInputsCommand {
                 continue;
             }
 
-            let string = format!("{} - {}", device.id, device.nick);
+            let string = format!("{} - {}", device.name, device.nick);
             input_devices_strings.push(string);
         }
         let response_message = input_devices_strings.join("; ");
@@ -221,9 +224,9 @@ impl Executable for GetAllInputsCommand {
 #[async_trait]
 impl Executable for SetCurrentInputCommand {
     async fn execute(&self) -> Response {
-        if let Some(id) = self.id {
+        if let Some(name) = &self.name {
             let mut audio_player = get_audio_player().await.lock().await;
-            match audio_player.set_current_input_device(id).await {
+            match audio_player.set_current_input_device(name).await {
                 Ok(_) => Response::new(true, "Input device was set"),
                 Err(err) => Response::new(false, err.to_string()),
             }

src/types/config.rs

@@ -4,7 +4,7 @@ use std::{collections::HashSet, error::Error, fs, path::PathBuf};
 
 #[derive(Default, Clone, Serialize, Deserialize)]
 pub struct DaemonConfig {
-    pub default_input_id: Option<u32>,
+    pub default_input_name: Option<String>,
     pub default_volume: Option<f32>,
 }
 

src/types/gui.rs

@@ -34,6 +34,6 @@ pub struct AudioPlayerState {
     pub new_position: Option<f32>,
     pub duration: f32,
 
-    pub current_input: u32,
-    pub all_inputs: HashMap<u32, String>,
+    pub current_input: String,
+    pub all_inputs: HashMap<String, String>,
 }

src/types/socket.rs

@@ -80,8 +80,8 @@ impl Request {
         Request::new("seek", vec![("position", &position.to_string())])
     }
 
-    pub fn set_input(id: u32) -> Self {
-        Request::new("set_input", vec![("input_id", &id.to_string())])
+    pub fn set_input(name: &str) -> Self {
+        Request::new("set_input", vec![("input_name", name)])
     }
 }
 

src/utils/commands.rs

@@ -44,13 +44,8 @@ pub fn parse_command(request: &Request) -> Option<Box<dyn Executable + Send>> {
         "get_input" => Some(Box::new(GetCurrentInputCommand {})),
         "get_inputs" => Some(Box::new(GetAllInputsCommand {})),
         "set_input" => {
-            let id = request
-                .args
-                .get("input_id")
-                .unwrap_or(&String::new())
-                .parse::<u32>()
-                .ok();
-            Some(Box::new(SetCurrentInputCommand { id }))
+            let name = Some(request.args.get("input_name").unwrap_or(&String::new())).cloned();
+            Some(Box::new(SetCurrentInputCommand { name }))
         }
         _ => None,
     }

src/utils/gui.rs

@@ -96,10 +96,8 @@ pub fn start_app_state_thread(audio_player_state_shared: Arc<Mutex<AudioPlayerSt
                     .collect::<Vec<&str>>()
                     .first()
                     .unwrap()
-                    .to_string()
-                    .parse::<u32>()
-                    .unwrap_or_default(),
-                false => 0,
+                    .to_string(),
+                false => String::new(),
             };
             let all_inputs = match all_inputs_res.status {
                 true => all_inputs_res
@@ -111,14 +109,11 @@ pub fn start_app_state_thread(audio_player_state_shared: Arc<Mutex<AudioPlayerSt
                         if entry.is_empty() {
                             return None;
                         }
-                        entry.split_once(" - ").and_then(|(k, v)| {
-                            k.trim()
-                                .parse::<u32>()
-                                .ok()
-                                .map(|key| (key, v.trim().to_string()))
-                        })
+                        entry
+                            .split_once(" - ")
+                            .map(|(k, v)| (k.trim().to_string(), v.trim().to_string()))
                     })
-                    .collect(),
+                    .collect::<HashMap<String, String>>(),
                 false => HashMap::new(),
             };
 

src/utils/pipewire.rs

@@ -199,12 +199,12 @@ pub async fn get_all_devices() -> Result<(Vec<AudioDevice>, Vec<AudioDevice>), B
     }
 }
 
-pub async fn get_device(node_id: u32) -> Result<AudioDevice, Box<dyn Error>> {
+pub async fn get_device(device_name: &str) -> Result<AudioDevice, Box<dyn Error>> {
     let (mut input_devices, output_devices) = get_all_devices().await?;
     input_devices.extend(output_devices);
 
     for device in input_devices {
-        if device.id == node_id {
+        if device.name == device_name {
             return Ok(device);
         }
     }