buildkite-builder 3.8.3 → 4.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2591489e3b57711b38926cf558ee362c2c579086b46ff800fccffb8db4598a92
-  data.tar.gz: cc12ee969c8e846008f13eea3ad61de0efa32d8706bda83ce76a3bf7e8b7772f
+  metadata.gz: 3d9385a6b39f83697f501167fac9988d2e0b351a3e1b877b24e6c93670959d27
+  data.tar.gz: d5fe308edf2e426eb4c7be9bf9bf5996056533eb70724e18d438109732101492
 SHA512:
-  metadata.gz: 6901d0afda1b70080615f5937ef7fa17c90023f1141606e4adf8f77b5ac3252da2f656070b22528b8eea2703686fb2e7af30575f6075e2d19e730a4b4c0df9e5
-  data.tar.gz: 39d6c40ec9a8f194997af75eb4776753d3b6904bbd19783e1592f92b260f918ccbda81c764efca332305e1f67073bcf7f060d52931a8738c1a2a7a56ffa5a924
+  metadata.gz: c406b7822bd571773bc3e2f29bf71b1067cacf1cf434b92457c8936a64757efdc48c8f8a63f9f56fc7885d5505d2505bd6f9ac8c9447f1f90d992b263508b3cb
+  data.tar.gz: ddfc34afe276f2716b86d5db4fe07a6854ef0135dd93f8e4b0bd8d5fb14aa07ffcd8c892661823bad17b90e6bcea3b977a7d5ca46694429250e7c9f818ea9baa

data/README.md

@@ -135,36 +135,6 @@ Buildkite::Builder.pipeline do
 end
 ```
 
-### Subpipeline
-
-While triggering another pipeline, you can predefine subpipeline's steps using `pipeline(NAME_OF_SUBPIPELINE)` in the main pipeline's `pipeline.rb` file and share with main pipeline's plugins and templates definition.
-
-`.buildkite/pipelines/pipeline-triggerer/pipeline.rb`
-
-```ruby
-Buildkite::Builder.pipeline do
-  pipeline('rspec-pipeline') do
-    command do
-      label "Run RSpec in separate pipeline"
-    end
-  end
-end
-```
-
-Inside your Buildkite pipeline setup, you can do the following:
-
-In `https://buildkite.com/your-org/rspec-pipeline/steps`
-
-```yaml
-steps:
-  - label: ":pipeline:"
-    commands:
-      - buildkite-agent artifact download $BKB_SUBPIPELINE_FILE . --build $BUILDKITE_TRIGGERED_FROM_BUILD_ID
-      - buildkite-agent pipeline upload $BKB_SUBPIPELINE_FILE
-```
-
-This will upload the pregenerated `pipeline.yml` to `rspec-pipeline`.
-
 ## Development
 
 After checking out the repo, run `bin/setup` to install dependencies. Then, run `rake spec` to run the tests. You can also run `bin/console` for an interactive prompt that will allow you to experiment.

data/VERSION

@@ -1 +1 @@
-3.8.3
+4.0.0

data/lib/buildkite/builder/extension_manager.rb

@@ -8,7 +8,7 @@ module Buildkite
         @context = context
         @extensions = {}
 
-        @loader = Loaders::Extensions.load(@context.root)
+        Loaders::Extensions.load(@context.root)
       end
 
       def use(extension, native: false, **args, &block)
@@ -34,6 +34,10 @@ module Buildkite
         @extensions.fetch(klass)
       end
 
+      def all
+        @extensions.keys
+      end
+
       private
 
       def log

data/lib/buildkite/builder/extensions/plugins.rb

@@ -0,0 +1,36 @@
+module Buildkite
+  module Builder
+    module Extensions
+      class Plugins < Extension
+        attr_reader :manager
+
+        dsl do
+          def plugin(name, uri, default_attributes = {})
+            context.extensions.find(Buildkite::Builder::Extensions::Plugins).manager.add(name, uri, default_attributes)
+          end
+        end
+
+        def prepare
+          @manager = PluginManager.new
+        end
+
+        def build
+          context.data.steps.each(:command) do |step|
+            next unless step.has?(:plugins)
+
+            step.get(:plugins).map! do |plugin|
+              resource, attributes = extract_resource_and_attributes(plugin)
+              resource.is_a?(Symbol) ? manager.build(resource, attributes) : plugin
+            end
+          end
+        end
+
+        private
+
+        def extract_resource_and_attributes(plugin)
+          [plugin.keys.first, plugin.values.first]
+        end
+      end
+    end
+  end
+end

