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

[github-actions[bot]]

Overview

This report analyzes console output patterns across 983 Go source files in pkg/ (excluding test files), focusing on fmt.Print* usage, Lipgloss styling, and Huh interactive form implementations.

Key Metrics

Metric	Count
Go source files analyzed	983
console.Format* / console.Render* calls in pkg/cli	~1,638
Bare fmt.Fprintf(os.Stderr) without console wrapper	~553
huh.NewForm calls	20
Forms with WithTheme + WithAccessible	20 / 20 ✅
Files using Lipgloss directly (cli layer)	4
Raw ANSI escape sequences	3 (console pkg only, documented)

---

✅ What's Working Well

Lipgloss: Excellent adaptive color system

pkg/styles/theme.go implements a well-designed adaptive color system:

• Dual-palette: Dracula-inspired dark palette + high-contrast light palette for every color token
• Custom adaptiveColor type satisfies color.Color, auto-selecting light/dark based on terminal detection
• Windows-safe background detection: Skips lipgloss.HasDarkBackground probe on Windows to avoid STATUS_DLL_INIT_FAILED/hang under ConPTY
• Rich semantic token set: ColorError, ColorWarning, ColorSuccess, ColorInfo, ColorPurple, ColorYellow, ColorComment, ColorForeground, ColorBackground, ColorBorder, ColorTableAltRow
• Pre-configured style vars: Error, Warning, Success, Info, FilePath, LineNumber, ContextLine, Highlight, Command, Progress, Prompt, Verbose, Header, TableHeader, TableCell, TableTotal, TableTitle, TableBorder, TreeEnumerator, TreeNode, plus schedule-calendar intensity styles
• Consistent borders: RoundedBorder (primary), NormalBorder (subtle), ThickBorder (reserved)

Huh: 100% theme/accessibility coverage

All 20 huh.NewForm calls across 12 files consistently apply:

.WithTheme(styles.HuhTheme).WithAccessible(console.IsAccessibleMode())

The styles.HuhTheme is a huh.ThemeFunc that maps the Dracula palette to every huh field variant: focused/blurred states, text inputs, selects, multi-selects, confirms, buttons, and navigation indicators. console.IsAccessibleMode() checks the ACCESSIBLE env var for screen-reader support.

Files covered: console/input.go, console/list.go, console/confirm.go, cli/add_interactive_auth.go, cli/add_interactive_git.go, cli/add_interactive_workflow.go, cli/add_interactive_orchestrator.go, cli/add_interactive_engine.go, cli/add_interactive_schedule.go, cli/run_interactive.go, cli/engine_secrets.go, cli/interactive.go

TTY detection: Consistent and correct

• console.applyStyle / console.applyStyleWithTTY wrap every lipgloss render call with a TTY check — ANSI codes are stripped in pipes/redirects
• RenderTitleBox, RenderErrorBox, RenderInfoSection each have graceful non-TTY text fallbacks
• RenderTable accepts a TTYFunc override for tables written to stderr vs stdout
• NewSpinner checks tty.IsStderrTerminal() and IsAccessibleMode() before enabling animation
• Spinner uses tea.WithInput(nil) to avoid consuming keyboard input from subsequent huh forms

console package: Well-documented naming convention

pkg/console/doc.go documents a clear Format* vs Render* convention:

• Format*: pure string transformations for a single message type
• Render*: multi-element/structured output — tables, boxes, trees, structs

FormatErrorChain deserves special mention: it walks the errors.Unwrap chain, strips the inner-error suffix from each level's message, and renders a visually nested chain with consistent indentation.

Lipgloss/tree used correctly in cli layer

The four pkg/cli files that import lipgloss directly all use it appropriately:

• compile_schedule_calendar.go: intensityStyle() returns a pre-defined styles.ScheduleCalendar* style
• mcp_inspect.go: lipgloss/tree with styles.TreeEnumerator + styles.TreeNode
• status_command.go: lipgloss/tree for workflow dependency tree with same style tokens
• gateway_logs_timeline_render.go: lipgloss for timeline display

---

⚠️ Issues and Improvement Opportunities

Issue 1: Inconsistent sub-step indentation in trial_confirmation.go

File: pkg/cli/trial_confirmation.go (lines 182–183, 199–200)

Sub-step items a. and b. are printed with bare fmt.Fprintf while surrounding numbered steps use console.FormatInfoMessage:

// Current — bare, unstyled
fmt.Fprintf(os.Stderr, "     a. Execute %s\n", workflowName)
fmt.Fprintf(os.Stderr, "     b. Auto-merge any pull requests...\n")

