fix(lfs): CommandExecutor.RawExecute forward os.Stdin

This is only used by LFS pre-push Hook which require Stdin.

internal/git/lfs.go

@@ -39,7 +39,7 @@ func IsLFSHook(hookName string) bool {
 }
 
 // DoesLFSHookConsumeStdin returns whether the LFS hookName will consume Stdin
-// meaning it won't be available to following commands
+// meaning it won't be available to following commands.
 func DoesLFSHookConsumeStdin(hookName string) bool {
 	for _, lfsHookName := range lfsHookConsumeStdin {
 		if lfsHookName == hookName {

internal/lefthook/runner/exec/execute_unix.go

@@ -72,12 +72,16 @@ func (e CommandExecutor) Execute(ctx context.Context, opts Options, out io.Write
 	return nil
 }
 
-func (e CommandExecutor) RawExecute(ctx context.Context, command []string, out io.Writer) error {
+func (e CommandExecutor) RawExecute(ctx context.Context, command []string, out io.Writer, forwardStdin bool) error {
 	cmd := exec.CommandContext(ctx, command[0], command[1:]...)
 
 	cmd.Stdout = out
 	cmd.Stderr = os.Stderr
 
+	if forwardStdin {
+		cmd.Stdin = os.Stdin
+	}
+
 	return cmd.Run()
 }
 

internal/lefthook/runner/exec/execute_windows.go

@@ -63,12 +63,16 @@ func (e CommandExecutor) Execute(ctx context.Context, opts Options, out io.Write
 	return nil
 }
 
-func (e CommandExecutor) RawExecute(ctx context.Context, command []string, out io.Writer) error {
+func (e CommandExecutor) RawExecute(ctx context.Context, command []string, out io.Writer, forwardStdin bool) error {
 	cmd := exec.Command(command[0], command[1:]...)
 
 	cmd.Stdout = out
 	cmd.Stderr = os.Stderr
 
+	if forwardStdin {
+		cmd.Stdin = os.Stdin
+	}
+
 	return cmd.Run()
 }
 

internal/lefthook/runner/exec/executor.go

@@ -17,5 +17,5 @@ type Options struct {
 // It is used here for testing purpose mostly.
 type Executor interface {
 	Execute(ctx context.Context, opts Options, out io.Writer) error
-	RawExecute(ctx context.Context, command []string, out io.Writer) error
+	RawExecute(ctx context.Context, command []string, out io.Writer, forwardStdin bool) error
 }

internal/lefthook/runner/runner.go

@@ -123,7 +123,7 @@ func (r *Runner) RunAll(ctx context.Context, sourceDirs []string) ([]Result, err
 	return results, nil
 }
 
-// returns whether it ran a LFS hook
+// returns whether it ran a LFS hook.
 func (r *Runner) runLFSHook(ctx context.Context) (bool, error) {
 	if !git.IsLFSHook(r.HookName) {
 		return false, nil
@@ -153,6 +153,7 @@ func (r *Runner) runLFSHook(ctx context.Context) (bool, error) {
 				r.GitArgs...,
 			),
 			out,
+			git.DoesLFSHookConsumeStdin(r.HookName),
 		)
 
 		output := strings.Trim(out.String(), "\n")

internal/lefthook/runner/runner_test.go

@@ -31,7 +31,7 @@ func (e TestExecutor) Execute(_ctx context.Context, opts exec.Options, _out io.W
 	return
 }
 
-func (e TestExecutor) RawExecute(_ctx context.Context, _command []string, _out io.Writer) error {
+func (e TestExecutor) RawExecute(_ctx context.Context, _command []string, _out io.Writer, forwardStdin bool) error {
 	return nil
 }