Add color output and spinner

- tidwall/pretty: colorized JSON on TTY (blue keys, green strings,
  yellow numbers, magenta bools, red null). Respects NO_COLOR env var.
- briandowns/spinner: braille dots spinner on stderr during API calls,
  only when stderr is a TTY. Same library as gh CLI.
- Fix CI: go test ./pkg/... in unit job (avoids e2e go build)

Author: ilyazub

.github/workflows/ci.yml

@@ -14,7 +14,7 @@ jobs:
       - uses: actions/setup-go@v5
         with:
           go-version-file: go.mod
-      - run: go test ./...
+      - run: go test ./pkg/...
 
   e2e:
     runs-on: ubuntu-latest

go.mod

@@ -10,8 +10,13 @@ require (
 )
 
 require (
+	github.com/briandowns/spinner v1.23.2 // indirect
+	github.com/fatih/color v1.7.0 // indirect
 	github.com/inconshreveable/mousetrap v1.1.0 // indirect
 	github.com/itchyny/timefmt-go v0.1.7 // indirect
+	github.com/mattn/go-colorable v0.1.2 // indirect
+	github.com/mattn/go-isatty v0.0.20 // indirect
 	github.com/spf13/pflag v1.0.9 // indirect
+	github.com/tidwall/pretty v1.2.1 // indirect
 	golang.org/x/sys v0.42.0 // indirect
 )

go.sum

@@ -1,18 +1,31 @@
 github.com/BurntSushi/toml v1.6.0 h1:dRaEfpa2VI55EwlIW72hMRHdWouJeRF7TPYhI+AUQjk=
 github.com/BurntSushi/toml v1.6.0/go.mod h1:ukJfTF/6rtPPRCnwkur4qwRxa8vTRFBF0uk2lLoLwho=
+github.com/briandowns/spinner v1.23.2 h1:Zc6ecUnI+YzLmJniCfDNaMbW0Wid1d5+qcTq4L2FW8w=
+github.com/briandowns/spinner v1.23.2/go.mod h1:LaZeM4wm2Ywy6vO571mvhQNRcWfRUnXOs0RcKV0wYKM=
 github.com/cpuguy83/go-md2man/v2 v2.0.6/go.mod h1:oOW0eioCTA6cOiMLiUPZOpcVxMig6NIQQ7OS05n1F4g=
+github.com/fatih/color v1.7.0 h1:DkWD4oS2D8LGGgTQ6IvwJJXSL5Vp2ffcQg58nFV38Ys=
+github.com/fatih/color v1.7.0/go.mod h1:Zm6kSWBoL9eyXnKyktHP6abPY2pDugNf5KwzbycvMj4=
 github.com/inconshreveable/mousetrap v1.1.0 h1:wN+x4NVGpMsO7ErUn/mUI3vEoE6Jt13X2s0bqwp9tc8=
 github.com/inconshreveable/mousetrap v1.1.0/go.mod h1:vpF70FUmC8bwa3OWnCshd2FqLfsEA9PFc4w1p2J65bw=
 github.com/itchyny/gojq v0.12.18 h1:gFGHyt/MLbG9n6dqnvlliiya2TaMMh6FFaR2b1H6Drc=
 github.com/itchyny/gojq v0.12.18/go.mod h1:4hPoZ/3lN9fDL1D+aK7DY1f39XZpY9+1Xpjz8atrEkg=
 github.com/itchyny/timefmt-go v0.1.7 h1:xyftit9Tbw+Dc/huSSPJaEmX1TVL8lw5vxjJLK4GMMA=
 github.com/itchyny/timefmt-go v0.1.7/go.mod h1:5E46Q+zj7vbTgWY8o5YkMeYb4I6GeWLFnetPy5oBrAI=
