ruby-grape / grape / 34032582409

99%

master: 100%

Pull #2907

github

Stop Endpoint::Options from appending to the caller's path Array

`Array(ary)` hands back the very Array it was given, so defaulting an
empty path by appending grew the Array the caller passed in.

A route declared inside a `namespace`, `resource`, `group` or
`route_param` block reaches `Grape::Endpoint` directly, so this is
visible from the public DSL — and a frozen Array raises at boot:

    PATHS = [].freeze

    class API < Grape::API
      namespace :v1 do
        get(PATHS) { 'ok' }   # FrozenError: can't modify frozen Array: []
      end
    end

A top-level route is shielded, but only incidentally: `Grape::API`
replays recorded setup steps through `evaluate_arguments`, which rebuilds
Array arguments with `map`, so the append lands on that copy instead.
Nested scopes run their block against the instance and skip it.

Build a new Array instead. Calling `super` with explicit keywords rather
than zsuper is what removes the need to assign back over the parameter.

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

1290 of 1332 branches covered (96.85%)

Branch coverage included in aggregate %.

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

4011 of 4011 relevant lines covered (100.0%)

27145.32 hits per line