ruby-grape / grape / 25960041002

97%

master: 97%

Pull #2726

github

Reuse one AttributesIterator per validator; drop Enumerable

AttributesIterator's attrs/scope are static per validator; only params
varies per request. Build the iterator once in Base#initialize (frozen)
via an overridable iterator_class, and thread params through #each
instead of storing request-derived state in @original_params/@params.
The reused instance is shared across threads, so it must hold no
mutable per-request state.

- Remove `include Enumerable` (dead + misleading: #each never returned
  an Enumerator and nothing used map/select/etc.) and the unused
  `attr_reader :scope`.
- MultipleParamsBase#validate! now lazily builds array_errors
  (`nil` -> `||= []`) like Base, avoiding an empty-array allocation
  on the no-error path.

Allocation: one fewer object (~80 B) per validator per request;
speed-neutral. Full validation/endpoint suites green.

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

1084 of 1167 branches covered (92.89%)

Branch coverage included in aggregate %.

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

3494 of 3551 relevant lines covered (98.39%)

32181.33 hits per line