Fix file naming inconsistencies and remove empty placeholder

[Copilot]

Two file naming issues identified: safe_outputs_env_test_helpers.go incorrectly contains _test_ in its name despite not being a test file, and config.go is an empty 4-line placeholder.

Changes

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

  • Removes misleading _test_ from non-test helper file
  • Contains assertEnvVarsInSteps() test helper used by safe_outputs_env_test.go and safe_outputs_env_integration_test.go
  • No import updates needed (same package)

• Deleted pkg/workflow/config.go

  • Empty placeholder with comment pointing to config_helpers.go
  • No references in codebase

Original prompt

---

This section details on the original issue you should resolve

<issue_title>[task] Fix file naming inconsistencies and remove empty placeholder</issue_title>
<issue_description>## 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</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes [task] Fix file naming inconsistencies and remove empty placeholder #3745

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[Copilot]

Pull Request Overview

This PR addresses two file naming inconsistencies in the codebase: renaming a misnamed test helper file and removing an empty placeholder file. The changes improve code organization and follow Go naming conventions.

• Renamed safe_outputs_env_test_helpers.go to safe_outputs_env_helpers.go, removing the misleading _test_ infix from a non-test file
• Deleted the empty config.go placeholder file that only contained a comment

Reviewed Changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File	Description
pkg/workflow/safe_outputs_env_helpers.go	Added (renamed from safe_outputs_env_test_helpers.go) - contains assertEnvVarsInSteps() test helper function used by test files
pkg/workflow/config.go	Deleted - empty 4-line placeholder with no references in codebase

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.