lex-telemetry 0.1.6 → 0.1.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f468345bff8197f58b2ef7736b8c03e2ede0e6521505bbd4c8b73e6f9c606a89
-  data.tar.gz: e4e9783b090af3812310e385e51c5a5950b57868faa7c87863d77a8ad7a6b1e2
+  metadata.gz: f37c3484695b8612aec589a24b49f98f8c1be3b0d2892a6e023b27447acd84e1
+  data.tar.gz: be1205311b7ee81cf281c3abe03b3f74844dd839bd41a6c9a934a2a7a3ef4b28
 SHA512:
-  metadata.gz: 0fac05554a0955707202e43c131dcfa1d54339e0750ff3f6640e5805cd6e92a60a3a71672f87b4415bd5d16de3e8ac4de3b8f005e754a01a570ea6c2a39eabc2
-  data.tar.gz: 4f5e12b043042b264e257f8cf99747aa020376083bbd9c727429e4cc702788f2f4b1a69a9d13962febc5fbc4cd22be00f893050609125b598588e1947e71951f
+  metadata.gz: b36a9fdd1e12e9d7fbe4bcde51bdd26ac6eb7fd1aea412f0522b8f31a8739b6e428c869193ea3b8776a7f62df5b7a51c81f05ba4d32397ff406cf5a8788d3fab
+  data.tar.gz: 90bcf6ca1386b77a474f0c3becb12327e850380dc6e67513ad1f6871d9e6f058f10c4c3d5b5c7a689bbf70b2b2848288172b5fab0555a946a97f5a025a783870

data/.github/workflows/ci.yml

@@ -10,8 +10,8 @@ jobs:
   ci:
     uses: LegionIO/.github/.github/workflows/ci.yml@main
 
-  lint:
-    uses: LegionIO/.github/.github/workflows/lint-patterns.yml@main
+  excluded-files:
+    uses: LegionIO/.github/.github/workflows/excluded-files.yml@main
 
   security:
     uses: LegionIO/.github/.github/workflows/security-scan.yml@main
@@ -27,8 +27,8 @@ jobs:
     uses: LegionIO/.github/.github/workflows/stale.yml@main
 
   release:
-    needs: [ci, lint]
+    needs: [ci, excluded-files]
     if: github.event_name == 'push' && github.ref == 'refs/heads/main'
     uses: LegionIO/.github/.github/workflows/release.yml@main
     secrets:
-      rubygems-api-key: ${{ secrets.RUBYGEMS_API_KEY }}
+      rubygems-api-key: ${{ secrets.RUBYGEMS_API_KEY }}

data/.rubocop.yml

@@ -1,67 +1,18 @@
-AllCops:
-  TargetRubyVersion: 3.4
-  NewCops: enable
-  SuggestExtensions: false
+inherit_gem:
+  rubocop-legion: config/lex.yml
 
-Layout/LineLength:
-  Max: 160
-
-Layout/SpaceAroundEqualsInParameterDefault:
-  EnforcedStyle: space
-
-Layout/HashAlignment:
-  EnforcedHashRocketStyle: table
-  EnforcedColonStyle: table
-
-Metrics/MethodLength:
-  Max: 50
-
-Metrics/ClassLength:
-  Max: 1500
-
-Metrics/ModuleLength:
-  Max: 1500
-
-Metrics/BlockLength:
-  Max: 40
-  Exclude:
-    - 'spec/**/*'
-
-Metrics/AbcSize:
-  Max: 60
-
-Metrics/CyclomaticComplexity:
-  Max: 15
-
-Metrics/PerceivedComplexity:
-  Max: 17
-
-Style/Documentation:
-  Enabled: false
-
-Style/SymbolArray:
-  Enabled: true
-
-Style/FrozenStringLiteralComment:
-  Enabled: true
-  EnforcedStyle: always
-
-Naming/FileName:
+# Actors define `def time` as instance method — cop looks for DSL call, false positive
+Legion/Extension/EveryActorRequiresTime:
   Enabled: false
 
-Naming/PredicateMethod:
+# Runner module is designed for include into actors; not standalone
+Legion/Extension/RunnerIncludeHelpers:
   Enabled: false
 
-Gemspec/DevelopmentDependencies:
+# Class instance variables use Concurrent::Hash/AtomicFixnum — thread-safe by design
+ThreadSafety/ClassInstanceVariable:
   Enabled: false
 
+# TelemetryEvent#build legitimately requires many named parameters for type safety
 Metrics/ParameterLists:
   Enabled: false
-
-Lint/EmptyClass:
-  Exclude:
-    - 'spec/**/*'
-
-Style/MultilineIfModifier:
-  Exclude:
-    - 'spec/**/*'

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## [0.1.7] - 2026-03-30
+
+### Changed
+- update to rubocop-legion 0.1.7, resolve all offenses
+
 ## [0.1.6] - 2026-03-26
 
 ### Changed

data/Gemfile

@@ -4,3 +4,4 @@ source 'https://rubygems.org'
 gemspec
 
 gem 'concurrent-ruby'
+gem 'rubocop-legion', '~> 0.1'

data/lib/legion/extensions/telemetry/parsers/claude_code.rb

@@ -18,11 +18,11 @@ module Legion
             first_line = File.open(path, &:readline)
             data = ::JSON.parse(first_line)
             data.is_a?(Hash) && (data.key?('sessionId') || data.key?('parentUuid'))
-          rescue StandardError
+          rescue StandardError => _e
             false
           end
 
-          def parse(path, offset: 0, &block)
+          def parse(path, offset: 0, &)
             pending_tools = {}
             session_id = nil
 
@@ -36,13 +36,13 @@ module Legion
 
                 if first_line && offset.zero?
                   session_id = data['sessionId'] || data['uuid'] || SecureRandom.uuid
