Move HEAD route creation into Grape::Router::Route#to_head

[ericproulx]

Endpoint#mount_in built the HEAD variant of a GET route inline:

route.dup.then do |head_route|
  head_route.convert_to_head_request!
  router.append(head_route.apply(self))
end

Duplicating a route and turning it into a HEAD request is the route's own concern, not the endpoint's.

Change

• Add Grape::Router::Route#to_head, which returns a fresh HEAD copy of the route.
• Demote convert_to_head_request! to protected — to_head is now its only caller, and it calls it on the dup with an explicit receiver (which protected permits).
• mount_in's loop body becomes router.append(route.to_head.apply(self)).

mount_in keeps the decision of whether to add a HEAD route (the do_not_route_head setting and the request_method != Rack::GET check) — that's the endpoint/router's call. to_head just builds the variant. The seam is: the endpoint decides, the route constructs.

Behavior is unchanged — HEAD on a GET route still 200s, do_not_route_head! still 405s on HEAD.

🤖 Generated with Claude Code

[github-actions]

Danger Report

No issues found.

View run