fix(clippy): satisfy new lints from nightly-2026-03-15

- collapsible_match: fold the bounds check into the KeyCode::Down match
  arm guard in vite_install package_manager
- unnecessary_sort_by: switch descending sorts in vite_js_runtime and
  vite_setup to sort_by_key with std::cmp::Reverse
- unnecessary_trailing_comma: drop trailing comma in a help.rs test
  assertion

Author: fengmk2

crates/vite_install/src/package_manager.rs

@@ -871,10 +871,8 @@ fn interactive_package_manager_menu() -> Result<PackageManagerType, Error> {
             KeyCode::Up => {
                 selected_index = selected_index.saturating_sub(1);
             }
-            KeyCode::Down => {
-                if selected_index < options.len() - 1 {
-                    selected_index += 1;
-                }
+            KeyCode::Down if selected_index < options.len() - 1 => {
+                selected_index += 1;
             }
             KeyCode::Enter | KeyCode::Char(' ') => {
                 break Ok(options[selected_index].1);

crates/vite_js_runtime/src/providers/node.rs

@@ -437,7 +437,7 @@ impl NodeProvider {
         }
 
         // Sort by major version descending (highest first)
-        lts_lines.sort_by(|a, b| b.1.cmp(&a.1));
+        lts_lines.sort_by_key(|b| std::cmp::Reverse(b.1));
 
         // offset is negative, so lts/-1 = index 1 (second highest)
         let index = (-offset) as usize;

crates/vite_setup/src/install.rs

@@ -475,7 +475,7 @@ pub async fn cleanup_old_versions(
     }
 
     // Sort newest first (by creation time, matching install.sh)
-    versions.sort_by(|a, b| b.0.cmp(&a.0));
+    versions.sort_by_key(|b| std::cmp::Reverse(b.0));
 
     // Remove versions beyond the keep limit, but never remove protected versions
     for (_time, path) in versions.into_iter().skip(max_keep) {

packages/cli/binding/src/cli/help.rs

@@ -225,7 +225,7 @@ mod tests {
         // accepted as task arguments instead of producing a parse error.
         let args = CLIArgs::try_parse_from(["vp", "run", "--yolo"]).unwrap();
         let debug = vite_str::format!("{args:?}");
-        assert!(debug.contains("\"--yolo\""), "Expected --yolo in task args, got: {debug}",);
+        assert!(debug.contains("\"--yolo\""), "Expected --yolo in task args, got: {debug}");
         assert!(matches!(args, CLIArgs::ViteTask(Command::Run(_))));
     }