Remove hook install_key (#153)

src/cli/run/run.rs

@@ -269,8 +269,8 @@ pub async fn install_hooks(hooks: &[Hook], reporter: &HookInstallReporter) -> Re
     let to_install = hooks
         .iter()
         .filter(|hook| !hook.installed())
-        .unique_by(|hook| hook.install_key())
-        .filter_map(|hook| hook.environment_dir().map(|env_dir| (hook, env_dir)));
+        .filter_map(|hook| hook.environment_dir().map(|env_dir| (hook, env_dir)))
+        .unique_by(|(_, env_dir)| env_dir.clone());
 
     let mut tasks = futures::stream::iter(to_install)
         .map(|(hook, env_dir)| async move {

src/hook.rs

@@ -530,16 +530,6 @@ impl Hook {
         )
     }
 
-    pub fn install_key(&self) -> String {
-        format!(
-            "{}-{}-{}-{}",
-            self.repo,
-            self.language,
-            self.language_version,
-            self.additional_dependencies.join(",")
-        )
-    }
-
     // TODO: health check
     /// Check if the hook is installed in the environment.
     pub fn installed(&self) -> bool {

src/store.rs

@@ -238,6 +238,7 @@ impl Store {
             );
             clone_repo(repo_config.repo.as_str(), &repo_config.rev, temp.path()).await?;
         } else {
+            // FIXME: Do not copy env dir.
             // TODO: use hardlink?
             // Optimization: This is an optimization from the Python pre-commit implementation.
             // Copy already cloned base remote repo.