lex-lex 0.3.4 → 0.3.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 60afc682e1c0a5fbf9ebd85ae1c9c2f16d91e7fe5bcdcf623984a9baae27a325
-  data.tar.gz: f1be1282c8be0613b8c77706975168440c546ccf5ac00fd4d6fd1d9fd055597e
+  metadata.gz: 3243ab6be48b9602778598ee63810fa357789ec4419f9899587e70977c1635c4
+  data.tar.gz: c8adbc3c6efc21cc6e307aab07403de34a582bfa275d94777b252ffe117375d5
 SHA512:
-  metadata.gz: 7d8050005b9b6eff43ee6c8d1f8be3f6a89e6dc0c4c3fdee0152abe48cf18b934be475de41e43b2f8a3b72b771e5d3c0a394566df50c3e0244acee25a3aa406e
-  data.tar.gz: 6c903ce0206c679a25edb8912704862e992cd4cdb23c7b4a626fcacafa5f95184e0d72a38cfed5673560fce1050ee2eafe7d0cebfddca5c67e3b95c1401aa5ff
+  metadata.gz: 938dae527776a91aee1ef16193f9da5517c827a199c5c1babdd7623d5f0365d9860b3c0b864659ee40694f893db80b0c7e44b111ef0acee105094d2d71f4f959
+  data.tar.gz: cc4ea57cabf435f6e3bc97eb999310534503f4b1b36d974e187d71b22c22002d6becd140d079cc8fc7c889e9b0907496ba433fe622de700f6fea99e94d8ad9c7

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,7 +27,7 @@ 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:

data/.rubocop.yml

@@ -1,65 +1,5 @@
-AllCops:
-  TargetRubyVersion: 3.4
-  NewCops: enable
-  SuggestExtensions: false
-
-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/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:
-  Enabled: false
-
-Naming/PredicateMethod:
-  Enabled: false
-
-Naming/PredicatePrefix:
-  Enabled: false
-
-Gemspec/DevelopmentDependencies:
-  Enabled: false
-
-Metrics/ParameterLists:
-  Enabled: false
-
-Metrics/BlockLength:
-  Max: 40
-  Exclude:
-    - 'spec/**/*'
+inherit_gem:
+  rubocop-legion: config/lex.yml
 
 Style/OneClassPerFile:
   Exclude:

data/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## [0.3.6] - 2026-05-27
+
+### Changed
+- Use Sequel static_cache for Extension/Runner/Function lookups in Register#save, eliminating per-message PG round-trips for existing records
+- Fall back to `.where` queries when static_cache plugin is not loaded (test environments)
+- Reload static caches after inserts/updates to keep in-memory state fresh
+
+## [0.3.5] - 2026-03-30
+
+### Changed
+- update to rubocop-legion 0.1.7, resolve all offenses
+
 ## [0.3.4] - 2026-03-29
 
 ### Fixed

data/Gemfile

@@ -3,3 +3,5 @@
 source 'https://rubygems.org'
 
 gemspec
+
+gem 'rubocop-legion'

data/lex-lex.gemspec

@@ -34,10 +34,10 @@ Gem::Specification.new do |spec|
   spec.add_dependency 'legion-settings',  '>= 1.3.14'
   spec.add_dependency 'legion-transport', '>= 1.3.9'
 
-  spec.add_development_dependency 'rake'
   spec.add_development_dependency 'rspec', '~> 3.13'
   spec.add_development_dependency 'rspec_junit_formatter'
-  spec.add_development_dependency 'rubocop', '~> 1.75'
+  spec.add_development_dependency 'rubocop'
+  spec.add_development_dependency 'rubocop-legion'
   spec.add_development_dependency 'rubocop-rspec'
   spec.add_development_dependency 'simplecov'
 end

data/lib/legion/extensions/lex/actors/agent_watcher.rb

@@ -4,7 +4,7 @@ module Legion
   module Extensions
     module Lex
       module Actor
-        class AgentWatcher < Legion::Extensions::Actors::Every
+        class AgentWatcher < Legion::Extensions::Actors::Every # rubocop:disable Legion/Extension/EveryActorRequiresTime
           def runner_class
             self.class
           end

data/lib/legion/extensions/lex/actors/sync.rb

@@ -27,11 +27,11 @@ module Legion
             false
           end
 
-          def enabled?
+          def enabled? # rubocop:disable Legion/Extension/ActorEnabledSideEffects
             return false unless defined?(Legion::Settings)
 
             Legion::Settings[:data][:connected] == true
-          rescue StandardError
+          rescue StandardError => _e
             false
           end
 

