refactor: merge duplicate dep_usage push, update stale comments

Author: RomarQ

src/check.rs

@@ -25,17 +25,9 @@ pub fn run_checks(workspace: &WorkspaceInfo, promotion_threshold: usize) -> Vec<
 
             let lookup_name = dep.package.as_deref().unwrap_or(&dep.name);
 
-            if let Some(ws_dep) = workspace.workspace_deps.get(lookup_name) {
-                // If registries differ, the workspace dep doesn't apply —
-                // treat as an independent dep eligible for promotion.
-                if dep.registry != ws_dep.registry {
-                    dep_usage
-                        .entry((lookup_name.to_string(), dep.registry.clone()))
-                        .or_default()
-                        .push((member_rel.clone(), dep.version.clone()));
-                    continue;
-                }
-
+            if let Some(ws_dep) = workspace.workspace_deps.get(lookup_name)
+                && dep.registry == ws_dep.registry
+            {
                 let kind = match (&dep.version, &ws_dep.version) {
                     (Some(dv), Some(wv)) if dv != wv => DiagnosticKind::VersionMismatch {
                         version: dep.version.clone(),

src/fix.rs

@@ -141,8 +141,8 @@ fn fix_member_dep(manifest_path: &Path, dep_name: &str) -> Result<bool, String>
 
 /// Rewrite a dependency entry to use `{ workspace = true }`.
 ///
-/// Strips `version` and `default-features` (which must be set at the workspace
-/// level to have any effect). Preserves other keys like `features` and `optional`.
+/// Strips `version`, `default-features`, and `registry` (which must be set at
+/// the workspace level). Preserves other keys like `features` and `optional`.
 fn rewrite_dep_entry(table: &mut dyn toml_edit::TableLike, key: &str) -> bool {
     let Some(item) = table.get_mut(key) else {
         return false;
@@ -173,7 +173,7 @@ fn rewrite_dep_entry(table: &mut dyn toml_edit::TableLike, key: &str) -> bool {
             if existing.get("workspace").and_then(|v| v.as_bool()) == Some(true) {
                 return false;
             }
-            // Rebuild with workspace = true, dropping version and default-features
+            // Rebuild with workspace = true, dropping version, default-features, and registry
             let mut rebuilt = InlineTable::new();
             rebuilt.insert("workspace", Value::from(true));
             for (k, v) in existing.iter() {