smart_container 0.8.1 → 0.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fbae5c2d40dc40e3114d685c1f13d6cbfb7003a1c8fda862ef5dbcfa432cf5cd
-  data.tar.gz: dd288fa47167c0da4e2aae656478170d0dd78b142eef52a351c461ddbc92bf29
+  metadata.gz: 929c6fcba625ebb836b753d4563082ea2d0aa793abbf840ed97fe6f209a0c7b0
+  data.tar.gz: b9cf58d9fc718aff98847e74f986b86da7984c3537f5258a37a2c67f9af611e8
 SHA512:
-  metadata.gz: 27129a91c84f8aad432c5462c2d3a921284b2b8e3b1fb4b0a215295f6f88d94063946cc24f2d065f6da9ba6e64baf421f2a192b68fa63ebda879a81715e2c88f
-  data.tar.gz: 5e6839adcfe1e3c31513b622fc3e46aab37ab3ec35a992c8a3b9c7ff029c976dbf331619d3df20da7ffe3d712a8839f04ac83f3055f9e2426869e9b9ecf556f6
+  metadata.gz: 93e6062a36edf2f06d37132dc53b55c87f5f0aac277eafec6a17572ff60f73e0c164ed4fae53ee5333c10556e605e8d4c9c29b935406f7a9dceb0280e7bfa4df
+  data.tar.gz: a653c4ded2e96d9fe4f42292eabe0467ad1cfe3a2633aa3bd4f77506d69f65c0a32703ecffb6875eb5d3de9d3e34592fdebc95c50417257c5a864273b9ae116c

data/.rubocop.yml

@@ -5,7 +5,8 @@ inherit_gem:
     - lib/rubocop.rspec.yml
 
 AllCops:
-  TargetRubyVersion: 2.7.1
+  TargetRubyVersion: 3.1
+  NewCops: enable
   Include:
     - lib/**/*.rb
     - spec/**/*.rb
@@ -13,3 +14,7 @@ AllCops:
     - Rakefile
     - smart_container.gemspec
     - bin/console
+
+Lint/EmptyBlock:
+  Exclude:
+    - spec/**/*.rb

data/CHANGELOG.md

@@ -1,6 +1,25 @@
 # Changelog
 All notable changes to this project will be documented in this file.
 
+## [0.10.0] - 2022-10-14
+### Changed
+- Simple `Mutex`-based locks was replaced with `SmartCore::Engine::ReadWriteLock` in order to decrease
+  context switching during method resolving inside RubyVM (to reduce thread locks when it is not necessary);
+- Development progress:
+  - Minimal ruby version - **2.5**;
+  - Updated development dependencies;
+  - Updated `smart_engine` dependency (`~> 0.11` -> `~> 0.17`);
+
+
+## [0.9.0] - 2020-01-17
+### Added
+- Support for **Ruby@3**;
+- Updated development dependencies;
+
+### Changed
+- No more TravisCI (todo: migrate to Github Actions);
+- Minimal `smart_engine` version: **0.11.0** (in order to support **Ruby@3**);
+
 ## [0.8.1] - 2020-07-09
 ### Changed
 - *Core*

data/Gemfile.lock

@@ -1,98 +1,137 @@
 PATH
   remote: .
   specs:
-    smart_container (0.8.1)
-      smart_engine (~> 0.5)
+    smart_container (0.10.0)
+      smart_engine (~> 0.17)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (6.0.2.2)
+    activesupport (7.0.4)
       concurrent-ruby (~> 1.0, >= 1.0.2)
-      i18n (>= 0.7, < 2)
-      minitest (~> 5.1)
-      tzinfo (~> 1.1)
-      zeitwerk (~> 2.2)
-    armitage-rubocop (0.81.0)
-      rubocop (= 0.81.0)
-      rubocop-performance (= 1.5.2)
-      rubocop-rails (= 2.5.2)
-      rubocop-rake (= 0.5.1)
-      rubocop-rspec (= 1.38.1)
-    ast (2.4.0)
-    coderay (1.1.2)
-    concurrent-ruby (1.1.6)
-    diff-lcs (1.3)
-    docile (1.3.2)
-    i18n (1.8.2)
+      i18n (>= 1.6, < 2)
+      minitest (>= 5.1)
+      tzinfo (~> 2.0)
+    armitage-rubocop (1.36.0)
+      rubocop (= 1.36.0)
+      rubocop-performance (= 1.15.0)
+      rubocop-rails (= 2.16.1)
+      rubocop-rake (= 0.6.0)
+      rubocop-rspec (= 2.13.2)
+    ast (2.4.2)
+    backport (1.2.0)
+    benchmark (0.2.0)
+    coderay (1.1.3)
+    concurrent-ruby (1.1.10)
+    diff-lcs (1.5.0)
+    docile (1.4.0)
+    e2mmap (0.1.0)
+    i18n (1.12.0)
       concurrent-ruby (~> 1.0)
     jaro_winkler (1.5.4)
