lex-mesh 0.4.7 → 0.4.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cd84b8f23017907b298f8f567e7bfeeda0a0fdcc4a8105dcb8b0228e57042e4b
-  data.tar.gz: f207ee8b98551427165a196528377b6b57270ef1afeda079941a13918c4c907f
+  metadata.gz: 1de324c2997f1d71286302c373a002c92bde5a389cfc9baf9bdce04d373f4fc5
+  data.tar.gz: 365de50aed497f545fb7deb881c9c3b87156acf6f8e2a5d443393004868ae278
 SHA512:
-  metadata.gz: 8d374bd0d78453bc8a0b1f3cfcde49a781fd1d43baf948ad56bbe0319b537329a19e0f397e76efc9341822d9d5fa57d8f528a710c7669f46e9d3eb407bede0ee
-  data.tar.gz: 4eeea99fe27e30c68febc9b5456ffaf2f1ed0637476362985ce5e6ec58526e9e9d967436b965135296d2997fea7b1547ea556742e81f078360cf3fa9daa651e4
+  metadata.gz: 506f44f20bec8e35b4b5fc3eb389796e4664511fa0feea50dd962ead40b10cb977a4c0ebc518d4ab20d5cdeaed2b1c2374e3a4ef5a9011121b4cb614b6540ce3
+  data.tar.gz: 188f168003f79c0ed067cc75f98888fa2ecb0dee5244ad79d76eeeb5f5f538ddbd414614be318137806ce69b2d87894b1ec0e479f46ab7d2238700f0286a7989

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

