Use the same structure for Blueprint JSON definitions and step handlers #215
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
Description
This Pull Request proposes a change in the structure of Blueprint JSON definitions and step handlers to make them consistent with each other.
In the current implementation, there is a discrepancy between how steps are defined in the Blueprint JSON and how their corresponding handlers are defined. While the Blueprint step definition includes the step name and its associated parameters, the handler function directly accepts the parameters without any reference to the step name. This lack of consistency can lead to confusion and errors while developing and maintaining the Blueprint.
This PR refactors step handler functions to accept parameters as an object that maps one to one with the related JSON structure. It also reuses the same TypeScript types. This makes the code more intuitive and less error-prone by providing a clear mapping between the step definitions and their corresponding handlers.
Before
After