[terminal-stylist] Terminal Stylist: Console Output & Lipgloss/Huh Consistency Report

[github-actions[bot]]

Terminal Stylist Report: Console Output Consistency

Scope: 24 .go files (excluding _test.go) scanned across pkg/, focused on fmt.Print*, console.*, lipgloss.*, and huh.* usage.

Key Metrics

Pattern	Count
Files using pkg/console formatters	242
Raw fmt.Print* in non-test, non-linter-testdata code	2 (both legitimate)
Files using Lipgloss directly	6
Files using Huh forms	15
Manual ANSI escape files	2 (both intentional, in stringutil and console/terminal.go)
Hardcoded (non-adaptive) colors outside pkg/styles/pkg/console	0

Overall Assessment: Healthy

The codebase shows strong, centralized console-styling discipline. Nearly all user-facing output routes through pkg/console (247 usage sites), which itself wraps Lipgloss with TTY-aware rendering (pkg/console/colorprofile_writer.go, pkg/console/terminal.go using tty.IsStderrTerminal()), and adaptive color definitions live solely in pkg/styles/theme.go — no stray hardcoded hex colors were found elsewhere.

Lipgloss usage details

• Adaptive colors: pkg/styles/theme.go defines all colors in light/dark pairs (colorErrorLight/Dark, etc.) and composes them via lipgloss.NewStyle() into semantic styles (Error, Warning, Success, Info, FilePath, Prompt, ...). No component reaches for raw ANSI codes or bypasses this palette.
• Dark-background detection: theme.go wires lipgloss.HasDarkBackground through a testable backgroundDetector seam (configureHasDarkBackground), a good pattern for terminal-aware rendering without hardcoding assumptions.
• Table rendering: pkg/console/console.go (RenderTable, ~line 244) correctly uses lipgloss/table rather than manual column alignment, including custom header-row styling.
• TTY-gated escapes: pkg/console/terminal.go guards ClearScreen/ClearLine behind tty.IsStderrTerminal(), avoiding raw escape codes leaking into piped/redirected output. pkg/stringutil/ansi.go provides ANSI-stripping utilities (not emission), which is the correct direction for safety.
• Minor gap: only 6 files touch lipgloss.* directly outside pkg/styles/pkg/console (pkg/logger/logger.go, pkg/cli/compile_schedule_calendar.go). Worth a follow-up check that these don't duplicate palette/style definitions instead of importing from pkg/styles.

Huh usage details

Huh (charm.land/huh/v2) is used across 15 files, mostly in interactive CLI flows: pkg/cli/add_interactive_*.go, run_interactive.go, interactive.go, engine_secrets.go, bootstrap_profile_*.go, plus core wrappers in pkg/console/{confirm,input,list,prompt_form}.go.

• Theming: pkg/styles/huh_theme.go defines HuhTheme as a huh.ThemeFunc, mapping the same Dracula-inspired palette (colorPurpleLight/Dark, colorSuccessLight/Dark, etc.) used by theme.go's Lipgloss styles via lipgloss.LightDark(isDark). This is a single source of truth for both static and interactive UI — a strong pattern worth highlighting as exemplary.
• Thin wrappers: pkg/console/confirm.go, input.go, list.go, prompt_form.go are each small (37–74 lines), suggesting focused, single-responsibility wrappers around Huh fields rather than ad hoc form-building scattered through pkg/cli.
• Not verified in this pass: explicit accessible-mode (huh.WithAccessible(true)) and validation (.Validate()) usage per field — recommend a follow-up grep of WithAccessible|\.Validate\( in the 15 Huh-consuming files to confirm coverage, since these weren't exhaustively checked here.

Remaining raw fmt.Print* calls (reviewed, no action needed)

• pkg/cli/status_command.go:288 — fmt.Print(console.RenderStruct(statuses)): prints already-styled output from the console package; correct usage (avoids double-wrapping with another console call).
• pkg/cli/view_command.go:168 — fmt.Print(output): prints a pre-rendered timeline stream (renderUnifiedTimelineStream); same pattern, output is built via console/Lipgloss upstream.
• All other fmt.Println hits are in linter testdata/ fixtures (intentionally "bad" example code for linter tests) or doc comments — not real output paths.

Recommendations

1. No urgent fixes required. The console/styles architecture is consistent and TTY-aware.
2. Follow up: confirm pkg/logger/logger.go and pkg/cli/compile_schedule_calendar.go reuse pkg/styles colors rather than defining their own Lipgloss styles, to keep the single-source-of-truth palette intact.
3. Follow up: spot-check Huh forms in pkg/cli/add_interactive_*.go for WithAccessible(true) and per-field .Validate() to confirm accessibility/validation parity across all 15 form call sites.
4. Consider a lint/grep-based CI check (or go vet-style analyzer, per this repo's existing pkg/linters pattern) that flags new raw fmt.Print* calls in pkg/cli outside of already-rendered console.* output, to keep this discipline as the codebase grows.

Generated by 💄 Terminal Stylist · auto · 22.5 AIC · ⌖ 2.38 AIC · ⊞ 8.1K · ◷

• expires on Aug 7, 2026, 1:37 AM UTC-08:00

[github-actions[bot]]

This discussion has been marked as outdated by Terminal Stylist.

A newer discussion is available at Discussion #51041.