gitlab-dangerfiles 4.8.1 → 4.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b42206a72b735d7a0016622666b37571d85d541a758d5454fbca3260e359f1f2
-  data.tar.gz: ac570434fc3dee200d7656a557668454e2ceb5719c43d49b80c364688d3d0e13
+  metadata.gz: dc6047699d31bc320127f651d567ddf984b6189eeeb7646eb4f0d46b4185e9fd
+  data.tar.gz: 655da17704203a02f6f138b8493bb8cf2002c2121d1015a63da0677ebf23b48d
 SHA512:
-  metadata.gz: '030678a153058f6c0ab3fdb198d109ca43b8693f763a6259585b2864c7d688faec108dbbe93285226a7b926ab6f99423b6b57d703f83004b588df27917582d22'
-  data.tar.gz: 7a3398d6bc48bdbd28a43329c311088c211b6343e00200a782b0eef9503dae39befeaaa139710847d3a674e1032508b8c938ef889f7907d9739548919fa12c97
+  metadata.gz: 02613c71246a09f46d15e0117e3f304ae626618c5736ff9b02f5e15bfbeccaa4921cefa694c97548dc5af2f4611866b58eb29f9eb5e0305d7412461097980a2c
+  data.tar.gz: 6703f6fe3502ad2336c33a4296a67d3e7d77d0dd9e46db97f510f0bd4c17a729f1bb20afcbb4ab6a0d5b24da477e4e50378e76cb943b9fc474fbed29320e619b

data/.gitlab/merge_request_templates/Release.md

@@ -2,7 +2,7 @@
 commit from this merge request, and `<NEW_VERSION>` with the upcoming version number. -->
 ## Diff
 
-https://gitlab.com/gitlab-org/ruby/gems/gitlab-dangerfiles/-/compare/v<PREVIOUS_VERSION>...<COMMIT_UPDATING_VERSION>
+https://gitlab.com/gitlab-org/ruby/gems/gitlab-dangerfiles/-/compare/<PREVIOUS_VERSION>...<COMMIT_UPDATING_VERSION>
 
 ## Checklist
 

data/.gitlab-ci.yml

@@ -23,7 +23,7 @@ default:
       - vendor/ruby
 
 .default-test-job:
-  image: "ruby:${RUBY_VERSION}"
+  image: "${GITLAB_DEPENDENCY_PROXY}ruby:${RUBY_VERSION}"
   stage: test
   needs: []
   before_script:

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    gitlab-dangerfiles (4.8.1)
+    gitlab-dangerfiles (4.10.0)
       danger (>= 9.3.0)
       danger-gitlab (>= 8.0.0)
       rake (~> 13.0)
@@ -117,6 +117,7 @@ GEM
     parallel (1.22.1)
     parser (3.2.1.0)
       ast (~> 2.4.1)
+      racc
     proc_to_ast (0.1.0)
       coderay
       parser
@@ -125,6 +126,7 @@ GEM
       coderay (~> 1.1)
       method_source (~> 1.0)
     public_suffix (5.0.1)
+    racc (1.8.1)
     rack (3.0.4.1)
     rainbow (3.1.1)
     rake (13.0.6)
@@ -220,6 +222,7 @@ DEPENDENCIES
   gitlab-styles (~> 10.0)
   guard-rspec (~> 4.7.3)
   lefthook (~> 1.3)
+  racc (~> 1.8)
   rspec (~> 3.8)
   rspec-parameterized
   rubocop-rails (< 2.21.2)

data/README.md

@@ -171,6 +171,15 @@ This rule ensures the merge request isn't too big to be reviewed, otherwise it s
 
 This rule posts a failure if the merge request has more than 20 commits.
 
+#### `danger_code_review`
+
+This rule posts a warning/failure when `@GitLabDuo` isn't assigned as a reviewer.
+
+##### Available configurations
+
+- `duo_code_review`: Whether a review from GitLab Duo Code is `:mandatory` or `:optional`.
+  Default to `:optional`.
+
 #### `metadata`
 
 This rule ensures basic metadata such as assignee, milestone and description are set on the merge request.
