ruby-grape / grape / 25877276008

97%

master: 97%

Pull #2719

github

Move content-type helpers from Middleware::Base into PrecomputedContentTypes

`Middleware::Base#content_types` / `#mime_types` / `#content_type_for` /
`#content_type` / `#content_types_indifferent_access` are only consumed by
middleware that include `PrecomputedContentTypes` — namely `Formatter`,
`Error`, and `Versioner::Base`. The other concrete middlewares
(`Filter`, `Auth::Base`) never call them.

Move the five methods into `PrecomputedContentTypes`. `Base` keeps the
ivar plumbing (`@options`, `@env`, `@app`), `Rack::Request` wrapper,
header merge, and `query_params`; the content-type concern lives with
the mixin that already takes responsibility for warming its caches.

This narrows `Base` to the storage / lifecycle layer and unblocks future
work to give per-middleware `Options` value objects without `Base`
reaching back into the option hash. No behaviour change — every
middleware that called those helpers already included
`PrecomputedContentTypes`.

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

1084 of 1167 branches covered (92.89%)

Branch coverage included in aggregate %.

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

3496 of 3553 relevant lines covered (98.4%)

32137.04 hits per line