ruby-grape / grape / 22399578588 / 14

97%

master: 97%

Pull #2660

github

Fix thread safety: move mutable ParamsScope state into per-request ParamScopeTracker

ParamsScope instances are shared across requests (definition-time objects).
The mutable `@index` and `@params_meeting_dependency` ivars were being
written during request processing, creating a race condition under concurrent
requests.

Introduces `ParamScopeTracker` which stores all per-request mutable state
(array indices + qualifying params) in a `Fiber[FIBER_KEY]` entry, isolating
each request's state from others. `Endpoint#run_validators` sets up the
tracker via `ParamScopeTracker.track {}`. `AttributesIterator#store_indices`
writes to it; `ParamsScope#full_name` and `#qualifying_params` read from it.

Uses `Fiber[]` (not `Thread.current[]`) so fiber-based servers (e.g. Falcon)
correctly isolate per-request state within each fiber.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

1055 of 1162 branches covered (90.79%)

Branch coverage included in aggregate %.

3286 of 3373 relevant lines covered (97.42%)

1398.16 hits per line