Skip to content

Conversation

@github-actions
Copy link
Contributor

Summary

Enhanced 5 Go files with debug logging statements to improve troubleshooting and development workflow visibility. All changes follow the project's logging guidelines from AGENTS.md.

Files Enhanced

  1. pkg/workflow/push_to_pull_request_branch.go (workflow:push_to_pull_request_branch)

    • Added logging for job building process
    • Added logging for GitHub App token minting
    • Added logging for configuration parsing
  2. pkg/workflow/schedule_preprocessing.go (workflow:schedule_preprocessing)

    • Added logging for schedule preprocessing entry
    • Added logging for shorthand schedule conversion
    • Added logging for schedule item processing with count
  3. pkg/workflow/tool_description_enhancer.go (workflow:tool_description_enhancer)

    • Added logging for tool description enhancement entry
    • Added logging when no constraints are found
    • Added logging with constraint count when constraints are added
  4. pkg/workflow/engine_firewall_support.go (workflow:engine_firewall_support)

    • Added logging for network support check entry with engine and strict mode
    • Added logging when no restrictions are defined
    • Added logging for engine firewall support status
    • Added logging for unsupported firewall warnings
  5. pkg/parser/ansi_strip.go (parser:ansi_strip)

    • Added logging with log.Enabled() check for ANSI stripping entry
    • Added logging with statistics (input/output length, escape sequences)

Guidelines Followed

  • ✅ Maximum 5 files modified
  • ✅ No test files modified
  • ✅ Logger declarations use correct naming convention (pkg:filename)
  • ✅ Logger arguments have no side effects
  • ✅ Logging messages are meaningful and helpful for debugging
  • ✅ Used log.Enabled() check for expensive debug operations (parser/ansi_strip.go)
  • ✅ Import statements properly formatted
  • ✅ Changes validated with make build (no compilation errors)
  • ✅ Workflow compilation tested with DEBUG=* ./gh-aw compile dev

Test Plan

  • Build succeeded: make build completes without errors
  • Debug logging works: DEBUG=* ./gh-aw compile dev shows log output
  • No test files were modified
  • Logger naming follows convention: pkg:filename

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 4.5 (noreply@anthropic.com)

AI generated by Go Logger Enhancement

Enhanced 5 Go files with debug logging following project guidelines:
- workflow/push_to_pull_request_branch.go: job building, app tokens, config parsing
- workflow/schedule_preprocessing.go: schedule conversion and processing
- workflow/tool_description_enhancer.go: tool enhancement with constraints
- workflow/engine_firewall_support.go: network support and firewall validation
- parser/ansi_strip.go: ANSI stripping with statistics (uses log.Enabled())

All loggers follow naming convention (pkg:filename) with no side effects.

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

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@github-actions github-actions bot added automation enhancement New feature or request labels Dec 11, 2025
@pelikhan pelikhan merged commit 7da1bd9 into main Dec 11, 2025
5 checks passed
@pelikhan pelikhan deleted the go-logger-session-13-b6c888551e13ee1a branch December 11, 2025 12:24
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.

2 participants