lex-agentic-memory 0.1.11 → 0.1.13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 44a4999991835bdd28a7189717f6a53f38928644a86c7f653517331687659901
-  data.tar.gz: b3aeb50fddc4b82580430829ec924549690b8dd6171be74ff9213db40b135888
+  metadata.gz: '0814efaa2ca66f0121dd6d783ec6383b09542f126628695cf2c375c056b5c2b1'
+  data.tar.gz: d6df0516525b8ff2883f9f4074eb488ac6df00c74ee24994ab7c629a9a449a0e
 SHA512:
-  metadata.gz: 32930b8b8092925f0793eef8ae8b1a6764d50cbc9692bb2952c46fb969137b41609eed96ecb455fdc7d0518373ebd912075eff8ac47c4deb8e97e1fc9176962b
-  data.tar.gz: a142cbf70545c4f80717ddda3d81abab26a1cc2cf5d8e40584625a416629d1551991d716fc6a9592fff194b523b0ebed2963be24fa414039cf11bdb3c934e60a
+  metadata.gz: ad170b27af1c27a90430d7ce82d89554b5a4cd5709504535d84bfc3ae52046495a790acf29cc9665739c2486cc736c36cbd118cf588f4f9418d3e90f307182e3
+  data.tar.gz: 8ee023a4360585eca2bd215dfc8601e4c9d2d96935a985c1bc8b0200e7953dbcd2f49d26032c35d23f8b5af69fa097a0d2c9c189291faa189e5892e22785405c

data/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## [0.1.13] - 2026-03-26
+
+### Fixed
+- `PostgresStore#serialize_trace` omitted `agent_id` column, causing `PG::NotNullViolation` on every insert against PostgreSQL (migration 022 declares `agent_id null: false`). Constructor now accepts `agent_id:` with fallback to `Legion::Settings.dig(:agent, :id)` or `'default'`
+- `Trace.create_store` factory now resolves and passes `agent_id` to PostgresStore
+- Spec schema for PostgresStore now includes `agent_id` column matching production migration
+
+## [0.1.12] - 2026-03-26
+
+### Fixed
+- `PostgresStore#store` used SQLite-specific `insert_conflict(:replace)` which fails on PostgreSQL with `TypeError: no implicit conversion of Symbol into Integer`. Replaced with proper `insert_conflict(target: :trace_id, update: ...)` syntax that generates correct `ON CONFLICT` SQL
+
 ## [0.1.11] - 2026-03-25
 
 ### Added

data/lib/legion/extensions/agentic/memory/trace/helpers/postgres_store.rb

@@ -15,8 +15,9 @@ module Legion
               TRACES_TABLE = :memory_traces
               ASSOCIATIONS_TABLE = :memory_associations
 
-              def initialize(tenant_id: nil)
+              def initialize(tenant_id: nil, agent_id: nil)
                 @tenant_id = tenant_id
+                @agent_id  = agent_id || resolve_agent_id
               end
 
               # Store (upsert) a trace by trace_id.
@@ -25,10 +26,11 @@ module Legion
                 return nil unless db_ready?
 
                 row = serialize_trace(trace)
-                begin
-                  db[TRACES_TABLE].insert_conflict(:replace).insert(row)
-                rescue Sequel::UniqueConstraintViolation
-                  db[TRACES_TABLE].where(trace_id: trace[:trace_id]).update(row.except(:trace_id))
+                ds  = db[TRACES_TABLE]
+                if db.adapter_scheme == :mysql2
+                  ds.insert_conflict(update: row.except(:trace_id)).insert(row)
+                else
+                  ds.insert_conflict(target: :trace_id, update: row.except(:trace_id)).insert(row)
                 end
                 HotTier.cache_trace(trace, tenant_id: @tenant_id) if HotTier.available?
                 trace[:trace_id]
@@ -261,6 +263,12 @@ module Legion
                 Legion::Data.connection
               end
 
+              def resolve_agent_id
+                Legion::Settings.dig(:agent, :id) || 'default'
+              rescue StandardError
+                'default'
+              end
+
               # Dataset for memory_traces scoped by tenant_id (if set).
               def traces_ds
                 ds = db[TRACES_TABLE]
