belt 0.3.40 → 0.3.41

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 848174ed726800ea7e2a60b16a43714e558bf8adae32017a195a4ef0982cff11
-  data.tar.gz: 3d71874ef9d72422b72059e768de032b07caeb9b719bfaaf307d5d4d634b48ee
+  metadata.gz: 1a7141bc9a4eff54bdc3b4c010d47f08c3917aed5072dfa2f976f83429c7c00e
+  data.tar.gz: 367789f59346408d662c87a955d22f95b7405d90f9711e819e216c58c155db17
 SHA512:
-  metadata.gz: f431f7d645a3e94cdbf799f0d6fdd44a817491fd991da85b3bce7df95e760359638b1eeb528b39f3dc33edc51a324f4d0a4627152e0e4c025749020b3549ab91
-  data.tar.gz: 59fc7c275194d8ac49c4f69a0eb0109dcdf516eb1f6f597d5c7a9a5f0e16046d2d0dd6c88920c94c58735e643a0e231b3ce331fa919a7bc1d240c91ecb710c2f
+  metadata.gz: a5c9427efe1511e4f429ff46a7f14043a17b06c490915f62c535471ac984abcccda54801f765bf24e2b39e8a8f83b8939af53c69ad1414ae677cdf15d1943dd8
+  data.tar.gz: 4e90da0c8d2391bd3f7170acffeb083d04c3585092df86f413d4f3d2663386c21bac8ca6e833aecdea6883882569360d3b02e4239a17f52cf457fe65ed31b5d0

data/CHANGELOG.md

@@ -1,5 +1,34 @@
 # Changelog
 
+## 0.3.41
+
+### Bug Fix
+
+- **Fix API Gateway sibling path parameter conflict**: When a resource has nested
+  resources (a block), member routes (show, update, destroy) now use `{param_name}`
+  (e.g., `{project_id}`) instead of `{id}` to match the nested routes. This prevents
+  API Gateway from rejecting routes due to sibling path segments having different
+  parameter names.
+
+  Without nested resources (no block):
+  ```
+  GET  /posts/{id}
+  PUT  /posts/{id}
+  DELETE /posts/{id}
+  ```
+
+  With nested resources:
+  ```
+  GET  /projects/{project_id}
+  PUT  /projects/{project_id}
+  DELETE /projects/{project_id}
+  GET  /projects/{project_id}/epics
+  GET  /projects/{project_id}/epics/{id}
+  ```
+
+  This fixes the error: "Unable to create resource at path '...': A sibling ({id})
+  of this resource already has a variable path part".
+
 ## 0.3.40
 
 ### Breaking Change

data/lib/belt/route_dsl.rb

@@ -416,9 +416,12 @@ module Belt
       resource_options = options.merge(route_type: :resources)
       actions = determine_actions(options)
 
-      add_resource_routes(resource_name, param_name, resource_options, actions)
+      # When there are nested resources, use {param_name} for member routes to avoid
+      # API Gateway sibling path parameter conflicts. Without nested resources, use {id}.
+      has_nested = block_given?
+      add_resource_routes(resource_name, param_name, resource_options, actions, use_param_name: has_nested)
 
-      return unless block_given?
+      return unless has_nested
 
       collection_prefix = "/#{resource_name}"
       member_prefix = "/#{resource_name}/{#{param_name}}"
@@ -497,8 +500,11 @@ module Belt
       options.merge(tables: [resource_name.to_sym])
     end
 
-    def add_resource_routes(resource_name, _param_name, resource_options, actions)
-      # Member routes (show/update/destroy) use {id}, not {singular_id} — Rails convention
+    def add_resource_routes(resource_name, param_name, resource_options, actions, use_param_name: false)
+      # Member routes (show/update/destroy) use {id} by default (Rails convention).
+      # When nested resources exist (use_param_name: true), use {param_name} instead
+      # to avoid API Gateway sibling path parameter conflicts.
+      member_param = use_param_name ? param_name : 'id'
       if actions.include?(:index)
         add_route(:get, "/#{resource_name}",
                   resolve_request_model_for(resource_options, :index))
@@ -508,16 +514,16 @@ module Belt
                   resolve_request_model_for(resource_options, :create))
       end
       if actions.include?(:show)
-        add_route(:get, "/#{resource_name}/{id}",
+        add_route(:get, "/#{resource_name}/{#{member_param}}",
                   resolve_request_model_for(resource_options, :show))
       end
       if actions.include?(:update)
-        add_route(:put, "/#{resource_name}/{id}",
+        add_route(:put, "/#{resource_name}/{#{member_param}}",
                   resolve_request_model_for(resource_options, :update))
       end
       return unless actions.include?(:destroy)
 
-      add_route(:delete, "/#{resource_name}/{id}", resolve_request_model_for(resource_options, :destroy))
+      add_route(:delete, "/#{resource_name}/{#{member_param}}", resolve_request_model_for(resource_options, :destroy))
     end
 
     def resolve_request_model_for(options, action)
@@ -799,8 +805,11 @@ module Belt
         end
       end
 
-      def add_scoped_resource_routes(resource_name, _param_name, resource_options, actions)
-        # Member routes (show/update/destroy) use {id}, not {singular_id} — Rails convention
+      def add_scoped_resource_routes(resource_name, param_name, resource_options, actions, use_param_name: false)
+        # Member routes (show/update/destroy) use {id} by default (Rails convention).
+        # When nested resources exist (use_param_name: true), use {param_name} instead
+        # to avoid API Gateway sibling path parameter conflicts.
+        member_param = use_param_name ? param_name : 'id'
         if actions.include?(:index)
           @gateway.send(:add_route, :get, build_path("/#{resource_name}"),
                         resolve_request_model_for(resource_options, :index))
@@ -810,16 +819,16 @@ module Belt
                         resolve_request_model_for(resource_options, :create))
         end
         if actions.include?(:show)
-          @gateway.send(:add_route, :get, build_path("/#{resource_name}/{id}"),
+          @gateway.send(:add_route, :get, build_path("/#{resource_name}/{#{member_param}}"),
                         resolve_request_model_for(resource_options, :show))
         end
         if actions.include?(:update)
-          @gateway.send(:add_route, :put, build_path("/#{resource_name}/{id}"),
+          @gateway.send(:add_route, :put, build_path("/#{resource_name}/{#{member_param}}"),
                         resolve_request_model_for(resource_options, :update))
         end
         return unless actions.include?(:destroy)
 
-        @gateway.send(:add_route, :delete, build_path("/#{resource_name}/{id}"),
+        @gateway.send(:add_route, :delete, build_path("/#{resource_name}/{#{member_param}}"),
                       resolve_request_model_for(resource_options, :destroy))
       end
 
@@ -842,7 +851,10 @@ module Belt
         resource_options = options.merge(route_type: :resources, controller: controller)
         actions = determine_scoped_actions(options)
 
-        add_scoped_resource_routes(resource_name, param_name, resource_options, actions)
+        # When there are nested resources, use {param_name} for member routes to avoid
+        # API Gateway sibling path parameter conflicts. Without nested resources, use {id}.
+        has_nested = !block.nil?
+        add_scoped_resource_routes(resource_name, param_name, resource_options, actions, use_param_name: has_nested)
         build_nested_resource_block(resource_name, param_name, options, controller, &block) if block
       end
 

data/lib/belt/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Belt
-  VERSION = '0.3.40'
+  VERSION = '0.3.41'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: belt
 version: !ruby/object:Gem::Version
-  version: 0.3.40
+  version: 0.3.41
 platform: ruby
 authors:
 - Stowzilla