openfeature-sdk 0.6.2 → 0.6.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 304772054cf10c0cc02870fdeb7a1a7412342912e3f8a08d80c33fe25bd99849
-  data.tar.gz: 6a27c2a8bf23074292594f4eec0ecd4dd9df11ddc1e3e48c4c7ff245e76a6381
+  metadata.gz: bd3ec1b654a844eaa7bcafe400d6d2075199fa49ba6823aad1c8e1ce8c05a526
+  data.tar.gz: 3d159cb4f4df832154cfa7a4657ee53f357827a3d55dcbe124bf4716248e24b8
 SHA512:
-  metadata.gz: bcaacd882ecdf0ebd0643e1a839bae3d4efb02deb30f2128c5feca91c93e85bee65b755147cf4bef6edabc010875746b777cd4570f26161451a620db396e02e8
-  data.tar.gz: 5a6ba82992f4d8e960d350a573e48a5e015c9b8eb953737354a4573d0a9f89ca6c4615553d0240a91c1490aad8fc3e1c2ca4bfeddc64d0b1203835ede55e1c77
+  metadata.gz: 66667a224162e00b882ad73b45e5d2825cc35f47007ca961749cc7115cbe1223d881be0cfff89eb4a5d3fd3f0235c2548f7a89420de9ff7bb3a6b0fd4bcada65
+  data.tar.gz: f799ac085e5593bc9b14a040ebddc8326c57ff46e36285ed0ea9b44008dd0d8d58a149a1c0f7fc9be50aeb3cf8d60ffc2859f2b3f9099d49e1db3df6f795b669

data/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "0.6.2"
+  ".": "0.6.3"
 }

