belt 0.3.42 → 0.3.43

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a46f69166308a770107e86d0ccee9499a62e60858ebf000c74990183670eacfc
-  data.tar.gz: 988f5e06c5b6f03fb42be923ea9864e9a9bdbf2f12bc4d8bd963c82066eaa9fe
+  metadata.gz: 35d8b66e560a7eedc35c0f801e54e80c5322c0aa81bbba2e84945c679ad37df8
+  data.tar.gz: 77cac61f6823651528069f69ba79e715acbe9b01054c101cdc51f7dbd5c0227c
 SHA512:
-  metadata.gz: 6f614e8b43dfbe1a6431b0ae8f8a04f3bd92a33f305afd4c75d5a84252a45714b9462fbbbdc63d6978a873da674da053581f7a7afb521f011a06badcd6bc5913
-  data.tar.gz: 766f648f1bc4b795698f605f729682ab44baf7e27fa66dd8ff73e6c618cd1bba21cd8ae15c75cdff9d8c1b6e9a98b84f5be4b1bd828d7e16c0efcc0071ef9a77
+  metadata.gz: 17067c3c5dca6d23f3a7c44576a2653fcb8d281d95951d0ac0faa3deececf9158a6cfeedd08ca2419b196bed86ce75989d0537f29bcdc8f4b288e284505c05d7
+  data.tar.gz: 7b4dad5df465c3f7ad7cf42bbd6dc803bfe5ef6443cfb92be9918c04a33ca5cb68c5b01082777933fd559cda4d7d9eb4e33dae07a22c1b1b1bf2d62cfa43a7ad

data/CHANGELOG.md

@@ -1,5 +1,38 @@
 # Changelog
 
+## 0.3.43
+
+### Bug Fix
+
+- **Fix nested resource member parameter conflicts**: Extends the fix from 0.3.41
+  to also cover deeply nested resources and custom member actions. Previously,
+  resources nested inside other nested resources (e.g., `epics` inside `projects`
+  with a block) still used `{id}` for member routes while custom member actions
+  (like `post :test, on: :member`) used `{param_name}`.
+
+  This fixes the error for routes like:
+  ```
+  /projects/{project_id}/webhooks/{id}           # show/update/destroy
+  /projects/{project_id}/webhooks/{webhook_id}/test  # custom member action
+  ```
+
+  Now both use consistent parameter names when the resource has a block:
+  ```
+  /projects/{project_id}/webhooks/{webhook_id}
+  /projects/{project_id}/webhooks/{webhook_id}/test
+  ```
+
+## 0.3.42
+
+### New Features
+
+- **Auto-sync apex DNS records**: When deploying to a production/apex environment,
+  `belt deploy` now automatically syncs A alias records (apex, www, api) to the
+  root zone in your shared DNS account. No more manual DNS steps for prod deploys.
+
+- **`belt dns doctor`**: New diagnostic command showing DNS health across all
+  environments — zone status, NS delegation, ACM certificates, and resolution tests.
+
 ## 0.3.41
 
 ### Bug Fix

data/lib/belt/route_dsl.rb

@@ -75,8 +75,11 @@ module Belt
       resource_options = options.merge(route_type: :resources)
       actions = @gateway.send(:determine_actions, options)
 
-      add_nested_resource_routes(resource_name, param_name, resource_options, actions)
-      return unless block
+      # When there's a block (nested resources or custom member actions), use {param_name}
+      # for member routes to avoid API Gateway sibling path parameter conflicts.
+      has_nested = !block.nil?
+      add_nested_resource_routes(resource_name, param_name, resource_options, actions, use_param_name: has_nested)
+      return unless has_nested
 
       nested_member_prefix = "#{@prefix}/#{resource_name}/{#{param_name}}"
       nested_collection_prefix = "#{@prefix}/#{resource_name}"
@@ -251,8 +254,11 @@ module Belt
       end
     end
 
-    def add_nested_resource_routes(resource_name, _param_name, resource_options, actions)
-      # Member routes (show/update/destroy) use {id}, not {singular_id} — Rails convention
+    def add_nested_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 or custom member actions 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, "#{@prefix}/#{resource_name}",
                       resolve_request_model_for(resource_options, :index))
@@ -262,16 +268,16 @@ module Belt
                       resolve_request_model_for(resource_options, :create))
       end
       if actions.include?(:show)
-        @gateway.send(:add_route, :get, "#{@prefix}/#{resource_name}/{id}",
+        @gateway.send(:add_route, :get, "#{@prefix}/#{resource_name}/{#{member_param}}",
                       resolve_request_model_for(resource_options, :show))
       end
       if actions.include?(:update)
-        @gateway.send(:add_route, :put, "#{@prefix}/#{resource_name}/{id}",
+        @gateway.send(:add_route, :put, "#{@prefix}/#{resource_name}/{#{member_param}}",
                       resolve_request_model_for(resource_options, :update))
       end
       return unless actions.include?(:destroy)
 
-      @gateway.send(:add_route, :delete, "#{@prefix}/#{resource_name}/{id}",
+      @gateway.send(:add_route, :delete, "#{@prefix}/#{resource_name}/{#{member_param}}",
                     resolve_request_model_for(resource_options, :destroy))
     end
 

data/lib/belt/version.rb

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

metadata

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