toggly 1.2.0 → 1.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 810c3cb9858347a1cb79212512e006bf2362ee97cd09c7a528eb334c0bf949cc
-  data.tar.gz: 7f6ef5ce376a28c1101ce65c1d82080c02e6b254487ed0908479c1a622adffeb
+  metadata.gz: a484fa27dd7cc53210f2af73984c93d9109205f797172203aea42c44c4722569
+  data.tar.gz: 79a64ed42c9497e8ce8ca0f59d4dc3c21e52e68ba5e24b3b76680767d2a4787b
 SHA512:
-  metadata.gz: e0c8fbd59c7cfa8baafd73f580da168fe0fe4114e802f0e55bec3c406994e8a4a169b773dbf6e97862f831157d7b011df18f6210cda33740a56a6f515c05fcfc
-  data.tar.gz: 92d791902d6bf591f9731647caa1963b2c589bf12255afe376b1f1fa9591437d9af10388503b54bbbf06e911afb41fb2b557be60c9564dd09eec423037fcbdab
+  metadata.gz: 07f924fb8addc26c7af35f31f0a023680201f097cfa6b26826c333a2dc3a2086c07ae094ca23413daf176546a886dbd39598914a79c0ae6cd35c8c0aeb5df564
+  data.tar.gz: 353559ba9d83853a589d843f01b52cbdb9287074b7be77505784515df99b2505317d54225456b7818c210d050b5cd170abe1a8bc315545dad5f534f9a7537244

data/CHANGELOG.md

@@ -5,6 +5,14 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.2.1] - 2026-09-28
+
+### Fixed
+
+- Always apply HTTP 200 definition bodies when the storage revision matches.
+  Same-revision responses still count as cache hits for telemetry after the
+  body is applied [OPS-1568].
+
 ## [1.2.0] - 2026-09-24
 
 ### Added

data/lib/toggly/client.rb

@@ -24,6 +24,8 @@ module Toggly
   #   )
   #   client = Toggly::Client.new(config)
   class Client
+    UNHANDLED_VARIANT_VALUE = Object.new.freeze
+
     include CacheTelemetry
     include SnapshotSupport
 
@@ -359,30 +361,43 @@ module Toggly
       return block.call(value) if block
       return nil if as.nil?
 
-      # Scalars / :boolean before Class#new — String.new({}) would soft-fail to "".
-      if as == String
-        return value.is_a?(String) ? value : nil
-      end
-      if as == Integer
-        return value.is_a?(Integer) && !value.is_a?(TrueClass) && !value.is_a?(FalseClass) ? value : nil
-      end
-      if as == Float
-        return value.is_a?(Numeric) && !value.is_a?(TrueClass) && !value.is_a?(FalseClass) ? value.to_f : nil
-      end
-      if [TrueClass, FalseClass, :boolean].include?(as)
-        return [true, false].include?(value) ? value : nil
-      end
+      scalar_value = decode_scalar_variant_value(value, as)
+      return scalar_value unless scalar_value.equal?(UNHANDLED_VARIANT_VALUE)
 
       # is_a? requires a Module; symbols like :boolean are handled above.
       return value if as.is_a?(Module) && value.is_a?(as)
-      return as.new(**value.transform_keys(&:to_sym)) if as.respond_to?(:new) && value.is_a?(Hash)
-      return as.json_create(value) if as.respond_to?(:json_create) && value.is_a?(Hash)
 
-      nil
+      decode_variant_object(value, as)
     rescue StandardError
       nil
     end
 
