ruby-grape / grape / 28751482172 / 11

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>

162 of 1220 branches covered (13.28%)

Branch coverage included in aggregate %.

2158 of 3652 relevant lines covered (59.09%)

1.02 hits per line