+    json (2.6.2)
+    kramdown (2.4.0)
+      rexml
+    kramdown-parser-gfm (1.1.0)
+      kramdown (~> 2.0)
     method_source (1.0.0)
-    minitest (5.14.0)
-    parallel (1.19.1)
-    parser (2.7.1.0)
-      ast (~> 2.4.0)
-    pry (0.13.1)
+    minitest (5.16.3)
+    nokogiri (1.13.8-arm64-darwin)
+      racc (~> 1.4)
+    parallel (1.22.1)
+    parser (3.1.2.1)
+      ast (~> 2.4.1)
+    pry (0.14.1)
       coderay (~> 1.1)
       method_source (~> 1.0)
-    rack (2.2.2)
-    rainbow (3.0.0)
-    rake (13.0.1)
-    rexml (3.2.4)
-    rspec (3.9.0)
-      rspec-core (~> 3.9.0)
-      rspec-expectations (~> 3.9.0)
-      rspec-mocks (~> 3.9.0)
-    rspec-core (3.9.1)
-      rspec-support (~> 3.9.1)
-    rspec-expectations (3.9.1)
+    racc (1.6.0)
+    rack (3.0.0)
+    rainbow (3.1.1)
+    rake (13.0.6)
+    regexp_parser (2.6.0)
+    reverse_markdown (2.1.1)
+      nokogiri
+    rexml (3.2.5)
+    rspec (3.11.0)
+      rspec-core (~> 3.11.0)
+      rspec-expectations (~> 3.11.0)
+      rspec-mocks (~> 3.11.0)
+    rspec-core (3.11.0)
+      rspec-support (~> 3.11.0)
+    rspec-expectations (3.11.1)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
-    rspec-mocks (3.9.1)
+      rspec-support (~> 3.11.0)
+    rspec-mocks (3.11.1)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
-    rspec-support (3.9.2)
-    rubocop (0.81.0)
-      jaro_winkler (~> 1.5.1)
+      rspec-support (~> 3.11.0)
+    rspec-support (3.11.1)
+    rubocop (1.36.0)
+      json (~> 2.3)
       parallel (~> 1.10)
-      parser (>= 2.7.0.1)
+      parser (>= 3.1.2.1)
       rainbow (>= 2.2.2, < 4.0)
-      rexml
+      regexp_parser (>= 1.8, < 3.0)
+      rexml (>= 3.2.5, < 4.0)
+      rubocop-ast (>= 1.20.1, < 2.0)
       ruby-progressbar (~> 1.7)
-      unicode-display_width (>= 1.4.0, < 2.0)
-    rubocop-performance (1.5.2)
-      rubocop (>= 0.71.0)
-    rubocop-rails (2.5.2)
-      activesupport
+      unicode-display_width (>= 1.4.0, < 3.0)
+    rubocop-ast (1.21.0)
+      parser (>= 3.1.1.0)
+    rubocop-performance (1.15.0)
+      rubocop (>= 1.7.0, < 2.0)
+      rubocop-ast (>= 0.4.0)
+    rubocop-rails (2.16.1)
+      activesupport (>= 4.2.0)
       rack (>= 1.1)
-      rubocop (>= 0.72.0)
-    rubocop-rake (0.5.1)
-      rubocop
-    rubocop-rspec (1.38.1)
-      rubocop (>= 0.68.1)
-    ruby-progressbar (1.10.1)
-    simplecov (0.18.5)
+      rubocop (>= 1.33.0, < 2.0)
+    rubocop-rake (0.6.0)
+      rubocop (~> 1.0)
+    rubocop-rspec (2.13.2)
+      rubocop (~> 1.33)
+    ruby-progressbar (1.11.0)
+    simplecov (0.21.2)
       docile (~> 1.1)
       simplecov-html (~> 0.11)
-    simplecov-html (0.12.2)
-    smart_engine (0.7.0)
-    thread_safe (0.3.6)
-    tzinfo (1.2.7)
-      thread_safe (~> 0.1)
-    unicode-display_width (1.7.0)
-    zeitwerk (2.3.0)
+      simplecov_json_formatter (~> 0.1)
+    simplecov-html (0.12.3)
+    simplecov_json_formatter (0.1.4)
+    smart_engine (0.17.0)
+    solargraph (0.47.2)
+      backport (~> 1.2)
+      benchmark
+      bundler (>= 1.17.2)
+      diff-lcs (~> 1.4)
+      e2mmap
+      jaro_winkler (~> 1.5)
+      kramdown (~> 2.3)
+      kramdown-parser-gfm (~> 1.1)
+      parser (~> 3.0)
+      reverse_markdown (>= 1.0.5, < 3)
+      rubocop (>= 0.52)
+      thor (~> 1.0)
+      tilt (~> 2.0)
+      yard (~> 0.9, >= 0.9.24)
+    thor (1.2.1)
+    tilt (2.0.11)
+    tzinfo (2.0.5)
+      concurrent-ruby (~> 1.0)
+    unicode-display_width (2.3.0)
+    webrick (1.7.0)
+    yard (0.9.28)
+      webrick (~> 1.7.0)
 
 PLATFORMS