+github.com/mattn/go-colorable v0.1.2 h1:/bC9yWikZXAL9uJdulbSfyVNIR3n3trXl+v8+1sx8mU=
+github.com/mattn/go-colorable v0.1.2/go.mod h1:U0ppj6V5qS13XJ6of8GYAs25YV2eR4EVcfRqFIhoBtE=
+github.com/mattn/go-isatty v0.0.8/go.mod h1:Iq45c/XA43vh69/j3iqttzPXn0bhXyGjM0Hdxcsrc5s=
+github.com/mattn/go-isatty v0.0.20 h1:xfD0iDuEKnDkl03q4limB+vH+GxLEtL/jb4xVJSWWEY=
+github.com/mattn/go-isatty v0.0.20/go.mod h1:W+V8PltTTMOvKvAeJH7IuucS94S2C6jfK/D7dTCTo3Y=
 github.com/russross/blackfriday/v2 v2.1.0/go.mod h1:+Rmxgy9KzJVeS9/2gXHxylqXiyQDYRxCVz55jmeOWTM=
 github.com/spf13/cobra v1.10.2 h1:DMTTonx5m65Ic0GOoRY2c16WCbHxOOw6xxezuLaBpcU=
 github.com/spf13/cobra v1.10.2/go.mod h1:7C1pvHqHw5A4vrJfjNwvOdzYu0Gml16OCs2GRiTUUS4=
 github.com/spf13/pflag v1.0.9 h1:9exaQaMOCwffKiiiYk6/BndUBv+iRViNW+4lEMi0PvY=
 github.com/spf13/pflag v1.0.9/go.mod h1:McXfInJRrz4CZXVZOBLb0bTZqETkiAhM9Iw0y3An2Bg=
+github.com/tidwall/pretty v1.2.1 h1:qjsOFOWWQl+N3RsoF5/ssm1pHmJJwhjlSbZ51I6wMl4=
+github.com/tidwall/pretty v1.2.1/go.mod h1:ITEVvHYasfjBbM0u2Pg8T2nJnzm8xPwvNhhsoaGGjNU=
 go.yaml.in/yaml/v3 v3.0.4/go.mod h1:DhzuOOF2ATzADvBadXxruRBLzYTpT36CKvDb3+aBEFg=
+golang.org/x/sys v0.0.0-20190222072716-a9d3bda3a223/go.mod h1:STP8DvDyc/dI5b8T5hshtkjS+E42TnysNCUPdjciGhY=
+golang.org/x/sys v0.6.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
 golang.org/x/sys v0.42.0 h1:omrd2nAlyT5ESRdCLYdm3+fMfNFE/+Rf4bDIQImRJeo=
 golang.org/x/sys v0.42.0/go.mod h1:4GL1E5IUh+htKOUEOaiffhrAeqysfVGipDYzABqnCmw=
 golang.org/x/term v0.41.0 h1:QCgPso/Q3RTJx2Th4bDLqML4W6iJiaXFq2/ftQF13YU=

pkg/cmd/account.go

@@ -23,8 +23,11 @@ func runAccount(cmd *cobra.Command, args []string) error {
 		return err
 	}
 
+	sp := newSpinner("Fetching account...")
+	sp.Start()
 	client := api.New(apiKey)
 	result, err := client.Account()
+	sp.Stop()
 	if err != nil {
 		return err
 	}

pkg/cmd/archive.go

@@ -23,8 +23,11 @@ func runArchive(cmd *cobra.Command, args []string) error {
 		return err
 	}
 
+	sp := newSpinner("Fetching archive...")
+	sp.Start()
 	client := api.New(apiKey)
 	result, err := client.Archive(args[0])
+	sp.Stop()
 	if err != nil {
 		return err
 	}

pkg/cmd/locations.go

@@ -29,8 +29,11 @@ func runLocations(cmd *cobra.Command, args []string) error {
 	}
 
 	paramsMap := params.ParamsToMap(parsed)
+	sp := newSpinner("Fetching locations...")
+	sp.Start()
 	client := api.New("")
 	result, err := client.Locations(paramsMap)
+	sp.Stop()
 	if err != nil {
 		return err
 	}

pkg/cmd/root.go

@@ -10,6 +10,7 @@ import (
 	clierrors "github.com/serpapi/serpapi-cli/pkg/errors"
 	"github.com/serpapi/serpapi-cli/pkg/jq"
 	"github.com/serpapi/serpapi-cli/pkg/output"
+	"github.com/serpapi/serpapi-cli/pkg/spinner"
 	"github.com/serpapi/serpapi-cli/pkg/version"
 )
 
@@ -89,3 +90,8 @@ func handleOutput(result any) error {
 	}
 	return nil
 }
