[terminal-stylist] Terminal Stylist: Console Output Analysis for pkg/

[github-actions[bot]]

Repository: github/gh-aw | Scope: pkg/ Go source files (non-test) | Files scanned: 986

---

Summary

The codebase demonstrates excellent console output discipline. A dedicated pkg/console package abstracts all user-facing output behind well-named formatter functions, pkg/styles centralises the entire Lipgloss style and adaptive-color system, and every Huh form is consistently themed. The issues found are minor and localised.

---

Key Metrics

Metric	Count
Non-test .go files in pkg/	986
Files using console.* output functions	170+
console.FormatInfoMessage call-sites	674
console.FormatWarningMessage call-sites	491
console.FormatSuccessMessage call-sites	216
Lipgloss style call-sites (pkg-wide)	104
Huh form instantiations	18
Forms with WithTheme + WithAccessible	18 ✅
Raw fmt.Print* for user-facing output (outside pkg/console)	3
Hardcoded ANSI escape sequences outside terminal helpers	0 ✅

---

Lipgloss Usage Analysis

Adaptive Color System (pkg/styles/theme.go)

The package implements a custom adaptive color type selecting light vs. dark hex values at runtime based on lipgloss.HasDarkBackground. 14 semantic color variables (Error, Warning, Success, Info, Purple, Yellow, Comment, Foreground, Background, Border, TableAltRow, ...) each have explicit light (darker/saturated) and dark (Dracula-inspired) variants. Background probing is skipped on Windows to avoid ConPTY crashes.

All pre-configured lipgloss.Style vars use these adaptive colors.

TTY-Aware Rendering (pkg/console/console.go)

All styling entry-points use applyStyleWithTTY which gates lipgloss.Style.Render behind tty.IsStdoutTerminal() / tty.IsStderrTerminal(). RenderTable, RenderTitleBox, RenderErrorBox, RenderInfoSection, and RenderComposedSections all follow this contract — plain text is emitted for pipes/redirects.

Direct Lipgloss in CLI (one file only)

pkg/cli/compile_schedule_calendar.go is the only pkg/cli/ file calling Lipgloss directly. It receives an isTerminal bool parameter and guards every render call. It uses named styles.ScheduleCalendar* style vars — no ad-hoc style construction.

Opportunity: Moving the heatmap renderer to a console.RenderCalendarGrid helper would remove the direct Lipgloss dependency from pkg/cli/. Low priority — correctness is not affected.

Border & Layout Patterns

Three named border vars (RoundedBorder, NormalBorder, ThickBorder) are centralised in pkg/styles/theme.go. RenderTitleBox uses an inline lipgloss.DoubleBorder() — the only deviation. Adding styles.DoubleBorder would complete centralisation.

Table rendering via lipgloss/table is encapsulated in console.RenderTable with a TableConfig struct. Zebra-striping, total rows, and per-column styling are handled inside buildTableStyleFunc.

---

Huh Form Analysis

Form Consistency & Theming

All 18 huh.NewForm(...) instantiations across 9 files call both .WithTheme(styles.HuhTheme).WithAccessible(console.IsAccessibleMode()). The custom styles.HuhTheme maps the Dracula-inspired palette to every Huh style token (titles, selectors, buttons, text-input cursor, etc.).

Field types in use: huh.NewConfirm, huh.NewSelect[string], huh.NewInput (with EchoModePassword). Appropriate for the current UX.

Non-TTY Fallbacks & Accessibility

pkg/console/confirm.go and pkg/console/list.go both check tty.IsStderrTerminal() before calling huh.NewForm, falling back to numbered text menus. console.IsAccessibleMode() reads the ACCESSIBLE env var and is passed to every form and spinner — animations are disabled in accessibility mode.

---

Findings & Recommendations

✅ Strengths

1. Full console abstraction — 20+ named formatter functions used consistently across 170+ files.
2. Adaptive colors — 14 semantic colors with explicit light/dark variants; Windows probe safety implemented.
3. Huh theming 100% consistent — every form uses styles.HuhTheme and IsAccessibleMode().
4. No raw ANSI escapes — the only \033[ sequences are the three constants in pkg/console/terminal.go (used by the Bubble Tea spinner).
5. formatCompilerMessage — uses console.FormatError(CompilerError{...}) for structured IDE-parseable output with Rust-like context rendering.

⚠️ Minor Issues

1. Unstyled sub-list items in pkg/cli/trial_confirmation.go

Lines 182–183 and 199–200 emit raw fmt.Fprintf(os.Stderr, " a. ...\n") without console styling, mixed with styled console.FormatInfoMessage(...) lines:

// styled parent:
fmt.Fprintf(os.Stderr, console.FormatInfoMessage("  %d. For each of %d executions:\n"), stepNum, count)
// unstyled children:
fmt.Fprintf(os.Stderr, "     a. Execute %s\n", workflowName)  // no styling
fmt.Fprintf(os.Stderr, "     b. Auto-merge any PRs created during execution\n")  // no styling

Consider a console.FormatSubItem(label string) helper for visual consistency.

2. Inline lipgloss.DoubleBorder() in pkg/console/console.go (RenderTitleBox)

Add styles.DoubleBorder = lipgloss.DoubleBorder() to pkg/styles/theme.go to complete border centralisation.

3. Direct styles.* usage in pkg/cli/compile_schedule_calendar.go

Extract the heatmap grid rendering into pkg/console to keep all Lipgloss dependencies out of pkg/cli/. Low priority.

---

References: §28508976423

Generated by 💄 Terminal Stylist · 79 AIC · ⌖ 11.5 AIC · ⊞ 5.7K · ◷

• expires on Jul 2, 2026, 2:02 AM UTC-08:00

[github-actions[bot]]

This discussion has been marked as outdated by Terminal Stylist.

A newer discussion is available at Discussion #42932.