-  ruby
+  arm64-darwin-21
 
 DEPENDENCIES
-  armitage-rubocop (~> 0.81)
-  bundler (~> 2.1)
-  pry (~> 0.13)
+  armitage-rubocop (~> 1.36)
+  bundler (~> 2.3)
+  pry (~> 0.14)
   rake (~> 13.0)
-  rspec (~> 3.9)
-  simplecov (~> 0.18)
+  rspec (~> 3.11)
+  simplecov (~> 0.21)
   smart_container!
+  solargraph (~> 0.47)
 
 BUNDLED WITH
-   2.1.4
+   2.3.23

data/LICENSE.txt

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2020 Rustam Ibragimov
+Copyright (c) 2020-2021 Rustam Ibragimov
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/README.md

@@ -1,6 +1,14 @@
-# SmartCore::Container · [![Gem Version](https://badge.fury.io/rb/smart_container.svg)](https://badge.fury.io/rb/smart_container) [![Build Status](https://travis-ci.org/smart-rb/smart_container.svg?branch=master)](https://travis-ci.org/smart-rb/smart_container)
+# SmartCore::Container &middot; <a target="_blank" href="https://github.com/Cado-Labs"><img src="https://github.com/Cado-Labs/cado-labs-logos/raw/main/cado_labs_badge.svg" alt="Supported by Cado Labs" style="max-width: 100%; height: 20px"></a> &middot; [![Gem Version](https://badge.fury.io/rb/smart_container.png)](https://badge.fury.io/rb/smart_container)
 
-Thread-safe semanticaly-defined IoC/DI Container.
+Thread-safe semanticaly-defined IoC/DI Container with a developer-friendly DSL and API.
+
+---
+
+<p>
+  <a href="https://github.com/Cado-Labs">
+    <img src="https://github.com/Cado-Labs/cado-labs-logos/blob/main/cado_labs_supporting.svg" alt="Supported by Cado Labs" />
+  </a>
+</p>
 
 ---
 
@@ -306,6 +314,8 @@ container.namespace('database') {} # no one to listen this changement... :)
 
 ## Roadmap
 
+- migrate to Github Actions;
+
 - convinient way to rebind registered dependnecies:
 
 ```ruby
@@ -343,6 +353,17 @@ resolve('logger', :allocate) # Draft
 
 - support for fallback block in `.resolve` operation (similar to `Hash#fetch` works);
 
+- inline temporary dependency switch:
+
+```ruby
+with(logger: Logger.new, db: DB.new) do
+  # logger is a new logger
+  # db is a new db
+end
+
+# out of block: logger is an old logger, db is an old db
+```
+
 ---
 
 ## Contributing
@@ -357,6 +378,12 @@ resolve('logger', :allocate) # Draft
 
 Released under MIT License.
 
