ruby-grape / grape / 26328124769

97%

master: 97%

Pull #2736

github

Lazy-allocate `validation_errors` in Endpoint#run_validators

The happy path (no validator raises) used to allocate an empty
`validation_errors = []` Array on every request, then check
`validation_errors.any?` at the end. Mirror the lazy `||= []` shape
already used in `Validators::Base#validate!`:

- Start with `validation_errors = nil`.
- Inside the rescues, do `(validation_errors ||= []) << e` and
  `(validation_errors ||= []).concat(e.errors)`.
- Use `if validation_errors` as the raise guard — non-nil iff at least
  one validator raised, which is the same predicate as
  `.any?` for the prior empty-or-populated array.

Zero behaviour change; one fewer allocation per request on the
no-validation-error path.

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

1088 of 1173 branches covered (92.75%)

Branch coverage included in aggregate %.

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

3532 of 3591 relevant lines covered (98.36%)

31650.18 hits per line