-
Notifications
You must be signed in to change notification settings - Fork 252
Closed
Closed
Copy link
Labels
Description
Objective
Rename the WorkflowStep type in pkg/cli/copilot_setup.go to CopilotWorkflowStep to eliminate naming confusion with the comprehensive WorkflowStep type in pkg/workflow/step_types.go.
Context
Two WorkflowStep types currently exist with different field sets:
pkg/workflow/step_types.go- Comprehensive version with 11 fields for workflow compilationpkg/cli/copilot_setup.go- Simplified version with 5 fields for Copilot setup scaffolding
While these serve different purposes, the name collision creates confusion.
Implementation Steps
- Open
pkg/cli/copilot_setup.go - Rename type from
WorkflowSteptoCopilotWorkflowStepat line 42-49 - Update all references within the same file (estimated 3-5 locations)
- Run tests to verify:
make test-unit - Run full validation:
make agent-finish
Files to Modify
pkg/cli/copilot_setup.go- Rename type and update references
Acceptance Criteria
- Type renamed to
CopilotWorkflowStepin copilot_setup.go - All references within the file updated
- No type name collisions remain
- All tests pass (
make test-unitandmake test) - Code builds successfully (
make build)
Estimated Effort
30 minutes
Priority
Low - Improves clarity but doesn't affect functionality
Related to #6836
AI generated by Plan Command for discussion #6829
Reactions are currently unavailable