magick-feature-flags 0.8.2 → 0.8.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a53f3c93e974822f126ed2148d9fb61adf7de3a374ba26acbd834a2a916545a0
-  data.tar.gz: 31dbbacc15565a496e65113364d3eaf5a6a5385251f3654a28ea1c5eb9e9b014
+  metadata.gz: a62d079b6f68d6c2b207fe51bf1698fd7b79eca4e9eecd509b614f4a33ee78f2
+  data.tar.gz: 7b01eafa0b42fa5555cfa7356bc299a3bb5170a8065714707e68c0d13a8e16a6
 SHA512:
-  metadata.gz: 9b39439b771eb158d39496758362675db1134ea160311b8af9d51b59e1b40c7440ee425ee7d8f6e28bd02dc3da6498ec24d79744cd5cb7b63f54e13e86fa8cf4
-  data.tar.gz: 0bc2519decc3164d6b3dddd6080bf79c00b9a1ee2473d95b25550edbea8ca8c433912b8655d72c86b993762526781377df27219f1957585990b59942e3597bb8
+  metadata.gz: e93c25f079be5541256bd3c89389a8370edb0c8cd2789e85c2010d9e6f6e8e20b9bb0ca74e87a7c22c4bfb471a53d88cde64ea1570cc06f2439746ee07a0e621
+  data.tar.gz: 9297dcd89a4bcfb487f1fe550960f1d1984a7a5ac631c80cb0f417a9792dbeffa7b1832b2d02fd0f07408ff9ba8e539f0c1127a6677415f9e81c62957ffd34cd

data/lib/magick/feature.rb

@@ -24,6 +24,8 @@ module Magick
       validate_type!
       validate_default_value!
       load_from_adapter
+      # Save description and display_name to adapter if they were provided and not already in adapter
+      save_metadata_if_new
     end
 
     def enabled?(context = {})
@@ -412,6 +414,21 @@ module Magick
       true
     end
 
+    # Reload feature state from adapter (useful when feature is changed externally)
+    def reload
+      load_from_adapter
+      # Update registered feature instance if it exists
+      if Magick.features.key?(name)
+        registered = Magick.features[name]
+        registered.instance_variable_set(:@stored_value, @stored_value)
+        registered.instance_variable_set(:@status, @status)
+        registered.instance_variable_set(:@description, @description)
+        registered.instance_variable_set(:@display_name, @display_name)
+        registered.instance_variable_set(:@targeting, @targeting.dup)
+      end
+      true
+    end
+
     def to_h
       {
         name: name,
@@ -435,28 +452,21 @@ module Magick
       load_value_from_adapter
     end
 
-    # Reload feature state from adapter (useful when feature is changed externally)
-    def reload
-      load_from_adapter
-      # Update registered feature instance if it exists
-      if Magick.features.key?(name)
-        registered = Magick.features[name]
-        registered.instance_variable_set(:@stored_value, @stored_value)
-        registered.instance_variable_set(:@status, @status)
-        registered.instance_variable_set(:@display_name, @display_name)
-        registered.instance_variable_set(:@targeting, @targeting.dup)
-      end
-      self
-    end
-
     def load_from_adapter
       # Clear cached value to force reload from adapter (which checks version)
       @stored_value = nil
       @stored_value = load_value_from_adapter
       status_value = adapter_registry.get(name, 'status')
       @status = status_value ? status_value.to_sym : status
+
+      # Load description from adapter (override initial value if present in adapter)
+      description_value = adapter_registry.get(name, 'description')
+      @description = description_value if description_value
+
+      # Load display_name from adapter (override initial value if present in adapter)
       display_name_value = adapter_registry.get(name, 'display_name')
       @display_name = display_name_value if display_name_value
+
       targeting_value = adapter_registry.get(name, 'targeting')
       if targeting_value.is_a?(Hash)
         # Normalize keys to symbols and handle nested structures
@@ -469,6 +479,17 @@ module Magick
       end
     end
 
+    def save_metadata_if_new
+      # Save description and display_name to adapter if they were provided in options
+      # but don't exist in adapter yet (to avoid overwriting existing values)
+      if @description && !adapter_registry.get(name, 'description')
+        adapter_registry.set(name, 'description', @description)
+      end
+      if @display_name && !adapter_registry.get(name, 'display_name')
+        adapter_registry.set(name, 'display_name', @display_name)
+      end
+    end
+
     def load_value_from_adapter
       value = adapter_registry.get(name, 'value')
       return nil if value.nil?

data/lib/magick/version.rb

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

metadata

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