+## Supporting
+
+<a href="https://github.com/Cado-Labs">
+  <img src="https://github.com/Cado-Labs/cado-labs-logos/blob/main/cado_labs_logo.png" alt="Supported by Cado Labs" />
+</a>
+
 ## Authors
 
 [Rustam Ibragimov](https://github.com/0exp)

data/lib/smart_core/container/definition_dsl.rb

@@ -17,7 +17,7 @@ class SmartCore::Container
       def included(base_klass)
         base_klass.instance_variable_set(:@__container_definition_commands__, CommandSet.new)
         base_klass.instance_variable_set(:@__container_instantiation_commands__, CommandSet.new)
-        base_klass.instance_variable_set(:@__container_definition_lock__, ArbitraryLock.new)
+        base_klass.instance_variable_set(:@__container_definition_lock__, SmartCore::Engine::ReadWriteLock.new)
         base_klass.singleton_class.send(:attr_reader, :__container_definition_commands__)
         base_klass.singleton_class.send(:attr_reader, :__container_instantiation_commands__)
         base_klass.extend(ClassMethods)
@@ -36,7 +36,7 @@ class SmartCore::Container
       def inherited(child_klass)
         child_klass.instance_variable_set(:@__container_definition_commands__, CommandSet.new)
         child_klass.instance_variable_set(:@__container_instantiation_commands__, CommandSet.new)
-        child_klass.instance_variable_set(:@__container_definition_lock__, ArbitraryLock.new)
+        child_klass.instance_variable_set(:@__container_definition_lock__, SmartCore::Engine::ReadWriteLock.new)
         SmartCore::Container::DefinitionDSL::Inheritance.inherit(base: self, child: child_klass)
         child_klass.singleton_class.prepend(ClassInheritance)
         super
@@ -53,7 +53,7 @@ class SmartCore::Container
       # @api public
       # @since 0.1.0
       def namespace(namespace_name, &dependencies_definition)
-        @__container_definition_lock__.thread_safe do
+        @__container_definition_lock__.write_sync do
           DependencyCompatability::Definition.prevent_dependency_overlap!(self, namespace_name)
 
           __container_definition_commands__ << Commands::Definition::Namespace.new(
@@ -75,7 +75,7 @@ class SmartCore::Container
         memoize: SmartCore::Container::Registry::DEFAULT_MEMOIZATION_BEHAVIOR,
         &dependency_definition
       )
-        @__container_definition_lock__.thread_safe do
+        @__container_definition_lock__.write_sync do
           DependencyCompatability::Definition.prevent_namespace_overlap!(self, dependency_name)
 
           __container_definition_commands__ << Commands::Definition::Register.new(
@@ -90,7 +90,7 @@ class SmartCore::Container
       # @api public
       # @since 0.1.0
       def compose(container_klass)
-        @__container_definition_lock__.thread_safe do
+        @__container_definition_lock__.write_sync do
           __container_definition_commands__ << Commands::Definition::Compose.new(
             container_klass
           )
@@ -106,7 +106,7 @@ class SmartCore::Container
       # @api public
       # @since 0.1.0
       def freeze_state!
-        @__container_definition_lock__.thread_safe do
+        @__container_definition_lock__.write_sync do
           __container_instantiation_commands__ << Commands::Instantiation::FreezeState.new
         end
       end

data/lib/smart_core/container/dependency_resolver.rb

@@ -63,7 +63,7 @@ module SmartCore::Container::DependencyResolver
       entity = extract(container, dependency_path)
 
       case
-      when memoized.nil?
+      when memoized == nil
         entity.is_a?(SmartCore::Container::Entities::Dependency)
       when !!memoized == true
         entity.is_a?(SmartCore::Container::Entities::MemoizedDependency)

data/lib/smart_core/container/dependency_watcher.rb

@@ -123,7 +123,7 @@ class SmartCore::Container::DependencyWatcher
   # @api private
   # @since 0.8.0
   def remove_listeners(entity_path) # TODO: support for pattern-based pathes
-    if entity_path.nil?
+    if entity_path == nil
       observers.each_value(&:clear)
     else
       entity_path = indifferently_accessable_path(entity_path)

data/lib/smart_core/container/entities/memoized_dependency.rb

@@ -3,6 +3,7 @@
 module SmartCore::Container::Entities
   # @api private
   # @since 0.2.0
+  # @version 0.10.0
   class MemoizedDependency < Dependency
     # @param dependency_name [String]
     # @param dependency_definition [Proc]
@@ -10,9 +11,10 @@ module SmartCore::Container::Entities
     #
     # @api private
     # @since 0.2.0
+    # @version 0.10.0
     def initialize(dependency_name, dependency_definition)
       super(dependency_name, dependency_definition)
-      @lock = SmartCore::Container::ArbitraryLock.new
+      @lock = SmartCore::Engine::ReadWriteLock.new
     end
 
     # @param host_container [SmartCore::Container, NilClass]
@@ -22,7 +24,7 @@ module SmartCore::Container::Entities
     # @since 0.2.0
     # @version 0.8.1
     def reveal(host_container = SmartCore::Container::NO_HOST_CONTAINER)
-      @lock.thread_safe do
+      @lock.read_sync do
         unless instance_variable_defined?(:@revealed_dependency)
           @revealed_dependency = dependency_definition.call
         else

data/lib/smart_core/container/entities/namespace.rb

@@ -2,6 +2,7 @@
 
 # @api private
 # @since 0.1.0
+# @version 0.10.0
 class SmartCore::Container::Entities::Namespace < SmartCore::Container::Entities::Base
   # @return [String]
   #
@@ -21,13 +22,13 @@ class SmartCore::Container::Entities::Namespace < SmartCore::Container::Entities
   #
   # @api private
   # @since 0.1.0
-  # @version 0.8.1
+  # @version 0.10.0
   def initialize(namespace_name, host_container = SmartCore::Container::NO_HOST_CONTAINER)
     super(namespace_name)
     @container_klass = Class.new(SmartCore::Container)
     @container_instance = nil
     @host_container = host_container
-    @lock = SmartCore::Container::ArbitraryLock.new
+    @lock = SmartCore::Engine::ReadWriteLock.new
   end
 
   # @param runtime_host_container [SmartCore::Container, NilClass]
@@ -35,26 +36,26 @@ class SmartCore::Container::Entities::Namespace < SmartCore::Container::Entities
   #
   # @api private
   # @since 0.1.0
-  # @version 0.8.1
+  # @version 0.10.0
   def reveal(runtime_host_container = SmartCore::Container::NO_HOST_CONTAINER)
-    thread_safe { container_instance(runtime_host_container) }
+    @lock.read_sync { container_instance(runtime_host_container) }
   end
 
   # @param dependencies_definition [Proc]
   # @return [void]
   #
   # @api private
-  # @since 0.1.0
+  # @since 0.10.0
   def append_definitions(dependencies_definition)
-    thread_safe { container_klass.instance_eval(&dependencies_definition) }
+    @lock.write_sync { container_klass.instance_eval(&dependencies_definition) }
   end
 
   # @return [void]
   #
   # @api private
-  # @since 0.1.0
+  # @since 0.10.0
   def freeze!
-    thread_safe { container_instance.freeze! }
+    @lock.write_sync { container_instance.freeze! }
   end
 
   private
@@ -78,13 +79,4 @@ class SmartCore::Container::Entities::Namespace < SmartCore::Container::Entities
       host_path: @host_container && namespace_name
     )
   end
-
-  # @param block [Block]
-  # @return [Any]
-  #
-  # @api private
-  # @since 0.1.0
-  def thread_safe(&block)
-    @lock.thread_safe(&block)
-  end
 end

data/lib/smart_core/container/host.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+using SmartCore::Ext::BasicObjectAsObject
+
 # @api private
 # @since 0.8.1
 class SmartCore::Container::Host
@@ -10,7 +12,8 @@ class SmartCore::Container::Host
     #
     # @api private
     # @since 0.8.1
-    def build(container, path) # rubocop:disable Metrics/AbcSize
+    # rubocop:disable Metrics/AbcSize, Style/NilComparison
+    def build(container, path)
       if (container.nil? && !path.nil?) || (!container.nil? && path.nil?)
         raise(SmartCore::Container::ArgumentError, <<~ERROR_MESSAGE)
           Host container requires both host container instance and host container path
@@ -28,6 +31,7 @@ class SmartCore::Container::Host
 
       new(container, path)
     end
+    # rubocop:enable Metrics/AbcSize, Style/NilComparison
   end
 
   # @return [SmartCore::Container]

data/lib/smart_core/container/registry.rb

@@ -2,6 +2,7 @@
 
 # @api private
 # @since 0.1.0
+# @version 0.10.0
 # rubocop:disable Metrics/ClassLength
 class SmartCore::Container::Registry
   # @since 0.1.0
@@ -35,9 +36,10 @@ class SmartCore::Container::Registry
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def initialize
     @registry = {}
-    @access_lock = SmartCore::Container::ArbitraryLock.new
+    @lock = SmartCore::Engine::ReadWriteLock.new
   end
 
   # @param entity_path [String, Symbol]
@@ -45,8 +47,9 @@ class SmartCore::Container::Registry
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def resolve(entity_path)
-    thread_safe { fetch_entity(entity_path) }
+    @lock.read_sync { fetch_entity(entity_path) }
   end
 
   # @param name [String, Symbol]
@@ -56,9 +59,9 @@ class SmartCore::Container::Registry
   #
   # @api private
   # @since 0.1.0
-  # @version 0.3.0
+  # @version 0.10.0
   def register_dependency(name, memoize: DEFAULT_MEMOIZATION_BEHAVIOR, &dependency_definition)
-    thread_safe { add_dependency(name, dependency_definition, memoize) }
+    @lock.write_sync { add_dependency(name, dependency_definition, memoize) }
   end
 
   # @param name [String, Symbol]
@@ -68,29 +71,31 @@ class SmartCore::Container::Registry
   #
   # @api private
   # @since 0.1.0
-  # @version 0.8.1
+  # @version 0.10.0
   def register_namespace(
     name,
     host_container = SmartCore::Container::NO_HOST_CONTAINER,
     &dependencies_definition
   )
-    thread_safe { add_namespace(name, host_container, dependencies_definition) }
+    @lock.write_sync { add_namespace(name, host_container, dependencies_definition) }
   end
 
   # @return [void]
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def freeze!
-    thread_safe { freeze_state! }
+    @lock.write_sync { freeze_state! }
   end
 
   # @return [Boolean]
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def frozen?
-    thread_safe { state_frozen? }
+    @lock.read_sync { state_frozen? }
   end
 
   # @param block [Block]
@@ -98,8 +103,9 @@ class SmartCore::Container::Registry
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def each(&block)
-    thread_safe { enumerate(&block) }
+    @lock.read_sync { enumerate(&block) }
   end
 
   # @param root_dependency_name [NilClass, String]
@@ -109,12 +115,13 @@ class SmartCore::Container::Registry
   #
   # @api private
   # @since 0.4.0
+  # @version 0.10.0
   def each_dependency(
     root_dependency_name = nil,
     yield_all: DEFAULT_ITERATION_YIELD_BEHAVIOUR,
     &block
   )
-    thread_safe { iterate(root_dependency_name, yield_all: yield_all, &block) }
+    @lock.read_sync { iterate(root_dependency_name, yield_all: yield_all, &block) }
   end
 
   # @option all_variants [Boolean]
@@ -122,28 +129,24 @@ class SmartCore::Container::Registry
   #
   # @api private
   # @since 0.4.0
+  # @version 0.10.0
   def keys(all_variants: DEFAULT_KEY_EXTRACTION_BEHAVIOUR)
-    thread_safe { extract_keys(all_variants: all_variants) }
+    @lock.read_sync { extract_keys(all_variants: all_variants) }
   end
 
   # @return [Hash<String|Symbol,SmartCore::Container::Entities::Base|Any>]
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def hash_tree(resolve_dependencies: false)
-    thread_safe { build_hash_tree(resolve_dependencies: resolve_dependencies) }
+    @lock.read_sync { build_hash_tree(resolve_dependencies: resolve_dependencies) }
   end
   alias_method :to_h, :hash_tree
   alias_method :to_hash, :hash_tree
 
   private
 
-  # @return [Mutex]
-  #
-  # @api private
-  # @since 0.1.0
-  attr_reader :lock
-
   # @return [Boolean]
   #
   # @api private
@@ -246,7 +249,6 @@ class SmartCore::Container::Registry
     dependencies_definition ||= proc {}
     prevent_dependency_overlap!(namespace_name)
 
-    # rubocop:disable Layout/RescueEnsureAlignment
     namespace_entity = begin
       fetch_entity(namespace_name)
     rescue SmartCore::Container::FetchError
@@ -254,8 +256,6 @@ class SmartCore::Container::Registry
         namespace_name, host_container
       )
     end
