dry-system 0.19.0 → 0.21.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2c09b2200301d76a2ac09297583d2a58711db80b43fbe191086bd859f1d3b4fe
-  data.tar.gz: 7f08ad0fb730b2a7281737f799722c30dc649926f36e18e2cd07144637edbb20
+  metadata.gz: 1c0977524fa33277150e8611a851c557461c14357e1a5d9c1f1929c283f3c9b3
+  data.tar.gz: 12de5eaa25ede9f5f959cd77e2af8f1d3b4b287aaebea1554c2915433743c290
 SHA512:
-  metadata.gz: 646a9ef47bce754608ae670ebb43aaad911e2c8b67d79a10f7c0f284cd379c4ce4992263fb22f48f42768e180cfdb6e9f1a3df8bb590245a9a29579858862000
-  data.tar.gz: 293fddcf0085814fc9628ee77ad6360d04ae076fb8b623117a1a336aceb39be1d5c1a8a44c92f10b7e48ca53e4939284ae91b8e6ed2a3514039f65c15d7d5d54
+  metadata.gz: 6976cb749c98ed536c5fb3e6ca67674aee2707b40cd5d196b6b82472edadfa10877bebac77631b0ebcc5fabe008514ed60e7d9be17ccd1343e8967836f319a11
+  data.tar.gz: fd894f982bf214dac3ccfa0de1d4fd7f95beeea1f91234cd2868dfab4687b76ea831c153b429d8ba9bd9f04bcb46dd2815f87a7829e6b347b2d8992cf0ebd478

data/CHANGELOG.md

@@ -1,5 +1,80 @@
 <!--- DO NOT EDIT THIS FILE - IT'S AUTOMATICALLY GENERATED VIA DEVTOOLS --->
 
