ruby-grape / grape / 25729167044

97%

master: 97%

Pull #2710

github

Tidy up Grape::DeclaredParamsHandler

- Split declared_hash_attr's two-arm Hash branch into two named
  helpers (declare_nested for the recursive case, declare_leaf for
  the terminal one) and lift the Hash check into a guard.
- Extract nested_path_for(parent_path, key) so the
  `path.dup << key.to_s` idiom is replaced with a single non-mutating
  helper, used by both branches and by build_memo_key.
- handle_passed_param: lift the type-nil shortcut into a guard,
  fold the has_children predicate inline into the Hash branch
  (route_params.keys.none? { ... }), and rewrite the if/elsif/elsif
  ladder as three early-return guards. Drop the unused &_block
  parameter (the method already uses an implicit `yield`).

No behaviour change.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

1074 of 1157 branches covered (92.83%)

Branch coverage included in aggregate %.

27 of 27 new or added lines in 1 file covered. (100.0%)

3477 of 3533 relevant lines covered (98.41%)

32019.48 hits per line