lex-agentic-executive 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a52da834db3ad024167228f957b019bfc33bb0a1cc86cc1cb58feabe4aa1bec4
-  data.tar.gz: 16f3a79dee77a608932cb3b837ee55aac9bc4cfbd596beb42f29582b530c1b54
+  metadata.gz: 0ac4e5f2b4c752913d15a4d25240de021a5e7821b2b50be4dd474b1a51ab482f
+  data.tar.gz: 9c407b92da789b2f6a11c340b921d772f32fdc5617b968ed5a3dad01c680ce53
 SHA512:
-  metadata.gz: e44439bae29cbf8793b0f69ece036bb097051f0434b4d33f4e373f661c5203b51beabb58d9b9f78f17ab9b0fc0041c0565dfbd7d6b11804707e39aa0ef78784a
-  data.tar.gz: 596a167041a980a4151babe00df9b876a0ebe647929fcea48e0cebe8b9b6704621a5499fcebde0494a8a9c3fff3000b22b2e29ce2da658f481a3ad1350400cd3
+  metadata.gz: 558348baa035852ac306d07d992a3fb8c65263e87bad2d359dfa3c6ab611a25c5c6477ab02cd5d6776f81cd5b5e648f6bcd5583a4984705ca52f587e41304f4d
+  data.tar.gz: 9c0eb7766904360b3c679c3357fb94006f24841f3eb392e9dcf7690e6282a4d65f67150b4cc204f0c67d7bafd67f1299ce5dcb2eccaed0e5ed50697d8b634fa8

data/CHANGELOG.md

@@ -1,6 +1,13 @@
 # Changelog
 
+## [0.2.2] - 2026-05-08
+### Fixed
+- `Decomposer#parse_sub_goals` now accesses JSON sub-goal hashes using string keys (with symbol fallback), preventing LLM-decomposed goals from getting empty content/default domain/0.5 priority.
+- `ExecutiveController#common_ef_level` formula corrected — blends minimum effective capacity (weighted 0.6) with average (weighted 0.4) instead of computing a no-op `avg * 0.6 + avg * 0.4`.
+- `FeedbackListener` tracks event handler references, enabling `stop_listening` and `restart_listening` to prevent duplicate handler accumulation on repeated start calls.
+
 ## [0.2.1] - 2026-05-07
+
 ### Fixed
 - Goal decomposer LLM strategy now parses native `Legion::LLM.chat` hash responses without requiring a legacy chat session.
 - Added regression coverage for cognition-domain dispatch to the MindGrowth analyzer.

data/lib/legion/extensions/agentic/executive/executive_function/helpers/executive_controller.rb

@@ -72,8 +72,9 @@ module Legion
 
               def common_ef_level
                 values = @components.values.map(&:effective_capacity)
+                min    = values.min
                 avg    = values.sum / values.size.to_f
-                (avg * (1.0 - COMMON_EF_WEIGHT)) + (avg * COMMON_EF_WEIGHT)
+                (avg * (1.0 - COMMON_EF_WEIGHT)) + (min * COMMON_EF_WEIGHT)
               end
 
               def can_inhibit?

data/lib/legion/extensions/agentic/executive/goal_management/helpers/decomposer.rb

@@ -92,9 +92,9 @@ module Legion
 
                 data.map do |sg|
                   {
-                    content:  sg[:content].to_s,
-                    domain:   (sg[:domain] || domain).to_sym,
-                    priority: (sg[:priority] || 0.5).to_f.clamp(0.0, 1.0)
+                    content:  sg.fetch('content', sg.fetch(:content, '')).to_s,
+                    domain:   sg.fetch('domain', sg.fetch(:domain, domain)).to_sym,
+                    priority: sg.fetch('priority', sg.fetch(:priority, 0.5)).to_f.clamp(0.0, 1.0)
                   }
                 end
               rescue StandardError => e

data/lib/legion/extensions/agentic/executive/goal_management/helpers/feedback_listener.rb

