ruby-grape / grape / 26331367761

97%

master: 97%

Pull #2736

github

Lazy-allocate validation_exceptions and unify rescue branches via ValidationErrors flatten

`Grape::Endpoint#run_validators`:

- Happy path used to allocate an empty `validation_errors = []` Array on
  every request. Start at `nil` and lazy-initialize the local (now named
  `validation_exceptions`) inside the rescue so the happy path allocates zero.
- Collapse the two rescue handlers into one. Both `Grape::Exceptions::Validation`
  (one error) and `Grape::Exceptions::ValidationArrayErrors` (a bundle of
  Validations from one validator) are now pushed onto the same list as-is.

`Grape::Exceptions::Validation#errors` returns `self` so callers can treat a
single Validation and a ValidationArrayErrors wrapper uniformly via
`flat_map(&:errors)` — Array returns flatten in, non-Array returns
(i.e. `self`) append as one element.

`Grape::Exceptions::ValidationErrors#initialize` keyword renamed
`errors:` → `exceptions:` (the input array is a mix of validation
exception objects, not the grouped errors Hash). Flattens any nested
`ValidationArrayErrors` via `flat_map(&:errors)` before grouping by
`params`. The unwrapping knowledge lives in the aggregator that *is* the
validation-errors collection rather than being duplicated in the
endpoint loop. UPGRADING entry added.

`if validation_exceptions` is the new raise guard — non-nil iff at least
one validator raised, equivalent to the prior `.any?` check on a
possibly-empty array.

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

1087 of 1171 branches covered (92.83%)

Branch coverage included in aggregate %.

8 of 8 new or added lines in 3 files covered. (100.0%)

3532 of 3591 relevant lines covered (98.36%)

31649.63 hits per line