[code-simplifier] Remove redundant ANSI stripping comments

[github-actions]

Code Simplification - 2026-01-28

This PR simplifies recently modified code to improve clarity by removing redundant comments.

Files Simplified

• pkg/workflow/compiler_yaml.go - Removed 6 self-documenting comments

Improvements Made

Enhanced Clarity

Removed redundant inline comments that stated "Strip ANSI escape codes" before StripANSIEscapeCodes() function calls. The function name itself is sufficiently clear about the operation being performed.

Before:

// Strip ANSI escape codes from source path
cleanSource := stringutil.StripANSIEscapeCodes(data.Source)

After:

cleanSource := stringutil.StripANSIEscapeCodes(data.Source)

The high-level function comment on line 47 already documents that "All ANSI escape codes are stripped from the output", making these inline comments redundant.

Applied Project Standards

• Self-documenting code: Well-named functions don't need explanatory comments
• DRY principle: Function-level documentation instead of repetitive inline comments
• Code clarity: Removed noise that doesn't add value

Changes Based On

Recent changes from:

• #12210 - Refactor generateYAML() into focused helper functions
• Commit 72f8665 - Recent refactoring work that split generateYAML into helpers

Testing

• ✅ No functional changes - comments only
• ✅ High-level function comment preserved (line 47)
• ✅ Code behavior is identical
• ✅ All 6 redundant comments removed consistently

Review Focus

Please verify:

• Function remains self-documenting without inline comments
• High-level comment on line 47 provides sufficient context
• No unintended changes to code logic
• Comments removal improves rather than harms readability

---

Automated by Code Simplifier Agent - analyzing code from the last 24 hours

AI generated by Code Simplifier

• expires on Feb 4, 2026, 2:23 PM UTC