firespring_dev_commands 2.1.3 → 2.1.4.pre.alpha.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 44213627e63b15337ede6416bf7dc17e6f70188d71ca2f2a19b601e61b21ae0e
-  data.tar.gz: 356826da99f48dcaa041afe7afa642fdb18f72b38536b2a46753fd0a336508fa
+  metadata.gz: 782f199b829c9205d5d6539b2e6be9d5daed015cf00e4dc4dd8743c53925b967
+  data.tar.gz: 960a71a68186fa19525cb8935ece0deb6ee398c0400913eff2a6cf39b9806635
 SHA512:
-  metadata.gz: 60b6e69d0bd7625fd033e528322d8d61512abdc3d1cd0637d295eeba000a65e8d55808a89c44c093fb09d136c6cba62a9641db4d30e54e0e248197a25693ca5e
-  data.tar.gz: 0b3f4c05a9d1900179e2bd8cc7322390243a990cf04ca48963508af32dc54e6f469dc4bb151af2ce4968a371fc2e387e554db80f4f23a79dc9deeff2ea78fafc
+  metadata.gz: a662422f7c487bce2824f657c86099508af9185b71bc95168cccd8ba854282d8a0d981c6a1f6adf0b570cfa7ae4b46c05d7f9276ba3fe201c8bee175529badd4
+  data.tar.gz: 0e4872ea27d593ca5e87ff92c044ddcf7e5d284f18d2fe5644ad9d7e380da95468202ee3f4697443464cef1e113587219daaf82ba287e95e18b7c578160c5de9

data/lib/firespring_dev_commands/jira/histories.rb

@@ -0,0 +1,13 @@
+require 'pry'
+
+module Dev
+  class Jira
+    class Histories
+      def self.populate(data)
+        return nil unless data.attrs.key?('changelog')
+
+        data.changelog['histories'].map { |it| Jira::History.new(it) }
+      end
+    end
+  end
+end

data/lib/firespring_dev_commands/jira/history.rb

@@ -0,0 +1,17 @@
+require 'date'
+
+module Dev
+  class Jira
+    class History
+      attr_accessor :date, :id, :author, :created, :items
+
+      def initialize(data)
+        @data = data
+        @id = data['id']
+        @author = data['author']
+        @items = data['items']
+        @created = Time.parse(data['created'])
+      end
+    end
+  end
+end

data/lib/firespring_dev_commands/jira/issue.rb

@@ -5,7 +5,7 @@ module Dev
       # Issue subtypes which do not map to a story type
       NON_STORY_TYPES = ['review', 'sub-task', 'code review sub-task', 'pre-deploy sub-task', 'deploy sub-task', 'devops sub-task'].freeze
 
-      attr_accessor :data, :project, :id, :title, :points, :assignee, :resolved_date
+      attr_accessor :data, :project, :id, :title, :points, :assignee, :resolved_date, :histories, :in_progress_history, :in_review_history, :closed_history
 
       def initialize(data)
         @data = data
@@ -15,10 +15,71 @@ module Dev
         @points = calculate_points(data)
         @assignee = Jira::User.lookup(data.assignee&.accountId)
         @resolved_date = data.resolutiondate
