Release v2.5.1: Fix table truncation, spinner grace period

Table: revert from Bubbles table (designed for interactive TUI)
back to manual lipgloss-styled table that respects full content width.
Bubbles table was truncating emails and dates too aggressively.

Spinner: add 200ms grace period before showing. Fast commands
(<200ms) complete without any spinner flash. Slow commands get
the animated spinner after the grace period.

Author: mubbie

VERSION

@@ -1 +1 @@
-2.5.0
+2.5.1

internal/ui/spinner.go

@@ -6,19 +6,31 @@ import (
 	"time"
 )
 
-// Spinner frames from charmbracelet/bubbles dot spinner
 var spinnerFrames = []string{"⣾", "⣽", "⣻", "⢿", "⡿", "⣟", "⣯", "⣷"}
 
+// Spinner shows an animated indicator while work is in progress.
+// Has a 200ms grace period before showing to avoid flashing on fast operations.
 type Spinner struct {
-	stop chan struct{}
-	done sync.WaitGroup
+	stop    chan struct{}
+	done    sync.WaitGroup
+	showed  bool
 }
 
+// StartSpinner begins a spinner that appears after 200ms.
 func StartSpinner(msg string) *Spinner {
 	s := &Spinner{stop: make(chan struct{})}
 	s.done.Add(1)
 	go func() {
 		defer s.done.Done()
+
+		// Grace period: don't show spinner for fast operations
+		select {
+		case <-s.stop:
+			return
+		case <-time.After(200 * time.Millisecond):
+		}
+
+		s.showed = true
 		i := 0
 		for {
 			select {
@@ -27,8 +39,7 @@ func StartSpinner(msg string) *Spinner {
 				return
 			default:
 				frame := spinnerFrames[i%len(spinnerFrames)]
-				styled := BranchStyle.Render(frame)
-				fmt.Printf("\r%s %s", styled, DimStyle.Render(msg))
+				fmt.Printf("\r%s %s", BranchStyle.Render(frame), DimStyle.Render(msg))
 				i++
 				time.Sleep(80 * time.Millisecond)
 			}
@@ -37,6 +48,7 @@ func StartSpinner(msg string) *Spinner {
 	return s
 }
 
+// Stop halts the spinner and clears the line.
 func (s *Spinner) Stop() {
 	close(s.stop)
 	s.done.Wait()

internal/ui/table.go

@@ -2,80 +2,66 @@ package ui
 
 import (
 	"fmt"
+	"strings"
 
-	"github.com/charmbracelet/bubbles/table"
 	"github.com/charmbracelet/lipgloss"
 )
 
 var (
-	titleStyle = lipgloss.NewStyle().Bold(true).Foreground(lipgloss.Color("5")).MarginBottom(1)
+	headerStyle = lipgloss.NewStyle().Bold(true).Foreground(lipgloss.Color("4"))
+	titleStyle  = lipgloss.NewStyle().Bold(true).Foreground(lipgloss.Color("5")).MarginBottom(1)
+	sepStyle    = lipgloss.NewStyle().Foreground(lipgloss.Color("240"))
 )
 
-const maxColWidth = 50
-
 func PrintTable(headers []string, rows [][]string, title string) {
 	if title != "" {
 		fmt.Println(titleStyle.Render(title))
 	}
 
-	// Calculate column widths based on content
+	// Calculate column widths from VISIBLE content width (ANSI-aware)
 	widths := make([]int, len(headers))
 	for i, h := range headers {
-		widths[i] = lipgloss.Width(h) + 2
+		widths[i] = len(h)
 	}
 	for _, row := range rows {
 		for i, cell := range row {
-			w := lipgloss.Width(cell) + 2
+			w := lipgloss.Width(cell)
 			if i < len(widths) && w > widths[i] {
 				widths[i] = w
 			}
 		}
 	}
 
-	// Cap non-last columns
-	lastCol := len(headers) - 1
+	// Add padding
 	for i := range widths {
-		if i != lastCol && widths[i] > maxColWidth {
-			widths[i] = maxColWidth
-		}
+		widths[i] += 2
 	}
 
-	// Build columns
-	cols := make([]table.Column, len(headers))
+	// Header
+	hParts := make([]string, len(headers))
 	for i, h := range headers {
-		cols[i] = table.Column{Title: h, Width: widths[i]}
+		hParts[i] = headerStyle.Width(widths[i]).Render(h)
+	}
+	fmt.Println(strings.Join(hParts, ""))
+
+	// Separator
+	sParts := make([]string, len(headers))
+	for i := range headers {
+		sParts[i] = sepStyle.Render(strings.Repeat("─", widths[i]))
 	}
+	fmt.Println(strings.Join(sParts, ""))
 
-	// Build rows
-	tableRows := make([]table.Row, len(rows))
-	for i, row := range rows {
-		r := make(table.Row, len(headers))
-		for j := range headers {
-			if j < len(row) {
-				r[j] = row[j]
+	// Rows - use lipgloss Width for ANSI-aware padding, MaxWidth for truncation
+	for _, row := range rows {
+		parts := make([]string, len(headers))
+		for i := range headers {
+			cell := ""
+			if i < len(row) {
+				cell = row[i]
 			}
+			parts[i] = lipgloss.NewStyle().Width(widths[i]).MaxWidth(widths[i]).Render(cell)
 		}
-		tableRows[i] = r
+		fmt.Println(strings.Join(parts, ""))
 	}
-
-	// Style the table
-	s := table.DefaultStyles()
-	s.Header = s.Header.
-		Bold(true).
-		Foreground(lipgloss.Color("4")).
-		BorderStyle(lipgloss.NormalBorder()).
-		BorderBottom(true).
-		BorderForeground(lipgloss.Color("240"))
-	s.Selected = lipgloss.NewStyle() // Disable selection highlighting for static display
-	s.Cell = s.Cell.Padding(0, 1)
-
-	t := table.New(
-		table.WithColumns(cols),
-		table.WithRows(tableRows),
-		table.WithHeight(len(tableRows)+1),
-		table.WithStyles(s),
-	)
-
-	fmt.Println(t.View())
 	fmt.Println()
 }

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "gx-git"
-version = "2.5.0"
+version = "2.5.1"
 description = "Git Productivity Toolkit"
 readme = "README.md"
 license = {text = "MIT"}