fix: optimize the migration experience from husky.

Author: yyz945947732

README.md

@@ -96,11 +96,6 @@ If you need multiple verbose commands per git hook, flexible configuration or au
 3. Run the CLI script to update the git hooks with the commands from the config:
 
    ```sh
-   # [Optional] These 2 steps can be skipped for non-husky users
-   git config core.hooksPath .git/hooks/
-   rm -rf .git/hooks
-
-   # Update ./git/hooks
    npx simple-git-hooks
    ```
 
@@ -211,34 +206,6 @@ If you have the option to set arguments or environment variables, you can use th
 
 If these options are not available, you may need to resort to using the terminal for skipping hooks.
 
-### When migrating from `husky` git hooks are not running
-
-**Why is this happening?**
-
-Husky might change the `core.gitHooks` value to `.husky`, this way, git hooks would search `.husky` directory instead of `.git/hooks/`.
-
-Read more on git configuration in [Git book](https://git-scm.com/docs/githooks)
-
-You can check it by running this command inside of your repo:
-
-`git config core.hooksPath`
-
-If it outputs `.husky` then this is your case
-
-**How to fix?**
-
-you need to point `core.gitHooks` value to `your-awesome-project/.git/hooks`. You can use this command:
-
-`git config core.hooksPath .git/hooks/`
-
-validate the value is set:
-
-`git config core.hooksPath`
-
-should output: `.git/hooks/`
-
-Then remove the `.husky` folder that are generated previously by `husky`.
-
 ### I am getting "npx: command not found" error in a GUI git client
 
 This happens when using a node version manager such as `nodenv`, `nvm`, `mise` which require

simple-git-hooks.js

@@ -1,6 +1,7 @@
 const fs = require('fs')
 const path = require('path')
 const url = require('url')
+const { execSync } = require('child_process');
 
 const VALID_GIT_HOOKS = [
     'applypatch-msg',
@@ -183,7 +184,36 @@ async function setHooksFromConfig(projectRootPath=process.cwd(), argv=process.ar
 }
 
 /**
- * Creates or replaces an existing executable script in .git/hooks/<hook> with provided command
+ * Returns the absolute path to the Git hooks directory.
+ * Respects user-defined core.hooksPath from Git config if present;
+ * otherwise defaults to <gitRoot>/.git/hooks.
+ *
+ * @param {string} gitRoot - The absolute path to the Git project root
+ * @returns {string} - The resolved absolute path to the hooks directory
+ * @private
+ */
+function _getHooksPath(gitRoot) {
+    const defaultHooksPath = path.join(gitRoot, '.git', 'hooks')
+    try {
+        const customHooksPath = execSync('git config core.hooksPath', {
+            cwd: gitRoot,
+            encoding: 'utf8'
+        }).trim()
+
+        if (!customHooksPath) {
+            return defaultHooksPath
+        }
+
+        return path.isAbsolute(customHooksPath)
+            ? customHooksPath
+            : path.resolve(gitRoot, customHooksPath)
+    } catch {
+        return defaultHooksPath
+    }
+}
+
+/**
+ * Creates or replaces an existing executable script in the git hooks directory with provided command
  * @param {string} hook
  * @param {string} command
  * @param {string} projectRoot
@@ -198,12 +228,12 @@ function _setHook(hook, command, projectRoot=process.cwd()) {
     }
 
     const hookCommand = PREPEND_SCRIPT + command
-    const hookDirectory = gitRoot + '/hooks/'
-    const hookPath = path.normalize(hookDirectory + hook)
+    const hookDirectory = _getHooksPath(gitRoot)
+    const hookPath = path.join(hookDirectory, hook)
 
     const normalizedHookDirectory = path.normalize(hookDirectory)
     if (!fs.existsSync(normalizedHookDirectory)) {
-        fs.mkdirSync(normalizedHookDirectory)
+        fs.mkdirSync(normalizedHookDirectory, { recursive: true })
     }
 
     fs.writeFileSync(hookPath, hookCommand)