ruby-grape / grape / 26327699911

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>

1088 of 1173 branches covered (92.75%)

Branch coverage included in aggregate %.

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

3534 of 3593 relevant lines covered (98.36%)

31652.6 hits per line