ruby-grape / grape / 25729167044 / 8

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>

1048 of 1155 branches covered (90.74%)

Branch coverage included in aggregate %.

3432 of 3524 relevant lines covered (97.39%)

1001.73 hits per line