feat: replace CLI output with styled Charm-based UI

Migrate all CLI output to the Charm ecosystem (Bubble Tea, lipgloss, Huh)
for a polished terminal experience with proper headless/CI mode fallback.

- Add Info, Success, ErrorMsg, Warn methods to UI with lipgloss styling
- Style spinner final/error messages with checkmark/cross prefixes
- Fix "No servers found" to exit 0 instead of 1 (not an error)
- Replace all RawMessage/PrintMixedGreen calls with typed UI methods
- Add interactive forms via charmbracelet/huh replacing survey
- Add styled tables via lipgloss replacing tablewriter
- Log spinner errors at error level in headless mode
- Update README libraries section to reflect new dependencies

Author: dirien

.golangci.yaml

@@ -1,31 +1,121 @@
-linters:
+version: "2"
+
+formatters:
   enable:
-    - thelper
     - gofumpt
-    - tparallel
-    - unconvert
-    - unparam
-    - wastedassign
-    - revive
+    - goimports
+
+linters:
+  enable:
+    # bugs
+    - bodyclose
+    - durationcheck
+    - errcheck
+    - errorlint
+    - gosec
+    - govet
+    - loggercheck
+    - nilerr
+    - noctx
+    - recvcheck
+    - sqlclosecheck
+
+    # performance
+    - copyloopvar
+    - intrange
+    - mirror
+    - perfsprint
+    - prealloc
+    - usestdlibvars
+
+    # style & conventions
+    - asciicheck
+    - bidichk
+    - dupword
+    - errname
     - forbidigo
-    - tagliatelle
-    - typecheck
     - goconst
     - gocritic
     - gocyclo
-    - govet
-    - errcheck
-    - gosec
-    - goimports
+    - goprintffuncname
+    - misspell
+    - modernize
     - nolintlint
+    - nosprintfhostport
+    - predeclared
+    - revive
+    - tagliatelle
+    - unconvert
+    - unparam
+    - wastedassign
+    - whitespace
+
+    # testing
+    - thelper
+    - tparallel
+    - usetesting
+
+    # unused / dead code
+    - fatcontext
+    - reassign
 
-linters-settings:
-  forbidigo:
-    forbid:
-      - 'ioutil\.*'
-  tagliatelle:
-    case:
-      use-field-name: false
+  settings:
+    forbidigo:
+      forbid:
+        - pattern: 'ioutil\.*'
+          msg: "use os and io packages instead of ioutil"
+      exclude-godoc-examples: true
+      analyze-types: true
+    tagliatelle:
+      case:
+        use-field-name: false
+        rules:
+          yaml: goCamel
+          json: snake
+    errorlint:
+      asserts: true
+      comparison: true
+      errorf: true
+    gocritic:
+      enabled-tags:
+        - diagnostic
+        - style
+        - performance
+      disabled-checks:
+        - hugeParam
+        - importShadow
+    revive:
       rules:
-        yaml: goCamel
-        json: snake
+        - name: blank-imports
+        - name: context-as-argument
+        - name: dot-imports
+        - name: error-naming
+        - name: error-return
+        - name: error-strings
+        - name: exported
+          disabled: true
+        - name: increment-decrement
+        - name: indent-error-flow
+        - name: package-comments
+          disabled: true
+        - name: range
+        - name: receiver-naming
+        - name: redefines-builtin-id
+        - name: superfluous-else
+        - name: unreachable-code
+        - name: var-naming
+    misspell:
+      locale: US
+      ignore-rules:
+        - organisation
+    gocyclo:
+      min-complexity: 15
+    perfsprint:
+      err-error: true
+      errorf: true
+      sprintf1: true
+    gosec:
+      excludes:
+        - G104
+        - G204
+        - G304

README.md

@@ -193,7 +193,10 @@ Feel free to join! See our [contribution guidelines](CONTRIBUTING.md).
 <details>
 <summary>Click to expand</summary>
 
-- https://github.com/fatih/color
+- https://github.com/charmbracelet/bubbletea
+- https://github.com/charmbracelet/lipgloss
+- https://github.com/charmbracelet/huh
+- https://github.com/charmbracelet/bubbles
 - https://github.com/melbahja/goph
 - https://github.com/spf13/cobra
 - https://github.com/goreleaser
@@ -202,7 +205,6 @@ Feel free to join! See our [contribution guidelines](CONTRIBUTING.md).
 - https://github.com/scaleway/scaleway-sdk-go
 - https://github.com/linode/linodego
 - https://github.com/hetznercloud/hcloud-go
