optify-config 1.6.0 → 1.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 36866b3812d07006be63012e27381c3cae692e3a4a4adc6c1c9d54dbfda32c26
-  data.tar.gz: df8fa299cbd2bc4891011e60a847c1819ab83de05b668924576515ffab648773
+  metadata.gz: c39f6b23931576d844b9c6c4991a373f96198f009e4f4a78227adcb373162f2c
+  data.tar.gz: 98cd040744bdfeab713795c276b58b472b5e9fff7d4c7bf164f5196aab94e812
 SHA512:
-  metadata.gz: 57eec9c48450af7a1b0541d2e241215f4b5b7c374d0b8e5a39dcfee1608cab60f29f02483c9b7a8a4422bb70cbde863c659c5e7f01821e21f06bcb1cb7005cb3
-  data.tar.gz: c8bec708ad806a205a3fe0d9dee33237e1c29b38c4c42ad8df21daf1d7c9910322e7b99abddeba512a3175b8b368b42db1e564be148e2731f078aec6e17e42fb
+  metadata.gz: b3c2ae32795dbca9e1a9a48eebbec2b6a854e44071554693dbad9c226ec94624cab294b6ed8fa790a0bbd7bbfec5a59bc42cb4d61d475299fde1c41ff0476738
+  data.tar.gz: 50bcc0e02ced878b270fd3d98a4f7ba46c66eb67f0b1913429e78ef4362dc30ec9045dc22c00c006424849cfe590b77c8219bac5dd8a6e7a05402f26a966b1c1

data/ext/optify_ruby/src/lib.rs

@@ -2,7 +2,6 @@ use magnus::{function, method, prelude::*, wrap, Object, Ruby};
 use optify::builder::OptionsProviderBuilder;
 use optify::builder::OptionsRegistryBuilder;
 use optify::builder::OptionsWatcherBuilder;
-use optify::convert_to_str_slice;
 use optify::provider::GetOptionsPreferences;
 use optify::provider::OptionsProvider;
 use optify::provider::OptionsRegistry;
