Replace '--reinstall' flag from 'install' by 'reinstall' action

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "fetchy"
-version = "0.13.12"
+version = "0.13.13"
 edition = "2021"
 description = "Quick packages installer"
 license = "Apache-2.0"

src/args.rs

@@ -22,12 +22,6 @@ pub enum Action {
         #[clap(help = "Name of the package(s) to install", required = true)]
         names: Vec<String>,
 
-        // #[clap(short, long, help = "Install from a specific repository")]
-        // repo: Option<String>,
-        //
-        #[clap(short, long, help = "Reinstall the package(s) if already installed")]
-        reinstall: bool,
-
         #[clap(
             short,
             long,
@@ -40,6 +34,12 @@ pub enum Action {
         discreet: bool,
     },
 
+    #[clap(about = "Re-install some already-installed package(s)")]
+    Reinstall {
+        #[clap(help = "Name of the package(s) to reinstall", required = true)]
+        names: Vec<String>,
+    },
+
     #[clap(about = "Update package(s)")]
     Update {
         #[clap(help = "Only update some package(s)")]

src/main.rs

@@ -91,17 +91,14 @@ async fn inner(action: Action) -> Result<()> {
     match action {
         Action::Install {
             names,
-            reinstall,
             check_updates,
             discreet,
         } => {
             let pkgs = resolve_pkgs_by_name_with_deps(names.as_slice(), &repos)?;
 
             install_pkgs(
                 pkgs,
-                if reinstall {
-                    InstalledPackagesHandling::Reinstall
-                } else if check_updates {
+                if check_updates {
                     InstalledPackagesHandling::CheckUpdates
                 } else {
                     InstalledPackagesHandling::Ignore
@@ -112,6 +109,18 @@ async fn inner(action: Action) -> Result<()> {
             .await?;
         }
 
+        Action::Reinstall { names } => {
+            let pkgs = resolve_installed_pkgs_by_name(&names, &db.installed, &repos)?;
+
+            let pkgs = pkgs
+                .into_iter()
+                .map(|(resolved, _)| resolved)
+                .map(refresh_pkg)
+                .collect::<Result<Vec<_>, _>>()?;
+
+            install_pkgs(pkgs, InstalledPackagesHandling::Reinstall, &mut db, false).await?;
+        }
+
         Action::Update { names } => {
             let pkgs = if !names.is_empty() {
                 resolve_installed_pkgs_by_name(&names, &db.installed, &repos)?