ruby-grape / grape / 29972459133

97%

master: 97%

Pull #2810

github

Make InheritableSetting's raw stores internal where the ecosystem allows

Enforce the per-key encapsulation that the semantic accessors already
provide: `namespace_inheritable` is now protected and
`namespace_stackable_with_hash` private, so every `namespace_inheritable`
key is reachable only through its dedicated accessor. Internal code reads
the `@namespace_inheritable` / `@namespace_stackable` ivars directly; the
protected/public readers exist for inherit_from / copy_state_from peer
access.

`namespace_stackable` deliberately stays a public reader: grape-swagger
reads it directly (and walks its inherited_values chain). Treat it as
read-only from the outside and prefer the dedicated accessors.

Two supporting changes:

* `Router::Pattern::Path` no longer receives a merged dump of both
  stores; `Endpoint#to_routes` passes the new
  `InheritableSetting#path_settings` snapshot — a `PathSettings` value
  object (`Data`) carrying exactly the six values `Path` reads.
* `InheritableSetting#mount_paths` exposes the full mount-path stack,
  complementing `#mount_path` (outermost entry only).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

1137 of 1230 branches covered (92.44%)

Branch coverage included in aggregate %.

3749 of 3816 relevant lines covered (98.24%)

22703.16 hits per line