data/lib/buildkite/builder/extensions/steps.rb

@@ -2,61 +2,76 @@ module Buildkite
   module Builder
     module Extensions
       class Steps < Extension
+        attr_reader :templates
+
         def prepare
-          context.data.steps = StepCollection.new(
-            TemplateManager.new(context.root),
-            PluginManager.new
-          )
+          @templates = TemplateManager.new(context.root)
+          context.data.steps = StepCollection.new
         end
 
-        dsl do
-          def group(label = nil, emoji: nil, &block)
-            raise "Group does not allow nested in another Group" if context.is_a?(Group)
+        def build_step(step_class, template_name, **args, &block)
+          template = @templates.find(template_name)
 
-            if emoji
-              emoji = Array(emoji).map { |name| ":#{name}:" }.join
-              label = [emoji, label].compact.join(' ')
-            end
+          step_class.new(**args).tap do |step|
+            step.process(template) if template
+            step.process(block) if block_given?
 
-            context.data.steps.push(Buildkite::Builder::Group.new(label, context, &block))
+            if @current_group
+              @current_group.steps.push(step)
+            else
+              context.data.steps.push(step)
+            end
           end
+        end
+
+        def with_group(group, &block)
+          raise "Group cannot be nested" if @current_group
 
-          def plugin(name, uri, default_attributes = {})
-            context.data.steps.plugins.add(name, uri, default_attributes)
+          @current_group = group
+
+          group.process(block)
+          context.data.steps.push(group).last
+        ensure
+          @current_group = nil
+        end
+
+        dsl do
+          def group(&block)
+            context.extensions.find(Steps).with_group(Pipelines::Steps::Group.new(context), &block)
           end
 
           def block(template = nil, **args, &block)
-            context.data.steps.add(Pipelines::Steps::Block, template, **args, &block)
+            context.extensions.find(Steps).build_step(Pipelines::Steps::Block, template, **args, &block)
           end
 
           def command(template = nil, **args, &block)
-            context.data.steps.add(Pipelines::Steps::Command, template, **args, &block)
+            context.extensions.find(Steps).build_step(Pipelines::Steps::Command, template, **args, &block)
           end
 
           def input(template = nil, **args, &block)
-            context.data.steps.add(Pipelines::Steps::Input, template, **args, &block)
+            context.extensions.find(Steps).build_step(Pipelines::Steps::Input, template, **args, &block)
           end
 
           def trigger(template = nil, **args, &block)
-            context.data.steps.add(Pipelines::Steps::Trigger, template, **args, &block)
+            context.extensions.find(Steps).build_step(Pipelines::Steps::Trigger, template, **args, &block)
           end
 
           def skip(template = nil, **args, &block)
-            step = context.data.steps.add(Pipelines::Steps::Skip, template, **args, &block)
-            # A skip step has a nil/noop command.
-            step.command(nil)
-            # Always set the skip attribute if it's in a falsey state.
-            step.skip(true) if !step.get(:skip) || step.skip.empty?
-            step
+            context.extensions.find(Steps).build_step(Pipelines::Steps::Skip, template, **args, &block).tap do |step|
+              # A skip step has a nil/noop command.
+              step.command(nil)
+              # Always set the skip attribute if it's in a falsey state.
+              step.skip(true) if !step.get(:skip) || step.skip.empty?
+            end
           end
 
           def wait(attributes = {}, &block)
-            step = context.data.steps.add(Pipelines::Steps::Wait, &block)
-            step.wait(nil)
-            attributes.each do |key, value|
-              step.set(key, value)
+            context.extensions.find(Steps).build_step(Pipelines::Steps::Wait, nil, &block).tap do |step|
+              step.wait(nil)
+              attributes.each do |key, value|
+                step.set(key, value)
+              end
             end
-            step
           end
         end
       end

data/lib/buildkite/builder/extensions.rb

@@ -6,8 +6,8 @@ module Buildkite
       autoload :Env, File.expand_path('extensions/env', __dir__)
       autoload :Lib, File.expand_path('extensions/lib', __dir__)
       autoload :Notify, File.expand_path('extensions/notify', __dir__)
-      autoload :SubPipelines, File.expand_path('extensions/sub_pipelines', __dir__)
       autoload :Steps, File.expand_path('extensions/steps', __dir__)
