belt 0.3.42 → 0.3.44

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a46f69166308a770107e86d0ccee9499a62e60858ebf000c74990183670eacfc
-  data.tar.gz: 988f5e06c5b6f03fb42be923ea9864e9a9bdbf2f12bc4d8bd963c82066eaa9fe
+  metadata.gz: 7977614906b8f7154ca9e00597f1eb5744bc00ac6814985cfda98a16687d5992
+  data.tar.gz: d60ca2ff84dec2ce5d8b6ee97be3f9ef189fc455881e30b4877a37ae3dafef36
 SHA512:
-  metadata.gz: 6f614e8b43dfbe1a6431b0ae8f8a04f3bd92a33f305afd4c75d5a84252a45714b9462fbbbdc63d6978a873da674da053581f7a7afb521f011a06badcd6bc5913
-  data.tar.gz: 766f648f1bc4b795698f605f729682ab44baf7e27fa66dd8ff73e6c618cd1bba21cd8ae15c75cdff9d8c1b6e9a98b84f5be4b1bd828d7e16c0efcc0071ef9a77
+  metadata.gz: b8cf0deed02541a8775b7f4000ba8f5336d09572d6752520bd5191407fd40b97412d5d8c2ee19790b6e8445231be9ffedbd09688c0233b642a96d2ba4494a38c
+  data.tar.gz: 025eb3373be5e8c345b29525fd6f81c2e21581b2b044d66dab03482b838e7e491556e6669b099cd128c0b5e5965b29a1ea741761c741b8dee5c489c7f587c34d

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/cli/environment_config.rb

@@ -41,11 +41,23 @@ module Belt
 
       # Apply the configured aws_profile and env vars to the current process.
       # Call this before running terraform, aws cli, etc.
+      #
+      # Note: If AWS credentials are already available via environment variables
+      # (AWS_ACCESS_KEY_ID, AWS_SECRET_ACCESS_KEY, AWS_SESSION_TOKEN) — as in GitHub
+      # Actions OIDC — we skip setting AWS_PROFILE to avoid overriding valid credentials.
       def apply!
-        ENV['AWS_PROFILE'] = @aws_profile if aws_profile?
+        ENV['AWS_PROFILE'] = @aws_profile if aws_profile? && !credentials_in_env?
         @env_vars.each { |key, value| ENV[key] = value }
       end
 
+      # Returns true if AWS credentials are available via environment variables.
+      # This happens in CI environments (GitHub Actions OIDC, CodeBuild, etc.)
+      def credentials_in_env?
+        key = ENV.fetch('AWS_ACCESS_KEY_ID', nil)
+        secret = ENV.fetch('AWS_SECRET_ACCESS_KEY', nil)
+        !key.to_s.empty? && !secret.to_s.empty?
+      end
+
       class ConfigEvaluator
         def initialize(config)
           @config = config

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.44'
 end

metadata

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