dry-system 0.21.0 → 0.22.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1c0977524fa33277150e8611a851c557461c14357e1a5d9c1f1929c283f3c9b3
-  data.tar.gz: 12de5eaa25ede9f5f959cd77e2af8f1d3b4b287aaebea1554c2915433743c290
+  metadata.gz: 2ce0d3b9c3094e2e7d1b20907a31107d6329221e6213839708f8cc7bac1321bd
+  data.tar.gz: 2e28bfe4758cace04f1978c2b17fa138921915d5db0b894796772fa428e12549
 SHA512:
-  metadata.gz: 6976cb749c98ed536c5fb3e6ca67674aee2707b40cd5d196b6b82472edadfa10877bebac77631b0ebcc5fabe008514ed60e7d9be17ccd1343e8967836f319a11
-  data.tar.gz: fd894f982bf214dac3ccfa0de1d4fd7f95beeea1f91234cd2868dfab4687b76ea831c153b429d8ba9bd9f04bcb46dd2815f87a7829e6b347b2d8992cf0ebd478
+  metadata.gz: 33e7e5e5cccf47481498a95c8dc37fdb282ab9005c595f4db30b05f84c842d5144ba7cfa0f3a15f895212dbb0034554f36d16649d004bf54eebc99dca70d53d1
+  data.tar.gz: dc441f94a5d1506e7c7bc4afd9e8ebfd43d43e7956af09e2a793df6d1b02b409ac8eb0252183cae6c388969f2e33b02522675794c1aa4c340f6c52efa0ad7344

data/CHANGELOG.md

@@ -1,5 +1,19 @@
 <!--- DO NOT EDIT THIS FILE - IT'S AUTOMATICALLY GENERATED VIA DEVTOOLS --->
 
+## 0.22.0 2022-01-06
+
+
+### Added
+
+- Expanded public interfaces for `Dry::System::Config::ComponentDirs` and `Dry::System::Config::Namespaces` to better support programmatic construction and inspection of these configs (@timriley in #195)
+
+### Changed
+
+- Deprecated `Dry::System::Config::Namespaces#root` as the way to add and configure a root namespace. Use `#add_root` instead (@timriley in #195)
+- Allow bootsnap plugin to use bootsnap on Ruby versions up to 3.0 (pusewicz in #196)
+
+[Compare v0.21.0...v0.22.0](https://github.com/dry-rb/dry-system/compare/v0.21.0...v0.22.0)
+
 ## 0.21.0 2021-11-01
 
 
@@ -29,6 +43,7 @@
   (@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)
+- Allow bootsnap for Rubies up to 3.0.x (via #196) (@pusewicz)
 
 [Compare v0.20.0...v0.21.0](https://github.com/dry-rb/dry-system/compare/v0.20.0...v0.21.0)
 

data/README.md

@@ -8,7 +8,7 @@
 # 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]

data/lib/dry/system/config/component_dir.rb

@@ -9,6 +9,7 @@ require_relative "namespaces"
 module Dry
   module System
     module Config
+      # @api public
       class ComponentDir
         include Dry::Configurable
 
@@ -37,6 +38,7 @@ module Dry
         #
         #   @see auto_register
         #   @see Component
+        #   @api public
         #
         # @!method auto_register
         #
@@ -45,6 +47,7 @@ module Dry
         #   @return [Boolean, Proc] the configured policy
         #
         #   @see auto_register=
+        #   @api public
         setting :auto_register, default: true
 
         # @!method add_to_load_path=(policy)
@@ -60,6 +63,7 @@ module Dry
         #
         #   @see add_to_load_path
         #   @see Container.configure
+        #   @api public
         #
         # @!method add_to_load_path
         #
@@ -68,6 +72,7 @@ module Dry
         #   @return [Boolean]
         #
         #   @see add_to_load_path=
+        #   @api public
         setting :add_to_load_path, default: true
 
         # @!method loader=(loader)
@@ -85,6 +90,7 @@ module Dry
         #   @see loader
         #   @see Loader
         #   @see Loader::Autoloading
+        #   @api public
         #
         # @!method loader
         #
@@ -93,6 +99,7 @@ module Dry
         #   @return [#call]
         #
         #   @see loader=
+        #   @api public
         setting :loader, default: Dry::System::Loader
 
         # @!method memoize=(policy)
@@ -119,6 +126,7 @@ module Dry
         #
         #   @see memoize
         #   @see Component
+        #   @api public
         #
         # @!method memoize
         #
@@ -127,6 +135,7 @@ module Dry
         #   @return [Boolean, Proc] the configured memoization policy
         #
         #   @see memoize=
+        #   @api public
         setting :memoize, default: false
 
         # @!method namespaces
@@ -135,11 +144,13 @@ module Dry
         #
         #   Allows namespaces to added on the returned object via {Namespaces#add}.
         #
-        #   @see Namespaces#add
-        #
         #   @return [Namespaces] the namespaces
+        #
+        #   @see Namespaces#add
+        #   @api public
         setting :namespaces, default: Namespaces.new, cloneable: true
 
+        # @api public
         def default_namespace=(namespace)
           Dry::Core::Deprecations.announce(
             "Dry::System::Config::ComponentDir#default_namespace=",
@@ -157,6 +168,7 @@ module Dry
           namespaces.add namespace_path, key: nil
         end
 
+        # @api public
         def default_namespace
           Dry::Core::Deprecations.announce(
             "Dry::System::Config::ComponentDir#default_namespace",
@@ -179,7 +191,7 @@ module Dry
         # @return [String] the path
         attr_reader :path
 
-        # @api private
+        # @api public
         def initialize(path)
           super()
           @path = path

data/lib/dry/system/config/component_dirs.rb

@@ -1,6 +1,5 @@
 # frozen_string_literal: true
 
-require "concurrent/map"
 require "dry/system/constants"
 require "dry/system/errors"
 require_relative "component_dir"
@@ -8,6 +7,9 @@ require_relative "component_dir"
 module Dry
   module System
     module Config
+      # The configured component dirs for a container
+      #
+      # @api public
       class ComponentDirs
         # @!group Settings
 
@@ -63,71 +65,141 @@ module Dry
         #
         #   @see memoize=
 
+        # rubocop:disable Layout/LineLength
+
         # @!method namespaces
         #
         #   Returns the default configured namespaces for all added component dirs
         #
-        #   Allows namespaces to added on the returned object via {Namespaces#add}.
+        #   Allows namespaces to added on the returned object via {Dry::System::Config::Namespaces#add}.
         #
-        #   @see Namespaces#add
+        #   @see Dry::System::Config::Namespaces#add
         #
         #   @return [Namespaces] the namespaces
 
+        # rubocop:enable Layout/LineLength
+
         # @!endgroup
 
         # A ComponentDir for configuring the default values to apply to all added
         # component dirs
         #
+        # @see #method_missing
         # @api private
         attr_reader :defaults
 
+        # Creates a new component dirs
+        #
         # @api private
         def initialize
-          @dirs = Concurrent::Map.new
+          @dirs = {}
           @defaults = ComponentDir.new(nil)
         end
 
         # @api private
         def initialize_copy(source)
-          @dirs = source.dirs.dup
+          @dirs = source.dirs.map { |path, dir| [path, dir.dup] }.to_h
           @defaults = source.defaults.dup
         end
 
-        # Adds and configures a component dir
+        # Returns and optionally yields a previously added component dir
+        #
+        # @param path [String] the path for the component dir
+        # @yieldparam dir [ComponentDir] the component dir
+        #
+        # @return [ComponentDir] the component dir
+        #
+        # @api public
+        def dir(path)
+          dirs[path].tap do |dir|
+            # Defaults can be (re-)applied first, since the dir has already been added
+            apply_defaults_to_dir(dir) if dir
+            yield dir if block_given?
+          end
+        end
+        alias_method :[], :dir
+
+        # @overload add(path)
+        #   Adds and configures a component dir for the given path
+        #
+        #   @param path [String] the path for the component dir, relative to the configured
+        #     container root
+        #   @yieldparam dir [ComponentDir] the component dir to configure
         #
-        # @param path [String] the path for the component dir, relative to the configured
-        #   container root
+        #   @return [ComponentDir] the added component dir
         #
-        # @yieldparam dir [ComponentDir] the component dir to configure
+        #   @example
+        #     component_dirs.add "lib" do |dir|
+        #       dir.default_namespace = "my_app"
+        #     end
         #
-        # @return [ComponentDir] the added component dir
+        #   @see ComponentDir
+        #   @api public
         #
-        # @example
-        #   component_dirs.add "lib" do |dir|
-        #     dir.default_namespace = "my_app"
-        #   end
+        # @overload add(dir)
+        #   Adds a configured component dir
         #
-        # @see ComponentDir
-        def add(path)
+        #   @param dir [ComponentDir] the configured component dir
+        #
+        #   @return [ComponentDir] the added component dir
+        #
+        #   @example
+        #     dir = Dry::System::ComponentDir.new("lib")
+        #     component_dirs.add dir
+        #
+        #   @see ComponentDir
+        #   @api public
+        def add(path_or_dir)
+          path, dir_to_add = path_and_dir(path_or_dir)
+
           raise ComponentDirAlreadyAddedError, path if dirs.key?(path)
 
-          dirs[path] = ComponentDir.new(path).tap do |dir|
-            yield dir if block_given?
+          dirs[path] = dir_to_add.tap do |dir|
+            # Defaults must be applied after yielding, since the dir is being newly added,
+            # and must have its configuration fully in place before we can know which
+            # defaults to apply
+            yield dir if path_or_dir == path && block_given?
             apply_defaults_to_dir(dir)
           end
         end
 
-        # Returns the added component dirs, with default settings applied
+        # Deletes and returns a previously added component dir
         #
-        # @return [Hash<String, ComponentDir>] the component dirs as a hash, keyed by path
-        def dirs
-          @dirs.each { |_, dir| apply_defaults_to_dir(dir) }
+        # @param path [String] the path for the component dir
+        #
+        # @return [ComponentDir] the removed component dir
+        #
+        # @api public
+        def delete(path)
+          dirs.delete(path)
         end
 
+        # Returns the paths of the component dirs
+        #
+        # @return [Array<String>] the component dir paths
+        #
+        # @api public
+        def paths
+          dirs.keys
+        end
+
+        # Returns the count of component dirs
+        #
+        # @return [Integer]
+        #
+        # @api public
+        def length
+          dirs.length
+        end
+        alias_method :size, :length
+
         # Returns the added component dirs, with default settings applied
         #
         # @return [Array<ComponentDir>]
+        #
+        # @api public
         def to_a
+          dirs.each { |_, dir| apply_defaults_to_dir(dir) }
           dirs.values
         end
 
@@ -135,12 +207,46 @@ module Dry
         # argument.
         #
         # @yieldparam dir [ComponentDir] the yielded component dir
+        #
+        # @api public
         def each(&block)
           to_a.each(&block)
         end
 
+        protected
+
+        # Returns the hash of component dirs, keyed by their paths
+        #
+        # Recently changed default configuration may not be applied to these dirs. Use
+        # #to_a or #each to access dirs with default configuration fully applied.
+        #
+        # This method exists to encapsulate the instance variable and to serve the needs
+        # of #initialize_copy
+        #
+        # @return [Hash{String => ComponentDir}]
+        #
+        # @api private
+        attr_reader :dirs
+
         private
 
+        # Converts a path string or pre-built component dir into a path and dir tuple
+        #
+        # @param path_or_dir [String,ComponentDir]
+        #
+        # @return [Array<(String, ComponentDir)>]
+        #
+        # @see #add
+        def path_and_dir(path_or_dir)
+          if path_or_dir.is_a?(ComponentDir)
+            dir = path_or_dir
+            [dir.path, dir]
+          else
+            path = path_or_dir
+            [path, ComponentDir.new(path)]
+          end
+        end
+
         # Applies default settings to a component dir. This is run every time the dirs are
         # accessed to ensure defaults are applied regardless of when new component dirs
         # are added. This method must be idempotent.

data/lib/dry/system/config/namespace.rb

@@ -21,16 +21,19 @@ module Dry
       #
       # @see Namespaces#add
       #
-      # @api private
+      # @api public
       class Namespace
         ROOT_PATH = nil
 
         include Dry::Equalizer(:path, :key, :const)
 
+        # @api public
         attr_reader :path
 
+        # @api public
         attr_reader :key
 
+        # @api public
         attr_reader :const
 
         # Returns a namespace configured to serve as the default root namespace for a
@@ -48,20 +51,24 @@ module Dry
           )
         end
 
+        # @api private
         def initialize(path:, key:, const:)
           @path = path
           @key = key
           @const = const
         end
 
+        # @api public
         def root?
           path == ROOT_PATH
         end
 
+        # @api public
         def path?
           !root?
         end
 
+        # @api private
         def default_key?
           key == path
         end

data/lib/dry/system/config/namespaces.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require "dry/core/deprecations"
 require "dry/system/errors"
 require_relative "namespace"
 
@@ -26,6 +27,39 @@ module Dry
           @namespaces = source.namespaces.dup
         end
 
+        # Returns the namespace configured for the path, or nil if no such namespace has
+        # been configured
+        #
+        # @return [Namespace, nil] the namespace, if configured
+        #
+        # @api public
+        def namespace(path)
+          namespaces[path]
+        end
+        alias_method :[], :namespace
+
+        # Returns the namespace configured for the root path, or nil if the root namespace
+        # has not been configured
+        #
+        # @return [Namespace, nil] the root namespace, if configured
+        #
+        # @api public
+        def root(**options)
+          if options.any?
+            Dry::Core::Deprecations.announce(
+              "Dry::System::Config::Namespaces#root (with arguments)",
+              "Use `#add_root(key: nil, const: nil)` instead",
+              tag: "dry-system",
+              uplevel: 1
+            )
+
+            add_root(**options)
+            return
+          end
+
+          namespaces[Namespace::ROOT_PATH]
+        end
+
         # rubocop:disable Layout/LineLength
 
         # Adds a component dir namespace
@@ -62,8 +96,8 @@ module Dry
         #
         # @param path [String] the path to the sub-directory of source files to which this
         #   namespace should apply, relative to the component dir
-        # @param identifier [String, nil] the leading namespace to apply to the registered
-        #   identifiers for the components. Set `nil` for the identifiers to be top-level.
+        # @param key [String, nil] the leading namespace to apply to the container keys
+        #   for the components. Set `nil` for the keys to be top-level.
         # @param const [String, nil] the Ruby constant namespace to expect for constants
         #   defined within the components. This should be provided in underscored string
         #   form, e.g. "hello_there/world" for a Ruby constant of `HelloThere::World`. Set
@@ -87,31 +121,84 @@ module Dry
         # @see #add
         #
         # @api public
-        def root(key: nil, const: nil)
+        def add_root(key: nil, const: nil)
           add(Namespace::ROOT_PATH, key: key, const: const)
         end
 
-        # @api private
+        # Deletes the configured namespace for the given path and returns the namespace
+        #
+        # If no namespace was previously configured for the given path, returns nil
+        #
+        # @param path [String] the path for the namespace
+        #
+        # @return [Namespace, nil]
+        #
+        # @api public
+        def delete(path)
+          namespaces.delete(path)
+        end
+
+        # Deletes the configured root namespace and returns the namespace
+        #
+        # If no root namespace was previously configured, returns nil
+        #
+        # @return [Namespace, nil]
+        #
+        # @api public
+        def delete_root
+          delete(Namespace::ROOT_PATH)
+        end
+
+        # Returns the paths of the configured namespaces
+        #
+        # @return [Array<String,nil>] the namespace paths, with nil representing the root
+        #   namespace
+        #
+        # @api public
+        def paths
+          namespaces.keys
+        end
+
+        # Returns the count of configured namespaces
+        #
+        # @return [Integer]
+        #
+        # @api public
+        def length
+          namespaces.length
+        end
+        alias_method :size, :length
+
+        # Returns true if there are no configured namespaces
+        #
+        # @return [Boolean]
+        #
+        # @api public
         def empty?
           namespaces.empty?
         end
 
         # Returns the configured namespaces as an array
         #
-        # This adds a root namespace to the end of the array if one was not configured
-        # manually. This fallback ensures that all components in the component dir can be
-        # loaded.
+        # Adds a default root namespace to the end of the array if one was not added
+        # explicitly. This fallback ensures that all components in the component dir can
+        # be loaded.
         #
         # @return [Array<Namespace>] the namespaces
         #
-        # @api private
+        # @api public
         def to_a
           namespaces.values.tap do |arr|
             arr << Namespace.default_root unless arr.any?(&:root?)
           end
         end
 
-        # @api private
+        # Calls the given block once for each configured namespace, passing the namespace
+        # as an argument.
+        #
+        # @yieldparam namespace [Namespace] the yielded namespace
+        #
+        # @api public
         def each(&block)
           to_a.each(&block)
         end

data/lib/dry/system/plugins/bootsnap.rb

@@ -36,7 +36,7 @@ module Dry
 
         # @api private
         def bootsnap_available?
-          RUBY_ENGINE == "ruby" && RUBY_VERSION >= "2.3.0" && RUBY_VERSION < "2.5.0"
+          RUBY_ENGINE == "ruby" && RUBY_VERSION >= "2.3.0" && RUBY_VERSION < "3.1.0"
         end
       end
     end

data/lib/dry/system/version.rb

@@ -2,6 +2,6 @@
 
 module Dry
   module System
-    VERSION = "0.21.0"
+    VERSION = "0.22.0"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dry-system
 version: !ruby/object:Gem::Version
-  version: 0.21.0
+  version: 0.22.0
 platform: ruby
 authors:
 - Piotr Solnica
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-10-31 00:00:00.000000000 Z
+date: 2022-01-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby