ruby-grape / grape / 30306029090 / 1

97%

master: 97%

push

github

Restore the per-attempt routing args in Grape::Router (#2834)

#2824 made process_route build a fresh routing-args Hash per attempt, so
a candidate tried after a cascading match (X-Cascade pass) could not
observe the previous attempt's :route_info or path captures. #2829 was
squash-merged from a branch cut before #2824 landed, so its squashed diff
replayed the stale router.rb over the fix and dropped the regression spec
along with it. Git reported no conflict, and #2824's CHANGELOG entry
survived, so the tree claims a fix it no longer has.

Symptoms are back on master: with a cascading v2 and a serving v1, the
`route` helper returns v2's route (origin "/:id" instead of "/:name") and
v2's capture leaks into params as {"id"=>"123", "name"=>"123"} -- an
endpoint sees a param it never declared.

Restore the fix and its spec verbatim. The router is the only writer of
env['grape.routing_args'], so the unconditional assignment is safe.

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

162 of 1282 branches covered (12.64%)

Branch coverage included in aggregate %.

2267 of 3854 relevant lines covered (58.82%)

0.96 hits per line