fix: resolve clippy collapsible_if warnings

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: killagu-claw

crates/pm/src/util/downloader.rs

@@ -44,11 +44,11 @@ pub async fn download_to_cache(name: &str, version: &str, tarball_url: &str) ->
     // Git packages are already cached during resolution.
     if tarball_url.starts_with("git+") || tarball_url.starts_with("git://") {
         let cache_dir = get_cache_dir();
-        if let Some(cache_path) = utoo_ruborist::git::git_cache_path(&cache_dir, tarball_url) {
-            if crate::fs::try_exists(&cache_path).await.unwrap_or(false) {
-                tracing::debug!("Git package cache hit: {}@{}", name, version);
-                return Some(cache_path);
-            }
+        if let Some(cache_path) = utoo_ruborist::git::git_cache_path(&cache_dir, tarball_url)
+            && crate::fs::try_exists(&cache_path).await.unwrap_or(false)
+        {
+            tracing::debug!("Git package cache hit: {}@{}", name, version);
+            return Some(cache_path);
         }
         tracing::warn!(
             "Git package {}@{} not found in cache, expected pre-resolution",

crates/ruborist/src/resolver/git.rs

@@ -125,11 +125,11 @@ fn clone_repo_blocking(
     let is_full_sha =
         commit_ish.is_some_and(|c| c.len() == 40 && c.chars().all(|ch| ch.is_ascii_hexdigit()));
 
-    if let Some(ref_name) = commit_ish {
-        if !is_full_sha {
-            // Try as branch/tag ref
-            prepare = prepare.with_ref_name(Some(ref_name))?;
-        }
+    if let Some(ref_name) = commit_ish
+        && !is_full_sha
+    {
+        // Try as branch/tag ref
+        prepare = prepare.with_ref_name(Some(ref_name))?;
     }
 
     let (checkout, _outcome) = prepare