ui: migrate charmbracelet stack to v2 (#1054)

Upgrades to charmbracelet's v2 libraries
(lipgloss, bubbletea, and bubbles, https://charm.land/blog/v2/).

This required significantly reworking our rendering pipeline,
as well as some workarounds on the test/simulation side
to accommodate improvements made on the Bubble Tea side.

## Render-time styling

Upgrade the TUI stack to `bubbles`, `bubbletea`, and `lipgloss`
v2,
and reshape the rendering pipeline around the API changes those
releases require.

Lip Gloss v2 resolves adaptive colors against the active output stream
at render time,
so the UI now carries semantic `ui.Style` and `ui.Theme` values until
it writes output.
That keeps call sites expressive while letting the output layer decide
how much styling the destination can support.

## Bubble Tea v2 integration

Bubble Tea v2 also changes the terminal contract.
Forms now implement `View() tea.View`,
key handling moves to `tea.KeyPressMsg`,
and startup sizing is configured through program options.
Those updates are mostly mechanical,
but they require the interactive layer to pass theme and window-size
state through every field consistently.

## Test harness and terminal emulation

The migration also forced the scripted UI harnesses to behave more like
real terminals.
The emulator now preserves raw-terminal newline semantics,
treats tabs as cursor movement,
and waits for the screen to settle before assertions snapshot the
result.
That keeps Bubble Tea v2 redraw bursts from leaking intermediate frames
into fixtures.

Robot-driven prompt fixtures need the same output-capability handling.
`lipgloss.Style.Render` still returns ANSI sequences even when tests
are recording prompt text into plain files.
`uitest.RobotView` now captures prompt and log output through a
`colorprofile.Writer` forced to `NoTTY`,
so script fixtures observe the same unstyled text that a non-terminal
destination would receive.
A regression test locks that behavior down for future UI changes.

The PTY-based `with-term` harness also needs a deterministic theme.
Its scripted terminal does not answer background-color probes,
so detecting the active theme can block interactive startup before the
first prompt is drawn.
`ui.detectTheme` now honors a test-only `__GIT_SPICE_THEME` override,
and `with-term` sets it explicitly so PTY-driven script tests can boot
without stalling on terminal queries.

## Output capability detection

The migration also changes how non-interactive output is written.
`ui.NewFileView` wraps outputs with `colorprofile.Writer`
and records the detected theme alongside the writer.
That preserves colored branch-tree and log output when the destination
supports it,
while still stripping or down-sampling styles when it does not.

Author: abhinav

.changes/unreleased/Changed-20260310-142852.yaml

@@ -0,0 +1,6 @@
+kind: Changed
+body: >-
+  Upgrade TUI rendering to v2 of underlying libraries.
+  No user-facing changes are expected, but significant internal refactoring was required.
+  If you encounter unexpected side effects, please [report them here](https://github.com/abhinav/git-spice/issues).
+time: 2026-03-10T14:28:52.586798-07:00

.golangci.yml

@@ -36,6 +36,11 @@ linters:
         - fmt.Fprintf
         - fmt.Fprintln
 
+        # Usually used in the same capacity as fmt.Print*.
+        - charm.land/lipgloss/v2.Fprint
+        - charm.land/lipgloss/v2.Fprintf
+        - charm.land/lipgloss/v2.Fprintln
+
         # This is always a strings.Builder, and can't fail.
         - (go.abhg.dev/gs/internal/ui.Writer).WriteString
         - (go.abhg.dev/gs/internal/ui.Writer).Write

branch_split.go

@@ -149,8 +149,9 @@ func (cmd *branchSplitCmd) Run(
 			}
 
 			fields := make([]ui.Field, 0, len(selected)+1) // +1 for deferred HEAD field
+			theme := view.Theme()
 			for i, commit := range selected {
-				desc := cmd.commitDescription(commit, false /* head */)
+				desc := cmd.commitDescription(theme, commit, false /* head */)
 				input := branchNameWidget(desc, &branchNames[i])
 				fields = append(fields, input)
 			}
@@ -168,7 +169,7 @@ func (cmd *branchSplitCmd) Run(
 					return nil
 				}
 
-				desc := cmd.commitDescription(headCommit, true /* head */) +
+				desc := cmd.commitDescription(theme, headCommit, true /* head */) +
 					" [" + cmd.Branch + "]"
 				return branchNameWidget(desc, &headBranchName)
 			}))
