git-lint 3.3.0 → 4.0.0

data/lib/git/lint/analyzers/commit_trailer_collaborator_email.rb

@@ -5,14 +5,16 @@ module Git
     module Analyzers
       # Analyzes commit trailer collaborator email address format.
       class CommitTrailerCollaboratorEmail < Abstract
+        # rubocop:disable Metrics/ParameterLists
         def initialize commit,
                        parser: Parsers::Trailers::Collaborator,
-                       validator: Validators::Email
-
-          super commit
+                       validator: Validators::Email,
+                       **dependencies
+          super commit, **dependencies
           @parser = parser
           @validator = validator
         end
+        # rubocop:enable Metrics/ParameterLists
 
         def valid? = affected_commit_trailers.empty?
 

data/lib/git/lint/analyzers/commit_trailer_collaborator_key.rb

@@ -5,8 +5,8 @@ module Git
     module Analyzers
       # Analyzes commit trailer collaborator key usage.
       class CommitTrailerCollaboratorKey < Abstract
-        def initialize commit, parser: Parsers::Trailers::Collaborator
-          super commit
+        def initialize commit, parser: Parsers::Trailers::Collaborator, **dependencies
+          super commit, **dependencies
           @parser = parser
         end
 

data/lib/git/lint/analyzers/commit_trailer_collaborator_name.rb

@@ -5,14 +5,16 @@ module Git
     module Analyzers
       # Analyzes commit trailer collaborator name construction.
       class CommitTrailerCollaboratorName < Abstract
+        # rubocop:disable Metrics/ParameterLists
         def initialize commit,
                        parser: Parsers::Trailers::Collaborator,
-                       validator: Validators::Name
-
-          super commit
+                       validator: Validators::Name,
+                       **dependencies
+          super commit, **dependencies
           @parser = parser
           @validator = validator
         end
+        # rubocop:enable Metrics/ParameterLists
 
         def valid? = affected_commit_trailers.empty?
 

data/lib/git/lint/cli/actions/analyze/branch.rb

@@ -7,9 +7,11 @@ module Git
         module Analyze
           # Handles analyze action for branch.
           class Branch
-            def initialize analyzer: Analyzer.new, container: Container
+            include Git::Lint::Import[:repository, :kernel, :logger]
+
+            def initialize analyzer: Analyzer.new, **dependencies
+              super(**dependencies)
               @analyzer = analyzer
-              @container = container
             end
 
             def call
@@ -21,7 +23,7 @@ module Git
 
             private
 
-            attr_reader :analyzer, :container
+            attr_reader :analyzer
 
             def parse
               analyzer.call do |collector, reporter|
@@ -29,12 +31,6 @@ module Git
                 kernel.abort if collector.errors?
               end
             end
-
-            def repository = container[__method__]
-
-            def kernel = container[__method__]
-
-            def logger = container[__method__]
           end
         end
       end

data/lib/git/lint/cli/actions/analyze/commit.rb

@@ -7,12 +7,14 @@ module Git
         module Analyze
           # Handles analyze action for commit(s) by SHA.
           class Commit
+            include Git::Lint::Import[:kernel, :logger]
+
             def initialize analyzer: Analyzer.new,
                            parser: GitPlus::Parsers::Commits::Saved::History.with_show,
-                           container: Container
+                           **dependencies
+              super(**dependencies)
               @analyzer = analyzer
               @parser = parser
-              @container = container
             end
 
             def call sha = nil
@@ -24,7 +26,7 @@ module Git
 
             private
 
-            attr_reader :analyzer, :parser, :container
+            attr_reader :analyzer, :parser
 
             def process sha
               analyzer.call commits: parser.call(*sha) do |collector, reporter|
@@ -32,10 +34,6 @@ module Git
                 kernel.abort if collector.errors?
               end
             end
-
-            def kernel = container[__method__]
-
-            def logger = container[__method__]
           end
         end
       end

data/lib/git/lint/cli/actions/config.rb

@@ -6,9 +6,11 @@ module Git
       module Actions
         # Handles gem configuration action.
         class Config
-          def initialize configuration: Configuration::Loader::CLIENT, container: Container
+          include Git::Lint::Import[:kernel, :logger]
+
+          def initialize configuration: Configuration::Loader::CLIENT, **dependencies
+            super(**dependencies)
             @configuration = configuration
