Skip to content

Conversation

@ndizazzo
Copy link
Owner

No description provided.

• Add generic BaseConstructor pattern to eliminate constructor boilerplate
• Create ParameterResolver for centralized parameter validation and resolution
• Implement OutputBuilder for standardized action output generation
• Refactor all File, System, and Docker actions to use new patterns
• Add comprehensive refactoring documentation and migration guide
• Reduce code duplication across 48+ action files while maintaining functionality
@ndizazzo ndizazzo merged commit 35b1f5d into main Aug 18, 2025
4 checks passed
@ndizazzo ndizazzo deleted the feature/dry-it-up branch August 18, 2025 22:13
@codecov
Copy link

codecov bot commented Aug 18, 2025

Codecov Report

❌ Patch coverage is 58.67769% with 400 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.42%. Comparing base (307e681) to head (814856e).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
actions/common/base_output_builder.go 0.00% 115 Missing ⚠️
actions/common/parameter_resolver.go 0.00% 101 Missing ⚠️
actions/common/output_builder.go 0.00% 83 Missing ⚠️
actions/common/base_constructor.go 0.00% 18 Missing ⚠️
actions/file/write_file_action.go 48.14% 12 Missing and 2 partials ⚠️
actions/docker/docker_ps_action.go 86.95% 4 Missing and 2 partials ⚠️
actions/docker/docker_image_list_action.go 82.75% 5 Missing ⚠️
actions/docker/docker_pull_action.go 68.75% 5 Missing ⚠️
actions/docker/docker_compose_ps_action.go 84.00% 4 Missing ⚠️
actions/file/read_file_action.go 75.00% 3 Missing and 1 partial ⚠️
... and 24 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #15      +/-   ##
==========================================
- Coverage   75.98%   74.42%   -1.57%     
==========================================
  Files          48       52       +4     
  Lines        6350     6177     -173     
==========================================
- Hits         4825     4597     -228     
- Misses       1237     1359     +122     
+ Partials      288      221      -67     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants