chore: wip

Author: chrisbbreuer

bin/cli.ts

@@ -22,16 +22,27 @@ if (['1', 'true'].includes(SKIP_INSTALL_GIT_HOOKS || '')) {
 cli
   .command('[configPath]', 'Install git hooks, optionally from specified config file')
   .option('--verbose', 'Enable verbose logging')
+  .option('--force', 'Force reinstall hooks even if already installed')
   .example('git-hooks')
   .example('git-hooks ../src/config.ts')
   .example('git-hooks --verbose')
-  .action(async (configPath?: string, options?: { verbose?: boolean }) => {
+  .example('git-hooks --force')
+  .action(async (configPath?: string, options?: { verbose?: boolean, force?: boolean }) => {
     try {
       if (options?.verbose) {
         log.debug(`Config path: ${configPath || 'using default'}`)
         log.debug(`Working directory: ${process.cwd()}`)
       }
 
+      // Check if hooks are already installed unless --force is used
+      if (!options?.force) {
+        const { areHooksInstalled } = await import('../src/git-hooks')
+        if (areHooksInstalled(process.cwd())) {
+          log.info('Git hooks are already installed. Use --force to reinstall.')
+          process.exit(0)
+        }
+      }
+
       if (configPath) {
         const config = await import(configPath)
         setHooksFromConfig(process.cwd(), { configFile: config, verbose: options?.verbose ?? false })

package.json

@@ -41,7 +41,7 @@
     "git-hooks": "./dist/bin/cli.js",
     "bun-git-hooks": "./dist/bin/cli.js"
   },
-  "files": ["README.md", "dist"],
+  "files": ["README.md", "dist", "scripts"],
   "scripts": {
     "build": "bun build.ts && bun run compile",
     "compile": "bun build ./bin/cli.ts --compile --minify --outfile bin/git-hooks",
@@ -51,7 +51,7 @@
     "compile:windows-x64": "bun build ./bin/cli.ts --compile --minify --target=bun-windows-x64 --outfile bin/git-hooks-windows-x64.exe",
     "compile:darwin-x64": "bun build ./bin/cli.ts --compile --minify --target=bun-darwin-x64 --outfile bin/git-hooks-darwin-x64",
     "compile:darwin-arm64": "bun build ./bin/cli.ts --compile --minify --target=bun-darwin-arm64 --outfile bin/git-hooks-darwin-arm64",
-    "postinstall": "bunx git-hooks && bun ./scripts/postinstall.ts",
+    "postinstall": "bun ./scripts/postinstall.ts",
     "uninstall": "bun ./scripts/uninstall.ts",
     "fresh": "bunx rimraf node_modules/ bun.lock && bun i",
     "prepublishOnly": "bun --bun run build && bun run compile:all",

scripts/postinstall.ts

@@ -3,7 +3,7 @@
 import { mkdir, symlink } from 'node:fs/promises'
 import { join } from 'node:path'
 import process from 'node:process'
-import { checkBunGitHooksInDependencies, getProjectRootDirectoryFromNodeModules, setHooksFromConfig } from '../src/git-hooks'
+import { areHooksInstalled, checkBunGitHooksInDependencies, getProjectRootDirectoryFromNodeModules, setHooksFromConfig } from '../src/git-hooks'
 
 /**
  * Creates the pre-commit from command in config by default
@@ -42,7 +42,14 @@ async function postinstall() {
 
   if (checkBunGitHooksInDependencies(projectDirectory)) {
     try {
+      // Check if hooks are already installed to avoid unnecessary reinstalls
+      if (areHooksInstalled(projectDirectory)) {
+        console.log('[INFO] Git hooks are already installed, skipping setup')
+        return
+      }
+
       setHooksFromConfig(projectDirectory)
+      console.log('[INFO] Git hooks installed successfully')
     }
     catch (err) {
       console.log(`[ERROR] Was not able to set git hooks. Reason: ${err}`)

src/git-hooks.ts

@@ -185,6 +185,50 @@ function _getPackageJson(projectPath = process.cwd()): { packageJsonContent: any
   return { packageJsonContent: JSON.parse(packageJsonDataRaw), packageJsonPath: targetPackageJson }
 }
 
+/**
+ * Checks if git hooks are already installed and up-to-date
+ */
+export function areHooksInstalled(projectRootPath: string = process.cwd()): boolean {
+  const gitRoot = getGitProjectRoot(projectRootPath)
+  if (!gitRoot) {
+    return false
+  }
+
+  if (!config || Object.keys(config).length === 0) {
+    return false
+  }
+
+  // Check if at least one configured hook exists and contains our script marker
+  const configuredHooks = Object.keys(config).filter(key =>
+    VALID_GIT_HOOKS.includes(key as typeof VALID_GIT_HOOKS[number])
+  )
+
+  if (configuredHooks.length === 0) {
+    return false
+  }
+
+  // Check if all configured hooks exist
+  for (const hook of configuredHooks) {
+    const hookPath = path.normalize(path.join(gitRoot, 'hooks', hook))
+    if (!fs.existsSync(hookPath)) {
+      return false
+    }
+
+    // Verify the hook contains our prepend script (marker of our installation)
+    try {
+      const hookContent = fs.readFileSync(hookPath, 'utf-8')
+      if (!hookContent.includes('SKIP_BUN_GIT_HOOKS')) {
+        return false
+      }
+    }
+    catch {
+      return false
+    }
+  }
+
+  return true
+}
+
 /**
  * Parses the config and sets git hooks
  */
@@ -524,6 +568,7 @@ const gitHooks: {
   getGitProjectRoot: typeof getGitProjectRoot
   runStagedLint: typeof runStagedLint
   getStagedFiles: typeof getStagedFiles
+  areHooksInstalled: typeof areHooksInstalled
 } = {
   PREPEND_SCRIPT,
   setHooksFromConfig,
@@ -532,7 +577,8 @@ const gitHooks: {
   getProjectRootDirectoryFromNodeModules,
   getGitProjectRoot,
   runStagedLint,
-  getStagedFiles
+  getStagedFiles,
+  areHooksInstalled
 }
 
 export default gitHooks