+    # Scalars / :boolean before Class#new — String.new({}) would soft-fail to "".
+    def decode_scalar_variant_value(value, as)
+      return value.is_a?(String) ? value : nil if as == String
+      return numeric_variant_value(value, Integer) if as == Integer
+      return numeric_variant_value(value, Float) if as == Float
+      return [true, false].include?(value) ? value : nil if [TrueClass, FalseClass, :boolean].include?(as)
+
+      UNHANDLED_VARIANT_VALUE
+    end
+
+    def numeric_variant_value(value, type)
+      return nil if value.is_a?(TrueClass) || value.is_a?(FalseClass)
+      return value if type == Integer && value.is_a?(Integer)
+      return value.to_f if type == Float && value.is_a?(Numeric)
+
+      nil
+    end
+
+    def decode_variant_object(value, as)
+      return nil unless value.is_a?(Hash)
+      return as.new(**value.transform_keys(&:to_sym)) if as.respond_to?(:new)
+      return as.json_create(value) if as.respond_to?(:json_create)
+
+      nil
+    end
+
     # `definitions` / `definitions-signed` → local rule eval. The sole
     # source of truth for `enabled?`; also carries `variants` / `allocation`
     # for the catalog-local `get_variant` assignment.

data/lib/toggly/definitions_provider.rb

@@ -189,9 +189,11 @@ module Toggly
         log_debug("Definitions not modified")
         FetchResult.new(definitions: nil, cache_outcome: :hit)
       when :same_revision
-        log_debug("Definitions revision matches existing (ETag or Last-Modified)")
-        store_revision_headers(response_etag, response_lm)
-        FetchResult.new(definitions: nil, cache_outcome: :hit)
+        # Same storage revision is a telemetry hit, but HTTP 200 still carries
+        # an authoritative body — apply it (parity with Go/Rust/Python).
+        log_debug("Definitions revision matches existing; applying HTTP 200 body")
+        result = handle_new_content(response, response_etag, response_lm)
+        FetchResult.new(definitions: result.definitions, cache_outcome: :hit)
       when :new_content
         handle_new_content(response, response_etag, response_lm)
       when :error_status

data/lib/toggly/variant_allocator.rb

@@ -112,13 +112,12 @@ module Toggly
       return nil if list.empty?
 
       pct = compute_percentile(identity, seed, feature_key, ignore_case)
-      list.each do |entry|
+      matching_entry = list.find do |entry|
         from = entry[:from].to_f
         to = entry[:to].to_f
-        matched = to >= 100.0 ? pct >= from : (pct >= from && pct < to)
-        return entry[:variant] if matched
+        to >= 100.0 ? pct >= from : (pct >= from && pct < to)
       end
-      nil
+      matching_entry&.fetch(:variant, nil)
     end
 
     # MF-parity percentile hash.

data/lib/toggly/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Toggly
-  VERSION = "1.2.0"
+  VERSION = "1.2.1"
 end

data/lib/toggly.rb

@@ -71,6 +71,8 @@ require_relative "toggly/client"
 #     }
 #   )
 module Toggly
+  NOT_CONFIGURED_ERROR = "Toggly not configured. Call Toggly.configure first."
+
   class << self
     # Global client instance
     attr_accessor :client
@@ -92,7 +94,7 @@ module Toggly
     # @param context [Context, nil] Optional evaluation context
     # @return [Boolean]
     def enabled?(feature_key, context: nil)
-      raise Error, "Toggly not configured. Call Toggly.configure first." unless @client
+      raise Error, NOT_CONFIGURED_ERROR unless @client
 
       @client.enabled?(feature_key, context: context)
     end
@@ -113,7 +115,7 @@ module Toggly
     # @param context [Context, nil]
     # @return [VariantResult, nil]
     def get_variant(feature_key, context: nil)
-      raise Error, "Toggly not configured. Call Toggly.configure first." unless @client
+      raise Error, NOT_CONFIGURED_ERROR unless @client
 
       @client.get_variant(feature_key, context: context)
     end
@@ -124,7 +126,7 @@ module Toggly
     # @param context [Context, nil]
     # @return [Object, nil]
     def get_variant_value(feature_key, context: nil, as: nil, &)
-      raise Error, "Toggly not configured. Call Toggly.configure first." unless @client
+      raise Error, NOT_CONFIGURED_ERROR unless @client
 
       @client.get_variant_value(feature_key, context: context, as: as, &)
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: toggly
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.2.1
 platform: ruby
 authors:
 - Ops.ai
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-09-25 00:00:00.000000000 Z
+date: 2026-10-04 00:00:00.000000000 Z
 dependencies: []
 description: High-performance Ruby SDK for Toggly feature flag management. Works with
   or without Toggly.io.