ruby-grape / grape / 25859964948

97%

master: 97%

Pull #2715

github

Normalize == / eql? aliasing across value-like classes

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>

1075 of 1157 branches covered (92.91%)

Branch coverage included in aggregate %.

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

3468 of 3524 relevant lines covered (98.41%)

32133.12 hits per line