+
+// newSpinner creates a spinner with the given label.
+func newSpinner(label string) *spinner.Spinner {
+	return spinner.New(label)
+}

pkg/cmd/search.go

@@ -1,6 +1,7 @@
 package cmd
 
 import (
+	"fmt"
 	"net/url"
 	"sort"
 
@@ -46,8 +47,11 @@ func runSearch(cmd *cobra.Command, args []string) error {
 	hasMaxPages := cmd.Flags().Changed("max-pages")
 
 	if !allPagesFlag && !hasMaxPages {
+		sp := newSpinner("Searching...")
+		sp.Start()
 		client := api.New(apiKey)
 		result, err := client.Search(paramsMap)
+		sp.Stop()
 		if err != nil {
 			return err
 		}
@@ -67,7 +71,10 @@ func runSearch(cmd *cobra.Command, args []string) error {
 	seen[canonicalParamsKey(currentParams)] = true
 
 	for {
+		sp := newSpinner(fmt.Sprintf("Fetching page %d...", pagesFetched+1))
+		sp.Start()
 		result, err := client.Search(currentParams)
+		sp.Stop()
 		if err != nil {
 			return err
 		}

pkg/output/output.go

@@ -5,14 +5,40 @@ import (
 	"fmt"
 	"io"
 	"os"
+
+	"github.com/tidwall/pretty"
+	"golang.org/x/term"
 )
 
+// darkStyle matches stripe-cli's darkTerminalStyle exactly.
+var darkStyle = &pretty.Style{
+	Key:    [2]string{"\x1B[34m", "\x1B[0m"},   // blue
+	String: [2]string{"\x1B[32m", "\x1B[0m"},   // green
+	Number: [2]string{"\x1B[33m", "\x1B[0m"},   // yellow
+	True:   [2]string{"\x1B[35m", "\x1B[0m"},   // magenta
+	False:  [2]string{"\x1B[35m", "\x1B[0m"},   // magenta
+	Null:   [2]string{"\x1B[31m", "\x1B[0m"},   // red
+}
+
+// shouldColor returns true when stdout is a TTY and NO_COLOR is not set.
+func shouldColor() bool {
+	if os.Getenv("NO_COLOR") != "" {
+		return false
+	}
+	return term.IsTerminal(int(os.Stdout.Fd()))
+}
+
 // PrintJSON prints a value to stdout as pretty-printed JSON.
+// Colorized when stdout is a TTY and NO_COLOR is not set.
 func PrintJSON(v any) error {
 	data, err := json.MarshalIndent(v, "", "  ")
 	if err != nil {
 		return err
 	}
+	if shouldColor() {
+		_, err = fmt.Fprint(os.Stdout, string(pretty.Color(data, darkStyle)))
+		return err
+	}
 	_, err = fmt.Fprintln(os.Stdout, string(data))
 	return err
 }
@@ -56,3 +82,4 @@ func formatNumber(f float64) string {
 	}
 	return fmt.Sprintf("%g", f)
 }
+

pkg/spinner/spinner.go

@@ -0,0 +1,43 @@
+// Package spinner provides a TTY-aware activity spinner for long-running operations.
+package spinner
+
+import (
+	"os"
+	"time"
+
+	"github.com/briandowns/spinner"
+	"golang.org/x/term"
+)
+
+// Spinner wraps briandowns/spinner with TTY awareness.
+type Spinner struct {
+	s *spinner.Spinner
+}
+
+// New creates a spinner that writes to stderr.
+// Returns a no-op spinner when stderr is not a TTY (pipes, CI, redirects).
+func New(label string) *Spinner {
+	if !term.IsTerminal(int(os.Stderr.Fd())) {
+		return &Spinner{}
+	}
+	s := spinner.New(spinner.CharSets[14], 80*time.Millisecond,
+		spinner.WithWriter(os.Stderr),
+		spinner.WithColor("cyan"),
+	)
+	s.Suffix = " " + label
+	return &Spinner{s: s}
+}
+
+// Start begins the spinner animation.
+func (sp *Spinner) Start() {
+	if sp.s != nil {
+		sp.s.Start()
+	}
+}
+
+// Stop halts the spinner and clears it from the terminal.
+func (sp *Spinner) Stop() {
+	if sp.s != nil {
+		sp.s.Stop()
+	}
+}