-    # rubocop:enable Layout/RescueEnsureAlignment
-
     namespace_entity.tap { namespace_entity.append_definitions(dependencies_definition) }
   end
 
@@ -275,8 +275,8 @@ class SmartCore::Container::Registry
         final_dependency_name =
           if root_dependency_name
             "#{root_dependency_name}" \
-            "#{SmartCore::Container::DependencyResolver::PATH_PART_SEPARATOR}" \
-            "#{dependency_name}"
+              "#{SmartCore::Container::DependencyResolver::PATH_PART_SEPARATOR}" \
+              "#{dependency_name}"
           else
             dependency_name
           end
@@ -343,14 +343,5 @@ class SmartCore::Container::Registry
       self, namespace_name
     )
   end
-
-  # @param block [Proc]
-  # @return [Any]
-  #
-  # @api private
-  # @since 0.1.0
-  def thread_safe(&block)
-    @access_lock.thread_safe(&block)
-  end
 end
 # rubocop:enable Metrics/ClassLength

data/lib/smart_core/container/version.rb

@@ -1,10 +1,12 @@
 # frozen_string_literal: true
 
 module SmartCore
-  class Container
+  class Container # rubocop:disable Style/StaticClass
+    # @return [String]
+    #
     # @api public
     # @since 0.1.0
