refactor(interactive): extract Prompt method and reduce duplication

internal/input/prompt.go

@@ -76,6 +76,14 @@ func (p *Prompter) Select(varName string, options []string) (string, error) {
 	return model.options[model.cursor], nil
 }
 
+// Prompt prompts for a variable value, using Select if enum is provided, Text otherwise
+func (p *Prompter) Prompt(varName string, enum []string) (string, error) {
+	if len(enum) > 0 {
+		return p.Select(varName, enum)
+	}
+	return p.Text(varName)
+}
+
 // textModel is the Bubble Tea model for text input
 type textModel struct {
 	varName   string

requires.go

@@ -9,16 +9,24 @@ import (
 	"github.com/go-task/task/v3/taskfile/ast"
 )
 
+func (e *Executor) canPrompt() bool {
+	return e.Interactive && (e.AssumeTerm || term.IsTerminal())
+}
+
+func (e *Executor) newPrompter() *input.Prompter {
+	return &input.Prompter{
+		Stdin:  e.Stdin,
+		Stdout: e.Stdout,
+		Stderr: e.Stderr,
+	}
+}
+
 // promptDepsVars traverses the dependency tree, collects all missing required
 // variables, and prompts for them upfront. This is used for deps which execute
 // in parallel, so all prompts must happen before execution to avoid interleaving.
 // Prompted values are stored in e.promptedVars for injection into task calls.
 func (e *Executor) promptDepsVars(calls []*Call) error {
-	if !e.Interactive {
+	if !e.canPrompt() {
-		return nil
-	}
-
-	if !e.AssumeTerm && !term.IsTerminal() {
 		return nil
 	}
 
@@ -73,32 +81,17 @@ func (e *Executor) promptDepsVars(calls []*Call) error {
 		return nil
 	}
 
-	// Prompt for all collected vars
+	prompter := e.newPrompter()
-	prompter := &input.Prompter{
-		Stdin:  e.Stdin,
-		Stdout: e.Stdout,
-		Stderr: e.Stderr,
-	}
-
 	e.promptedVars = ast.NewVars()
 
 	for _, v := range varsMap {
-		var value string
+		value, err := prompter.Prompt(v.Name, v.Enum)
-		var err error
-
-		if len(v.Enum) > 0 {
-			value, err = prompter.Select(v.Name, v.Enum)
-		} else {
-			value, err = prompter.Text(v.Name)
-		}
-
 		if err != nil {
 			if errors.Is(err, input.ErrCancelled) {
 				return &errors.TaskCancelledByUserError{TaskName: "interactive prompt"}
 			}
 			return err
 		}
-
 		e.promptedVars.Set(v.Name, ast.Var{Value: value})
 	}
 
@@ -109,11 +102,7 @@ func (e *Executor) promptDepsVars(calls []*Call) error {
 // Used for sequential task calls (cmds) where we can prompt just-in-time.
 // Returns true if any vars were prompted (caller should recompile the task).
 func (e *Executor) promptTaskVars(t *ast.Task, call *Call) (bool, error) {
-	if !e.Interactive || t.Requires == nil || len(t.Requires.Vars) == 0 {
+	if !e.canPrompt() || t.Requires == nil || len(t.Requires.Vars) == 0 {
-		return false, nil
-	}
-
-	if !e.AssumeTerm && !term.IsTerminal() {
 		return false, nil
 	}
 
@@ -135,22 +124,10 @@ func (e *Executor) promptTaskVars(t *ast.Task, call *Call) (bool, error) {
 		return false, nil
 	}
 
-	prompter := &input.Prompter{
+	prompter := e.newPrompter()
-		Stdin:  e.Stdin,
-		Stdout: e.Stdout,
-		Stderr: e.Stderr,
-	}
 
 	for _, v := range missing {
-		var value string
+		value, err := prompter.Prompt(v.Name, v.Enum)
-		var err error
-
-		if len(v.Enum) > 0 {
-			value, err = prompter.Select(v.Name, v.Enum)
-		} else {
-			value, err = prompter.Text(v.Name)
-		}
-
 		if err != nil {
 			if errors.Is(err, input.ErrCancelled) {
 				return false, &errors.TaskCancelledByUserError{TaskName: t.Name()}