ruby-grape / grape / 26338323231 / 26

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>

370 of 1171 branches covered (31.6%)

Branch coverage included in aggregate %.

2600 of 3585 relevant lines covered (72.52%)

7.89 hits per line