feat(update): only list selected updates in summary

Author: JamieMason

src/commands/update.rs

@@ -117,6 +117,29 @@ struct Bucket {
   indices: Vec<InstanceIdx>,
 }
 
+/// Keep only the rows the user picked, then rewrite each remaining row's
+/// `dependency_outdated_count` to the sum of bucket counts that survived
+/// the filter for its (group, dependency). Otherwise a multi-bucket dep
+/// with one bucket left would still print the original total.
+pub fn filter_rows_for_display(rows: &[UpdateRow], selection: &[bool]) -> Vec<UpdateRow> {
+  let mut filtered: Vec<UpdateRow> = rows
+    .iter()
+    .zip(selection.iter())
+    .filter(|&(_, &picked)| picked)
+    .map(|(row, _)| row.clone())
+    .collect();
+  let mut totals: std::collections::HashMap<(usize, String), usize> = std::collections::HashMap::new();
+  for row in &filtered {
+    *totals.entry((row.group_idx, row.dependency_name.clone())).or_insert(0) += row.bucket_count;
+  }
+  for row in &mut filtered {
+    if let Some(total) = totals.get(&(row.group_idx, row.dependency_name.clone())) {
+      row.dependency_outdated_count = *total;
+    }
+  }
+  filtered
+}
+
 pub fn run<D: DiskIo>(mut ctx: Context, registry_updates: RegistryUpdates, io: &D, tui: &dyn Tui) -> Result<Context, SyncpackError> {
   let now = update_row::unix_now();
   let rows = build_update_rows(&ctx, &registry_updates, now);
@@ -158,9 +181,12 @@ pub fn run<D: DiskIo>(mut ctx: Context, registry_updates: RegistryUpdates, io: &
     (None, (0..rows.len()).collect())
   };
 
-  update_row::render_rows(&rows, selection.as_deref());
+  let filtered_for_display: Option<Vec<UpdateRow>> = selection.as_deref().map(|sel| filter_rows_for_display(&rows, sel));
+  let display_rows: &[UpdateRow] = filtered_for_display.as_deref().unwrap_or(&rows);
+
+  update_row::render_rows(display_rows, None);
 
-  let counts = update_row::count_diffs(&rows);
+  let counts = update_row::count_diffs(display_rows);
   update_row::render_summary(counts);
 
   // Apply selected rows by copying expected → actual through the

src/commands/update_test.rs

@@ -1,6 +1,6 @@
 use {
   crate::{
-    commands::update::build_update_rows,
+    commands::update::{build_update_rows, filter_rows_for_display},
     test::{builder::TestBuilder, mock_tui::MockTui},
   },
   serde_json::json,
@@ -256,6 +256,45 @@ async fn mock_tui_cancel_returns_none() {
   assert!(tui.pick(&rows, 80, 24).is_none());
 }
 
+#[tokio::test]
+async fn filter_rows_for_display_drops_unselected_and_recomputes_dep_total() {
+  let (ctx, updates) = TestBuilder::new()
+    .with_packages(vec![
+      json!({
+        "name": "package-a",
+        "version": "1.0.0",
+        "dependencies": {"foo": "^1.0.0"}
+      }),
+      json!({
+        "name": "package-b",
+        "version": "1.0.0",
+        "dependencies": {"foo": "~1.0.0"}
+      }),
+      json!({
+        "name": "package-c",
+        "version": "1.0.0",
+        "dependencies": {"foo": "1.0.0"}
+      }),
+    ])
+    .with_registry_updates(json!({"foo": ["1.0.0", "1.0.1"]}))
+    .run_with_updates()
+    .await;
+  let rows = build_update_rows(&ctx, &updates.unwrap(), FROZEN_NOW);
+  assert_eq!(rows.len(), 3, "three buckets for foo");
+  for row in &rows {
+    assert_eq!(row.dependency_outdated_count, 3, "pre-filter total reflects all buckets");
+  }
+
+  let selection = vec![true, false, false];
+  let filtered = filter_rows_for_display(&rows, &selection);
+  assert_eq!(filtered.len(), 1, "only the selected bucket survives");
+  assert_eq!(
+    filtered[0].dependency_outdated_count, 1,
+    "dep total recomputed to selected bucket counts"
+  );
+  assert_eq!(filtered[0].bucket_count, 1);
+}
+
 mod apply {
   use {
     super::*,