smart_initializer 0.9.1 → 0.10.0

data/lib/smart_core/initializer/constructor/definer.rb

@@ -2,6 +2,7 @@
 
 # @api private
 # @since 0.1.0
+# @version 0.10.0
 # rubocop:disable Metrics/ClassLength
 class SmartCore::Initializer::Constructor::Definer
   # @param klass [Class]
@@ -9,9 +10,10 @@ class SmartCore::Initializer::Constructor::Definer
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def initialize(klass)
     @klass = klass
-    @lock = SmartCore::Engine::Lock.new
+    @lock = SmartCore::Engine::ReadWriteLock.new
   end
 
   # @param block [Proc]
@@ -19,8 +21,9 @@ class SmartCore::Initializer::Constructor::Definer
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def define_init_extension(block)
-    thread_safe do
+    @lock.write_sync do
       add_init_extension(build_init_extension(block))
     end
   end
@@ -37,7 +40,7 @@ class SmartCore::Initializer::Constructor::Definer
   #
   # @api private
   # @since 0.1.0
-  # @version 0.8.0
+  # @version 0.10.0
   def define_parameter(
     name,
     type,
@@ -48,7 +51,7 @@ class SmartCore::Initializer::Constructor::Definer
     mutable,
     as
   )
-    thread_safe do
+    @lock.write_sync do
       attribute = build_param_attribute(
         name,
         type,
@@ -71,9 +74,9 @@ class SmartCore::Initializer::Constructor::Definer
   #
   # @api private
   # @since 0.1.0
-  # @version 0.8.0
+  # @version 0.10.0
   def define_parameters(*names, mutable:, privacy:)
-    thread_safe do
+    @lock.write_sync do
       names.map do |name|
         build_param_attribute(
           name,
@@ -107,7 +110,7 @@ class SmartCore::Initializer::Constructor::Definer
   #
   # @api private
   # @since 0.1.0
-  # @version 0.8.0
+  # @version 0.10.0
   def define_option(
     name,
     type,
@@ -120,7 +123,7 @@ class SmartCore::Initializer::Constructor::Definer
     default,
     optional
   )
-    thread_safe do
+    @lock.write_sync do
       attribute = build_option_attribute(
         name,
         type,
@@ -145,9 +148,9 @@ class SmartCore::Initializer::Constructor::Definer
   #
   # @api private
   # @since 0.1.0
-  # @version 0.8.0
+  # @version 0.10.0
   def define_options(*names, mutable:, privacy:)
-    thread_safe do
+    @lock.write_sync do
       names.map do |name|
         build_option_attribute(
           name,
@@ -352,14 +355,5 @@ class SmartCore::Initializer::Constructor::Definer
       ERROR_MESSAGE
     end
   end
-
-  # @param block [Block]
-  # @return [Any]
-  #
-  # @api private
-  # @since 0.1.0
-  def thread_safe(&block)
-    @lock.synchronize(&block)
-  end
 end
 # rubocop:enable Metrics/ClassLength

data/lib/smart_core/initializer/dsl.rb

@@ -2,6 +2,7 @@
 
 # @api private
 # @since 0.1.0
+# @version 0.10.0
 module SmartCore::Initializer::DSL
   require_relative 'dsl/inheritance'
 
@@ -11,13 +12,13 @@ module SmartCore::Initializer::DSL
     #
     # @api private
     # @since 0.1.0
+    # @version 0.10.0
     def extended(base_klass)
       base_klass.instance_eval do
         instance_variable_set(:@__params__, SmartCore::Initializer::Attribute::List.new)
         instance_variable_set(:@__options__, SmartCore::Initializer::Attribute::List.new)
         instance_variable_set(:@__init_extensions__, SmartCore::Initializer::Extensions::List.new)
         instance_variable_set(:@__definer__, SmartCore::Initializer::Constructor::Definer.new(self))
-        instance_variable_set(:@__deflock__, SmartCore::Engine::Lock.new)
         instance_variable_set(:@__initializer_settings__, SmartCore::Initializer::Settings.new)
       end
       base_klass.extend(ClassMethods)
@@ -27,20 +28,20 @@ module SmartCore::Initializer::DSL
 
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   module ClassInheritance
     # @param child_klass [Class]
     # @return [void]
     #
     # @api private
     # @since 0.1.0
-    # @version 0.3.2
+    # @version 0.10.0
     def inherited(child_klass)
       child_klass.instance_exec(__initializer_settings__) do |init_settings|
         instance_variable_set(:@__params__, SmartCore::Initializer::Attribute::List.new)
         instance_variable_set(:@__options__, SmartCore::Initializer::Attribute::List.new)
         instance_variable_set(:@__init_extensions__, SmartCore::Initializer::Extensions::List.new)
         instance_variable_set(:@__definer__, SmartCore::Initializer::Constructor::Definer.new(self))
-        instance_variable_set(:@__deflock__, SmartCore::Engine::Lock.new)
         instance_variable_set(:@__initializer_settings__, init_settings.dup)
       end
       child_klass.extend(ClassMethods)

data/lib/smart_core/initializer/extensions/list.rb

@@ -2,6 +2,7 @@
 
 # @api private
 # @since 0.1.0
+# @version 0.10.0
 class SmartCore::Initializer::Extensions::List
   # @since 0.1.0
   include Enumerable
@@ -10,9 +11,10 @@ class SmartCore::Initializer::Extensions::List
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def initialize
     @extensions = []
-    @lock = SmartCore::Engine::Lock.new
+    @lock = SmartCore::Engine::ReadWriteLock.new
   end
 
   # @param extension [SmartCore::Initializer::Extensions::Abstract]
@@ -20,8 +22,9 @@ class SmartCore::Initializer::Extensions::List
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def add(extension)
-    thread_safe { extensions << extension }
+    @lock.write_sync { extensions << extension }
   end
   alias_method :<<, :add
 
@@ -30,8 +33,9 @@ class SmartCore::Initializer::Extensions::List
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def concat(list)
-    thread_safe do
+    @lock.write_sync do
       list.each { |extension| add(extension.dup) }
     end
   end
@@ -41,8 +45,9 @@ class SmartCore::Initializer::Extensions::List
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def each(&block)
-    thread_safe do
+    @lock.read_sync do
       block_given? ? extensions.each(&block) : extensions.each
     end
   end
@@ -51,8 +56,9 @@ class SmartCore::Initializer::Extensions::List
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def size
-    thread_safe { extensions.size }
+    @lock.read_sync { extensions.size }
   end
 
   private
@@ -62,13 +68,4 @@ class SmartCore::Initializer::Extensions::List
   # @api private
   # @since 0.1.0
   attr_reader :extensions
-
-  # @param block [Block]
-  # @return [Any]
-  #
-  # @api private
-  # @since 0.1.0
-  def thread_safe(&block)
-    @lock.synchronize(&block)
-  end
 end

data/lib/smart_core/initializer/plugins/abstract.rb

@@ -2,6 +2,7 @@
 
 # @api private
 # @since 0.1.0
+# @version 0.10.0
 class SmartCore::Initializer::Plugins::Abstract
   class << self
     # @param child_klass [Class]
@@ -9,9 +10,10 @@ class SmartCore::Initializer::Plugins::Abstract
     #
     # @api private
     # @since 0.1.0
+    # @version 0.10.0
     def inherited(child_klass)
       child_klass.instance_variable_set(:@__loaded__, false)
-      child_klass.instance_variable_set(:@__lock__, Mutex.new)
+      child_klass.instance_variable_set(:@__lock__, SmartCore::Engine::Lock.new)
       super
     end
 

data/lib/smart_core/initializer/plugins/registry.rb

@@ -2,6 +2,7 @@
 
 # @api private
 # @since 0.1.0
+# @version 0.10.0
 class SmartCore::Initializer::Plugins::Registry
   # @since 0.1.0
   include Enumerable
@@ -10,9 +11,10 @@ class SmartCore::Initializer::Plugins::Registry
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def initialize
     @plugin_set = {}
-    @access_lock = Mutex.new
+    @access_lock = SmartCore::Engine::ReadWriteLock.new
   end
 
   # @param plugin_name [Symbol, String]
@@ -20,8 +22,9 @@ class SmartCore::Initializer::Plugins::Registry
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def [](plugin_name)
-    thread_safe { fetch(plugin_name) }
+    @access_lock.read_sync { fetch(plugin_name) }
   end
 
   # @param plugin_name [Symbol, String]
@@ -30,8 +33,9 @@ class SmartCore::Initializer::Plugins::Registry
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def register(plugin_name, plugin_module)
-    thread_safe { apply(plugin_name, plugin_module) }
+    @access_lock.write_sync { apply(plugin_name, plugin_module) }
   end
   alias_method :[]=, :register
 
@@ -39,16 +43,18 @@ class SmartCore::Initializer::Plugins::Registry
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def names
-    thread_safe { plugin_names }
+    @access_lock.read_sync { plugin_names }
   end
 
   # @return [Hash<String,Class<SmartCore::Initializer::Plugins::Abstract>>]
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def loaded
-    thread_safe { loaded_plugins }
+    @access_lock.read_sync { loaded_plugins }
   end
 
   # @param block [Block]
@@ -56,8 +62,9 @@ class SmartCore::Initializer::Plugins::Registry
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def each(&block)
-    thread_safe { iterate(&block) }
+    @access_lock.read_sync { iterate(&block) }
   end
 
   private
@@ -74,14 +81,6 @@ class SmartCore::Initializer::Plugins::Registry
   # @since 0.1.0
   attr_reader :access_lock
 
-  # @return [void]
-  #
-  # @api private
-  # @since 0.1.0
-  def thread_safe
-    access_lock.synchronize { yield if block_given? }
-  end
-
   # @return [Array<String>]
   #
   # @api private

data/lib/smart_core/initializer/plugins/registry_interface.rb

@@ -2,6 +2,7 @@
 
 # @api private
 # @since 0.1.0
+# @version 0.10.0
 module SmartCore::Initializer::Plugins::RegistryInterface
   class << self
     # @param base_module [Class, Module]
@@ -9,12 +10,13 @@ module SmartCore::Initializer::Plugins::RegistryInterface
     #
     # @api private
     # @since 0.1.0
+    # @version 0.10.0
     def extended(base_module)
       base_module.instance_variable_set(
         :@plugin_registry, SmartCore::Initializer::Plugins::Registry.new
       )
       base_module.instance_variable_set(
-        :@access_lock, SmartCore::Engine::Lock.new
+        :@access_lock, SmartCore::Engine::ReadWriteLock.new
       )
     end
   end
@@ -24,24 +26,27 @@ module SmartCore::Initializer::Plugins::RegistryInterface
   #
   # @api public
   # @since 0.1.0
+  # @version 0.10.0
   def load(plugin_name)
-    thread_safe { plugin_registry[plugin_name].load! }
+    @access_lock.read_sync { plugin_registry[plugin_name].load! }
   end
 
   # @return [Array<String>]
   #
   # @api public
   # @since 0.1.0
+  # @version 0.10.0
   def loaded_plugins
-    thread_safe { plugin_registry.loaded.keys }
+    @access_lock.read_sync { plugin_registry.loaded.keys }
   end
 
   # @return [Array<String>]
   #
   # @api public
   # @since 0.1.0
+  # @version 0.10.0
   def names
-    thread_safe { plugin_registry.names }
+    @access_lock.read_sync { plugin_registry.names }
   end
 
   # @param plugin_name [Symbol, String]
@@ -49,8 +54,9 @@ module SmartCore::Initializer::Plugins::RegistryInterface
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def register_plugin(plugin_name, plugin_module)
-    thread_safe { plugin_registry[plugin_name] = plugin_module }
+    @access_lock.write_sync { plugin_registry[plugin_name] = plugin_module }
   end
 
   private
@@ -60,18 +66,4 @@ module SmartCore::Initializer::Plugins::RegistryInterface
   # @api private
   # @since 0.1.0
   attr_reader :plugin_registry
-
-  # @return [SmartCore::Engine::Lock]
-  #
-  # @api private
-  # @since 0.1.0
-  attr_reader :access_lock
-
-  # @return [void]
-  #
-  # @api private
-  # @since 0.1.0
-  def thread_safe
-    access_lock.synchronize { yield if block_given? }
-  end
 end

data/lib/smart_core/initializer/settings/auto_cast.rb

@@ -2,22 +2,15 @@
 
 # @api private
 # @since 0.8.0
+# @version 0.10.0
 class SmartCore::Initializer::Settings::AutoCast < SmartCore::Initializer::Settings::Base
-  # @return [void]
-  #
-  # @api private
-  # @since 0.8.0
-  def initialize
-    @value = nil
-    @lock = SmartCore::Engine::Lock.new
-  end
-
   # @return [Boolean]
   #
   # @api private
   # @since 0.8.0
+  # @version 0.10.0
   def resolve
-    thread_safe do
+    @lock.read_sync do
       @value == nil ? SmartCore::Initializer::Configuration[:auto_cast] : @value
     end
   end
@@ -27,8 +20,9 @@ class SmartCore::Initializer::Settings::AutoCast < SmartCore::Initializer::Setti
   #
   # @api private
   # @since 0.8.0
+  # @version 0.10.0
   def assign(value)
-    thread_safe do
+    @lock.write_sync do
       raise(
         SmartCore::Initializer::SettingArgumentError,
         ":auto_cast setting should be a type of boolean (got: `#{value.class}`)"

data/lib/smart_core/initializer/settings/base.rb

@@ -2,14 +2,16 @@
 
 # @api private
 # @since 0.8.0
+# @version 0.10.0
 class SmartCore::Initializer::Settings::Base
   # @return [void]
   #
   # @api private
   # @since 0.8.0
+  # @version 0.10.0
   def initialize
     @value = nil
-    @lock = SmartCore::Engine::Lock.new
+    @lock = SmartCore::Engine::ReadWriteLock.new
   end
 
   # @!method resolve
@@ -28,22 +30,12 @@ class SmartCore::Initializer::Settings::Base
   #
   # @api private
   # @since 0.8.0
+  # @version 0.10.0
   def dup
-    thread_safe do
+    @lock.write_sync do
       self.class.new.tap do |duplicate|
         duplicate.instance_variable_set(:@value, @value)
       end
     end
   end
-
-  private
-
-  # @param block [Block]
-  # @return [Any]
-  #
-  # @api private
-  # @since 0.8.0
-  def thread_safe(&block)
-    @lock.synchronize(&block)
-  end
 end

data/lib/smart_core/initializer/settings/strict_options.rb

@@ -2,22 +2,15 @@
 
 # @api private
 # @since 0.8.0
+# @version 0.10.0
 class SmartCore::Initializer::Settings::StrictOptions < SmartCore::Initializer::Settings::Base
-  # @return [void]
-  #
-  # @api private
-  # @since 0.8.0
-  def initialize
-    @value = nil
-    @lock = SmartCore::Engine::Lock.new
-  end
-
   # @return [Boolean]
   #
   # @api private
   # @since 0.8.0
+  # @version 0.10.0
   def resolve
-    thread_safe do
+    @lock.read_sync do
       @value == nil ? SmartCore::Initializer::Configuration[:strict_options] : @value
     end
   end
@@ -27,8 +20,9 @@ class SmartCore::Initializer::Settings::StrictOptions < SmartCore::Initializer::
   #
   # @api private
   # @since 0.8.0
+  # @version 0.10.0
   def assign(value)
-    thread_safe do
+    @lock.write_sync do
       raise(
         SmartCore::Initializer::SettingArgumentError,
         ":strict_options setting should be a type of boolean (got: `#{value.class}`)"

data/lib/smart_core/initializer/settings/type_system.rb

@@ -2,24 +2,15 @@
 
 # @api private
 # @since 0.1.0
-# @version 0.8.0
+# @version 0.10.0
 class SmartCore::Initializer::Settings::TypeSystem < SmartCore::Initializer::Settings::Base
-  # @return [void]
-  #
-  # @api private
-  # @since 0.1.0
-  # @version 0.8.0
-  def initialize
-    @value = nil
-    @lock = SmartCore::Engine::Lock.new
-  end
-
   # @return [Any]
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def generic_type_object
-    thread_safe do
+    @lock.read_sync do
       SmartCore::Initializer::TypeSystem.resolve(resolve).generic_type_object
     end
   end
@@ -28,9 +19,9 @@ class SmartCore::Initializer::Settings::TypeSystem < SmartCore::Initializer::Set
   #
   # @api private
   # @since 0.1.0
-  # @version 0.8.0
+  # @version 0.10.0
   def resolve
-    thread_safe do
+    @lock.read_sync do
       @value == nil ? SmartCore::Initializer::Configuration[:default_type_system] : @value
     end
   end
@@ -40,9 +31,9 @@ class SmartCore::Initializer::Settings::TypeSystem < SmartCore::Initializer::Set
   #
   # @api private
   # @since 0.1.0
-  # @version 0.8.0
+  # @version 0.10.0
   def assign(value)
-    thread_safe do
+    @lock.write_sync do
       # NOTE: type system existence validation
       SmartCore::Initializer::TypeSystem.resolve(value)
       @value = value

data/lib/smart_core/initializer/type_system/interop/aliasing/alias_list.rb

@@ -2,32 +2,36 @@
 
 # @api private
 # @since 0.1.0
+# @version 0.10.0
 class SmartCore::Initializer::TypeSystem::Interop::Aliasing::AliasList
   # @param interop_klass [Class<SmartCore::Initializer::TypeSystem::Interop>]
   # @return [void]
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def initialize(interop_klass)
     @interop_klass = interop_klass
     @list = {}
-    @lock = SmartCore::Engine::Lock.new
+    @lock = SmartCore::Engine::ReadWriteLock.new
   end
 
   # @return [Array<String>]
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def keys
-    thread_safe { registered_aliases }
+    @lock.read_sync { registered_aliases }
   end
 
   # @return [Hash<String,Any>]
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def to_h
-    thread_safe { transform_to_hash }
+    @lock.read_sync { transform_to_hash }
   end
   alias_method :to_hash, :to_h
 
@@ -37,9 +41,10 @@ class SmartCore::Initializer::TypeSystem::Interop::Aliasing::AliasList
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def associate(alias_name, type)
     interop_klass.prevent_incompatible_type!(type)
-    thread_safe { set_alias(alias_name, type) }
+    @lock.write_sync { set_alias(alias_name, type) }
   end
 
   # @param alias_name [String, Symbol]
@@ -47,8 +52,9 @@ class SmartCore::Initializer::TypeSystem::Interop::Aliasing::AliasList
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def resolve(alias_name)
-    thread_safe { get_alias(alias_name) }
+    @lock.read_sync { get_alias(alias_name) }
   end
 
   private
@@ -65,15 +71,6 @@ class SmartCore::Initializer::TypeSystem::Interop::Aliasing::AliasList
   # @since 0.1.0
   attr_reader :interop_klass
 
-  # @param block [Block]
-  # @return [Any]
-  #
-  # @api private
-  # @since 0.1.0
-  def thread_safe(&block)
-    @lock.synchronize(&block)
-  end
-
   # @param alias_name [String, Symbol]
   # @param type [Any]
   # @return [void]
@@ -97,16 +94,16 @@ class SmartCore::Initializer::TypeSystem::Interop::Aliasing::AliasList
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def get_alias(alias_name)
     alias_name = normalized_alias(alias_name)
 
-    begin
-      list.fetch(alias_name)
-    rescue ::KeyError
-      raise(SmartCore::Initializer::TypeAliasNotFoundError, <<~ERROR_MESSAGE)
-        Alias with name "#{alias_name}" not found.
-      ERROR_MESSAGE
-    end
+    raise(
+      SmartCore::Initializer::TypeAliasNotFoundError,
+      "Alias with name `#{alias_name}` not found."
+    ) unless list.key?(alias_name)
+
+    list.fetch(alias_name)
   end
 
   # @param alias_name [String, Symbol]