-    # @version 0.8.1
-    VERSION = '0.8.1'
+    # @version 0.10.0
+    VERSION = '0.10.0'
   end
 end

data/lib/smart_core/container.rb

@@ -7,10 +7,10 @@ require 'smart_core'
 module SmartCore
   # @api public
   # @since 0.1.0
+  # @version 0.10.0
   class Container # rubocop:disable Metrics/ClassLength
     require_relative 'container/version'
     require_relative 'container/errors'
-    require_relative 'container/arbitrary_lock'
     require_relative 'container/key_guard'
     require_relative 'container/entities'
     require_relative 'container/definition_dsl'
@@ -82,13 +82,13 @@ module SmartCore
     #
     # @api public
     # @since 0.1.0
-    # @version 0.8.1
+    # @version 0.10.0
     def initialize(host_container: NO_HOST_CONTAINER, host_path: NO_HOST_PATH)
       @host = SmartCore::Container::Host.build(host_container, host_path)
       build_registry!
       @watcher = SmartCore::Container::DependencyWatcher.new(self)
       @host_path = host_path
-      @access_lock = ArbitraryLock.new
+      @lock = SmartCore::Engine::ReadWriteLock.new
     end
 
     # @param dependency_name [String, Symbol]
@@ -97,13 +97,14 @@ module SmartCore
     #
     # @api public
     # @sicne 0.1.0
-    # @version 0.8.0
+    # @version 0.10.0
     def register(
       dependency_name,
       memoize: SmartCore::Container::Registry::DEFAULT_MEMOIZATION_BEHAVIOR,
       &dependency_definition
     )
-      thread_safe do
+
+      @lock.write_sync do
         registry.register_dependency(dependency_name, memoize: memoize, &dependency_definition)
         watcher.notify(dependency_name)
       end
@@ -115,9 +116,9 @@ module SmartCore
     #
     # @api public
     # @since 0.1.0
-    # @version 0.8.0
+    # @version 0.10.0
     def namespace(namespace_name, &dependencies_definition)
-      thread_safe do
+      @lock.write_sync do
         registry.register_namespace(namespace_name, self, &dependencies_definition)
         watcher.notify(namespace_name)
       end
@@ -128,9 +129,9 @@ module SmartCore
     #
     # @api public
     # @since 0.1.0
-    # @version 0.1.0
+    # @version 0.10.0
     def resolve(dependency_path)
-      thread_safe { DependencyResolver.resolve(self, dependency_path) }
+      @lock.read_sync { DependencyResolver.resolve(self, dependency_path) }
     end
     alias_method :[], :resolve
 
@@ -139,32 +140,36 @@ module SmartCore
     #
     # @api public
     # @since 0.1.0
