chore: venv update

Author: denis256

internal/shell/run_cmd.go

@@ -33,12 +33,13 @@ const SignalForwardingDelay = time.Second * 15
 
 // ShellOptions contains the configuration needed to run shell commands.
 type ShellOptions struct {
-	Writers         writer.Writers
-	EngineOptions   *engine.EngineOptions
-	EngineConfig    *engine.EngineConfig
-	Telemetry       *telemetry.Options
-	Env             map[string]string
-	Exec            vexec.Exec
+	Writers       writer.Writers
+	EngineOptions *engine.EngineOptions
+	EngineConfig  *engine.EngineConfig
+	Telemetry     *telemetry.Options
+	Env           map[string]string
+	Exec          vexec.Exec
+
 	RootWorkingDir  string
 	WorkingDir      string
 	TFPath          string
@@ -153,8 +154,7 @@ func (o *ShellOptions) WithForwardTFStdout(f bool) *ShellOptions {
 	return o
 }
 
-// WithExec installs a vexec.Exec backend that replaces the default os/exec
-// path used by RunCommandWithOutput. Pass nil to clear it. Intended for tests.
+// WithExec sets the vexec.Exec used by RunCommandWithOutput.
 func (o *ShellOptions) WithExec(e vexec.Exec) *ShellOptions {
 	o.Exec = e
 
@@ -258,6 +258,10 @@ func RunCommandWithOutput(
 		}
 
 		if runOpts.Exec != nil {
+			if needsPTY {
+				return errors.New("shell: vexec backend does not support PTY mode")
+			}
+
 			injectedCmd := runOpts.Exec.Command(ctx, command, args...)
 			injectedCmd.SetDir(commandDir)
 			injectedCmd.SetEnv(util.EnvSliceFromMap(runOpts.Env))
@@ -305,6 +309,7 @@ func RunCommandWithOutput(
 
 // procExecError builds a ProcessExecutionError using the standard fields from ShellOptions.
 // Pass nil for output when the failure occurred before any output was captured.
+// Caller guarantees a non-nil ShellOptions; the receiver is dereferenced unconditionally.
 func (o *ShellOptions) procExecError(err error, command, dir string, args []string, output *util.CmdOutput) error {
 	pe := util.ProcessExecutionError{
 		Err:             err,

pkg/config/dependency.go

@@ -41,6 +41,7 @@ import (
 	"github.com/gruntwork-io/terragrunt/internal/shell"
 	"github.com/gruntwork-io/terragrunt/internal/tf"
 	"github.com/gruntwork-io/terragrunt/internal/util"
+	"github.com/gruntwork-io/terragrunt/internal/vexec"
 	"github.com/gruntwork-io/terragrunt/pkg/config/hclparse"
 )
 
@@ -1449,6 +1450,11 @@ func runTerragruntOutputJSON(ctx context.Context, pctx *ParsingContext, l log.Lo
 
 // shellRunOptsFromPctx builds a *shell.ShellOptions from ParsingContext flat fields.
 func shellRunOptsFromPctx(pctx *ParsingContext) *shell.ShellOptions {
+	exec := pctx.Exec
+	if exec == nil {
+		exec = vexec.NewOSExec()
+	}
+
 	return shell.NewShellOptions().
 		WithWorkingDir(pctx.WorkingDir).
 		WithEnv(pctx.Env).
@@ -1460,7 +1466,7 @@ func shellRunOptsFromPctx(pctx *ParsingContext) *shell.ShellOptions {
 		WithExperiments(pctx.Experiments).
 		WithHeadless(pctx.Headless).
 		WithForwardTFStdout(pctx.ForwardTFStdout).
-		WithExec(pctx.Exec)
+		WithExec(exec)
 }
 
 // tfRunOptsFromPctx builds a *tf.RunOptions from ParsingContext flat fields.

pkg/config/parsing_context.go

@@ -53,9 +53,6 @@ type ParsingContext struct {
 	Features            *cty.Value
 	Locals              *cty.Value
 
-	// Exec, when non-nil, replaces the default os/exec backend used by run_cmd
-	// and other shell-outs that read shell.ShellOptions from this context.
-	// Intended for tests; production code leaves this nil.
 	Exec vexec.Exec
 
 	Env                 map[string]string