fix: don't install custom hooks to hooks dir (#1246)

Author: scop

internal/command/install.go

@@ -298,12 +298,16 @@ func (l *Lefthook) createHooksIfNeeded(cfg *config.Config, hooks []string, force
 			continue
 		}
 
-		hookNames = append(hookNames, hook)
-
 		if err = l.cleanHook(hook, force); err != nil {
 			return fmt.Errorf("could not replace the hook: %w", err)
 		}
 
+		if _, ok := config.AvailableHooks[hook]; !ok {
+			continue
+		}
+
+		hookNames = append(hookNames, hook)
+
 		templateArgs := templates.Args{
 			Rc:                      cfg.Rc,
 			AssertLefthookInstalled: cfg.AssertLefthookInstalled,

internal/command/install_test.go

@@ -213,6 +213,36 @@ post-commit:
 				infoPath(config.ChecksumFileName),
 			},
 		},
+		{
+			name: "with custom hook",
+			config: `
+my-custom-hook:
+  commands:
+    custom:
+      run: echo 'Hello from custom!'
+`,
+			wantNotExist: []string{
+				hookPath("my-custom-hook"),
+			},
+		},
+		{
+			name: "with custom existing hook",
+			config: `
+my-custom-hook:
+  commands:
+    custom:
+      run: echo 'Hello from custom!'
+`,
+			existingFiles: map[string]string{
+				hookPath("my-custom-hook"): "",
+			},
+			wantExist: []string{
+				hookPath("my-custom-hook.old"),
+			},
+			wantNotExist: []string{
+				hookPath("my-custom-hook"),
+			},
+		},
 		{
 			name: "with unfetched remote",
 			config: `