wip

Author: asottile

src/clientlib.rs

@@ -59,25 +59,25 @@ pub(crate) struct ConfigHook;
 #[derive(Cfgv, Debug)]
 pub(crate) struct LocalRepo {
     #[cfgv_id]
-    repo: String,
-    hooks: Vec<ManifestHook>,
+    pub(crate) repo: String,
+    pub(crate) hooks: Vec<ManifestHook>,
 }
 
 #[allow(dead_code)]
 #[derive(Cfgv, Debug)]
 pub(crate) struct MetaRepo {
     #[cfgv_id]
-    repo: String,
-    hooks: Vec<ConfigHook>,
+    pub(crate) repo: String,
+    pub(crate) hooks: Vec<ConfigHook>,
 }
 
 #[allow(dead_code)]
 #[derive(Cfgv, Debug)]
 pub(crate) struct RemoteRepo {
     #[cfgv_id]
-    repo: String,
-    rev: String,
-    hooks: Vec<ConfigHook>,
+    pub(crate) repo: String,
+    pub(crate) rev: String,
+    pub(crate) hooks: Vec<ConfigHook>,
 }
 
 #[allow(dead_code)]
@@ -107,7 +107,7 @@ impl Cfgv for Repo {
 #[allow(dead_code)]
 #[derive(Cfgv, Debug)]
 pub(crate) struct Config {
-    repos: Vec<Repo>,
+    pub(crate) repos: Vec<Repo>,
 
     #[cfgv_default_expr(vec!["pre-commit".into()])]
     default_install_hook_types: Vec<String>,

src/commands/run.rs

@@ -1,5 +1,7 @@
+use crate::clientlib;
 use crate::env_ext;
 use crate::git;
+use crate::repository;
 use crate::staged_files_only;
 use crate::store;
 use crate::PreCommitEnv;
@@ -35,10 +37,15 @@ pub(crate) fn cmd(
     if stash {
         ctx = Some(staged_files_only::StagedFilesOnly::new(
             &repo,
-            store.directory,
+            &store.directory,
         )?);
     }
 
+    let config = clientlib::load_config(&config)?;
+    let hooks = repository::all_hooks(config, &store);
+    dbg!(cmd.hook_stage);
+    dbg!(hooks);
+
     drop(ctx);
     Ok(())
 }

src/git.rs

@@ -44,3 +44,63 @@ pub(crate) fn has_unstaged_config(repo: &gix::Repository, config: &str) -> anyho
 
     Ok(retc == 1)
 }
+
+// from `git rev-parse --local-env-vars`
+// TODO: use gix for this?
+const _LOCAL_ENV_VARS: &[&str] = &[
+    "GIT_ALTERNATE_OBJECT_DIRECTORIES",
+    "GIT_COMMON_DIR",
+    "GIT_CONFIG",
+    "GIT_CONFIG_COUNT",
+    "GIT_CONFIG_PARAMETERS",
+    "GIT_DIR",
+    "GIT_GRAFT_FILE",
+    "GIT_IMPLICIT_WORK_TREE",
+    "GIT_INDEX_FILE",
+    "GIT_INTERNAL_SUPER_PREFIX",
+    "GIT_NO_REPLACE_OBJECTS",
+    "GIT_OBJECT_DIRECTORY",
+    "GIT_PREFIX",
+    "GIT_REPLACE_REF_BASE",
+    "GIT_SHALLOW_FILE",
+    "GIT_WORK_TREE",
+];
+
+pub(crate) trait GitNoLocalEnv {
+    fn git_no_local_env(&mut self) -> &mut Self;
+}
+
+impl GitNoLocalEnv for process::Command {
+    fn git_no_local_env(&mut self) -> &mut Self {
+        for var in _LOCAL_ENV_VARS {
+            self.env_remove(var);
+        }
+        self
+    }
+}
+
+pub(crate) fn git_no_fs_monitor() -> process::Command {
+    let mut ret = process::Command::new("git");
+    ret.args(["-c", "core.useBuiltinFSMonitor=false"]);
+    ret
+}
+
+pub(crate) fn init_repo(path: &str, remote: &str) -> anyhow::Result<()> {
+    // TODO:
+    // if os.path.isdir(remote):
+    //      remote = os.path.abspath(remote)
+
+    // TODO: add this to gix?
+    // avoid the user's template so that hooks do not recurse
+    git_no_fs_monitor()
+        .git_no_local_env()
+        .args(["init", "--template=", path])
+        .output()?;
+    git_no_fs_monitor()
+        .git_no_local_env()
+        .current_dir(path)
+        .args(["remote", "add", "origin", remote])
+        .output()?;
+
+    Ok(())
+}

src/main.rs

@@ -9,6 +9,7 @@ mod clientlib;
 mod commands;
 mod env_ext;
 mod git;
+mod repository;
 mod staged_files_only;
 mod store;
 

src/repository.rs

@@ -0,0 +1,37 @@
+use crate::clientlib::{Config, LocalRepo, ManifestHook, MetaRepo, RemoteRepo, Repo};
+use crate::store::Store;
+use std::path;
+
+#[derive(Debug)]
+pub(crate) struct Hook {
+    src: String,
+    prefix: path::PathBuf,
+    hook: ManifestHook,
+}
+
+fn _local_hooks(repo: LocalRepo, store: &Store) -> anyhow::Result<Vec<Hook>> {
+    anyhow::bail!("not implemented!")
+}
+
+fn _meta_hooks(repo: MetaRepo, store: &Store) -> anyhow::Result<Vec<Hook>> {
+    anyhow::bail!("not implemented!")
+}
+
+fn _cloned_hooks(repo: RemoteRepo, store: &Store) -> anyhow::Result<Vec<Hook>> {
+    let path = store.clone_repo(&repo.repo, &repo.rev)?;
+    anyhow::bail!("not implemented!")
+}
+
+pub(crate) fn all_hooks(config: Config, store: &Store) -> anyhow::Result<Vec<Hook>> {
+    let mut ret: Vec<Hook> = Vec::new();
+
+    for repo in config.repos.into_iter() {
+        match repo {
+            Repo::Local(local_repo) => ret.extend(_local_hooks(local_repo, store)?),
+            Repo::Meta(meta_repo) => ret.extend(_meta_hooks(meta_repo, store)?),
+            Repo::Remote(repo) => ret.extend(_cloned_hooks(repo, store)?),
+        }
+    }
+
+    Ok(ret)
+}

src/store.rs

@@ -41,6 +41,29 @@ fn _readonly(d: &path::Path) -> bool {
     d.exists() && d.access(AccessMode::WRITE).is_err()
 }
 
+trait MakeRepo {
+    fn make(&self, dest: &str) -> anyhow::Result<()>;
+}
+
+struct LocalRepoMaker {}
+
+impl MakeRepo for LocalRepoMaker {
+    fn make(&self, dest: &str) -> anyhow::Result<()> {
+        anyhow::bail!("not implemented!");
+    }
+}
+
+struct ClonedRepoMaker<'a> {
+    repo: &'a str,
+    rev: &'a str,
+}
+
+impl MakeRepo for ClonedRepoMaker<'_> {
+    fn make(&self, dest: &str) -> anyhow::Result<()> {
+        anyhow::bail!("not implemented!");
+    }
+}
+
 pub(crate) struct Store {
     pub(crate) directory: path::PathBuf,
     pub(crate) readonly: bool,
@@ -121,6 +144,10 @@ impl Store {
         Ok(ret)
     }
 
+    pub(crate) fn clone_repo(&self, repo: &str, rev: &str) -> anyhow::Result<String> {
+        anyhow::bail!("not implemented!");
+    }
+
     pub(crate) fn mark_config_used(&self, path: &str) -> anyhow::Result<()> {
         if self.readonly {
             return Ok(());