-- https://github.com/olekukonko/tablewriter
 - https://github.com/sethvargo/go-password
 - https://github.com/ovh/go-ovh
 - https://github.com/dirien/ovh-go-sdk
@@ -217,8 +219,7 @@ Feel free to join! See our [contribution guidelines](CONTRIBUTING.md).
 - https://github.com/tcnksm/go-latest
 - https://github.com/uber-go/zap
 - https://github.com/oracle/oci-go-sdk
-- https://github.com/AlecAivazis/survey
-- https://github.com/aws/aws-sdk-go
+- https://github.com/aws/aws-sdk-go-v2
 - https://github.com/gophercloud/gophercloud
 - https://github.com/exoscale/egoscale
 

cmd/minectl/create.go

@@ -2,18 +2,17 @@ package minectl
 
 import (
 	"fmt"
-	"os"
 
 	"github.com/dirien/minectl/internal/provisioner"
-	"github.com/olekukonko/tablewriter"
+	"github.com/dirien/minectl/internal/ui"
 
 	"github.com/pkg/errors"
 	"github.com/spf13/cobra"
 )
 
 func init() {
 	createCmd.Flags().StringP("filename", "f", "", "Location of the manifest file")
-	createCmd.Flags().SetAnnotation("filename", cobra.BashCompFilenameExt, []string{"yaml"}) //nolint:errcheck
+	_ = createCmd.Flags().SetAnnotation("filename", cobra.BashCompFilenameExt, []string{"yaml"})
 	createCmd.Flags().BoolP("wait", "w", true, "Wait for Minecraft Server is started")
 }
 
@@ -29,15 +28,15 @@ var createCmd = &cobra.Command{
 
 func runCreate(cmd *cobra.Command, _ []string) error {
 	filename, err := cmd.Flags().GetString("filename")
-	if len(filename) == 0 {
+	if filename == "" {
 		return errors.New("Please provide a valid manifest file via -f|--filename flag")
 	}
 	if err != nil {
 		return errors.Wrap(err, "Please provide a valid manifest file")
 	}
 	p, err := provisioner.NewProvisioner(&provisioner.MinectlProvisionerOpts{
 		ManifestPath: filename,
-	}, minectlLog)
+	}, minectlUI)
 	if err != nil {
 		return err
 	}
@@ -50,20 +49,17 @@ func runCreate(cmd *cobra.Command, _ []string) error {
 		return err
 	}
 	if !headless {
-		table := tablewriter.NewTable(os.Stdout,
-			tablewriter.WithHeader([]string{"ID", "NAME", "REGION", "TAGS", "IP"}),
-		)
+		table := ui.NewTable(minectlUI, "ID", "NAME", "REGION", "TAGS", "IP")
 		table.Append([]string{res.ID, res.Name, res.Region, res.Tags, res.PublicIP})
 
 		fmt.Println("")
 		table.Render()
 
-		minectlLog.PrintMixedGreen("\n🪓 To delete the server type:\n\n %s", fmt.Sprintf("minectl delete -f %s --id %s\n", filename, res.ID))
-		minectlLog.PrintMixedGreen("\n🆙 To update the server type:\n\n %s", fmt.Sprintf("minectl update -f %s --id %s\n", filename, res.ID))
-		minectlLog.PrintMixedGreen("\n🔌 Connected to RCON type:\n\n %s", fmt.Sprintf("minectl rcon -f %s --id %s\n", filename, res.ID))
-		minectlLog.RawMessage("🚧 Beta features:")
-		minectlLog.PrintMixedGreen("⤴️ To upload a plugin type:\n\n %s",
-			fmt.Sprintf("minectl plugins -f %s --id %s --plugin <folder>/x.jar --destination /minecraft/plugins\n", filename, res.ID))
+		minectlUI.Info(fmt.Sprintf("To delete the server:\n\n  minectl delete -f %s --id %s", filename, res.ID))
+		minectlUI.Info(fmt.Sprintf("To update the server:\n\n  minectl update -f %s --id %s", filename, res.ID))
+		minectlUI.Info(fmt.Sprintf("To connect via RCON:\n\n  minectl rcon -f %s --id %s", filename, res.ID))
+		minectlUI.Warn("Beta features:")
+		minectlUI.Info(fmt.Sprintf("To upload a plugin:\n\n  minectl plugins -f %s --id %s --plugin <folder>/x.jar --destination /minecraft/plugins", filename, res.ID))
 	}
 	return nil
 }

cmd/minectl/delete.go

@@ -1,19 +1,15 @@
 package minectl
 
 import (
-	"bufio"
-	"fmt"
-	"os"
-	"strings"
-
 	"github.com/dirien/minectl/internal/provisioner"
+	"github.com/dirien/minectl/internal/ui"
 	"github.com/pkg/errors"
 	"github.com/spf13/cobra"
 )
 
 func init() {
 	deleteCmd.Flags().StringP("filename", "f", "", "Location of the manifest file")
-	deleteCmd.Flags().SetAnnotation("filename", cobra.BashCompFilenameExt, []string{"yaml"}) //nolint:errcheck
+	_ = deleteCmd.Flags().SetAnnotation("filename", cobra.BashCompFilenameExt, []string{"yaml"})
 	deleteCmd.Flags().String("id", "", "Contains the server id")
 	deleteCmd.Flags().BoolP("yes", "y", false, "Automatically delete the server")
 }
@@ -35,20 +31,20 @@ func runDelete(cmd *cobra.Command, _ []string) error {
 	if err != nil {
 		return errors.Wrap(err, "failed to get 'filename' value")
 	}
-	if len(filename) == 0 {
+	if filename == "" {
 		return errors.New("Please provide a valid manifest file via -f|--filename flag")
 	}
 	id, err := cmd.Flags().GetString("id")
 	if err != nil {
 		return err
 	}
-	if len(id) == 0 {
+	if id == "" {
 		return errors.New("Please provide a valid id")
 	}
 	newProvisioner, err := provisioner.NewProvisioner(&provisioner.MinectlProvisionerOpts{
 		ManifestPath: filename,
 		ID:           id,
-	}, minectlLog)
+	}, minectlUI)
 	if err != nil {
 		return err
 	}
@@ -60,15 +56,17 @@ func runDelete(cmd *cobra.Command, _ []string) error {
 			return err
 		}
 	} else {
-		fmt.Print("🗑️Do you want to delete the Minecraft server? [y/N]: ")
-		reader := bufio.NewReader(os.Stdin)
-		text, _ := reader.ReadString('\n')
-		text = strings.ReplaceAll(text, "\n", "")
-		if strings.Compare("y", text) == 0 {
+		confirmed, err := ui.Confirm("Do you want to delete the Minecraft server?")
+		if err != nil {
+			return err
+		}
+		if confirmed {
 			err = newProvisioner.DeleteServer()
 			if err != nil {
 				return err
 			}
+		} else {
+			minectlUI.Warn("Delete canceled.")
 		}
 	}
 

cmd/minectl/list.go

@@ -2,10 +2,9 @@ package minectl
 
 import (
 	"fmt"
-	"os"
 
 	"github.com/dirien/minectl/internal/provisioner"
-	"github.com/olekukonko/tablewriter"
+	"github.com/dirien/minectl/internal/ui"
 	"github.com/pkg/errors"
 	"github.com/spf13/cobra"
 )
@@ -31,7 +30,7 @@ func runList(cmd *cobra.Command, _ []string) error {
 	if err != nil {
 		return errors.Wrap(err, "failed to get 'provider' value")
 	}
-	if len(provider) == 0 {
+	if provider == "" {
 		return errors.New("Please provide a valid 'provider' value")
 	}
 	region, err := cmd.Flags().GetString("region")
@@ -42,7 +41,7 @@ func runList(cmd *cobra.Command, _ []string) error {
 	newProvisioner, err := provisioner.ListProvisioner(&provisioner.MinectlProvisionerListOpts{
 		Provider: provider,
 		Region:   region,
-	}, minectlLog)
+	}, minectlUI)
 	if err != nil {
 		return err
 	}
@@ -51,19 +50,18 @@ func runList(cmd *cobra.Command, _ []string) error {
 		return err
 	}
 
+	if len(servers) == 0 {
+		minectlUI.Info("No servers found")
+		return nil
+	}
+
 	if !headless {
-		if len(servers) > 0 {
-			fmt.Println("")
-			table := tablewriter.NewTable(os.Stdout,
-				tablewriter.WithHeader([]string{"ID", "NAME", "REGION", "TAGS", "IP"}),
-			)
-			for _, server := range servers {
-				table.Append([]string{server.ID, server.Name, server.Region, server.Tags, server.PublicIP})
-			}
-			table.Render()
-		} else {
-			return errors.New("🤷 No server found")
+		fmt.Println("")
+		table := ui.NewTable(minectlUI, "ID", "NAME", "REGION", "TAGS", "IP")
+		for _, server := range servers {
+			table.Append([]string{server.ID, server.Name, server.Region, server.Tags, server.PublicIP})
 		}
+		table.Render()
 	}
 	return nil
 }