quonfig-openfeature 0.0.2 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f06d605c99762aee198d13fb3cc5c1e717af2c0d1868c8d9f621d00cd4eccace
-  data.tar.gz: bad9becb07bc9965e47b053d5aeaa7ff8e94ca1b09c261d74dcd911306c727c7
+  metadata.gz: 6c1221d0ff74f947b7e5d54dfa126994f293cb2924411af33f23ecd5c821f864
+  data.tar.gz: bd7ca301c3723978a72fd88cc04f662544fab16c8a2b69729d20098921438e6d
 SHA512:
-  metadata.gz: e3a473d41dbbbea5ceca03421fe2cd882284cfb10a47857707f0422bd66c04cc24fea0d87bd14f6ec2ce1f96770fcd0b2e7f387adcf4f365d07024a6b341dbaf
-  data.tar.gz: dec998e3352b5ef04bec93f0637e846bc4cdfe3085da75baf995e4923fca4fc010e4dc6846c17f5660d12ba8b3303aa9a24a951e00d0b19cf682eb9cf90c5521
+  metadata.gz: 91fb7599243c068204364b9ae3b785146bbd61d0bed1ff40f9ec5c42001c1b18b1f981c5e05b5078dd73c2a16a60cc49434774fe678bc32116a6eda1ad079dc9
+  data.tar.gz: a9393d16f9722633b5e6ddbda84a3768600440fbc681e1cd29d179716bb7ac84c430e8dff5ff2cfda7d8a476cac67b3a5596b3737e0b1bef7da1f1a9f6354979

data/VERSION

@@ -1 +1 @@
-0.0.2
+0.0.4

data/lib/quonfig/openfeature/provider.rb

@@ -97,27 +97,13 @@ module Quonfig
 
       def fetch_boolean_value(flag_key:, default_value:, evaluation_context: nil)
         evaluate(flag_key, default_value, evaluation_context) do |client, mapped_ctx|
-          value = client.get_bool(flag_key, default: nil, context: mapped_ctx)
-          if value.nil?
-            ResolutionDetails.new(value: default_value, reason: Reason::ERROR,
-                                  error_code: ErrorCode::FLAG_NOT_FOUND,
-                                  error_message: "flag not found: #{flag_key}")
-          else
-            ResolutionDetails.new(value: value, reason: Reason::TARGETING_MATCH)
-          end
+          to_resolution(client.get_bool_details(flag_key, context: mapped_ctx), default_value)
         end
       end
 
       def fetch_string_value(flag_key:, default_value:, evaluation_context: nil)
         evaluate(flag_key, default_value, evaluation_context) do |client, mapped_ctx|
-          value = client.get_string(flag_key, default: nil, context: mapped_ctx)
-          if value.nil?
-            ResolutionDetails.new(value: default_value, reason: Reason::ERROR,
-                                  error_code: ErrorCode::FLAG_NOT_FOUND,
-                                  error_message: "flag not found: #{flag_key}")
-          else
-            ResolutionDetails.new(value: value, reason: Reason::TARGETING_MATCH)
-          end
+          to_resolution(client.get_string_details(flag_key, context: mapped_ctx), default_value)
         end
       end
 
@@ -126,76 +112,81 @@ module Quonfig
         # first, fall back to float so we transparently handle both Quonfig
         # int and double configs.
         evaluate(flag_key, default_value, evaluation_context) do |client, mapped_ctx|
-          value = nil
-          begin
-            value = client.get_int(flag_key, default: nil, context: mapped_ctx)
-          rescue ::Quonfig::Errors::TypeMismatchError
-            value = client.get_float(flag_key, default: nil, context: mapped_ctx)
-          end
-
-          if value.nil?
-            ResolutionDetails.new(value: default_value, reason: Reason::ERROR,
-                                  error_code: ErrorCode::FLAG_NOT_FOUND,
-                                  error_message: "flag not found: #{flag_key}")
-          else
-            ResolutionDetails.new(value: value, reason: Reason::TARGETING_MATCH)
+          details = client.get_int_details(flag_key, context: mapped_ctx)
+          if details.error_code == ::Quonfig::EvaluationDetails::ERROR_TYPE_MISMATCH
+            details = client.get_float_details(flag_key, context: mapped_ctx)
           end
+          to_resolution(details, default_value)
         end
       end
 
       def fetch_integer_value(flag_key:, default_value:, evaluation_context: nil)
         evaluate(flag_key, default_value, evaluation_context) do |client, mapped_ctx|
-          value = client.get_int(flag_key, default: nil, context: mapped_ctx)
-          if value.nil?
-            ResolutionDetails.new(value: default_value, reason: Reason::ERROR,
-                                  error_code: ErrorCode::FLAG_NOT_FOUND,
-                                  error_message: "flag not found: #{flag_key}")
-          else
-            ResolutionDetails.new(value: value, reason: Reason::TARGETING_MATCH)
-          end
+          to_resolution(client.get_int_details(flag_key, context: mapped_ctx), default_value)
         end
       end
 
       def fetch_float_value(flag_key:, default_value:, evaluation_context: nil)
         evaluate(flag_key, default_value, evaluation_context) do |client, mapped_ctx|
