Merge pull request #103 from siketyan/feat/open-in-default-app

feat: Open in default application (explorer, open)

src/application.rs

@@ -40,6 +40,26 @@ impl Application {
     }
 }
 
+#[cfg(windows)]
+impl Default for Application {
+    fn default() -> Self {
+        Self {
+            cmd: "explorer.exe".to_string(),
+            args: vec!["%p".to_string()],
+        }
+    }
+}
+
+#[cfg(not(windows))]
+impl Default for Application {
+    fn default() -> Self {
+        Self {
+            cmd: "open".to_string(),
+            args: vec!["%p".to_string()],
+        }
+    }
+}
+
 #[derive(Debug, Default, Deserialize)]
 pub struct Applications {
     #[serde(flatten)]
@@ -61,6 +81,16 @@ impl Applications {
         self.open(name, &path)
             .unwrap_or_else(|| Application::intermediate(name).open(&path))
     }
+
+    pub fn open_or_intermediate_or_default<P>(&self, name: Option<&str>, path: P) -> Result<()>
+    where
+        P: AsRef<Path>,
+    {
+        match name {
+            Some(n) => self.open_or_intermediate(n, path),
+            _ => Application::default().open(path),
+        }
+    }
 }
 
 impl Deref for Applications {

src/cmd/clone.rs

@@ -29,7 +29,7 @@ pub struct Cmd {
 
     /// Opens the directory after cloned a repository.
     #[clap(long)]
-    open: Option<String>,
+    open: Option<Option<String>>,
 }
 
 impl Cmd {
@@ -103,8 +103,11 @@ impl Cmd {
         };
 
         let open = if let Some(app) = &self.open {
-            config.applications.open_or_intermediate(app, &path)?;
-            Some(app.to_string())
+            config
+                .applications
+                .open_or_intermediate_or_default(app.as_deref(), &path)?;
+
+            Some(app.as_deref().unwrap_or("<default>").to_string())
         } else {
             None
         };

src/cmd/init.rs

@@ -23,7 +23,7 @@ pub struct Cmd {
 
     /// Opens the directory after cloned a repository.
     #[clap(long)]
-    open: Option<String>,
+    open: Option<Option<String>>,
 }
 
 impl Cmd {
@@ -68,11 +68,13 @@ impl Cmd {
         }
 
         if let Some(app) = self.open {
-            config.applications.open_or_intermediate(&app, &path)?;
+            config
+                .applications
+                .open_or_intermediate_or_default(app.as_deref(), &path)?;
 
             info!(
                 "Opened the repository in [{}] successfully.",
-                style(&app).bold(),
+                style(app.as_deref().unwrap_or("<default>")).bold(),
             );
         }
 

src/cmd/open.rs

@@ -14,7 +14,7 @@ pub struct Cmd {
     repo: String,
 
     /// Name of the application entry.
-    application: String,
+    application: Option<String>,
 }
 
 impl Cmd {
@@ -31,7 +31,7 @@ impl Cmd {
 
         config
             .applications
-            .open_or_intermediate(&self.application, path)?;
+            .open_or_intermediate_or_default(self.application.as_deref(), path)?;
 
         Ok(())
     }