add comments

Author: gregschmit

lib/rest_framework/routers.rb

@@ -109,6 +109,8 @@ def _rest_resources(default_singular, name, **kwargs, &block)
             next unless controller_class.method_defined?(action)
 
             [ methods ].flatten.each do |m|
+              # Anchor the route since Rails 8.1 OPTIONS routes are non-anchored by default, which
+              # causes parent OPTIONS routes to greedily intercept sub-path requests.
               public_send(m, "", action: action, anchor: true) if self.respond_to?(m)
             end
           end
@@ -122,6 +124,8 @@ def _rest_resources(default_singular, name, **kwargs, &block)
               next if bulk_exclude.include?(action)
 
               [ methods ].flatten.each do |m|
+                # Anchor the route since Rails 8.1 OPTIONS routes are non-anchored by default, which
+                # causes parent OPTIONS routes to greedily intercept sub-path requests.
                 public_send(m, "", action: action, anchor: true) if self.respond_to?(m)
               end
             end
@@ -184,6 +188,8 @@ def rest_route(name = nil, **kwargs, &block)
             next unless controller_class.method_defined?(action)
 
             [ methods ].flatten.each do |m|
+              # Anchor the route since Rails 8.1 OPTIONS routes are non-anchored by default, which
+              # causes parent OPTIONS routes to greedily intercept sub-path requests.
               public_send(m, "", action: action, anchor: true) if self.respond_to?(m)
             end
           end