ruby-grape / grape / 35269842466

99%

master: 100%

Pull #2947

github

Speed up declared filling in a missing Hash param

When `declared(params)` fills in a Hash-typed param the request left out,
it answers `{}` unless the route declares a child of it, and it looked for
one by interpolating `"#{key}["` inside the `none?` block: a new String
for every param on the route. The prefix is now built once and passed to
the check. Its `k != key` guard is dropped too -- a key that starts with
`"#{key}["` is longer than `key`, so it never equals it.

On a request to a route with 19 optional String params and a Hash param,
sending only one of them, this takes 123 -> 105 objects per request and
is +3.8% without a JIT, +6.3% with YJIT; with 49 String params,
243 -> 195 objects and +4.4% / +8.9%.

Two same-shape cleanups on route compilation, with no throughput effect:
`routes` uses `flat_map` (`Endpoint#routes` is already flat), and a
route's requirements are merged into one Hash with `merge!` instead of a
fresh Hash per namespace.

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>

1427 of 1482 branches covered (96.29%)

Branch coverage included in aggregate %.

5 of 5 new or added lines in 3 files covered. (100.0%)

4290 of 4290 relevant lines covered (100.0%)

28158.74 hits per line