refactor(actions): take rename's prompt off the TUI; drop tui from restack

internal/actions/rename.go

@@ -6,7 +6,6 @@ import (
 	"github.com/getstackit/stackit/internal/actions/validation"
 	"github.com/getstackit/stackit/internal/app"
 	"github.com/getstackit/stackit/internal/output"
-	"github.com/getstackit/stackit/internal/tui"
 	"github.com/getstackit/stackit/internal/utils"
 )
 
@@ -16,8 +15,18 @@ type RenameOptions struct {
 	Force   bool
 }
 
+// RenameHandler supplies a new branch name interactively when one is not given
+// on the command line. The CLI adapter implements it; the action layer does not
+// import the TUI.
+type RenameHandler interface {
+	// PromptNewName asks the user for a new branch name, given the current one.
+	// Implementations should return an error when prompting is not possible
+	// (e.g. non-interactive mode).
+	PromptNewName(currentName string) (string, error)
+}
+
 // RenameAction renames the current branch and updates metadata
-func RenameAction(ctx *app.Context, opts RenameOptions) error {
+func RenameAction(ctx *app.Context, opts RenameOptions, handler RenameHandler) error {
 	eng := ctx.Engine
 	out := ctx.Output
 
@@ -29,12 +38,8 @@ func RenameAction(ctx *app.Context, opts RenameOptions) error {
 
 	newName := opts.NewName
 	if newName == "" {
-		if !utils.IsInteractive() {
-			return fmt.Errorf("branch name is required in non-interactive mode")
-		}
-
 		var err error
-		newName, err = tui.PromptTextInput("Enter new branch name:", currentBranch)
+		newName, err = handler.PromptNewName(currentBranch)
 		if err != nil {
 			return err
 		}

internal/actions/restack.go

@@ -11,7 +11,6 @@ import (
 	"github.com/getstackit/stackit/internal/engine"
 	"github.com/getstackit/stackit/internal/handlers"
 	"github.com/getstackit/stackit/internal/rerere"
-	"github.com/getstackit/stackit/internal/tui"
 	"github.com/getstackit/stackit/internal/utils"
 )
 
@@ -154,7 +153,7 @@ func RestackAction(ctx *app.Context, plan *RestackPlan, handler handlers.Restack
 		handler = &handlers.NullRestackHandler{}
 	}
 
-	interactiveRererePrompt := ctx.Interactive && !ctx.Quiet && tui.IsTTY()
+	interactiveRererePrompt := ctx.Interactive && !ctx.Quiet && utils.IsTTY()
 	if _, jsonOutput := handler.(*handlers.JSONRestackHandler); jsonOutput {
 		interactiveRererePrompt = false
 	}

internal/cli/branch/rename.go

@@ -1,13 +1,29 @@
 package branch
 
 import (
+	"fmt"
+
 	"github.com/spf13/cobra"
 
 	"github.com/getstackit/stackit/internal/actions"
 	"github.com/getstackit/stackit/internal/app"
 	"github.com/getstackit/stackit/internal/cli/common"
+	"github.com/getstackit/stackit/internal/tui"
+	"github.com/getstackit/stackit/internal/utils"
 )
 
+// renameHandler prompts for a new branch name via the TUI when running
+// interactively, and reports that a name is required otherwise. It keeps the
+// TUI dependency in the CLI adapter, out of the action layer.
+type renameHandler struct{}
+
+func (renameHandler) PromptNewName(currentName string) (string, error) {
+	if !utils.IsInteractive() {
+		return "", fmt.Errorf("branch name is required in non-interactive mode")
+	}
+	return tui.PromptTextInput("Enter new branch name:", currentName)
+}
+
 // NewRenameCmd creates the rename command
 func NewRenameCmd() *cobra.Command {
 	var (
@@ -35,7 +51,7 @@ Note that this removes any association to a pull request, as GitHub pull request
 					Force:   force,
 				}
 
-				return actions.RenameAction(ctx, opts)
+				return actions.RenameAction(ctx, opts, renameHandler{})
 			})
 		},
 	}