ruby-grape / grape

97%

master: 97%

LAST BUILD ON BRANCH refactor/router-pattern-delegation

branch: refactor/router-pattern-delegation

CHANGE BRANCH

x

Reset

• refactor/router-pattern-delegation
• DateFormat
• Handle_json_array
• ISSUE-2321
• Issue-1908
• active_support_6_1_minimal
• add_irb_in_gemfile_for_bundle_console_warning
• add_rack_3_2_gemfile
• add_rails_8_1_on_ci
• add_ruby_3_4
• api_override_singleton_methods
• body-metadata
• bugfix/1577
• bugfix/1970
• bugfix/1986
• bugfix/2041
• bugfix/autoload
• bugfix/memory-leak
• build_with_registry
• chore/cleanup-on-compilation
• chore/coercion-tests
• chore/primitive-coercer-spec
• cleanup-deprecated-code
• cookies_delegation
• crush_desclared_params
• declared_params_optional_array
• deprecated_endpoint_return
• doc
• doc/remove_manual_toc_and_toc_danger_check
• doc/remove_toc
• doc/update_reload_documentation_for_current_Rails_versions
• drop_support_ruby_2_7_0
• dynamic_registration
• empty_string_and_bool
• enable_branch_coverage
• enable_frozen_string_literal_ci
• entity-dsl
• erik/remove-faulty-documentation-about-blank-values
• exposes-to-s
• failing_spec_for_1967
• faster_normalize_path
• feature/grape-api-setup-var-array
• feature_rack_lint
• fix-636
• fix-badges
• fix-docs
• fix-helper-inheritance
• fix-typo
• fix/avoid_multiple_mounts_pollution
• fix/sendfile_test_fix
• fix_2566_memory_leak
• fix_all_media_types_regression
• fix_coercer_cache
• fix_endpoint_status_when_not_calling_error!
• fix_helpers_methods_in_public_scope
• fix_leaky_slash
• fix_middleware_keywords
• fix_registry_warning_message
• fix_require_logger
• fix_routes_memoization
• fix_shared_params_empty_args
• format_header_best_q_match
• funding
• happy_rubocop
• hook-readme
• instance_to_s_delegation
• issue-1775
• lazy_block
• lazy_compile
• less_active_support_concern
• less_parse_nested_query
• master
• migrate-to-dry-configurable
• mountable_apis
• optimize/parameters-extract-options-non-mutating
• optimize_api_documentation
• private_namespace_reverse_stackable_with_hash
• private_within_namespace
• real-api-integration-test
• reduce_array_alloc_on_setup
• refactor_http_headers
• refactor_route_setting_internal_usage
• refactor_setting_get_or_set
• refs/heads/master
• refs/tags/v1.7.1
• refs/tags/v1.8.0
• refs/tags/v2.0.0
• remove-unused-constant
• remove_namespace_reverse_stackable_from_public_interface
• remove_namespace_stackable_and_inheritable_from_public_space
• remove_namespace_stackable_with_hash_from_public_interface
• remove_non_supported_jsonapi
• remove_obsolete_docker_compose_version
• remove_settings_unset_functions
• remove_strict_hash_configuration
• rescue_from
• resolve_performace_issue_with_given
• revert-1953-chore/micto-optimization-3
• revert-wrong-commit
• revisit_auth_middleware
• revisit_endpoint_helpers_and_new
• revisit_middleware_default_options
• ruby3_handling_argument_delegation
• ruby_style_send
• small_refactor_versioner_middleware
• spec-only-rack-class
• try_pattern_instead_of_respond_to
• update_return_upgrading_notes_and_simplify_execute
• update_rubocop
• update_rubocop_1_71_2
• upgrade-mustermann
• upgrade-rubocop
• upgrading-description
• use_forwardable
• use_require_relative_in_gemspec
• v0.19.2
• v1.0.0
• v1.0.1
• v1.0.2
• v1.0.3
• v1.1.0
• v1.2.0
• v1.2.1
• v1.2.2
• v1.2.3
• v1.2.4
• v1.2.5
• v1.3.0
• v1.3.1
• v1.3.2
• v1.3.3
• v1.4.0
• v1.5.0
• v1.5.1
• v2.1.0
• v2.1.1
• v2.1.2
• v2.1.3
• v2.2.0
• v2.3.0
• v2.4.0
• v3.0.0
• v3.0.1
• validators_bad_encoding

Build # 19808742858

Build Type

Pull #2629

github

Committed by ericproulx

Commit Message

Refactor router architecture for better encapsulation

- Convert Route classes to use explicit parameters instead of options hashes
- Add Forwardable delegation for cleaner interfaces
- Move Pattern construction to caller for better separation of concerns
- Improve API design with keyword arguments throughout router system
- Replace nested option access (route.options[:endpoint]) with direct methods
- Simplify code with flat_map and remove unnecessary splat operators
- Extract DSL_METHODS constant in ApiDescription for reusability

This refactoring improves code maintainability, testability, and performance
while maintaining backward compatibility with the public API.

Pull Request Pull Request #2629: Refactor Router Architecture: Improve Encapsulation and API Design

Run Details

1064 of 1153 branches covered (92.28%)

Branch coverage included in aggregate %.

77 of 77 new or added lines in 9 files covered. (100.0%)

3354 of 3417 relevant lines covered (98.16%)

56078.53 hits per line