@@ -90,7 +90,7 @@ module Legion
             confidence = SOURCE_CONFIDENCE.fetch(source.to_s, 0.5)
             memory_runner.store_trace(
               type:            :semantic,
-              content_payload: { domain: domain, value: value, source: source }.to_s,
+              content_payload: pref_json_dump({ domain: domain, value: value, source: source }),
               domain_tags:     ['preference', "preference:#{domain}", "owner:#{owner_id}"],
               origin:          :direct_experience,
               confidence:      confidence
@@ -251,14 +251,31 @@ module Legion
             payload = trace[:content_payload]
             return nil unless payload.is_a?(String)
 
-            match = payload.match(/domain.*?(\w+).*?value.*?(\w+).*?source.*?(\w+)/)
-            return nil unless match
+            parsed = parse_preference_payload_json(payload)
+            parsed ||= parse_preference_payload_legacy(payload)
+            return nil unless parsed
+
+            { domain: parsed[:domain], value: parsed[:value], source: parsed[:source], confidence: trace[:confidence] }
+          rescue StandardError => _e
+            nil
+          end
 
-            { domain: match[1], value: match[2], source: match[3], confidence: trace[:confidence] }
+          def parse_preference_payload_json(payload)
+            data = pref_json_load(payload)
+            return nil unless data.is_a?(Hash) && data[:domain] && data[:value] && data[:source]
+
+            data
           rescue StandardError => _e
             nil
           end
 
+          def parse_preference_payload_legacy(payload)
+            match = payload.match(/domain.*?(\w+).*?value.*?(\w+).*?source.*?(\w+)/)
+            return nil unless match
+
+            { domain: match[1], value: match[2], source: match[3] }
+          end
+
           def compute_compatibility(profile)
             return nil unless profile.is_a?(Hash) && profile[:personality]
             return nil unless personality_available?
@@ -353,7 +370,15 @@ module Legion
             unique_channels = signals.map { |s| s[:channel] }.uniq
             return nil unless unique_channels.size >= 3
 
-            { domain: 'format', value: 'adaptive', source: 'observation', confidence: 0.55 }
+            { domain: 'format', value: 'structured', source: 'observation', confidence: 0.55 }
+          end
+
+          def pref_json_dump(data)
+            Legion::JSON.dump(data) # rubocop:disable Legion/HelperMigration/DirectJson
+          end
+
+          def pref_json_load(payload)
+            Legion::JSON.load(payload) # rubocop:disable Legion/HelperMigration/DirectJson
           end
         end
       end

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

@@ -3,7 +3,7 @@
 module Legion
   module Extensions
     module Mesh
-      VERSION = '0.4.7'
+      VERSION = '0.4.8'
     end
   end
 end

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

@@ -104,6 +104,24 @@ RSpec.describe Legion::Extensions::Mesh::Helpers::PreferenceProfile do
       expect(result[:stored]).to be false
       expect(result[:reason]).to eq(:memory_not_available)
     end
+
+    it 'serializes payload as JSON string' do
+      captured_payload = nil
+      runner_double = double('memory_runner')
+      allow(profile_mod).to receive(:memory_available?).and_return(true)
+      allow(profile_mod).to receive(:memory_runner).and_return(runner_double)
+      allow(runner_double).to receive(:store_trace) do |**args|
+        captured_payload = args[:content_payload]
+        { stored: true }
+      end
+
+      profile_mod.store_preference(owner_id: 'u1', domain: 'verbosity', value: 'concise', source: 'explicit')
+
+      parsed = Legion::JSON.load(captured_payload)
+      expect(parsed[:domain]).to eq('verbosity')
+      expect(parsed[:value]).to eq('concise')
+      expect(parsed[:source]).to eq('explicit')
+    end
   end
 
   describe '.clear_preferences' do
@@ -136,6 +154,38 @@ RSpec.describe Legion::Extensions::Mesh::Helpers::PreferenceProfile do
     end
   end
 
+  describe '.parse_preference_trace' do
+    it 'parses JSON-encoded payload and returns symbol-keyed hash' do
+      payload = Legion::JSON.dump({ domain: 'verbosity', value: 'concise', source: 'explicit' })
+      trace = { content_payload: payload, confidence: 0.9 }
+      result = profile_mod.parse_preference_trace(trace)
+      expect(result[:domain]).to eq('verbosity')
+      expect(result[:value]).to eq('concise')
+      expect(result[:source]).to eq('explicit')
+      expect(result[:confidence]).to eq(0.9)
+    end
+
+    it 'falls back to legacy regex for old .to_s hash format' do
+      legacy_payload = '{:domain=>"verbosity", :value=>"concise", :source=>"explicit"}'
+      trace = { content_payload: legacy_payload, confidence: 0.7 }
+      result = profile_mod.parse_preference_trace(trace)
+      expect(result[:domain]).to eq('verbosity')
+      expect(result[:value]).to eq('concise')
+      expect(result[:source]).to eq('explicit')
+      expect(result[:confidence]).to eq(0.7)
+    end
+
+    it 'returns nil for a non-string payload' do
+      trace = { content_payload: nil, confidence: 0.5 }
+      expect(profile_mod.parse_preference_trace(trace)).to be_nil
+    end
+
+    it 'returns nil for a payload that cannot be parsed by either method' do
+      trace = { content_payload: 'garbage data without structure', confidence: 0.5 }
+      expect(profile_mod.parse_preference_trace(trace)).to be_nil
+    end
+  end
+
   describe '.erase_partner!' do
     before do
       described_class.clear_observations
@@ -321,7 +371,7 @@ RSpec.describe Legion::Extensions::Mesh::Helpers::PreferenceProfile do
       expect(tone[:source]).to eq('observation')
     end
 
-    it 'derives :adaptive format for mixed channel usage after threshold' do
+    it 'derives :structured format for mixed channel usage after threshold' do
       channels = %i[cli api chat rest]
       20.times.with_index do |i, _|
         ch = channels[i % channels.length]
@@ -333,10 +383,27 @@ RSpec.describe Legion::Extensions::Mesh::Helpers::PreferenceProfile do
       inferred = described_class.inferred_preferences('user-mix')
       format = inferred.find { |p| p[:domain] == 'format' }
       expect(format).not_to be_nil
-      expect(format[:value]).to eq('adaptive')
+      expect(format[:value]).to eq('structured')
       expect(format[:source]).to eq('observation')
     end
 
+    it 'derive_format never returns a value outside VALID_VALUES[:format]' do
+      channels = %i[cli api chat rest grpc websocket]
+      20.times.with_index do |i, _|
+        ch = channels[i % channels.length]
+        described_class.update_from_observation(
+          owner_id: 'user-valid',
+          signals:  { content_type: :text, channel: ch, direct_address: false }
+        )
+      end
+      inferred = described_class.inferred_preferences('user-valid')
+      format_pref = inferred.find { |p| p[:domain] == 'format' }
+      if format_pref
+        valid_format_values = described_class::VALID_VALUES['format'].map(&:to_s)
+        expect(valid_format_values).to include(format_pref[:value])
+      end
+    end
+
     it 'keeps separate observation counts per owner_id' do
       3.times { described_class.update_from_observation(owner_id: 'user-a', signals: cli_signal) }
       7.times { described_class.update_from_observation(owner_id: 'user-b', signals: cli_signal) }

data/spec/spec_helper.rb

@@ -35,8 +35,8 @@ module Legion
   end
 end
 
-Legion::Settings.loader.settings[:client] ||= {}
-Legion::Settings.loader.settings[:client][:name] = 'test-agent' # rubocop:disable Legion/Llm/SettingsAccessPath
+Legion::Settings[:client] ||= {}
+Legion::Settings[:client][:name] = 'test-agent'
 
 require 'legion/extensions/mesh'
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lex-mesh
 version: !ruby/object:Gem::Version
-  version: 0.4.7
+  version: 0.4.8
 platform: ruby
 authors:
 - Esity