optify-from_hash 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ab3e37acc0afa16b17fbfb4bcbbf95e31b38d75c321d3d3d3a9a9f8a9fb7afb5
-  data.tar.gz: b087d7433c8329a6e62262d129f80cd538dae2459337e619ad2ab760df21bbc5
+  metadata.gz: 0e24ce9b23b87819fc6688195ca5a90cb43dfbe23b25264e269cf37daa97f531
+  data.tar.gz: 18e142eb6c75bf80cfab5fb8d64079ec1f7fd2d212e97043def5555d595fcdd0
 SHA512:
-  metadata.gz: 968c5aa5b223c043a8558f5e2ac2b4cf8efb7925ed408bf6dffc458734a51a6da0d4b85e781688354b8432c0b6307067c4102521f7792d801859a1a80bdf391c
-  data.tar.gz: 57b11b0707b45887e544ad8d32286e0780a37d89f1fc87ea293139ed13a26140e2cde81cf1afe534827e4c9be69e6a3c5812b236b2ed5fd6438e73e1985ed206
+  metadata.gz: ecfa102d64db21405131a02545bfa524eebde5150c9030555faa89e8e802ec6bdf8006fbd90d30a713450a1beb28167302996bbee2507c01f6d67cea3196ad56
+  data.tar.gz: 1943ab3ab4ec77e5441de4006b1699683a5bf0d9e8151d3b51ef4fcedd00e6eedf5b8baa5e9d42e7edff367f22d29109495e63ee2e6cff197facd2f5edd8ed79

data/lib/optify_from_hash/from_hashable.rb

@@ -14,7 +14,6 @@ module Optify
 
     # Create a new immutable instance of the class from a hash.
     #
-    # This is a class method so that it can set members with private setters.
     # @param hash The hash to create the instance from.
     # @return The new instance.
     #: (Hash[untyped, untyped]) -> instance
@@ -40,7 +39,7 @@ module Optify
       instance.freeze
     end
 
-    #: (untyped, untyped) -> untyped
+    #: (untyped, T::Types::Base) -> untyped
     def self._convert_value(value, type)
       if type.is_a?(T::Types::Untyped)
         # No preferred type is given, so return the value as is.
@@ -52,7 +51,10 @@ module Optify
       case value
       when Array
         # Handle `T.nilable(T::Array[...])`
-        type = type.unwrap_nilable if type.respond_to?(:unwrap_nilable)
+        if type.respond_to?(:unwrap_nilable)
+          type = type #: as untyped
+                 .unwrap_nilable
+        end
         inner_type = type.type
         return value.map { |v| _convert_value(v, inner_type) }.freeze
       when Hash
@@ -61,7 +63,8 @@ module Optify
         # `T.any(...)` because using `.types` works for both cases.
         if type.respond_to?(:types)
           # Find a type that works for the hash.
-          type.types.each do |t|
+          type #: as untyped
+            .types.each do |t|
             return _convert_hash(value, t).freeze
           rescue StandardError
             # Ignore and try the next type.
@@ -76,12 +79,13 @@ module Optify
       value
     end
 
-    #: (Hash[untyped, untyped], untyped) -> untyped
+    #: (Hash[untyped, untyped], T::Types::Base) -> untyped
     def self._convert_hash(hash, type)
       if type.respond_to?(:raw_type)
         # There is an object for the hash.
         # It could be a custom class, a String, or maybe something else.
-        type_for_hash = type.raw_type
+        type_for_hash = type #: as untyped
+                        .raw_type
         return type_for_hash.from_hash(hash) if type_for_hash.respond_to?(:from_hash)
       elsif type.is_a?(T::Types::TypedHash)
         # The hash should be a hash, but the values might be objects to convert.
@@ -116,9 +120,9 @@ module Optify
     end
 
     # Convert this object to a JSON string.
-    #: (*untyped) -> String
-    def to_json(*args)
-      to_h.to_json(args)
+    #: (?JSON::State?) -> String
+    def to_json(state = nil)
+      to_h.to_json(state)
     end
 
     # Convert this object to a Hash recursively.
@@ -134,21 +138,19 @@ module Optify
         # Remove the @ prefix to get the method name
         method_name = var_name.to_s[1..] #: as !nil
         value = instance_variable_get(var_name)
-        result[method_name.to_sym] = _convert_value_to_hash(value)
+        result[method_name.to_sym] = self.class.send(:_convert_value_for_to_h, value)
       end
 
       result
     end
 
-    private
-
     #: (untyped) -> untyped
-    def _convert_value_to_hash(value)
+    def self._convert_value_for_to_h(value)
       case value
       when Array
-        value.map { |v| _convert_value_to_hash(v) }
+        value.map { |v| _convert_value_for_to_h(v) }
       when Hash
-        value.transform_values { |v| _convert_value_to_hash(v) }
+        value.transform_values { |v| _convert_value_for_to_h(v) }
       when nil
         nil
       else
@@ -159,5 +161,7 @@ module Optify
         end
       end
     end
+
+    private_class_method :_convert_value_for_to_h
   end
 end

data/rbi/optify_from_hash.rbi

@@ -9,15 +9,14 @@ module Optify
 
     # Create a new instance of the class from a hash.
     #
-    # This is a class method that so that it can set members with private setters.
     # @param hash The hash to create the instance from.
     # @return The new instance.
     sig { params(hash: T::Hash[T.untyped, T.untyped]).returns(T.attached_class) }
     def self.from_hash(hash); end
 
     # Convert this object to a JSON string.
-    sig { params(args: T.untyped).returns(String) }
-    def to_json(*args); end
+    sig { params(state: T.nilable(JSON::State)).returns(String) }
+    def to_json(state = nil); end
 
     # Convert this object to a Hash recursively.
     # This is mostly the reverse operation of `from_hash`,

data/sig/optify_from_hash.rbs

@@ -6,13 +6,12 @@ end
 class Optify::FromHashable
   # Create a new instance of the class from a hash.
   #
-  # This is a class method that so that it can set members with private setters.
   # @param hash The hash to create the instance from.
   # @return The new instance.
   def self.from_hash: (::Hash[untyped, untyped] hash) -> instance
 
   # Convert this object to a JSON string.
-  def to_json: (*untyped args) -> String
+  def to_json: (?JSON::State? state) -> String
 
   # Convert this object to a Hash recursively.
   # This is mostly the reverse operation of `from_hash`,

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: optify-from_hash
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Justin D. Harris