[deep-report] Add console.Print* wrappers to eliminate 1,640 two-step stderr calls

[github-actions]

Description

The pkg/console package exposes Format* styled-output functions but no Print* wrappers, so every caller re-implements fmt.Fprintln(os.Stderr, console.Format*(...)) by hand. The Repository Quality report (discussion #47087) profiled 1,640 two-step calls, 94 Fprintf variants, and 239 raw fmt.Fprintf(os.Stderr, ...) calls that bypass styling entirely. Add thin Print* helpers (mirroring each Format*) that write to os.Stderr internally, then migrate the two worst files: pkg/cli/mcp_inspect_mcp.go (70 raw calls) and pkg/cli/engine_secrets.go (63 raw calls).

Expected Impact

Removes the stdout/stderr mis-routing footgun, centralizes newline handling, and creates a single interception point for testing/WASM/observability. A follow-up linter can lock it in permanently.

Suggested Agent

Code Simplifier / File Diet agent (Copilot).

Estimated Effort

Medium (1-4 hours) — wrappers are trivial; scope migration to the 2 worst files.

Data Source

DeepReport analysis 2026-07-21; discussion #47087.

Generated by 🔬 Deep Report · age00 200.9 AIC · ⌖ 17.2 AIC · ⊞ 9.9K · ◷

• expires on Jul 23, 2026, 7:27 AM UTC-08:00

[github-actions]

🍪 Issue Monster selected this for Copilot

I have identified this issue as a good candidate for automated resolution and requested assignment to the Copilot coding agent.

If assignment succeeds, the Copilot coding agent will analyze the issue and create a pull request with the fix.

Om nom nom! 🍪

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

• pi.dev

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "pi.dev"

See Network Configuration for more information.

🍪 Om nom nom by Issue Monster · gpt54 2.83 AIC · ⌖ 7.74 AIC · ⊞ 7.5K · ◷