+      autoload :Plugins, File.expand_path('extensions/plugins', __dir__)
       autoload :Use, File.expand_path('extensions/use', __dir__)
     end
   end

data/lib/buildkite/builder/pipeline.rb

@@ -19,7 +19,6 @@ module Buildkite
       attr_reader :logger,
                   :root,
                   :artifacts,
-                  :plugins,
                   :dsl,
                   :data,
                   :extensions
@@ -28,7 +27,6 @@ module Buildkite
         @root = root
         @logger = logger || Logger.new(File::NULL)
         @artifacts = []
-        @plugins = {}
         @dsl = Dsl.new(self)
         @extensions = ExtensionManager.new(self)
         @data = Data.new
@@ -38,7 +36,7 @@ module Buildkite
         use(Extensions::Env)
         use(Extensions::Notify)
         use(Extensions::Steps)
-        use(Extensions::SubPipelines)
+        use(Extensions::Plugins)
       end
 
       def upload

data/lib/buildkite/builder/plugin.rb

@@ -3,16 +3,16 @@
 module Buildkite
   module Builder
     class Plugin
-      attr_reader :uri, :source, :version, :attributes
+      attr_reader \
+        :uri,
+        :source,
+        :version,
+        :default_attributes
 
-      def initialize(uri, attributes = {})
+      def initialize(uri, default_attributes = {})
         @uri = uri
         @source, @version = uri.split('#')
-        @attributes = attributes
-      end
-
-      def to_h
-        Buildkite::Pipelines::Helpers.sanitize(uri => (attributes&.empty? ? nil : attributes))
+        @default_attributes = default_attributes
       end
     end
   end

data/lib/buildkite/builder/plugin_manager.rb

@@ -7,23 +7,16 @@ module Buildkite
 
       def add(name, uri, default_attributes = {})
         name = name.to_s
+        raise(ArgumentError, "Plugin already defined: #{name}") if @plugins.key?(name)
 
-        if @plugins.key?(name)
-          raise ArgumentError, "Plugin already defined: #{name}"
-        end
-
-        @plugins[name] = {
-          uri: uri,
-          default_attributes: default_attributes
-        }
+        @plugins[name] = Plugin.new(uri, default_attributes)
       end
 
-      def fetch(name)
-        @plugins[name]
-      end
+      def build(name, attributes = {})
+        plugin = @plugins[name.to_s]
+        raise(ArgumentError, "Plugin is not registered: #{resource}") unless plugin
 
-      def to_definition
-        # No-op
+        { plugin.uri => plugin.default_attributes.merge(attributes) }
       end
     end
   end

data/lib/buildkite/builder/step_collection.rb

@@ -1,13 +1,9 @@
 module Buildkite
   module Builder
     class StepCollection
-      attr_reader :templates
-      attr_reader :plugins
       attr_reader :steps
 
-      def initialize(templates, plugins)
-        @templates = templates
-        @plugins = plugins
+      def initialize
         @steps = []
       end
 
@@ -15,12 +11,12 @@ module Buildkite
         types = types.flatten
 
         @steps.each do |step|
-          if types.include?(step.class.to_sym)
-            yield step
-          elsif step.is_a?(Group)
-            step.data.steps.each(*types) do |step|
+          if step.class.to_sym == :group
+            step.steps.each(*types) do |step|
               yield step
             end
+          elsif types.include?(step.class.to_sym)
+            yield step
           elsif types.empty?
             yield step
           end
@@ -35,10 +31,6 @@ module Buildkite
         find(key) || raise(ArgumentError, "Can't find step with key: #{key}")
       end
 
-      def add(step_class, template = nil, **args, &block)
-        @steps.push(step_class.new(self, template, **args, &block)).last
-      end
-
       def push(step)
         @steps.push(step)
       end

data/lib/buildkite/builder.rb

@@ -5,7 +5,6 @@ require 'pathname'
 module Buildkite
   module Builder
     autoload :Commands, File.expand_path('builder/commands', __dir__)
-    autoload :Group, File.expand_path('builder/group', __dir__)
     autoload :Pipeline, File.expand_path('builder/pipeline', __dir__)
     autoload :Definition, File.expand_path('builder/definition', __dir__)
     autoload :Data, File.expand_path('builder/data', __dir__)
