Skip to content

[log] add debug logging to 5 Go packages#30061

Merged
pelikhan merged 1 commit intomainfrom
main-baef1df8593c44af
May 4, 2026
Merged

[log] add debug logging to 5 Go packages#30061
pelikhan merged 1 commit intomainfrom
main-baef1df8593c44af

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented May 4, 2026

Adds logger.New(...) declarations and meaningful log.Printf/log.Print calls to 5 previously unlogged files across the workflow, parser, and cli packages.

Files changed

  • pkg/workflow/model_identifier.go — logs identifier parse entry, which branch is taken (provider-scoped, glob, bare name), query param count, and final result
  • pkg/workflow/model_aliases.go — logs import/frontmatter layer merge counts and final alias map size
  • pkg/workflow/heredoc_validation.go — logs delimiter validation attempts and detected injection attempts
  • pkg/parser/sub_agent_extractor.go — logs extraction entry, marker count, duplicate detection, per-agent extraction, and completion summary
  • pkg/cli/experiments_analyze_statistics.go — logs analysis entry, degenerate-case short-circuit, chi-square balance result, and final recommendation

Logger naming convention

All loggers follow the "pkg:filename" convention already used in the codebase:

  • modelIdentifierLog = logger.New("workflow:model_identifier")
  • modelAliasesLog = logger.New("workflow:model_aliases")
  • heredocLog = logger.New("workflow:heredoc_validation")
  • subAgentLog = logger.New("parser:sub_agent_extractor")
  • experimentsStatsLog = logger.New("cli:experiments_statistics")

Testing

make build fails in this environment due to a pre-existing Go toolchain version constraint (go.mod requires 1.25.8, environment has 1.24.13). The changes are purely additive (new logger var + Printf calls), follow patterns used in hundreds of other files in the codebase, and do not alter any logic paths.

Generated by Go Logger Enhancement · ● 1.6M ·

  • expires on May 6, 2026, 4:44 AM UTC

…ackages

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions github-actions Bot added automation enhancement New feature or request labels May 4, 2026
@pelikhan pelikhan merged commit 1abbed1 into main May 4, 2026
@pelikhan pelikhan deleted the main-baef1df8593c44af branch May 4, 2026 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant