mirror of
https://github.com/Twigpine/zero.git
synced 2026-11-11 23:40:28 -10:00
* Migrate TUI to Bubble Tea v2 Move the TUI stack to charm.land Bubble Tea, Bubbles, and Lip Gloss v2, including key, paste, mouse, and view handling changes for the v2 APIs. Add compatibility helpers for the new input event shapes and update TUI tests accordingly. Refresh related Go dependencies and make the PDF truncation fixture deterministic across poppler and pure-Go extraction. Tested: GOCACHE=/tmp/zero-go-cache GOMODCACHE=/tmp/zero-go-mod-cache go test ./internal/tui Tested: GOCACHE=/tmp/zero-go-cache GOMODCACHE=/tmp/zero-go-mod-cache go test ./internal/imageinput -count=1 -v Tested: GOCACHE=/tmp/zero-go-cache GOMODCACHE=/tmp/zero-go-mod-cache go test ./... * Address provider wizard OAuth review feedback Remove the unused mouseButton compatibility helper flagged by lint. Correlate provider wizard OAuth and device-code async results with the active provider and attempt id so stale results from abandoned attempts are ignored. Add regression coverage for stale browser OAuth and stale device-code results. Tested: GOCACHE=/tmp/zero-go-cache GOMODCACHE=/tmp/zero-go-mod-cache go test ./internal/tui -run 'ProviderWizard.*OAuth|ProviderWizard.*Device|ApplyProviderWizardOAuth|RenderCredentialStepShowsOAuth' -count=1 -v Tested: GOCACHE=/tmp/zero-go-cache GOMODCACHE=/tmp/zero-go-mod-cache go test ./internal/tui Tested: GOCACHE=/tmp/zero-go-cache GOMODCACHE=/tmp/zero-go-mod-cache go test ./... Tested: git diff --check * Smooth TUI mouse scrolling
134 lines
4.3 KiB
Go
134 lines
4.3 KiB
Go
package tui
|
|
|
|
import (
|
|
"context"
|
|
"fmt"
|
|
"strings"
|
|
"testing"
|
|
|
|
tea "charm.land/bubbletea/v2"
|
|
"charm.land/lipgloss/v2"
|
|
|
|
"github.com/Gitlawb/zero/internal/tools"
|
|
)
|
|
|
|
func TestTranscriptCommandTogglesDetailedView(t *testing.T) {
|
|
m := transcriptViewTestModel()
|
|
m.input.SetValue("/transcript")
|
|
|
|
updated, _ := m.Update(testKey(tea.KeyEnter))
|
|
next := updated.(model)
|
|
assertContains(t, plainRender(t, next.View()), "Transcript")
|
|
|
|
next.input.SetValue("/transcript")
|
|
updated, _ = next.Update(testKey(tea.KeyEnter))
|
|
next = updated.(model)
|
|
assertNotContains(t, plainRender(t, next.View()), "Transcript")
|
|
}
|
|
|
|
func TestCtrlOTogglesDetailedTranscriptView(t *testing.T) {
|
|
m := transcriptViewTestModel()
|
|
|
|
updated, _ := m.Update(testKeyCtrl('o'))
|
|
next := updated.(model)
|
|
assertContains(t, plainRender(t, next.View()), "Transcript")
|
|
|
|
updated, _ = next.Update(testKeyCtrl('o'))
|
|
next = updated.(model)
|
|
assertNotContains(t, plainRender(t, next.View()), "Transcript")
|
|
}
|
|
|
|
func TestEscExitsDetailedTranscriptView(t *testing.T) {
|
|
m := transcriptViewTestModel()
|
|
updated, _ := m.Update(testKeyCtrl('o'))
|
|
next := updated.(model)
|
|
assertContains(t, plainRender(t, next.View()), "Transcript")
|
|
|
|
updated, _ = next.Update(testKey(tea.KeyEsc))
|
|
next = updated.(model)
|
|
assertNotContains(t, plainRender(t, next.View()), "Transcript")
|
|
}
|
|
|
|
func TestDetailedTranscriptIncludesToolOutputBeyondLiveCap(t *testing.T) {
|
|
m := transcriptViewTestModel()
|
|
output := numberedLines(flushCardBodyMaxLines + 4)
|
|
row := transcriptRow{kind: rowToolResult, id: "tool-1", tool: "custom_tool", status: tools.StatusOK, detail: output}
|
|
m.transcript = append(m.transcript, row)
|
|
m.flushed = len(m.transcript)
|
|
|
|
// Long generic tool output collapses in the live view (click to expand);
|
|
// the body is hidden behind a one-line summary.
|
|
compact := plainRender(t, m.renderRow(row, m.width, buildRowContext(m.transcript)))
|
|
assertNotContains(t, compact, "line-020")
|
|
assertContains(t, compact, "click to expand")
|
|
|
|
// The detailed transcript view (Ctrl+O) still shows the full, uncapped output.
|
|
updated, _ := m.Update(testKeyCtrl('o'))
|
|
next := updated.(model)
|
|
view := plainRender(t, next.View())
|
|
assertContains(t, view, "line-404")
|
|
assertNotContains(t, view, "more lines")
|
|
assertNotContains(t, view, "click to expand")
|
|
}
|
|
|
|
func TestDetailedTranscriptViewNeverExceedsTerminalWidth(t *testing.T) {
|
|
for _, width := range []int{24, 40, 58, 80, 120} {
|
|
m := transcriptViewTestModel()
|
|
m.width = width
|
|
m.transcript = append(m.transcript,
|
|
transcriptRow{kind: rowUser, text: "please inspect this very long request and show the transcript without overflowing"},
|
|
transcriptRow{kind: rowToolResult, id: "wide", tool: "custom_tool", status: tools.StatusOK, detail: strings.Repeat("wide-output-", 30)},
|
|
transcriptRow{kind: rowAssistant, text: "done with a final answer that also needs wrapping", final: true, turnTools: 1},
|
|
)
|
|
m.flushed = len(m.transcript)
|
|
|
|
updated, _ := m.Update(testKeyCtrl('o'))
|
|
next := updated.(model)
|
|
view := viewString(next.View())
|
|
assertContains(t, plainRender(t, view), "Transcript")
|
|
for index, line := range strings.Split(view, "\n") {
|
|
if got := lipgloss.Width(line); got > chatWidth(width) {
|
|
t.Fatalf("width %d: detailed transcript line %d is %d cells wide: %q", width, index, got, line)
|
|
}
|
|
}
|
|
}
|
|
}
|
|
|
|
func TestDetailedTranscriptSwallowsNormalChatSubmit(t *testing.T) {
|
|
m := transcriptViewTestModel()
|
|
updated, _ := m.Update(testKeyCtrl('o'))
|
|
next := updated.(model)
|
|
next.input.SetValue("this should not launch")
|
|
|
|
updated, cmd := next.Update(testKey(tea.KeyEnter))
|
|
next = updated.(model)
|
|
|
|
if cmd != nil {
|
|
t.Fatal("detailed transcript should not return a run command for Enter")
|
|
}
|
|
if transcriptContains(next.transcript, "this should not launch") {
|
|
t.Fatalf("detailed transcript should not submit composer text, got %#v", next.transcript)
|
|
}
|
|
assertContains(t, plainRender(t, next.View()), "Transcript")
|
|
}
|
|
|
|
func transcriptViewTestModel() model {
|
|
m := newModel(context.Background(), Options{
|
|
Cwd: "/work/zero",
|
|
ProviderName: "openai",
|
|
ModelName: "gpt-test",
|
|
})
|
|
m.width = 96
|
|
m.height = 30
|
|
m.headerPrinted = true
|
|
return m
|
|
}
|
|
|
|
func numberedLines(count int) string {
|
|
lines := make([]string, 0, count)
|
|
for i := 1; i <= count; i++ {
|
|
lines = append(lines, fmt.Sprintf("line-%03d", i))
|
|
}
|
|
return strings.Join(lines, "\n")
|
|
}
|