dependabot-common 0.216.0 → 0.216.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 645d08a2a5cfd122e1ec0bb4302d7c4e80bd0348e0c0e52d28dc210d196b963b
-  data.tar.gz: 25d81216a7b48d60b332d241e06f3a7dcc384b06ea3f5fac1dcc02949da226dc
+  metadata.gz: cc44d20c1eea5ffcfebda79597511add6d56ecad33107329dd60223d39f14185
+  data.tar.gz: 1f483e056ca6d4e065a6b81d8d2491afda80f2c7820b1dee3ece09efc3ce0fa5
 SHA512:
-  metadata.gz: 8ef726592da7c2ff04784322801d12902ab42613e53b084687acdbfb479982e60a5303cb6e7f249e5525547a7b7681af48c3afe9df16cfda435b771e5e3f84ad
-  data.tar.gz: ef3a1010b2dd5736c9595754b1eb1cf1cca4cd7eecdfae551b97a2d84c71952ccad43bedafab0f9b6cea07892afbff41b766d9d8941ae10bcd7eedb434970aa3
+  metadata.gz: de74b6faf4093eb8ccc59c99e85f561f0967d8f453cd20fd52f973a1def74c0fc588f88e8251adae1bb0a62a17efb4c81c80b7dedcc9098d79c290e36f43b3f8
+  data.tar.gz: cfc902894c13a7c0631def4ef00439e0eb0b8bb668224954113cdf48d5a4010d083efe5d650996a90c4bda4a5e2410d6d0a002d3cfb18837b9f46a93852a43e5

data/lib/dependabot/dependency_group.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+module Dependabot
+  class DependencyGroup
+    attr_reader :name, :rules
+
+    def initialize(name:, rules:)
+      @name = name
+      @rules = rules
+    end
+  end
+end

data/lib/dependabot/errors.rb

@@ -80,7 +80,7 @@ module Dependabot
 
     def initialize(file_path, msg = nil)
       @file_path = file_path
-      super("#{file_path} not found" || msg)
+      super(msg || "#{file_path} not found")
     end
 
     def file_name
@@ -98,7 +98,7 @@ module Dependabot
 
     def initialize(file_path, msg = nil)
       @file_path = file_path
-      super(msg)
+      super(msg || "#{file_path} not parseable")
     end
 
     def file_name

data/lib/dependabot/pull_request_creator/branch_namer/dependency_group_strategy.rb

@@ -0,0 +1,55 @@
+# frozen_string_literal: true
+
+module Dependabot
+  class PullRequestCreator
+    class BranchNamer
+      class DependencyGroupStrategy
+        def initialize(dependencies:, files:, target_branch:, dependency_group:,
+                       separator: "/", prefix: "dependabot", max_length: nil)
+          @dependencies     = dependencies
+          @files            = files
+          @target_branch    = target_branch
+          @dependency_group = dependency_group
+          @separator        = separator
+          @prefix           = prefix
+          @max_length       = max_length
+        end
+
+        # FIXME: Incorporate max_length truncation once we allow user config
+        #
+        # For now, we are using a placeholder DependencyGroup with a
+        # fixed-length name, so we can punt on handling truncation until
+        # we determine the strict validation rules for names
+        def new_branch_name
+          File.join(prefixes, dependency_group.name, prototype_suffix).gsub("/", separator)
+        end
+
+        private
+
+        attr_reader :dependencies, :dependency_group, :files, :target_branch, :separator, :prefix, :max_length
+
+        def prefixes
+          [
+            prefix,
+            package_manager,
+            directory,
+            target_branch
+          ].compact
+        end
+
+        # FIXME: Remove once grouped PRs can supersede each other
+        def prototype_suffix
+          "prototype-#{Time.now.utc.to_i}"
+        end
+
+        def package_manager
+          dependencies.first.package_manager
+        end
+
+        def directory
+          files.first.directory.tr(" ", "-")
+        end
+      end
+    end
+  end
+end

data/lib/dependabot/pull_request_creator/branch_namer.rb

@@ -5,18 +5,19 @@ require "digest"
 require "dependabot/metadata_finders"
 require "dependabot/pull_request_creator"
 require "dependabot/pull_request_creator/branch_namer/solo_strategy"
+require "dependabot/pull_request_creator/branch_namer/dependency_group_strategy"
 
 module Dependabot
   class PullRequestCreator
     class BranchNamer
-      attr_reader :dependencies, :files, :target_branch, :separator, :prefix, :max_length, :group_rule
+      attr_reader :dependencies, :files, :target_branch, :separator, :prefix, :max_length, :dependency_group
 
