gitlab-triage 1.14.1 → 1.17.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fbce3d75b063c517b80bbbfa5a0cf8ef2cf0f6ffe9eabcebe743f7ea5e6fbc23
-  data.tar.gz: '09687c2d578d80f351a54a614113bcf36b70a15144f848409b8143be53c95f9b'
+  metadata.gz: 68a42526c02ea2ea87e7f0ca3d24ac0fb5acbdd145fb30fc2bbd79e533ce9b65
+  data.tar.gz: 274ba6fb0d4040abc095207eb3349c3719e57e6e37d6087af1bedaed78e8b6b5
 SHA512:
-  metadata.gz: f88d9507fd36e15941fa9b1345887e138893e8e7e5ae1b872408cfea449c2565c48c4267c1df278fa0f1ddf3f1c857050e4868eef88268b4acf9c7078dda0a53
-  data.tar.gz: 77dfd58b8e61d3115e9057b688d7f2b91b8473dd4e7a177a32cad0585f1f42c115d371f9d10f10a0a27fd3ee1b4230ee757982dcc85207395ebef82274165e70
+  metadata.gz: be15fd2ef8f9e4d12116f83906905bd43a45f0ba81978f03504aa751e97ae367f92110072abb2b2c4ae2419fd01c9599d89db3469f96c871341291dfdcddc43a
+  data.tar.gz: e2a9669fd8373304b66d8470fe24fae0c55349c229accf9a523462ead4e63ea4f8ef3f3bc3c1d9a6765073a5d7226f8787f2feeb8a1c6c4491d6dcedbaa3af1f

data/.gitlab/merge_request_templates/Release.md

