fix: move user config to ~/.config/lefthook.yaml (#15)

~/.local/etc is, unfortunately, not a thing

Co-authored-by: Claude Code <noreply@anthropic.com>

Author: alecthomas

README.md

@@ -29,7 +29,7 @@ for repo configs), where `<ext>` is `yml`, `yaml`, `json`, `jsonc`, or `toml`.
 
 - Creates shell wrapper scripts for all standard git hooks in `~/.local/libexec/lhm/hooks/`, each invoking `lhm run-hook <hook>`
 - Sets `git config --global core.hooksPath ~/.local/libexec/lhm/hooks`
-- Writes a default `~/.local/etc/lefthook.yaml` if no user config exists
+- Writes a default `~/.config/lefthook.yaml` if no user config exists
 
 ### `lhm install --system`
 
@@ -82,14 +82,14 @@ When git triggers a hook, it runs the wrapper script in the hooks directory. Eac
 Config is resolved as a three-layer merge, where later layers override earlier ones:
 
 1. **System** (`/usr/local/etc/lefthook.yaml`) — organizational baseline
-2. **User global** (`~/.local/etc/lefthook.yaml`) — per-user overrides
+2. **User global** (`~/.config/lefthook.yaml`) — per-user overrides
 3. **Repo** (`$REPO/lefthook.yaml` or adapter) — per-repo overrides
 
 Any layer may be absent. When a repo has no lefthook config, the adapter system is used in its place (see below).
 
 ### Adapters
 
-When a repo has no `lefthook.yaml`, lhm checks for other git hook managers and transparently adapts them. The generated adapter config is merged with `~/.local/etc/lefthook.yaml` using the standard merging system, so global hooks still apply.
+When a repo has no `lefthook.yaml`, lhm checks for other git hook managers and transparently adapts them. The generated adapter config is merged with `~/.config/lefthook.yaml` using the standard merging system, so global hooks still apply.
 
 Adapters are tried in this order (first match wins):
 

src/config.rs

@@ -84,7 +84,7 @@ pub fn find_config(dir: &Path, check_dot_config: bool) -> Option<PathBuf> {
     None
 }
 
-/// Find the user-level lefthook config in the given directory (e.g. `~/.local/etc`).
+/// Find the user-level lefthook config in the given directory (e.g. `~/.config`).
 pub fn global_config(home: &Path, overrides: &ConfigOverrides) -> Option<PathBuf> {
     if let Some(ref p) = overrides.global_config {
         debug!("using global config override: {}", p.display());
@@ -121,7 +121,7 @@ pub fn install_default_global_config(dir: &Path) -> Result<(), String> {
     Ok(())
 }
 
-/// Load the user-level config from `~/.local/etc/lefthook.yaml` (or override) if it exists.
+/// Load the user-level config from `~/.config/lefthook.yaml` (or override) if it exists.
 pub fn load_global_config(dir: &Path, overrides: &ConfigOverrides) -> Result<Option<Value>, String> {
     match global_config(dir, overrides) {
         Some(path) => read_yaml(&path).map(Some),

src/main.rs

@@ -130,7 +130,10 @@ impl InstallScope {
     }
 
     fn config_dir(&self) -> PathBuf {
-        self.base_dir().join("etc")
+        match self {
+            Self::User => home_dir().join(".config"),
+            Self::System => self.base_dir().join("etc"),
+        }
     }
 }
 
@@ -566,16 +569,20 @@ mod tests {
 
     #[test]
     fn test_install_scope_user_config_dir() {
-        assert!(InstallScope::User.config_dir().ends_with(".local/etc"));
+        assert!(InstallScope::User.config_dir().ends_with(".config"));
     }
 
     #[test]
     fn test_install_scope_base_dir_shared_structure() {
         for scope in [InstallScope::User, InstallScope::System] {
             let base = scope.base_dir();
             assert_eq!(scope.hooks_dir(), base.join("libexec/lhm/hooks"));
-            assert_eq!(scope.config_dir(), base.join("etc"));
         }
+        assert_eq!(
+            InstallScope::System.config_dir(),
+            InstallScope::System.base_dir().join("etc")
+        );
+        assert!(InstallScope::User.config_dir().ends_with(".config"));
     }
 
     #[test]