data/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## [0.6.3](https://github.com/open-feature/ruby-sdk/compare/v0.6.2...v0.6.3) (2026-03-07)
+
+
+### Features
+
+* close spec compliance gaps for OpenFeature v0.8.0 ([#237](https://github.com/open-feature/ruby-sdk/issues/237)) ([9a87d04](https://github.com/open-feature/ruby-sdk/commit/9a87d04d5f261ea06e073f405c15613db7099d8a))
+* enable Gherkin feature tests ([#50](https://github.com/open-feature/ruby-sdk/issues/50)) ([#233](https://github.com/open-feature/ruby-sdk/issues/233)) ([95845ba](https://github.com/open-feature/ruby-sdk/commit/95845ba6ec26357d9c0895d310361e411f85da11))
+
+
+### Bug Fixes
+
+* close remaining MUST-level spec compliance gaps ([#238](https://github.com/open-feature/ruby-sdk/issues/238)) ([1d08491](https://github.com/open-feature/ruby-sdk/commit/1d084911964c8672dd66b23834eec6f14e453749))
+
 ## [0.6.2](https://github.com/open-feature/ruby-sdk/compare/v0.6.1...v0.6.2) (2026-03-07)
 
 

data/Gemfile

@@ -4,3 +4,6 @@ source "https://rubygems.org"
 
 # Specify your gem's dependencies in openfeature-sdk.gemspec
 gemspec
+
+gem "cucumber", "~> 10.0", group: :test
+gem "logger", group: :test

data/Gemfile.lock

@@ -1,12 +1,40 @@
 PATH
   remote: .
   specs:
-    openfeature-sdk (0.6.2)
+    openfeature-sdk (0.6.3)
 
 GEM
   remote: https://rubygems.org/
   specs:
     ast (2.4.3)
+    base64 (0.3.0)
+    bigdecimal (4.0.1)
+    builder (3.3.0)
+    cucumber (10.2.0)
+      base64 (~> 0.2)
+      builder (~> 3.2)
+      cucumber-ci-environment (> 9, < 12)
+      cucumber-core (> 15, < 17)
+      cucumber-cucumber-expressions (> 17, < 20)
+      cucumber-html-formatter (> 21, < 23)
+      diff-lcs (~> 1.5)
+      logger (~> 1.6)
+      mini_mime (~> 1.1)
+      multi_test (~> 1.1)
+      sys-uname (~> 1.3)
+    cucumber-ci-environment (11.0.0)
+    cucumber-core (16.2.0)
+      cucumber-gherkin (> 36, < 40)
+      cucumber-messages (> 31, < 33)
+      cucumber-tag-expressions (> 6, < 9)
+    cucumber-cucumber-expressions (19.0.0)
+      bigdecimal
+    cucumber-gherkin (39.0.0)
+      cucumber-messages (>= 31, < 33)
+    cucumber-html-formatter (22.3.0)
+      cucumber-messages (> 23, < 33)
+    cucumber-messages (32.2.0)
+    cucumber-tag-expressions (8.1.0)
     date (3.5.1)
     debug (1.11.1)
       irb (~> 1.10)
@@ -14,6 +42,7 @@ GEM
     diff-lcs (1.6.2)
     docile (1.4.1)
     erb (6.0.2)
+    ffi (1.17.3)
     io-console (0.8.2)
     irb (1.17.0)
       pp (>= 0.6.0)
@@ -23,7 +52,11 @@ GEM
     json (2.18.1)
     language_server-protocol (3.17.0.5)
     lint_roller (1.1.0)
+    logger (1.7.0)
     markly (0.15.2)
+    memoist3 (1.0.0)
+    mini_mime (1.1.5)
+    multi_test (1.1.0)
     parallel (1.27.0)
     parser (3.3.10.2)
       ast (~> 2.4.1)
@@ -100,6 +133,9 @@ GEM
       lint_roller (~> 1.1)
       rubocop-performance (~> 1.26.0)
     stringio (3.2.0)
+    sys-uname (1.5.0)
+      ffi (~> 1.1)
+      memoist3 (~> 1.0.0)
     timecop (0.9.10)
     tsort (0.2.0)
     unicode-display_width (3.2.0)
@@ -120,7 +156,9 @@ PLATFORMS
   x86_64-linux
 
 DEPENDENCIES
+  cucumber (~> 10.0)
   debug
+  logger
   markly
   openfeature-sdk!
   rake (~> 13.0)

data/README.md

@@ -17,8 +17,8 @@
   </a>
   <!-- x-release-please-start-version -->
 
-  <a href="https://github.com/open-feature/ruby-sdk/releases/tag/v0.6.2">
-    <img alt="Release" src="https://img.shields.io/static/v1?label=release&message=v0.6.2&color=blue&style=for-the-badge" />
+  <a href="https://github.com/open-feature/ruby-sdk/releases/tag/v0.6.3">
+    <img alt="Release" src="https://img.shields.io/static/v1?label=release&message=v0.6.3&color=blue&style=for-the-badge" />
   </a>
 
   <!-- x-release-please-end -->

data/Rakefile

@@ -7,4 +7,9 @@ RSpec::Core::RakeTask.new(:spec)
 
 require "standard/rake"
 
+desc "Run Cucumber Gherkin feature tests"
+task :cucumber do
+  sh "bundle exec cucumber"
+end
+
 task default: %i[spec standard]

data/cucumber.yml

@@ -0,0 +1,2 @@
+default: spec/open-feature-spec/specification/assets/gherkin --require features --tags "not @deprecated" --publish-quiet
+deprecated: spec/open-feature-spec/specification/assets/gherkin --require features --publish-quiet

data/lib/open_feature/sdk/api.rb

@@ -34,7 +34,7 @@ module OpenFeature
       include Singleton # Satisfies Flag Evaluation API Requirement 1.1.1
       extend Forwardable
 
-      def_delegators :configuration, :provider, :set_provider, :set_provider_and_wait, :hooks, :evaluation_context
+      def_delegators :configuration, :provider, :set_provider, :set_provider_and_wait, :hooks, :add_hooks, :evaluation_context
 
       def configuration
         @configuration ||= Configuration.new
@@ -54,6 +54,11 @@ module OpenFeature
         Client.new(provider: Provider::NoOpProvider.new, evaluation_context:)
       end
 
+      def provider_metadata(domain: nil)
+        prov = provider(domain: domain)
+        prov.metadata if prov&.respond_to?(:metadata)
+      end
+
       def add_handler(event_type, handler)
         configuration.add_handler(event_type, handler)
       end

data/lib/open_feature/sdk/client.rb

@@ -28,6 +28,10 @@ module OpenFeature
         @hooks = []
       end
 
+      def add_hooks(*new_hooks)
+        @hooks.concat(new_hooks.flatten)
+      end
+
       def provider_status
         OpenFeature::SDK.configuration.provider_state(@provider)
       end
@@ -70,27 +74,40 @@ module OpenFeature
       def fetch_details(type:, flag_key:, default_value:, evaluation_context: nil, invocation_hooks: [], hook_hints: nil)
         validate_default_value_type(type, default_value)
 
-        if OpenFeature::SDK.configuration.provider_tracked?(@provider)
-          error_code = short_circuit_error_code(provider_status)
-          if error_code
-            resolution = Provider::ResolutionDetails.new(
-              value: default_value,
-              error_code: error_code,
-              reason: Provider::Reason::ERROR
-            )
-            return EvaluationDetails.new(flag_key: flag_key, resolution_details: resolution)
-          end
-        end
-
         built_context = build_evaluation_context(evaluation_context)
 
         # Assemble ordered hooks: API → Client → Invocation → Provider (spec 4.4.2)
         provider_hooks = @provider.respond_to?(:hooks) ? Array(@provider.hooks) : []
         ordered_hooks = [*OpenFeature::SDK.hooks, *@hooks, *invocation_hooks, *provider_hooks]
 
+        # Check for short-circuit conditions (spec 1.7.6 + 1.7.7)
+        short_circuit_code = nil
+        if OpenFeature::SDK.configuration.provider_tracked?(@provider)
+          short_circuit_code = short_circuit_error_code(provider_status)
+        end
+
         # Fast path: skip hook ceremony when no hooks are registered
         if ordered_hooks.empty?
-          return evaluate_flag(type: type, flag_key: flag_key, default_value: default_value, evaluation_context: built_context)
+          if short_circuit_code
+            resolution = Provider::ResolutionDetails.new(
+              value: default_value,
+              error_code: short_circuit_code,
+              reason: Provider::Reason::ERROR
+            )
+            return EvaluationDetails.new(flag_key: flag_key, resolution_details: resolution)
+          end
+
+          begin
+            return evaluate_flag(type: type, flag_key: flag_key, default_value: default_value, evaluation_context: built_context)
+          rescue => e
+            resolution = Provider::ResolutionDetails.new(
+              value: default_value,
+              error_code: Provider::ErrorCode::GENERAL,
+              reason: Provider::Reason::ERROR,
+              error_message: e.message
+            )
+            return EvaluationDetails.new(flag_key: flag_key, resolution_details: resolution)
+          end
         end
 
         hook_context = Hooks::HookContext.new(
@@ -111,8 +128,21 @@ module OpenFeature
         end
 
         executor = Hooks::HookExecutor.new(logger: OpenFeature::SDK.configuration.logger)
-        executor.execute(ordered_hooks: ordered_hooks, hook_context: hook_context, hints: hints) do |hctx|
-          evaluate_flag(type: type, flag_key: flag_key, default_value: default_value, evaluation_context: hctx.evaluation_context)
+
+        if short_circuit_code
+          # Spec 1.7.6 + 1.7.7: short-circuit must still run error hooks and finally hooks
+          resolution = Provider::ResolutionDetails.new(
+            value: default_value,
+            error_code: short_circuit_code,
+            reason: Provider::Reason::ERROR
+          )
+          evaluation_details = EvaluationDetails.new(flag_key: flag_key, resolution_details: resolution)
+          executor.run_short_circuit(ordered_hooks: ordered_hooks, hook_context: hook_context, hints: hints, evaluation_details: evaluation_details)
+          evaluation_details
+        else
+          executor.execute(ordered_hooks: ordered_hooks, hook_context: hook_context, hints: hints) do |hctx|
+            evaluate_flag(type: type, flag_key: flag_key, default_value: default_value, evaluation_context: hctx.evaluation_context)
+          end
         end
       end
 
@@ -128,6 +158,7 @@ module OpenFeature
           resolution_details.value = default_value
           resolution_details.error_code = Provider::ErrorCode::TYPE_MISMATCH
           resolution_details.reason = Provider::Reason::ERROR
+          resolution_details.variant = nil
         end
 
         EvaluationDetails.new(flag_key: flag_key, resolution_details: resolution_details)

data/lib/open_feature/sdk/client_metadata.rb

@@ -2,6 +2,8 @@
 
 module OpenFeature
   module SDK
-    ClientMetadata = Struct.new(:domain, keyword_init: true)
+    ClientMetadata = Struct.new(:domain, keyword_init: true) do
+      alias_method :name, :domain
+    end
   end
 end

data/lib/open_feature/sdk/configuration.rb

@@ -31,6 +31,10 @@ module OpenFeature
         @client_handlers_mutex = Mutex.new
       end
 
+      def add_hooks(*new_hooks)
+        @hooks.concat(new_hooks.flatten)
+      end
+
       def provider(domain: nil)
         @providers[domain] || @providers[nil]
       end
@@ -85,9 +89,11 @@ module OpenFeature
       end
 
       def shutdown
-        providers_to_shutdown = @provider_mutex.synchronize { @providers.values.uniq }
+        providers_to_shutdown = @provider_mutex.synchronize { @providers.values.uniq(&:object_id) }
 
         providers_to_shutdown.each do |prov|
+          # Spec 1.7.9: Set provider state to NOT_READY before shutdown
+          @provider_state_registry.set_initial_state(prov, ProviderState::NOT_READY)
           prov.shutdown if prov.respond_to?(:shutdown)
         rescue => e
           @logger&.warn("Error shutting down provider #{prov&.class&.name || "unknown"}: #{e.message}")
@@ -107,34 +113,46 @@ module OpenFeature
         @provider_mutex.synchronize do
           @providers.clear
         end
+        @hooks.clear
+        @evaluation_context = nil
+        @transaction_context_propagator = nil
       end
 
       def set_provider_internal(provider, domain:, wait_for_init:)
         # Capture evaluation context before acquiring mutex to prevent race conditions
         context_for_init = @evaluation_context
 
-        old_provider, provider_to_init = nil
+        old_provider = nil
+        needs_init = false
+        needs_shutdown = false
 
         @provider_mutex.synchronize do
           old_provider = @providers[domain]
 
-          # Remove old provider state to prevent memory leaks
-          @provider_state_registry.remove_provider(old_provider)
-
           new_providers = @providers.dup
           new_providers[domain] = provider
           @providers = new_providers
 
-          @provider_state_registry.set_initial_state(provider)
+          # Spec 1.1.2.2: Only initialize if the provider is not already active
+          # (i.e., not already bound to another domain)
+          already_active = @providers.any? { |d, p| d != domain && p.equal?(provider) && @provider_state_registry.tracked?(p) }
+          needs_init = !already_active
 
-          provider.send(:attach, self) if provider.is_a?(Provider::EventEmitter)
+          if needs_init
+            @provider_state_registry.set_initial_state(provider)
+            provider.send(:attach, self) if provider.is_a?(Provider::EventEmitter)
+          end
 
-          provider_to_init = provider
+          # Spec 1.1.2.3: Only shutdown old provider if it's no longer bound to any domain
+          if old_provider && !old_provider.equal?(provider)
+            still_bound = @providers.any? { |_, p| p.equal?(old_provider) }
+            needs_shutdown = !still_bound
+            @provider_state_registry.remove_provider(old_provider) unless still_bound
+          end
         end
 
         # Shutdown old provider outside mutex to avoid blocking other operations
-        # Only shutdown if it's a different provider to prevent race condition
-        if old_provider && old_provider != provider
+        if needs_shutdown
           begin
             old_provider.shutdown if old_provider.respond_to?(:shutdown)
           rescue => e
@@ -143,12 +161,17 @@ module OpenFeature
         end
 
         # Initialize provider outside the mutex to avoid blocking other operations
-        if wait_for_init
-          init_provider(provider_to_init, context_for_init, raise_on_error: true)
-        else
-          Thread.new do
-            init_provider(provider_to_init, context_for_init, raise_on_error: false)
+        if needs_init
+          if wait_for_init
+            init_provider(provider, context_for_init, raise_on_error: true)
+          else
+            Thread.new do
+              init_provider(provider, context_for_init, raise_on_error: false)
+            end
           end
+        elsif wait_for_init
+          # Provider already active; no init needed but still dispatch READY
+          dispatch_provider_event(provider, ProviderEvent::PROVIDER_READY)
         end
       end
 
@@ -164,16 +187,22 @@ module OpenFeature
 
         dispatch_provider_event(provider, ProviderEvent::PROVIDER_READY)
       rescue => e
+        # Spec 1.7.8: Propagate error code from provider if available
+        error_code = if e.respond_to?(:error_code) && e.error_code
+          e.error_code
+        else
+          Provider::ErrorCode::GENERAL
+        end
+
         dispatch_provider_event(provider, ProviderEvent::PROVIDER_ERROR,
-          error_code: Provider::ErrorCode::GENERAL,
+          error_code: error_code,
           message: e.message)
 
         if raise_on_error
-          # Re-raise as ProviderInitializationError for synchronous callers
           raise ProviderInitializationError.new(
             "Provider #{provider.class.name} initialization failed: #{e.message}",
             provider:,
-            error_code: Provider::ErrorCode::GENERAL,
+            error_code: error_code,
             original_error: e
           )
         end

data/lib/open_feature/sdk/hooks/hook_executor.rb

@@ -18,6 +18,20 @@ module OpenFeature
           @logger = logger
         end
 
+        # Runs error hooks and finally hooks for a short-circuit evaluation
+        # (spec 1.7.6 + 1.7.7). Before hooks and after hooks are NOT run.
+        #
+        # @param ordered_hooks [Array] hooks in before-order
+        # @param hook_context [HookContext] the hook context
+        # @param hints [Hints] hook hints
+        # @param evaluation_details [EvaluationDetails] the short-circuit result
+        def run_short_circuit(ordered_hooks:, hook_context:, hints:, evaluation_details:)
+          error = StandardError.new(evaluation_details.error_code)
+          run_error_hooks(ordered_hooks, hook_context, error, hints)
+        ensure
+          run_finally_hooks(ordered_hooks, hook_context, evaluation_details, hints)
+        end
+
         # Executes the full hook lifecycle around the flag evaluation block.
         #
         # @param ordered_hooks [Array] hooks in before-order (API, Client, Invocation, Provider)
@@ -31,7 +45,15 @@ module OpenFeature
           begin
             run_before_hooks(ordered_hooks, hook_context, hints)
             evaluation_details = evaluate_block.call(hook_context)
-            run_after_hooks(ordered_hooks, hook_context, evaluation_details, hints)
+
+            # Spec 4.3.6: If evaluation resulted in an error (e.g. FLAG_NOT_FOUND,
+            # TYPE_MISMATCH), run error hooks instead of after hooks.
+            if evaluation_details.error_code
+              error = StandardError.new(evaluation_details.error_message || evaluation_details.error_code)
+              run_error_hooks(ordered_hooks, hook_context, error, hints)
+            else
+              run_after_hooks(ordered_hooks, hook_context, evaluation_details, hints)
+            end
           rescue => e
             run_error_hooks(ordered_hooks, hook_context, e, hints)
 

data/lib/open_feature/sdk/provider/in_memory_provider.rb

@@ -29,8 +29,10 @@ module OpenFeature
         end
 
         def update_flags(new_flags)
+          old_keys = @flags.keys
           @flags = new_flags.dup
-          emit_provider_changed(new_flags.keys)
+          changed_keys = (old_keys | new_flags.keys)
+          emit_provider_changed(changed_keys)
         end
 
         def fetch_boolean_value(flag_key:, default_value:, evaluation_context: nil)

data/lib/open_feature/sdk/version.rb

@@ -2,6 +2,6 @@
 
 module OpenFeature
   module SDK
-    VERSION = "0.6.2"
+    VERSION = "0.6.3"
   end
 end

data/renovate.json

@@ -3,7 +3,7 @@
   "extends": [
     "config:recommended",
     ":automergeStableNonMajor",
-    "npm:unpublishSafe"
+    "security:minimumReleaseAgeNpm"
   ],
   "semanticCommits": "enabled"
 }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: openfeature-sdk
 version: !ruby/object:Gem::Version
-  version: 0.6.2
+  version: 0.6.3
 platform: ruby
 authors:
 - OpenFeature Authors
@@ -159,6 +159,7 @@ files:
 - LICENSE
 - README.md
 - Rakefile
+- cucumber.yml
 - lib/open_feature/sdk.rb
 - lib/open_feature/sdk/api.rb
 - lib/open_feature/sdk/client.rb