ruby-grape / grape / 26755015790

97%

Build Type

push

github

Committed by web-flow

Commit Message

Merge routing args in place in Router#process_route (#2754)

process_route built `args = env[GRAPE_ROUTING_ARGS] || { route_info: route }`
and then reassigned `env[GRAPE_ROUTING_ARGS] = args.merge(route_params)`, which
allocates a second Hash for every matched route that carries params.

Set the routing-args Hash once with `||=` and `merge!` the route params into it
in place, dropping the extra allocation. The resulting routing args are
identical (same keys and values); only the merge is now in-place. `route_params`
is a freshly built Hash, so nothing shared is mutated.

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Coverage Stats

1113 of 1205 branches covered (92.37%)

Branch coverage included in aggregate %.

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

1 existing line in 1 file now uncovered.

3541 of 3603 relevant lines covered (98.28%)

9700.96 hits per line