diff --git a/pkg/cli/codemod_github_app.go b/pkg/cli/codemod_github_app.go index 05217577c14..2214a33d548 100644 --- a/pkg/cli/codemod_github_app.go +++ b/pkg/cli/codemod_github_app.go @@ -37,6 +37,7 @@ func hasDeprecatedAppField(frontmatter map[string]any) bool { if githubAny, hasGitHub := toolsMap["github"]; hasGitHub { if githubMap, ok := githubAny.(map[string]any); ok { if _, hasApp := githubMap["app"]; hasApp { + githubAppCodemodLog.Print("Deprecated 'app' field found in tools.github") return true } } @@ -48,6 +49,7 @@ func hasDeprecatedAppField(frontmatter map[string]any) bool { if soAny, hasSO := frontmatter["safe-outputs"]; hasSO { if soMap, ok := soAny.(map[string]any); ok { if _, hasApp := soMap["app"]; hasApp { + githubAppCodemodLog.Print("Deprecated 'app' field found in safe-outputs") return true } } @@ -57,6 +59,7 @@ func hasDeprecatedAppField(frontmatter map[string]any) bool { if checkoutAny, hasCheckout := frontmatter["checkout"]; hasCheckout { if checkoutMap, ok := checkoutAny.(map[string]any); ok { if _, hasApp := checkoutMap["app"]; hasApp { + githubAppCodemodLog.Print("Deprecated 'app' field found in checkout") return true } } @@ -64,6 +67,7 @@ func hasDeprecatedAppField(frontmatter map[string]any) bool { for _, item := range checkoutArr { if itemMap, ok := item.(map[string]any); ok { if _, hasApp := itemMap["app"]; hasApp { + githubAppCodemodLog.Print("Deprecated 'app' field found in checkout array item") return true } } diff --git a/pkg/workflow/compiler_yaml_ai_execution.go b/pkg/workflow/compiler_yaml_ai_execution.go index c0ac972ff18..1c0cf107abe 100644 --- a/pkg/workflow/compiler_yaml_ai_execution.go +++ b/pkg/workflow/compiler_yaml_ai_execution.go @@ -9,6 +9,7 @@ import ( func (c *Compiler) generateEngineExecutionSteps(yaml *strings.Builder, data *WorkflowData, engine CodingAgentEngine, logFile string) { steps := engine.GetExecutionSteps(data, logFile) + compilerYamlLog.Printf("Generating engine execution steps: engine=%s, steps=%d", engine.GetID(), len(steps)) for _, step := range steps { for _, line := range step { diff --git a/pkg/workflow/github_tool_to_toolset.go b/pkg/workflow/github_tool_to_toolset.go index 0ae9371a7e3..ce489c9b5d6 100644 --- a/pkg/workflow/github_tool_to_toolset.go +++ b/pkg/workflow/github_tool_to_toolset.go @@ -22,6 +22,7 @@ func init() { if err := json.Unmarshal(githubToolToToolsetJSON, &GitHubToolToToolsetMap); err != nil { panic("failed to load GitHub tool to toolset mapping: " + err.Error()) } + githubToolToToolsetLog.Printf("Loaded GitHub tool-to-toolset mapping: %d entries", len(GitHubToolToToolsetMap)) } // GitHubToolToToolsetMap is the last declaration in this file; ValidateGitHubToolsAgainstToolsets diff --git a/pkg/workflow/lock_schema.go b/pkg/workflow/lock_schema.go index d45d1b39f9b..1e5335c74f1 100644 --- a/pkg/workflow/lock_schema.go +++ b/pkg/workflow/lock_schema.go @@ -87,6 +87,8 @@ func formatSupportedVersions() string { // GenerateLockMetadata creates a LockMetadata struct for embedding in lock files // For release builds, the compiler version is included in the metadata func GenerateLockMetadata(frontmatterHash string, stopTime string, strict bool) *LockMetadata { + lockSchemaLog.Printf("Generating lock metadata: schema=%s, strict=%t, hasStopTime=%t", LockSchemaV2, strict, stopTime != "") + metadata := &LockMetadata{ SchemaVersion: LockSchemaV2, FrontmatterHash: frontmatterHash, @@ -97,6 +99,7 @@ func GenerateLockMetadata(frontmatterHash string, stopTime string, strict bool) // Include compiler version only for release builds if IsRelease() { metadata.CompilerVersion = GetVersion() + lockSchemaLog.Printf("Including compiler version in lock metadata: %s", metadata.CompilerVersion) } return metadata diff --git a/pkg/workflow/safe_outputs_config_helpers.go b/pkg/workflow/safe_outputs_config_helpers.go index c3a7958b383..e2a2b2cb3ad 100644 --- a/pkg/workflow/safe_outputs_config_helpers.go +++ b/pkg/workflow/safe_outputs_config_helpers.go @@ -199,6 +199,9 @@ func generateHideCommentConfig(max *string, defaultMax int, allowedReasons []str // - "allowed_repos" uses underscore to match JavaScript handler expectations (see repo_helpers.cjs) // This inconsistency is intentional to maintain compatibility with existing handler code. func generateTargetConfigWithRepos(targetConfig SafeOutputTargetConfig, max *string, defaultMax int, additionalFields map[string]any) map[string]any { + safeOutputsConfigGenLog.Printf("Generating target config: target=%s, targetRepo=%s, allowedReposCount=%d, additionalFieldsCount=%d", + targetConfig.Target, targetConfig.TargetRepoSlug, len(targetConfig.AllowedRepos), len(additionalFields)) + config := generateMaxConfig(max, defaultMax) // Add target if specified