-            @container = container
           end
 
           def call action
@@ -21,15 +23,11 @@ module Git
 
           private
 
-          attr_reader :configuration, :container
+          attr_reader :configuration
 
           def edit = kernel.system("$EDITOR #{configuration.current}")
 
           def view = kernel.system("cat #{configuration.current}")
-
-          def kernel = container[__method__]
-
-          def logger = container[__method__]
         end
       end
     end

data/lib/git/lint/cli/actions/container.rb

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+require "dry/container"
+
+module Git
+  module Lint
+    module CLI
+      module Actions
+        # Provides a single container with application and action specific dependencies.
+        module Container
+          extend Dry::Container::Mixin
+
+          config.registry = ->(container, key, value, _options) { container[key.to_s] = value }
+
+          merge Git::Lint::Container
+
+          register(:analyze_branch) { Analyze::Branch.new }
+          register(:analyze_commit) { Analyze::Commit.new }
+          register(:config) { Config.new }
+          register(:hook) { Hook.new }
+        end
+      end
+    end
+  end
+end

data/lib/git/lint/cli/actions/hook.rb

@@ -6,9 +6,11 @@ module Git
       module Actions
         # Handles unsaved Git commit action.
         class Hook
-          def initialize analyzer: Analyzer.new, container: Container
+          include Git::Lint::Import[:repository, :kernel, :logger]
+
+          def initialize analyzer: Analyzer.new, **dependencies
+            super(**dependencies)
             @analyzer = analyzer
-            @container = container
           end
 
           def call path
@@ -20,13 +22,7 @@ module Git
 
           private
 
-          attr_reader :analyzer, :container
-
-          def repository = container[__method__]
-
-          def kernel = container[__method__]
-
-          def logger = container[__method__]
+          attr_reader :analyzer
         end
       end
     end

data/lib/git/lint/cli/actions/import.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+require "auto_injector"
+
+module Git
+  module Lint
+    module CLI
+      module Actions
+        Import = AutoInjector[Container]
+      end
+    end
+  end
+end

data/lib/git/lint/cli/parser.rb

@@ -7,26 +7,29 @@ module Git
     module CLI
       # Assembles and parses all Command Line Interface (CLI) options.
       class Parser
+        include Import[:configuration]
+
         CLIENT = OptionParser.new nil, 40, "  "
         SECTIONS = [Parsers::Core, Parsers::Analyze].freeze # Order matters.
 
-        def initialize sections: SECTIONS, client: CLIENT, container: Container
+        def initialize sections: SECTIONS, client: CLIENT, **dependencies
+          super(**dependencies)
           @sections = sections
           @client = client
-          @configuration = container[:configuration].dup
+          @configuration_duplicate = configuration.dup
         end
 
         def call arguments = []
-          sections.each { |section| section.call configuration, client: }
+          sections.each { |section| section.call configuration_duplicate, client: }
           client.parse arguments
-          configuration.freeze
+          configuration_duplicate.freeze
         end
 
         def to_s = client.to_s
 
         private
 
-        attr_reader :sections, :client, :configuration
+        attr_reader :sections, :client, :configuration_duplicate
       end
     end
   end

data/lib/git/lint/cli/parsers/core.rb

@@ -8,16 +8,18 @@ module Git
       module Parsers
         # Handles parsing of Command Line Interface (CLI) core options.
         class Core
+          include Import[:specification]
+
           using ::Refinements::Structs
 
           def self.call(...) = new(...).call
 
           def initialize configuration = Container[:configuration],
                          client: Parser::CLIENT,
-                         container: Container
+                         **dependencies
+            super(**dependencies)
             @configuration = configuration
             @client = client
-            @container = container
           end
 
           def call arguments = []
@@ -30,7 +32,7 @@ module Git
 
           private
 
-          attr_reader :configuration, :client, :container
+          attr_reader :configuration, :client
 
           def collate = private_methods.sort.grep(/add_/).each { |method| __send__ method }
 
@@ -68,8 +70,6 @@ module Git
               configuration.merge! action_help: true
             end
           end
-
-          def specification = container[__method__]
         end
       end
     end

data/lib/git/lint/cli/shell.rb

@@ -5,17 +5,18 @@ module Git
     module CLI
       # The main Command Line Interface (CLI) object.
       class Shell
