ruby-grape / grape / 26785176904 / 3

97%

master: 97%

Pull #2757

github

Build the cookie jar only on use and drop its deferral Proc

Two related steps that make the response-cookie path lazy at the request
boundary:

- `build_response_cookies` runs on every request and went through
  `cookies` -> `request.cookies`, materializing the `Grape::Cookies` jar
  even when the handler never touched a cookie. Gate it on a new
  `Grape::Request#cookies?` predicate (true only once the jar exists),
  so a cookie-free request allocates no jar at all.

- With the jar now built only when a cookie is actually read or written,
  its `-> { rack_cookies }` deferral Proc no longer earns its keep: any
  access immediately forces the parse, and a write (`[]=`) always did.
  Parse `rack_cookies` eagerly in the constructor and replace the
  `is_a?(Proc)` lazy reader with a plain `attr_reader`, dropping the
  closure allocation and the per-access branch.

The `ActiveSupport::HashWithIndifferentAccess` wrapping is unchanged --
it backs Grape's string/symbol-indifferent cookie access.

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

1088 of 1203 branches covered (90.44%)

Branch coverage included in aggregate %.

3504 of 3600 relevant lines covered (97.33%)

968.45 hits per line