-          value = client.get_float(flag_key, default: nil, context: mapped_ctx)
-          if value.nil?
-            ResolutionDetails.new(value: default_value, reason: Reason::ERROR,
-                                  error_code: ErrorCode::FLAG_NOT_FOUND,
-                                  error_message: "flag not found: #{flag_key}")
-          else
-            ResolutionDetails.new(value: value, reason: Reason::TARGETING_MATCH)
-          end
+          to_resolution(client.get_float_details(flag_key, context: mapped_ctx), default_value)
         end
       end
 
-      # Object resolution tries +get_string_list+ first (so Quonfig
+      # Object resolution tries +get_string_list_details+ first (so Quonfig
       # +string_list+ configs surface as native arrays), then falls back to
-      # +get_json+ for any other JSON-shaped config.
+      # +get_json_details+ for any other JSON-shaped config.
       def fetch_object_value(flag_key:, default_value:, evaluation_context: nil)
         evaluate(flag_key, default_value, evaluation_context) do |client, mapped_ctx|
-          value = nil
-          begin
-            value = client.get_string_list(flag_key, default: nil, context: mapped_ctx)
-          rescue ::Quonfig::Errors::TypeMismatchError
-            value = nil
-          end
-          if value.nil?
-            value = client.get_json(flag_key, default: nil, context: mapped_ctx)
-          end
-
-          if value.nil?
-            ResolutionDetails.new(value: default_value, reason: Reason::ERROR,
-                                  error_code: ErrorCode::FLAG_NOT_FOUND,
-                                  error_message: "flag not found: #{flag_key}")
-          else
-            ResolutionDetails.new(value: value, reason: Reason::TARGETING_MATCH)
+          details = client.get_string_list_details(flag_key, context: mapped_ctx)
+          if details.error_code == ::Quonfig::EvaluationDetails::ERROR_TYPE_MISMATCH
+            details = client.get_json_details(flag_key, context: mapped_ctx)
           end
+          to_resolution(details, default_value)
         end
       end
 
       private
 
+      # Map a Quonfig::EvaluationDetails to the OpenFeature ResolutionDetails
+      # the SDK consumes. The Quonfig SDK's *_details methods don't raise, so
+      # this is a pure mapping layer.
+      def to_resolution(details, default_value)
+        case details.reason
+        when ::Quonfig::EvaluationDetails::REASON_STATIC
+          ResolutionDetails.new(value: details.value, reason: Reason::STATIC)
+        when ::Quonfig::EvaluationDetails::REASON_TARGETING_MATCH
+          ResolutionDetails.new(value: details.value, reason: Reason::TARGETING_MATCH)
+        when ::Quonfig::EvaluationDetails::REASON_SPLIT
+          ResolutionDetails.new(value: details.value, reason: Reason::SPLIT)
+        when ::Quonfig::EvaluationDetails::REASON_DEFAULT
+          ResolutionDetails.new(value: default_value, reason: Reason::DEFAULT)
+        when ::Quonfig::EvaluationDetails::REASON_ERROR
+          ResolutionDetails.new(
+            value: default_value,
+            reason: Reason::ERROR,
+            error_code: map_error_code(details.error_code),
+            error_message: details.error_message
+          )
+        else
+          # Defensive default: surface as ERROR so unknown reasons don't
+          # silently leak the SDK value back.
+          ResolutionDetails.new(
+            value: default_value,
+            reason: Reason::ERROR,
+            error_code: ErrorCode::GENERAL,
+            error_message: "unknown reason: #{details.reason.inspect}"
+          )
+        end
+      end
+
+      def map_error_code(error_code)
+        case error_code
+        when ::Quonfig::EvaluationDetails::ERROR_FLAG_NOT_FOUND then ErrorCode::FLAG_NOT_FOUND
+        when ::Quonfig::EvaluationDetails::ERROR_TYPE_MISMATCH  then ErrorCode::TYPE_MISMATCH
+        else ErrorCode::GENERAL
+        end
+      end
+
       def evaluate(flag_key, default_value, evaluation_context)
         client = @client
         if client.nil?
@@ -209,14 +200,6 @@ module Quonfig
 
         mapped_ctx = Context.map_context(evaluation_context, @targeting_key_mapping)
         yield(client, mapped_ctx)
-      rescue ::Quonfig::Errors::MissingDefaultError => e
-        ResolutionDetails.new(value: default_value, reason: Reason::ERROR,
-                              error_code: ErrorCode::FLAG_NOT_FOUND,
-                              error_message: e.message)
-      rescue ::Quonfig::Errors::TypeMismatchError => e
-        ResolutionDetails.new(value: default_value, reason: Reason::ERROR,
-                              error_code: ErrorCode::TYPE_MISMATCH,
-                              error_message: e.message)
       rescue ::Quonfig::Errors::UninitializedError, ::Quonfig::Errors::InitializationTimeoutError => e
         ResolutionDetails.new(value: default_value, reason: Reason::ERROR,
                               error_code: ErrorCode::PROVIDER_NOT_READY,

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: quonfig-openfeature
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.4
 platform: ruby
 authors:
 - Jeff Dwyer
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-04-27 00:00:00.000000000 Z
+date: 2026-05-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: openfeature-sdk
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.0.8
+        version: 0.0.12
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.0.8
+        version: 0.0.12
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement