fix: properly handle Ctrl+C and Ctrl+D in interactive prompts

- Add TTY check to prevent prompting in non-interactive contexts
- Fix zone selection to properly propagate ErrInterrupt and ErrEOF
- Add interrupt handling for zone selection in config add
- Add test for Ctrl+C cancellation (exits 130, shows error)
- Update test for Ctrl+D cancellation (exits 0, graceful)
- All 18 E2E scenarios now passing

Author: natalie-o-perret

cmd/config/config.go

@@ -2,14 +2,14 @@ package config
 
 import (
 	"fmt"
-	"io"
 	"log"
 	"os"
 	"path"
 	"strings"
 
 	"github.com/manifoldco/promptui"
 	"github.com/spf13/cobra"
+	"golang.org/x/term"
 
 	exocmd "github.com/exoscale/cli/cmd"
 	"github.com/exoscale/cli/pkg/account"
@@ -33,6 +33,12 @@ func configCmdRun(cmd *cobra.Command, _ []string) error {
 		log.Fatalf("remove ENV credentials variables to use %s", cmd.CalledAs())
 	}
 
+	// Check if running non-interactively (no TTY)
+	if !isTerminal(os.Stdin.Fd()) {
+		printNoConfigMessage()
+		return fmt.Errorf("interactive terminal required for config setup")
+	}
+
 	if exocmd.GConfigFilePath != "" && account.CurrentAccount.Key != "" {
 		accounts := listAccounts(defaultAccountMark)
 		accounts = append(accounts, newAccountLabel)
@@ -246,15 +252,24 @@ func chooseZone(client *v3.Client, zones []string) (string, error) {
 
 	_, result, err := prompt.Run()
 	if err != nil {
-		if err == promptui.ErrInterrupt {
-			return "", io.EOF // Return io.EOF to signal cancellation
+		switch err {
+		case promptui.ErrInterrupt:
+			return "", promptui.ErrInterrupt // Propagate Ctrl+C
+		case promptui.ErrEOF:
+			return "", promptui.ErrEOF // Propagate Ctrl+D
+		default:
+			return "", fmt.Errorf("prompt failed: %w", err)
 		}
-		return "", fmt.Errorf("prompt failed: %w", err)
 	}
 
 	return result, nil
 }
 
+// isTerminal checks if the given file descriptor is a terminal
+func isTerminal(fd uintptr) bool {
+	return term.IsTerminal(int(fd))
+}
+
 func printNoConfigMessage() {
 	fmt.Print(`No Exoscale CLI configuration found
 

cmd/config/config_add.go

@@ -230,9 +230,28 @@ func promptAccountInformation() (*account.Account, error) {
 	}
 	account.DefaultZone, err = chooseZone(client, nil)
 	if err != nil {
+		// Handle prompt cancellation
+		if err == promptui.ErrInterrupt {
+			fmt.Fprintln(os.Stderr, "Error: Operation Cancelled")
+			os.Exit(exocmd.ExitCodeInterrupt)
+		}
+		if err == promptui.ErrEOF {
+			fmt.Fprintln(os.Stderr, "")
+			os.Exit(0)
+		}
+		// API error - try with fallback zones
 		for {
 			defaultZone, err := chooseZone(globalstate.EgoscaleV3Client, utils.AllZones)
 			if err != nil {
+				// Handle prompt cancellation in fallback
+				if err == promptui.ErrInterrupt {
+					fmt.Fprintln(os.Stderr, "Error: Operation Cancelled")
+					os.Exit(exocmd.ExitCodeInterrupt)
+				}
+				if err == promptui.ErrEOF {
+					fmt.Fprintln(os.Stderr, "")
+					os.Exit(0)
+				}
 				return nil, err
 			}
 			if defaultZone != "" {

tests/e2e/scenarios/without-api/config/add/add_cancel_with_ctrl_c.txtar

@@ -0,0 +1,9 @@
+# Test cancelling with Ctrl+C (interrupt) during API key prompt
+# Ctrl+C should show error message and exit with code 130
+
+# Attempt to add account and cancel with Ctrl+C
+! execpty --stdin=inputs exo config add
+stderr 'Error: Operation Cancelled'
+
+-- inputs --
+@ctrl+c

tests/e2e/scenarios/without-api/config/add/add_cancel_with_ctrl_d.txtar

@@ -1,9 +1,8 @@
 # Test cancelling with Ctrl+D (EOF) during API key prompt
-# Ctrl+D sends EOF signal which should be handled like Ctrl+C
+# Ctrl+D should exit gracefully with code 0 (not an error like Ctrl+C)
 
-# Attempt to add account and cancel with Ctrl+D
-! execpty --stdin=inputs exo config add
-stderr 'Error: Operation Cancelled'
+# Attempt to add account and cancel with Ctrl+D - should exit gracefully
+execpty --stdin=inputs exo config add
 
 -- inputs --
 @ctrl+d