Made hidden features hidden on cargo add when not activated

src/cargo/ops/cargo_add/mod.rs

@@ -705,9 +705,13 @@ fn print_msg(shell: &mut Shell, dep: &Dependency, section: &[String]) -> CargoRe
             shell.write_stderr(format_args!(" {}\n", feat), &ColorSpec::new())?;
         }
         for feat in deactivated {
-            shell.write_stderr(&prefix, &ColorSpec::new())?;
-            shell.write_stderr('-', &ColorSpec::new().set_bold(true).set_fg(Some(Red)))?;
-            shell.write_stderr(format_args!(" {}\n", feat), &ColorSpec::new())?;
+            let is_hidden = feat.starts_with('_');
+
+            if !is_hidden {
+                shell.write_stderr(&prefix, &ColorSpec::new())?;
+                shell.write_stderr('-', &ColorSpec::new().set_bold(true).set_fg(Some(Red)))?;
+                shell.write_stderr(format_args!(" {}\n", feat), &ColorSpec::new())?;
+            }
         }
     }
 

tests/testsuite/cargo_add/add_hidden_activated/in/Cargo.toml

@@ -0,0 +1,9 @@
+[package]
+name = "main"
+version = "0.1.0"
+edition = "2021"
+
+# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
+
+[dependencies]
+hidden = { version = "0.1.0", path = "../hidden" }

tests/testsuite/cargo_add/add_hidden_activated/in/src/main.rs

@@ -0,0 +1,3 @@
+fn main() {
+    println!("Hello, world!");
+}

tests/testsuite/cargo_add/add_hidden_activated/mod.rs

@@ -0,0 +1,20 @@
+use cargo_test_support::curr_dir;
+use cargo_test_support::prelude::*;
+use cargo_test_support::Project;
+
+use crate::cargo_add::init_registry;
+
+#[cargo_test]
+fn add_hidden_activated() {
+    init_registry();
+    let project = Project::from_template(curr_dir!().join("in"));
+    let cwd = &project.root();
+
+    snapbox::cmd::Command::cargo_ui()
+        .current_dir(cwd)
+        .args(&["add", "hidden-feature-test", "--features", "_secret"])
+        .assert()
+        .success()
+        .stdout_matches_path(curr_dir!().join("stdout.log"))
+        .stderr_matches_path(curr_dir!().join("stderr.log"));
+}

tests/testsuite/cargo_add/add_hidden_activated/stderr.log

@@ -0,0 +1,5 @@
+    Updating `dummy-registry` index
+      Adding hidden-feature-test v99999.0.0 to dependencies.
+             Features:
+             + _secret
+             - not_secret

tests/testsuite/cargo_add/add_hidden_unactivated/in/Cargo.toml

@@ -0,0 +1,9 @@
+[package]
+name = "main"
+version = "0.1.0"
+edition = "2021"
+
+# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
+
+[dependencies]
+hidden = { version = "0.1.0", path = "../hidden" }

tests/testsuite/cargo_add/add_hidden_unactivated/in/src/main.rs

@@ -0,0 +1,3 @@
+fn main() {
+    println!("Hello, world!");
+}

tests/testsuite/cargo_add/add_hidden_unactivated/mod.rs

@@ -0,0 +1,20 @@
+use cargo_test_support::curr_dir;
+use cargo_test_support::prelude::*;
+use cargo_test_support::Project;
+
+use crate::cargo_add::init_registry;
+
+#[cargo_test]
+fn add_hidden_unactivated() {
+    init_registry();
+    let project = Project::from_template(curr_dir!().join("in"));
+    let cwd = &project.root();
+
+    snapbox::cmd::Command::cargo_ui()
+        .current_dir(cwd)
+        .args(&["add", "hidden-feature-test"])
+        .assert()
+        .success()
+        .stdout_matches_path(curr_dir!().join("stdout.log"))
+        .stderr_matches_path(curr_dir!().join("stderr.log"));
+}

tests/testsuite/cargo_add/add_hidden_unactivated/stderr.log

@@ -0,0 +1,4 @@
+    Updating `dummy-registry` index
+      Adding hidden-feature-test v99999.0.0 to dependencies.
+             Features:
+             - not_secret

tests/testsuite/cargo_add/mod.rs

@@ -1,4 +1,6 @@
 mod add_basic;
+mod add_hidden_activated;
+mod add_hidden_unactivated;
 mod add_multiple;
 mod add_normalized_name_external;
 mod build;
@@ -185,4 +187,10 @@ fn add_registry_packages(alt: bool) {
         .feature("eyes", &[])
         .feature("ears", &[])
         .publish();
+
+    cargo_test_support::registry::Package::new("hidden-feature-test", "99999.0.0+my-package")
+        .alternative(alt)
+        .feature("_secret", &[])
+        .feature("not_secret", &[])
+        .publish();
 }