ruby-grape / grape
97%
master: 97%

LAST BUILD BRANCH: fix/adversarial-sweep-2026-08
DEFAULT BRANCH: master
Repo Added
Files 167
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH fix-router-cascade-sibling-routes
branch: fix-router-cascade-sibling-routes
CHANGE BRANCH
x

coverage: 96.96% (+0.004%) from 96.956%
30239832808

Pull #2829

github

ericproulx 
Let a cascading route hand over to every remaining route

Router#transaction sent a cascading match straight to the greedy
neighbour — the last route registered for the path — and returned its
answer, so any route registered between the first match and that last
one was unreachable. With three mounted API versions plus a catch-all
`route :any, '*path'`, v1 and v3 were served while v2 answered 406; the
shortcut only ever looked correct with exactly two versions, where "the
last route" happens to be the sibling.

Try the method's remaining routes (#rotation) as soon as a match
cascades, before the ANY routes and the greedy neighbour, and keep the
priority order otherwise intact. A consequence, and what cascade: true
means: with a catch-all present, an unmatched version now falls through
to it rather than surfacing the versioner's 406. cascade: false still
answers 406 immediately and never consults the catch-all.

Also close the body of every superseded cascading response — Rack
requires it, and rotation previously leaked them (Rack::Lint flagged
this as soon as a third sibling made the path reachable).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Pull Request #2829: Let a cascading route hand over to every remaining route

1192 of 1282 branches covered (92.98%)

Branch coverage included in aggregate %.

30 of 31 new or added lines in 1 file covered. (96.77%)

1 existing line in 1 file now uncovered.

3816 of 3883 relevant lines covered (98.27%)

26782.35 hits per line

Relevant lines Covered
3883 RELEVANT LINES 3816 COVERED LINES
26782.35 HITS PER LINE
Source Files on fix-router-cascade-sibling-routes

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
30239832808 fix-router-cascade-sibling-routes Let a cascading route hand over to every remaining route Router#transaction sent a cascading match straight to the greedy neighbour — the last route registered for the path — and returned its answer, so any route registered between the first matc... Pull #2829 ericproulx github
96.96
30239089784 fix-router-cascade-sibling-routes Let a cascading route hand over to every remaining route Router#transaction sent a cascading match straight to the greedy neighbour — the last route registered for the path — and returned its answer, so any route registered between the first matc... Pull #2829 ericproulx github
96.94
30218074287 fix-router-cascade-sibling-routes Let a cascading route hand over to every remaining route Router#transaction sent a cascading match straight to the greedy neighbour — the last route registered for the path — and returned its answer, so any route registered between the first matc... Pull #2829 ericproulx github
96.92
30218047118 fix-router-cascade-sibling-routes Let a cascading route hand over to every remaining route Router#transaction sent a cascading match straight to the greedy neighbour — the last route registered for the path — and returned its answer, so any route registered between the first matc... push ericproulx github
96.92
30218032275 fix-router-cascade-sibling-routes Let a cascading route hand over to every remaining route Router#transaction sent a cascading match straight to the greedy neighbour — the last route registered for the path — and returned its answer, so any route registered between the first matc... push ericproulx github
96.92
See All Builds (3445)