shipit-engine 0.28.1 → 0.33.0

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

@@ -0,0 +1,107 @@
+# frozen_string_literal: true
+
+module Shipit
+  module Webhooks
+    module Handlers
+      module PullRequest
+        class UnlabeledHandler < 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 repository
+            @repository ||=
+              Shipit::Repository.from_github_repo_name(params.repository.full_name) ||
+              Shipit::NullRepository.new
+          end
+
+          def stack
+            @stack ||=
+              Shipit::Webhooks::Handlers::PullRequest::ReviewStackAdapter
+                .new(params, scope: repository.review_stacks)
+          end
+
+          def pull_request
+            params.pull_request
+          end
+
+          def pull_request_state
+            pull_request["state"]
+          end
+
+          def respond_to_label_change?
+            params.action == "unlabeled" &&
+              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

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

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+module Shipit
+  module Webhooks
+    module Handlers
+      class PushHandler < Handler
+        params do
+          requires :ref
+        end
+        def process
+          stacks.where(branch: branch).each(&:sync_github)
+        end
+
+        private
+
+        def branch
+          params.ref.gsub('refs/heads/', '')
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+module Shipit
+  module Webhooks
+    module Handlers
+      class StatusHandler < Handler
+        params do
+          requires :sha, String
+          requires :state, String
+          accepts :description, String
+          accepts :target_url, String
+          accepts :context, String
+          accepts :created_at, String
+
+          accepts :branches, Array do
+            requires :name, String
+          end
+        end
+
+        def process
+          Commit.where(sha: params.sha).each do |commit|
+            commit.create_status_from_github!(params)
+          end
+        end
+      end
+    end
+  end
+end

data/app/serializers/concerns/shipit/conditional_attributes.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   module ConditionalAttributes
     extend ActiveSupport::Concern

data/app/serializers/shipit/anonymous_user_serializer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class AnonymousUserSerializer < UserSerializer
   end

data/app/serializers/shipit/command_line_user_serializer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class CommandLineUserSerializer < UserSerializer
   end

data/app/serializers/shipit/commit_serializer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class CommitSerializer < ShortCommitSerializer
     include GithubUrlHelper

data/app/serializers/shipit/deploy_serializer.rb

@@ -1,10 +1,11 @@
+# frozen_string_literal: true
 module Shipit
   class DeploySerializer < TaskSerializer
     include GithubUrlHelper
 
     has_many :commits
 
-    attributes :compare_url, :rollback_url, :additions, :deletions
+    attributes :compare_url, :rollback_url, :additions, :deletions, :rollback_once_aborted_to
 
     def html_url
       stack_deploy_url(object.stack, object)
@@ -21,5 +22,11 @@ module Shipit
     def type
       :deploy
     end
+
+    def rollback_once_aborted_to
+      return nil unless object.rollback_once_aborted_to
+
+      DeploySerializer.new(object.rollback_once_aborted_to)
+    end
   end
 end

data/app/serializers/shipit/hook_serializer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class HookSerializer < ActiveModel::Serializer
     include ConditionalAttributes

data/app/serializers/shipit/merge_request_serializer.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+module Shipit
+  class MergeRequestSerializer < ActiveModel::Serializer
+    include GithubUrlHelper
+    include ConditionalAttributes
+
+    has_one :merge_requested_by
+    has_one :head, serializer: ShortCommitSerializer
+
+    attributes :id, :number, :title, :github_id, :additions, :deletions, :state, :merge_status, :mergeable,
+               :merge_requested_at, :rejection_reason, :html_url, :branch, :base_ref
+
+    def html_url
+      github_pull_request_url(object)
+    end
+
+    def include_rejection_reason?
+      object.rejection_reason?
+    end
+  end
+end

data/app/serializers/shipit/pull_request_serializer.rb

@@ -1,20 +1,18 @@
+# frozen_string_literal: true
+
 module Shipit
   class PullRequestSerializer < ActiveModel::Serializer
     include GithubUrlHelper
     include ConditionalAttributes
 
-    has_one :merge_requested_by
+    has_one :user
     has_one :head, serializer: ShortCommitSerializer
+    has_many :assignees, serializer: UserSerializer
 
-    attributes :id, :number, :title, :github_id, :additions, :deletions, :state, :merge_status, :mergeable,
-               :merge_requested_at, :rejection_reason, :html_url, :branch, :base_ref
+    attributes :id, :number, :title, :github_id, :additions, :deletions, :state, :html_url
 
     def html_url
-      github_pull_request_url(object)
-    end
-
-    def include_rejection_reason?
-      object.rejection_reason?
+      github_pull_request_url(object) if object.stack.present?
     end
   end
 end

data/app/serializers/shipit/review_stack_serializer.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+module Shipit
+  class ReviewStackSerializer < Shipit::StackSerializer
+    has_one :pull_request
+  end
+end

data/app/serializers/shipit/rollback_serializer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class RollbackSerializer < DeploySerializer
     def type

data/app/serializers/shipit/short_commit_serializer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class ShortCommitSerializer < ActiveModel::Serializer
     attributes :sha, :message

data/app/serializers/shipit/stack_serializer.rb

@@ -1,11 +1,14 @@
+# frozen_string_literal: true
 module Shipit
   class StackSerializer < ActiveModel::Serializer
     include ConditionalAttributes
 
     has_one :lock_author
