magick-feature-flags 0.9.4 → 0.9.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 47d460bf4a24d016000bfaffcaaf5f57a0af0b9152e48d4af3678f0775788039
-  data.tar.gz: c9fe1129eff787c85eff1ce7305926dde59139a0011a50f7fce4d069fe89d360
+  metadata.gz: c89e62bee50614e51b3e844968d557e90217cd48e77bb68ec50c47d48881389b
+  data.tar.gz: d39574add8463bb4a927ea94db047d3418317d966b77f400892f67ed6bf4a31c
 SHA512:
-  metadata.gz: ba685c76fff52bade17967a0986241d7a4f09b9313ef5e607c3df88f4b2f44be9c02f3ddb852879abf4edf099c631cb4de0187adb4446921b4f178008b819826
-  data.tar.gz: 7ad42b6a66bf885737b2719840e7e999175143e632bbdcfffd085126b5d85a76534c32dd2743599969a92459a0eebc3ef66818db20c58f172c849b153140eb79
+  metadata.gz: 133bbce3e15f14c39181eaf2078e0bec80fcc2aa295d85397bcb78f95400637e046dd9749f5c8ed65291f2c2e8b562ac8ced6b3df39dd3d625536a58851cefbb
+  data.tar.gz: 3a8578be1de567da37d0393e07302934176100bed9d9239bad978c757ed65a6f58770cc3a9b6b6fb5d73fd2393007053bc43c32b466b9bd167e97d5e775c6be0

data/lib/magick/adapters/registry.rb

@@ -86,6 +86,12 @@ module Magick
         (memory_features + redis_features).uniq
       end
 
+      # Explicitly trigger cache invalidation for a feature
+      # This is useful for targeting updates that need immediate cache invalidation
+      def invalidate_cache(feature_name)
+        publish_cache_invalidation(feature_name)
+      end
+
       private
 
       attr_reader :memory_adapter, :redis_adapter, :circuit_breaker

data/lib/magick/feature.rb

@@ -74,8 +74,10 @@ module Magick
       return false if status == :inactive
       return false if status == :deprecated && !context[:allow_deprecated]
 
-      # Check feature dependencies
-      return false if !dependencies.empty? && !dependencies.all? { |dep_name| Magick.enabled?(dep_name, context) }
+      # NOTE: We don't check dependencies here because:
+      # - Main features can be enabled independently
+      # - Dependencies are only prevented from being enabled if the main feature is disabled
+      # - The dependency check is handled in the enable() method, not in enabled?()
 
       # Check date/time range targeting
       return false if targeting[:date_range] && !date_range_active?(targeting[:date_range])
@@ -461,6 +463,7 @@ module Magick
     def delete
       adapter_registry.delete(name)
       @stored_value = nil
+      @stored_value_initialized = false # Reset initialization flag so get_value returns default_value
       @targeting = {}
       # Also remove from Magick.features if registered
       Magick.features.delete(name.to_s)
@@ -820,11 +823,20 @@ module Magick
     end
 
     def save_targeting
+      # Save targeting to adapter (this triggers cache invalidation via Pub/Sub)
       adapter_registry.set(name, 'targeting', targeting)
+
       # Update the feature in Magick.features if it's registered
       return unless Magick.features.key?(name)
 
       Magick.features[name].instance_variable_set(:@targeting, targeting.dup)
+
+      # Explicitly trigger cache invalidation for targeting updates
+      # Targeting changes affect enabled? checks, so we need immediate cache invalidation
+      # even if async updates are enabled
+      return unless adapter_registry.respond_to?(:invalidate_cache)
+
+      adapter_registry.invalidate_cache(name)
     end
 
     def default_for_type

data/lib/magick/version.rb

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

metadata

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