[log] Add debug logging to 5 workflow files

[github-actions]

Summary

This PR adds comprehensive debug logging to 5 workflow files to improve troubleshooting and development visibility. All changes follow the project's logging guidelines from AGENTS.md.

Files Enhanced

1. pkg/workflow/manual_approval.go

• Logger: manualApprovalLog (workflow:manual_approval)
• Logging Added:
  • Manual approval configuration processing entry
  • On: section format detection (string vs object)
  • Manual approval extraction results (found/not found)
  • Configuration success or absence messages

2. pkg/workflow/copilot_participant_steps.go

• Logger: copilotParticipantLog (workflow:copilot_participant_steps)
• Logging Added:
  • Function entry with participant type and count
  • Empty participants handling
  • Token preference selection (Copilot vs regular)
  • Separate logging for assignee vs reviewer step generation

3. pkg/workflow/time_delta.go

• Logger: timeDeltaLog (workflow:time_delta)
• Logging Added:
  • Time delta parsing entry with input parameters
  • Validation failure messages (e.g., missing '+' prefix)
  • Successful parsing confirmation with formatted output
  • Absolute date-time parsing entry and success messages

4. pkg/workflow/xml_comments.go

• Logger: xmlCommentsLog (workflow:xml_comments)
• Logging Added:
  • XML comment removal entry with input line count
  • Code block detection (with marker type)
  • Removal completion with statistics (removed lines, output lines)

5. pkg/workflow/secret_masking.go

• Logger: secretMaskingLog (defined in redact_secrets.go)
• Logging Added:
  • Extraction function entry
  • Steps count extracted from frontmatter
  • Configuration status messages (found/not found)
• Note: File already had extensive logging in MergeSecretMasking function

Validation

✅ Build: Compiled successfully with make build
✅ Testing: Verified logging output with DEBUG=* ./gh-aw compile dev
✅ No Side Effects: All logger arguments are pure values, no function calls
✅ Naming Convention: All loggers follow pkg:filename pattern

Logging Best Practices Followed

• ✅ No side effects in logger arguments
• ✅ Meaningful debug messages for troubleshooting
• ✅ Logger naming convention: pkg:filename
• ✅ Strategic placement at function entry and key decision points
• ✅ No duplicate logging with existing logs
• ✅ Test files (*_test.go) not modified
• ✅ Maximum 5 files per PR for focused review

Example Output

workflow:manual_approval Processing manual-approval configuration
workflow:manual_approval on: section is object format but no manual-approval field found
workflow:manual_approval No manual approval configured for workflow
workflow:secret_masking Extracting secret-masking configuration from frontmatter
workflow:secret_masking No secret-masking configuration found in frontmatter

Cache Updated

The logging enhancement cache has been updated to track these 5 new files, bringing the total to 25 files with enhanced logging across the codebase.

🤖 Generated with Claude Code

AI generated by Go Logger Enhancement