Display only feature list or summarized on cargo add

Author: Angelo Ross

src/cargo/ops/cargo_add/mod.rs

@@ -943,56 +943,49 @@ fn print_dep_table_msg(shell: &mut Shell, dep: &DependencyUI) -> CargoResult<()>
     if matches!(shell.verbosity(), crate::core::shell::Verbosity::Quiet) {
         return Ok(());
     }
+
     let (activated, deactivated) = dep.features();
     if !activated.is_empty() || !deactivated.is_empty() {
         let prefix = format!("{:>13}", " ");
-        let suffix = format_feature_version_suffix(&dep);
-
-        shell.write_stderr(format_args!("{}Features{}:\n", prefix, suffix), &style::NOP)?;
+        let suffix = format_features_version_suffix(&dep);
 
-        const MAX_FEATURE_PRINTS: usize = 50;
+        shell.write_stderr(format_args!("{prefix}Features{suffix}:\n"), &style::NOP)?;
 
-        let mut activated_printed = 0;
+        const MAX_FEATURE_PRINTS: usize = 30;
         let total_activated = activated.len();
-        for feat in activated {
-            if activated_printed >= MAX_FEATURE_PRINTS {
-                let remaining = total_activated - activated_printed;
-                shell.write_stderr(
-                    format_args!("{prefix}... {remaining} more activated features\n"),
-                    &style::NOP,
-                )?;
-                break;
-            }
+        let total_deactivated = deactivated.len();
 
-            shell.write_stderr(&prefix, &style::NOP)?;
-            shell.write_stderr('+', &style::GOOD)?;
-            shell.write_stderr(format_args!(" {}\n", feat), &style::NOP)?;
-            activated_printed += 1;
+        if total_activated <= MAX_FEATURE_PRINTS {
+            for feat in activated {
+                shell.write_stderr(&prefix, &style::NOP)?;
+                shell.write_stderr('+', &style::GOOD)?;
+                shell.write_stderr(format_args!(" {feat}\n"), &style::NOP)?;
+            }
+        } else {
+            shell.write_stderr(
+                format_args!("{prefix}{total_activated} activated features\n"),
+                &style::NOP,
+            )?;
         }
 
-        let mut deactivated_printed = 0;
-        let total_deactivated = deactivated.len();
-        for feat in deactivated {
-            if activated_printed + deactivated_printed >= MAX_FEATURE_PRINTS {
-                let remaining = total_deactivated - deactivated_printed;
-                shell.write_stderr(
-                    format_args!("{prefix}... {remaining} more deactivated features\n"),
-                    &style::NOP,
-                )?;
-                break;
+        if total_activated + total_deactivated <= MAX_FEATURE_PRINTS {
+            for feat in deactivated {
+                shell.write_stderr(&prefix, &style::NOP)?;
+                shell.write_stderr('-', &style::ERROR)?;
+                shell.write_stderr(format_args!(" {feat}\n"), &style::NOP)?;
             }
-
-            shell.write_stderr(&prefix, &style::NOP)?;
-            shell.write_stderr('-', &style::ERROR)?;
-            shell.write_stderr(format_args!(" {}\n", feat), &style::NOP)?;
-            deactivated_printed += 1;
+        } else {
+            shell.write_stderr(
+                format_args!("{prefix}{total_deactivated} deactivated features\n"),
+                &style::NOP,
+            )?;
         }
     }
 
     Ok(())
 }
 
-fn format_feature_version_suffix(dep: &DependencyUI) -> String {
+fn format_features_version_suffix(dep: &DependencyUI) -> String {
     if let Some(version) = &dep.available_version {
         let mut version = version.clone();
         version.build = Default::default();

tests/testsuite/cargo_add/features_too_many_activated/in/Cargo.toml renamed to tests/testsuite/cargo_add/features_activated_over_limit/in/Cargo.toml

@@ -0,0 +1,5 @@
+    Updating `dummy-registry` index
+      Adding your-face v99999.0.0 to dependencies.
+             Features:
+             100 activated features
+             100 deactivated features

tests/testsuite/cargo_add/features_too_many_activated/in/src/lib.rs renamed to tests/testsuite/cargo_add/features_activated_over_limit/in/src/lib.rs

@@ -8,7 +8,7 @@ use cargo_test_support::curr_dir;
 #[cargo_test]
 fn case() {
     const MANY_FEATURES_COUNT: usize = 200;
-    const ACTIVATED_FEATURES_COUNT: usize = 10;
+    const ACTIVATED_FEATURES_COUNT: usize = 30;
 
     cargo_test_support::registry::init();
     let mut test_package =