@@ -18,7 +17,6 @@ module Buildkite
     autoload :Processors, File.expand_path('builder/processors', __dir__)
     autoload :Rainbow, File.expand_path('builder/rainbow', __dir__)
     autoload :Plugin, File.expand_path('builder/plugin', __dir__)
-    autoload :PluginCollection, File.expand_path('builder/plugin_collection', __dir__)
     autoload :StepCollection, File.expand_path('builder/step_collection', __dir__)
     autoload :PipelineCollection, File.expand_path('builder/pipeline_collection', __dir__)
     autoload :TemplateManager, File.expand_path('builder/template_manager', __dir__)

data/lib/buildkite/pipelines/helpers/plugins.rb

@@ -5,12 +5,7 @@ module Buildkite
     module Helpers
       module Plugins
         def plugin(name_or_source, plugin_attributes = {})
-          attributes['plugins'] ||= Buildkite::Builder::PluginCollection.new(step_collection.plugins)
-          attributes['plugins'].add(name_or_source, plugin_attributes)
-        end
-
-        def plugins
-          attributes['plugins']
+          append(:plugins, { name_or_source => plugin_attributes })
         end
       end
     end

data/lib/buildkite/pipelines/steps/abstract.rb

@@ -11,20 +11,16 @@ module Buildkite
 
         def_delegator :@context, :data
 
-        attr_reader :template
-        attr_reader :step_collection
-
         def self.to_sym
           name.split('::').last.downcase.to_sym
         end
 
-        def initialize(step_collection, template_name, **args, &block)
-          @step_collection = step_collection
-          @template = step_collection.templates.find(template_name)
+        def initialize(**args)
           @context = StepContext.new(self, **args)
+        end
 
-          instance_exec(@context, &template) if template
-          instance_exec(@context, &block) if block_given?
+        def process(block)
+          instance_exec(@context, &block)
         end
       end
     end

data/lib/buildkite/pipelines/steps/group.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+module Buildkite
+  module Pipelines
+    module Steps
+      class Group < Abstract
+        attribute :depends_on, append: true
+        attribute :key
+        attribute :label
+
+        attr_reader :steps
+
+        def initialize(pipeline, **args)
+          @pipeline = pipeline
+          @context = StepContext.new(self, **args)
+          @steps = Buildkite::Builder::StepCollection.new
+        end
+
+        def method_missing(method_name, ...)
+          if @pipeline.dsl.respond_to?(method_name)
+            @pipeline.dsl.public_send(method_name, ...)
+          else
+            super
+          end
+        end
+
+        def respond_to_missing?(...)
+          @pipeline.dsl.respond_to?(...) || super
+        end
+
+        def to_h
+          super.merge(group: nil, steps: steps.to_definition)
+        end
+      end
+    end
+  end
+end

data/lib/buildkite/pipelines/steps.rb

@@ -6,6 +6,7 @@ module Buildkite
       autoload :Abstract, File.expand_path('steps/abstract', __dir__)
       autoload :Block, File.expand_path('steps/block', __dir__)
       autoload :Command, File.expand_path('steps/command', __dir__)
+      autoload :Group, File.expand_path('steps/group', __dir__)
       autoload :Input, File.expand_path('steps/input', __dir__)
       autoload :Skip, File.expand_path('steps/skip', __dir__)
       autoload :Trigger, File.expand_path('steps/trigger', __dir__)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: buildkite-builder
 version: !ruby/object:Gem::Version
-  version: 3.8.3
+  version: 4.0.0
 platform: ruby
 authors:
 - Ngan Pham
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-12-09 00:00:00.000000000 Z
+date: 2022-12-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rainbow
@@ -113,19 +113,16 @@ files:
 - lib/buildkite/builder/extensions/env.rb
 - lib/buildkite/builder/extensions/lib.rb
 - lib/buildkite/builder/extensions/notify.rb
+- lib/buildkite/builder/extensions/plugins.rb
 - lib/buildkite/builder/extensions/steps.rb
-- lib/buildkite/builder/extensions/sub_pipelines.rb
 - lib/buildkite/builder/extensions/use.rb
-- lib/buildkite/builder/group.rb
 - lib/buildkite/builder/loaders.rb
 - lib/buildkite/builder/loaders/abstract.rb
 - lib/buildkite/builder/loaders/extensions.rb
 - lib/buildkite/builder/loaders/templates.rb
 - lib/buildkite/builder/logging_utils.rb
 - lib/buildkite/builder/pipeline.rb
-- lib/buildkite/builder/pipeline_collection.rb
 - lib/buildkite/builder/plugin.rb
-- lib/buildkite/builder/plugin_collection.rb
 - lib/buildkite/builder/plugin_manager.rb
 - lib/buildkite/builder/rainbow.rb
 - lib/buildkite/builder/step_collection.rb
@@ -150,6 +147,7 @@ files:
 - lib/buildkite/pipelines/steps/abstract.rb
 - lib/buildkite/pipelines/steps/block.rb
 - lib/buildkite/pipelines/steps/command.rb
+- lib/buildkite/pipelines/steps/group.rb
 - lib/buildkite/pipelines/steps/input.rb
 - lib/buildkite/pipelines/steps/skip.rb
 - lib/buildkite/pipelines/steps/trigger.rb
@@ -177,7 +175,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
+rubygems_version: 3.3.26
 signing_key:
 specification_version: 4
 summary: A gem for programmatically creating Buildkite pipelines.

data/lib/buildkite/builder/extensions/sub_pipelines.rb

@@ -1,107 +0,0 @@
-require 'securerandom'
-
-module Buildkite
-  module Builder
-    module Extensions
-      class SubPipelines < Extension
-        class Pipeline
-          include Buildkite::Pipelines::Attributes
-
-          attr_reader :data, :name, :dsl
-
-          # These attributes are for triggered step
-          attribute :label
-          attribute :key
-          attribute :skip
-          attribute :if, as: :condition
-          attribute :depends_on, append: true
-          attribute :allow_dependency_failure
-          attribute :branches
-          attribute :async
-          attribute :build
-
-          def self.to_sym
-            name.split('::').last.downcase.to_sym
-          end
-
-          def initialize(name, context, &block)
-            @context = context
-            @name = name
-            @data = Data.new
-            @data.steps = StepCollection.new(
-              context.data.steps.templates,
-              context.data.steps.plugins
-            )
-            @data.notify = []
-            @data.env = {}
-
-            # Use `clone` to copy over dsl's extended extensions
-            @dsl = context.dsl.clone
-            # Override dsl context to current pipeline
-            @dsl.instance_variable_set(:@context, self)
-
-            instance_eval(&block) if block_given?
-            self
-          end
-
-          def to_h
-            # Merge envs from main pipeline, since ruby does not have `reverse_merge` and
-            # `data` does not allow keys override, we have to reset the data hash per key.
-            @context.data.env.merge(data.env).each do |key, value|
-              data.env[key] = value
-            end
-            data.to_definition
-          end
-
-          def method_missing(method_name, *args, **kwargs, &_block)
-            @dsl.public_send(method_name, *args, **kwargs, &_block)
-          end
-
-          def pipeline_yml
-            @pipeline_yml ||= "tmp/buildkite-builder/#{SecureRandom.urlsafe_base64}.yml"
-          end
-        end
-
-        def prepare
-          context.data.pipelines = PipelineCollection.new(context.artifacts)
-        end
-
-        dsl do
-          def pipeline(name, &block)
-            raise "Subpipeline must have a name" if name.empty?
-            raise "Subpipeline does not allow nested in another Subpipeline" if context.is_a?(Buildkite::Builder::Extensions::SubPipelines::Pipeline)
-
-            sub_pipeline = Buildkite::Builder::Extensions::SubPipelines::Pipeline.new(name, context, &block)
-            context.data.pipelines.add(sub_pipeline)
-
-            trigger_step = context.data.steps.add(Pipelines::Steps::Trigger)
-            trigger_step.trigger(name)
-
-            build_options = {
-              message: '${BUILDKITE_MESSAGE}',
-              commit: '${BUILDKITE_COMMIT}',
-              branch: '${BUILDKITE_BRANCH}',
-              env: {
-                BUILDKITE_PULL_REQUEST: '${BUILDKITE_PULL_REQUEST}',
-                BUILDKITE_PULL_REQUEST_BASE_BRANCH: '${BUILDKITE_PULL_REQUEST_BASE_BRANCH}',
-                BUILDKITE_PULL_REQUEST_REPO: '${BUILDKITE_PULL_REQUEST_REPO}',
-                BKB_SUBPIPELINE_FILE: sub_pipeline.pipeline_yml
-              }
-            }
-            build_options.merge!(sub_pipeline.build) if sub_pipeline.build
-
-            trigger_step.build(build_options)
-            trigger_step.key(sub_pipeline.key || "subpipeline_#{name}_#{context.data.pipelines.count}")
-            trigger_step.label(sub_pipeline.label || name.capitalize)
-            trigger_step.async(sub_pipeline.async || false)
-            trigger_step.branches(sub_pipeline.branches) if sub_pipeline.branches
-            trigger_step.condition(sub_pipeline.condition) if sub_pipeline.condition
-            trigger_step.depends_on(*sub_pipeline.get('depends_on')) if sub_pipeline.get('depends_on')
-            trigger_step.allow_dependency_failure(sub_pipeline.allow_dependency_failure || false)
-            trigger_step.skip(sub_pipeline.skip || false)
-          end
-        end
-      end
-    end
-  end
-end