+## 0.21.0 2021-11-01
+
+
+### Added
+
+- Added **component dir namespaces** as a way to specify multiple, ordered, independent namespace rules within a given component dir. This replaces and expands upon the namespace support we previously provided via the singular `default_namespace` component dir setting (@timriley in #181)
+
+### Changed
+
+- `default_namespace` setting on component dirs has been deprecated. Add a component dir namespace instead, e.g. instead of:
+
+  ```ruby
+  # Inside Dry::System::Container.configure
+  config.component_dirs.add "lib" do |dir|
+    dir.default_namespace = "admin"
+  end
+  ```
+
+  Add this:
+
+  ```ruby
+  config.component_dirs.add "lib" do |dir|
+    dir.namespaces.add "admin", key: nil
+  end
+  ```
+
+  (@timriley in #181)
+- `Dry::System::Component#path` has been removed and replaced by `Component#require_path` and `Component#const_path` (@timriley in #181)
+- Unused `Dry::System::FileNotFoundError` and `Dry::System::InvalidComponentIdentifierTypeError` errors have been removed (@timriley in #194)
+
+[Compare v0.20.0...v0.21.0](https://github.com/dry-rb/dry-system/compare/v0.20.0...v0.21.0)
+
+## 0.20.0 2021-09-12
+
+
+### Fixed
+
+- Fixed dependency graph plugin to work with internal changes introduced in 0.19.0 (@wuarmin in #173)
+- Fixed behavior of `Dry::System::Identifier#start_with?` for components identified by a single segment, or if all matching segments are provided (@wuarmin in #177)
+- Fixed compatibility of `finalize!` signature provided in `Container::Stubs` (@mpokrywka in #178)
+
+### Changed
+
+- [internal] Upgraded to new `setting` API provided in dry-configurable 0.13.0 (@timriley in #179)
+
+[Compare v0.19.2...v0.20.0](https://github.com/dry-rb/dry-system/compare/v0.19.2...v0.20.0)
+
+## 0.19.2 2021-08-30
+
+
+### Changed
+
+- [internal] Improved compatibility with upcoming dry-configurable 0.13.0 release (@timriley in #186)
+
+[Compare v0.18.2...v0.19.2](https://github.com/dry-rb/dry-system/compare/v0.18.2...v0.19.2)
+
+## 0.18.2 2021-08-30
+
+
+### Changed
+
+- [internal] Improved compatibility with upcoming dry-configurable 0.13.0 release (@timriley in #187)
+
+[Compare v0.19.1...v0.18.2](https://github.com/dry-rb/dry-system/compare/v0.19.1...v0.18.2)
+
+## 0.19.1 2021-07-11
+
+
+### Fixed
+
+- Check for registered components (@timriley in #175)
+
+
+[Compare v0.19.0...v0.19.1](https://github.com/dry-rb/dry-system/compare/v0.19.0...v0.19.1)
+
 ## 0.19.0 2021-04-22
 
 This release marks a huge step forward for dry-system, bringing support for Zeitwerk and other autoloaders, plus clearer configuration and improved consistency around component resolution for both finalized and lazy loading containers. [Read the announcement post](https://dry-rb.org/news/2021/04/22/dry-system-0-19-released-with-zeitwerk-support-and-more-leading-the-way-for-hanami-2-0/) for a high-level tour of the new features.

data/README.md

@@ -1,3 +1,4 @@
+<!--- this file is synced from dry-rb/template-gem project -->
 [gem]: https://rubygems.org/gems/dry-system
 [actions]: https://github.com/dry-rb/dry-system/actions
 [codacy]: https://www.codacy.com/gh/dry-rb/dry-system
@@ -7,22 +8,22 @@
 # dry-system [![Join the chat at https://dry-rb.zulipchat.com](https://img.shields.io/badge/dry--rb-join%20chat-%23346b7a.svg)][chat]
 
 [![Gem Version](https://badge.fury.io/rb/dry-system.svg)][gem]
-[![CI Status](https://github.com/dry-rb/dry-system/workflows/ci/badge.svg)][actions]
+[![CI Status](https://github.com/dry-rb/dry-system/workflows/CI/badge.svg)][actions]
 [![Codacy Badge](https://api.codacy.com/project/badge/Grade/3a0e30d0ae2542c7ba047ba5f923c0bb)][codacy]
 [![Codacy Badge](https://api.codacy.com/project/badge/Coverage/3a0e30d0ae2542c7ba047ba5f923c0bb)][codacy]
 [![Inline docs](http://inch-ci.org/github/dry-rb/dry-system.svg?branch=master)][inchpages]
 
 ## Links
 
-* [User documentation](http://dry-rb.org/gems/dry-system)
+* [User documentation](https://dry-rb.org/gems/dry-system)
 * [API documentation](http://rubydoc.info/gems/dry-system)
 
 ## Supported Ruby versions
 
 This library officially supports the following Ruby versions:
 
-* MRI >= `2.5`
-* jruby >= `9.2`
+* MRI `>= 2.6.0`
+* jruby `>= 9.3`
 
 ## License
 

data/dry-system.gemspec

@@ -1,37 +1,38 @@
 # frozen_string_literal: true
-# this file is managed by dry-rb/devtools project
 
-lib = File.expand_path('lib', __dir__)
+# this file is synced from dry-rb/template-gem project
+
+lib = File.expand_path("lib", __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require 'dry/system/version'
+require "dry/system/version"
 
 Gem::Specification.new do |spec|
-  spec.name          = 'dry-system'
+  spec.name          = "dry-system"
   spec.authors       = ["Piotr Solnica"]
   spec.email         = ["piotr.solnica@gmail.com"]
-  spec.license       = 'MIT'
+  spec.license       = "MIT"
   spec.version       = Dry::System::VERSION.dup
 
   spec.summary       = "Organize your code into reusable components"
   spec.description   = spec.summary
-  spec.homepage      = 'https://dry-rb.org/gems/dry-system'
+  spec.homepage      = "https://dry-rb.org/gems/dry-system"
   spec.files         = Dir["CHANGELOG.md", "LICENSE", "README.md", "dry-system.gemspec", "lib/**/*"]
-  spec.bindir        = 'bin'
+  spec.bindir        = "bin"
   spec.executables   = []
-  spec.require_paths = ['lib']
+  spec.require_paths = ["lib"]
 
-  spec.metadata['allowed_push_host'] = 'https://rubygems.org'
-  spec.metadata['changelog_uri']     = 'https://github.com/dry-rb/dry-system/blob/master/CHANGELOG.md'
-  spec.metadata['source_code_uri']   = 'https://github.com/dry-rb/dry-system'
-  spec.metadata['bug_tracker_uri']   = 'https://github.com/dry-rb/dry-system/issues'
+  spec.metadata["allowed_push_host"] = "https://rubygems.org"
+  spec.metadata["changelog_uri"]     = "https://github.com/dry-rb/dry-system/blob/master/CHANGELOG.md"
+  spec.metadata["source_code_uri"]   = "https://github.com/dry-rb/dry-system"
+  spec.metadata["bug_tracker_uri"]   = "https://github.com/dry-rb/dry-system/issues"
 
-  spec.required_ruby_version = ">= 2.5.0"
+  spec.required_ruby_version = ">= 2.6.0"
 
   # to update dependencies edit project.yml
   spec.add_runtime_dependency "concurrent-ruby", "~> 1.0"
   spec.add_runtime_dependency "dry-auto_inject", ">= 0.4.0"
-  spec.add_runtime_dependency "dry-configurable", "~> 0.12", ">= 0.12.1"
-  spec.add_runtime_dependency "dry-container", "~> 0.7", ">= 0.7.2"
+  spec.add_runtime_dependency "dry-configurable", "~> 0.13", ">= 0.13.0"
+  spec.add_runtime_dependency "dry-container", "~> 0.9", ">= 0.9.0"
   spec.add_runtime_dependency "dry-core", "~> 0.5", ">= 0.5"
   spec.add_runtime_dependency "dry-inflector", "~> 0.1", ">= 0.1.2"
   spec.add_runtime_dependency "dry-struct", "~> 1.0"

data/lib/dry/system/auto_registrar.rb

@@ -29,29 +29,17 @@ module Dry
 
       # @api private
       def call(component_dir)
-        components(component_dir).each do |component|
+        component_dir.each_component do |component|
           next unless register_component?(component)
 
-          container.register(component.identifier, memoize: component.memoize?) { component.instance }
+          container.register(component.key, memoize: component.memoize?) { component.instance }
         end
       end
 
       private
 
-      def components(component_dir)
-        files(component_dir.full_path).map { |file_path|
-          component_dir.component_for_path(file_path)
-        }
-      end
-
-      def files(dir)
-        raise ComponentDirNotFoundError, dir unless Dir.exist?(dir)
-
-        Dir["#{dir}/**/#{RB_GLOB}"].sort
-      end
-
       def register_component?(component)
-        !container.registered?(component) && component.auto_register?
+        !container.registered?(component.key) && component.auto_register?
       end
     end
   end

data/lib/dry/system/booter/component_registry.rb

@@ -21,13 +21,13 @@ module Dry
         end
 
         def exists?(name)
-          components.any? { |component| component.identifier == name }
+          components.any? { |component| component.name == name }
         end
 
         def [](name)
-          component = components.detect { |component| component.identifier == name }
+          component = components.detect { |c| c.name == name }
 
-          component || raise(InvalidComponentIdentifierError, name)
+          component || raise(InvalidComponentNameError, name)
         end
       end
     end

data/lib/dry/system/booter.rb

@@ -179,15 +179,15 @@ module Dry
       end
 
       def load_component(path)
-        identifier = Pathname(path).basename(RB_EXT).to_s.to_sym
+        name = Pathname(path).basename(RB_EXT).to_s.to_sym
 
-        Kernel.require path unless components.exists?(identifier)
+        Kernel.require path unless components.exists?(name)
 
         self
       end
 
-      def require_boot_file(identifier)
-        boot_file = find_boot_file(identifier)
+      def require_boot_file(name)
+        boot_file = find_boot_file(name)
 
         Kernel.require boot_file if boot_file
       end

data/lib/dry/system/component.rb

@@ -17,7 +17,7 @@ module Dry
     #
     # @api public
     class Component
-      include Dry::Equalizer(:identifier, :file_path, :options)
+      include Dry::Equalizer(:identifier, :namespace, :options)
 
       DEFAULT_OPTIONS = {
         separator: DEFAULT_SEPARATOR,
@@ -26,43 +26,34 @@ module Dry
       }.freeze
 
       # @!attribute [r] identifier
-      #   @return [String] component's unique identifier
+      #   @return [String] the component's unique identifier
       attr_reader :identifier
 
-      # @!attribute [r] file_path
-      #   @return [String, nil] full path to the component's file, if found
-      attr_reader :file_path
+      # @!attribute [r] namespace
+      #   @return [Dry::System::Config::Namespace] the component's namespace
+      attr_reader :namespace
 
       # @!attribute [r] options
-      #   @return [Hash] component's options
+      #   @return [Hash] the component's options
       attr_reader :options
 
       # @api private
-      def self.new(identifier, options = EMPTY_HASH)
-        options = DEFAULT_OPTIONS.merge(options)
-
-        namespace = options.delete(:namespace)
-        separator = options.delete(:separator)
-
-        identifier =
-          if identifier.is_a?(Identifier)
-            identifier
-          else
-            Identifier.new(
-              identifier,
-              namespace: namespace,
-              separator: separator
-            )
-          end
-
-        super(identifier, **options)
+      def initialize(identifier, namespace:, **options)
+        @identifier = identifier
+        @namespace = namespace
+        @options = DEFAULT_OPTIONS.merge(options)
       end
 
+      # Returns true, indicating that the component is directly loadable from the files
+      # managed by the container
+      #
+      # This is the inverse of {IndirectComponent#loadable?}
+      #
+      # @return [TrueClass]
+      #
       # @api private
-      def initialize(identifier, file_path: nil, **options)
-        @identifier = identifier
-        @file_path = file_path
-        @options = options
+      def loadable?
+        true
       end
 
       # Returns the component's instance
@@ -74,39 +65,95 @@ module Dry
       end
       ruby2_keywords(:instance) if respond_to?(:ruby2_keywords, true)
 
-      # @api private
-      def bootable?
-        false
-      end
-
+      # Returns the component's unique key
+      #
+      # @return [String] the key
+      #
+      # @see Identifier#key
+      #
+      # @api public
       def key
-        identifier.to_s
-      end
-
-      def path
-        identifier.path
+        identifier.key
       end
 
+      # Returns the root namespace segment of the component's key, as a symbol
+      #
+      # @see Identifier#root_key
+      #
+      # @return [Symbol] the root key
+      #
+      # @api public
       def root_key
         identifier.root_key
       end
 
-      # Returns true if the component has a corresponding file
+      # Returns a path-delimited representation of the compnent, appropriate for passing
+      # to `Kernel#require` to require its source file
       #
-      # @return [Boolean]
-      # @api private
-      def file_exists?
-        !!file_path
+      # The path takes into account the rules of the namespace used to load the component.
+      #
+      # @example Component from a root namespace
+      #   component.key # => "articles.create"
+      #   component.require_path # => "articles/create"
+      #
+      # @example Component from an "admin/" path namespace (with `key: nil`)
+      #   component.key # => "articles.create"
+      #   component.require_path # => "admin/articles/create"
+      #
+      # @see Config::Namespaces#add
+      # @see Config::Namespace
+      #
+      # @return [String] the require path
+      #
+      # @api public
+      def require_path
+        if namespace.path
+          "#{namespace.path}#{PATH_SEPARATOR}#{path_in_namespace}"
+        else
+          path_in_namespace
+        end
+      end
+
+      # Returns an "underscored", path-delimited representation of the component,
+      # appropriate for passing to the inflector for constantizing
+      #
+      # The const path takes into account the rules of the namespace used to load the
+      # component.
+      #
+      # @example Component from a namespace with `const: nil`
+      #   component.key # => "articles.create_article"
+      #   component.const_path # => "articles/create_article"
+      #   component.inflector.constantize(component.const_path) # => Articles::CreateArticle
+      #
+      # @example Component from a namespace with `const: "admin"`
+      #   component.key # => "articles.create_article"
+      #   component.const_path # => "admin/articles/create_article"
+      #   component.inflector.constantize(component.const_path) # => Admin::Articles::CreateArticle
+      #
+      # @see Config::Namespaces#add
+      # @see Config::Namespace
+      #
+      # @return [String] the const path
+      #
+      # @api public
+      def const_path
+        namespace_const_path = namespace.const&.gsub(identifier.separator, PATH_SEPARATOR)
+
+        if namespace_const_path
+          "#{namespace_const_path}#{PATH_SEPARATOR}#{path_in_namespace}"
+        else
+          path_in_namespace
+        end
       end
 
       # @api private
       def loader
-        options[:loader]
+        options.fetch(:loader)
       end
 
       # @api private
       def inflector
-        options[:inflector]
+        options.fetch(:inflector)
       end
 
       # @api private
@@ -121,6 +168,17 @@ module Dry
 
       private
 
+      def path_in_namespace
+        identifier_in_namespace =
+          if namespace.key
+            identifier.namespaced(from: namespace.key, to: nil)
+          else
+            identifier
+          end
+
+        identifier_in_namespace.key_with_separator(PATH_SEPARATOR)
+      end
+
       def callable_option?(value)
         if value.respond_to?(:call)
           !!value.call(self)

data/lib/dry/system/component_dir.rb

@@ -1,4 +1,7 @@
+# frozen_string_literal: true
+
 require "pathname"
+require "dry/system/constants"
 require_relative "constants"
 require_relative "identifier"
 require_relative "magic_comments_parser"
@@ -28,40 +31,101 @@ module Dry
         @container = container
       end
 
-      # Returns a component for a given identifier if a matching component file could be
-      # found within the component dir
+      # Returns a component for the given key if a matching source file is found within
+      # the component dir
       #
-      # This will search within the component dir's configured default_namespace first,
-      # then fall back to searching for a non-namespaced file
+      # This searches according to the component dir's configured namespaces, in order of
+      # definition, with the first match returned as the component.
       #
-      # @param identifier [String] the identifier string
+      # @param key [String] the component's key
       # @return [Dry::System::Component, nil] the component, if found
       #
       # @api private
-      def component_for_identifier(identifier)
-        identifier = Identifier.new(
-          identifier,
-          namespace: default_namespace,
-          separator: container.config.namespace_separator
-        )
-
-        if (file_path = find_component_file(identifier.path))
-          return build_component(identifier, file_path)
+      def component_for_key(key)
+        namespaces.each do |namespace|
+          identifier = Identifier.new(key, separator: container.config.namespace_separator)
+
+          next unless identifier.start_with?(namespace.key)
+
+          if (file_path = find_component_file(identifier, namespace))
+            return build_component(identifier, namespace, file_path)
+          end
+        end
+
+        nil
+      end
+
+      def each_component
+        return enum_for(:each_component) unless block_given?
+
+        each_file do |file_path, namespace|
+          yield component_for_path(file_path, namespace)
         end
+      end
+
+      private
+
+      def namespaces
+        config.namespaces.to_a.map { |namespace| normalize_namespace(namespace) }
+      end
 
-        identifier = identifier.with(namespace: nil)
-        if (file_path = find_component_file(identifier.path))
-          build_component(identifier, file_path)
+      # Returns an array of "normalized" namespaces, safe for loading components
+      #
+      # This works around the issue of a namespace being added for a nested path but
+      # _without_ specifying a key namespace. In this case, the key namespace will defaut
+      # to match the path, meaning it will contain path separators instead of the
+      # container's configured `namespace_separator` (due to `Config::Namespaces` not
+      # being able to know the configured `namespace_separator`), so we need to replace
+      # the path separators with the proper `namespace_separator` here (where we _do_ know
+      # what it is).
+      def normalize_namespace(namespace)
+        if namespace.path&.include?(PATH_SEPARATOR) && namespace.default_key?
+          namespace = namespace.class.new(
+            path: namespace.path,
+            key: namespace.key.gsub(PATH_SEPARATOR, container.config.namespace_separator),
+            const: namespace.const
+          )
         end
+
+        namespace
+      end
+
+      def each_file
+        return enum_for(:each_file) unless block_given?
+
+        raise ComponentDirNotFoundError, full_path unless Dir.exist?(full_path)
+
+        namespaces.each do |namespace|
+          files(namespace).each do |file|
+            yield file, namespace
+          end
+        end
+      end
+
+      def files(namespace)
+        if namespace.path?
+          Dir[File.join(full_path, namespace.path, "**", RB_GLOB)].sort
+        else
+          non_root_paths = namespaces.to_a.reject(&:root?).map(&:path)
+
+          Dir[File.join(full_path, "**", RB_GLOB)].reject { |file_path|
+            Pathname(file_path).relative_path_from(full_path).to_s.start_with?(*non_root_paths)
+          }.sort
+        end
+      end
+
+      # Returns the full path of the component directory
+      #
+      # @return [Pathname]
+      def full_path
+        container.root.join(path)
       end
 
       # Returns a component for a full path to a Ruby source file within the component dir
       #
       # @param path [String] the full path to the file
       # @return [Dry::System::Component] the component
-      #
-      # @api private
-      def component_for_path(path)
+      def component_for_path(path, namespace)
         separator = container.config.namespace_separator
 
         key = Pathname(path).relative_path_from(full_path).to_s
@@ -70,46 +134,49 @@ module Dry
           .join(separator)
 
         identifier = Identifier.new(key, separator: separator)
+          .namespaced(
+            from: namespace.path&.gsub(PATH_SEPARATOR, separator),
+            to: namespace.key
+          )
+
+        build_component(identifier, namespace, path)
+      end
 
-        if identifier.start_with?(default_namespace)
-          identifier = identifier.dequalified(default_namespace, namespace: default_namespace)
+      def find_component_file(identifier, namespace)
+        # To properly find the file within a namespace with a key, we should strip the key
+        # from beginning of our given identifier
+        if namespace.key
+          identifier = identifier.namespaced(from: namespace.key, to: nil)
         end
 
-        build_component(identifier, path)
-      end
+        file_name = "#{identifier.key_with_separator(PATH_SEPARATOR)}#{RB_EXT}"
 
-      # Returns the full path of the component directory
-      #
-      # @return [Pathname]
-      # @api private
-      def full_path
-        container.root.join(path)
-      end
+        component_file =
+          if namespace.path?
+            full_path.join(namespace.path, file_name)
+          else
+            full_path.join(file_name)
+          end
 
-      # @api private
-      def component_options
-        {
-          auto_register: auto_register,
-          loader: loader,
-          memoize: memoize
-        }
+        component_file if component_file.exist?
       end
 
-      private
-
-      def build_component(identifier, file_path)
+      def build_component(identifier, namespace, file_path)
         options = {
           inflector: container.config.inflector,
           **component_options,
           **MagicCommentsParser.(file_path)
         }
 
-        Component.new(identifier, file_path: file_path, **options)
+        Component.new(identifier, namespace: namespace, **options)
       end
 
-      def find_component_file(component_path)
-        component_file = full_path.join("#{component_path}#{RB_EXT}")
-        component_file if component_file.exist?
+      def component_options
+        {
+          auto_register: auto_register,
+          loader: loader,
+          memoize: memoize
+        }
       end
 
       def method_missing(name, *args, &block)