ruby-grape / grape / 26758489730

97%

master: 97%

Build Type

Pull #2755

github

Committed by ericproulx

Commit Message

Inline mustermann-grape into Grape::Router::MustermannPattern

mustermann-grape is a ~40-line Mustermann::AST::Pattern subclass that exists
only for Grape's path syntax, and it had already grown Grape-specific: the
Integer constraint reads Grape's own `params` option. Maintaining it as a
separate gem meant a two-step release for every syntax tweak (cut a
mustermann-grape release, then bump Grape's pin) plus its own CI/versioning.

Move the grammar into Grape as Grape::Router::MustermannPattern (attribution to
the original authors preserved), depend on `mustermann` directly, and
instantiate it directly from Grape::Router::Pattern as Grape always did. The
`mustermann` core dependency is unchanged; the graph collapses from
grape -> mustermann-grape -> mustermann to grape -> mustermann.

The class is no longer registered as a Mustermann `type: :grape` (Grape never
used the registry). Apps that called `Mustermann.new(_, type: :grape)` relying
on Grape pulling in mustermann-grape transitively should depend on it directly;
see UPGRADING.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Pull Request Pull Request #2755: Inline mustermann-grape into Grape::Router::MustermannPattern

Coverage Stats

1110 of 1207 branches covered (91.96%)

Branch coverage included in aggregate %.

20 of 25 new or added lines in 3 files covered. (80.0%)

2 existing lines in 2 files now uncovered.

3547 of 3612 relevant lines covered (98.2%)

23178.04 hits per line