@@ -193,6 +202,38 @@ project. To use it in your project, perform the following steps:
      dangerfiles.import_dangerfiles(only: %w[simple_roulette])
    end
    ```
+   
+##### Auto-assignment
+
+By default, the simple roulette only suggests reviewers in a comment. You can, optionally, configure it to automatically assign reviewers from the roulette to your merge requests.
+
+Configure `auto_assign_for_roulette_roles` to specify which roles to assign (`:reviewer`, `:maintainer`, or both). If empty (default), no auto-assignment occurs.
+
+Optionally, configure `auto_assign_for_roulette_labels` to only auto-assign for Merge Requests with specific labels. If empty (default), auto-assignment applies to all MRs.
+
+###### Examples
+
+``` ruby
+# Auto-assign reviewers for all MRs
+helper.config.auto_assign_for_roulette_roles = [:reviewer]
+
+# Auto-assign both reviewer and maintainer for all MRs
+helper.config.auto_assign_for_roulette_roles = [:reviewer, :maintainer]
+
+# Auto-assign reviewers only for a specific MR label
+helper.config.auto_assign_for_roulette_roles = [:reviewer]
+helper.config.auto_assign_for_roulette_labels = ["maintenance::dependency"]
+```
+
+##### Available configurations
+
+Note: These options aren't actually used in the `simple_roulette` rule, but can be used in your own
+implementation of the roulette if you use `roulette.required_approvals`/`roulette.codeowners_approvals`.
+
+- `included_optional_codeowners_sections_for_roulette`: Indicates which optional codeowners sections
+  should be included in roulette. Default to `[]`.
+- `excluded_required_codeowners_sections_for_roulette`: Indicates which required codeowners sections
+  should be excluded from roulette. Default to `[]`.
 
 #### `type_label`
 
@@ -281,6 +322,16 @@ To run the Danger Rake task in a project that doesn't have `master` as the defau
 DANGER_LOCAL_BASE="origin/main" bundle exec rake danger_local
 ```
 
