ποΈ Architect: Refactor Endpoints for SOLID Principles #606
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.
ποΈ Architect: Refactor Endpoint Logic
ποΈ Design Change:
_list_impl(handling both sync/async viainspect) to explicit_list_syncand_list_asyncmethods inListGetEndpointMixin._get_implinto_get_syncand_get_async._process_filtershook to allow subclasses to extract parameters without overriding execution logic.β»οΈ DRY Gains:
_prepare_execution.JobsEndpointandRecordsEndpoint.π‘οΈ Solidity:
List[T]vsAny).BaseEndpoint._auto_filternow returns a copy, preventing side effects._list_impland_get_implare removed. Subclasses relying on overriding these (likeUsersEndpoint,RecordsEndpoint) have been updated to use_process_filters._list_implwere updated to mock_list_syncor_list_async.PR created automatically by Jules for task 1845034591522279126 started by @fderuiter