optify-config 1.15.1-arm64-darwin → 1.16.0-arm64-darwin

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d6adfc1ceddfd67792e9d0054d9cf76aa78440eb0aa5c61196d242e7cb0aa2ec
-  data.tar.gz: 4c3b240675bf8cf90feab309d9bac1f0335b598bb517886495eb2f74db30a3f1
+  metadata.gz: 9383437ad25eff80824038a4cf9e484577eec5702891243d18e10b3d61dff186
+  data.tar.gz: 3c0f3472609f462f68cf93f940353459a0dea8c99aead366533a2dbe6f38be7a
 SHA512:
-  metadata.gz: da721944061f4013ff7a7b457e84fc3b122d7fb9b3850cd2e2d5f85fc78770a26b1b2c36ac06a2718cedde71139e8ad5ff373939f35603a6488b3ad3b122c9d1
-  data.tar.gz: b70dcd20fc0a361363ab909830bf68701c660264daadac48b64512137422f5948f1e5911bf038fa312bfc785eec5a52b601fe881e9ced1be1cc8f001ecbd4a8a
+  metadata.gz: 188d638d949e9989c8deba5da789465929ffb630dbc09c3216cef0bf023a87c28759b50900e545018f4d9f773c5a8382b05e0854a6f7ab3378c5454e68104768
+  data.tar.gz: '0962d70021c874c52c020b8dfc04941d947ca062353f0351934d9bb37ee795c85f975177ad851b06289f34c43ec3a2855a597baaadf4fca4b4e655cf238599f0'

data/lib/optify.rb

@@ -15,7 +15,7 @@ require_relative 'optify_ruby/watcher_implementation'
 # but that doesn't work when building for multiple versions of Ruby.
 # So we have to do this which is similar to something from 'https://github.com/matsadler/halton-rb/blob/main/lib/halton.rb'.
 begin
-  ruby_version = T.must(RUBY_VERSION.match(/\d+\.\d+/))[0]
+  ruby_version = RUBY_VERSION.match(/\d+\.\d+/)&.[](0)
   require_relative "optify_ruby/#{ruby_version}/optify_ruby"
 rescue LoadError
   begin

data/lib/optify_ruby/options_metadata.rb

@@ -11,7 +11,7 @@ module Optify
     sig { returns(T.nilable(T::Array[String])) }
     attr_reader :aliases
 
-    # The canonical names of features that depend on this one.
+    # The canonical names of features that import this one.
     sig { returns(T.nilable(T::Array[String])) }
     attr_reader :dependents
 

data/lib/optify_ruby/provider_module.rb

@@ -6,8 +6,6 @@ require 'sorbet-runtime'
 module Optify
   # @!visibility private
   module ProviderModule
-    extend T::Sig
-
     #: (Array[String] feature_names) -> Array[String]
     def get_canonical_feature_names(feature_names)
       # Try to optimize a typical case where there are just a few features.
@@ -40,7 +38,7 @@ module Optify
       end
       result.freeze
 
-      @features_with_metadata = T.let(result, T.nilable(T::Hash[String, OptionsMetadata]))
+      @features_with_metadata = result
       result
     end
 
@@ -71,13 +69,14 @@ module Optify
                        get_options_json(key, feature_names)
                      end
       hash = JSON.parse(options_json)
-      T.unsafe(config_class).from_hash(hash)
+      config_class #: as untyped
+        .from_hash(hash)
     end
 
     #: -> void
     def _init
-      @cache = T.let({}, T.nilable(T::Hash[T.untyped, T.untyped]))
-      @features_with_metadata = T.let(nil, T.nilable(T::Hash[String, OptionsMetadata]))
+      @cache = {} #: Hash[untyped, untyped]?
+      @features_with_metadata = nil #: Hash[String, OptionsMetadata]?
     end
 
     NOT_FOUND_IN_CACHE_SENTINEL = Object.new
@@ -93,28 +92,31 @@ module Optify
       end
 
       init unless @cache
-      feature_names = get_canonical_feature_names(feature_names) unless preferences&.skip_feature_name_conversion
 
-      cache_key = [key, feature_names, preferences&.constraints_json, config_class]
-      result = @cache&.fetch(cache_key, NOT_FOUND_IN_CACHE_SENTINEL)
+      if preferences.nil?
+        feature_names = get_filtered_features(feature_names, GetOptionsPreferences.new)
+      elsif !preferences.skip_feature_name_conversion || preferences.constraints_json
+        feature_names = get_filtered_features(feature_names, preferences)
+      end
+
+      # Features are filtered, so we don't need the constraints in the cache key.
+      cache_key = [key, feature_names, config_class]
+      result = @cache #: as !nil
+               .fetch(cache_key, NOT_FOUND_IN_CACHE_SENTINEL)
       return result unless result.equal?(NOT_FOUND_IN_CACHE_SENTINEL)
 
       # Handle a cache miss.
 
-      # We can avoid converting the features names because they're already converted, if that was desired.
-      if preferences.nil?
-        preferences = GetOptionsPreferences.new
-        preferences.skip_feature_name_conversion = true
-      else
-        # Indeed the copying of preferences could be wasteful, but this only happens on a cache miss
-        # and when no custom preferences are provided.
-        preferences = preferences.dup
-        preferences.skip_feature_name_conversion = true
-      end
+      # We can avoid converting the features names because they're already converted from filtering above, if that was desired.
+      # We don't need the constraints because we filtered the features above.
+      # We already know there are no overrides because we checked above.
+      preferences = GetOptionsPreferences.new
+      preferences.skip_feature_name_conversion = true
 
       result = get_options(key, feature_names, config_class, nil, preferences)
 
-      T.must(@cache)[cache_key] = result
+      @cache #: as !nil
+        .[]= cache_key, result
     end
   end
 end

data/lib/optify_ruby/watcher_implementation.rb

@@ -33,7 +33,7 @@ module Optify
     #: -> OptionsWatcher
     def init
       _init
-      @cache_creation_time = T.let(Time.now, T.nilable(Time))
+      @cache_creation_time = Time.now #: Time?
       self
     end
 

data/rbi/optify.rbi

@@ -45,7 +45,7 @@ module Optify
     sig { returns(T.nilable(T::Array[String])) }
     def aliases; end
 
-    # The canonical names of features that depend on this one.
+    # The canonical names of features that import this one.
     sig { returns(T.nilable(T::Array[String])) }
     def dependents; end
 
@@ -183,6 +183,18 @@ module Optify
     sig { params(canonical_feature_name: String).returns(T.nilable(OptionsMetadata)) }
     def get_feature_metadata(canonical_feature_name); end
 
+    # Filters `feature_names` based on the preferences,
+    # such as the `preferences`'s constraints.
+    # Also converts the feature names to canonical feature names if `preferences.skip_feature_name_conversion` is `false`.
+    sig do
+      params(
+        feature_names: T::Array[String],
+        preferences: GetOptionsPreferences
+      )
+        .returns(T::Array[String])
+    end
+    def get_filtered_features(feature_names, preferences); end
+
     # Fetches options based on the provided key and feature names.
     #
     # @param key The key to fetch options for.

data/sig/optify.rbs

@@ -38,7 +38,7 @@ end
 class Optify::OptionsMetadata < BaseConfig
   def aliases: () -> ::Array[String]?
 
-  # The canonical names of features that depend on this one.
+  # The canonical names of features that import this one.
   def dependents: () -> ::Array[String]?
 
   def details: () -> untyped
@@ -142,6 +142,8 @@ module Optify::ProviderModule
   # @return The metadata for the feature.
   def get_feature_metadata: (String canonical_feature_name) -> OptionsMetadata?
 
+  def get_filtered_features: (::Array[String] feature_names, GetOptionsPreferences preferences) -> ::Array[String]
+
   def get_options: [Config] (String key, ::Array[String] feature_names, T::Class[Config] config_class, ?CacheOptions? cache_options, ?Optify::GetOptionsPreferences? preferences) -> Config
 
   # Fetches options in JSON format based on the provided key and feature names.

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: optify-config
 version: !ruby/object:Gem::Version
-  version: 1.15.1
+  version: 1.16.0
 platform: arm64-darwin
 authors:
 - Justin D. Harris
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2025-08-25 00:00:00.000000000 Z
+date: 2025-09-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sorbet-runtime
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.12167
+        version: 0.6.12477
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.12167
+        version: 0.6.12477
 - !ruby/object:Gem::Dependency
   name: rake-compiler
   requirement: !ruby/object:Gem::Requirement
@@ -58,28 +58,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.12167
+        version: 0.6.12477
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.12167
+        version: 0.6.12477
 - !ruby/object:Gem::Dependency
   name: tapioca
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.17.2
+        version: 0.17.7
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.17.2
+        version: 0.17.7
 - !ruby/object:Gem::Dependency
   name: test-unit
   requirement: !ruby/object:Gem::Requirement