-    attributes :id, :repo_owner, :repo_name, :environment, :html_url, :url, :tasks_url, :deploy_url, :pull_requests_url,
-               :deploy_spec, :undeployed_commits_count, :is_locked, :lock_reason, :continuous_deployment, :created_at,
-               :updated_at, :locked_since, :last_deployed_at, :branch, :merge_queue_enabled
+
+    attributes :id, :repo_owner, :repo_name, :environment, :html_url, :url, :tasks_url, :deploy_url,
+      :merge_requests_url, :deploy_spec, :undeployed_commits_count, :is_locked, :lock_reason, :continuous_deployment,
+      :created_at, :updated_at, :locked_since, :last_deployed_at, :branch, :merge_queue_enabled, :is_archived,
+      :archived_since
 
     def url
       api_stack_url(object)
@@ -19,8 +22,8 @@ module Shipit
       api_stack_tasks_url(object)
     end
 
-    def pull_requests_url
-      api_stack_pull_requests_url(object)
+    def merge_requests_url
+      api_stack_merge_requests_url(object)
     end
 
     def is_locked
@@ -39,6 +42,10 @@ module Shipit
       object.locked?
     end
 
+    def is_archived
+      object.archived?
+    end
+
     def deploy_spec
       object.cached_deploy_spec.cacheable.config
     end

data/app/serializers/shipit/tail_task_serializer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class TailTaskSerializer < ActiveModel::Serializer
     include ChunksHelper
@@ -7,7 +8,7 @@ module Shipit
 
     def url
       return @url if defined? @url
-      @url = next_chunks_url(task)
+      @url = next_chunks_url(task, last_byte: next_offset)
     end
 
     def include_url?
@@ -15,7 +16,7 @@ module Shipit
     end
 
     def output
-      task.chunks.tail(context[:last_id]).pluck(:text).join
+      @output ||= task.tail_output(last_byte)
     end
 
     def rollback_url
@@ -31,6 +32,14 @@ module Shipit
     alias_method :task, :object
     delegate :stack, to: :object
 
+    def next_offset
+      last_byte + output.bytesize
+    end
+
+    def last_byte
+      context[:last_byte].to_i
+    end
+
     def rollback
       return @rollback if defined? @rollback
       @rollback = stack.rollbacks.where(parent_id: task.id).last

data/app/serializers/shipit/task_serializer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class TaskSerializer < ActiveModel::Serializer
     include ConditionalAttributes
@@ -5,23 +6,7 @@ module Shipit
     has_one :author
     has_one :revision, serializer: ShortCommitSerializer
 
-    attributes(*%i(
-      id
-      url
-      html_url
-      output_url
-      type
-      status
-      action
-      title
-      description
-      started_at
-      ended_at
-      updated_at
-      created_at
-      env
-      ignored_safeties
-    ))
+    attributes(:id, :url, :html_url, :output_url, :type, :status, :action, :title, :description, :started_at, :ended_at, :updated_at, :created_at, :env, :ignored_safeties, :max_retries, :retry_attempt)
 
     def revision
       object.until_commit

data/app/serializers/shipit/user_serializer.rb

@@ -1,5 +1,10 @@
+# frozen_string_literal: true
 module Shipit
   class UserSerializer < ActiveModel::Serializer
-    attributes :id, :name, :email, :login, :avatar_url, :created_at, :updated_at, :github_id
+    attributes :id, :name, :email, :login, :avatar_url, :created_at, :updated_at, :github_id, :anonymous
+
+    def anonymous
+      !object.logged_in?
+    end
   end
 end

data/app/validators/ascii_only_validator.rb

@@ -1,7 +1,8 @@
+# frozen_string_literal: true
 class AsciiOnlyValidator < ActiveModel::EachValidator
   def validate_each(record, attribute, value)
-    value.encode(Encoding::ASCII)
-  rescue Encoding::UndefinedConversionError
-    record.errors.add(attribute, :ascii)
+    if value && !value.ascii_only?
+      record.errors.add(attribute, :ascii)
+    end
   end
 end

data/app/validators/subset_validator.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class SubsetValidator < ActiveModel::EachValidator
   def validate_each(record, attribute, value)
     superset = options[:of]

data/app/views/layouts/shipit.html.erb

@@ -2,7 +2,7 @@
 <html lang="<%= I18n.locale %>" data-controller="<%= controller_name %>" data-action="<%= action_name %>">
 <head>
   <% if @stack %>
-    <title><%= Shipit.app_name %> - <%= @stack.repo_name %>/<%= @stack.environment %></title>
+    <title><%= @stack.repo_name %>/<%= @stack.environment %> - <%= Shipit.app_name %></title>
   <% else %>
   <title><%= Shipit.app_name %></title>
   <% end %>
@@ -12,6 +12,8 @@
   <%= javascript_include_tag :shipit %>
   <%= yield :update_subscription %>
   <%= csrf_meta_tags %>
+
+  <%= render 'layouts/head' %>
 </head>
 <body>
   <% flash.each do |name, msg| %>
@@ -52,11 +54,11 @@
     <% end %>
 
     <header class="header">
-      <%= link_to "Shipit v#{Shipit::VERSION}", "https://github.com/Shopify/shipit-engine/tree/v#{Shipit::VERSION}", class: 'powered-by' %>
+      <%= link_to "Powered by Shipit v#{Shipit::VERSION}", "https://github.com/Shopify/shipit-engine/tree/v#{Shipit::VERSION}", class: 'powered-by' %>
       <div class="wrapper">
         <div class="header__inner">
           <a href="/" class="logo">
-            <span class="visually-hidden">Shipit</span>
+            <span class="visually-hidden"><%= Shipit.app_name %></span>
           </a>
           <div class="header__page-title">
             <%= yield :page_title %>