ruby-grape / grape / 26721499467 / 17

97%

master: 97%

Pull #2752

github

Skip per-request instrumentation work when no subscribers are listening

Grape fires up to six ActiveSupport::Notifications.instrument calls per
request (endpoint run / render / run_validators / run_filters and
format_response). Each builds a payload Hash and enters the notification
machinery on every request, even when nothing is subscribed to the event.

Guard each emission behind ActiveSupport::Notifications.notifier.listening?
via a small private instrument_<event> method on the owning class. With no
subscriber the block runs directly, skipping the payload Hash and the
instrument dispatch; the block is forwarded anonymously so nothing extra is
allocated unless a subscriber is present. Behaviour is unchanged when a
subscriber exists - same events, same payloads.

Saves one Hash allocation per emitted event (-6 allocations/request on an
endpoint with before/after filters and params), ~+4.6% median throughput
under YJIT on that endpoint; scales with events emitted per request.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

1088 of 1201 branches covered (90.59%)

Branch coverage included in aggregate %.

3504 of 3600 relevant lines covered (97.33%)

968.42 hits per line