Objective
Replace raw fmt.Fprintf(os.Stderr, ...) calls in pkg/cli/org_runner.go and pkg/cli/mcp_list_tools.go with proper console.FormatInfoMessageStderr / console.FormatListItemStderr wrappers.
Context
Identified in Terminal Stylist analysis: discussion #43520. These files output user-facing status messages directly to stderr, bypassing the console formatting layer. Wrapping them gives users the consistent i / • prefix, color, and TTY guards the rest of the codebase already uses.
Implementation Plan
Files to Modify
pkg/cli/org_runner.go lines 87–103: Replace fmt.Fprintf(os.Stderr, " - ...") bullet lines with console.FormatListItemStderr(...)pkg/cli/mcp_list_tools.go lines 77–136: Replace raw info strings to stderr with console.FormatInfoMessageStderr(...)
Approach
- Identify all raw
fmt.Fprintf(os.Stderr, ...) user-facing message callsites in each file (skip blank-line spacers).
- Replace bullet/list item lines with
console.FormatListItemStderr.
- Replace informational status lines with
console.FormatInfoMessageStderr.
- Run
make build (or go build ./...) and existing tests.
Notes
- Blank-line spacers (
fmt.Fprintln(os.Stderr, "")) are acceptable — leave as-is.
- Do NOT change
pkg/cli/logs_format_compact.go — its [tag] value format is intentional machine output.
Acceptance Criteria
Generated by 📋 Plan Command · 25.5 AIC · ⌖ 8.2 AIC · ⊞ 4.9K · ◷
Comment /plan to run again
Objective
Replace raw
fmt.Fprintf(os.Stderr, ...)calls inpkg/cli/org_runner.goandpkg/cli/mcp_list_tools.gowith properconsole.FormatInfoMessageStderr/console.FormatListItemStderrwrappers.Context
Identified in Terminal Stylist analysis: discussion #43520. These files output user-facing status messages directly to stderr, bypassing the console formatting layer. Wrapping them gives users the consistent
i/•prefix, color, and TTY guards the rest of the codebase already uses.Implementation Plan
Files to Modify
pkg/cli/org_runner.golines 87–103: Replacefmt.Fprintf(os.Stderr, " - ...")bullet lines withconsole.FormatListItemStderr(...)pkg/cli/mcp_list_tools.golines 77–136: Replace raw info strings to stderr withconsole.FormatInfoMessageStderr(...)Approach
fmt.Fprintf(os.Stderr, ...)user-facing message callsites in each file (skip blank-line spacers).console.FormatListItemStderr.console.FormatInfoMessageStderr.make build(orgo build ./...) and existing tests.Notes
fmt.Fprintln(os.Stderr, "")) are acceptable — leave as-is.pkg/cli/logs_format_compact.go— its[tag] valueformat is intentional machine output.Acceptance Criteria
org_runner.gobullet lines useconsole.FormatListItemStderrmcp_list_tools.goinfo lines useconsole.FormatInfoMessageStderrfmt.Fprintf(os.Stderr, ...)messages remain in those two files