ruby-grape / grape / 39052227118

100%

master: 100%

Pull #3037

github

Hold every method to RuboCop's default perceived complexity

Metrics/PerceivedComplexity was raised to 15. Router#transaction is the
only method above the default of 8 (9), and the override is dropped.

The static route table moves into the tuple @optimized_map already holds
per method, next to the union and the buckets. #transaction then reads
everything a request can be matched with through one lookup, and its
three ways of matching -- the static table, a bucket, the union -- read
as one if/elsif/else instead of an if/else nested in another. A table is
still left out (nil) for a method whose routes spell out no path in
full, so such a request skips straight to the union. A request for a
dynamic route now makes one Hash lookup where it made two.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

1501 of 1503 branches covered (99.87%)

Branch coverage included in aggregate %.

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

4462 of 4463 relevant lines covered (99.98%)

29385.63 hits per line