data/lib/legion/extensions/lex/runners/extension.rb

@@ -8,13 +8,14 @@ module Legion
           include Legion::Extensions::Helpers::Lex if defined?(Legion::Extensions::Helpers::Lex)
 
           def create(name:, namespace:, active: true, **opts)
-            existing = Legion::Data::Model::Extension.where(name: name).first
-            return update(extension_id: existing.values[:id], namespace: namespace, active: active, **opts) if existing
+            existing = find_cached_extension(name)
+            return update(extension_id: existing.values[:id], namespace: namespace, active: active, **opts) if existing # rubocop:disable Legion/Extension/RunnerReturnHash
 
             insert = { name: name, namespace: namespace, active: active }
             insert[:exchange] = opts.fetch(:exchange, name)
             insert[:uri] = opts.fetch(:uri, name)
             id = Legion::Data::Model::Extension.insert(insert)
+            reload_static_caches
             { success: true, extension_id: id }
           end
 
@@ -33,6 +34,7 @@ module Legion
             return { success: true, changed: false, extension_id: extension_id } if changes.empty?
 
             extension.update(changes)
+            reload_static_caches
             { success: true, changed: true, updates: changes, extension_id: extension_id }
           end
 
@@ -52,8 +54,26 @@ module Legion
             return { success: false, reason: 'not found' } if record.nil?
 
             record.delete
+            reload_static_caches
             { success: true, extension_id: extension_id }
           end
+
+          private
+
+          def find_cached_extension(name)
+            model = Legion::Data::Model::Extension
+            if model.respond_to?(:cache) && model.respond_to?(:all)
+              model.all.find { |e| e.values[:name] == name }
+            else
+              model.where(name: name).first
+            end
+          end
+
+          def reload_static_caches
+            [Legion::Data::Model::Extension, Legion::Data::Model::Runner, Legion::Data::Model::Function].each do |m|
+              m.load_cache if m.respond_to?(:load_cache)
+            end
+          end
         end
       end
     end

data/lib/legion/extensions/lex/runners/function.rb

@@ -7,14 +7,17 @@ module Legion
         module Function
           include Legion::Extensions::Helpers::Lex if defined?(Legion::Extensions::Helpers::Lex)
 
+          RESERVED_ARG_NAMES = %w[opts options].freeze
+
           def create(runner_id:, name:, active: true, **opts)
-            existing = Legion::Data::Model::Function.where(name: name.to_s, runner_id: runner_id).first
-            return update(function_id: existing.values[:id], name: name, active: active, **opts) if existing
+            existing = find_cached_function(name, runner_id)
+            return update(function_id: existing.values[:id], name: name, active: active, **opts) if existing # rubocop:disable Legion/Extension/RunnerReturnHash
 
             insert = { runner_id: runner_id, name: name.to_s, active: active }
-            insert[:args] = Legion::JSON.dump(opts[:formatted_args]) if opts.key?(:formatted_args)
+            insert[:args] = json_dump(opts[:formatted_args]) if opts.key?(:formatted_args)
 
             id = Legion::Data::Model::Function.insert(insert)
+            reload_static_caches
             { success: true, function_id: id }
           end
 
@@ -26,13 +29,14 @@ module Legion
             changes[:active] = opts[:active] if opts.key?(:active) && function.values[:active] != opts[:active]
 
             if opts.key?(:formatted_args)
-              args = Legion::JSON.dump(opts[:formatted_args])
+              args = json_dump(opts[:formatted_args])
               changes[:args] = args unless args == function.values[:args]
             end
 
             return { success: true, changed: false, function_id: function_id } if changes.empty?
 
             function.update(changes)
+            reload_static_caches
             { success: true, changed: true, updates: changes, function_id: function_id }
           end
 
@@ -48,16 +52,34 @@ module Legion
             return { success: false, reason: 'not found' } if record.nil?
 
             record.delete
+            reload_static_caches
             { success: true, function_id: function_id }
           end
 
           def build_args(raw_args:, **_opts)
             args = {}
             raw_args.each do |arg|
-              args[arg[1]] = arg[0] unless %w[opts options].include?(arg[1].to_s)
+              args[arg[1]] = arg[0] unless RESERVED_ARG_NAMES.include?(arg[1].to_s)
             end
             { success: true, formatted_args: args }
           end
