Enhance workflow and job management logging (#3946)

Author: github-actions[bot]

pkg/workflow/filters.go

@@ -149,6 +149,7 @@ func (c *Compiler) applyPullRequestForkFilter(data *WorkflowData, frontmatter ma
 	// If "*" wildcard is present, skip fork filtering (allow all forks)
 	for _, pattern := range allowedForks {
 		if pattern == "*" {
+			filtersLog.Print("Wildcard fork pattern detected, allowing all forks")
 			return // No fork filtering needed
 		}
 	}
@@ -368,6 +369,7 @@ func (c *Compiler) applyLabelFilter(data *WorkflowData, frontmatter map[string]a
 
 	// If we have label conditions, combine them and apply to the workflow
 	if len(labelConditions) > 0 {
+		filtersLog.Printf("Applying label name filters: %d conditions found", len(labelConditions))
 		var finalCondition ConditionNode
 		if len(labelConditions) == 1 {
 			finalCondition = labelConditions[0]

pkg/workflow/jobs.go

@@ -101,6 +101,7 @@ func (jm *JobManager) ValidateDependencies() error {
 
 // detectCycles uses DFS to detect cycles in the job dependency graph
 func (jm *JobManager) detectCycles() error {
+	jobLog.Print("Detecting cycles in job dependency graph")
 	// Track visit states: 0=unvisited, 1=visiting, 2=visited
 	visitState := make(map[string]int)
 
@@ -118,6 +119,7 @@ func (jm *JobManager) detectCycles() error {
 		}
 	}
 
+	jobLog.Print("No cycles detected in job dependencies")
 	return nil
 }
 
@@ -145,6 +147,7 @@ func (jm *JobManager) dfsVisit(jobName string, visitState map[string]int) error
 
 // RenderToYAML generates the jobs section of a GitHub Actions workflow
 func (jm *JobManager) RenderToYAML() string {
+	jobLog.Printf("Rendering %d jobs to YAML", len(jm.jobs))
 	if len(jm.jobs) == 0 {
 		return "jobs:\n"
 	}
@@ -341,6 +344,7 @@ func (jm *JobManager) renderJob(job *Job) string {
 
 // GetTopologicalOrder returns jobs in topological order (dependencies before dependents)
 func (jm *JobManager) GetTopologicalOrder() ([]string, error) {
+	jobLog.Printf("Computing topological order for %d jobs", len(jm.jobs))
 	// First validate dependencies to ensure no cycles
 	if err := jm.ValidateDependencies(); err != nil {
 		return nil, err

pkg/workflow/runtime_setup.go

@@ -226,6 +226,7 @@ func DetectRuntimeRequirements(workflowData *WorkflowData) []RuntimeRequirement
 
 // detectFromCustomSteps scans custom steps YAML for runtime commands
 func detectFromCustomSteps(customSteps string, requirements map[string]*RuntimeRequirement) {
+	log.Print("Scanning custom steps for runtime commands")
 	lines := strings.Split(customSteps, "\n")
 	for _, line := range lines {
 		// Look for run: commands
@@ -277,6 +278,7 @@ func detectRuntimeFromCommand(cmdLine string, requirements map[string]*RuntimeRe
 
 // detectFromMCPConfigs scans MCP server configurations for runtime commands
 func detectFromMCPConfigs(tools map[string]any, requirements map[string]*RuntimeRequirement) {
+	log.Printf("Scanning %d MCP configurations for runtime commands", len(tools))
 	for _, tool := range tools {
 		// Handle structured MCP config with command field
 		if toolMap, ok := tool.(map[string]any); ok {
@@ -369,6 +371,7 @@ func GenerateRuntimeSetupSteps(requirements []RuntimeRequirement) []GitHubAction
 
 // generateSetupStep creates a setup step for a given runtime
 func generateSetupStep(runtime *Runtime, version string) GitHubActionStep {
+	log.Printf("Generating setup step for runtime: %s, version=%s", runtime.ID, version)
 	// Use default version if none specified
 	if version == "" {
 		version = runtime.DefaultVersion
@@ -426,6 +429,7 @@ func ShouldSkipRuntimeSetup(workflowData *WorkflowData) bool {
 
 // applyRuntimeOverrides applies runtime version overrides from frontmatter
 func applyRuntimeOverrides(runtimes map[string]any, requirements map[string]*RuntimeRequirement) {
+	log.Printf("Applying runtime overrides for %d configured runtimes", len(runtimes))
 	for runtimeID, configAny := range runtimes {
 		// Parse runtime configuration
 		configMap, ok := configAny.(map[string]any)

pkg/workflow/template.go

@@ -32,16 +32,19 @@ func wrapExpressionsInTemplateConditionals(markdown string) string {
 		// Check if expression is already wrapped in ${{ ... }}
 		// Look for the pattern starting with "${{"
 		if strings.HasPrefix(expr, "${{") {
+			templateLog.Print("Expression already wrapped, skipping")
 			return match // Already wrapped, return as-is
 		}
 
 		// Check if expression is an environment variable reference (starts with ${)
 		// These don't need ${{ }} wrapping as they're already evaluated
 		if strings.HasPrefix(expr, "${") {
+			templateLog.Print("Environment variable reference detected, skipping wrap")
 			return match // Environment variable reference, return as-is
 		}
 
 		// Always wrap expressions that don't start with ${{ or ${
+		templateLog.Printf("Wrapping expression: %s", expr)
 		return "{{#if ${{ " + expr + " }} }}"
 	})
 
@@ -73,6 +76,7 @@ func (c *Compiler) generateInterpolationAndTemplateStep(yaml *strings.Builder, e
 
 	// Skip if neither interpolation nor template rendering is needed
 	if !hasExpressions && !hasTemplates {
+		templateLog.Print("No interpolation or template rendering needed, skipping step generation")
 		return
 	}