ruby-grape / grape / 27479679455

97%

master: 97%

Build Type

Pull #2764

github

Committed by ericproulx

Commit Message

Route Grape::Json through the non-deprecated MultiJSON API

multi_json 1.21.0 deprecated the `MultiJson` constant (in favor of
`MultiJSON`) and the `MultiJSON.dump`/`MultiJSON.load` method names (in
favor of `generate`/`parse`), all removed in 2.0. Grape detected the
backend via `::MultiJson` and called `dump`/`load`, so users on
multi_json >= 1.21 saw deprecation warnings (#2763).

Detect `::MultiJSON` and give `Grape::Json` a small facade whose
`dump`/`load` delegate to `MultiJSON.generate`/`parse` (byte-identical
output) and which re-exposes `parse`. Behavior is unchanged on every
path; the stdlib and legacy multi_json (< 1.21) fallbacks are untouched.

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

Pull Request Pull Request #2764: Route Grape::Json through the non-deprecated MultiJSON API

Coverage Stats

1122 of 1221 branches covered (91.89%)

Branch coverage included in aggregate %.

10 of 11 new or added lines in 1 file covered. (90.91%)

1 existing line in 1 file now uncovered.

3569 of 3637 relevant lines covered (98.13%)

23066.99 hits per line