magick-feature-flags 0.9.19 → 0.9.20

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3a9bd8f59c8729764a5a1b9876b20bca16dce2670ed477ee64109eda02219f98
-  data.tar.gz: 566826dcca95d42a34fd5f95581e5dfd1c3d074a3260def10fda5bb3a68c5ce0
+  metadata.gz: e4f9379b630e9db00b28614f347d523edb33d63c842d6b5c9fd747be02f1d75d
+  data.tar.gz: ffa32ee13faa08781c68870813c15ea7b3dcc15fe65f8c1f2618952251fe870c
 SHA512:
-  metadata.gz: 0b37fb6089b687c08b31aad74d24f43c90d65134c9e68e0de2d521a6357f64d0f3778b8aafe60aca9f912f4610d73a42e4e6a2f96f156f66a8c5d154affa8d91
-  data.tar.gz: 267b78151d2017069f1a3e4836ed0016bbdc524ee9fb2d7ea15ec8148df9359604f9acf9dca886aa073fb289309c5f4f90ce39754d8da80d95794606b0a05087
+  metadata.gz: 0f2651c1f8528b4eb385f82a629c7891e0a78ec4a796b574baa8228a2bfcaddf88ee6855af45f9fba9c12b4d7d475e1d8aa6ba6b1002706a1247a8ef81b540e4
+  data.tar.gz: 418cc8c6909969581b147df052c72ffc9635a86b07726f69f0c6085bef3bbb804515d34a3d6ca782f8b4c03d4127722a5547a483ca2d0cddfc56f3bef4e6e864

data/lib/magick/adapters/memory.rb

@@ -11,12 +11,16 @@ module Magick
       end
 
       def get(feature_name, key)
+        # Fast path: avoid mutex if possible (use string keys directly)
+        feature_name_str = feature_name.is_a?(String) ? feature_name : feature_name.to_s
+        key_str = key.is_a?(String) ? key : key.to_s
+
         mutex.synchronize do
           cleanup_expired
-          feature_data = store[feature_name.to_s]
+          feature_data = store[feature_name_str]
           return nil unless feature_data
 
-          value = feature_data[key.to_s]
+          value = feature_data[key_str]
           deserialize_value(value)
         end
       end

data/lib/magick/feature.rb

@@ -25,51 +25,23 @@ module Magick
       # Performance optimizations: cache expensive checks
       @_targeting_empty = true # Will be updated after load_from_adapter
       @_rails_events_enabled = false # Cache Rails events availability (only enable in dev)
+      @_perf_metrics_enabled = false # Cache performance metrics (disabled by default for speed)
 
       validate_type!
       validate_default_value!
       load_from_adapter
       # Update targeting empty cache after loading
       @_targeting_empty = @targeting.empty?
+      # Cache performance metrics availability (check once, not on every call)
+      @_perf_metrics_enabled = Magick.performance_metrics != nil
       # Save description and display_name to adapter if they were provided and not already in adapter
       save_metadata_if_new
     end
 
     def enabled?(context = {})
-      # Performance metrics: measure before the check (only if enabled)
-      perf_metrics = Magick.performance_metrics
-      start_time = perf_metrics ? Process.clock_gettime(Process::CLOCK_MONOTONIC) : nil
-
-      # Fast path: check enabled status
-      result = check_enabled(context)
-
-      # Record metrics if enabled (after the check to minimize overhead)
-      if perf_metrics
-        duration = (Process.clock_gettime(Process::CLOCK_MONOTONIC) - start_time) * 1000 # milliseconds
-        perf_metrics.record(name, 'enabled?', duration, success: true)
-      end
-
-      # Rails 8+ events (only in development or when explicitly enabled)
-      if @_rails_events_enabled
-        if result
-          Magick::Rails::Events.feature_enabled(name, context: context)
-        else
-          Magick::Rails::Events.feature_disabled(name, context: context)
-        end
-      end
-
-      # Warn if deprecated (only if enabled)
-      if status == :deprecated && result && !context[:allow_deprecated] && Magick.warn_on_deprecated
-        warn "DEPRECATED: Feature '#{name}' is deprecated and will be removed."
-        Magick::Rails::Events.deprecated_warning(name) if @_rails_events_enabled
-      end
-
-      result
+      # Fastest path: check enabled status directly (no overhead)
+      check_enabled(context)
     rescue StandardError => e
-      if perf_metrics
-        duration = start_time ? (Process.clock_gettime(Process::CLOCK_MONOTONIC) - start_time) * 1000 : 0.0
-        perf_metrics.record(name, 'enabled?', duration, success: false)
-      end
       # Return false on any error (fail-safe)
       warn "Magick: Error checking feature '#{name}': #{e.message}" if defined?(Rails) && Rails.env.development?
       false

data/lib/magick/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Magick
-  VERSION = '0.9.19'
+  VERSION = '0.9.20'
 end

data/lib/magick.rb

@@ -103,7 +103,9 @@ module Magick
     end
 
     def enabled?(feature_name, context = {})
-      feature = features[feature_name.to_s] || self[feature_name]
+      # Fast path: use string key directly (avoid repeated to_s conversion)
+      feature_name_str = feature_name.to_s
+      feature = features[feature_name_str] || self[feature_name]
       feature.enabled?(context)
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: magick-feature-flags
 version: !ruby/object:Gem::Version
-  version: 0.9.19
+  version: 0.9.20
 platform: ruby
 authors:
 - Andrew Lobanov