Skip to content

[log] Add debug logging to safe outputs and runtime step generation#11461

Merged
pelikhan merged 2 commits intomainfrom
add-logging-safe-outputs-runtime-9ba24e3c5627464c
Jan 23, 2026
Merged

[log] Add debug logging to safe outputs and runtime step generation#11461
pelikhan merged 2 commits intomainfrom
add-logging-safe-outputs-runtime-9ba24e3c5627464c

Conversation

@github-actions
Copy link
Contributor

Summary

Enhanced 5 Go files with debug logging to improve troubleshooting and development capabilities in the workflow compiler.

Files Modified

  1. pkg/workflow/compiler_safe_outputs_env.go

    • Added logger declaration: safeOutputsEnvLog
    • Added 3 logging calls for safe output environment variable configuration
    • Logs function entry and control flow decisions
  2. pkg/workflow/compiler_safe_outputs_specialized.go

    • Added logger declaration: specializedOutputsLog
    • Added 3 logging calls for specialized safe output step config building
    • Logs configuration for assign-to-agent, create-agent-session, and create-project
  3. pkg/workflow/safe_inputs_generator.go

    • Added logger declaration: safeInputsGeneratorLog
    • Added 3 logging calls for safe inputs tools config generation
    • Logs tool count and config generation progress
  4. pkg/workflow/update_entity_helpers.go

    • Added logger declaration: updateEntityHelpersLog
    • Added 3 logging calls for update entity config parsing
    • Logs configuration parsing and validation
  5. pkg/workflow/runtime_step_generator.go

    • Added logger declaration: runtimeStepGeneratorLog
    • Added 4 logging calls for runtime setup step generation
    • Logs step generation progress and completion

Logging Guidelines Compliance

All changes follow the logging guidelines from AGENTS.md:

  • ✅ Using pkg:filename naming convention for logger IDs
  • ✅ Logging at function entry and important control flow decisions
  • ✅ Avoiding side effects in log arguments
  • ✅ Providing meaningful context for debugging

Testing

The code changes follow Go syntax and import conventions. Build validation should be performed in CI to ensure no compilation errors.

To test logging output:

DEBUG=* ./gh-aw compile dev

Context

This is part of an ongoing effort to enhance the codebase with debug logging. Previous sessions have added logging to 9 other files. This session (#52) adds logging to 5 more files, bringing the total to 14 files with enhanced logging.

🤖 Generated with [Claude Code]((redacted)

AI generated by Go Logger Enhancement

Enhanced 5 Go files with debug logging to improve troubleshooting and development:

- pkg/workflow/compiler_safe_outputs_env.go: Added logger declaration and 3 logging calls for safe output environment variable configuration
- pkg/workflow/compiler_safe_outputs_specialized.go: Added logger declaration and 3 logging calls for specialized safe output step config building (assign-to-agent, create-agent-session, create-project)
- pkg/workflow/safe_inputs_generator.go: Added logger declaration and 3 logging calls for safe inputs tools config generation
- pkg/workflow/update_entity_helpers.go: Added logger declaration and 3 logging calls for update entity config parsing
- pkg/workflow/runtime_step_generator.go: Added logger declaration and 4 logging calls for runtime setup step generation

All logging follows the guidelines from AGENTS.md:
- Using pkg:filename naming convention for logger IDs
- Logging at function entry and control flow decisions
- Avoiding side effects in log arguments
- Providing meaningful context for debugging

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@github-actions github-actions bot added automation enhancement New feature or request labels Jan 23, 2026
@pelikhan
Copy link
Contributor

@copilot generate a shared agentic workflow that exposes a "go" safe input tool and a "make" safe input tool that act as a way to invoke go/ make. Import these tools in this workflow.

Copy link
Contributor

Copilot AI commented Jan 23, 2026

@pelikhan I've opened a new pull request, #11462, to work on those changes. Once the pull request is ready, I'll request review from you.

@pelikhan pelikhan merged commit 5c608ac into main Jan 23, 2026
49 checks passed
@pelikhan pelikhan deleted the add-logging-safe-outputs-runtime-9ba24e3c5627464c branch January 23, 2026 11:13
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