Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions pkg/cli/codemod_github_app.go
Original file line number Diff line number Diff line change
Expand Up @@ -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
}
}
Expand All @@ -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
}
}
Expand All @@ -57,13 +59,15 @@ 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
}
}
if checkoutArr, ok := checkoutAny.([]any); ok {
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
}
}
Expand Down
1 change: 1 addition & 0 deletions pkg/workflow/compiler_yaml_ai_execution.go
Original file line number Diff line number Diff line change
Expand Up @@ -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 {
Expand Down
1 change: 1 addition & 0 deletions pkg/workflow/github_tool_to_toolset.go
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
3 changes: 3 additions & 0 deletions pkg/workflow/lock_schema.go
Original file line number Diff line number Diff line change
Expand Up @@ -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,
Expand All @@ -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
Expand Down
3 changes: 3 additions & 0 deletions pkg/workflow/safe_outputs_config_helpers.go
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down