stack_master 2.17.0 → 2.17.1

data/lib/stack_master/stack.rb

@@ -17,10 +17,13 @@ module StackMaster
     include Utils::Initializable
 
     def template_default_parameters
-      TemplateUtils.template_hash(template).fetch('Parameters', {}).inject({}) do |result, (parameter_name, description)|
-        result[parameter_name] = description['Default']
-        result
-      end
+      TemplateUtils
+        .template_hash(template)
+        .fetch('Parameters', {})
+        .inject({}) do |result, (parameter_name, description)|
+          result[parameter_name] = description['Default']&.to_s
+          result
+        end
     end
 
     def parameters_with_defaults
@@ -31,6 +34,7 @@ module StackMaster
       cf = StackMaster.cloud_formation_driver
       cf_stack = cf.describe_stacks({ stack_name: stack_name }).stacks.first
       return unless cf_stack
+
       parameters = cf_stack.parameters.inject({}) do |params_hash, param_struct|
         params_hash[param_struct.parameter_key] = param_struct.parameter_value
         params_hash
@@ -56,14 +60,27 @@ module StackMaster
     end
 
     def self.generate(stack_definition, config)
-      parameter_hash = ParameterLoader.load(parameter_files: stack_definition.all_parameter_files, parameters: stack_definition.parameters)
+      parameter_hash = ParameterLoader.load(
+        parameter_files: stack_definition.all_parameter_files,
+        parameters: stack_definition.parameters
+      )
       template_parameters = ParameterResolver.resolve(config, stack_definition, parameter_hash[:template_parameters])
-      compile_time_parameters = ParameterResolver.resolve(config, stack_definition, parameter_hash[:compile_time_parameters])
-      template_body = TemplateCompiler.compile(config, stack_definition.compiler, stack_definition.template_dir, stack_definition.template, compile_time_parameters, stack_definition.compiler_options)
+      compile_time_parameters = ParameterResolver.resolve(
+        config,
+        stack_definition,
+        parameter_hash[:compile_time_parameters]
+      )
+      template_body = TemplateCompiler.compile(
+        config,
+        stack_definition.compiler,
+        stack_definition.template_dir,
+        stack_definition.template,
+        compile_time_parameters,
+        stack_definition.compiler_options
+      )
       template_format = TemplateUtils.identify_template_format(template_body)