@@ -70,11 +69,10 @@ impl WrappedOptionsProvider {
         preferences: &MutGetOptionsPreferences,
     ) -> Result<String, magnus::Error> {
         let preferences = convert_preferences(preferences);
-        let features = convert_to_str_slice!(feature_names);
         match rb_self
             .0
             .borrow()
-            .get_all_options(&features, None, Some(&preferences))
+            .get_all_options(&feature_names, None, Some(&preferences))
         {
             Ok(options) => Ok(options.to_string()),
             Err(e) => Err(magnus::Error::new(ruby.exception_runtime_error(), e)),
@@ -98,11 +96,10 @@ impl WrappedOptionsProvider {
         rb_self: &Self,
         feature_names: Vec<String>,
     ) -> Result<Vec<String>, magnus::Error> {
-        let features = convert_to_str_slice!(feature_names);
         rb_self
             .0
             .borrow()
-            .get_canonical_feature_names(&features)
+            .get_canonical_feature_names(&feature_names)
             .map_err(|e| magnus::Error::new(ruby.exception_arg_error(), e))
     }
 
@@ -129,11 +126,10 @@ impl WrappedOptionsProvider {
         key: String,
         feature_names: Vec<String>,
     ) -> Result<String, magnus::Error> {
-        let features = convert_to_str_slice!(feature_names);
         match rb_self
             .0
             .borrow()
-            .get_options_with_preferences(&key, &features, None, None)
+            .get_options_with_preferences(&key, &feature_names, None, None)
         {
             Ok(options) => Ok(options.to_string()),
             Err(e) => Err(magnus::Error::new(ruby.exception_runtime_error(), e)),
@@ -148,10 +144,9 @@ impl WrappedOptionsProvider {
         preferences: &MutGetOptionsPreferences,
     ) -> Result<String, magnus::Error> {
         let preferences = convert_preferences(preferences);
-        let features = convert_to_str_slice!(feature_names);
         match rb_self.0.borrow().get_options_with_preferences(
             &key,
-            &features,
+            &feature_names,
             None,
             Some(&preferences),
         ) {
@@ -216,11 +211,10 @@ impl WrappedOptionsWatcher {
         preferences: &MutGetOptionsPreferences,
     ) -> Result<String, magnus::Error> {
         let preferences = convert_preferences(preferences);
-        let features = convert_to_str_slice!(feature_names);
         match rb_self
             .0
             .borrow()
-            .get_all_options(&features, None, Some(&preferences))
+            .get_all_options(&feature_names, None, Some(&preferences))
         {
             Ok(options) => Ok(options.to_string()),
             Err(e) => Err(magnus::Error::new(ruby.exception_runtime_error(), e)),
@@ -244,11 +238,10 @@ impl WrappedOptionsWatcher {
         rb_self: &Self,
         feature_names: Vec<String>,
     ) -> Result<Vec<String>, magnus::Error> {
-        let features = convert_to_str_slice!(feature_names);
         rb_self
             .0
             .borrow()
-            .get_canonical_feature_names(&features)
+            .get_canonical_feature_names(&feature_names)
             .map_err(|e| magnus::Error::new(ruby.exception_arg_error(), e))
     }
 
@@ -273,11 +266,10 @@ impl WrappedOptionsWatcher {
         key: String,
         feature_names: Vec<String>,
     ) -> Result<String, magnus::Error> {
-        let features = convert_to_str_slice!(feature_names);
         match rb_self
             .0
             .borrow()
-            .get_options_with_preferences(&key, &features, None, None)
+            .get_options_with_preferences(&key, &feature_names, None, None)
         {
             Ok(options) => Ok(options.to_string()),
             Err(e) => Err(magnus::Error::new(ruby.exception_runtime_error(), e)),
@@ -292,10 +284,9 @@ impl WrappedOptionsWatcher {
         preferences: &MutGetOptionsPreferences,
     ) -> Result<String, magnus::Error> {
         let preferences = convert_preferences(preferences);
-        let features = convert_to_str_slice!(feature_names);
         match rb_self.0.borrow().get_options_with_preferences(
             &key,
-            &features,
+            &feature_names,
             None,
             Some(&preferences),
         ) {

data/lib/optify_ruby/base_config.rb

@@ -8,8 +8,7 @@ module Optify
   # A base class for classes from configuration files.
   # Classes that derive from this can easily be used with `Optify::OptionsProvider.get_options`
   # because they will have an implementation of `from_hash` that works recursively.
-  # This class is a work in progress with minimal error handling
-  # and doesn't handle certain cases such as nilable types yet.
+  # This class is a work in progress with minimal error handling.
   # It may be moved to another gem in the future.
   class BaseConfig
     extend T::Sig
@@ -26,7 +25,10 @@ module Optify
       instance = new
 
       hash.each do |key, value|
-        sig_return_type = T::Utils.signature_for_method(instance_method(key)).return_type
+        sig = T::Utils.signature_for_method(instance_method(key))
+        raise "A Sorbet signature is required for `#{name}.#{key}`." if sig.nil?
+
+        sig_return_type = sig.return_type
         value = _convert_value(value, sig_return_type)
         instance.instance_variable_set("@#{key}", value)
       end
@@ -104,8 +106,47 @@ module Optify
       return true if other.equal?(self)
       return false unless other.is_a?(self.class)
 
-      instance_variables.all? do |var|
-        instance_variable_get(var) == other.instance_variable_get(var)
+      instance_variables.all? do |name|
+        instance_variable_get(name) == other.instance_variable_get(name)
+      end
+    end
+
+    # Convert this object to a Hash recursively.
+    # This is mostly the reverse operation of `from_hash`,
+    # as keys will be symbols
+    # and `from_hash` will convert strings to symbols if that's how the attribute is declared.
+    # @return [Hash] The hash representation of this object.
+    #: () -> Hash[Symbol, untyped]
+    def to_h
+      result = Hash.new(instance_variables.size)
+
+      instance_variables.each do |var_name|
+        # 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)
+      end
+
+      result
+    end
+
+    private
+
+    #: (untyped value) -> untyped
+    def _convert_value_to_hash(value)
+      case value
+      when Array
+        value.map { |v| _convert_value_to_hash(v) }
+      when Hash
+        value.transform_values { |v| _convert_value_to_hash(v) }
+      when nil
+        nil
+      else
+        if value.respond_to?(:to_h)
+          value.to_h
+        else
+          value
+        end
       end
     end
   end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: optify-config
 version: !ruby/object:Gem::Version
-  version: 1.6.0
+  version: 1.7.0
 platform: ruby
 authors:
 - Justin D. Harris
 bindir: exe
 cert_chain: []
-date: 2025-06-11 00:00:00.000000000 Z
+date: 2025-06-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rb_sys
@@ -29,14 +29,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.12083
+        version: 0.5.12167
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.12083
+        version: 0.5.12167
 - !ruby/object:Gem::Dependency
   name: rake-compiler
   requirement: !ruby/object:Gem::Requirement
@@ -57,42 +57,42 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.9.3
+        version: 4.0.0.dev.4
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.9.3
+        version: 4.0.0.dev.4
 - !ruby/object:Gem::Dependency
   name: sorbet
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.12083
+        version: 0.5.12167
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.12083
+        version: 0.5.12167
 - !ruby/object:Gem::Dependency
   name: tapioca
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.16.11
+        version: 0.17.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.16.11
+        version: 0.17.2
 - !ruby/object:Gem::Dependency
   name: test-unit
   requirement: !ruby/object:Gem::Requirement