ruby-grape / grape / 25973725275 / 32

97%

master: 97%

push

github

Reuse one AttributesIterator per validator; drop Enumerable (#2726)

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>

1060 of 1169 branches covered (90.68%)

Branch coverage included in aggregate %.

3477 of 3571 relevant lines covered (97.37%)

994.75 hits per line