@@ -213,7 +214,11 @@ func (cmd *branchSplitCmd) Run(
 	return nil
 }
 
-func (cmd *branchSplitCmd) commitDescription(c git.CommitDetail, head bool) string {
+func (cmd *branchSplitCmd) commitDescription(
+	theme ui.Theme,
+	c git.CommitDetail,
+	head bool,
+) string {
 	var desc strings.Builder
 	if head {
 		desc.WriteString("  ■ ")
@@ -224,7 +229,7 @@ func (cmd *branchSplitCmd) commitDescription(c git.CommitDetail, head bool) stri
 		ShortHash:  c.ShortHash,
 		Subject:    c.Subject,
 		AuthorDate: c.AuthorDate,
-	}).Render(&desc, commit.DefaultSummaryStyle, nil)
+	}).Render(&desc, theme, commit.DefaultSummaryStyle, nil)
 
 	return desc.String()
 }

commit_amend.go

@@ -88,14 +88,14 @@ func (cmd *commitAmendCmd) Run(
 					WithItems(
 						ui.ListItem[bool]{
 							Title: "Yes",
-							Description: func(bool) string {
+							Description: func(ui.Theme, bool) string {
 								return "Amend the commit on trunk"
 							},
 							Value: true,
 						},
 						ui.ListItem[bool]{
 							Title: "No",
-							Description: func(bool) string {
+							Description: func(ui.Theme, bool) string {
 								return "Create a branch and commit there instead"
 							},
 							Value: false,
@@ -162,12 +162,12 @@ func (cmd *commitAmendCmd) Run(
 						WithItems(
 							ui.ListItem[bool]{
 								Title:       "Yes",
-								Description: func(bool) string { return "Continue with commit amend" },
+								Description: func(ui.Theme, bool) string { return "Continue with commit amend" },
 								Value:       true,
 							},
 							ui.ListItem[bool]{
 								Title:       "No",
-								Description: func(bool) string { return "Abort the operation" },
+								Description: func(ui.Theme, bool) string { return "Abort the operation" },
 								Value:       false,
 							},
 						).

commit_pick.go

@@ -94,14 +94,14 @@ func (cmd *commitPickCmd) Run(
 				WithItems(
 					ui.ListItem[bool]{
 						Title: "Yes",
-						Description: func(bool) string {
+						Description: func(ui.Theme, bool) string {
 							return fmt.Sprintf("Cherry-pick commit %v on trunk", commit.Short())
 						},
 						Value: true,
 					},
 					ui.ListItem[bool]{
 						Title: "No",
-						Description: func(bool) string {
+						Description: func(ui.Theme, bool) string {
 							return "Abort the operation"
 						},
 						Value: false,

doc/mise.lock

@@ -3,11 +3,14 @@ module go.abhg.dev/gs
 go 1.26.0
 
 require (
+	charm.land/bubbles/v2 v2.0.0
+	charm.land/bubbletea/v2 v2.0.1
+	charm.land/lipgloss/v2 v2.0.0
 	github.com/alecthomas/kong v1.14.0
 	github.com/buildkite/shellwords v1.0.1
-	github.com/charmbracelet/bubbles v1.0.0
-	github.com/charmbracelet/bubbletea v1.3.10
-	github.com/charmbracelet/lipgloss v1.1.0
+	github.com/charmbracelet/colorprofile v0.4.3
+	github.com/charmbracelet/x/ansi v0.11.6
+	github.com/charmbracelet/x/term v0.2.2
 	github.com/cli/browser v1.3.0
 	github.com/creack/pty v1.1.24
 	github.com/dustin/go-humanize v1.0.1
@@ -18,13 +21,13 @@ require (
 	github.com/shurcooL/githubv4 v0.0.0-20260209031235-2402fdf4a9ed
 	github.com/stretchr/testify v1.11.1
 	github.com/tidwall/gjson v1.18.0
-	github.com/vito/midterm v0.2.3
+	github.com/vito/midterm v0.2.4
 	github.com/zalando/go-keyring v0.2.6
 	gitlab.com/gitlab-org/api/client-go v1.46.0
 	go.abhg.dev/container/ring v0.3.0
 	go.abhg.dev/io/ioutil v0.1.0
 	go.abhg.dev/komplete v0.1.0
-	go.abhg.dev/log/silog v0.2.0
+	go.abhg.dev/log/silog v0.3.0
 	go.abhg.dev/testing/stub v0.2.0
 	go.uber.org/mock v0.6.0
 	golang.org/x/oauth2 v0.35.0
@@ -37,20 +40,17 @@ require (
 	al.essio.dev/pkg/shellescape v1.6.0 // indirect
 	github.com/atotto/clipboard v0.1.4 // indirect
 	github.com/aymanbagabas/go-osc52/v2 v2.0.1 // indirect
-	github.com/charmbracelet/colorprofile v0.4.1 // indirect
-	github.com/charmbracelet/x/ansi v0.11.6 // indirect
-	github.com/charmbracelet/x/cellbuf v0.0.15 // indirect
-	github.com/charmbracelet/x/term v0.2.2 // indirect
-	github.com/clipperhouse/displaywidth v0.9.0 // indirect
-	github.com/clipperhouse/stringish v0.1.1 // indirect
-	github.com/clipperhouse/uax29/v2 v2.5.0 // indirect
+	github.com/charmbracelet/ultraviolet v0.0.0-20260205113103-524a6607adb8 // indirect
+	github.com/charmbracelet/x/termios v0.1.1 // indirect
+	github.com/charmbracelet/x/windows v0.2.2 // indirect
+	github.com/clipperhouse/displaywidth v0.11.0 // indirect
+	github.com/clipperhouse/uax29/v2 v2.7.0 // indirect
 	github.com/danielgatis/go-ansicode v1.0.7 // indirect
 	github.com/danielgatis/go-iterator v0.0.1 // indirect
 	github.com/danielgatis/go-utf8 v1.0.0 // indirect
 	github.com/danielgatis/go-vte v1.0.8 // indirect
 	github.com/danieljoos/wincred v1.2.3 // indirect
 	github.com/davecgh/go-spew v1.1.1 // indirect
-	github.com/erikgeiser/coninput v0.0.0-20211004153227-1c3628e74d0f // indirect
 	github.com/fatih/color v1.18.0 // indirect
 	github.com/godbus/dbus/v5 v5.2.2 // indirect
 	github.com/google/go-cmp v0.7.0 // indirect
@@ -61,9 +61,7 @@ require (
 	github.com/hexops/valast v1.5.0 // indirect
 	github.com/lucasb-eyer/go-colorful v1.3.0 // indirect
 	github.com/mattn/go-colorable v0.1.14 // indirect
-	github.com/mattn/go-localereader v0.0.1 // indirect
-	github.com/mattn/go-runewidth v0.0.19 // indirect
-	github.com/muesli/ansi v0.0.0-20230316100256-276c6243b2f6 // indirect
+	github.com/mattn/go-runewidth v0.0.20 // indirect
 	github.com/muesli/cancelreader v0.2.2 // indirect
 	github.com/muesli/termenv v0.16.0 // indirect
 	github.com/nightlyone/lockfile v1.0.0 // indirect
@@ -76,8 +74,7 @@ require (
 	go.yaml.in/yaml/v4 v4.0.0-rc.3 // indirect
 	golang.org/x/mod v0.33.0 // indirect
 	golang.org/x/sync v0.19.0 // indirect
-	golang.org/x/sys v0.41.0 // indirect
-	golang.org/x/text v0.32.0 // indirect
+	golang.org/x/sys v0.42.0 // indirect
 	golang.org/x/time v0.14.0 // indirect
 	golang.org/x/tools v0.41.0 // indirect
 	mvdan.cc/gofumpt v0.9.2 // indirect