// Better — consistent with surrounding context
fmt.Fprintf(os.Stderr, console.FormatInfoMessage("     a. Execute %s\n"), workflowName)
fmt.Fprintf(os.Stderr, console.FormatInfoMessage("     b. Auto-merge any pull requests...\n"))

Issue 2: Hardcoded [hint] prefix in logs_orchestrator.go

File: pkg/cli/logs_orchestrator.go (line 872)

// Current — raw prefix string, no styling
fmt.Fprintf(w, "[hint] %s\n", message)

The console package already formats hints in FormatError using styles.Info styling. A console.FormatHintMessage helper would unify this pattern.

Recommendation: Add FormatHintMessage(string) string to the console package (using styles.Info with a "hint: " prefix) and replace this raw format call.

Issue 3: Hardcoded separator lines without TTY guard

File: pkg/cli/add_interactive_orchestrator.go (lines 326, 328)

// Current — always emits Unicode ━ characters, no TTY guard
fmt.Fprintln(os.Stderr, "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━")

Compare with trial_confirmation.go:221 which wraps the separator in console.FormatInfoMessage for at least some styling, though still hardcodes the string.

Recommendation: Use console.RenderTitleBox or a dedicated console.RenderSeparator() helper that respects TTY detection.

Issue 4: Bare fmt.Fprintf in org_runner.go, mcp_list_tools.go, remove_command.go

Several files emit plain text to stderr without console formatters:

pkg/cli/org_runner.go (lines 87–103): Repository/workflow preview list items

// Current
fmt.Fprintf(os.Stderr, "  - Repository: %s\n", preview.Repo)
// Better
fmt.Fprintln(os.Stderr, console.FormatListItem("Repository: "+preview.Repo))

pkg/cli/mcp_list_tools.go (lines 77–79, 134–136): Server availability messages

// Current
fmt.Fprintf(os.Stderr, "Available MCP servers: ")
fmt.Fprintf(os.Stderr, "%s\n", strings.Join(serverNames, ", "))
// Better
fmt.Fprintln(os.Stderr, console.FormatInfoMessage("Available MCP servers: "+strings.Join(serverNames, ", ")))

pkg/cli/remove_command.go (lines 58–60, 104–106): Workflow/file listing — could use console.RenderTable for aligned multi-column display.

Issue 5 (low priority): Markdown table output without TTY detection

Files: pkg/cli/audit_diff_render.go, pkg/cli/audit_cross_run_render.go

These files emit hardcoded markdown table borders (|--------|) to stdout — intentional for downstream markdown consumption (e.g., GitHub comment formatting). However, there is no TTY detection, so raw markdown appears in interactive terminal sessions.

Recommendation: Detect stdout TTY and switch between console.RenderTable (terminal) and markdown format (pipe).

Issue 6 (low priority): update_container_pins.go and experiments_command.go

pkg/cli/update_container_pins.go (lines 160–185, 344): Uses bare fmt.Fprintln(os.Stderr, "") for blank lines and fmt.Fprintf(os.Stderr, " %s: %s\n", f.image, f.reason) for image/status pairs. console.RenderTable would give aligned, styled output.

pkg/cli/experiments_command.go (lines 765–766, 807): Uses bare fmt.Fprintf(os.Stderr, " Branch: %s\n", ...). console.RenderStruct with appropriate struct tags would give consistent rendering.

---

Recommendations Summary

Priority	Action	Files
🔴 High	Wrap sub-step fmt.Fprintf with console.FormatInfoMessage	cli/trial_confirmation.go
🟡 Medium	Add console.FormatHintMessage(), replace [hint] raw format	cli/logs_orchestrator.go, console/console.go
🟡 Medium	Replace hardcoded ━━━ separators with TTY-aware helper	cli/add_interactive_orchestrator.go, cli/trial_confirmation.go
🟢 Low	Use console.FormatListItem / FormatInfoMessage consistently	cli/org_runner.go, cli/mcp_list_tools.go, cli/remove_command.go
🟢 Low	Use console.RenderTable for aligned output	cli/update_container_pins.go, cli/experiments_command.go
🟢 Low	TTY-detect stdout in audit render files	cli/audit_diff_render.go, cli/audit_cross_run_render.go

References:

• §28435637759

Generated by 💄 Terminal Stylist · 121.9 AIC · ⌖ 10.6 AIC · ⊞ 5.7K · ◷

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

[github-actions[bot]]

This discussion was automatically closed because it expired on 2026-07-01T10:02:12.739Z.

Closed by Workflow