fix: skip git-lfs on pre-push hook when calling manually

internal/git/lfs.go

@@ -9,11 +9,11 @@ const (
  LFSConfigFile = ".lfsconfig"
 )
 
-var lfsHooks = [...]string{
- "post-checkout",
- "post-commit",
- "post-merge",
- "pre-push",
+var lfsHooks = map[string]struct{}{
+ "post-checkout": {},
+ "post-commit": {},
+ "post-merge": {},
+ "pre-push": {},
 }
 
 // IsLFSAvailable returns 'true' if git-lfs is installed.
@@ -25,11 +25,6 @@ func IsLFSAvailable() bool {
 
 // IsLFSHook returns whether the hookName is supported by Git LFS.
 func IsLFSHook(hookName string) bool {
- for _, lfsHookName := range lfsHooks {
- if lfsHookName == hookName {
- return true
- }
- }
-
- return false
+ _, ok := lfsHooks[hookName]
+ return ok
 }

internal/lefthook/runner/runner.go

@@ -129,6 +129,11 @@ func (r *Runner) runLFSHook(ctx context.Context) error {
  return nil
  }
 
+ // Skip triggering LFS for pre-push hook when triggered manually
+ if len(r.GitArgs) == 0 && r.HookName == "pre-push" {
+ return nil
+ }
+
  lfsRequiredFile := filepath.Join(r.Repo.RootPath, git.LFSRequiredFile)
  lfsConfigFile := filepath.Join(r.Repo.RootPath, git.LFSConfigFile)