hooked-book/source/cli/install.md

@@ -20,7 +20,3 @@ Below is the default script template that Hooked uses, where `hook_type` is the
 ```sh
 {{#include ../../../hooked-cli/source/templates/default.sh}}
 ```
-
-You can provide your own template by using the `general.template` configuration setting. If you do, make sure you include a line somewhere that says `# Installed by Hooked.` for the [uninstall CLI command][cli-uninstall].
-
-[cli-uninstall]: ./uninstall.md

hooked-book/source/configuration/README.md

@@ -10,13 +10,11 @@ The `general` [table][toml-table] is for main Hooked configuration.
 |-----|------|---------|-------------|
 | config | String | Hooked.toml | The configuration file to use. If your configuration file isn't `Hooked.toml` you should set this accordingly. |
 | directory | String | hooks | The directory Hooked looks in for anything related to files. For example: scripts, templates, etc. |
-| template | Optional string | | Path to a custom template to be used when installing scripts. See the [install CLI command][cli-install] for more details. |
 
 ```toml
 [general]
 config = "Hooked.toml"
 directory = "hooks"
-template = "template.sh"
 ```
 
 ## Pre-commit
@@ -45,6 +43,5 @@ on_failure = "continue"
 
 [^command-and-script]: When both a command and script are defined in a hook, *only* the command will be run.
 
-[cli-install]: ../cli/install.md
 [toml-table]: https://toml.io/en/v1.0.0#table
 [toml-arrays-of-tables]: https://toml.io/en/v1.0.0#array-of-tables

hooked-cli/source/cli/install.rs

@@ -1,50 +0,0 @@
-//! The `install` subcommand.
-use std::{
-  fs::{read_to_string, set_permissions, write, Permissions},
-  os::unix::fs::PermissionsExt,
-  path::PathBuf,
-};
-
-use {
-  color_eyre::{eyre::eyre, Result},
-  hooked_config::Config,
-  tera::{Context, Tera},
-};
-
-use crate::{cli::InstallArgs, DEFAULT_TEMPLATE, HOOK_TYPES};
-
-/// The `install` subcommand.
-pub fn hooked_install(config: Config, args: InstallArgs) -> Result<()> {
-  let git_hooks_dir = PathBuf::from(".git/hooks/");
-  if !git_hooks_dir.exists() {
-    return Err(eyre!("The \".git/hooks/\" directory does not exist"));
-  }
-
-  let hooked_directory = config.general.directory;
-  for hook_type in HOOK_TYPES {
-    let mut context = Context::new();
-    context.insert("config_path", &config.general.config);
-    context.insert("hook_type", hook_type);
-
-    let hook_path = git_hooks_dir.join(hook_type);
-    if hook_path.exists() && !args.overwrite {
-      println!(
-        "{:?} exists, use --overwrite to replace the existing file",
-        hook_path
-      );
-      continue;
-    }
-
-    let template = match config.general.template.as_ref() {
-      Some(template_path) => {
-        read_to_string(hooked_directory.join(template_path))?
-      }
-      None => DEFAULT_TEMPLATE.to_string(),
-    };
-
-    write(&hook_path, Tera::one_off(&template, &context, false)?)?;
-    set_permissions(hook_path, Permissions::from_mode(0o775))?;
-  }
-
-  Ok(())
-}

hooked-cli/source/cli/mod.rs

@@ -2,15 +2,11 @@
 
 use std::path::PathBuf;
 
-use clap::{Args as Arguments, Parser, Subcommand};
+use clap::{Parser, Subcommand};
 
-mod install;
 mod run;
-mod uninstall;
 
-pub use install::hooked_install;
 pub use run::hooked_run;
-pub use uninstall::hooked_uninstall;
 
 /// CLI arguments struct using [`clap::Parser`].
 #[derive(Debug, Parser)]
