Skip to content

[log] Add debug logging to artifact manager, update command, and MCP config utils#20261

Merged
pelikhan merged 1 commit intomainfrom
add-debug-logging-artifact-manager-update-cmd-01256dedb20ec332
Mar 9, 2026
Merged

[log] Add debug logging to artifact manager, update command, and MCP config utils#20261
pelikhan merged 1 commit intomainfrom
add-debug-logging-artifact-manager-update-cmd-01256dedb20ec332

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Mar 9, 2026

Adds meaningful debug log statements to 5 Go files in pkg/ that had minimal logging coverage.

Changes

  • pkg/workflow/artifact_manager.go: Log when NewArtifactManager() creates a new manager instance
  • pkg/workflow/create_code_scanning_alert.go: Log custom driver name, target repo, allowed repos count, and full parsed config summary in parseCodeScanningAlertsConfig
  • pkg/cli/logs_utils.go: Log count of discovered .lock.yml files and each extracted workflow name in getAgenticWorkflowNames
  • pkg/workflow/mcp_config_utils.go: Log the boolean decision result of shouldRewriteLocalhostToDocker to aid sandbox connectivity debugging
  • pkg/cli/update_command.go: Log before UpdateActions and UpdateActionsInWorkflowFiles calls with relevant flags, plus completion status in RunUpdateWorkflows

Test plan

  • make build passes with no compilation errors
  • Enable DEBUG=* and run gh aw compile to verify logs appear at expected callsites
  • Run gh aw update with DEBUG=* to verify update-phase logs appear in sequence

Generated by Go Logger Enhancement ·

Warning

⚠️ Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • proxy.golang.org

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "proxy.golang.org"

See Network Configuration for more information.

  • expires on Mar 11, 2026, 8:32 PM UTC

… utils

- artifact_manager.go: Log when NewArtifactManager() constructs a new manager
- create_code_scanning_alert.go: Log driver name, target-repo, allowed-repos count,
  and parsed config summary in parseCodeScanningAlertsConfig
- logs_utils.go: Log count of .lock.yml files found and each discovered workflow name
- mcp_config_utils.go: Log the shouldRewriteLocalhostToDocker decision result
- update_command.go: Log before UpdateActions and UpdateActionsInWorkflowFiles calls,
  and log completion status in RunUpdateWorkflows

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions github-actions bot added automation enhancement New feature or request labels Mar 9, 2026
@pelikhan pelikhan merged commit fbfcab4 into main Mar 9, 2026
@pelikhan pelikhan deleted the add-debug-logging-artifact-manager-update-cmd-01256dedb20ec332 branch March 9, 2026 22:06
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