quonfig 0.0.17 → 0.0.18

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 68d9721e3220acc150e33b43e993c8b8b2380056453939b62b06b05cc4ef4255
-  data.tar.gz: 643c409f2b8fa3d5291d92fcf8a0d39cf5a2a67b4d697036497a19296f584d72
+  metadata.gz: 125753bc634b155cdae7cf4772705ee74c5ec37f4a612c9c52ea873a94d0c5ca
+  data.tar.gz: 2c6e09f01e7ed54cf7e6f0fbb33784ea62e0d2bd069a87e3d74dafe3ed97aeb1
 SHA512:
-  metadata.gz: c133fdcdf47da1b026465f42dfc71e98be03590bb0df80ebd247a572bce6404a59b5f411b014e83349ca2278af2c4c62974c22c1ea9c0c7b1af474d933e91725
-  data.tar.gz: c982a887a21dcfe7545e2b50b71a09f2fb7465827819676a238bdbd87cbeaa7626f26e723eb3535c8e3f893de4f7bcebc93e31264783b76716add5057ece836c
+  metadata.gz: 7e9474c7aa96611977db52658ba730efab5aafddb811e68dbd8ce90833d3c3017f6d5bc9b870db54be4b125844f43d46d96c067179a5cfd744880e4ca32cbd79
+  data.tar.gz: 26e638dbdeb223f06d9742cd155fd2b5b33073b20ee6e397a3322564979042c9c9dff8a3f893127c30ca70544202fc04ba2e3ce6c6c2f58332613dba884f9c33

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 0.0.18 - 2026-05-21
+
+- **Fix (SSE): give Net `read_timeout` headroom over the watchdog deadline (qfg-6y44).** `stream_once` armed two read deadlines at the identical `sse_read_timeout` value: `Net::HTTP#read_timeout` and the `ReadDeadlineWatchdog`. On the body read both were live, and the watchdog carries up to `POLL_INTERVAL` (0.25 s) of polling latency on top of its deadline — so when Net's (unreliable on the `read_body` path) stdlib timeout did fire, it could beat the watchdog and surface a `Net::ReadTimeout` instead of the `SSEReadDeadlineExceeded` the SDK is instrumented around. A new `READ_TIMEOUT_HEADROOM` (30 s) keeps Net's `read_timeout` as a redundant backstop while guaranteeing the watchdog fires first.
+- **Fix (datadir): coerce int/double config values to numbers at load time (qfg-38sf.8).** Config files store `int`/`double` Value fields as JSON strings (`{"type":"int","value":"123"}`). api-delivery normalizes these to real numbers at config-load time (`Value.UnmarshalJSON`), so every HTTP/SSE envelope already carries JSON numbers — but the datadir loader read the files directly and passed the strings through verbatim. `Quonfig::Datadir` now runs a generic recursive `coerce_numeric_values` walk over each parsed config document before projecting it to a `ConfigResponse`: any Value node (`type` of `"int"`/`"double"` with a String `value`) is coerced via `Integer(value, 10)` / `Float(value)`, covering `default.rules[].value`, environment rules, `criteria[].valueToMatch`, weighted-value arms, and variants. An unparseable numeric string is left untouched (passthrough — never raises). Brings the datadir loader in line with sdk-go and api-delivery so the loaded envelope always carries real numbers regardless of who consumes it.
+- **CI: pin `integration-test-data` to v2026.05.20 and guard against stale generated tests (#12).** The integration suite now resolves its YAML specs from a pinned tag, and a guard fails the build if the generated tests drift from the templates.
+- **CI: skip the Chaos workflow on Dependabot PRs and de-flake the datadir-reload test (8f60d9c).**
+- **Dependency bumps (CI actions):** `actions/setup-go` 5.6.0 → 6.4.0 (#7), `actions/checkout` 4.3.1 → 6.0.2 (#8), `actions/upload-artifact` 4.6.2 → 7.0.1 (#9), `ruby/setup-ruby` 1.306.0 → 1.310.0 (#10).
+
 ## 0.0.17 - 2026-05-19
 
 - **Feat (datadir): opt-in `data_dir_auto_reload` (qfg-mol-2da).** Datadir mode previously loaded the workspace once at construction and served purely from memory. Set `data_dir_auto_reload: true` to have the SDK watch the configured `datadir`, re-read `Quonfig::Datadir.load_envelope`, and fire the existing `on_update` callback whenever files change. Adds `listen ~> 3.8` (FSEvents on macOS, inotify on Linux, polling fallback on Windows) as a runtime dep. Behavior: parse-then-swap (a failed parse keeps the previous envelope and skips the callback), debounced (`data_dir_auto_reload_debounce_ms`, default 200 ms — bursts coalesce to one reload), and gracefully downgrades when watch registration fails (read-only fs, immutable container, missing native backend). Symlinked datadirs are resolved to their real path before watching. Default is `false`; opt-in only.

data/lib/quonfig/datadir.rb

@@ -42,6 +42,7 @@ module Quonfig
           raw = JSON.parse(File.read(path))
           raise ArgumentError, "[quonfig] config has empty key — file is not a Quonfig Config: #{path}" if raw['key'].nil? || raw['key'].to_s.empty?
 
+          coerce_numeric_values(raw)
           configs << to_config_response(raw, env_id)
         end
       end
@@ -94,5 +95,42 @@ module Quonfig
 
       raw || false
     end
+
+    # Config files store int/double Value fields as JSON strings
+    # (`{"type":"int","value":"123"}`). api-delivery normalizes these to real
+    # numbers at config-load time (`Value.UnmarshalJSON`), so every envelope it
+    # emits over HTTP/SSE already carries JSON numbers. In datadir mode we read
+    # the files directly, so we must coerce here to match.
+    #
+    # Walks the parsed config document in place, coercing every Value node — any
+    # Hash with a `type` of `"int"`/`"double"` and a String `value` — to a real
+    # number. A generic recursive walk covers `default.rules[].value`,
+    # environment rules, `criteria[].valueToMatch`, weighted-value arms, and
+    # variants without enumerating each location. On parse failure the original
+    # string is left in place (passthrough — never raise).
+    def coerce_numeric_values(node)
+      case node
+      when Hash
+        coerce_numeric_value_field(node)
+        node.each_value { |child| coerce_numeric_values(child) }
+      when Array
+        node.each { |child| coerce_numeric_values(child) }
+      end
+      node
+    end
+
+    def coerce_numeric_value_field(hash)
+      value = hash['value']
+      return unless value.is_a?(String)
+
+      case hash['type']
+      when 'int'
+        hash['value'] = Integer(value, 10)
+      when 'double'
+        hash['value'] = Float(value)
+      end
+    rescue ArgumentError, TypeError
+      # Unparseable numeric string — leave the original value untouched.
+    end
   end
 end

data/lib/quonfig/sse_config_client.rb

@@ -56,6 +56,17 @@ module Quonfig
     # Anything else (5xx, 429, network errors) stays on the transient path.
     TERMINAL_HTTP_CODES = [401, 403, 404].freeze
 
+    # qfg-6y44: headroom added to +sse_read_timeout+ when configuring
+    # Net::HTTP#read_timeout. The ReadDeadlineWatchdog already covers both the
+    # header and body reads at exactly +sse_read_timeout+; Net's own
+    # read_timeout is only a redundant backstop. Arming it at the *same* value
+    # as the watchdog makes the two race — and on the body path Net's
+    # (unreliable) timeout can fire first, surfacing a Net::ReadTimeout instead
+    # of the SSEReadDeadlineExceeded the SDK is instrumented around. Giving
+    # Net's read_timeout this much headroom keeps it a backstop without ever
+    # letting it win the race.
+    READ_TIMEOUT_HEADROOM = 30
+
     # +on_error+: optional callable invoked on every SSE error edge. Parent
     # Quonfig::Client wires this to drive @sse_state -> :error so that
     # +connection_state+ reflects the disconnect (qfg-47c2.27).
@@ -254,8 +265,11 @@ module Quonfig
       http.use_ssl = (uri.scheme == 'https')
       http.open_timeout = @options.sse_connect_timeout
       # Keep Net::HTTP's read_timeout as a backstop for the header read
-      # (where it does apply reliably). The watchdog covers the body path.
-      http.read_timeout = @options.sse_read_timeout
+      # (where it does apply reliably). The watchdog covers the body path —
+      # so read_timeout gets READ_TIMEOUT_HEADROOM over the watchdog deadline
+      # to guarantee the watchdog fires first and we get a deterministic
+      # SSEReadDeadlineExceeded rather than a racy Net::ReadTimeout (qfg-6y44).
+      http.read_timeout = @options.sse_read_timeout + READ_TIMEOUT_HEADROOM
 
       req = Net::HTTP::Get.new(uri.request_uri, headers)
 

data/lib/quonfig/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Quonfig
-  VERSION = '0.0.17'
+  VERSION = '0.0.18'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: quonfig
 version: !ruby/object:Gem::Version
-  version: 0.0.17
+  version: 0.0.18
 platform: ruby
 authors:
 - Jeff Dwyer
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-05-19 00:00:00.000000000 Z
+date: 2026-05-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport