ruby-grape / grape
97%
master: 97%

LAST BUILD BRANCH: perf/build-headers-each-header
DEFAULT BRANCH: master
Repo Added
Files 167
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH perf/precompute-middleware-caches
branch: perf/precompute-middleware-caches
CHANGE BRANCH
x

coverage: 96.893% (-0.01%) from 96.907%
24630181277

Pull #2686

github

ericproulx 
Add Grape::Middleware::PrecomputedContentTypes and short-circuit merge_headers

Grape::Middleware::Base#call does dup.call!(env) per request, so the
+||=+ memoization on content_types / mime_types / content_types_indifferent_access
lived on the per-request dup — it was computed fresh every request.

This change introduces +Grape::Middleware::PrecomputedContentTypes+, an
opt-in mixin that warms the three caches on the parent instance at
initialization time. Per-request dups then inherit the cached values
via +dup+'s ivar copy instead of recomputing +with_indifferent_access+
(~1 µs each).

Grape::Middleware::Error, Formatter, and Versioner::Base include the
module. Plain Grape::Middleware::Base subclasses continue to compute
content_types lazily on first access — no behavior change for custom
middlewares. The module advertises intent at the class declaration,
making the warmup a named capability rather than an initializer side
effect.

Also short-circuits Grape::Middleware::Base#merge_headers when no
headers were set, avoiding both the lazy +Grape::Util::Header.new+
allocation and an empty +merge!+ against the response.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Pull Request #2686: Add Grape::Middleware::PrecomputedContentTypes and short-circuit merge_headers

1070 of 1158 branches covered (92.4%)

Branch coverage included in aggregate %.

16 of 17 new or added lines in 5 files covered. (94.12%)

3359 of 3413 relevant lines covered (98.42%)

33310.69 hits per line

Relevant lines Covered
3413 RELEVANT LINES 3359 COVERED LINES
33310.69 HITS PER LINE
Source Files on perf/precompute-middleware-caches

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
24630181277 perf/precompute-middleware-caches Add Grape::Middleware::PrecomputedContentTypes and short-circuit merge_headers Grape::Middleware::Base#call does dup.call!(env) per request, so the +||=+ memoization on content_types / mime_types / content_types_indifferent_access lived on the pe... Pull #2686 ericproulx github
96.89
24630096760 perf/precompute-middleware-caches Add Grape::Middleware::PrecomputedContentTypes and short-circuit merge_headers Grape::Middleware::Base#call does dup.call!(env) per request, so the +||=+ memoization on content_types / mime_types / content_types_indifferent_access lived on the pe... Pull #2686 ericproulx github
96.89
24629950730 perf/precompute-middleware-caches Precompute middleware content-type caches on the parent instance Grape::Middleware::Base#call does dup.call!(env) per request, so the +||=+ memoization on content_types / mime_types / content_types_indifferent_access lived on the per-request dup ... Pull #2686 ericproulx github
96.89
24629943149 perf/precompute-middleware-caches Precompute middleware content-type caches on the parent instance Grape::Middleware::Base#call does dup.call!(env) per request, so the +||=+ memoization on content_types / mime_types / content_types_indifferent_access lived on the per-request dup ... Pull #2686 ericproulx github
96.89
See All Builds (3142)