-      stack_policy_body = if stack_definition.stack_policy_file_path
-                            File.read(stack_definition.stack_policy_file_path)
-                          end
+      stack_policy_body =
+        (File.read(stack_definition.stack_policy_file_path) if stack_definition.stack_policy_file_path)
       new(region: stack_definition.region,
           stack_name: stack_definition.stack_name,
           tags: stack_definition.tags,
@@ -76,13 +93,26 @@ module StackMaster
     end
 
     def self.generate_without_parameters(stack_definition, config)
-      parameter_hash = ParameterLoader.load(parameter_files: stack_definition.all_parameter_files, parameters: stack_definition.parameters)
-      compile_time_parameters = ParameterResolver.resolve(config, stack_definition, parameter_hash[:compile_time_parameters])
-      template_body = TemplateCompiler.compile(config, stack_definition.compiler, stack_definition.template_dir, stack_definition.template, compile_time_parameters, stack_definition.compiler_options)
+      parameter_hash = ParameterLoader.load(
+        parameter_files: stack_definition.all_parameter_files,
+        parameters: stack_definition.parameters
+      )
+      compile_time_parameters = ParameterResolver.resolve(
+        config,
+        stack_definition,
+        parameter_hash[:compile_time_parameters]
+      )
+      template_body = TemplateCompiler.compile(
+        config,
+        stack_definition.compiler,
+        stack_definition.template_dir,
+        stack_definition.template,
+        compile_time_parameters,
+        stack_definition.compiler_options
+      )
       template_format = TemplateUtils.identify_template_format(template_body)
-      stack_policy_body = if stack_definition.stack_policy_file_path
-                            File.read(stack_definition.stack_policy_file_path)
-                          end
+      stack_policy_body =
+        (File.read(stack_definition.stack_policy_file_path) if stack_definition.stack_policy_file_path)
       new(region: stack_definition.region,
           stack_name: stack_definition.stack_name,
           tags: stack_definition.tags,
@@ -96,6 +126,7 @@ module StackMaster
 
     def max_template_size(use_s3)
       return TemplateUtils::MAX_S3_TEMPLATE_SIZE if use_s3
+
       TemplateUtils::MAX_TEMPLATE_SIZE
     end
 

data/lib/stack_master/stack_definition.rb

@@ -65,6 +65,7 @@ module StackMaster
 
     def template_file_path
       return unless template
+
       File.expand_path(template, template_dir)
     end
 
@@ -85,6 +86,7 @@ module StackMaster
 
     def s3_template_file_name
       return template if ['.json', '.yaml', '.yml'].include?(File.extname(template))
+
       Utils.change_extension(template, 'json')
     end
 
@@ -101,7 +103,7 @@ module StackMaster
     end
 
     def parameter_file_globs
-      [ default_parameter_glob, region_parameter_glob ] + additional_parameter_lookup_globs
+      [default_parameter_glob, region_parameter_glob] + additional_parameter_lookup_globs
     end
 
     def stack_policy_file_path

data/lib/stack_master/stack_differ.rb

@@ -10,12 +10,14 @@ module StackMaster
 
     def proposed_template
       return @proposed_stack.template_body unless @proposed_stack.template_format == :json
+
       JSON.pretty_generate(JSON.parse(@proposed_stack.template_body)) + "\n"
     end
 
     def current_template
       return '' unless @current_stack
       return @current_stack.template_body unless @current_stack.template_format == :json
+
       JSON.pretty_generate(TemplateUtils.template_hash(@current_stack.template_body)) + "\n"
     end
 
@@ -81,8 +83,10 @@ module StackMaster
 
     def single_param_update?(param_name)
       return false if param_name.blank? || @current_stack.blank? || body_different?
+
       differences = Hashdiff.diff(@current_stack.parameters_with_defaults, @proposed_stack.parameters_with_defaults)
       return false if differences.count != 1
+
       diff = differences[0]
       diff[0] == "~" && diff[1] == param_name
     end

data/lib/stack_master/stack_events/fetcher.rb

@@ -31,7 +31,9 @@ module StackMaster
       end
 
       def fetch_events
-        PagedResponseAccumulator.call(cf, :describe_stack_events, { stack_name: @stack_name }, :stack_events).stack_events
+        PagedResponseAccumulator
+          .call(cf, :describe_stack_events, { stack_name: @stack_name }, :stack_events)
+          .stack_events
       end
     end
   end

data/lib/stack_master/stack_events/presenter.rb

@@ -10,7 +10,12 @@ module StackMaster
       end
 
       def print_event(event)
-        @io.puts Rainbow("#{event.timestamp.localtime} #{event.logical_resource_id} #{event.resource_type} #{event.resource_status} #{event.resource_status_reason}").color(event_colour(event))
+        @io.puts(
+          Rainbow(
+            "#{event.timestamp.localtime} #{event.logical_resource_id} #{event.resource_type} " \
+            "#{event.resource_status} #{event.resource_status_reason}"
+          ).color(event_colour(event))
+        )
       end
 
       def event_colour(event)

data/lib/stack_master/stack_events/streamer.rb

@@ -7,7 +7,8 @@ module StackMaster
         new(stack_name, region, **args, &block).stream
       end
 
-      def initialize(stack_name, region, from: Time.now, break_on_finish_state: true, sleep_between_fetches: 1, io: nil, &block)
+      def initialize(stack_name, region, from: Time.now, break_on_finish_state: true, sleep_between_fetches: 1,
+                     io: nil, &block)
         @stack_name = stack_name
         @region = region
         @block = block
@@ -42,6 +43,7 @@ module StackMaster
         [].tap do |unseen_events|
           events.each do |event|
             next if @seen_events.include?(event.event_id)
+
             @seen_events << event.event_id
             unseen_events << event
           end

data/lib/stack_master/stack_status.rb

@@ -39,6 +39,7 @@ module StackMaster
 
     def stack
       return @stack if defined?(@stack)
+
       StackMaster.cloud_formation_driver.set_region(stack_definition.region)
       @stack = find_stack
     end

data/lib/stack_master/template_compilers/sparkle_formation.rb

@@ -4,7 +4,6 @@ require 'stack_master/sparkle_formation/compile_time/state_builder'
 
 module StackMaster::TemplateCompilers
   class SparkleFormation
-
     CompileTime = StackMaster::SparkleFormation::CompileTime
 
     def self.require_dependencies
@@ -28,11 +27,12 @@ module StackMaster::TemplateCompilers
     private
 
     def self.compile_sparkle_template(template_dir, template, compiler_options)
-      sparkle_path = if compiler_options['sparkle_path']
-        File.expand_path(compiler_options['sparkle_path'])
-      else
-        template_dir
-      end
+      sparkle_path =
+        if compiler_options['sparkle_path']
+          File.expand_path(compiler_options['sparkle_path'])
+        else
+          template_dir
+        end
 
       collection = ::SparkleFormation::SparkleCollection.new
       root_pack = ::SparkleFormation::Sparkle.new(
@@ -48,7 +48,10 @@ module StackMaster::TemplateCompilers
       end
 
       if compiler_options['sparkle_pack_template']
-        raise ArgumentError.new("Template #{template.inspect} not found in any sparkle pack") unless collection.templates['aws'].include? template
+        unless collection.templates['aws'].include? template
+          raise ArgumentError.new("Template #{template.inspect} not found in any sparkle pack")
+        end
+
         template_file_path = collection.templates['aws'][template].top['path']
       else
         template_file_path = File.join(template_dir, template)

data/lib/stack_master/template_utils.rb

@@ -16,8 +16,9 @@ module StackMaster
       end
     end
 
-    def template_hash(template_body=nil)
+    def template_hash(template_body = nil)
       return unless template_body
+
       template_format = identify_template_format(template_body)
       case template_format
       when :json
@@ -30,7 +31,10 @@ module StackMaster
     def maybe_compressed_template_body(template_body)
       # Do not compress the template if it's not JSON because parsing YAML as a hash ignores
       # CloudFormation-specific tags such as !Ref
-      return template_body if template_body.size <= MAX_TEMPLATE_SIZE || identify_template_format(template_body) != :json
+      if template_body.size <= MAX_TEMPLATE_SIZE || identify_template_format(template_body) != :json
+        return template_body
+      end
+
       JSON.dump(template_hash(template_body))
     end
   end

data/lib/stack_master/test_driver/cloud_formation.rb

@@ -99,11 +99,27 @@ module StackMaster
         change_set_id = options.fetch(:change_set_name)
         change_set = @change_sets.fetch(change_set_id)
         change_details = [
-          OpenStruct.new(evaluation: 'Static', change_source: 'ResourceReference', target: OpenStruct.new(attribute: 'Properties', requires_recreation: 'Always', name: 'blah'))
+          OpenStruct.new(
+            evaluation: 'Static',
+            change_source: 'ResourceReference',
+            target: OpenStruct.new(
+              attribute: 'Properties',
+              requires_recreation: 'Always',
+              name: 'blah'
+            )
+          )
         ]
-        change = OpenStruct.new(action: 'Modify', replacement: 'True', scope: ['Properties'], details: change_details)
+        change = OpenStruct.new(
+          action: 'Modify',
+          replacement: 'True',
+          scope: ['Properties'],
+          details: change_details
+        )
         changes = [
-          OpenStruct.new(type: 'AWS::Resource', resource_change: change)
+          OpenStruct.new(
+            type: 'AWS::Resource',
+            resource_change: change
+          )
         ]
         OpenStruct.new(change_set.merge(changes: changes, status: 'CREATE_COMPLETE'))
       end
@@ -121,15 +137,16 @@ module StackMaster
 
       def describe_stacks(options = {})
         stack_name = options[:stack_name]
-        stacks = if stack_name
-          if @stacks[stack_name]
-            [@stacks[stack_name]]
+        stacks =
+          if stack_name
+            if @stacks[stack_name]
+              [@stacks[stack_name]]
+            else
+              raise Aws::CloudFormation::Errors::ValidationError.new('', 'Stack does not exist')
+            end
           else
-            raise Aws::CloudFormation::Errors::ValidationError.new('', 'Stack does not exist')
+            @stacks.values
           end
-        else
-          @stacks.values
-        end
         OpenStruct.new(stacks: stacks, next_token: nil)
       end
 

data/lib/stack_master/utils.rb

@@ -30,6 +30,7 @@ module StackMaster
 
     def hash_to_aws_tags(tags)
       return [] if tags.nil?
+
       tags.inject([]) do |aws_tags, (key, value)|
         aws_tags << { key: key, value: value }
         aws_tags

data/lib/stack_master/validator.rb

@@ -35,11 +35,12 @@ module StackMaster
     end
 
     def stack
-      @stack ||= if validate_template_parameters?
-        Stack.generate(@stack_definition, @config)
-      else
-        Stack.generate_without_parameters(@stack_definition, @config)
-      end
+      @stack ||=
+        if validate_template_parameters?
+          Stack.generate(@stack_definition, @config)
+        else
+          Stack.generate_without_parameters(@stack_definition, @config)
+        end
     end
 
     def parameter_validator

data/lib/stack_master/version.rb

@@ -1,3 +1,3 @@
 module StackMaster
-  VERSION = "2.17.0"
+  VERSION = "2.17.1"
 end

data/lib/stack_master.rb

@@ -140,6 +140,7 @@ module StackMaster
 
   def debug(message)
     return unless debug?
+
     stderr.puts Rainbow("[DEBUG] #{message}").color(:green)
   end
 
@@ -167,6 +168,7 @@ module StackMaster
   end
 
   attr_accessor :non_interactive_answer
+
   @non_interactive_answer = 'y'
 
   def base_dir

metadata

@@ -1,15 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: stack_master
 version: !ruby/object:Gem::Version
-  version: 2.17.0
+  version: 2.17.1
 platform: ruby
 authors:
 - Steve Hodgkiss
 - Glen Stampoultzis
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-07-11 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -123,6 +122,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: os
   requirement: !ruby/object:Gem::Requirement
@@ -542,7 +555,6 @@ files:
 - lib/stack_master/paged_response_accumulator.rb
 - lib/stack_master/parameter_loader.rb
 - lib/stack_master/parameter_resolver.rb
-- lib/stack_master/parameter_resolvers/accounts_by_tags.rb
 - lib/stack_master/parameter_resolvers/acm_certificate.rb
 - lib/stack_master/parameter_resolvers/ami_finder.rb
 - lib/stack_master/parameter_resolvers/ejson.rb
@@ -610,9 +622,8 @@ licenses:
 metadata:
   bug_tracker_uri: https://github.com/envato/stack_master/issues
   changelog_uri: https://github.com/envato/stack_master/blob/master/CHANGELOG.md
-  documentation_uri: https://www.rubydoc.info/gems/stack_master/2.17.0
-  source_code_uri: https://github.com/envato/stack_master/tree/v2.17.0
-post_install_message:
+  documentation_uri: https://www.rubydoc.info/gems/stack_master/2.17.1
+  source_code_uri: https://github.com/envato/stack_master/tree/v2.17.1
 rdoc_options: []
 require_paths:
 - lib
@@ -627,8 +638,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3.1
-signing_key:
+rubygems_version: 4.0.2
 specification_version: 4
 summary: StackMaster is a sure-footed way of creating, updating and keeping track
   of Amazon (AWS) CloudFormation stacks.

data/lib/stack_master/parameter_resolvers/accounts_by_tags.rb

@@ -1,60 +0,0 @@
-module StackMaster
-  module ParameterResolvers
-    class AccountsByTags < Resolver
-      array_resolver class_name: 'AccountsByTags'
-
-      def initialize(config, stack_definition)
-        @config = config
-        @stack_definition = stack_definition
-      end
-
-      def resolve(team)
-        
-      end
-
-      private
-
-      def fetch_all_aws_accounts()
-        return @aws_accounts if defined(@aws_accounts)
-        @aws_accounts = []
-        next_token = nil
-        client = Aws::Organizations::Client.new(region: @stack_definition.region)
-        loop do
-          resp = client.list_accounts(next_token: next_token)
-          @aws_accounts << resp.accounts
-          next_token = resp.next_token
-          break if next_token.nil?
-        end
-        @aws_accounts.flatten! || @aws_accounts  
-      end
-
-      def_fetch_tags_for_account(account)
-        client = Aws::Organizations::Client.new(region: @stack_definition.region)
-        next_token = nil
-        tags = [] 
-        loop do
-          client.list_tags_for_resource(resource_id: account.id, next_token: next_token)
-          tags << resp.tags
-          next_token = resp.next_token
-          break if next_token.nil?
-        end
-        tags.flatten! || tags
-      end
-
-      def fetch_tags_for_accounts()
-        return @tagged_aws_accounts if defined?(@tagged_aws_accounts )
-        @tagged_aws_accounts = Hash.new()
-        fetch_all_aws_accounts()
-        @aws_accounts.each do |account|
-          @tagged_aws_accounts[account.id.to_sym] = { :account => account.id, :tags => fetch_tags_for_account(account.id) }
-        end
-      end
-
-      def account_in_group?(account, group)
-        fetch_tags_for_accounts if defined?(@tagged_aws_accounts)
-        @tagged_aws_accounts[:account.to_sym][:tags].select {|tag| tag['key'] =='Group' && tag['value']==group}
-      end
-
-    end
-  end
-end