refactor(ruborist): remove From<bool> for PeerDeps impl

The implicit bool→enum conversion had counter-intuitive semantics
(true=Skip, false=Include). Use explicit match at the call site instead.

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

Author: elrrrrrrr

crates/pm/src/util/user_config.rs

@@ -4,6 +4,7 @@ use std::sync::{LazyLock, OnceLock};
 
 use anyhow::Result;
 use dashmap::DashMap;
+use utoo_ruborist::builder::PeerDeps;
 use utoo_ruborist::manifest::PackageJson;
 use utoo_ruborist::spec::Catalogs;
 
@@ -83,8 +84,11 @@ pub fn set_legacy_peer_deps(value: Option<bool>) {
     LEGACY_PEER_DEPS.set(value);
 }
 
-pub async fn get_peer_deps() -> utoo_ruborist::builder::PeerDeps {
-    LEGACY_PEER_DEPS.get().await.into()
+pub async fn get_peer_deps() -> PeerDeps {
+    match LEGACY_PEER_DEPS.get().await {
+        true => PeerDeps::Skip,
+        false => PeerDeps::Include,
+    }
 }
 
 static OMIT: OnceLock<HashSet<OmitType>> = OnceLock::new();

crates/ruborist/src/model/node.rs

@@ -22,13 +22,6 @@ pub enum PeerDeps {
     Skip,
 }
 
-impl From<bool> for PeerDeps {
-    /// `true` (legacy_peer_deps) → `Skip`, `false` → `Include`.
-    fn from(legacy: bool) -> Self {
-        if legacy { Self::Skip } else { Self::Include }
-    }
-}
-
 /// Controls whether dev dependencies are included during edge iteration.
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]
 pub enum DevDeps {