shipit-engine 0.28.1 → 0.33.0

data/app/models/shipit/team.rb

@@ -1,5 +1,6 @@
+# frozen_string_literal: true
 module Shipit
-  class Team < ActiveRecord::Base
+  class Team < Record
     REQUIRED_HOOKS = %i(membership).freeze
 
     has_many :memberships

data/app/models/shipit/undeployed_commit.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class UndeployedCommit < DelegateClass(Commit)
     attr_reader :index

data/app/models/shipit/unlimited_api_client.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class UnlimitedApiClient
     def stack_id?

data/app/models/shipit/user.rb

@@ -1,5 +1,6 @@
+# frozen_string_literal: true
 module Shipit
-  class User < ActiveRecord::Base
+  class User < Record
     DEFAULT_AVATAR = URI.parse('https://avatars.githubusercontent.com/u/583231?')
 
     has_many :memberships
@@ -23,9 +24,9 @@ module Shipit
     end
 
     def self.find_or_create_author_from_github_commit(github_commit)
-      if github_commit.commit.message =~ /^#{PullRequest::MERGE_REQUEST_FIELD}: ([\w\-\.]+)$/
+      if (match_info = github_commit.commit.message.match(/^#{MergeRequest::MERGE_REQUEST_FIELD}: ([\w\-\.]+)$/))
         begin
-          return find_or_create_by_login!($1)
+          return find_or_create_by_login!(match_info[1])
         rescue Octokit::NotFound
           # Corner case where the merge-requested-by user cannot be found (renamed/deleted).
           # In this case we carry on and search for the commit author
@@ -68,7 +69,7 @@ module Shipit
     end
 
     def identifiers_for_ping
-      {github_id: github_id, name: name, email: email, github_login: login}
+      { github_id: github_id, name: name, email: email, github_login: login }
     end
 
     def logged_in?
@@ -79,6 +80,11 @@ module Shipit
       @authorized ||= Shipit.github_teams.empty? || teams.where(id: Shipit.github_teams.map(&:id)).exists?
     end
 
+    def repositories_contributed_to
+      return [] unless id
+      Stack.where(id: stacks_contributed_to).distinct.pluck(:repository_id)
+    end
+
     def stacks_contributed_to
       return [] unless id
       Commit.where('author_id = :id or committer_id = :id', id: id).distinct.pluck(:stack_id)
@@ -137,11 +143,12 @@ module Shipit
 
       begin
         github_api.emails
-                  .sort_by { |e| e.primary ? 0 : 1 }
-                  .map(&:email)
-                  .find { |e| email_valid_and_preferred?(e) }
-      rescue Octokit::NotFound, Octokit::Forbidden
+          .sort_by { |e| e.primary ? 0 : 1 }
+          .map(&:email)
+          .find { |e| email_valid_and_preferred?(e) }
+      rescue Octokit::NotFound, Octokit::Forbidden, Octokit::Unauthorized
         # If the user hasn't agreed to the necessary permission, we can't access their private emails.
+        Rails.logger.warn("Failed to retrieve emails for user '#{github_user.name || github_user.login}'")
         nil
       end
     end

data/app/models/shipit/variable_definition.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class VariableDefinition
     attr_reader :name, :title, :default, :select

data/app/models/shipit/webhooks.rb

@@ -0,0 +1,43 @@
+# frozen_string_literal: true
+module Shipit
+  module Webhooks
+    class << self
+      def default_handlers
+        {
+          'push' => [Handlers::PushHandler],
+          'pull_request' => [
+            Handlers::PullRequest::OpenedHandler,
+            Handlers::PullRequest::ClosedHandler,
+            Handlers::PullRequest::ReopenedHandler,
+            Handlers::PullRequest::EditedHandler,
+            Handlers::PullRequest::AssignedHandler,
+            Handlers::PullRequest::LabeledHandler,
+            Handlers::PullRequest::UnlabeledHandler,
+            Handlers::PullRequest::LabelCapturingHandler,
+          ],
+          'status' => [Handlers::StatusHandler],
+          'membership' => [Handlers::MembershipHandler],
+          'check_suite' => [Handlers::CheckSuiteHandler],
+        }
+      end
+
+      def handlers
+        @handlers ||= reset_handlers!
+      end
+
+      def reset_handlers!
+        @handlers = default_handlers
+      end
+
+      def register_handler(event, callable = nil, &block)
+        handlers[event] ||= []
+        handlers[event] << callable if callable
+        handlers[event] << block if block_given?
+      end
+
+      def for_event(event)
+        handlers.fetch(event) { [] }
+      end
+    end
+  end
+end

data/app/models/shipit/webhooks/handlers/check_suite_handler.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+module Shipit
+  module Webhooks
+    module Handlers
+      class CheckSuiteHandler < Handler
+        params do
+          requires :check_suite do
+            requires :head_sha, String
+            requires :head_branch, String
+          end
+        end
+        def process
+          stacks.where(branch: params.check_suite.head_branch).each do |stack|
+            stack.commits.where(sha: params.check_suite.head_sha).each(&:schedule_refresh_check_runs!)
+          end
+        end
+      end
+    end
+  end
+end

data/app/models/shipit/webhooks/handlers/handler.rb

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+module Shipit
+  module Webhooks
+    module Handlers
+      class Handler
+        class << self
+          attr_reader :param_parser
+
+          def params(&block)
+            @param_parser = ExplicitParameters::Parameters.define(&block)
+          end
+        end
+
+        def self.call(params)
+          new(params).process
+        end
+
+        attr_reader :params, :payload
+
+        def initialize(payload)
+          @payload = payload
+          @params = self.class.param_parser.parse!(payload)
+        end
+
+        def process
+          raise NotImplementedError
+        end
+
+        private
+
+        def stacks
+          @stacks ||= Repository.from_github_repo_name(repository_name)&.stacks || Stack.none
+        end
+
+        def repository_name
+          payload.dig('repository', 'full_name')
+        end
+      end
+    end
+  end
+end

data/app/models/shipit/webhooks/handlers/membership_handler.rb

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+module Shipit
+  module Webhooks
+    module Handlers
+      class MembershipHandler < Handler
+        params do
+          requires :action, String
+          requires :team do
+            requires :id, Integer
+            requires :name, String
+            requires :slug, String
+            requires :url, String
+          end
+          requires :organization do
+            requires :login, String
+          end
+          requires :member do
+            requires :login, String
+          end
+        end
+        def process
+          team = find_or_create_team!
+          member = User.find_or_create_by_login!(params.member.login)
+
+          case params.action
+          when 'added'
+            team.add_member(member)
+          when 'removed'
+            team.members.delete(member)
+          else
+            raise ArgumentError, "Don't know how to perform action: `#{action.inspect}`"
+          end
+        end
+
+        private
+
+        def find_or_create_team!
+          Team.find_or_create_by!(github_id: params.team.id) do |team|
+            team.github_team = params.team
+            team.organization = params.organization.login
+          end
+        end
+      end
+    end
+  end
+end

data/app/models/shipit/webhooks/handlers/pull_request/assigned_handler.rb

@@ -0,0 +1,74 @@
+# frozen_string_literal: true
+
+module Shipit
+  module Webhooks
+    module Handlers
+      module PullRequest
+        class AssignedHandler < Shipit::Webhooks::Handlers::Handler
+          params do
+            requires :action, String
+            requires :number, Integer
+            requires :pull_request do
+              requires :id, Integer
+              requires :number, Integer
+              requires :url, String
+              requires :title, String
+              requires :state, String
+              requires :additions, Integer
+              requires :deletions, Integer
+              requires :head do
+                requires :sha, String
+                requires :ref, String
+              end
+              requires :user do
+                requires :login, String
+              end
+              requires :assignees, Array do
+                requires :login, String
+              end
+              requires :labels, Array do
+                requires :name, String
+              end
+            end
+            requires :repository do
+              requires :full_name, String
+            end
+            requires :sender do
+              requires :login, String
+            end
+          end
+
+          def process
+            return unless respond_to_assignee_change?
+
+            pull_request.update(github_pull_request: params.pull_request) if pull_request.present?
+          end
+
+          private
+
+          def respond_to_assignee_change?
+            %w[assigned unassigned].include?(params.action)
+          end
+
+          def pull_request
+            @pull_request ||= Shipit::PullRequest
+              .joins(:stack, stack: :repository)
+              .find_by(
+                number: params.number,
+                stacks: {
+                  repositories:
+                    {
+                      id: repository.id,
+                    },
+                }
+              )
+          end
+
+          def repository
+            Shipit::Repository.from_github_repo_name(params.repository.full_name) || Shipit::NullRepository.new
+          end
+        end
+      end
+    end
+  end
+end

data/app/models/shipit/webhooks/handlers/pull_request/closed_handler.rb

@@ -0,0 +1,68 @@
+# frozen_string_literal: true
+
+module Shipit
+  module Webhooks
+    module Handlers
+      module PullRequest
+        class ClosedHandler < Shipit::Webhooks::Handlers::Handler
+          params do
+            requires :action, String
+            requires :number, Integer
+            requires :pull_request do
+              requires :id, Integer
+              requires :number, Integer
+              requires :url, String
+              requires :title, String
+              requires :state, String
+              requires :additions, Integer
+              requires :deletions, Integer
+              requires :head do
+                requires :sha, String
+                requires :ref, String
+              end
+              requires :user do
+                requires :login, String
+              end
+              requires :assignees, Array do
+                requires :login, String
+              end
+              requires :labels, Array do
+                requires :name, String
+              end
+            end
+            requires :repository do
+              requires :full_name, String
+            end
+            requires :sender do
+              requires :login, String
+            end
+          end
+
+          def process
+            return unless respond_to_pull_request_closed?
+
+            review_stack.archive!
+          end
+
+          private
+
+          def repository
+            @repository ||=
+              Shipit::Repository.from_github_repo_name(params.repository.full_name) ||
+              Shipit::NullRepository.new
+          end
+
+          def review_stack
+            @review_stack ||=
+              Shipit::Webhooks::Handlers::PullRequest::ReviewStackAdapter
+                .new(params, scope: repository.review_stacks)
+          end
+
+          def respond_to_pull_request_closed?
+            params.action == "closed"
+          end
+        end
+      end
+    end
+  end
+end

data/app/models/shipit/webhooks/handlers/pull_request/edited_handler.rb

@@ -0,0 +1,74 @@
+# frozen_string_literal: true
+
+module Shipit
+  module Webhooks
+    module Handlers
+      module PullRequest
+        class EditedHandler < Shipit::Webhooks::Handlers::Handler
+          params do
+            requires :action, String
+            requires :number, Integer
+            requires :pull_request do
+              requires :id, Integer
+              requires :number, Integer
+              requires :url, String
+              requires :title, String
+              requires :state, String
+              requires :additions, Integer
+              requires :deletions, Integer
+              requires :head do
+                requires :sha, String
+                requires :ref, String
+              end
+              requires :user do
+                requires :login, String
+              end
+              requires :assignees, Array do
+                requires :login, String
+              end
+              requires :labels, Array do
+                requires :name, String
+              end
+            end
+            requires :repository do
+              requires :full_name, String
+            end
+            requires :sender do
+              requires :login, String
+            end
+          end
+
+          def process
+            return unless respond_to_pull_request_edited?
+
+            pull_request.update(github_pull_request: params.pull_request) if pull_request.present?
+          end
+
+          private
+
+          def pull_request
+            @pull_request ||= Shipit::PullRequest
+              .joins(:stack, stack: :repository)
+              .find_by(
+                number: params.number,
+                stacks: {
+                  repositories:
+                    {
+                      id: repository.id,
+                    },
+                }
+              )
+          end
+
+          def repository
+            Shipit::Repository.from_github_repo_name(params.repository.full_name) || Shipit::NullRepository.new
+          end
+
+          def respond_to_pull_request_edited?
+            params.action == "edited"
+          end
+        end
+      end
+    end
+  end
+end