@@ -30,35 +26,23 @@ pub struct Args {
 #[derive(Debug, Subcommand)]
 pub enum MainSubcommands {
   /// Install Hooked into ".git/hooks".
-  Install(InstallArgs),
+  Install {
+    /// Overwrite existing files.
+    #[clap(long)]
+    overwrite: bool,
+  },
 
   /// Remove installed hooks.
-  Uninstall(UninstallArgs),
+  Uninstall {
+    /// Remove hooks not installed by Hooked.
+    #[clap(long)]
+    all: bool,
+  },
 
   /// Manually run hooks.
-  Run(RunArgs),
+  Run {
-}
+    /// The hook type to run.
-
+    #[clap(value_parser = crate::HOOK_TYPES)]
-/// The `install` subcommand arguments.
+    hook_type: String,
-#[derive(Debug, Arguments)]
+  },
-pub struct InstallArgs {
-  /// Overwrite existing files.
-  #[clap(long)]
-  pub overwrite: bool,
-}
-
-/// The `uninstall` subcommand arguments.
-#[derive(Debug, Arguments)]
-pub struct UninstallArgs {
-  /// Remove hooks not installed by Hooked.
-  #[clap(long)]
-  pub all: bool,
-}
-
-/// The `run` subcommand arguments.
-#[derive(Debug, Arguments)]
-pub struct RunArgs {
-  /// The hook type to run.
-  #[clap(value_parser = crate::HOOK_TYPES)]
-  pub hook_type: String,
 }

hooked-cli/source/cli/uninstall.rs

@@ -1,40 +0,0 @@
-//! The `uninstall` subcommand.
-
-use std::{
-  fs::{read_to_string, remove_file},
-  path::PathBuf,
-};
-
-use {
-  color_eyre::{eyre::eyre, Result},
-  hooked_config::Config,
-};
-
-use crate::{cli::UninstallArgs, HOOK_TYPES};
-
-/// The `uninstall` subcommand.
-pub fn hooked_uninstall(_config: Config, args: UninstallArgs) -> Result<()> {
-  let git_hooks_dir = PathBuf::from(".git/hooks/");
-  if !git_hooks_dir.exists() {
-    return Err(eyre!("The \".git/hooks/\" directory does not exist"));
-  }
-
-  for hook_type in HOOK_TYPES {
-    let hook_path = git_hooks_dir.join(hook_type);
-    if !hook_path.exists() {
-      continue;
-    }
-
-    let hook_contents = read_to_string(&hook_path)?;
-    if args.all || hook_contents.contains("# Installed by Hooked.") {
-      remove_file(hook_path)?;
-    } else {
-      println!(
-        "{:?} wasn't installed by Hooked, use --all to remove it",
-        hook_path
-      );
-    }
-  }
-
-  Ok(())
-}

hooked-cli/source/main.rs

@@ -5,10 +5,17 @@
 #![forbid(unsafe_code)]
 #![warn(missing_docs, clippy::missing_docs_in_private_items)]
 
+use std::{
+  fs::{read_to_string, remove_file, set_permissions, write, Permissions},
+  os::unix::fs::PermissionsExt,
+  path::PathBuf,
+};
+
 use {
   clap::Parser,
-  color_eyre::{install, Result},
+  color_eyre::{eyre::eyre, install, Result},
   hooked_config::Config,
+  tera::{Context, Tera},
 };
 
 use crate::cli::{Args, MainSubcommands};
@@ -28,17 +35,61 @@ fn main() -> Result<()> {
   let args = Args::parse();
   let config = Config::from_toml_file(args.config)?;
 
+  let git_hooks_dir = PathBuf::from(".git/hooks/");
+
   match args.command {
-    MainSubcommands::Install(sub_args) => {
+    MainSubcommands::Install { overwrite } => {
-      cli::hooked_install(config, sub_args)?;
+      if !git_hooks_dir.exists() {
+        return Err(eyre!("The \".git/hooks/\" directory does not exist"));
+      }
+
+      for hook_type in HOOK_TYPES {
+        let mut context = Context::new();
+        context.insert("config_path", &config.general.config);
+        context.insert("hook_type", hook_type);
+
+        let hook_path = git_hooks_dir.join(hook_type);
+        if hook_path.exists() && !overwrite {
+          println!(
+            "{:?} exists, use --overwrite to replace the existing file",
+            hook_path
+          );
+          continue;
+        }
+
+        write(
+          &hook_path,
+          Tera::one_off(DEFAULT_TEMPLATE, &context, false)?,
+        )?;
+        set_permissions(hook_path, Permissions::from_mode(0o775))?;
+      }
     }
 
-    MainSubcommands::Uninstall(sub_args) => {
+    MainSubcommands::Uninstall { all } => {
-      cli::hooked_uninstall(config, sub_args)?;
+      if !git_hooks_dir.exists() {
+        return Err(eyre!("The \".git/hooks/\" directory does not exist"));
+      }
+
+      for hook_type in HOOK_TYPES {
+        let hook_path = git_hooks_dir.join(hook_type);
+        if !hook_path.exists() {
+          continue;
+        }
+
+        let hook_contents = read_to_string(&hook_path)?;
+        if all || hook_contents.contains("# Installed by Hooked.") {
+          remove_file(hook_path)?;
+        } else {
+          println!(
+            "{:?} wasn't installed by Hooked, use --all to remove it",
+            hook_path
+          );
+        }
+      }
     }
 
-    MainSubcommands::Run(sub_args) => {
+    MainSubcommands::Run { hook_type } => {
-      cli::hooked_run(config, sub_args.hook_type)?;
+      cli::hooked_run(config, hook_type)?;
     }
   }
 

hooked-config/source/config/general.rs

@@ -13,9 +13,6 @@ pub struct General {
 
   /// The directory to use for hooks.
   pub directory: PathBuf,
-
-  /// Path to a script template for use with the install subcommand.
-  pub template: Option<PathBuf>,
 }
 
 impl Default for General {
@@ -23,7 +20,6 @@ impl Default for General {
     Self {
       config: PathBuf::from("Hooked.toml"),
       directory: PathBuf::from("hooks"),
-      template: None,
     }
   }
 }

hooked-config/tests/parsing/with-hooks.toml

@@ -1,6 +1,5 @@
 [general]
 directory = "hooked"
-template = "test.sh"
 
 [[pre_commit]]
 name = "Pre Commit 1"

hooked-config/tests/snapshots/parsing__defaults.snap

@@ -6,7 +6,6 @@ Config {
     general: General {
         config: "Hooked.toml",
         directory: "hooks",
-        template: None,
     },
     pre_commit: [
         PreCommit {

hooked-config/tests/snapshots/parsing__with-hooks.snap

@@ -6,9 +6,6 @@ Config {
     general: General {
         config: "Hooked.toml",
         directory: "hooked",
-        template: Some(
-            "test.sh",
-        ),
     },
     pre_commit: [
         PreCommit {