ruby-grape / grape / 26327699911 / 22

97%

master: 97%

Pull #2734

github

Extract `options_route_enabled` from Endpoint options Hash into a dedicated accessor

`options_route_enabled` is not a constructor input — it's set later by
`Grape::API::Instance#configure_options_routes` via
`last_route.app.options[:options_route_enabled] = true`. Threading it
through the `**options` Hash was the only reason `Endpoint#options` had
to remain externally mutable.

This change:

- Adds `attr_accessor :options_route_enabled` on `Grape::Endpoint`,
  initialized to `false`.
- Rewrites `options?` to read the ivar instead of `options[:options_route_enabled]`.
- Rewrites the writer at `Grape::API::Instance` to use
  `last_route.app.options_route_enabled = true`.

No behavior change. Sets the stage for tightening the rest of the
Endpoint constructor surface in a follow-up.

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

203 of 1171 branches covered (17.34%)

Branch coverage included in aggregate %.

2186 of 3585 relevant lines covered (60.98%)

7.72 hits per line