From 9a24a80d59e7bd7109d0538edfd1d6bddbddbbdc Mon Sep 17 00:00:00 2001
From: peoplemiau1 <eznaevsamil70@gmail.com>
Date: Fri, 15 May 2026 12:22:05 +0300
Subject: [PATCH] fix: address clippy warnings in tree.rs

---
 src/cli/tree.rs | 53 ++++++++++++++++++++-----------------------------
 1 file changed, 21 insertions(+), 32 deletions(-)

diff --git a/src/cli/tree.rs b/src/cli/tree.rs
index 9a9e6ae..7db9bc3 100644
--- a/src/cli/tree.rs
+++ b/src/cli/tree.rs
@@ -57,17 +57,15 @@ fn print_deps(
 
         if let Some(path) = &dep_path {
             let dcr_toml = path.join("dcr.toml");
-            if dcr_toml.exists() {
-                if let Ok(config) = Config::open(&dcr_toml.to_string_lossy()) {
-                    if let Some(v) = config.get("package.version").and_then(|v| v.as_str()) {
-                        version = format!(" v{}", v);
-                    }
-                    sub_deps = config
-                        .get("dependencies")
-                        .and_then(|v| v.as_table())
-                        .cloned();
-                    resolved_path = Some(path.clone());
+            if dcr_toml.exists() && let Ok(config) = Config::open(&dcr_toml.to_string_lossy()) {
+                if let Some(v) = config.get("package.version").and_then(|v| v.as_str()) {
+                    version = format!(" v{}", v);
                 }
+                sub_deps = config
+                    .get("dependencies")
+                    .and_then(|v| v.as_table())
+                    .cloned();
+                resolved_path = Some(path.clone());
             }
         }
 
@@ -91,35 +89,26 @@ fn print_deps(
 
         println!("{}{}{}{}", prefix, connector, name, version);
 
-        if let Some(s_deps) = sub_deps {
-            if !seen.contains(*name) {
-                seen.insert(name.to_string());
-                let new_prefix = format!("{}{}", prefix, child_prefix);
-                if let Some(path) = resolved_path {
-                    print_deps(&s_deps, &path, &new_prefix, seen);
-                }
-                seen.remove(*name);
+        if let Some(s_deps) = sub_deps && !seen.contains(*name) {
+            seen.insert(name.to_string());
+            let new_prefix = format!("{}{}", prefix, child_prefix);
+            if let Some(path) = resolved_path {
+                print_deps(&s_deps, &path, &new_prefix, seen);
             }
+            seen.remove(*name);
         }
     }
 }
 
 fn resolve_dep_path(value: &Value, base_path: &Path) -> Option<PathBuf> {
     match value {
-        Value::String(s) => {
-            if s.starts_with("path:") {
-                Some(base_path.join(&s[5..]))
-            } else {
-                None
-            }
-        }
-        Value::Table(t) => {
-            if let Some(path) = t.get("path").and_then(|v| v.as_str()) {
-                Some(base_path.join(path))
-            } else {
-                None
-            }
-        }
+        Value::String(s) => s
+            .strip_prefix("path:")
+            .map(|stripped| base_path.join(stripped)),
+        Value::Table(t) => t
+            .get("path")
+            .and_then(|v| v.as_str())
+            .map(|path| base_path.join(path)),
         _ => None,
     }
 }