@@ -32,4 +32,4 @@ with the latest commit from https://gitlab.com/gitlab-org/gitlab-triage/commits/
 - Checklist after merging:
   - [ ] [Update the release notes for the newly created tag](docs/release_process.md#how-to).
 
-/label ~"Engineering Productivity" ~"ep::triage" ~"tooling::workflow"
+/label ~"Engineering Productivity" ~"ep::triage" ~tooling ~"tooling::workflow"

data/README.md

@@ -38,6 +38,7 @@ The format of the file is [YAML](https://en.wikipedia.org/wiki/YAML).
 project.
 
 Select which resource to add the policy to:
+- `epics`
 - `issues`
 - `merge_requests`
 
@@ -47,9 +48,28 @@ For example:
 
 ```yml
 resource_rules:
+  epics:
+    rules:
+      - name: My epic policy
+        conditions:
+          date:
+            attribute: updated_at
+            condition: older_than
+            interval_type: days
+            interval: 5
+          state: opened
+          labels:
+            - None
+        actions:
+          labels:
+            - needs attention
+          mention:
+            - markglenfletcher
+          comment: |
+            {{author}} This epic is unlabelled after 5 days. It needs attention. Please take care of this before the end of #{2.days.from_now.strftime('%Y-%m-%d')}
   issues:
     rules:
-      - name: My policy
+      - name: My issue policy
         conditions:
           date:
             attribute: updated_at
@@ -86,7 +106,7 @@ resource_rules:
               /label ~"needs attention"
   merge_requests:
     rules:
-      - name: My policy
+      - name: My merge request policy
         conditions:
           state: opened
           labels:
@@ -585,6 +605,7 @@ Available action types:
 - [`comment` action](#comment-action)
 - [`comment_type` action option](#comment-type-action-option)
 - [`summarize` action](#summarize-action)
+- [`comment_on_summary` action](#comment-on-summary-action)
 
 ##### Labels action
 
@@ -796,8 +817,8 @@ Accepts a hash of fields.
 
 | Field         | Type   | Description                                     | Required | Placeholders | Ruby expression | Default        |
 | ----          | ----   | ----                                            | ----     | ----         | ----            | ----           |
-| `title`       | string | The title of the generated issue                | yes      | yes          | no              |                |
-| `destination` | integer or string | The project ID or path to create the generated issue in | no       | no           | no              | source project |
+| `title`       | string | The title of the generated issue                | yes      | yes          | yes             |                |
+| `destination` | integer or string | The project ID or path to create the generated issue in | no   | no              | no             | source project |
 | `item`        | string | Template representing each triaged resource     | no       | yes          | yes             |                |
 | `summary`     | string | The description of the generated issue          | no       | Only `{{title}}`, `{{items}}`, `{{type}}` | yes | |
 | `redact_confidential_resources` | boolean | Whether redact fields for confidential resources | no | no | no | true |
@@ -867,6 +888,56 @@ Which could generate an issue like:
   /label ~"needs attention"
   ```
 
+##### Comment on summary action
+
+Generates one comment for each resource, attaching these comments to the summary
+created by the [`summarize` action](#summarize-action).
+
+The use case for this is wanting to create a summary with an overview, and then
+a threaded discussion for each resource, with a header comment starting each
+discussion.
+
+Accepts a single string value: the template used to generate the comments. For
+details of the syntax of this template, see the [comment action](#comment-action).
+
+Since this action depends on the summary, it is invalid to supply a
+`comment_on_summary` action without an accompanying `summarize` sibling action.
+The `summarize` action will always be completed first.
+
+Just like for [comment action](#comment-action), setting `comment_type` in the
+`actions` set controls whether the comment must be resolved for merge requests.
+See: [`comment_type` action option](#comment-type-action-option).
+
+Example:
+
+```yml
+resource_rules:
+  issues:
+    rules:
+      - name: List of issues to discuss
+        limits:
+          most_recent: 15
+        actions:
+          comment_type: thread
+          comment_on_summary: |
+            # {{title}}
+
+            author: {{author}}
+          summarize:
+            title: |
+              #{resource[:type].capitalize} require labels
+            item: |
+              - [ ] [{{title}}]({{web_url}}) {{labels}}
+            summary: |
+              The following {{type}} require labels:
+
+              {{items}}
+
+              Please take care of them before the end of #{7.days.from_now.strftime('%Y-%m-%d')}
+
+              /label ~"needs attention"
+```
+
 ### Summary policies
 
 Summary policies are special policies that join multiple rule policies together

data/bin/gitlab-triage

@@ -3,6 +3,7 @@
 require 'yaml'
 require_relative '../lib/gitlab/triage/option_parser'
 require_relative '../lib/gitlab/triage/engine'
+require_relative '../lib/gitlab/triage/ui'
 
 options = Gitlab::Triage::OptionParser.parse(ARGV)
 options.policies_files << '.triage-policies.yml' if options.policies_files.empty?
@@ -10,7 +11,9 @@ options.policies_files << '.triage-policies.yml' if options.policies_files.empty
 options.policies_files.each do |policies_file|
   policies = HashWithIndifferentAccess.new(YAML.load_file(policies_file))
 
-  Gitlab::Triage::Engine
+  policy_engine = Gitlab::Triage::Engine
     .new(policies: policies, options: options)
-    .perform
+
+  puts Gitlab::Triage::UI.header("Executing policies from #{policies_file}.", char: '*')
+  policy_engine.perform
 end

data/lib/gitlab/triage/action.rb

@@ -1,14 +1,18 @@
 require_relative 'action/summarize'
 require_relative 'action/comment'
+require_relative 'action/comment_on_summary'
 
 module Gitlab
   module Triage
     module Action
       def self.process(policy:, **args)
-        {
-          Summarize => policy.summarize?,
-          Comment => policy.comment?
-        }.compact.each do |action, active|
+        policy.validate!
+
+        [
+          [Summarize, policy.summarize?],
+          [Comment, policy.comment?],
+          [CommentOnSummary, policy.comment_on_summary?]
+        ].each do |action, active|
           act(action: action, policy: policy, **args) if active
         end
       end

data/lib/gitlab/triage/action/comment.rb

@@ -57,14 +57,17 @@ module Gitlab
         end
 
         def build_post_url(resource)
-          # POST /projects/:id/issues/:issue_iid/notes
-          post_url = UrlBuilders::UrlBuilder.new(
+          url_builder_opts = {
             network_options: network.options,
-            source_id: resource[:project_id],
+            source: policy.source,
+            source_id: resource[policy.source_id_sym],
             resource_type: policy.type,
             resource_id: resource['iid'],
             sub_resource_type: sub_resource_type
-          ).build
+          }
+
+          # POST /(groups|projects)/:id/(epics|issues|merge_requests)/:iid/notes
+          post_url = UrlBuilders::UrlBuilder.new(url_builder_opts).build
 
           puts Gitlab::Triage::UI.debug "post_url: #{post_url}" if network.options.debug
 

data/lib/gitlab/triage/action/comment_on_summary.rb

@@ -0,0 +1,83 @@
+# frozen_string_literal: true
+
+require_relative 'base'
+require_relative '../command_builders/text_content_builder'
+require_relative '../command_builders/comment_command_builder'
+require_relative '../command_builders/label_command_builder'
+require_relative '../command_builders/remove_label_command_builder'
+require_relative '../command_builders/cc_command_builder'
+require_relative '../command_builders/status_command_builder'
+require_relative '../command_builders/move_command_builder'
+
+module Gitlab
+  module Triage
+    module Action
+      class CommentOnSummary < Base
+        class Dry < CommentOnSummary
+          def act
+            puts "The following comments would be posted for the rule **#{policy.name}**:\n\n"
+
+            super
+          end
+
+          private
+
+          def perform(comment)
+            puts "# #{summary[:web_url]}\n```\n#{comment}\n```\n"
+          end
+        end
+
+        attr_reader :summary
+
+        def initialize(policy:, network:)
+          super(policy: policy, network: network)
+          @summary = policy.summary
+        end
+
+        def act
+          policy.resources.each do |resource|
+            comment = build_comment(resource).strip
+
+            perform(comment) unless comment.empty?
+          end
+        end
+
+        private
+
+        def build_comment(resource)
+          CommandBuilders::TextContentBuilder.new(policy.actions[:comment_on_summary], resource: resource, network: network).build_command
+        end
+
+        def perform(comment)
+          network.post_api(build_post_url, body: comment)
+        end
+
+        def build_post_url
+          # POST /projects/:id/issues/:issue_iid/notes
+          post_url = UrlBuilders::UrlBuilder.new(
+            network_options: network.options,
+            source_id: summary['project_id'],
+            resource_type: policy.type,
+            resource_id: summary['iid'],
+            sub_resource_type: sub_resource_type
+          ).build
+
+          puts Gitlab::Triage::UI.debug "post_url: #{post_url}" if network.options.debug
+
+          post_url
+        end
+
+        def sub_resource_type
+          case type = policy.actions[:comment_type]
+          when 'comment', nil # nil is default
+            'notes'
+          when 'thread'
+            'discussions'
+          else
+            raise ArgumentError, "Unknown comment type: #{type}"
+          end
+        end
+      end
+    end
+  end
+end

data/lib/gitlab/triage/action/summarize.rb

@@ -10,6 +10,12 @@ module Gitlab
           private
 
           def perform
+            policy.summary = {
+              web_url: '[the-created-issue-url]',
+              project_id: 'some-id',
+              iid: 'some-iid'
+            }.with_indifferent_access
+
             if group_summary_without_destination?
               puts Gitlab::Triage::UI.warn("No issue will be created: No summary destination specified when source is 'groups'.")
               return
@@ -35,7 +41,7 @@ module Gitlab
             return
           end
 
-          network.post_api(post_issue_url, post_issue_body)
+          policy.summary = network.post_api(post_issue_url, post_issue_body)
         end
 
         def issue

data/lib/gitlab/triage/api_query_builders/base_query_param_builder.rb

@@ -2,16 +2,25 @@ module Gitlab
   module Triage
     module APIQueryBuilders
       class BaseQueryParamBuilder
-        attr_reader :param_name, :param_contents
+        attr_reader :param_name, :param_contents, :allowed_values
 
-        def initialize(param_name, param_contents)
+        def initialize(param_name, param_contents, allowed_values: nil)
           @param_name = param_name
           @param_contents = param_contents
+          @allowed_values = allowed_values
+
+          validate_allowed_values! if allowed_values
         end
 
         def build_param
           "&#{param_name}=#{param_content.strip}"
         end
+
+        private
+
+        def validate_allowed_values!
+          ParamsValidator.new([{ name: param_name, type: String, values: allowed_values }], { param_name => param_contents }).validate!
+        end
       end
     end
   end

data/lib/gitlab/triage/api_query_builders/date_query_param_builder.rb

@@ -1,57 +1,32 @@
-require_relative '../validators/params_validator'
+require_relative '../param_builders/date_param_builder'
+require_relative 'base_query_param_builder'
 
 module Gitlab
   module Triage
     module APIQueryBuilders
-      class DateQueryParamBuilder
+      class DateQueryParamBuilder < BaseQueryParamBuilder
         ATTRIBUTES = %w[updated_at created_at].freeze
-        CONDITIONS = %w[older_than newer_than].freeze
-        INTERVAL_TYPES = %w[days weeks months years].freeze
-
-        def self.filter_parameters
-          [
-            {
-              name: :attribute,
-              type: String,
-              values: ATTRIBUTES
-            },
-            {
-              name: :condition,
-              type: String,
-              values: CONDITIONS
-            },
-            {
-              name: :interval_type,
-              type: String,
-              values: INTERVAL_TYPES
-            },
-            {
-              name: :interval,
-              type: Numeric
-            }
-          ]
-        end
 
         def self.applicable?(condition)
           ATTRIBUTES.include?(condition[:attribute].to_s)
         end
 
         def initialize(condition_hash)
-          @attribute = condition_hash[:attribute].to_s
-          @interval_condition = condition_hash[:condition].to_sym
-          @interval_type = condition_hash[:interval_type]
-          @interval = condition_hash[:interval]
-          validate_condition(condition_hash)
+          date_param_builder = ParamBuilders::DateParamBuilder.new(ATTRIBUTES, condition_hash)
+
+          super(build_param_name(condition_hash), date_param_builder.param_content)
         end
 
-        def validate_condition(condition)
-          ParamsValidator.new(self.class.filter_parameters, condition).validate!
+        def param_content
+          param_contents
         end
 
-        def param_name
-          prefix = attribute.sub(/_at\z/, '')
+        private
+
+        def build_param_name(condition_hash)
+          prefix = condition_hash[:attribute].to_s.sub(/_at\z/, '')
           suffix =
-            case interval_condition
+            case condition_hash[:condition].to_sym
             when :older_than
               'before'
             when :newer_than
@@ -60,18 +35,6 @@ module Gitlab
 
           "#{prefix}_#{suffix}"
         end
-
-        def param_content
-          interval.public_send(interval_type).ago.to_date # rubocop:disable GitlabSecurity/PublicSend
-        end
-
-        def build_param
-          "&#{param_name}=#{param_content.strip}"
-        end
-
-        private
-
-        attr_reader :condition_hash, :attribute, :interval_condition, :interval_type, :interval
       end
     end
   end

data/lib/gitlab/triage/api_query_builders/multi_query_param_builder.rb

@@ -6,14 +6,22 @@ module Gitlab
       class MultiQueryParamBuilder < BaseQueryParamBuilder
         attr_reader :separator
 
-        def initialize(param_name, param_contents, separator)
+        def initialize(param_name, param_contents, separator, allowed_values: nil)
           @separator = separator
-          super(param_name, param_contents)
+          super(param_name, Array(param_contents), allowed_values: allowed_values)
         end
 
         def param_content
           param_contents.map(&:strip).join(separator)
         end
+
+        private
+
+        def validate_allowed_values!
+          param_contents.each do |param|
+            ParamsValidator.new([{ name: param_name, type: String, values: allowed_values }], { param_name => param }).validate!
+          end
+        end
       end
     end
   end