[task] Fix file naming inconsistencies and remove empty placeholder

[github-actions]

Objective

Correct file naming inconsistencies and remove unnecessary files to maintain codebase cleanliness and follow Go conventions.

Context

Two quick wins identified:

1. pkg/workflow/safe_outputs_env_test_helpers.go - Incorrectly named (looks like test file but isn't)
2. pkg/workflow/config.go - Empty placeholder file (4 lines, just a comment)

Related to #3713 - Issue #5 and Issue #5 (Empty Placeholder File).

Approach

Task 1: Fix Test Helper Naming

Current: safe_outputs_env_test_helpers.go
Problem: Name suggests it's a test file (contains _test_) but it doesn't end with _test.go and is not a test file
Solution: Rename to safe_outputs_env_helpers.go

Steps:

1. Rename file: safe_outputs_env_test_helpers.go → safe_outputs_env_helpers.go
2. Update any imports that reference this file
3. Verify no build or test issues

Task 2: Remove Empty Placeholder

Current: pkg/workflow/config.go (4 lines - just comment saying content moved)
Solution: Remove the file entirely

Steps:

1. Delete pkg/workflow/config.go
2. Verify no imports reference this file
3. Run full test suite to ensure no issues

Files to Rename

• pkg/workflow/safe_outputs_env_test_helpers.go → pkg/workflow/safe_outputs_env_helpers.go

Files to Delete

• pkg/workflow/config.go

Files to Check

• Any files that import safe_outputs_env_test_helpers - update import paths
• Any files that import config - verify they don't exist (file is empty)

Acceptance Criteria

• File renamed: safe_outputs_env_test_helpers.go → safe_outputs_env_helpers.go
• All imports updated to use new file name
• config.go deleted
• No broken imports or references
• make build succeeds
• make test passes
• File naming follows Go conventions
• No functionality changed
  Related to [refactor] 🔧 Semantic Function Clustering Analysis: Refactoring Opportunities #3713

AI generated by Plan Command for #3713