+    # @version 0.10.0
     def fetch(dependency_path)
-      thread_safe { DependencyResolver.fetch(self, dependency_path) }
+      @lock.read_sync { DependencyResolver.fetch(self, dependency_path) }
     end
 
     # @return [void]
     #
     # @api public
     # @since 0.1.0
+    # @version 0.10.0
     def freeze!
-      thread_safe { registry.freeze! }
+      @lock.write_sync { registry.freeze! }
     end
 
     # @return [Boolean]
     #
     # @api public
     # @since 0.1.0
+    # @version 0.10.0
     def frozen?
-      thread_safe { registry.frozen? }
+      @lock.read_sync { registry.frozen? }
     end
 
     # @return [void]
     #
     # @api public
     # @since 0.1.0
+    # @version 0.10.0
     def reload!
-      thread_safe { build_registry! }
+      @lock.write_sync { build_registry! }
     end
 
     # @option all_variants [Boolean]
@@ -172,8 +177,9 @@ module SmartCore
     #
     # @api public
     # @since 0.4.0
+    # @version 0.10.0
     def keys(all_variants: SmartCore::Container::Registry::DEFAULT_KEY_EXTRACTION_BEHAVIOUR)
-      thread_safe { registry.keys(all_variants: all_variants) }
+      @lock.read_sync { registry.keys(all_variants: all_variants) }
     end
 
     # @param key [String, Symbol]
@@ -181,8 +187,9 @@ module SmartCore
     #
     # @api public
     # @since 0.5.0
+    # @version 0.10.0
     def key?(key)
-      thread_safe { DependencyResolver.key?(self, key) }
+      @lock.read_sync { DependencyResolver.key?(self, key) }
     end
 
     # @param namespace_path [String, Symbol]
@@ -190,8 +197,9 @@ module SmartCore
     #
     # @api public
     # @since 0.5.0
+    # @version 0.10.0
     def namespace?(namespace_path)
-      thread_safe { DependencyResolver.namespace?(self, namespace_path) }
+      @lock.read_sync { DependencyResolver.namespace?(self, namespace_path) }
     end
 
     # @param dependency_path [String, Symbol]
@@ -200,8 +208,9 @@ module SmartCore
     #
     # @api public
     # @since 0.5.0
+    # @version 0.10.0
     def dependency?(dependency_path, memoized: nil)
-      thread_safe { DependencyResolver.dependency?(self, dependency_path, memoized: memoized) }
+      @lock.read_sync { DependencyResolver.dependency?(self, dependency_path, memoized: memoized) }
     end
 
     # @option yield_all [Boolean]
@@ -213,11 +222,12 @@ module SmartCore
     #
     # @api public
     # @since 0.4.0
+    # @version 0.10.0
     def each_dependency(
       yield_all: SmartCore::Container::Registry::DEFAULT_ITERATION_YIELD_BEHAVIOUR,
       &block
     )
-      thread_safe { registry.each_dependency(yield_all: yield_all, &block) }
+      @lock.read_sync { registry.each_dependency(yield_all: yield_all, &block) }
     end
     alias_method :each, :each_dependency
     alias_method :each_pair, :each_dependency
@@ -227,8 +237,9 @@ module SmartCore
     #
     # @api public
     # @since 0.1.0
+    # @version 0.10.0
     def hash_tree(resolve_dependencies: false)
-      thread_safe { registry.hash_tree(resolve_dependencies: resolve_dependencies) }
+      @lock.read_sync { registry.hash_tree(resolve_dependencies: resolve_dependencies) }
     end
     alias_method :to_h, :hash_tree
     alias_method :to_hash, :hash_tree
@@ -242,8 +253,9 @@ module SmartCore
     #
     # @api public
     # @since 0.8.0
+    # @version 0.10.0
     def observe(entity_path, &observer) # TODO: support for pattern-based pathes
-      thread_safe { watcher.watch(entity_path, &observer) }
+      @lock.write_sync { watcher.watch(entity_path, &observer) }
     end
     alias_method :subscribe, :observe
 
@@ -252,8 +264,9 @@ module SmartCore
     #
     # @api public
     # @since 0.8.0
+    # @version 0.10.0
     def unobserve(observer)
-      thread_safe { watcher.unwatch(observer) }
+      @lock.write_sync { watcher.unwatch(observer) }
     end
     alias_method :unsubscribe, :unobserve
 
@@ -262,8 +275,9 @@ module SmartCore
     #
     # @api public
     # @since 0.8.0
+    # @version 0.10.0
     def clear_observers(entity_path = nil) # TODO: support for pattern-based pathes
-      thread_safe { watcher.clear_listeners(entity_path) }
+      @lock.write_sync { watcher.clear_listeners(entity_path) }
     end
     alias_method :clear_listeners, :clear_observers
 
@@ -277,14 +291,5 @@ module SmartCore
     def build_registry!
       @registry = RegistryBuilder.build(self)
     end
