lex-mesh 0.3.1 → 0.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 85f99c905c7878c400b57d6d7e1868838521ab8d2f7f5fad973d6a813c09f411
-  data.tar.gz: b567bb9e7d1defaf1224aa3184fe63c80bc20458fce0d70fb936251e81247634
+  metadata.gz: aea56c9f61a9f536a407fdfee61d76717c92e9dd688365f58c33d6442ccb9d77
+  data.tar.gz: abfe8aa793907b98e051f5083c149a6f414fa0135261a603fbd50822bc0c9a93
 SHA512:
-  metadata.gz: a7b28d8162556724cae30e898607e38633ad7749d98d10a273188c67c203517fb8aff0fa7408e365c2808f1e4dfc95bca5625077187ce7c814cc6ee30e2d5053
-  data.tar.gz: e0e21868a89d6856b83c92d3034aa85d6ba9a1cda02f8afc62a8921c434211e05ad2764395c03b30f3439ca1eb89aa7ae3f616a07e15988b0af28932713f80c2
+  metadata.gz: 4fdba6f4567ca4bf713964d309ef421297c469c82bd619153297a39ad17327b784b3fac74ddf187a1e97f5c12c40ebc6983862639bb5a02f13f26af928487e6c
+  data.tar.gz: a7078556f34fd735dcb9186fa9c5d6b04ecbbe73b08db08baf2d0056be9afcf94534c34aaa1d0b03a32291e9088e947670c65d0921d78d912b2fc7ca008c5b72

data/lib/legion/extensions/mesh/helpers/registry.rb

@@ -50,10 +50,16 @@ module Legion
             agent_ids.filter_map { |id| @agents[id] }
           end
 
-          def route_message(from:, to: nil, capability: nil, pattern: :unicast, payload: {})
+          def route_message(from:, to: nil, capability: nil, pattern: :unicast, payload: {}, hops: 0)
+            if hops >= Topology::MAX_HOPS
+              return { from: from, to: to, capability: capability, pattern: pattern,
+                       payload: payload, hops: hops, at: Time.now.utc,
+                       delivered_to: [], rejected: :max_hops_exceeded }
+            end
+
             msg = {
               from: from, to: to, capability: capability,
-              pattern: pattern, payload: payload, at: Time.now.utc
+              pattern: pattern, payload: payload, hops: hops, at: Time.now.utc
             }
 
             targets = case pattern

data/lib/legion/extensions/mesh/runners/delegation.rb

@@ -9,7 +9,7 @@ module Legion
         module Delegation
           include Legion::Extensions::Helpers::Lex if defined?(Legion::Extensions::Helpers::Lex)
 