-                  block.call(Helpers::TelemetryEvent.build(
-                               event_type: :session_start,
-                               session_id: session_id,
-                               source:     source_name,
-                               timestamp:  timestamp,
-                               metadata:   { version: data['version'] }
-                             ))
+                  yield(Helpers::TelemetryEvent.build(
+                    event_type: :session_start,
+                    session_id: session_id,
+                    source:     source_name,
+                    timestamp:  timestamp,
+                    metadata:   { version: data['version'] }
+                  ))
                   first_line = false
                   next
                 end
@@ -51,9 +51,9 @@ module Legion
                 msg = data['message']
                 next unless msg.is_a?(Hash)
 
-                process_assistant(msg, session_id, timestamp, pending_tools, &block) if msg['role'] == 'assistant'
-                process_tool_result(msg, session_id, timestamp, pending_tools, &block) if msg['role'] == 'user'
-              rescue ::JSON::ParserError
+                process_assistant(msg, session_id, timestamp, pending_tools, &) if msg['role'] == 'assistant'
+                process_tool_result(msg, session_id, timestamp, pending_tools, &) if msg['role'] == 'user'
+              rescue ::JSON::ParserError => _e
                 next
               end
 
@@ -63,7 +63,7 @@ module Legion
 
           private
 
-          def process_assistant(msg, session_id, timestamp, pending_tools, &block)
+          def process_assistant(msg, session_id, timestamp, pending_tools)
             content = msg['content']
             usage = msg['usage']
 
@@ -85,16 +85,16 @@ module Legion
             has_tool_use = content.is_a?(Array) && content.any? { |b| b['type'] == 'tool_use' }
             return if has_tool_use
 
-            block.call(Helpers::TelemetryEvent.build(
-                         event_type: :llm_request,
-                         session_id: session_id,
-                         source:     source_name,
-                         timestamp:  timestamp,
-                         tokens:     extract_tokens(usage)
-                       ))
+            yield(Helpers::TelemetryEvent.build(
+              event_type: :llm_request,
+              session_id: session_id,
+              source:     source_name,
+              timestamp:  timestamp,
+              tokens:     extract_tokens(usage)
+            ))
           end
 
-          def process_tool_result(msg, session_id, timestamp, pending_tools, &block)
+          def process_tool_result(msg, session_id, timestamp, pending_tools)
             content = msg['content']
             return unless content.is_a?(Array)
 
@@ -106,16 +106,16 @@ module Legion
 
               duration_ms = ((timestamp - pending[:timestamp]) * 1000).to_i if timestamp && pending[:timestamp]
 
-              block.call(Helpers::TelemetryEvent.build(
-                           event_type:  :tool_call,
-                           session_id:  session_id,
-                           source:      source_name,
-                           timestamp:   pending[:timestamp],
-                           tool_name:   pending[:tool_name],
-                           tool_input:  pending[:tool_input],
-                           duration_ms: duration_ms,
-                           tokens:      pending[:tokens]
-                         ))
+              yield(Helpers::TelemetryEvent.build(
+                event_type:  :tool_call,
+                session_id:  session_id,
+                source:      source_name,
+                timestamp:   pending[:timestamp],
+                tool_name:   pending[:tool_name],
+                tool_input:  pending[:tool_input],
+                duration_ms: duration_ms,
+                tokens:      pending[:tokens]
+              ))
             end
           end
 
@@ -134,7 +134,7 @@ module Legion
             return nil unless str
 
             Time.parse(str)
-          rescue ArgumentError
+          rescue ArgumentError => _e
             nil
           end
 

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

@@ -94,7 +94,7 @@ module Legion
           def publish_pending(**_opts)
             if privacy_mode?
               count = event_store.pending.length
-              Legion::Logging.info "lex-telemetry: privacy mode active, suppressing #{count} pending events (local logging only)" if defined?(Legion::Logging)
+              Legion::Logging.info "lex-telemetry: privacy mode active, suppressing #{count} pending events (local logging only)" # rubocop:disable Legion/HelperMigration/DirectLogging
               event_store.pending.clear
               return { success: true, published: 0, suppressed: true, reason: 'enterprise_data_privacy is enabled' }
             end
@@ -109,7 +109,7 @@ module Legion
                 Transport::Messages::TelemetryMessage.new.publish(event, routing_key: routing_key)
                 published += 1
               end
-            rescue StandardError
+            rescue StandardError => _e
               event_store.pending.push(event)
             end
 
@@ -149,7 +149,7 @@ module Legion
                 high_water_mark.set(path: path, offset: new_offset)
                 files_processed += 1
                 events_ingested += count
-              rescue StandardError
+              rescue StandardError => _e
                 next
               end
             end
@@ -174,7 +174,7 @@ module Legion
             affinity = if defined?(Legion::Settings)
                          begin
                            Legion::Settings.dig(:region, :default_affinity)
-                         rescue StandardError
+                         rescue StandardError => _e
                            'prefer_local'
                          end
                        else

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

@@ -3,7 +3,7 @@
 module Legion
   module Extensions
     module Telemetry
-      VERSION = '0.1.6'
+      VERSION = '0.1.7'
     end
   end
 end

data/lib/legion/extensions/telemetry.rb

@@ -13,7 +13,7 @@ require 'legion/extensions/telemetry/runners/telemetry'
 module Legion
   module Extensions
     module Telemetry
-      extend Legion::Extensions::Core if Legion::Extensions.const_defined? :Core
+      extend Legion::Extensions::Core if Legion::Extensions.const_defined? :Core, false
 
       def self.remote_invocable?
         false

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lex-telemetry
 version: !ruby/object:Gem::Version
-  version: 0.1.6
+  version: 0.1.7
 platform: ruby
 authors:
 - Esity