+        @histories = Jira::Histories.populate(data)
+        @in_progress_history = nil
+        @in_review_history = nil
+        @closed_history = nil
+      end
+
+      def cycle_time
+        # Calculate the difference and convert to days
+        ((last_closed_history.created - first_in_progress_history.created) / 60 / 60 / 24).round(2)
+      end
+
+      def in_progress_cycle_time
+        # Calculate the difference and convert to days
+        ((first_in_review_history.created - first_in_progress_history.created) / 60 / 60 / 24).round(2)
+      end
+
+      def in_review_cycle_time
+        # Calculate the difference and convert to days
+        ((last_closed_history.created - first_in_review_history.created) / 60 / 60 / 24).round(2)
+      end
+
+      private def first_in_progress_history
+        raise 'you must expand the changelog field to calculate cycle time' if histories.nil?
+
+        # Find the first instance in the histoy where the status moved to "In Progress"
+        @in_progress_history ||= histories.select do |history|
+          history.items.find do |item|
+            item['fieldId'] == 'status' && item['toString'] == 'In Progress'
+          end
+        end.min_by(&:created)
+        raise 'unable to find "In Progress" history entry needed to calculate cycle time' unless @in_progress_history
+
+        @in_progress_history
+      end
+
+      private def first_in_review_history
+        raise 'you must expand the changelog field to calculate cycle time' if histories.nil?
+
+        # Find the first instance in the histoy where the status moved to "In Review"
+        @in_review_history ||= histories.select do |history|
+          history.items.find do |item|
+            item['fieldId'] == 'status' && item['toString'] == 'In Review'
+          end
+        end.min_by(&:created)
+        raise 'unable to find "In Review" history entry needed to calculate cycle time' unless @in_review_history
+
+        @in_review_history
+      end
+
+      private def last_closed_history
+        raise 'you must expand the changelog field to calculate cycle time' if histories.nil?
+
+        # Find the last instance in the histoy where the status moved to "Closed"
+        @closed_history ||= histories.select do |history|
+          history.items.find do |item|
+            item['fieldId'] == 'status' && item['toString'] == 'Closed'
+          end
+        end.max_by(&:created)
+        raise 'unable to find "Closed" history entry needed to calculate cycle time' unless @closed_history
+
+        @closed_history
       end
 
       # Returns the value of the jira points field or 0 if the field is not found
-      def calculate_points(data)
+      private def calculate_points(data)
         return data.send(Dev::Jira.config.points_field_name).to_i if Dev::Jira.config.points_field_name && data.respond_to?(Dev::Jira.config.points_field_name)
 
         0

data/lib/firespring_dev_commands/jira.rb

@@ -9,15 +9,16 @@ module Dev
     # "user_lookup_list" should be an array of Jira::User objects representing the usernames, ids, etc for all jira users
     #    This is a bit clumsy but currently the jira api only returns the user id with issues
     #    and there is no way to query this information from Jira directly.
-    Config = Struct.new(:username, :token, :url, :points_field_name, :user_lookup_list, :read_timeout, :http_debug) do
+    Config = Struct.new(:username, :token, :url, :points_field_name, :expand, :user_lookup_list, :read_timeout, :http_debug) do
       def initialize
         self.username = nil
         self.token = nil
         self.url = nil
         self.points_field_name = nil
+        self.expand = []
         self.user_lookup_list = []
         self.read_timeout = 120
-        self.http_debug = false
+        self.http_debug = true
       end
     end
 
@@ -62,15 +63,16 @@ module Dev
     def issues(jql, &)
       start_at = 0
       max_results = 100
+      expand = self.class.config.expand
 
       # Query Jira and yield all issues it returns
-      issues = @client.Issue.jql(jql, start_at:, max_results:)
+      issues = @client.Issue.jql(jql, start_at:, max_results:, expand:)
       issues.map { |data| Issue.new(data) }.each(&)
 
       # If we returned the max_results then there may be more - add the max results to where we start at and query again
       while issues.length >= max_results
         start_at += max_results
-        issues = @client.Issue.jql(jql, start_at:, max_results:)
+        issues = @client.Issue.jql(jql, start_at:, max_results:, expand:)
         issues.map { |data| Issue.new(data) }.each(&)
       end
     end

data/lib/firespring_dev_commands/templates/config.rb

@@ -6,11 +6,14 @@ module Dev
     class Application
       # Contains rake tasks for displaying and setting application variables in parameter store
       class Config < Dev::Template::ApplicationInterface
-        attr_reader :path
+        attr_reader :path_prefix, :key_parameter_path
+
+        # Allow for custom config path_prefix for the application
+        # Allow for custom key parameter path (otherwise base it off the path prefix)
+        def initialize(name, path_prefix, key_parameter_path: nil, exclude: [])
+          @path_prefix = path_prefix
+          @key_parameter_path = key_parameter_path || "#{path_prefix}/kms/id"
 
