Route new users from setup to signup wizard

internal/commands/setup.go

@@ -28,7 +28,7 @@
 var setupCmd = &cobra.Command{
 	Use:   "setup",
 	Short: "Interactive setup wizard",
-	Long:  "Configure Fizzy CLI with your API token, account, and default board.",
+	Long:  "Configure Fizzy CLI with your API token, account, and default board.\nNew users without an account will be guided through signup.",
 	RunE:  runSetup,
 }
 
@@ -45,6 +45,26 @@
 	fmt.Println("Welcome to Fizzy CLI setup!")
 	fmt.Println()
 
+	// Ask if user has an account before checking existing config
+	var hasAccount string
+	err := huh.NewSelect[string]().
+		Title("Do you have a Fizzy account?").
+		Options(
+			huh.NewOption("Yes, I have an account", "yes"),
+			huh.NewOption("No, I'd like to sign up", "no"),
+		).
+		Value(&hasAccount).
+		Run()
+
+	if err != nil {
+		fmt.Println("Setup cancelled.")
+		return nil //nolint:nilerr // user cancelled prompt
+	}
+
+	if hasAccount == "no" {
+		return runSignup(cmd, args)
+	}
+
 	// Check for existing config
 	globalExists := config.Exists()
 	localPath := config.LocalConfigPath()
@@ -56,7 +76,7 @@
 			configLocation = "local config (" + localPath + ")"
 		}
 
 		err := huh.NewConfirm().
 			Title(fmt.Sprintf("Existing %s found. Reconfigure?", configLocation)).
 			Value(&reconfigure).
 			Run()
@@ -74,7 +94,7 @@
 
 	// Ask hosted vs self-hosted
 	var hostingType string
-	err := huh.NewSelect[string]().
+	err = huh.NewSelect[string]().
 		Title("Are you using the hosted or self-hosted version?").
 		Options(
 			huh.NewOption("Hosted (app.fizzy.do)", "hosted"),

internal/commands/setup_test.go

@@ -3,12 +3,62 @@ package commands
 import (
 	"os"
 	"path/filepath"
+	"strings"
 	"testing"
 
 	"github.com/basecamp/fizzy-cli/internal/config"
 	"gopkg.in/yaml.v3"
 )
 
+func TestSetupCommandDescription(t *testing.T) {
+	t.Run("long description mentions signup for new users", func(t *testing.T) {
+		if !strings.Contains(setupCmd.Long, "signup") {
+			t.Error("expected setup command long description to mention signup")
+		}
+	})
+}
+
+func TestSetupRejectsMachineOutput(t *testing.T) {
+	t.Run("rejects agent mode", func(t *testing.T) {
+		defer ResetTestMode()
+		ResetTestMode()
+		cfgAgent = true
+		err := runSetup(setupCmd, nil)
+		if err == nil {
+			t.Fatal("expected error when running setup with --agent")
+		}
+		if !strings.Contains(err.Error(), "interactive terminal") {
+			t.Errorf("unexpected error: %v", err)
+		}
+	})
+
+	t.Run("rejects JSON mode", func(t *testing.T) {
+		defer ResetTestMode()
+		ResetTestMode()
+		cfgJSON = true
+		err := runSetup(setupCmd, nil)
+		if err == nil {
+			t.Fatal("expected error when running setup with --json")
+		}
+		if !strings.Contains(err.Error(), "interactive terminal") {
+			t.Errorf("unexpected error: %v", err)
+		}
+	})
+
+	t.Run("rejects quiet mode", func(t *testing.T) {
+		defer ResetTestMode()
+		ResetTestMode()
+		cfgQuiet = true
+		err := runSetup(setupCmd, nil)
+		if err == nil {
+			t.Fatal("expected error when running setup with --quiet")
+		}
+		if !strings.Contains(err.Error(), "interactive terminal") {
+			t.Errorf("unexpected error: %v", err)
+		}
+	})
+}
+
 func TestParseAccounts(t *testing.T) {
 	t.Run("parses accounts from identity response", func(t *testing.T) {
 		data := map[string]any{