+## Warnings vs Errors
+
+The following lint violations generate warnings:
+
+1. The commit subject is too short ([code](https://gitlab.com/gitlab-org/ruby/gems/gitlab-dangerfiles/blob/bccb167/lib/gitlab/dangerfiles/base_linter.rb#L74-76)).
+1. The commit changes too many lines across too many files without describing the changes in the commit body ([code](https://gitlab.com/gitlab-org/ruby/gems/gitlab-dangerfiles/blob/bccb167/lib/gitlab/dangerfiles/commit_linter.rb#L83-85)).
+1. The commit body contains too many characters per line ([code](https://gitlab.com/gitlab-org/ruby/gems/gitlab-dangerfiles/blob/bccb167/lib/gitlab/dangerfiles/commit_linter.rb#L87-96)).
+
+All other lint violations generate errors, except when the MR has [Squash commits when merge request is accepted](https://docs.gitlab.com/ee/user/project/merge_requests/squash_and_merge.html#squash-commits-in-a-merge-request) enabled, which downgrades lint violations to warnings.
+
 ## Documentation
 
 Latest documentation can be found at <https://www.rubydoc.info/gems/gitlab-dangerfiles>.
@@ -307,8 +358,8 @@ For example, to test `gitlab-dangerfiles` changes from the `your-branch-name` br
 
 ```ruby
 group :development, :test, :danger do
-  gem 'gitlab-dangerfiles', '~> 3.4.3', require: false,
-  git: 'https://gitlab.com/gitlab-org/ruby/gems/gitlab-dangerfiles.git',
+  gem 'gitlab-dangerfiles', require: false, feature_category: :tooling,
+  git: 'https://gitlab.com/gitlab-org/ruby/gems/gitlab-dangerfiles.git', # rubocop:disable Cop/GemFetcher -- Testing unreleased changes
   ref: 'your-branch-name'
 end
 ```

data/gitlab-dangerfiles.gemspec

@@ -37,6 +37,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "lefthook", "~> 1.3"
   spec.add_development_dependency "rspec-parameterized"
   spec.add_development_dependency "rspec", "~> 3.8"
+  spec.add_development_dependency "racc", "~> 1.8"
   # we do not commit the bundle lockfile, so this temporary workaround needs to be
   # present until 2.21.3 or 2.22.x is released
   # See https://gitlab.com/gitlab-org/ruby/gems/gitlab-styles/-/issues/63

data/lib/danger/plugins/duo_code.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+module Danger
+  # Contains method to check if Duo code suggestion added to MR as a reviewer.
+  class DuoCode < Danger::Plugin
+    def suggestion_added?
+      return false unless helper.ci?
+
+      helper.mr_reviewers.include?("GitLabDuo")
+    end
+  end
+end

data/lib/danger/plugins/internal/helper.rb

@@ -30,7 +30,11 @@ module Danger
       Authentication: '~"group::authentication"',
       Authorization: '~"group::authorization"',
       Compliance: '~"group::compliance"',
-      Verify: '~"devops::verify"'
+      Verify: '~"devops::verify"',
+      "AST:Composition Analysis": '~"group::composition analysis"',
+      "AST:Dynamic Analysis": '~"group::dynamic analysis"',
+      "AST:Secret Detection": '~"group::secret detection"',
+      "AST:Static Analysis": '~"group::static analysis"'
     }.freeze
     # rubocop:enable Style/HashSyntax
 
@@ -181,7 +185,9 @@ module Danger
     # @return [{Symbol => Array<String>}] a hash of the type +{ category1: ["file1", "file2"], category2: ["file3", "file4"] }+
     #   using filename regex (+filename_regex+) and specific change regex (+changes_regex+) from the given +categories+ hash.
     def changes_by_category(categories = [])
-      all_changed_files.each_with_object(Hash.new { |h, k| h[k] = [] }) do |file, hash|
+      changed_and_deleted_files = all_changed_files + changes.deleted.files
+
+      changed_and_deleted_files.each_with_object(Hash.new { |h, k| h[k] = [] }) do |file, hash|
         categories_for_file(file, categories).each { |category| hash[category] << file }
       end
     end
@@ -244,7 +250,7 @@ module Danger
     # @return [String] the GFM for a category label, making its best guess if it's not
     #   a category we know about.
     def label_for_category(category)
-      CATEGORY_LABELS[category] ||
+      helper.config.custom_labels_for_categories[category] || CATEGORY_LABELS[category] ||
 
         if category.start_with?("`")
           category.to_s
@@ -288,6 +294,13 @@ module Danger
       gitlab_helper.mr_json["assignees"]
     end
 
+    # @return [Array<Hash>] +[]+ when not in the CI context, and the MR reviewers otherwise.
+    def mr_reviewers
+      return [] unless ci?
+
+      gitlab_helper.mr_json["reviewers"]
+    end
+
     # @return [String] +""+ when not in the CI context, and the MR title otherwise.
     def mr_title
       return "" unless ci?

data/lib/danger/plugins/roulette.rb

@@ -45,7 +45,7 @@ module Danger
     # @param categories [Array<Symbol>] An array of categories symbols.
     #
     # @return [Array<Spin>]
-    def spin(project = nil, categories = [:none], ux_fallback_wider_community_reviewer: teammate_pedroms)
+    def spin(project = nil, categories = [:none], ux_fallback_wider_community_reviewer: ux_fallback_reviewer)
       # TODO: Deprecate the project argument. It prevents us from
       # memorizing Spinner and can cause unexpected results if it's
       # passing a different project than the merge request project.
@@ -87,10 +87,40 @@ module Danger
       Gitlab::Dangerfiles::Teammate.warnings
     end
 
+    # Automatically assigns reviewers from roulette spins if configured to do so
+    #
+    # @param spins [Array<Spin>] The roulette spins to potentially assign from
+    def assign_reviewers_from_roulette(spins)
+      return if helper.mr_reviewers.any?
+
+      reviewers_to_assign = find_reviewers_to_assign(spins)
+
+      if reviewers_to_assign.any?
+        post_assignment_message(reviewers_to_assign)
+      else
+        warn("No reviewers available for assignment")
+      end
+    end
+
+    # Determines if auto-assignment should happen based on configuration
+    #
+    # @return [Boolean]
+    def auto_assign_reviewers?
+      return false if helper.config.auto_assign_for_roulette_roles.empty?
+
+      configured_labels = helper.config.auto_assign_for_roulette_labels
+
+      return true if configured_labels.empty?
+
+      mr_labels = helper.mr_labels
+      configured_labels.any? { |label| mr_labels.include?(label) }
+    end
+
     private
 
-    def teammate_pedroms
-      @teammate_pedroms ||= Gitlab::Dangerfiles::Teammate.find_member("pedroms")
+    def ux_fallback_reviewer
+      teammates = %w[pedroms annabeldunstone seggenberger jmiocene clavimoniere]
+      Gitlab::Dangerfiles::Teammate.find_member(teammates.sample)
     end
 
     def spin_for_approval_rule?(rule)
@@ -157,5 +187,41 @@ module Danger
     def labels
       @labels ||= helper.mr_labels
     end
+
+    # Find reviewers to assign based on configured roles
+    #
+    # @param spins [Array<Spin>] The roulette spins
+    # @return [Array<String>] Array of usernames to assign
+    def find_reviewers_to_assign(spins)
+      roles_to_assign = helper.config.auto_assign_for_roulette_roles
+      reviewers_to_assign = []
+
+      spins.each do |spin|
+        if roles_to_assign.include?(:reviewer) && spin.reviewer&.username
+          reviewers_to_assign << spin.reviewer.username
+        end
+
+        if roles_to_assign.include?(:maintainer) && spin.maintainer&.username
+          reviewers_to_assign << spin.maintainer.username
+        end
+
+        if reviewers_to_assign.any?
+          break
+        end
+      end
+
+      reviewers_to_assign
+    end
+
+    # Posts the assignment message with the selected reviewers
+    #
+    # @param reviewers_to_assign [Array<String>] Array of usernames to assign
+    def post_assignment_message(reviewers_to_assign)
+      role_text = helper.config.auto_assign_for_roulette_roles.map(&:to_s).join(' and ')
+      message = "🎲 Assigned #{role_text}s based on reviewer roulette.\n/assign_reviewer #{reviewers_to_assign.map { |u| "@#{u}" }.join(' ')}"
+      markdown(message)
+    rescue StandardError => e
+      warn("Failed to assign reviewers: #{e.message}")
+    end
   end
 end

data/lib/danger/rules/commit_messages/Dangerfile

@@ -113,7 +113,7 @@ def warn_or_fail_commits(failed_linters, default_to_fail: true)
   Array(failed_linters).each do |linter|
     linter.problems.each do |problem_key, problem_desc|
       case problem_key
-      when :subject_too_short, :subject_above_warning, :details_too_many_changes, :details_line_too_long
+      when :subject_too_short, :details_too_many_changes, :details_line_too_long
         warn_commit(linter.commit, problem_desc)
       else
         self.__send__(:"#{level}_commit", linter.commit, problem_desc)

data/lib/danger/rules/duo_code_review/Dangerfile

@@ -0,0 +1,10 @@
+# frozen_string_literal: true
+
+return if duo_code.suggestion_added?
+
+case helper.config.duo_code_review
+when :optional
+  markdown("We advise getting a review from GitLab Duo Code. You can assign `@GitLabDuo` as a reviewer to this merge request.")
+when :mandatory
+  failure("A review from GitLab Duo Code is mandatory. Please assign `@GitLabDuo` as a reviewer to this merge request.")
+end

data/lib/danger/rules/simple_roulette/Dangerfile

@@ -8,9 +8,6 @@ MESSAGE = <<MARKDOWN
 Changes that require review have been detected! A merge request is normally
 reviewed by both a reviewer and a maintainer in its primary category and by a
 maintainer in all other categories.
-MARKDOWN
-
-TABLE_MARKDOWN = <<MARKDOWN
 
 To spread load more evenly across eligible reviewers, Danger has picked a candidate for each
 review slot. Feel free to
@@ -26,9 +23,11 @@ Please consider assigning a reviewer or maintainer who is a
 [domain expert](https://about.gitlab.com/handbook/engineering/projects/##{PROJECT_NAME})
 in the area of the merge request.
 
+MARKDOWN
+
+NO_AUTO_ASSIGN = <<MARKDOWN
 Once you've decided who will review this merge request, mention them as you
 normally would! Danger does not automatically notify them for you.
-
 MARKDOWN
 
 TABLE_HEADER_WITH_CATEGORIES = <<MARKDOWN
@@ -101,5 +100,11 @@ if changes.any?
 
   markdown(format(WARNING_MESSAGE, warnings: warnings)) if warnings
 
-  markdown(TABLE_MARKDOWN + table_header + rows.join("\n")) unless rows.empty?
+  markdown(table_header + rows.join("\n")) unless rows.empty?
+
+  if roulette.auto_assign_reviewers?
+    roulette.assign_reviewers_from_roulette(random_roulette_spins)
+  else
+    markdown(NO_AUTO_ASSIGN)
+  end
 end

data/lib/gitlab/dangerfiles/capability.rb

@@ -62,15 +62,13 @@ module Gitlab
 
       class ImportIntegrateBE < Capability
         def has_capability?(teammate)
-          kind == :reviewer &&
-            teammate.role.match?(/Backend Engineer.+Manage:Import and Integrate/)
+          kind == :reviewer && teammate.role.match?(/Backend Engineer.+:Import and Integrate/)
         end
       end
 
       class ImportIntegrateFE < Capability
         def has_capability?(teammate)
-          kind == :reviewer &&
-            teammate.role.match?(/Frontend Engineer.+Manage:Import and Integrate/)
+          kind == :reviewer && teammate.role.match?(/Frontend Engineer.+:Import and Integrate/)
         end
       end
 

data/lib/gitlab/dangerfiles/config.rb

@@ -21,31 +21,54 @@ module Gitlab
       #           match changed lines in files that match +filename_regex+. Used in `helper.changes_by_category`, `helper.changes`, and `helper.categories_for_file`.
       attr_accessor :files_to_category
 
+      # @!attribute custom_labels_for_categories
+      #   @return [{String => String}] A hash of the form +{ category_name => label }+.
+      #           Used in `helper.custom_labels_for_categories`.
+      attr_accessor :custom_labels_for_categories
+
+      # @!attribute disabled_roulette_categories
+      #   @return [Array] indicating which categories would be disabled for the simple roulette. Default to `[]` (all categories are enabled)
+      attr_accessor :disabled_roulette_categories
+
+      # Rule: changes_size
       # @!attribute code_size_thresholds
       #   @return [{ high: Integer, medium: Integer }] a hash of the form +{ high: 42, medium: 12 }+ where +:high+ is the lines changed threshold which triggers an error, and +:medium+ is the lines changed threshold which triggers a warning. Also, see +DEFAULT_CHANGES_SIZE_THRESHOLDS+ for the format of the hash.
       attr_accessor :code_size_thresholds
 
+      # Rule: commit_messages
       # @!attribute max_commits_count
       #   @return [Integer] the maximum number of allowed non-squashed/non-fixup commits for a given MR. A warning is triggered if the MR has more commits.
       attr_accessor :max_commits_count
 
-      # @!attribute disabled_roulette_categories
-      #   @return [Array] indicating which categories would be disabled for the simple roulette. Default to `[]` (all categories are enabled)
-      attr_accessor :disabled_roulette_categories
+      # Rule: duo_code_review
+      # @!attribute duo_code_review
+      #   @return [Symbol] whether a review from GitLab Duo Code is `:mandatory` or `:optional`. Default to `:optional`.
+      attr_accessor :duo_code_review
 
+      # Rule: simple_roulette
       # @!attribute included_optional_codeowners_sections_for_roulette
       #   @return [Array] indicating which optional codeowners sections should be included in roulette. Default to `[]`.
       attr_accessor :included_optional_codeowners_sections_for_roulette
 
+      # Rule: simple_roulette
       # @!attribute excluded_required_codeowners_sections_for_roulette
       #   @return [Array] indicating which required codeowners sections should be excluded from roulette. Default to `[]`.
       attr_accessor :excluded_required_codeowners_sections_for_roulette
 
+      # @!attribute auto_assign_for_roulette_roles
+      #   @return [Array<Symbol>] which roles to auto assign as reviewers, given roulette recommendations (:reviewer, :maintainer, or both). If empty, auto-assignment is disabled. Default to `[]`.
+      attr_accessor :auto_assign_for_roulette_roles
+
+      # @!attribute auto_assign_for_roulette_labels
+      #   @return [Array<String>] MR labels that allow auto reviewer assignment. If empty, applies to all MRs, provided :auto_assign_for_roulette_roles is not empty. Default to `[]`.
+      attr_accessor :auto_assign_for_roulette_labels
+
       DEFAULT_CHANGES_SIZE_THRESHOLDS = { high: 2_000, medium: 500 }.freeze
       DEFAULT_COMMIT_MESSAGES_MAX_COMMITS_COUNT = 10
 
       def initialize
         @files_to_category = {}
+        @custom_labels_for_categories = {}
         @project_root = nil
         @project_name = ENV["CI_PROJECT_NAME"]
         @ci_only_rules = []
@@ -54,6 +77,9 @@ module Gitlab
         @disabled_roulette_categories = []
         @included_optional_codeowners_sections_for_roulette = []
         @excluded_required_codeowners_sections_for_roulette = []
+        @auto_assign_for_roulette_roles = []
+        @auto_assign_for_roulette_labels = []
+        @duo_code_review = :optional
       end
     end
   end

data/lib/gitlab/dangerfiles/spec_helper.rb

@@ -168,8 +168,8 @@ RSpec.shared_context "with teammates" do
   let(:software_engineer_in_import_integrate_fe) do
     Gitlab::Dangerfiles::Teammate.new(
       "username" => "software-engineer-in-import-and-integrate-fe",
-      "name" => "Software Engineer in Import and Integrate FE",
-      "role" => "Frontend Engineer, Manage:Import and Integrate",
+      "name" => "Frontend Engineer in Import and Integrate",
+      "role" => "AnyRole Frontend Engineer, AnyStage:Import and Integrate",
       "projects" => { "gitlab" => "reviewer frontend" },
       "available" => true,
       "tz_offset_hours" => 2.0
@@ -179,8 +179,8 @@ RSpec.shared_context "with teammates" do
   let(:software_engineer_in_import_integrate_be) do
     Gitlab::Dangerfiles::Teammate.new(
       "username" => "software-engineer-in-import-and-integrate-be",
-      "name" => "Software Engineer in Import and Integrate BE",
-      "role" => "Backend Engineer, Manage:Import and Integrate",
+      "name" => "Backend Engineer in Import and Integrate",
+      "role" => "AnyRole Backend Engineer, AnyStage:Import and Integrate",
       "projects" => { "gitlab" => "reviewer backend" },
       "available" => true,
       "tz_offset_hours" => 2.0
@@ -224,7 +224,7 @@ RSpec.shared_context "with teammates" do
     Gitlab::Dangerfiles::Teammate.new(
       "username" => "import-and-integrate-backend-reviewer",
       "name" => "Import and Integrate BE engineer",
-      "role" => "Backend Engineer, Manage:Import and Integrate",
+      "role" => "AnyRole Backend Engineer, AnyStage:Import and Integrate",
       "projects" => { "gitlab" => "reviewer backend" },
       "available" => backend_reviewer_available,
       "tz_offset_hours" => 2.0
@@ -235,7 +235,7 @@ RSpec.shared_context "with teammates" do
     Gitlab::Dangerfiles::Teammate.new(
       "username" => "import-and-integrate-frontend-reviewer",
       "name" => "Import and Integrate FE engineer",
-      "role" => "Frontend Engineer, Manage:Import and Integrate",
+      "role" => "AnyRole Frontend Engineer, AnyStage:Import and Integrate",
       "projects" => { "gitlab" => "reviewer frontend" },
       "available" => true,
       "tz_offset_hours" => 2.0
@@ -285,7 +285,7 @@ RSpec.shared_context "with teammates" do
   end
 
   let(:teammate_json) { teammates.to_json }
-  let(:teammate_pedroms) { instance_double(Gitlab::Dangerfiles::Teammate, name: "Pedro") }
+  let(:ux_fallback_reviewer) { instance_double(Gitlab::Dangerfiles::Teammate) }
   let(:company_members) { Gitlab::Dangerfiles::Teammate.fetch_company_members }
 
   before do

data/lib/gitlab/dangerfiles/spinner.rb

@@ -18,7 +18,7 @@ module Gitlab
         @team_author = team_author
         @labels = labels
         @categories = categories.reject do |category|
-          import_and_integrate_reject_category?(category)
+          import_and_integrate_reject_category?(project, category, labels)
         end
         @random = random
         @ux_fallback_wider_community_reviewer = ux_fallback_wider_community_reviewer
@@ -98,12 +98,13 @@ module Gitlab
 
       attr_reader :random, :ux_fallback_wider_community_reviewer
 
+      # Reject Import and Integrate categories if the MR author has reviewing abilities for the category.
+      #
       # @param [String] category name
       # @return [Boolean]
-      def import_and_integrate_reject_category?(category)
-        # Reject Import and Integrate categories if the MR author has reviewing abilities for the category.
-        team_author&.import_integrate_be?(project, category, labels) ||
-          team_author&.import_integrate_fe?(project, category, labels)
+      def import_and_integrate_reject_category?(project, category, labels)
+        [:import_integrate_be, :import_integrate_fe].include?(category) && team_author &&
+          (team_author.reviewer?(project, category, labels) || team_author.maintainer?(project, category, labels))
       end
 
       # MR includes QA changes, but also other changes, and author isn't an SET

data/lib/gitlab/dangerfiles/teammate.rb

@@ -141,18 +141,6 @@ module Gitlab
         has_capability?(project, category, :maintainer, labels)
       end
 
-      def import_integrate_be?(project, category, labels)
-        return false unless category == :import_integrate_be
-
-        has_capability?(project, category, :reviewer, labels)
-      end
-
-      def import_integrate_fe?(project, category, labels)
-        return false unless category == :import_integrate_fe
-
-        has_capability?(project, category, :reviewer, labels)
-      end
-
       def markdown_name(author: nil)
         "#{@markdown_name}#{utc_offset_text(author)}"
       end

data/lib/gitlab/dangerfiles/version.rb

@@ -1,5 +1,5 @@
 module Gitlab
   module Dangerfiles
-    VERSION = "4.8.1"
+    VERSION = "4.10.0"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitlab-dangerfiles
 version: !ruby/object:Gem::Version
-  version: 4.8.1
+  version: 4.10.0
 platform: ruby
 authors:
 - GitLab
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-01-09 00:00:00.000000000 Z
+date: 2025-07-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -136,6 +136,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.8'
+- !ruby/object:Gem::Dependency
+  name: racc
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.8'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.8'
 - !ruby/object:Gem::Dependency
   name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement
@@ -225,12 +239,14 @@ files:
 - gitlab-dangerfiles.gemspec
 - lefthook.yml
 - lib/danger/plugins/changelog.rb
+- lib/danger/plugins/duo_code.rb
 - lib/danger/plugins/internal/helper.rb
 - lib/danger/plugins/roulette.rb
 - lib/danger/rules/changelog/Dangerfile
 - lib/danger/rules/changes_size/Dangerfile
 - lib/danger/rules/commit_messages/Dangerfile
 - lib/danger/rules/commits_counter/Dangerfile
+- lib/danger/rules/duo_code_review/Dangerfile
 - lib/danger/rules/metadata/Dangerfile
 - lib/danger/rules/simple_roulette/Dangerfile
 - lib/danger/rules/type_label/Dangerfile
@@ -282,7 +298,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.27
+rubygems_version: 3.5.22
 signing_key:
 specification_version: 4
 summary: This gem provides common Dangerfile and plugins for GitLab projects.