[terminal-stylist] Terminal Stylist: Console Output Analysis Report

[github-actions[bot]]

Terminal Stylist — Console Output Analysis

Analyzed 939 Go source files (excluding tests) in pkg/ for console output patterns, Lipgloss, and Huh usage.

---

Key Metrics

Metric	Value
Files using console.Format*	192 (1,694 calls)
Files using console.Render* / Show*	~30
Files using styles package	24
Files importing lipgloss directly	6
Files using huh package	13 (20 form instances)

---

✅ What's Working Well

Lipgloss: All color constants in styles/theme.go use compat.AdaptiveColor{Light, Dark} — no hardcoded hex values leak outside the package. TTY detection is consistently applied: console.go uses applyStyle(), compile_schedule_calendar.go passes isTerminal to its intensityStyle() helper, and gateway_logs_timeline_render.go uses a streamColor() closure. Only 6 files import lipgloss directly (all in infrastructure layers).

Huh: All 20 form instances consistently use WithTheme(styles.HuhTheme).WithAccessible(console.IsAccessibleMode()). Non-TTY fallbacks exist in confirm.go and list.go. IsAccessibleMode() checks ACCESSIBLE, TERM=dumb, and NO_COLOR.

Console package: Rich formatter set (Success ✓, Error ✗, Warning ⚠, Info i, Command ⚡, Progress 🔨, Prompt ❓, Verbose 🔍), plus RenderTable, RenderStruct, RenderTree, RenderTitleBox, LogVerbose, Bubble Tea spinner, and WASM fallbacks.

---

⚠️ Findings & Recommendations

1. Raw separator in `actionlint.go` bypasses console infrastructure

pkg/cli/actionlint.go lines 118–123, 168 — hardcoded strings.Repeat("━", 60) output unconditionally to stderr. console.RenderTitleBox() already handles this with proper TTY branching:

// Current
separator := strings.Repeat("━", 60)
fmt.Fprintf(os.Stderr, "\n%s\n", separator)
fmt.Fprintf(os.Stderr, "%s\n", console.FormatInfoMessage("Actionlint Summary"))

// Recommended
for _, line := range console.RenderTitleBox("Actionlint Summary", 60) {
    fmt.Fprintln(os.Stderr, line)
}

Bullet-list of ErrorsByKind on line 146 also emits " • %s: %d\n" directly; console.FormatListItem() covers this.

2. Verbose debug output bypasses `console.LogVerbose`

pkg/cli/add_interactive_workflow.go lines 50, 56, 60, 64, 181–209 use raw if c.Verbose { fmt.Fprintf(os.Stderr, ...) }. The project already has console.LogVerbose(verbose, msg) (used in token_usage.go, run_workflow_tracking.go):

// Recommended
console.LogVerbose(c.Verbose, fmt.Sprintf("Checking workflow status (attempt %d/5) for: %s", i+1, workflowName))

3. Inconsistent severity icons in `observability_insights.go`

pkg/cli/observability_insights.go lines 340–351 — "high" gets 🔴 but "medium" falls back to "[medium]" plain text. Should use semantic console formatters:

// Recommended per severity
case "high":   fmt.Fprintln(os.Stderr, console.FormatErrorMessage(insight.Title))
case "medium": fmt.Fprintln(os.Stderr, console.FormatWarningMessage(insight.Title))
case "low":    fmt.Fprintln(os.Stderr, console.FormatInfoMessage(insight.Title))

4. Unstyled plain text mixed with styled output in `add_interactive_orchestrator.go`

Lines 247, 334 — direct fmt.Fprintf for list items and completion messages:

// Recommended
fmt.Fprintln(os.Stderr, console.FormatListItem(fmt.Sprintf(".github/workflows/%s", f)))
fmt.Fprintln(os.Stderr, console.FormatSuccessMessage(fmt.Sprintf("The workflow '%s' has been added.", wf.Spec.WorkflowName)))

5. WASM/non-WASM console asymmetry (minor)

Three functions in console_wasm.go have no styled counterpart in console.go: FormatLocationMessage (📁), FormatCountMessage (📊), FormatListHeader. Not called externally today, but if shared code ever calls them they'll fail to compile for non-WASM targets. Should add styled implementations or remove from the WASM stub.

---

Summary Priority

Priority	Finding	Files
Medium	Migrate verbose fmt to console.LogVerbose	add_interactive_workflow.go
Medium	Replace separator with console.RenderTitleBox	actionlint.go
Low	Fix inconsistent severity icons	observability_insights.go
Low	Style mixed plain-text list/completion messages	add_interactive_orchestrator.go
Minor	Fix WASM/non-WASM function parity	console.go / console_wasm.go

References: §27946278659

Generated by 💄 Terminal Stylist · 146.4 AIC · ⌖ 8.17 AIC · ⊞ 5.6K · ◷

• expires on Jun 23, 2026, 2:39 AM UTC-08:00

[github-actions[bot]]

This discussion was automatically closed because it expired on 2026-06-23T10:39:14.927Z.

Closed by Workflow