data/lib/buildkite/builder/group.rb

@@ -1,44 +0,0 @@
-module Buildkite
-  module Builder
-    class Group
-      include Buildkite::Pipelines::Attributes
-
-      attr_reader :label
-      attr_reader :data
-
-      attribute :depends_on, append: true
-      attribute :key
-
-      def self.to_sym
-        name.split('::').last.downcase.to_sym
-      end
-
-      def initialize(label, context, &block)
-        @label = label
-        @data = Data.new
-        @data.steps = StepCollection.new(
-          context.data.steps.templates,
-          context.data.steps.plugins
-        )
-        @data.notify = []
-
-        # Use `clone` to copy over dsl's extended extensions
-        @dsl = context.dsl.clone
-        # Override dsl context to current group
-        @dsl.instance_variable_set(:@context, self)
-
-        instance_eval(&block) if block_given?
-        self
-      end
-
-      def to_h
-        attributes = super
-        { group: label }.merge(attributes).merge(data.to_definition)
-      end
-
-      def method_missing(method_name, *args, **kwargs, &_block)
-        @dsl.public_send(method_name, *args, **kwargs, &_block)
-      end
-    end
-  end
-end

data/lib/buildkite/builder/pipeline_collection.rb

@@ -1,40 +0,0 @@
-require "forwardable"
-
-module Buildkite
-  module Builder
-    class PipelineCollection
-      extend Forwardable
-
-      attr_reader :pipelines
-
-      def_delegator :@pipelines, :count
-      def_delegator :@pipelines, :each
-
-      def initialize(artifacts)
-        @artifacts = artifacts
-        @pipelines = []
-      end
-
-      def add(pipeline)
-        unless pipeline.is_a?(Buildkite::Builder::Extensions::SubPipelines::Pipeline)
-          raise "`#{pipeline}` must be a Buildkite::Builder::Extensions::SubPipelines::Pipeline"
-        end
-
-        pipelines << pipeline
-      end
-
-      def to_definition
-        # Instead of generates pipeline.yml, subpipelines save generated file to artifacts
-        pipelines.each do |pipeline|
-          file = Pathname.new(pipeline.pipeline_yml)
-          file.dirname.mkpath
-          file.write(YAML.dump(Pipelines::Helpers.sanitize(pipeline.to_h)))
-
-          @artifacts << file
-        end
-
-        nil
-      end
-    end
-  end
-end

data/lib/buildkite/builder/plugin_collection.rb

@@ -1,49 +0,0 @@
-module Buildkite
-  module Builder
-    class PluginCollection
-      attr_reader :plugin_manager
-
-      def initialize(plugin_manager)
-        @plugin_manager = plugin_manager
-        @collection = []
-      end
-
-      def add(resource, attributes = {})
-        plugin =
-          case resource
-          when Symbol
-            registered_plugin = plugin_manager.fetch(resource.to_s)
-
-            raise ArgumentError, "Plugin `#{resource}` does not exist" unless registered_plugin
-
-            Plugin.new(registered_plugin[:uri], registered_plugin[:default_attributes].merge(attributes))
-          when String
-            Plugin.new(resource, attributes)
-          when Plugin
-            resource
-          else
-            raise ArgumentError, "Unknown plugin `#{resource.inspect}`"
-          end
-
-        @collection.push(plugin).last
-      end
-
-      def find(source)
-        source_string =
-          case source
-          when String then source
-          when Plugin then source.source
-          else raise ArgumentError, "Unknown source #{source.inspect}"
-          end
-
-        @collection.select do |plugin|
-          plugin.source == source_string
-        end
-      end
-
-      def to_definition
-        @collection.map(&:to_h)
-      end
-    end
-  end
-end