ruby-grape / grape / 25872754667

97%

push

github

Normalize == / eql? aliasing across value-like classes (#2715)

Several classes defined `==` without aliasing it to `eql?`, leaving
hash-table membership (`include?`, `Set`, `Hash` keys) inconsistent with
their custom equality. Normalize every value-like class to the same
shape: `def ==(other); ...; end; alias eql? ==`.

- middleware/stack.rb (Stack::Middleware): add `alias eql? ==`.
- serve_stream/stream_response.rb: same.
- serve_stream/file_body.rb: same.
- util/media_type.rb: collapse the `def ==(other); eql?(other); end`
  trampoline + separate `def eql?` into a single `==` body with
  `alias eql? ==`. Same behaviour, less indirection.
- exceptions/error_response.rb: same collapse.
- namespace.rb: flip from `def eql? ... alias == eql?` to
  `def == ... alias eql? ==` so the project follows one direction
  consistently.

`endpoint.rb` already used the canonical pattern; left untouched.

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

1073 of 1155 branches covered (92.9%)

Branch coverage included in aggregate %.

30 of 30 new or added lines in 7 files covered. (100.0%)

3470 of 3526 relevant lines covered (98.41%)

32019.4 hits per line