ruby-grape / grape / 28699839219

97%

master: 97%

Pull #2774

github

Make forward_match an internal kwarg instead of a route option

`forward_match` was threaded from the mount/route DSL through
`Endpoint::Options` and the route options hash, only to be read once in
`Route#initialize` to pick the match function. Since the value it derives
from (`config.app`) already lives on the endpoint, compute it locally in
`to_routes` and pass it to `Grape::Router::Route.new` as a keyword
argument.

This drops `forward_match` from `Endpoint::Options`, from the route
options hash, and removes the now-dead `BaseRoute#forward_match`
delegator (which had no callers). Adds a direct unit test for
`Route#match?` under both forward/non-forward modes and an UPGRADING note
for the removed introspection surface.

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

1129 of 1226 branches covered (92.09%)

Branch coverage included in aggregate %.

8 of 8 new or added lines in 4 files covered. (100.0%)

3585 of 3653 relevant lines covered (98.14%)

23123.08 hits per line