@@ -276,6 +284,7 @@ module Legion
 
                 {
                   trace_id:                trace[:trace_id],
+                  agent_id:                @agent_id,
                   tenant_id:               @tenant_id,
                   trace_type:              trace[:trace_type].to_s,
                   content:                 payload.is_a?(Hash) ? Legion::JSON.dump(payload) : payload.to_s,

data/lib/legion/extensions/agentic/memory/trace.rb

@@ -34,7 +34,7 @@ module Legion
             def create_store
               if postgres_available?
                 Legion::Logging.debug '[memory] Using shared PostgresStore (write-through)'
-                Helpers::PostgresStore.new(tenant_id: resolve_tenant_id)
+                Helpers::PostgresStore.new(tenant_id: resolve_tenant_id, agent_id: resolve_agent_id)
               elsif defined?(Legion::Cache) && Legion::Cache.respond_to?(:connected?) && Legion::Cache.connected?
                 Legion::Logging.debug '[memory] Using shared CacheStore (memcached)'
                 Helpers::CacheStore.new
@@ -60,6 +60,12 @@ module Legion
             rescue StandardError
               nil
             end
+
+            def resolve_agent_id
+              Legion::Settings.dig(:agent, :id) || 'default'
+            rescue StandardError
+              'default'
+            end
           end
         end
       end

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

@@ -4,7 +4,7 @@ module Legion
   module Extensions
     module Agentic
       module Memory
-        VERSION = '0.1.11'
+        VERSION = '0.1.13'
       end
     end
   end

data/spec/legion/extensions/agentic/memory/hologram/helpers/hologram_spec.rb

@@ -99,7 +99,9 @@ RSpec.describe Legion::Extensions::Agentic::Memory::Hologram::Helpers::Hologram
 
   describe '#reconstruct' do
     context 'with sufficient fragments' do
-      let(:fragments) { hologram.fragment!(4) }
+      # Force completeness above RECONSTRUCTION_THRESHOLD (0.3) so the context
+      # is deterministic — fragment!(4) uses rand and can produce all-insufficient sets.
+      let(:fragments) { hologram.fragment!(4).each { |f| f.completeness = 1.0 } }
 
       it 'returns success: true' do
         expect(hologram.reconstruct(fragments)[:success]).to be true

data/spec/legion/extensions/agentic/memory/trace/helpers/postgres_store_spec.rb

@@ -12,6 +12,7 @@ RSpec.describe Legion::Extensions::Agentic::Memory::Trace::Helpers::PostgresStor
     d.create_table(:memory_traces) do
       primary_key :id
       String  :trace_id, size: 36, null: false, unique: true
+      String  :agent_id, size: 64, null: false, default: 'test-agent'
       String  :tenant_id, size: 64
       String  :trace_type, null: false
       String  :content, text: true, null: false
@@ -110,6 +111,40 @@ RSpec.describe Legion::Extensions::Agentic::Memory::Trace::Helpers::PostgresStor
     end
   end
 
+  # --- store insert_conflict syntax ---
+
+  describe '#store insert_conflict syntax' do
+    it 'calls insert_conflict with target: :trace_id (not :replace symbol)' do
+      ds = double('dataset')
+      allow(db).to receive(:[]).with(:memory_traces).and_return(ds)
+      allow(ds).to receive(:where).and_return(ds)
+      allow(ds).to receive(:insert_conflict).and_return(ds)
+      allow(ds).to receive(:insert)
+
+      store.store(semantic_trace)
+
+      expect(ds).to have_received(:insert_conflict)
+        .with(hash_including(target: :trace_id))
+    end
+  end
+
+  # --- store agent_id population ---
+
+  describe '#store agent_id population' do
+    it 'writes agent_id to the database row' do
+      store.store(semantic_trace)
+      row = db[:memory_traces].where(trace_id: semantic_trace[:trace_id]).first
+      expect(row[:agent_id]).not_to be_nil
+    end
+
+    it 'uses the resolved agent_id from settings' do
+      custom_store = described_class.new(tenant_id: tenant_id, agent_id: 'my-agent')
+      custom_store.store(semantic_trace)
+      row = db[:memory_traces].where(trace_id: semantic_trace[:trace_id]).first
+      expect(row[:agent_id]).to eq('my-agent')
+    end
+  end
+
   # --- store + retrieve ---
 
   describe '#store and #retrieve' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lex-agentic-memory
 version: !ruby/object:Gem::Version
-  version: 0.1.11
+  version: 0.1.13
 platform: ruby
 authors:
 - Esity