vident 0.4.1 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eb36bbcf0d3a46ec51c1a68e80dbf0ef4b85076412591d30215779d1ff2816e8
-  data.tar.gz: 6c4b6f98a4ac9bcbebaea4003a6532231a2e188aa94ec34889f4c1c624c9ae46
+  metadata.gz: ba2054320ceb13a2646f5f88f4dcef219ad4a5482118f1245472b07d8848f313
+  data.tar.gz: 7e0af03ee04bc52c6b7f35cd0da6490116428581bae587249380df7c71c8d863
 SHA512:
-  metadata.gz: 6ccf024bee94cccffccc5b8cf7999b0b1b2e47c1a4185ab2f5a2f5129b7494d66cdde1d7d8ee8364123386dedd3f1f15ae60c27798db73cea3be4f22f68b24f6
-  data.tar.gz: f09643a2d324a23012b5466b6bd7ad4a205f21238441891557a8d60d5a71860f5a64e866366bdb63744a62f6d7efed53bdfcb17f5711eab54e35675cefd60f88
+  metadata.gz: 8cd4c1933065215f12d8640a4274164ddb869f9b013891f6ef569293b84dbcafea761c8442ebee81c0656c3c2d0e560f7d7ee4a36b5390cba2bc2148d5bc62b0
+  data.tar.gz: bd3549f2eaf0157bea9b8489c2fc9ea83f71d11afa998b6a013ef65ea86ff4f41cbb39caafb94b4ea5049c00b36e3be3a3b67c3cb9ec24020f842cda85476523

data/lib/vident/attributes/typed.rb

@@ -132,22 +132,28 @@ if Gem.loaded_specs.has_key? "dry-struct"
 
           def map_primitive_to_dry_type(signature, options, converter)
             strict = !options[:convert]
-            type, subtype = extract_member_type_and_subclass(signature, options)
-            dry_type = dry_type_from_primary_type(type, strict, converter)
-            if subtype && dry_type.respond_to?(:of)
-              subtype_info = dry_type_from_primary_type(subtype, strict, converter)
-              # Sub types of collections currently can be nil - this should be an option
-              dry_type.of(subtype_info.optional.meta(required: false))
-            else
-              dry_type
+            signatures = extract_member_type_and_subclass(signature, options)
+            types = signatures.map do |type, subtype|
+              dry_type = dry_type_from_primary_type(type, strict, converter)
+              if subtype && dry_type.respond_to?(:of)
+                subtype_info = dry_type_from_primary_type(subtype, strict, converter)
+                # Sub types of collections currently can be nil - this should be an option
+                dry_type.of(subtype_info.optional.meta(required: false))
+              else
+                dry_type
+              end
             end
+            types.reduce(:|)
           end
 
           def extract_member_type_and_subclass(signature, options)
-            if signature.is_a?(Array)
-              [Array, signature.first]
+            case signature
+            when Set
+              signature.flat_map { |s| extract_member_type_and_subclass(s, options) }
+            when Array
+              [[Array, signature.first]]
             else
-              [signature, options[:type] || options[:sub_type]]
+              [[signature, options[:type] || options[:sub_type]]]
             end
           end
 
@@ -185,7 +191,16 @@ if Gem.loaded_specs.has_key? "dry-struct"
               # values using the default constructor, `new`.
               Types.Instance(type)
             else
-              Types.Constructor(type) { |values| converter ? converter.call(values) : type.new(**values) }
+              # dry calls this when initialising the Type. Check if type of input is correct or coerce
+              Types.Constructor(type) do |value|
+                next value if value.is_a?(type)
+
+                if converter
+                  converter.call(value)
+                else
+                  type.new(**value)
+                end
+              end
             end
           end
         end

data/lib/vident/base.rb

@@ -104,7 +104,12 @@ module Vident
         else
           RootComponent::UsingViewComponent
         end
-        klass.new(**stimulus_options_for_component(options))
+        element_attrs = options
+          .except(:id, :element_tag, :html_options, :controller, :controllers, :actions, :targets, :named_classes, :data_maps)
+          .merge(
+            stimulus_options_for_component(options)
+          )
+        klass.new(**element_attrs)
       end
     end
     alias_method :root, :parent_element
@@ -148,12 +153,11 @@ module Vident
     end
     module_function :stimulus_identifier_from_path
 
-    protected
+    private
 
     # Prepare the stimulus attributes for a StimulusComponent
     def stimulus_options_for_component(options)
       {
-        **options.except(:id, :element_tag, :html_options, :controller, :controllers, :actions, :targets, :named_classes, :data_maps),
         id: respond_to?(:id) ? id : (attribute(:id) || options[:id]),
         element_tag: attribute(:element_tag) || options[:element_tag] || :div,
         html_options: prepare_html_options(options[:html_options]),
@@ -167,8 +171,6 @@ module Vident
       }
     end
 
-    private
-
     def prepare_html_options(erb_options)
       # Options should override in this order:
       # - defined on component class methods (lowest priority)

data/lib/vident/caching/cache_key.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "benchmark"
-
 # Rails fragment caching works by either expecting the cached key object to respond to `cache_key` or for that object
 # to be an array or hash. In our case the object maybe an instance of Core::Presenter so here we add a default
 # `cache_key` implementation.
@@ -26,10 +24,6 @@ module Vident
           named_cache_key_includes(name, *attrs)
         end
 
-        def enable_cache_key_benchmarking
-          @enable_cache_key_benchmarking = true
-        end
-
         attr_reader :named_cache_key_attributes
 
         # TypedComponents can be used with fragment caching, but you need to be careful! Read on...
@@ -100,17 +94,7 @@ module Vident
             else
               @cache_key ||= {}
             end
-
-            # TODO: remove the benchmarking code here
-
-            if @enable_cache_key_benchmarking
-              time = ::Benchmark.measure { generate_cache_key(n) }
-              ::Logging::Log.debug "Cache key #{self.class.name}: #{time.real}"
-              ::Thread.current[:total_key_generation_time] ||= 0
-              ::Thread.current[:total_key_generation_time] += time.real
-            else
-              generate_cache_key(n)
-            end
+            generate_cache_key(n)
             @cache_key[n]
           end
         end

data/lib/vident/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Vident
-  VERSION = "0.4.1"
+  VERSION = "0.5.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: vident
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.5.0
 platform: ruby
 authors:
 - Stephen Ierodiaconou
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-01-26 00:00:00.000000000 Z
+date: 2023-02-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport