ruby-grape / grape / 28751482172

97%

master: 97%

Pull #2783

github

Read a route's version, anchor and requirements from its pattern

`version`, `anchor` and `requirements` are the attributes that shape the
matcher — they are passed to `Grape::Router::Pattern` to build the
mustermann pattern. They were also passed a second time to
`Grape::Router::Route` and stored as ivars, purely to back the readers.

Expose them on `Pattern` instead (`attr_reader`) and delegate the
`BaseRoute` readers to `@pattern`, alongside the existing `path`/`origin`
delegation. `to_routes` no longer passes them to `Route.new`. Metadata
that does not shape matching — `namespace`, `prefix`, `settings` — stays
as route keyword arguments.

grape-swagger already passes these to `Pattern.new` and none of them to
`Route.new`, so its routes' readers become correct rather than `nil`.

Also collapse `extract_capture` to a guard plus a single merge. With the
`format` keyword gone (#2782) it just adds an optional `version`
constraint to `requirements`, so the intermediate accumulator was noise.

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

1127 of 1222 branches covered (92.23%)

Branch coverage included in aggregate %.

9 of 9 new or added lines in 2 files covered. (100.0%)

3588 of 3656 relevant lines covered (98.14%)

23334.59 hits per line