ruby-grape / grape
97%
master: 97%

LAST BUILD BRANCH: fix/adversarial-sweep-2026-08
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 freeze-coercers-and-sync-caches
branch: freeze-coercers-and-sync-caches
CHANGE BRANCH
x

coverage: 96.835% (+0.007%) from 96.828%
30172405085

Pull #2819

github

ericproulx 
Freeze coercers at construction and synchronize Grape::Util::Cache

Extends the validator freeze contract to the whole coercion layer,
naming the mechanism as Grape::Util::FreezeOnNew — extend into a class
whose instances are shared across requests and new returns a frozen
instance. A future request-time lazy ivar write raises FrozenError in
the first spec run instead of being a latent race. The module wraps new,
never initialize, so the freeze lands after the entire initialize chain
(SetCoercer assigns ivars after super); extending a hierarchy base
covers its subclasses via singleton-class inheritance.

- Extended into DryTypeCoercer (covers Primitive/Array/Set),
  CustomTypeCoercer (+ collection subclass), MultipleTypeCoercer
  (internal @type_coercers also frozen) and VariantCollectionCoercer.
  Validators::Base's hand-written new/super.freeze and
  ContractScopeValidator's end-of-initialize freeze now use the same
  module. DeepFreeze's obsolete "coercers memoize lazily" note updated.

- Grape::Util::Cache lookups are Monitor-synchronized. The caches are
  written at API *definition* time (params blocks run at class-body
  evaluation), which the compile-time Instance::LOCK does not cover and
  which can be concurrent (parallel eager loading, concurrent autoload
  of different API files). Monitor rather than Mutex because CoercerCache
  re-enters itself when a multiple-type coercer builds member coercers.

- Route#regexp_capture_index is assigned eagerly in #to_regexp instead
  of memoized on first read: the reader runs during request-time route
  matching on shared instances. Previously safe only because to_regexp
  happened to warm it during compilation.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Pull Request #2819: Freeze coercers at construction and synchronize Grape::Util::Cache

1137 of 1230 branches covered (92.44%)

Branch coverage included in aggregate %.

3758 of 3825 relevant lines covered (98.25%)

22893.09 hits per line

Relevant lines Covered
3825 RELEVANT LINES 3758 COVERED LINES
22893.09 HITS PER LINE
Source Files on freeze-coercers-and-sync-caches

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
30172405085 freeze-coercers-and-sync-caches Freeze coercers at construction and synchronize Grape::Util::Cache Extends the validator freeze contract to the whole coercion layer, naming the mechanism as Grape::Util::FreezeOnNew — extend into a class whose instances are shared across request... Pull #2819 ericproulx github
96.83
30171837642 freeze-coercers-and-sync-caches Merge 9205570b3 into a8b4b8523 Pull #2819 GitHub github
96.83
30171768031 freeze-coercers-and-sync-caches Freeze coercers at construction and synchronize Grape::Util::Cache push Eric Proulx github
96.83
30171759041 freeze-coercers-and-sync-caches Merge d35752e35 into a8b4b8523 Pull #2819 GitHub github
96.83
30171739110 freeze-coercers-and-sync-caches Freeze coercers at construction and synchronize Grape::Util::Cache push Eric Proulx github
96.83
See All Builds (3445)