Merge pull request #8 from Daniel-Ric/feature/2026-02-03/improve-cli-usability-and-aesthetics

CLI: Verbesserte Eingabe-UI, dezente Farben, Spinner und Page-Rendering

Author: Daniel-Ric

internal/cli/cli.go

@@ -75,13 +75,14 @@ func (a *App) askEdition() (ping.Edition, error) {
 }
 
 func (a *App) askHost() (string, error) {
+	var errMsg string
 	for {
-		value, err := promptInput("Server Host")
+		value, err := promptInput("Server Host", "z.B. play.example.com", errMsg)
 		if err != nil {
 			return "", err
 		}
 		if strings.TrimSpace(value) == "" {
-			fmt.Println("Host darf nicht leer sein")
+			errMsg = "Host darf nicht leer sein"
 			continue
 		}
 		return value, nil
@@ -90,8 +91,9 @@ func (a *App) askHost() (string, error) {
 
 func (a *App) askPort(edition ping.Edition) (int, error) {
 	defaultPort := ping.DefaultPort(edition)
+	var errMsg string
 	for {
-		value, err := promptInput(fmt.Sprintf("Port (%d)", defaultPort))
+		value, err := promptInput(fmt.Sprintf("Port (%d)", defaultPort), "Leer lassen für Standardport", errMsg)
 		if err != nil {
 			return 0, err
 		}
@@ -100,11 +102,11 @@ func (a *App) askPort(edition ping.Edition) (int, error) {
 		}
 		port, err := ping.ParsePort(value)
 		if err != nil {
-			fmt.Println(err)
+			errMsg = err.Error()
 			continue
 		}
 		if port == 0 {
-			fmt.Println("Port darf nicht leer sein")
+			errMsg = "Port darf nicht leer sein"
 			continue
 		}
 		return port, nil
@@ -115,14 +117,18 @@ func (a *App) execute(config Config) error {
 	ctx, cancel := context.WithTimeout(context.Background(), a.inputTimeout)
 	defer cancel()
 
-	result, err := ping.Execute(ctx, config.Edition, config.Host, config.Port)
+	resultText, err := withSpinner("Abfrage", "Server wird abgefragt", 120*time.Millisecond, func() (string, error) {
+		result, err := ping.Execute(ctx, config.Edition, config.Host, config.Port)
+		if err != nil {
+			return "", err
+		}
+		return result.String(), nil
+	})
 	if err != nil {
 		return err
 	}
 
-	fmt.Println()
-	fmt.Println(result.String())
-	fmt.Println()
+	renderTextPage("Ergebnis", resultText)
 	return nil
 }
 

internal/cli/ui.go

@@ -6,11 +6,44 @@ import (
 	"fmt"
 	"os"
 	"strings"
+	"time"
 )
 
-func promptInput(label string) (string, error) {
+const (
+	colorReset  = "\033[0m"
+	colorDim    = "\033[2m"
+	colorAccent = "\033[36m"
+	colorWarn   = "\033[33m"
+	colorBold   = "\033[1m"
+)
+
+func supportsColor() bool {
+	if os.Getenv("NO_COLOR") != "" {
+		return false
+	}
+	term := strings.TrimSpace(os.Getenv("TERM"))
+	return term != "" && term != "dumb"
+}
+
+func style(text, color string) string {
+	if !supportsColor() || color == "" {
+		return text
+	}
+	return color + text + colorReset
+}
+
+func promptInput(label, hint, errMsg string) (string, error) {
+	clearScreen()
+	fmt.Println(style(label, colorAccent))
+	if hint != "" {
+		fmt.Println(style(hint, colorDim))
+	}
+	if errMsg != "" {
+		fmt.Println(style(errMsg, colorWarn))
+	}
+	fmt.Println()
+	fmt.Print(style("> ", colorAccent))
 	reader := bufio.NewReader(os.Stdin)
-	fmt.Printf("%s: ", label)
 	value, err := reader.ReadString('\n')
 	if err != nil {
 		return "", err
@@ -19,6 +52,9 @@ func promptInput(label string) (string, error) {
 }
 
 func selectOption(title string, options []string) (int, error) {
+	if len(options) == 0 {
+		return 0, errors.New("keine Auswahloptionen verfügbar")
+	}
 	fd := int(os.Stdin.Fd())
 	state, err := makeRaw(fd)
 	if err != nil {
@@ -27,7 +63,7 @@ func selectOption(title string, options []string) (int, error) {
 	defer restore(fd, state)
 
 	selected := 0
-	renderMenu(title, options, selected)
+	renderMenu(title, options, selected, "")
 
 	reader := bufio.NewReader(os.Stdin)
 	for {
@@ -39,6 +75,16 @@ func selectOption(title string, options []string) (int, error) {
 		switch b {
 		case 3:
 			return 0, errors.New("abgebrochen")
+		case 'w', 'k':
+			if selected > 0 {
+				selected--
+			}
+			renderMenu(title, options, selected, "")
+		case 's', 'j':
+			if selected < len(options)-1 {
+				selected++
+			}
+			renderMenu(title, options, selected, "")
 		case 13, 10:
 			clearScreen()
 			return selected, nil
@@ -47,17 +93,29 @@ func selectOption(title string, options []string) (int, error) {
 			if err != nil {
 				return 0, err
 			}
-			if seq == "[A" {
+			if seq == "[A" || seq == "OA" {
 				if selected > 0 {
 					selected--
 				}
 			}
-			if seq == "[B" {
+			if seq == "[B" || seq == "OB" {
 				if selected < len(options)-1 {
 					selected++
 				}
 			}
-			renderMenu(title, options, selected)
+			renderMenu(title, options, selected, "")
+		case 0, 224:
+			code, err := reader.ReadByte()
+			if err != nil {
+				return 0, err
+			}
+			if code == 72 && selected > 0 {
+				selected--
+			}
+			if code == 80 && selected < len(options)-1 {
+				selected++
+			}
+			renderMenu(title, options, selected, "")
 		}
 	}
 }
@@ -71,24 +129,85 @@ func readEscapeSequence(reader *bufio.Reader) (string, error) {
 	if err != nil {
 		return "", err
 	}
+	if b1 == '[' && b2 >= '0' && b2 <= '9' {
+		b3, err := reader.ReadByte()
+		if err != nil {
+			return "", err
+		}
+		return string([]byte{b1, b2, b3}), nil
+	}
 	return string([]byte{b1, b2}), nil
 }
 
-func renderMenu(title string, options []string, selected int) {
+func renderMenu(title string, options []string, selected int, hint string) {
 	clearScreen()
-	fmt.Println(title)
+	fmt.Println(style(title, colorAccent))
 	fmt.Println()
 	for i, option := range options {
 		if i == selected {
-			fmt.Printf("> %s\n", option)
+			fmt.Printf("%s %s\n", style(">", colorAccent+colorBold), style(option, colorBold))
 			continue
 		}
 		fmt.Printf("  %s\n", option)
 	}
 	fmt.Println()
-	fmt.Println("Pfeiltasten zum Navigieren, Enter zum Bestätigen")
+	if hint == "" {
+		hint = "Pfeiltasten (oder W/S), Enter zum Bestätigen"
+	}
+	fmt.Println(style(hint, colorDim))
 }
 
 func clearScreen() {
 	fmt.Print("\033[H\033[2J")
 }
+
+func renderTextPage(title, content string) {
+	lines := strings.Split(content, "\n")
+	renderPage(title, lines)
+}
+
+func renderPage(title string, lines []string) {
+	clearScreen()
+	fmt.Println(style(title, colorAccent))
+	fmt.Println()
+	for _, line := range lines {
+		if strings.TrimSpace(line) == "" {
+			fmt.Println()
+			continue
+		}
+		fmt.Println(line)
+	}
+}
+
+func renderSpinnerPage(title, message, frame string) {
+	renderPage(title, []string{fmt.Sprintf("%s %s", style(message, colorDim), style(frame, colorAccent))})
+}
+
+func withSpinner(title, message string, tick time.Duration, action func() (string, error)) (string, error) {
+	resultCh := make(chan struct {
+		result string
+		err    error
+	}, 1)
+	go func() {
+		result, err := action()
+		resultCh <- struct {
+			result string
+			err    error
+		}{result: result, err: err}
+	}()
+
+	frames := []string{"⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"}
+	ticker := time.NewTicker(tick)
+	defer ticker.Stop()
+
+	frame := 0
+	for {
+		select {
+		case res := <-resultCh:
+			return res.result, res.err
+		case <-ticker.C:
+			renderSpinnerPage(title, message, frames[frame])
+			frame = (frame + 1) % len(frames)
+		}
+	}
+}