ask-observability 0.1.0 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d0e28b911f472b83c6a7c93bcd632b9fe981fb39aa6a73e43dbeadc287199faa
-  data.tar.gz: 8480c74048d04b7e9cf83bfe9614d5d927bdb31865f11a5e159db2ba3ccbce26
+  metadata.gz: ffbf52d9051f77149044a23bff976f8e1fe9558a9915e4f48f5b2c0fbf6cbe52
+  data.tar.gz: ba7ea476ca3526ba574c653563e30d1fdeab6a61865345c54bf21205cd560053
 SHA512:
-  metadata.gz: 49f733327ed7c5c5f3c53931919c41bb7234ec2b487f0dae8d85814cfa3eb3416009cc53e2deba9901d6bb00f1991db5e1ececfed726fe02c274628c031ee65e
-  data.tar.gz: ac2fa10d43298f99c045c3136bc9c524b1f50f1c9251378907d9df69c9b8f4ee9144284faa5afa3bd6e041784f7253bdaba796232f751224901f5c28d6931baa
+  metadata.gz: 60276bbf6712ffb8bf5fe68bcbc70fa22ef7f48667ace0e1de3c220b63bf35e7d6879a8b2d5d9dba6e8df1650c081f581b27778faf7807e1a9eb73fc158f40f0
+  data.tar.gz: 9b4ac302576e3fca200f30490c536dc854002d64dd20b787d9d556d27168e8022d17bab2415564f4567524c40d0f158ae8781c80eb9b099836f711f20eef58ca

data/CHANGELOG.md

@@ -7,6 +7,29 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.1.2] - 2026-08-06
+
+### Fixed
+
+- **Token metrics read from the nested `usage` payload hash.** ask-agent now
+  enriches `chat.ask` events with a shared nested `usage` hash (tokens/cost
+  are only known after the LLM call returns, and the instrumenter
+  shallow-copies the top-level payload). The subscriber reads
+  `payload[:usage]` first and falls back to top-level
+  `input_tokens`/`output_tokens` for other emitters, so
+  `ask_llm_tokens_total` counts correctly again.
+
+## [0.1.1] - 2026-08-06
+
+### Added
+
+- `Ask::Observability.counter` / `Ask::Observability.histogram` — reload-safe
+  register-or-resolve accessors on the shared registry. App files that
+  register metrics are re-evaluated by Zeitwerk during development reloads,
+  which re-registered the same names against a registry that still held them
+  and raised `AlreadyRegisteredError` (surfacing as a 500 in the host app);
+  the accessors resolve the existing metric instead.
+
 ## [0.1.0] - 2026-08-06
 
 ### Added

data/lib/ask/observability/subscriber.rb

@@ -51,8 +51,12 @@ module Ask
       end
 
       def record_tokens(metrics, labels, payload)
-        input = payload[:input_tokens].to_i
-        output = payload[:output_tokens].to_i
+        # Chat events emitted by ask-agent enrich a shared nested +usage+
+        # hash after the call returns (tokens/cost are only known then);
+        # other emitters pass input/output tokens at the top level.
+        usage = payload[:usage] || payload
+        input = usage[:input_tokens].to_i
+        output = usage[:output_tokens].to_i
         return if input.zero? && output.zero?
 
         metrics.tokens_total.increment(by: input, labels: labels.merge(direction: 'input')) if input.positive?

data/lib/ask/observability/version.rb

@@ -2,6 +2,6 @@
 
 module Ask
   module Observability
-    VERSION = '0.1.0'
+    VERSION = '0.1.2'
   end
 end

data/lib/ask/observability.rb

@@ -65,6 +65,30 @@ module Ask
 
       attr_writer :registry
 
+      # Register-or-resolve a counter on the shared registry. Reload-safe:
+      # in development, Zeitwerk re-evaluates app files that register
+      # metrics, which would re-register the same name against a registry
+      # that still holds it and raise AlreadyRegisteredError (surfacing as
+      # a 500 in the app). Resolving the existing metric keeps registrations
+      # idempotent instead.
+      #
+      # @param name [Symbol] metric name, e.g. :voice_turns_total
+      # @return [Prometheus::Client::Counter]
+      def counter(name, **kwargs)
+        registry.get(name) || registry.counter(name, **kwargs)
+      rescue Prometheus::Client::Registry::AlreadyRegisteredError
+        registry.get(name)
+      end
+
+      # Register-or-resolve a histogram (see #counter).
+      #
+      # @return [Prometheus::Client::Histogram]
+      def histogram(name, **kwargs)
+        registry.get(name) || registry.histogram(name, **kwargs)
+      rescue Prometheus::Client::Registry::AlreadyRegisteredError
+        registry.get(name)
+      end
+
       # Subscribe to every ask-instrumentation event and start maintaining
       # Prometheus metrics. Idempotent — subsequent calls are no-ops.
       def install

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ask-observability
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.2
 platform: ruby
 authors:
 - Kaka Ruto