Skip to content

Conversation

@github-actions
Copy link
Contributor

Summary

Enhanced 5 Go files with meaningful debug logging statements following the project's logging guidelines from AGENTS.md. All files already had logger declarations, so this PR focuses on adding strategic logging calls to improve troubleshooting and development visibility.

Files Enhanced

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

    • Added logging for new runtime requirement additions with version tracking
    • Added logging for automatic UV → Python dependency detection and addition
  2. pkg/workflow/repo_memory.go (workflow:repo_memory)

    • Added logging for default repo-memory configuration detection (nil value)
    • Added logging for boolean true/false configuration handling
    • Added logging for backward-compatible object-style configuration processing
  3. pkg/cli/run_command.go (cli:run_command)

    • Added logging for workflow enable/disable state detection with workflow ID
    • Added logging for temporary workflow enablement operations
    • Added logging for workflow already-enabled state confirmation
    • Added logging for lock file discovery and validation
  4. pkg/cli/mcp_server.go (mcp:server)

    • Added logging for MCP status tool execution with pattern and jq filter parameters
    • Added logging for MCP compile tool execution with comprehensive flag details
  5. pkg/workflow/gateway.go (workflow:gateway)

    • Enhanced gateway configuration logging with port, container, command, and server count

Logging Characteristics

All logging statements follow project guidelines:

  • ✅ Use existing logger declarations (no new loggers added)
  • ✅ Log function parameters without side effects
  • ✅ Log important state changes and control flow decisions
  • ✅ Provide meaningful context for debugging
  • ✅ No duplication of user-facing console messages
  • ✅ Use log.Printf for formatted output with context

Validation

  • ✅ Successfully compiled with make build
  • ✅ Tested workflow compilation with DEBUG=* ./gh-aw compile dev
  • ✅ Verified debug logging output appears correctly
  • ✅ No compilation errors or warnings
  • ✅ Cache updated with session 34 processing details

Example Debug Output

With DEBUG=workflow:runtime_setup enabled:

workflow:runtime_setup Detecting runtime requirements from workflow data
workflow:runtime_setup Applying runtime overrides for 0 configured runtimes
workflow:runtime_setup Detected 0 runtime requirements: []

With workflow that uses UV package manager:

workflow:runtime_setup UV detected without Python, automatically adding Python runtime
workflow:runtime_setup Adding new runtime requirement: python (version=)

Session Info

  • Session: 34
  • Files enhanced: 5
  • Total project files enhanced: 49 (across all sessions)
  • Commit: e140f2c

AI generated by Go Logger Enhancement

…ver, and gateway

Enhanced 5 Go files with meaningful debug logging statements following
the project's logging guidelines. All files already had logger declarations,
so this focuses on adding strategic logging calls to improve troubleshooting.

Files enhanced:
- pkg/workflow/runtime_setup.go: Runtime requirement and UV/Python detection
- pkg/workflow/repo_memory.go: Configuration parsing (nil/boolean/object)
- pkg/cli/run_command.go: Workflow enable/disable and lock file operations
- pkg/cli/mcp_server.go: MCP tool execution (status and compile)
- pkg/workflow/gateway.go: Gateway configuration logging

🤖 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 31, 2025
@pelikhan pelikhan merged commit 34546ba into main Dec 31, 2025
4 checks passed
@pelikhan pelikhan deleted the go-logger/session-34-runtime-repo-memory-run-mcp-gateway-e40b6fbd15a4e7bb branch December 31, 2025 07:02
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