-          def delegate(from:, to:, task_context:, consent_level: :execute, parent_delegation_id: nil, **) # rubocop:disable Metrics/ParameterLists
+          def delegate(from:, to:, task_context:, consent_level: :execute, parent_delegation_id: nil, **)
             result = delegation_tracker.create(
               from:                 from,
               to:                   to,

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

@@ -32,7 +32,7 @@ module Legion
             result ? { alive: true } : { error: :not_registered }
           end
 
-          def send_message(from:, to: nil, capability: nil, pattern: :unicast, payload: {}, **) # rubocop:disable Metrics/ParameterLists
+          def send_message(from:, to: nil, capability: nil, pattern: :unicast, payload: {}, **)
             return { error: :invalid_pattern } unless Helpers::Topology.valid_pattern?(pattern)
 
             msg = mesh_registry.route_message(from: from, to: to, capability: capability,

data/lib/legion/extensions/mesh/runners/task_request.rb

@@ -15,7 +15,7 @@ module Legion
 
           DEFAULT_TIMEOUT = 30
 
-          def request_task(from:, to:, task:, payload:, timeout: DEFAULT_TIMEOUT, consent_level: :execute, **) # rubocop:disable Metrics/ParameterLists
+          def request_task(from:, to:, task:, payload:, timeout: DEFAULT_TIMEOUT, consent_level: :execute, **)
             target_id = resolve_target(to)
             return { success: false, reason: :no_agent_found, requested: to } unless target_id
 

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

@@ -3,7 +3,7 @@
 module Legion
   module Extensions
     module Mesh
-      VERSION = '0.3.1'
+      VERSION = '0.3.2'
     end
   end
 end

data/spec/legion/extensions/mesh/helpers/registry_spec.rb

@@ -239,6 +239,40 @@ RSpec.describe Legion::Extensions::Mesh::Helpers::Registry do
       end
     end
 
+    describe 'hop-count enforcement' do
+      it 'delivers normally when hops is below MAX_HOPS' do
+        result = registry.route_message(from: 'sender', to: 'a1', pattern: :unicast, hops: 0)
+        expect(result[:delivered_to]).to eq(['a1'])
+        expect(result[:rejected]).to be_nil
+      end
+
+      it 'rejects when hops equals MAX_HOPS' do
+        result = registry.route_message(from: 'sender', to: 'a1', pattern: :unicast,
+                                        hops: Legion::Extensions::Mesh::Helpers::Topology::MAX_HOPS)
+        expect(result[:delivered_to]).to eq([])
+        expect(result[:rejected]).to eq(:max_hops_exceeded)
+      end
+
+      it 'rejects when hops exceeds MAX_HOPS' do
+        result = registry.route_message(from: 'sender', to: 'a1', pattern: :unicast,
+                                        hops: Legion::Extensions::Mesh::Helpers::Topology::MAX_HOPS + 1)
+        expect(result[:delivered_to]).to eq([])
+        expect(result[:rejected]).to eq(:max_hops_exceeded)
+      end
+
+      it 'includes :max_hops_exceeded in the rejected field' do
+        result = registry.route_message(from: 'sender', pattern: :broadcast,
+                                        hops: Legion::Extensions::Mesh::Helpers::Topology::MAX_HOPS)
+        expect(result[:rejected]).to eq(:max_hops_exceeded)
+      end
+
+      it 'defaults hops to 0 for backward compatibility' do
+        result = registry.route_message(from: 'sender', to: 'a1', pattern: :unicast)
+        expect(result[:delivered_to]).to eq(['a1'])
+        expect(result[:rejected]).to be_nil
+      end
+    end
+
     describe 'message buffer cap' do
       it 'shifts old messages when buffer exceeds 1000' do
         1001.times { |i| registry.route_message(from: 'sender', to: "a#{i}", pattern: :unicast) }

data/spec/legion/extensions/mesh/transport/queues/gossip_spec.rb

@@ -0,0 +1,48 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+unless defined?(Legion::Transport::Queue)
+  module Legion
+    module Transport
+      class Queue
+        def initialize(**); end
+      end
+    end
+  end
+end
+
+unless defined?(Legion::Transport::Exchanges::Node)
+  module Legion
+    module Transport
+      module Exchanges
+        class Node # rubocop:disable Lint/EmptyClass
+        end
+      end
+    end
+  end
+end
+
+require 'legion/extensions/mesh/transport/queues/gossip'
+
+RSpec.describe Legion::Extensions::Mesh::Transport::Queues::Gossip do
+  subject(:queue) { described_class.allocate }
+
+  describe '#queue_name' do
+    it "returns 'mesh.gossip'" do
+      expect(queue.queue_name).to eq('mesh.gossip')
+    end
+  end
+
+  describe '#exchange' do
+    it 'returns Legion::Transport::Exchanges::Node' do
+      expect(queue.exchange).to eq(Legion::Transport::Exchanges::Node)
+    end
+  end
+
+  describe '#routing_key' do
+    it "returns 'mesh.gossip'" do
+      expect(queue.routing_key).to eq('mesh.gossip')
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lex-mesh
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.3.2
 platform: ruby
 authors:
 - Esity
@@ -190,6 +190,7 @@ files:
 - spec/legion/extensions/mesh/transport/messages/mesh_departure_spec.rb
 - spec/legion/extensions/mesh/transport/messages/preference_query_spec.rb
 - spec/legion/extensions/mesh/transport/messages/preference_response_spec.rb
+- spec/legion/extensions/mesh/transport/queues/gossip_spec.rb
 - spec/legion/extensions/mesh/transport/queues/preference_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/LegionIO/lex-mesh