shipit-engine 0.32.0 → 0.33.0

data/app/models/shipit/user.rb

@@ -24,7 +24,7 @@ module Shipit
     end
 
     def self.find_or_create_author_from_github_commit(github_commit)
-      if (match_info = github_commit.commit.message.match(/^#{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!(match_info[1])
         rescue Octokit::NotFound
@@ -80,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)

data/app/models/shipit/webhooks.rb

@@ -5,6 +5,16 @@ module Shipit
       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],

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

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

@@ -0,0 +1,127 @@
+# frozen_string_literal: true
+
+module Shipit
+  module Webhooks
+    module Handlers
+      module PullRequest
+        class LabelCapturingHandler < 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 capture_labels?
+
+            capture_labels
+
+            stack
+          end
+
+          private
+
+          def capture_labels?
+            opened_active_stack? ||
+              labeled_active_stack? ||
+              unlabeled_active_stack? ||
+              reopened_active_stack?
+          end
+
+          def opened_active_stack?
+            opened? && stack.present?
+          end
+
+          def labeled_active_stack?
+            labeled? && stack.present? && !stack.archived?
+          end
+
+          def unlabeled_active_stack?
+            unlabeled? && stack.present? && !stack.archived?
+          end
+
+          def reopened_active_stack?
+            reopened? && stack.present? && !stack.archived?
+          end
+
+          def opened?
+            action == "opened"
+          end
+
+          def labeled?
+            action == "labeled"
+          end
+
+          def unlabeled?
+            action == "unlabeled"
+          end
+
+          def reopened?
+            action == "reopened"
+          end
+
+          def action
+            params.action
+          end
+
+          def pull_request
+            params.pull_request
+          end
+
+          def capture_labels
+            return unless pull_request = stack.pull_request
+
+            pull_request.update!(labels: params.pull_request.labels.map(&:name))
+          end
+
+          def review_stack
+            @review_stack ||=
+              Shipit::Webhooks::Handlers::PullRequest::ReviewStackAdapter
+                .new(params, scope: repository.review_stacks)
+          end
+
+          def repository
+            @repository ||=
+              Shipit::Repository
+                .from_github_repo_name(params.repository.full_name) || NullRepository.new
+          end
+
+          def stack
+            @stack ||= review_stack.stack
+          end
+
+          def labels
+            Array.new(pull_request.labels).map(&:name)
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,106 @@
+# frozen_string_literal: true
+
+module Shipit
+  module Webhooks
+    module Handlers
+      module PullRequest
+        class LabeledHandler < 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_label_change?
+
+            handle
+          end
+
+          private
+
+          def handle
+            if archive?
+              stack.archive!
+            elsif unarchive?
+              stack.unarchive!
+            end
+
+            stack
+          end
+
+          def stack
+            @stack ||=
+              Shipit::Webhooks::Handlers::PullRequest::ReviewStackAdapter
+                .new(params, scope: repository.review_stacks)
+          end
+
+          def repository
+            @repository ||= Shipit::Repository.from_github_repo_name(params.repository.full_name) ||
+              Shipit::NullRepository.new
+          end
+
+          def pull_request
+            params.pull_request
+          end
+
+          def pull_request_state
+            pull_request.state
+          end
+
+          def respond_to_label_change?
+            params.action == "labeled" &&
+              pull_request_state == "open" &&
+              repository.review_stacks_enabled &&
+              (archive? || unarchive?)
+          end
+
+          def archive?
+            (repository.provisioning_behavior_allow_with_label? && !pull_request_has_provisioning_label?) ||
+              (repository.provisioning_behavior_prevent_with_label? && pull_request_has_provisioning_label?)
+          end
+
+          def unarchive?
+            (repository.provisioning_behavior_allow_with_label? && pull_request_has_provisioning_label?) ||
+              (repository.provisioning_behavior_prevent_with_label? && !pull_request_has_provisioning_label?)
+          end
+
+          def pull_request_has_provisioning_label?
+            pull_request_label_names.include?(repository.provisioning_label_name)
+          end
+
+          def pull_request_label_names
+            Array.new(pull_request.labels).map(&:name)
+          end
+        end
+      end
+    end
+  end
+end