-        ACTIONS = {
-          analyze_branch: Actions::Analyze::Branch.new,
-          analyze_commit: Actions::Analyze::Commit.new,
-          config: Actions::Config.new,
-          hook: Actions::Hook.new
-        }.freeze
-
-        def initialize parser: Parser.new, actions: ACTIONS, container: Container
+        include Actions::Import[
+          :analyze_branch,
+          :analyze_commit,
+          :config,
+          :hook,
+          :specification,
+          :logger
+        ]
+
+        def initialize parser: Parser.new, **dependencies
+          super(**dependencies)
           @parser = parser
-          @actions = actions
-          @container = container
         end
 
         def call arguments = []
@@ -26,32 +27,18 @@ module Git
 
         private
 
-        attr_reader :parser, :actions, :container
+        attr_reader :parser
 
         def perform configuration
           case configuration
-            in action_analyze: true, analyze_sha: nil then analyze_branch
-            in action_analyze: true, analyze_sha: String => sha then analyze_commit sha
-            in action_config: Symbol => action then config action
-            in action_hook: Pathname => path then hook path
+            in action_analyze: true, analyze_sha: nil then analyze_branch.call
+            in action_analyze: true, analyze_sha: String => sha then analyze_commit.call sha
+            in action_config: Symbol => action then config.call action
+            in action_hook: Pathname => path then hook.call path
             in action_version: true then logger.info { specification.labeled_version }
-            else usage
+            else logger.any { parser.to_s }
           end
         end
-
-        def analyze_branch = actions.fetch(__method__).call
-
-        def analyze_commit(sha) = actions.fetch(__method__).call(sha)
-
-        def config(action) = actions.fetch(__method__).call(action)
-
-        def hook(path) = actions.fetch(__method__).call(path)
-
-        def usage = logger.unknown { parser.to_s }
-
-        def specification = container[__method__]
-
-        def logger = container[__method__]
       end
     end
   end

data/lib/git/lint/commits/loader.rb

@@ -7,19 +7,16 @@ module Git
     module Commits
       # Automatically detects and loads system.
       class Loader
-        using ::Refinements::Strings
-
-        SYSTEMS = {
-          circle_ci: Systems::CircleCI.new,
-          git_hub_action: Systems::GitHubAction.new,
-          netlify_ci: Systems::NetlifyCI.new,
-          local: Systems::Local.new
-        }.freeze
+        include Systems::Import[
+          :circle_ci,
+          :git_hub_action,
+          :netlify_ci,
+          :local,
+          :repository,
+          :environment
+        ]
 
-        def initialize systems: SYSTEMS, container: Container
-          @systems = systems
-          @container = container
-        end
+        using ::Refinements::Strings
 
         def call
           message = "Invalid repository. Are you within a Git repository?"
@@ -30,21 +27,15 @@ module Git
 
         private
 
-        attr_reader :systems, :container
-
         def load_system
-          if key? "CIRCLECI" then systems.fetch :circle_ci
-          elsif key? "GITHUB_ACTIONS" then systems.fetch :git_hub_action
-          elsif key? "NETLIFY" then systems.fetch :netlify_ci
-          else systems.fetch :local
+          if key? "CIRCLECI" then circle_ci
+          elsif key? "GITHUB_ACTIONS" then git_hub_action
+          elsif key? "NETLIFY" then netlify_ci
+          else local
           end
         end
 
         def key?(key) = environment.fetch(key, "false").to_bool
-
-        def repository = container[__method__]
-
-        def environment = container[__method__]
       end
     end
   end

data/lib/git/lint/commits/systems/circle_ci.rb

@@ -6,19 +6,13 @@ module Git
       module Systems
         # Provides Circle CI build environment feature branch information.
         class CircleCI
-          def initialize container: Container
-            @container = container
-          end
+          include Git::Lint::Import[:repository]
 
           def call = repository.commits("origin/#{repository.branch_default}..#{name}")
 
           private
 
-          attr_reader :container
-
           def name = "origin/#{repository.branch_name}"
-
-          def repository = container[__method__]
         end
       end
     end

