ruby-grape / grape / 24755531894

97%

master: 97%

Build Type

Pull #2693

github

Committed by ericproulx

Commit Message

Introduce Grape::Exceptions::ErrorResponse value object

`Middleware::Error#error_response` previously handled three input shapes
polymorphically: a Hash from `throw :error, {...}`, a `Grape::Exceptions::Base`
subclass, and an arbitrary user-handler return. Hash-style access like
`error[:status]` only worked on exceptions because `Grape::Exceptions::Base#[]`
delegated to `__send__` — a "pretend to be a Hash" trick that existed solely
to make `error_response` work uniformly.

Introduce `Grape::Exceptions::ErrorResponse`, a small value object with
`status`, `message`, `headers`, `backtrace`, and `original_exception` readers
plus a `coerce(input)` classifier that normalizes Hash, Exception, and
ErrorResponse inputs into a uniform value at the boundary. The middleware now
calls `coerce` once at entry and reads typed attributes — no more polymorphism
puzzle in the body.

Migrated the seven internal `throw :error, hash` sites to throw `ErrorResponse`
instances. User-defined `throw :error, hash` from custom middleware or
`rescue_from` blocks continues to work via `coerce`.

Removed `Grape::Exceptions::Base#[]`. Code that read `exception[:status]`
must switch to `exception.status` — see UPGRADING for migration notes.

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

Pull Request Pull Request #2693: Introduce Grape::Exceptions::ErrorResponse value object

Coverage Stats

1081 of 1166 branches covered (92.71%)

Branch coverage included in aggregate %.

37 of 39 new or added lines in 6 files covered. (94.87%)

1 existing line in 1 file now uncovered.

3374 of 3429 relevant lines covered (98.4%)

32022.76 hits per line