-      def initialize(dependencies:, files:, target_branch:, group_rule: nil,
+      def initialize(dependencies:, files:, target_branch:, dependency_group: nil,
                      separator: "/", prefix: "dependabot", max_length: nil)
         @dependencies  = dependencies
         @files         = files
         @target_branch = target_branch
-        @group_rule    = group_rule
+        @dependency_group = dependency_group
         @separator     = separator
         @prefix        = prefix
         @max_length    = max_length
@@ -30,7 +31,7 @@ module Dependabot
 
       def strategy
         @strategy ||=
-          if group_rule.nil?
+          if dependency_group.nil?
             SoloStrategy.new(
               dependencies: dependencies,
               files: files,
@@ -40,11 +41,11 @@ module Dependabot
               max_length: max_length
             )
           else
-            GroupRuleStrategy.new(
+            DependencyGroupStrategy.new(
               dependencies: dependencies,
               files: files,
               target_branch: target_branch,
-              group_rule: group_rule,
+              dependency_group: dependency_group,
               separator: separator,
               prefix: prefix,
               max_length: max_length

data/lib/dependabot/pull_request_creator/message_builder.rb

@@ -52,7 +52,7 @@ module Dependabot
       end
 
       def pr_message
-        suffixed_pr_message_header + commit_message_intro + \
+        suffixed_pr_message_header + commit_message_intro +
           metadata_cascades + prefixed_pr_message_footer
       rescue StandardError => e
         Dependabot.logger.error("Error while generating PR message: #{e.message}")

data/lib/dependabot/pull_request_creator.rb

@@ -49,7 +49,7 @@ module Dependabot
                 :commit_message_options, :vulnerabilities_fixed,
                 :reviewers, :assignees, :milestone, :branch_name_separator,
                 :branch_name_prefix, :branch_name_max_length, :github_redirection_service,
-                :custom_headers, :provider_metadata
+                :custom_headers, :provider_metadata, :dependency_group
 
     def initialize(source:, base_commit:, dependencies:, files:, credentials:,
                    pr_message_header: nil, pr_message_footer: nil,
@@ -61,7 +61,7 @@ module Dependabot
                    automerge_candidate: false,
                    github_redirection_service: DEFAULT_GITHUB_REDIRECTION_SERVICE,
                    custom_headers: nil, require_up_to_date_base: false,
-                   provider_metadata: {}, message: nil)
+                   provider_metadata: {}, message: nil, dependency_group: nil)
       @dependencies               = dependencies
       @source                     = source
       @base_commit                = base_commit
@@ -87,6 +87,7 @@ module Dependabot
       @require_up_to_date_base    = require_up_to_date_base
       @provider_metadata          = provider_metadata
       @message                    = message
+      @dependency_group           = dependency_group
 
       check_dependencies_have_previous_version
     end
@@ -235,7 +236,7 @@ module Dependabot
           dependencies: dependencies,
           files: files,
           target_branch: source.branch,
-          group_rule: nil,
+          dependency_group: dependency_group,
           separator: branch_name_separator,
           prefix: branch_name_prefix,
           max_length: branch_name_max_length

data/lib/dependabot.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Dependabot
-  VERSION = "0.216.0"
+  VERSION = "0.216.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dependabot-common
 version: !ruby/object:Gem::Version
-  version: 0.216.0
+  version: 0.216.1
 platform: ruby
 authors:
 - Dependabot
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-04-12 00:00:00.000000000 Z
+date: 2023-04-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aws-sdk-codecommit
@@ -318,14 +318,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.48.0
+        version: 1.50.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.48.0
+        version: 1.50.0
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
@@ -432,6 +432,7 @@ files:
 - lib/dependabot/config/update_config.rb
 - lib/dependabot/dependency.rb
 - lib/dependabot/dependency_file.rb
+- lib/dependabot/dependency_group.rb
 - lib/dependabot/errors.rb
 - lib/dependabot/experiments.rb
 - lib/dependabot/file_fetchers.rb
@@ -447,7 +448,6 @@ files:
 - lib/dependabot/file_updaters/vendor_updater.rb
 - lib/dependabot/git_commit_checker.rb
 - lib/dependabot/git_metadata_fetcher.rb
-- lib/dependabot/group_rule.rb
 - lib/dependabot/logger.rb
 - lib/dependabot/metadata_finders.rb
 - lib/dependabot/metadata_finders/README.md
@@ -460,7 +460,7 @@ files:
 - lib/dependabot/pull_request_creator/azure.rb
 - lib/dependabot/pull_request_creator/bitbucket.rb
 - lib/dependabot/pull_request_creator/branch_namer.rb
-- lib/dependabot/pull_request_creator/branch_namer/group_rule_strategy.rb
+- lib/dependabot/pull_request_creator/branch_namer/dependency_group_strategy.rb
 - lib/dependabot/pull_request_creator/branch_namer/solo_strategy.rb
 - lib/dependabot/pull_request_creator/codecommit.rb
 - lib/dependabot/pull_request_creator/commit_signer.rb

data/lib/dependabot/group_rule.rb

@@ -1,11 +0,0 @@
-# frozen_string_literal: true
-
-module Dependabot
-  class GroupRule
-    attr_reader :name
-
-    def initialize(name)
-      @name = name
-    end
-  end
-end

data/lib/dependabot/pull_request_creator/branch_namer/group_rule_strategy.rb

@@ -1,28 +0,0 @@
-# frozen_string_literal: true
-
-module Dependabot
-  class PullRequestCreator
-    class BranchNamer
-      class GroupRuleStrategy
-        def initialize(dependencies:, files:, target_branch:, group_rule:,
-                       separator: "/", prefix: "dependabot", max_length: nil)
-          @dependencies  = dependencies
-          @files         = files
-          @target_branch = target_branch
-          @group_rule    = group_rule
-          @separator     = separator
-          @prefix        = prefix
-          @max_length    = max_length
-        end
-
-        def new_branch_name
-          group_rule.name
-        end
-
-        private
-
-        attr_reader :group_rule
-      end
-    end
-  end
-end