ruby-grape / grape / 26494895924

97%

push

github

Prune unused requires in lib/grape.rb (#2742)

Three removals (verified via grep across `lib/` + full RSpec run):

- `active_support/version` — redundant; `require 'active_support'`
  itself loads the version file first.
- `pathname` — no `Pathname` reference anywhere in the gem.

One narrow:

- `active_support/inflector` → `active_support/core_ext/string/inflections`
  — only the String inflector extensions (`demodulize`, `underscore`) are
  used (in `validations.rb` and `util/registry.rb`); the rest of the
  Inflector class/rule machinery isn't reached.

Kept with a comment after a false start during the audit:

- `active_support/core_ext/hash/indifferent_access` — looked unused
  (we only construct `ActiveSupport::HashWithIndifferentAccess.new(...)`
  directly, never call `Hash#with_indifferent_access`), but
  `HashWithIndifferentAccess.new(hash)` internally calls
  `nested_under_indifferent_access` on its input, which is added by
  this core_ext. Dropping it broke ~200 specs.

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

1097 of 1183 branches covered (92.73%)

Branch coverage included in aggregate %.

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

3535 of 3592 relevant lines covered (98.41%)

31279.68 hits per line