fix(cli/uninstall): arg '-g' can't be used with additional packages

Author: edilson258

cli/args/flags.rs

@@ -288,7 +288,7 @@ pub struct JupyterFlags {
 
 #[derive(Clone, Debug, Eq, PartialEq)]
 pub struct UninstallFlagsGlobal {
-  pub name: String,
+  pub packages: Vec<String>,
   pub root: Option<String>,
 }
 
@@ -2876,7 +2876,6 @@ The installation root is determined, in order of precedence:
       .arg(
         Arg::new("additional-packages")
           .help("List of additional packages to remove")
-          .conflicts_with("global")
           .num_args(1..)
           .action(ArgAction::Append)
       )
@@ -5189,19 +5188,19 @@ fn jupyter_parse(flags: &mut Flags, matches: &mut ArgMatches) {
 
 fn uninstall_parse(flags: &mut Flags, matches: &mut ArgMatches) {
   let name = matches.remove_one::<String>("name-or-package").unwrap();
+  let packages: Vec<_> = vec![name]
+    .into_iter()
+    .chain(
+      matches
+        .remove_many::<String>("additional-packages")
+        .unwrap_or_default(),
+    )
+    .collect();
 
   let kind = if matches.get_flag("global") {
     let root = matches.remove_one::<String>("root");
-    UninstallKind::Global(UninstallFlagsGlobal { name, root })
+    UninstallKind::Global(UninstallFlagsGlobal { packages, root })
   } else {
-    let packages: Vec<_> = vec![name]
-      .into_iter()
-      .chain(
-        matches
-          .remove_many::<String>("additional-packages")
-          .unwrap_or_default(),
-      )
-      .collect();
     UninstallKind::Local(RemoveFlags { packages })
   };
 
@@ -8916,7 +8915,7 @@ mod tests {
       Flags {
         subcommand: DenoSubcommand::Uninstall(UninstallFlags {
           kind: UninstallKind::Global(UninstallFlagsGlobal {
-            name: "file_server".to_string(),
+            packages: vec!["file_server".to_string()],
             root: None,
           }),
         }),
@@ -8937,7 +8936,7 @@ mod tests {
       Flags {
         subcommand: DenoSubcommand::Uninstall(UninstallFlags {
           kind: UninstallKind::Global(UninstallFlagsGlobal {
-            name: "file_server".to_string(),
+            packages: vec!["file_server".to_string()],
             root: Some("/user/foo/bar".to_string()),
           }),
         }),

cli/tools/installer.rs

@@ -245,31 +245,31 @@ pub async fn uninstall(
     }
   }
 
-  let file_path = installation_dir.join(&uninstall_flags.name);
+  for package_name in &uninstall_flags.packages {
+    let file_path = installation_dir.join(package_name);
 
-  let mut removed = remove_file_if_exists(&file_path)?;
+    let mut removed = remove_file_if_exists(&file_path)?;
 
-  if cfg!(windows) {
-    let file_path = file_path.with_extension("cmd");
-    removed |= remove_file_if_exists(&file_path)?;
-  }
+    if cfg!(windows) {
+      let file_path = file_path.with_extension("cmd");
+      removed |= remove_file_if_exists(&file_path)?;
+    }
 
-  if !removed {
-    return Err(anyhow!(
-      "No installation found for {}",
-      uninstall_flags.name
-    ));
-  }
+    if !removed {
+      return Err(anyhow!("No installation found for {}", package_name));
+    }
+
+    // There might be some extra files to delete
+    // Note: tsconfig.json is legacy. We renamed it to deno.json.
+    // Remove cleaning it up after January 2024
+    for ext in ["tsconfig.json", "deno.json", "lock.json"] {
+      let file_path = file_path.with_extension(ext);
+      remove_file_if_exists(&file_path)?;
+    }
 
-  // There might be some extra files to delete
-  // Note: tsconfig.json is legacy. We renamed it to deno.json.
-  // Remove cleaning it up after January 2024
-  for ext in ["tsconfig.json", "deno.json", "lock.json"] {
-    let file_path = file_path.with_extension(ext);
-    remove_file_if_exists(&file_path)?;
+    log::info!("✅ Successfully uninstalled {}", package_name);
   }
 
-  log::info!("✅ Successfully uninstalled {}", uninstall_flags.name);
   Ok(())
 }
 
@@ -1611,7 +1611,7 @@ mod tests {
       Default::default(),
       UninstallFlags {
         kind: UninstallKind::Global(UninstallFlagsGlobal {
-          name: "echo_test".to_string(),
+          packages: vec!["echo_test".to_string()],
           root: Some(temp_dir.path().to_string()),
         }),
       },