ruby-grape / grape / 38960101986

100%

master: 100%

Pull #3033

github

Allocate less per request when presenting an entity or reading path params

Profiling the grape-on-rack app under live load showed four per-request
allocations whose only use was to feed another one:

- `present` collected the `env`/`version` embeds into a Hash and splatted
  it into `represent`, whose keywords build a Hash anyway: two Hashes per
  call where one does.
- `present` built `[nil, object]` only to destructure it in the
  one-argument form.
- `Request#make_params` copied the routing args with `except` only to
  `deep_merge!` the copy into the params. A capture is a String or an
  Array of them, never a Hash, so the merge only ever assigned; the
  captures are now assigned directly, skipping Grape's own keys.
- `format_from_extension` sliced the extension out with an endless
  Range, an object of its own on every call.

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

1504 of 1505 branches covered (99.93%)

Branch coverage included in aggregate %.

10 of 10 new or added lines in 2 files covered. (100.0%)

4410 of 4411 relevant lines covered (99.98%)

29525.2 hits per line