@@ -10,6 +10,7 @@ module Legion
               def initialize(engine:)
                 @engine    = engine
                 @listening = false
+                @handlers  = []
               end
 
               def handle_task_event(task_id:, status:, result: nil)
@@ -26,7 +27,7 @@ module Legion
                 return if @listening
                 return unless defined?(Legion::Events)
 
-                Legion::Events.on('task.completed') do |event|
+                handler_completed = Legion::Events.on('task.completed') do |event|
                   handle_task_event(
                     task_id: event[:task_id],
                     status:  event[:status] || 'task.completed',
@@ -34,7 +35,7 @@ module Legion
                   )
                 end
 
-                Legion::Events.on('task.failed') do |event|
+                handler_failed = Legion::Events.on('task.failed') do |event|
                   handle_task_event(
                     task_id: event[:task_id],
                     status:  event[:status] || 'task.failed',
@@ -42,9 +43,28 @@ module Legion
                   )
                 end
 
+                @handlers << { event: 'task.completed', block: handler_completed }
+                @handlers << { event: 'task.failed', block: handler_failed }
                 @listening = true
               end
 
+              def stop_listening
+                return unless @listening
+
+                @handlers.each do |entry|
+                  Legion::Events.off(entry[:event], entry[:block])
+                rescue StandardError => e
+                  log.debug "[feedback_listener] stop_listening cleanup failed: #{e.message}"
+                end
+                @handlers.clear
+                @listening = false
+              end
+
+              def restart_listening
+                stop_listening
+                start_listening
+              end
+
               def listening?
                 @listening
               end

data/lib/legion/extensions/agentic/executive/version.rb

@@ -4,7 +4,7 @@ module Legion
   module Extensions
     module Agentic
       module Executive
-        VERSION = '0.2.1'
+        VERSION = '0.2.2'
       end
     end
   end

data/spec/legion/extensions/agentic/executive/goal_management/helpers/feedback_listener_spec.rb

@@ -101,4 +101,71 @@ RSpec.describe Legion::Extensions::Agentic::Executive::GoalManagement::Helpers::
       expect(listener.listening?).to be false
     end
   end
+
+  describe '#stop_listening' do
+    it 'removes registered handlers and resets listening state' do
+      events_spy = Class.new do
+        attr_reader :registered, :removed
+
+        def initialize
+          @registered = []
+          @removed = []
+        end
+
+        def on(event, &block)
+          @registered << { event: event, block: block }
+          block
+        end
+
+        def off(event, block)
+          @removed << { event: event, block: block }
+        end
+      end.new
+      stub_const('Legion::Events', events_spy)
+      listener.start_listening
+      expect(listener.listening?).to be true
+
+      listener.stop_listening
+      expect(listener.listening?).to be false
+      expect(events_spy.removed.size).to eq(2)
+      removed_events = events_spy.removed.map { |r| r[:event] }
+      expect(removed_events).to include('task.completed')
+      expect(removed_events).to include('task.failed')
+    end
+
+    it 'does nothing when not listening' do
+      expect { listener.stop_listening }.not_to raise_error
+      expect(listener.listening?).to be false
+    end
+  end
+
+  describe '#restart_listening' do
+    it 'stops and starts listening' do
+      events_spy = Class.new do
+        attr_reader :registered, :removed
+
+        def initialize
+          @registered = []
+          @removed = []
+        end
+
+        def on(event, &block)
+          @registered << { event: event, block: block }
+          block
+        end
+
+        def off(event, block)
+          @removed << { event: event, block: block }
+        end
+      end.new
+      stub_const('Legion::Events', events_spy)
+      listener.start_listening
+
+      listener.restart_listening
+      expect(listener.listening?).to be true
+      # Original handlers removed, new ones registered
+      expect(events_spy.removed.size).to eq(2)
+      expect(events_spy.registered.size).to eq(4) # 2 original + 2 new
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lex-agentic-executive
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - Esity