ruby-grape / grape / 26493796179

97%

push

github

Make rescue_from reject meta selectors mixed with exception classes (#2737)

Previously the `*args` signature meant that:

  rescue_from :all, MyError, with: :handler

silently dropped `MyError`: the `if args.include?(:all)` branch fired
and the trailing argument was never registered. Same for
`:grape_exceptions` and `:internal_grape_exceptions`.

Keep the existing `*args` signature, add a single guard above the
dispatch: if any meta selector is present and `args.size > 1`, raise
`ArgumentError`. Valid forms (`rescue_from :all`,
`rescue_from :all, with:`, `rescue_from MyError, OtherError, with:`)
are unaffected.

CHANGELOG + UPGRADING entries added; three new request_response_spec
examples cover the new ArgumentError contract.

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

1089 of 1175 branches covered (92.68%)

Branch coverage included in aggregate %.

17 of 17 new or added lines in 1 file covered. (100.0%)

3549 of 3608 relevant lines covered (98.36%)

31616.67 hits per line