Skip to content

Comments

[log] Add debug logging to 5 workflow files#16639

Merged
pelikhan merged 1 commit intomainfrom
main-f0b6c6abeaa66b12
Feb 18, 2026
Merged

[log] Add debug logging to 5 workflow files#16639
pelikhan merged 1 commit intomainfrom
main-f0b6c6abeaa66b12

Conversation

@github-actions
Copy link
Contributor

Adds meaningful debug logging to 5 workflow files that had loggers declared but minimal log calls, improving troubleshooting of workflow compilation and configuration parsing.

Changes

pkg/workflow/inputs.go

  • Added inputsLog.Printf at the end of ParseInputDefinition to log parsed type, required flag, and option count

pkg/workflow/label_trigger_parser.go

  • Added entry log to expandLabelTriggerShorthand showing entity type and labels being expanded
  • Added fallback log when an unknown entity type defaults to the issues trigger key
  • Added result log showing the final trigger key and label count

pkg/workflow/push_to_pull_request_branch.go

  • Added entry log to buildCheckoutRepository showing target repo and trial mode
  • Added logs for trial mode logical repo slug override
  • Added log when checking out a non-default repository
  • Added log when appending an authentication token to the checkout step

pkg/workflow/markdown_unfencing.go

  • Added entry log to UnfenceMarkdown showing content size
  • Added log when no outer fence is detected (early return)
  • Added log when an opening markdown fence is detected (char type and length)
  • Added log when no matching closing fence is found (early return)

pkg/workflow/permissions.go

  • Added permissionsLog.Printf warning when convertStringToPermissionScope receives an unknown permission scope key

Validation

  • make build passes with no compilation errors
  • go vet ./pkg/workflow/... reports no issues
  • All log calls use the existing per-file logger variables following the pkg:filename naming convention
  • No log arguments compute values or cause side effects

AI generated by Go Logger Enhancement

  • expires on Feb 20, 2026, 8:32 PM UTC

Add meaningful debug logging to ParseInputDefinition, expandLabelTriggerShorthand,
buildCheckoutRepository, UnfenceMarkdown, and convertStringToPermissionScope to
improve troubleshooting of workflow compilation and configuration parsing.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions github-actions bot added automation enhancement New feature or request labels Feb 18, 2026
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