-
-    # @param block [Block]
-    # @return [Any]
-    #
-    # @api private
-    # @since 0.1.0
-    def thread_safe(&block)
-      @access_lock.thread_safe(&block)
-    end
   end
 end

data/smart_container.gemspec

@@ -3,7 +3,7 @@
 require_relative 'lib/smart_core/container/version'
 
 Gem::Specification.new do |spec|
-  spec.required_ruby_version = Gem::Requirement.new('>= 2.4.10')
+  spec.required_ruby_version = Gem::Requirement.new('>= 2.5')
 
   spec.name    = 'smart_container'
   spec.version = SmartCore::Container::VERSION
@@ -27,12 +27,13 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ['lib']
 
-  spec.add_dependency 'smart_engine', '~> 0.5'
+  spec.add_dependency 'smart_engine', '~> 0.17'
 
-  spec.add_development_dependency 'bundler',          '~> 2.1'
+  spec.add_development_dependency 'solargraph',       '~> 0.47'
+  spec.add_development_dependency 'bundler',          '~> 2.3'
   spec.add_development_dependency 'rake',             '~> 13.0'
-  spec.add_development_dependency 'rspec',            '~> 3.9'
-  spec.add_development_dependency 'armitage-rubocop', '~> 0.81'
-  spec.add_development_dependency 'simplecov',        '~> 0.18'
-  spec.add_development_dependency 'pry',              '~> 0.13'
+  spec.add_development_dependency 'rspec',            '~> 3.11'
+  spec.add_development_dependency 'armitage-rubocop', '~> 1.36'
+  spec.add_development_dependency 'simplecov',        '~> 0.21'
+  spec.add_development_dependency 'pry',              '~> 0.14'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: smart_container
 version: !ruby/object:Gem::Version
-  version: 0.8.1
+  version: 0.10.0
 platform: ruby
 authors:
 - Rustam Ibragimov
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2020-07-09 00:00:00.000000000 Z
+date: 2022-10-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: smart_engine
@@ -16,28 +16,42 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.5'
+        version: '0.17'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.5'
+        version: '0.17'
+- !ruby/object:Gem::Dependency
+  name: solargraph
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.47'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.47'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.1'
+        version: '2.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.1'
+        version: '2.3'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -58,56 +72,56 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.9'
+        version: '3.11'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.9'
+        version: '3.11'
 - !ruby/object:Gem::Dependency
   name: armitage-rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.81'
+        version: '1.36'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.81'
+        version: '1.36'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.18'
+        version: '0.21'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.18'
+        version: '0.21'
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.13'
+        version: '0.14'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.13'
+        version: '0.14'
 description: Thread-safe semanticaly-defined IoC/DI Container
 email:
 - iamdaiver@gmail.com
@@ -118,7 +132,6 @@ files:
 - ".gitignore"
 - ".rspec"
 - ".rubocop.yml"
-- ".travis.yml"
 - CHANGELOG.md
 - CODE_OF_CONDUCT.md
 - Gemfile
@@ -129,7 +142,6 @@ files:
 - bin/console
 - bin/setup
 - lib/smart_core/container.rb
-- lib/smart_core/container/arbitrary_lock.rb
 - lib/smart_core/container/definition_dsl.rb
 - lib/smart_core/container/definition_dsl/command_set.rb
 - lib/smart_core/container/definition_dsl/commands.rb
@@ -173,7 +185,7 @@ metadata:
   homepage_uri: https://github.com/smart-rb/smart-container
   source_code_uri: https://github.com/smart-rb/smart-container
   changelog_uri: https://github.com/smart-rb/smart-container/CHANGELOG.md
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -181,15 +193,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.4.10
+      version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
-signing_key: 
+rubygems_version: 3.3.11
+signing_key:
 specification_version: 4
 summary: IoC/DI Container
 test_files: []

data/.travis.yml

@@ -1,22 +0,0 @@
----
-language: ruby
-cache: bundler
-os: linux
-dist: xenial
-before_install:
-- gem install bundler
-script:
-- bundle exec rake rubocop
-- bundle exec rake rspec
-jobs:
-  fast_finish: true
-  include:
-  - rvm: 2.4.10
-  - rvm: 2.5.8
-  - rvm: 2.6.6
-  - rvm: 2.7.1
-  - rvm: ruby-head
-  - rvm: jruby-head
-  allow_failures:
-  - rvm: ruby-head
-  - rvm: jruby-head

data/lib/smart_core/container/arbitrary_lock.rb

@@ -1,22 +0,0 @@
-# frozen_string_literal: true
-
-# @api private
-# @since 0.1.0
-class SmartCore::Container::ArbitraryLock
-  # @return [void]
-  #
-  # @api private
-  # @since 0.1.0
-  def initialize
-    @lock = Mutex.new
-  end
-
-  # @param block [Proc]
-  # @return [Any]
-  #
-  # @api private
-  # @since 0.1.0
-  def thread_safe(&block)
-    @lock.owned? ? yield : @lock.synchronize(&block)
-  end
-end