style: improve console output formatting across workflow engines and CLI

- Fix unstyled fmt.Printf calls in workflow engines to use console.FormatErrorMessage, console.FormatWarningMessage, and console.FormatVerboseMessage
- Update CLI success messages to use console.FormatSuccessMessage
- Add console package imports to workflow files that were missing them
- Ensure consistent terminal styling across claude_engine, custom_engine, codex_engine, cache, network_proxy, and push_to_pull_request_branch
- Maintain proper stderr output routing and TTY detection for color support

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: claude

pkg/cli/commands.go

@@ -352,9 +352,9 @@ func addWorkflowsWithPR(workflows []string, number int, verbose bool, engineOver
 	}
 
 	if len(workflows) == 1 {
-		fmt.Printf("Successfully created PR for workflow: %s\n", workflows[0])
+		fmt.Fprintln(os.Stderr, console.FormatSuccessMessage(fmt.Sprintf("Successfully created PR for workflow: %s", workflows[0])))
 	} else {
-		fmt.Printf("Successfully created PR for workflows: %s\n", strings.Join(workflows, ", "))
+		fmt.Fprintln(os.Stderr, console.FormatSuccessMessage(fmt.Sprintf("Successfully created PR for workflows: %s", strings.Join(workflows, ", "))))
 	}
 	return nil
 }

pkg/workflow/cache.go

@@ -2,8 +2,10 @@ package workflow
 
 import (
 	"fmt"
+	"os"
 	"strings"
 
+	"github.com/githubnext/gh-aw/pkg/console"
 	"github.com/goccy/go-yaml"
 )
 
@@ -87,7 +89,7 @@ func generateCacheSteps(builder *strings.Builder, data *WorkflowData, verbose bo
 	var topLevel map[string]any
 	if err := yaml.Unmarshal([]byte(data.Cache), &topLevel); err != nil {
 		if verbose {
-			fmt.Printf("Warning: Failed to parse cache configuration: %v\n", err)
+			fmt.Fprintln(os.Stderr, console.FormatWarningMessage(fmt.Sprintf("Failed to parse cache configuration: %v", err)))
 		}
 		return
 	}
@@ -96,7 +98,7 @@ func generateCacheSteps(builder *strings.Builder, data *WorkflowData, verbose bo
 	cacheConfig, exists := topLevel["cache"]
 	if !exists {
 		if verbose {
-			fmt.Printf("Warning: No cache key found in parsed configuration\n")
+			fmt.Fprintln(os.Stderr, console.FormatWarningMessage("No cache key found in parsed configuration"))
 		}
 		return
 	}

pkg/workflow/claude_engine.go

@@ -3,10 +3,13 @@ package workflow
 import (
 	"encoding/json"
 	"fmt"
+	"os"
 	"slices"
 	"sort"
 	"strings"
 	"time"
+
+	"github.com/githubnext/gh-aw/pkg/console"
 )
 
 // ClaudeEngine represents the Claude Code agentic engine
@@ -593,7 +596,7 @@ func (e *ClaudeEngine) RenderMCPConfig(yaml *strings.Builder, tools map[string]a
 			if toolConfig, ok := tools[toolName].(map[string]any); ok {
 				if hasMcp, _ := hasMCPConfig(toolConfig); hasMcp {
 					if err := e.renderClaudeMCPConfig(yaml, toolName, toolConfig, isLast); err != nil {
-						fmt.Printf("Error generating custom MCP configuration for %s: %v\n", toolName, err)
+						fmt.Fprintln(os.Stderr, console.FormatErrorMessage(fmt.Sprintf("Error generating custom MCP configuration for %s: %v", toolName, err)))
 					}
 				}
 			}
@@ -855,7 +858,7 @@ func (e *ClaudeEngine) parseClaudeJSONLog(logContent string, verbose bool) LogMe
 	if err := json.Unmarshal([]byte(logContent), &logEntries); err != nil {
 		// If that fails, try to parse as mixed format (debug logs + JSONL)
 		if verbose {
-			fmt.Printf("Failed to parse Claude log as JSON array, trying JSONL format: %v\n", err)
+			fmt.Fprintln(os.Stderr, console.FormatVerboseMessage(fmt.Sprintf("Failed to parse Claude log as JSON array, trying JSONL format: %v", err)))
 		}
 
 		logEntries = []map[string]any{}
@@ -914,7 +917,7 @@ func (e *ClaudeEngine) parseClaudeJSONLog(logContent string, verbose bool) LogMe
 			if err := json.Unmarshal([]byte(trimmedLine), &jsonEntry); err != nil {
 				// Skip invalid JSON lines (could be partial debug output)
 				if verbose {
-					fmt.Printf("Skipping invalid JSON line: %s\n", trimmedLine)
+					fmt.Fprintln(os.Stderr, console.FormatVerboseMessage(fmt.Sprintf("Skipping invalid JSON line: %s", trimmedLine)))
 				}
 				continue
 			}
@@ -924,13 +927,13 @@ func (e *ClaudeEngine) parseClaudeJSONLog(logContent string, verbose bool) LogMe
 
 		if len(logEntries) == 0 {
 			if verbose {
-				fmt.Printf("No valid JSON entries found in Claude log\n")
+				fmt.Fprintln(os.Stderr, console.FormatVerboseMessage("No valid JSON entries found in Claude log"))
 			}
 			return metrics
 		}
 
 		if verbose {
-			fmt.Printf("Extracted %d JSON entries from mixed format Claude log\n", len(logEntries))
+			fmt.Fprintln(os.Stderr, console.FormatVerboseMessage(fmt.Sprintf("Extracted %d JSON entries from mixed format Claude log", len(logEntries))))
 		}
 	}
 
@@ -982,8 +985,8 @@ func (e *ClaudeEngine) parseClaudeJSONLog(logContent string, verbose bool) LogMe
 				}
 
 				if verbose {
-					fmt.Printf("Extracted from Claude result payload: tokens=%d, cost=%.4f, turns=%d\n",
-						metrics.TokenUsage, metrics.EstimatedCost, metrics.Turns)
+					fmt.Fprintln(os.Stderr, console.FormatVerboseMessage(fmt.Sprintf("Extracted from Claude result payload: tokens=%d, cost=%.4f, turns=%d",
+						metrics.TokenUsage, metrics.EstimatedCost, metrics.Turns)))
 				}
 				break
 			} else if typeStr == "assistant" {
@@ -1027,8 +1030,8 @@ func (e *ClaudeEngine) parseClaudeJSONLog(logContent string, verbose bool) LogMe
 		for _, seq := range metrics.ToolSequences {
 			totalTools += len(seq)
 		}
-		fmt.Printf("Claude parser extracted %d tool sequences with %d total tool calls\n",
-			len(metrics.ToolSequences), totalTools)
+		fmt.Fprintln(os.Stderr, console.FormatVerboseMessage(fmt.Sprintf("Claude parser extracted %d tool sequences with %d total tool calls",
+			len(metrics.ToolSequences), totalTools)))
 	}
 
 	// Convert tool call map to slice

pkg/workflow/codex_engine.go

@@ -2,11 +2,14 @@ package workflow
 
 import (
 	"fmt"
+	"os"
 	"regexp"
 	"sort"
 	"strconv"
 	"strings"
 	"time"
+
+	"github.com/githubnext/gh-aw/pkg/console"
 )
 
 // convertToIdentifier converts a workflow name to a valid identifier format
@@ -250,7 +253,7 @@ func (e *CodexEngine) RenderMCPConfig(yaml *strings.Builder, tools map[string]an
 			if toolConfig, ok := expandedTools[toolName].(map[string]any); ok {
 				if hasMcp, _ := hasMCPConfig(toolConfig); hasMcp {
 					if err := e.renderCodexMCPConfig(yaml, toolName, toolConfig); err != nil {
-						fmt.Printf("Error generating custom MCP configuration for %s: %v\n", toolName, err)
+						fmt.Fprintln(os.Stderr, console.FormatErrorMessage(fmt.Sprintf("Error generating custom MCP configuration for %s: %v", toolName, err)))
 					}
 				}
 			}

pkg/workflow/custom_engine.go

@@ -2,7 +2,10 @@ package workflow
 
 import (
 	"fmt"
+	"os"
 	"strings"
+
+	"github.com/githubnext/gh-aw/pkg/console"
 )
 
 // CustomEngine represents a custom agentic engine that executes user-defined GitHub Actions steps
@@ -167,7 +170,7 @@ func (e *CustomEngine) RenderMCPConfig(yaml *strings.Builder, tools map[string]a
 			if toolConfig, ok := tools[toolName].(map[string]any); ok {
 				if hasMcp, _ := hasMCPConfig(toolConfig); hasMcp {
 					if err := e.renderCustomMCPConfig(yaml, toolName, toolConfig, isLast); err != nil {
-						fmt.Printf("Error generating custom MCP configuration for %s: %v\n", toolName, err)
+						fmt.Fprintln(os.Stderr, console.FormatErrorMessage(fmt.Sprintf("Error generating custom MCP configuration for %s: %v", toolName, err)))
 					}
 				}
 			}
@@ -300,7 +303,7 @@ func (e *CustomEngine) ParseLogMetrics(logContent string, verbose bool) LogMetri
 		if claudeMetrics.Turns > 0 || claudeMetrics.TokenUsage > 0 || claudeMetrics.EstimatedCost > 0 {
 			// Found structured data, use Claude parsing
 			if verbose {
-				fmt.Println("Custom engine: Using Claude-style parsing for logs")
+				fmt.Fprintln(os.Stderr, console.FormatVerboseMessage("Custom engine: Using Claude-style parsing for logs"))
 			}
 			return claudeMetrics
 		}
@@ -313,15 +316,15 @@ func (e *CustomEngine) ParseLogMetrics(logContent string, verbose bool) LogMetri
 		if codexMetrics.Turns > 0 || codexMetrics.TokenUsage > 0 {
 			// Found some data, use Codex parsing
 			if verbose {
-				fmt.Println("Custom engine: Using Codex-style parsing for logs")
+				fmt.Fprintln(os.Stderr, console.FormatVerboseMessage("Custom engine: Using Codex-style parsing for logs"))
 			}
 			return codexMetrics
 		}
 	}
 
 	// Fall back to basic parsing if neither Claude nor Codex approaches work
 	if verbose {
-		fmt.Println("Custom engine: Using basic fallback parsing for logs")
+		fmt.Fprintln(os.Stderr, console.FormatVerboseMessage("Custom engine: Using basic fallback parsing for logs"))
 	}
 
 	lines := strings.Split(logContent, "\n")

pkg/workflow/network_proxy.go

@@ -3,7 +3,10 @@ package workflow
 import (
 	_ "embed"
 	"fmt"
+	"os"
 	"strings"
+
+	"github.com/githubnext/gh-aw/pkg/console"
 )
 
 //go:embed config/squid.conf
@@ -56,14 +59,14 @@ func (c *Compiler) generateInlineProxyConfig(yaml *strings.Builder, toolName str
 	mcpConfig, err := getMCPConfig(toolConfig, toolName)
 	if err != nil {
 		if c.verbose {
-			fmt.Printf("Error getting MCP config for %s: %v\n", toolName, err)
+			fmt.Fprintln(os.Stderr, console.FormatVerboseMessage(fmt.Sprintf("Error getting MCP config for %s: %v", toolName, err)))
 		}
 		return
 	}
 
 	if mcpConfig.Container == "" {
 		if c.verbose {
-			fmt.Printf("Proxy-enabled tool '%s' missing container configuration\n", toolName)
+			fmt.Fprintln(os.Stderr, console.FormatVerboseMessage(fmt.Sprintf("Proxy-enabled tool '%s' missing container configuration", toolName)))
 		}
 		return
 	}
@@ -76,7 +79,7 @@ func (c *Compiler) generateInlineProxyConfig(yaml *strings.Builder, toolName str
 	}
 
 	if c.verbose {
-		fmt.Printf("Generating inline proxy configuration for tool '%s'\n", toolName)
+		fmt.Fprintln(os.Stderr, console.FormatVerboseMessage(fmt.Sprintf("Generating inline proxy configuration for tool '%s'", toolName)))
 	}
 
 	// Generate squid.conf inline

pkg/workflow/push_to_pull_request_branch.go

@@ -2,6 +2,9 @@ package workflow
 
 import (
 	"fmt"
+	"os"
+
+	"github.com/githubnext/gh-aw/pkg/console"
 )
 
 // PushToPullRequestBranchConfig holds configuration for pushing changes to a specific branch from agent output
@@ -157,7 +160,7 @@ func (c *Compiler) parsePushToPullRequestBranchConfig(outputMap map[string]any)
 					default:
 						// Invalid value, use default and log warning
 						if c.verbose {
-							fmt.Printf("Warning: invalid if-no-changes value '%s', using default 'warn'\n", ifNoChangesStr)
+							fmt.Fprintln(os.Stderr, console.FormatWarningMessage(fmt.Sprintf("invalid if-no-changes value '%s', using default 'warn'", ifNoChangesStr)))
 						}
 						pushToBranchConfig.IfNoChanges = "warn"
 					}