ruby-grape / grape / 28751236776

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`.

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

1130 of 1226 branches covered (92.17%)

Branch coverage included in aggregate %.

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

3591 of 3659 relevant lines covered (98.14%)

23389.55 hits per line