refactor: Simplify extract_binary_path_from_wrapper with shared parsing logic

- Create shared utility function extract_quoted_strings() in desktop_file.rs
- Extract strings enclosed in specified quote characters
- Refactor extract_binary_path_from_wrapper() to use shared parsing logic
- Reduce code duplication from 6 lines to 1 line for extraction logic
- Add comprehensive tests for the utility function
- Maintain identical functionality and behavior

Author: ranfdev

src/backends/desktop_file.rs

@@ -5,6 +5,22 @@ pub struct DesktopEntry {
     pub icon: String,
 }
 
+/// Extracts the first string enclosed in the specified quote character from a line of text.
+/// Returns the extracted string without quotes, or None if no quoted string is found.
+/// 
+/// This is useful for parsing shell scripts and config files that use quoted strings.
+/// 
+/// # Examples
+/// ```
+/// let line = "exec '/usr/bin/vim' \"$@\"";
+/// assert_eq!(extract_quoted_string(line, '\''), Some("/usr/bin/vim"));
+/// ```
+pub fn extract_quoted_string(line: &str, quote_char: char) -> Option<String> {
+    let start = line.find(quote_char)?;
+    let end = line[start + 1..].find(quote_char)?;
+    Some(line[start + 1..start + 1 + end].to_string())
+}
+
 pub fn parse_desktop_file(content: &str) -> anyhow::Result<DesktopEntry> {
     let mut name = None;
     let mut exec = None;
@@ -118,4 +134,34 @@ Icon=test-icon
         assert_eq!(&entry.name, "Test=App");
         assert_eq!(&entry.exec, "/usr/bin/test --param=value");
     }
+
+    #[test]
+    fn test_extract_quoted_string_single_quotes() {
+        let line = "exec '/usr/bin/vim' \"$@\"";
+        assert_eq!(extract_quoted_string(line, '\''), Some("/usr/bin/vim".to_string()));
+    }
+
+    #[test]
+    fn test_extract_quoted_string_double_quotes() {
+        let line = r#"exec "distrobox-enter" -n test"#;
+        assert_eq!(extract_quoted_string(line, '"'), Some("distrobox-enter".to_string()));
+    }
+
+    #[test]
+    fn test_extract_quoted_string_no_quotes() {
+        let line = "exec /usr/bin/vim";
+        assert_eq!(extract_quoted_string(line, '\''), None);
+    }
+
+    #[test]
+    fn test_extract_quoted_string_incomplete_quotes() {
+        let line = "exec '/usr/bin/vim";
+        assert_eq!(extract_quoted_string(line, '\''), None);
+    }
+
+    #[test]
+    fn test_extract_quoted_string_empty() {
+        let line = "exec ''";
+        assert_eq!(extract_quoted_string(line, '\''), Some("".to_string()));
+    }
 }

src/backends/distrobox/distrobox.rs

@@ -848,6 +848,9 @@ impl Distrobox {
 
     /// Extracts the original binary path from a distrobox exported wrapper script.
     /// The wrapper script contains lines like: exec '/usr/bin/binary' "$@"
+    /// 
+    /// Uses the shared `extract_quoted_string` utility from desktop_file module for
+    /// consistent string parsing across the codebase.
     async fn extract_binary_path_from_wrapper(&self, wrapper_path: &str) -> Option<String> {
         // Read the wrapper script content
         let cmd = Command::new_with_args("cat", [wrapper_path]);
@@ -856,17 +859,15 @@ impl Distrobox {
         // Look for the pattern: exec ... '/path/to/binary' or exec '/path/to/binary'
         // The binary path is typically in single quotes in the else branch
         for line in output.lines() {
-            let line = line.trim();
+            let trimmed = line.trim();
             // Look for lines with exec that contain a quoted path
-            if line.starts_with("exec") {
-                // Try to extract path between single quotes
-                if let Some(start) = line.find('\'') {
-                    if let Some(end) = line[start + 1..].find('\'') {
-                        let path = &line[start + 1..start + 1 + end];
-                        // Validate it looks like a path
-                        if path.starts_with('/') && !path.contains("distrobox") {
-                            return Some(path.to_string());
-                        }
+            if trimmed.starts_with("exec") {
+                // Reuse the shared quoted string extraction logic from desktop_file module
+                if let Some(path) = extract_quoted_string(trimmed, '\'') {
+                    // Validate it looks like an absolute path to the actual binary
+                    // (not a distrobox wrapper command)
+                    if path.starts_with('/') && !path.contains("distrobox") {
+                        return Some(path);
                     }
                 }
             }