ruby-grape / grape / 26338323231

97%

master: 97%

Pull #2738

github

Skip redundant `GRAPE_ROUTING_ARGS` rewrite when route has no params to merge

`Router#process_route` previously did:

  args = env[Grape::Env::GRAPE_ROUTING_ARGS] || { route_info: route }
  route_params = route.params(input)
  env[Grape::Env::GRAPE_ROUTING_ARGS] = route_params.blank? ? args : args.merge(route_params)

When `env[GRAPE_ROUTING_ARGS]` was already set and `route_params` was
blank, the ternary returned `args` unchanged and we wrote the same Hash
back to env — a no-op assignment on every cascade match through a route
with no URL placeholders (the common case).

Switch to `||=` for the default, and only re-assign env when there are
new params to merge. Same observable behavior; one fewer env write per
no-param route match.

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

1088 of 1173 branches covered (92.75%)

Branch coverage included in aggregate %.

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

3534 of 3593 relevant lines covered (98.36%)

31633.32 hits per line