data/lib/git/lint/commits/systems/container.rb

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+require "dry/container"
+
+module Git
+  module Lint
+    module Commits
+      module Systems
+        # Provides a single container with application and system specific dependencies.
+        module Container
+          extend Dry::Container::Mixin
+
+          config.registry = ->(container, key, value, _options) { container[key.to_s] = value }
+
+          merge Git::Lint::Container
+
+          register(:circle_ci) { CircleCI.new }
+          register(:git_hub_action) { GitHubAction.new }
+          register(:netlify_ci) { NetlifyCI.new }
+          register(:local) { Local.new }
+        end
+      end
+    end
+  end
+end

data/lib/git/lint/commits/systems/git_hub_action.rb

@@ -6,19 +6,13 @@ module Git
       module Systems
         # Provides GitHub Action build environment feature branch information.
         class GitHubAction
-          def initialize container: Container
-            @container = container
-          end
+          include Git::Lint::Import[:repository]
 
           def call = repository.commits("origin/#{repository.branch_default}..#{name}")
 
           private
 
-          attr_reader :container
-
           def name = "origin/#{repository.branch_name}"
-
-          def repository = container[__method__]
         end
       end
     end

data/lib/git/lint/commits/systems/import.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+require "auto_injector"
+
+module Git
+  module Lint
+    module Commits
+      module Systems
+        Import = AutoInjector[Container]
+      end
+    end
+  end
+end

data/lib/git/lint/commits/systems/local.rb

@@ -6,19 +6,13 @@ module Git
       module Systems
         # Provides local build environment feature branch information.
         class Local
-          def initialize container: Container
-            @container = container
-          end
+          include Git::Lint::Import[:repository]
 
           def call = repository.commits("#{repository.branch_default}..#{name}")
 
           private
 
-          attr_reader :container
-
           def name = repository.branch_name
-
-          def repository = container[__method__]
         end
       end
     end

data/lib/git/lint/commits/systems/netlify_ci.rb

@@ -8,27 +8,17 @@ module Git
       module Systems
         # Provides Netlify CI build environment feature branch information.
         class NetlifyCI
-          def initialize container: Container
-            @container = container
-          end
+          include Git::Lint::Import[:repository, :executor, :environment]
 
           def call
-            shell.capture3 "git remote add -f origin #{environment["REPOSITORY_URL"]}"
-            shell.capture3 "git fetch origin #{name}:#{name}"
+            executor.capture3 "git remote add -f origin #{environment["REPOSITORY_URL"]}"
+            executor.capture3 "git fetch origin #{name}:#{name}"
             repository.commits "origin/#{repository.branch_default}..origin/#{name}"
           end
 
           private
 
-          attr_reader :container
-
           def name = environment["HEAD"]
-
-          def repository = container[__method__]
-
-          def shell = container[__method__]
-
-          def environment = container[__method__]
         end
       end
     end

data/lib/git/lint/container.rb

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
 
+require "cogger"
 require "dry-container"
 require "git_plus"
-require "logger"
-require "pastel"
+require "open3"
 require "spek"
 
 module Git
@@ -13,31 +13,12 @@ module Git
       extend Dry::Container::Mixin
 
       register(:configuration) { Configuration::Loader.call }
+      register(:environment) { ENV }
       register(:repository) { GitPlus::Repository.new }
       register(:specification) { Spek::Loader.call "#{__dir__}/../../../git-lint.gemspec" }
-      register(:colorizer) { Pastel.new enabled: $stdout.tty? }
       register(:kernel) { Kernel }
-
-      register :log_colors do
-        {
-          "DEBUG" => self[:colorizer].white.detach,
-          "INFO" => self[:colorizer].green.detach,
-          "WARN" => self[:colorizer].yellow.detach,
-          "ERROR" => self[:colorizer].red.detach,
-          "FATAL" => self[:colorizer].white.bold.on_red.detach,
-          "ANY" => self[:colorizer].white.bold.detach
-        }
-      end
-
-      register :logger do
-        Logger.new $stdout,
-                   level: Logger.const_get(ENV.fetch("LOG_LEVEL", "INFO")),
-                   formatter: (
-                     lambda do |severity, _at, _name, message|
-                       self[:log_colors][severity].call "#{message}\n"
-                     end
-                   )
-      end
+      register(:executor) { Open3 }
+      register(:logger) { Cogger::Client.new }
     end
   end
 end

data/lib/git/lint/import.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+require "auto_injector"
+
+module Git
+  module Lint
+    Import = AutoInjector[Container]
+  end
+end