+
+          private
+
+          def find_cached_function(name, runner_id)
+            model = Legion::Data::Model::Function
+            if model.respond_to?(:cache) && model.respond_to?(:all)
+              model.all.find { |f| f.values[:name] == name.to_s && f.values[:runner_id] == runner_id }
+            else
+              model.where(name: name.to_s, runner_id: runner_id).first
+            end
+          end
+
+          def reload_static_caches
+            [Legion::Data::Model::Extension, Legion::Data::Model::Runner, Legion::Data::Model::Function].each do |m|
+              m.load_cache if m.respond_to?(:load_cache)
+            end
+          end
         end
       end
     end

data/lib/legion/extensions/lex/runners/register.rb

@@ -8,6 +8,9 @@ module Legion
           include Legion::Extensions::Helpers::Lex if defined?(Legion::Extensions::Helpers::Lex)
 
           def save(opts:, **_options)
+            log.unknown "save(opts: #{opts.class}, #{opts.length}, #{opts&.keys}"
+            log.unknown "full: #{opts}" if opts.empty?
+
             return { success: false, reason: 'no opts provided' } if opts.nil? || opts.empty?
 
             extension_id = nil

data/lib/legion/extensions/lex/runners/runner.rb

@@ -4,12 +4,12 @@ module Legion
   module Extensions
     module Lex
       module Runners
-        module Runner
+        module Runner # rubocop:disable Legion/Extension/RunnerPluralModule
           include Legion::Extensions::Helpers::Lex if defined?(Legion::Extensions::Helpers::Lex)
 
           def create(extension_id:, name:, active: true, **opts)
-            existing = Legion::Data::Model::Runner.where(name: name.to_s, extension_id: extension_id).first
-            return update(runner_id: existing.values[:id], name: name, active: active, **opts) if existing
+            existing = find_cached_runner(name, extension_id)
+            return update(runner_id: existing.values[:id], name: name, active: active, **opts) if existing # rubocop:disable Legion/Extension/RunnerReturnHash
 
             insert = {
               extension_id: extension_id,
@@ -20,6 +20,7 @@ module Legion
             insert[:queue] = opts.fetch(:queue, name.to_s)
             insert[:uri] = opts.fetch(:uri, name.to_s)
             id = Legion::Data::Model::Runner.insert(insert)
+            reload_static_caches
             { success: true, runner_id: id }
           end
 
@@ -38,6 +39,7 @@ module Legion
             return { success: true, changed: false, runner_id: runner_id } if changes.empty?
 
             runner.update(changes)
+            reload_static_caches
             { success: true, changed: true, updates: changes, runner_id: runner_id }
           end
 
@@ -53,8 +55,26 @@ module Legion
             return { success: false, reason: 'not found' } if record.nil?
 
             record.delete
+            reload_static_caches
             { success: true, runner_id: runner_id }
           end
+
+          private
+
+          def find_cached_runner(name, extension_id)
+            model = Legion::Data::Model::Runner
+            if model.respond_to?(:cache) && model.respond_to?(:all)
+              model.all.find { |r| r.values[:name] == name.to_s && r.values[:extension_id] == extension_id }
+            else
+              model.where(name: name.to_s, extension_id: extension_id).first
+            end
+          end
+
+          def reload_static_caches
+            [Legion::Data::Model::Extension, Legion::Data::Model::Runner, Legion::Data::Model::Function].each do |m|
+              m.load_cache if m.respond_to?(:load_cache)
+            end
+          end
         end
       end
     end

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

@@ -3,7 +3,7 @@
 module Legion
   module Extensions
     module Lex
-      VERSION = '0.3.4'
+      VERSION = '0.3.6'
     end
   end
 end

data/lib/legion/extensions/lex.rb

@@ -6,14 +6,14 @@ require_relative 'lex/runners/runner'
 require_relative 'lex/runners/function'
 require_relative 'lex/runners/register'
 require_relative 'lex/runners/sync'
-require_relative 'lex/actors/agent_watcher' if defined?(Legion::Extensions::Actors::Every)
+require_relative 'lex/actors/agent_watcher'
 
 module Legion
   module Extensions
     module Lex
-      extend Legion::Extensions::Core if Legion::Extensions.const_defined?(:Core)
+      extend Legion::Extensions::Core if Legion::Extensions.const_defined?(:Core, false)
 
-      def self.data_required?
+      def self.data_required? # rubocop:disable Legion/Extension/DataRequiredWithoutMigrations
         true
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lex-lex
 version: !ruby/object:Gem::Version
-  version: 0.3.4
+  version: 0.3.6
 platform: ruby
 authors:
 - Esity
@@ -108,35 +108,35 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 1.3.9
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.13'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.13'
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: rspec_junit_formatter
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '3.13'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '3.13'
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: rspec_junit_formatter
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -150,19 +150,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: rubocop-legion
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.75'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.75'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement