lex-swarm 0.2.2 → 0.2.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 44c1897aa216153d964378805ac63718afc2782bfeee0b9e69fb3c6b84e7c750
-  data.tar.gz: 225dacb3e327dd976379dc39760d77e72f28350b4c3881547be8cfd735716736
+  metadata.gz: 872cfd908e1911c140ec4fa764f8d992c468e0fc258aceb709551254f2723111
+  data.tar.gz: f55c01c18b5a46c1c704f626c9b23b8d15312d86467507739cf7fda934386295
 SHA512:
-  metadata.gz: 3e0331bfb72bc0f67669fd6a3b84a752387adc733e14c7eab07913d8a7438f51c3d727515246c2552faaef2c68b1faa8a035b4c99d50ae283548eba404e1bd27
-  data.tar.gz: '089cd79bcc1b9a1642fe55723373d012220619eea2455e808dd5b19983e5c291218c2367bac81f92c1e3a4a74514d07948932c751eb4e1164c68a7cd1f872ac9'
+  metadata.gz: d874292dcff40204e4af91353e0b2b99e02a87282219f949d2000aadce61a47267e44553e6d33a3badc392f9e5fd697c2f90089080ce43d4ea6ed1823d828014
+  data.tar.gz: 0eb4bad49126b2864f2258340e01518796d8664585c7502ab66f9d6c97c75c896302a0bb9ecfe45701bcb72f45ee55b59c7495143ca385eaad62d0906539060a

data/Gemfile

@@ -6,6 +6,7 @@ gemspec
 
 gem 'rspec', '~> 3.13'
 gem 'rubocop', '~> 1.75', require: false
+gem 'rubocop-legion', '~> 0.1'
 gem 'rubocop-rspec', require: false
 
 gem 'legion-gaia'

data/lib/legion/extensions/swarm/actors/orphan_sweeper.rb

@@ -4,10 +4,11 @@ module Legion
   module Extensions
     module Swarm
       module Actor
-        class OrphanSweeper < Legion::Extensions::Actors::Every
+        class OrphanSweeper < Legion::Extensions::Actors::Every # rubocop:disable Legion/Extension/EveryActorRequiresTime
           def time = 300
           def run_now? = false
           def use_runner? = false
+          def runner_class = self.class
           def check_subtask? = false
           def generate_task? = false
 

data/lib/legion/extensions/swarm/actors/stale_check.rb

@@ -6,7 +6,7 @@ module Legion
   module Extensions
     module Swarm
       module Actor
-        class StaleCheck < Legion::Extensions::Actors::Every
+        class StaleCheck < Legion::Extensions::Actors::Every # rubocop:disable Legion/Extension/EveryActorRequiresTime
           def runner_class
             Legion::Extensions::Swarm::Runners::Swarm
           end

data/lib/legion/extensions/swarm/helpers/sub_agent.rb

@@ -38,19 +38,19 @@ module Legion
 
           def max_depth
             Legion::Settings.dig(:swarm, :max_depth) || 3
-          rescue StandardError
+          rescue StandardError => _e
             3
           end
 
           def max_concurrent
             Legion::Settings.dig(:swarm, :max_concurrent) || 20
-          rescue StandardError
+          rescue StandardError => _e
             20
           end
 
           def max_per_parent
             Legion::Settings.dig(:swarm, :max_per_parent) || 10
-          rescue StandardError
+          rescue StandardError => _e
             10
           end
 
@@ -61,7 +61,7 @@ module Legion
                         .exclude(parent_id: nil)
                         .exclude(status: %w[complete failed])
                         .count
-          rescue StandardError
+          rescue StandardError => _e
             0
           end
 
@@ -72,7 +72,7 @@ module Legion
                         .where(parent_id: parent_task_id)
                         .exclude(status: %w[complete failed])
                         .count
-          rescue StandardError
+          rescue StandardError => _e
             0
           end
         end

data/lib/legion/extensions/swarm/{actors → helpers}/workspace_sync.rb

@@ -1,20 +1,20 @@
 # frozen_string_literal: true
 
 require 'time'
-require_relative '../helpers/workspace'
+require_relative 'workspace'
 
 module Legion
   module Extensions
     module Swarm
-      module Actors
+      module Helpers
         class WorkspaceSync
-          include Legion::Extensions::Helpers::Lex if Legion::Extensions.const_defined?(:Helpers) &&
-                                                      Legion::Extensions::Helpers.const_defined?(:Lex)
+          include Legion::Extensions::Helpers::Lex if Legion::Extensions.const_defined?(:Helpers, false) &&
+                                                      Legion::Extensions::Helpers.const_defined?(:Lex, false)
 
           ROUTING_PREFIX = 'swarm.workspace'
 
-          def publish_change(charter_id:, key:, operation:, value: nil, author: nil, version: nil, **) # rubocop:disable Metrics/ParameterLists
-            return { success: true, skipped: :no_transport } unless defined?(Legion::Transport)
+          def publish_change(charter_id:, key:, operation:, value: nil, author: nil, version: nil, **)
+            return { success: true, skipped: :no_transport } unless Legion.const_defined?(:Transport, false)
 
             routing_key = "#{ROUTING_PREFIX}.#{charter_id}"
             payload     = { charter_id: charter_id, key: key, value: value,
@@ -29,7 +29,7 @@ module Legion
             { success: true, skipped: :publish_error, message: e.message }
           end
 
-          def apply_incoming(charter_id:, key:, operation:, value: nil, author: nil, version: nil, timestamp: nil, **) # rubocop:disable Metrics/ParameterLists
+          def apply_incoming(charter_id:, key:, operation:, value: nil, author: nil, version: nil, timestamp: nil, **)
             op = operation.to_s
             case op
             when 'put'

data/lib/legion/extensions/swarm/runners/spawn_child.rb

@@ -4,7 +4,7 @@ module Legion
   module Extensions
     module Swarm
       module Runners
-        module SpawnChild
+        module SpawnChild # rubocop:disable Legion/Extension/RunnerIncludeHelpers
           include Helpers::SubAgent if defined?(Helpers::SubAgent)
 
           def spawn_child(runner:, function:, payload:, parent_task_id:, **)

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

@@ -5,10 +5,10 @@ module Legion
     module Swarm
       module Runners
         module Swarm
-          include Legion::Extensions::Helpers::Lex if Legion::Extensions.const_defined?(:Helpers) &&
-                                                      Legion::Extensions::Helpers.const_defined?(:Lex)
+          include Legion::Extensions::Helpers::Lex if Legion::Extensions.const_defined?(:Helpers, false) &&
+                                                      Legion::Extensions::Helpers.const_defined?(:Lex, false)
 
-          def create_swarm(name:, objective:, roles: [], max_agents: 10, timeout: 3600, **) # rubocop:disable Metrics/ParameterLists
+          def create_swarm(name:, objective:, roles: [], max_agents: 10, timeout: 3600, **)
             charter = Helpers::Charter.new_charter(name: name, objective: objective,
                                                    roles: roles, max_agents: max_agents, timeout: timeout)
             id = swarm_store.create(charter)

data/lib/legion/extensions/swarm/runners/workspace.rb

@@ -7,8 +7,8 @@ module Legion
     module Swarm
       module Runners
         module Workspace
-          include Legion::Extensions::Helpers::Lex if Legion::Extensions.const_defined?(:Helpers) &&
-                                                      Legion::Extensions::Helpers.const_defined?(:Lex)
+          include Legion::Extensions::Helpers::Lex if Legion::Extensions.const_defined?(:Helpers, false) &&
+                                                      Legion::Extensions::Helpers.const_defined?(:Lex, false)
 
           def workspace_put(charter_id:, key:, value:, author:, **)
             entry = workspace.put(charter_id, key: key, value: value, author: author)

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

@@ -3,7 +3,7 @@
 module Legion
   module Extensions
     module Swarm
-      VERSION = '0.2.2'
+      VERSION = '0.2.4'
     end
   end
 end

data/lib/legion/extensions/swarm.rb

@@ -12,7 +12,7 @@ require 'legion/extensions/swarm/runners/workspace'
 module Legion
   module Extensions
     module Swarm
-      extend Legion::Extensions::Core if Legion::Extensions.const_defined? :Core
+      extend Legion::Extensions::Core if Legion::Extensions.const_defined? :Core, false
     end
   end
 end

data/spec/legion/extensions/swarm/actors/workspace_sync_spec.rb

@@ -1,16 +1,16 @@
 # frozen_string_literal: true
 
 require 'spec_helper'
-require_relative '../../../../../lib/legion/extensions/swarm/actors/workspace_sync'
+require_relative '../../../../../lib/legion/extensions/swarm/helpers/workspace_sync'
 
-RSpec.describe Legion::Extensions::Swarm::Actors::WorkspaceSync do
-  subject(:actor) { described_class.allocate }
+RSpec.describe Legion::Extensions::Swarm::Helpers::WorkspaceSync do
+  subject(:sync) { described_class.allocate }
 
   describe '#publish_change' do
     it 'returns skipped when transport publish fails' do
       allow(Legion::Transport).to receive(:publish).and_raise(StandardError, 'not connected')
-      result = actor.publish_change(charter_id: 'c1', key: 'k', value: 'v',
-                                    author: 'a', version: 1, operation: :put)
+      result = sync.publish_change(charter_id: 'c1', key: 'k', value: 'v',
+                                   author: 'a', version: 1, operation: :put)
       expect(result[:success]).to be true
       expect(result[:skipped]).to eq(:publish_error)
     end
@@ -19,10 +19,10 @@ RSpec.describe Legion::Extensions::Swarm::Actors::WorkspaceSync do
   describe '#apply_incoming' do
     let(:workspace) { Legion::Extensions::Swarm::Helpers::Workspace.new }
 
-    before { actor.instance_variable_set(:@workspace, workspace) }
+    before { sync.instance_variable_set(:@workspace, workspace) }
 
     it 'applies a remote put operation' do
-      result = actor.apply_incoming(
+      result = sync.apply_incoming(
         charter_id: 'c1', key: 'shared', value: 'remote-data',
         author: 'remote-agent', version: 1, timestamp: Time.now.utc.to_s, operation: 'put'
       )
@@ -33,7 +33,7 @@ RSpec.describe Legion::Extensions::Swarm::Actors::WorkspaceSync do
 
     it 'applies a remote delete operation' do
       workspace.put('c1', key: 'doomed', value: 'bye', author: 'local')
-      result = actor.apply_incoming(
+      result = sync.apply_incoming(
         charter_id: 'c1', key: 'doomed', operation: 'delete'
       )
       expect(result[:success]).to be true
@@ -41,7 +41,7 @@ RSpec.describe Legion::Extensions::Swarm::Actors::WorkspaceSync do
     end
 
     it 'rejects unknown operations' do
-      result = actor.apply_incoming(charter_id: 'c1', key: 'x', operation: 'explode')
+      result = sync.apply_incoming(charter_id: 'c1', key: 'x', operation: 'explode')
       expect(result[:success]).to be false
       expect(result[:reason]).to eq(:unknown_operation)
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lex-swarm
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: 0.2.4
 platform: ruby
 authors:
 - Esity
@@ -133,12 +133,12 @@ files:
 - lib/legion/extensions/swarm.rb
 - lib/legion/extensions/swarm/actors/orphan_sweeper.rb
 - lib/legion/extensions/swarm/actors/stale_check.rb
-- lib/legion/extensions/swarm/actors/workspace_sync.rb
 - lib/legion/extensions/swarm/client.rb
 - lib/legion/extensions/swarm/helpers/charter.rb
 - lib/legion/extensions/swarm/helpers/sub_agent.rb
 - lib/legion/extensions/swarm/helpers/swarm_store.rb
 - lib/legion/extensions/swarm/helpers/workspace.rb
+- lib/legion/extensions/swarm/helpers/workspace_sync.rb
 - lib/legion/extensions/swarm/runners/spawn_child.rb
 - lib/legion/extensions/swarm/runners/swarm.rb
 - lib/legion/extensions/swarm/runners/workspace.rb