-        # Allow for custom config path for the application
-        def initialize(name, path, exclude: [])
-          @path = path
           super(name, exclude:)
         end
 
@@ -18,7 +21,7 @@ module Dev
         def create_list_task!
           # Have to set a local variable to be accessible inside of the instance_eval block
           application = @name
-          path = @path
+          path_prefix = @path_prefix
           exclude = @exclude
 
           DEV_COMMANDS_TOP_LEVEL.instance_eval do
@@ -29,8 +32,10 @@ module Dev
                 desc "List all #{application} configs"
                 task list: %w(init) do
                   puts
-                  Dev::Aws::Parameter.new.list(path).each do |it|
-                    puts "  #{it.name} => #{it.value} (#{it.type})"
+                  puts "Listing all parameters which start with #{path_prefix}:".light_green
+                  puts
+                  Dev::Aws::Parameter.new.list(path_prefix).each do |it|
+                    puts "  #{it.name.light_white} => #{it.value.light_white} (#{it.type})"
                   end
                   puts
                 end
@@ -44,7 +49,8 @@ module Dev
         def create_set_task!
           # Have to set a local variable to be accessible inside of the instance_eval block
           application = @name
-          path = @path
+          path_prefix = @path_prefix
+          key_parameter_path = @key_parameter_path
           exclude = @exclude
 
           DEV_COMMANDS_TOP_LEVEL.instance_eval do
@@ -58,7 +64,7 @@ module Dev
                      "\n\toptionally specify ENCRYPT=true to change a String type to a SecureString type"
                 task set: %w(ensure_aws_credentials) do
                   raise 'NAME is required' if ENV['NAME'].to_s.strip.blank?
-                  raise 'NAME is not found in this apps parameters' if Dev::Aws::Parameter.new.list(path).none? { |it| it.name == ENV['NAME'] }
+                  raise 'NAME is not found in this apps parameters' if Dev::Aws::Parameter.new.list(path_prefix).none? { |it| it.name == ENV['NAME'] }
                   raise 'VALUE is required' if ENV['VALUE'].to_s.strip.blank?
 
                   param_path = ENV.fetch('NAME', nil)
@@ -68,7 +74,7 @@ module Dev
                   params = {type: 'String'}
                   if ENV['ENCRYPT'].to_s.strip == 'true' || old_value.type == 'SecureString'
                     params[:type] = 'SecureString'
-                    params[:key_id] = Dev::Aws::Parameter.new.get_value("#{path}/kms/id")
+                    params[:key_id] = Dev::Aws::Parameter.new.get_value(key_parameter_path)
                   end
 
                   message = 'This will change '.light_green +

data/lib/firespring_dev_commands/version.rb

@@ -6,6 +6,6 @@ module Dev
     # Use 'v.v.v.pre.alpha.v' for pre-release vesions
     # Use 'v.v.v.beta.v for beta versions
     # Use semantic versioning for any releases (https://semver.org/)
-    VERSION = '2.1.3'.freeze
+    VERSION = '2.1.4.pre.alpha.1'.freeze
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: firespring_dev_commands
 version: !ruby/object:Gem::Version
-  version: 2.1.3
+  version: 2.1.4.pre.alpha.1
 platform: ruby
 authors:
 - Firespring
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-08-22 00:00:00.000000000 Z
+date: 2023-08-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -284,6 +284,8 @@ files:
 - lib/firespring_dev_commands/git.rb
 - lib/firespring_dev_commands/git/info.rb
 - lib/firespring_dev_commands/jira.rb
+- lib/firespring_dev_commands/jira/histories.rb
+- lib/firespring_dev_commands/jira/history.rb
 - lib/firespring_dev_commands/jira/issue.rb
 - lib/firespring_dev_commands/jira/project.rb
 - lib/firespring_dev_commands/jira/user.rb
@@ -331,9 +333,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '3.1'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubygems_version: 3.4.10
 signing_key: