ruby-grape / grape / 25855125689

97%

master: 97%

Build Type

Pull #2712

github

Committed by ericproulx

Commit Message

Switch error_formatter to keyword arguments; expose status and headers

The error_formatter contract previously took five positional arguments —
`(message, backtrace, options, env, original_exception)` — which made it
impossible to thread additional context through to custom formatters
without breaking every existing override. Switch to keyword arguments and
add two fields that have been asked for in #2527:

    def call(message:, backtrace:, options:, env:,
             status:, headers:, original_exception:)

`status:` makes JSON:API-style error bodies straightforward (the spec
embeds the HTTP status code in the response). `headers:` lets formatters
react to the response content-type or trace the marker headers set by
`error!`. Both were previously only reachable via
`env[Grape::Env::API_ENDPOINT].status` and friends.

This is a contract break — existing custom formatters that re-declare
`call` with the positional signature will need to migrate. See
UPGRADING.md for the before/after.

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

Pull Request Pull Request #2712: Switch error_formatter to keyword arguments; expose status and headers

Coverage Stats

1075 of 1157 branches covered (92.91%)

Branch coverage included in aggregate %.

8 of 9 new or added lines in 2 files covered. (88.89%)

1 existing line in 1 file now uncovered.

3469 of 3526 relevant lines covered (98.38%)

32139.41 hits per line