ruby-grape / grape / 29093951006

97%

master: 97%

Pull #2791

github

Build helper routes from a settings copy, not shared mutation

add_head_not_allowed_methods_and_options_methods used
without_root_prefix_and_versioning to delete the root-prefix/versioning
keys from the shared class-level namespace_inheritable and restore them in
an ensure. Compilation runs when a fresh API instance is built, which can
happen at runtime (mount, helpers, or route definitions call change!) while
another instance is still serving. A request handled during that window
could observe the temporarily-missing keys — e.g. Instance#cascade? reads
namespace_inheritable[:version_options] and would fall back to true.

Read the settings the helper routes need from a local copy
(namespace_inheritable.to_hash minus the versioning keys) and pass it into
collect_route_config_per_pattern instead. Nothing in that path reads the
stripped keys, so behavior is unchanged, and the shared settings object is
no longer mutated during compilation.

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

1127 of 1222 branches covered (92.23%)

Branch coverage included in aggregate %.

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

3585 of 3653 relevant lines covered (98.14%)

23174.72 hits per line