lex-metering 0.1.2 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6e70bf639989419388bb625a4f105d7ad6376b534b35f839d8623427bfc3fe43
-  data.tar.gz: b6c1250c1e19d2d80ee88772416aa4b3f05f4123b774698b6b4ae6ef5b6f02a4
+  metadata.gz: db002082af15a2d97671a7b9ed75897f21ec03d16e360962517fc848527c9ec6
+  data.tar.gz: cb642644b254256b1fef4a17933f9276a3a972ebee52a37c80e4bf88786e1b57
 SHA512:
-  metadata.gz: d65a2b2829f638b80ce726b19c616f81a4dc4c436482bea8bee837d489053070b929f59fe240e09dff4478be52cdce3dee43aa127cc21dea0b49833b920ec246
-  data.tar.gz: 614785e8c7af6964c0312c5f18ff362c0fd1f7e29656e6db76cd94c2029774108050671363adb8b17012f334ce71e86750f6547eae298490875fde11637c092c
+  metadata.gz: 7e107e84a2d199254a12daad43880e9f3b593b3076b0e42964a7fe0ef07410c4a75923761da4128cb0942c4b45db1b791b4923e4e2263484c3b61621d54cf979
+  data.tar.gz: 60c7c096ea83b68c01fef84244449764ac3ff357465a9aa2a5e74f71f72b94f838ac2bd8805dd969c9cbbe613bdf3a04145e62a47c8d6923134b96399828a767

data/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog
 
+## [0.1.4] - 2026-03-18
+
+### Changed
+- `record` no longer writes directly to database; returns hash only for RabbitMQ publishing
+- `data_required?` changed to `false` — extension loads without legion-data
+
+## [0.1.3] - 2026-03-18
+
+### Fixed
+- `worker_costs` period filtering now uses cross-DB `Sequel.lit('recorded_at >= ?', cutoff)` with Ruby time arithmetic instead of PostgreSQL-only `CURRENT_TIMESTAMP - INTERVAL` syntax
+
 ## [0.1.2] - 2026-03-17
 
 ### Changed

data/CLAUDE.md

@@ -11,7 +11,7 @@ Captures LLM token usage metrics per task for cost attribution and intelligent r
 ## Gem Info
 
 - **Gem name**: `lex-metering`
-- **Version**: `0.1.0`
+- **Version**: `0.1.4`
 - **Module**: `Legion::Extensions::Metering`
 - **Ruby**: `>= 3.4`
 - **License**: MIT
@@ -62,14 +62,14 @@ lib/legion/extensions/metering/
 
 ## Integration Points
 
-- **legion-data**: `data_required? true` — will not load if DB unavailable. Accesses `metering_records` as a raw Sequel dataset (no Sequel::Model subclass).
+- **legion-data**: `data_required? false` — loads without DB. `record` returns hash only (for RabbitMQ publishing). Query methods (`worker_costs`, `team_costs`, `routing_stats`, `cleanup_old_records`) still access `metering_records` as a raw Sequel dataset when `Legion::Data` is available.
 - **LegionIO MCP**: `legion.routing_stats` MCP tool calls `routing_stats` runner
 - **REST API**: `GET /api/tasks/:id` includes a `:metering` block when lex-metering data exists for the task
 - **Digital Workers**: `legion worker costs` CLI command delegates to `worker_costs` runner
 
 ## Development Notes
 
-- Extension has `data_required? true` (both at module level and instance level) — will skip loading if `legion-data` is not connected
+- Extension has `data_required? false` — loads without `legion-data`; `record` builds hash only (no DB insert), query methods still require `Legion::Data`
 - No explicit actors — gets auto-generated subscription actors from the framework
 - `routing_stats` uses `select_append { avg(latency_ms).as(avg_latency) }` — Sequel virtual row syntax
-- Time interval filtering uses `Sequel.lit("CURRENT_TIMESTAMP - INTERVAL '...'")` which is PostgreSQL syntax; SQLite uses different interval syntax (known limitation)
+- Time interval filtering uses `Sequel.lit('recorded_at >= ?', cutoff)` with Ruby `Time` arithmetic for cross-database compatibility (PostgreSQL, SQLite, MySQL)

data/lib/legion/extensions/metering/runners/metering.rb

@@ -5,6 +5,8 @@ module Legion
     module Metering
       module Runners
         module Metering
+          PERIOD_DAYS = { 'daily' => 1, 'weekly' => 7, 'monthly' => 30 }.freeze
+
           def record(worker_id: nil, task_id: nil, provider: nil, model_id: nil,
                      input_tokens: 0, output_tokens: 0, thinking_tokens: 0,
                      input_context_bytes: 0, latency_ms: 0, routing_reason: nil,
@@ -27,7 +29,6 @@ module Legion
               recorded_at:         Time.now.utc
             }
 
-            Legion::Data.connection[:metering_records].insert(record) if defined?(Legion::Data) && Legion::Data.connection
             Legion::Logging.debug "[metering] recorded: provider=#{provider} model=#{model_id} " \
                                   "tokens=#{record[:total_tokens]} latency=#{latency_ms}ms wall_clock=#{wall_clock_ms}ms"
             record
@@ -35,15 +36,7 @@ module Legion
 
           def worker_costs(worker_id:, period: 'daily', **)
             ds = Legion::Data.connection[:metering_records].where(worker_id: worker_id)
-
-            case period
-            when 'daily'
-              ds = ds.where { recorded_at >= Sequel.lit("CURRENT_TIMESTAMP - INTERVAL '1 day'") }
-            when 'weekly'
-              ds = ds.where { recorded_at >= Sequel.lit("CURRENT_TIMESTAMP - INTERVAL '7 days'") }
-            when 'monthly'
-              ds = ds.where { recorded_at >= Sequel.lit("CURRENT_TIMESTAMP - INTERVAL '30 days'") }
-            end
+            ds = apply_period_filter(ds, period)
 
             {
               worker_id:       worker_id,
@@ -93,6 +86,16 @@ module Legion
             Legion::Logging.info "[metering] cleanup: purged=#{count} retention_days=#{retention_days} cutoff=#{cutoff}"
             { purged: count, retention_days: retention_days, cutoff: cutoff }
           end
+
+          private
+
+          def apply_period_filter(dataset, period)
+            days = PERIOD_DAYS[period]
+            return dataset unless days
+
+            cutoff = Time.now.utc - (days * 86_400)
+            dataset.where(::Sequel.lit('recorded_at >= ?', cutoff))
+          end
         end
       end
     end

data/lib/legion/extensions/metering/version.rb

@@ -3,7 +3,7 @@
 module Legion
   module Extensions
     module Metering
-      VERSION = '0.1.2'
+      VERSION = '0.1.4'
     end
   end
 end

data/lib/legion/extensions/metering.rb

@@ -8,11 +8,11 @@ module Legion
       extend Legion::Extensions::Core if Legion::Extensions.const_defined? :Core
 
       def self.data_required?
-        true
+        false
       end
 
       def data_required?
-        true
+        false
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lex-metering
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.4
 platform: ruby
 authors:
 - Esity