ruby-grape / grape / 24754755957

97%

push

github

Precompute prefix list in Versioner::Path (#2691)

`Versioner::Path#before` built `[mount_path, Grape::Router.normalize_path(prefix)]`
on every request and iterated it, filtering nil/empty/'/' entries with a guard
clause on each element. Both `mount_path` and `prefix` are fixed at mount time,
so this work is redundant.

Memoize a pre-filtered `@prefixes` array in `initialize` (the middleware dup in
`Middleware::Base#call` shallow-copies ivars, so the frozen array is shared
across request dups for free). The per-request loop simplifies to
`if path_info.start_with?(path)`.

  current                    1.09 M i/s, 80KB / 2k objects
  variant A (memoize normalize) 1.34 M i/s, 80KB / 2k objects
  variant B (memoize list)   2.61 M i/s, 40KB / 1k objects  (2.38x faster, 2x fewer)

Shipping variant B.

No behavior change; all 2,236 specs pass.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

1077 of 1161 branches covered (92.76%)

Branch coverage included in aggregate %.

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

3354 of 3407 relevant lines covered (98.44%)

32198.21 hits per line