lex-agentic-memory 0.1.35 → 0.1.36

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bb940d62021d4a86ed5de171d6b275c9287a9ca727efadd3c60c3a87ca0f921e
-  data.tar.gz: b02842d4a5939e436d7ae4f72cca295839061f5fa4cdbf9b47736f37e462e76e
+  metadata.gz: f135722a81c15e36374e61e6348db1f421db9b4fc5257bc2915d8526a9b7a145
+  data.tar.gz: 86dfb884036c9d7d8617a3a785b6b35c6cc0db413b60a9866aad04ad3e899d59
 SHA512:
-  metadata.gz: e28e286f59952562c8c00ad37cb06f84914f2b93af864cb95a1d9904c218dc72d28bb09a4df049c418f5a66df752a199171c0e553e684c234ae888ab73be24a8
-  data.tar.gz: 12febef87343cde5c4ca8f372c62a6f7d5b053a47ad5362710ee54224be60ceb899f7f0f32eeb65cc9474ebe5b47535731e2f6a3e83a89f3d315c6780774ea77
+  metadata.gz: bd080984039d516a8980ed88723a34f6f8738439f55a921e572cbba852f73af3c48010cffc9fec5b134305f2e7d0f7d774d032bf8a41e88003b9f7cef8cba34a
+  data.tar.gz: 5e2fbe8775082a3e64533acce43937923b0e5f6c97cc7bcb623fc4a42f9bc21be513f824fef9ea80f13afb392da5e30648b68fd6e9a9fbadd1e1eeed985259b5

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## [0.1.36] - 2026-05-17
+### Fixed
+- `PostgresStore#db_ready?` now checks `Legion::Data.can_write?(:memory_traces)` before attempting writes, preventing `PG::InsufficientPrivilege` errors when connected with a read-only role.
+- `Trace.postgres_available?` also checks INSERT privilege so `create_store` correctly falls back to a local store instead of selecting PostgresStore and failing at runtime.
+
 ## [0.1.34] - 2026-05-08
 ### Fixed
 - Deferred GAIA heartbeat maintenance no longer initializes the shared trace store just to return a deferred summary, avoiding idle local trace table scans.

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

@@ -264,12 +264,14 @@ module Legion
               # No-op — this store is write-through; nothing to flush.
               def flush; end
 
-              # Returns true when both required tables exist in the connected DB.
+              # Returns true when both required tables exist and the user has INSERT privilege on both.
               def db_ready?
                 defined?(Legion::Data) &&
                   Legion::Data.respond_to?(:connection) &&
                   Legion::Data.connection&.table_exists?(TRACES_TABLE) &&
-                  Legion::Data.connection.table_exists?(ASSOCIATIONS_TABLE)
+                  Legion::Data.connection.table_exists?(ASSOCIATIONS_TABLE) &&
+                  Legion::Data.can_write?(TRACES_TABLE) &&
+                  Legion::Data.can_write?(ASSOCIATIONS_TABLE)
               rescue StandardError => e
                 log.error "[trace_persistence] db_ready?: #{e.message}"
                 false

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

@@ -79,7 +79,9 @@ module Legion
                 Legion::Data.connection &&
                 %i[postgres mysql2].include?(Legion::Data.connection.adapter_scheme) &&
                 Legion::Data.connection.table_exists?(:memory_traces) &&
-                Legion::Data.connection.table_exists?(:memory_associations)
+                Legion::Data.connection.table_exists?(:memory_associations) &&
+                Legion::Data.can_write?(:memory_traces) &&
+                Legion::Data.can_write?(:memory_associations)
             rescue StandardError => _e
               false
             end

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

@@ -4,7 +4,7 @@ module Legion
   module Extensions
     module Agentic
       module Memory
-        VERSION = '0.1.35'
+        VERSION = '0.1.36'
       end
     end
   end

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

@@ -65,6 +65,9 @@ RSpec.describe Legion::Extensions::Agentic::Memory::Hologram::Helpers::HologramE
     let(:fragments) { engine.fragment_hologram(hologram_id: hologram.id, count: 4) }
 
     it 'returns success: true when sufficient fragments are used' do
+      # Enhance all fragments to guarantee they exceed the reconstruction threshold
+      # regardless of the random completeness assigned during fragmentation.
+      fragments.each { |f| f.enhance!(0.5) }
       ids = fragments.select(&:sufficient?).map(&:id)
       result = engine.reconstruct_from_fragments(hologram_id: hologram.id, fragment_ids: ids)
       expect(result[:success]).to be true

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

@@ -81,8 +81,11 @@ RSpec.describe Legion::Extensions::Agentic::Memory::Trace::Helpers::PostgresStor
   end
 
   before do
+    allow(Legion::Data).to receive(:respond_to?).and_call_original
     allow(Legion::Data).to receive(:respond_to?).with(:connection).and_return(true)
     allow(Legion::Data).to receive(:connection).and_return(db)
+    allow(Legion::Data).to receive(:can_write?).with(:memory_traces).and_return(true)
+    allow(Legion::Data).to receive(:can_write?).with(:memory_associations).and_return(true)
     # SQLite adapter_scheme is :sqlite — postgres_available? checks for :postgres/:mysql2,
     # but PostgresStore itself only calls db_ready? which just needs the tables to exist.
     allow(db).to receive(:adapter_scheme).and_return(:sqlite)
@@ -109,6 +112,27 @@ RSpec.describe Legion::Extensions::Agentic::Memory::Trace::Helpers::PostgresStor
       allow(Legion::Data).to receive(:connection).and_raise(StandardError, 'no db')
       expect(store.db_ready?).to be false
     end
+
+    it 'returns false when user lacks INSERT privilege on memory_traces' do
+      allow(Legion::Data).to receive(:can_write?).with(:memory_traces).and_return(false)
+      expect(store.db_ready?).to be false
+    end
+
+    it 'returns true when user has INSERT privilege on memory_traces' do
+      allow(Legion::Data).to receive(:can_write?).with(:memory_traces).and_return(true)
+      expect(store.db_ready?).to be true
+    end
+
+    it 'returns false when user lacks INSERT privilege on memory_associations' do
+      allow(Legion::Data).to receive(:can_write?).with(:memory_associations).and_return(false)
+      expect(store.db_ready?).to be false
+    end
+
+    it 'returns true when user has INSERT privilege on both tables' do
+      allow(Legion::Data).to receive(:can_write?).with(:memory_traces).and_return(true)
+      allow(Legion::Data).to receive(:can_write?).with(:memory_associations).and_return(true)
+      expect(store.db_ready?).to be true
+    end
   end
 
   # --- store insert_conflict syntax ---

data/spec/legion/extensions/agentic/memory/trace/memory_spec.rb

@@ -34,5 +34,43 @@ RSpec.describe Legion::Extensions::Agentic::Memory::Trace do
 
       expect(described_class.shared_store).to eq(:shared_store)
     end
+
+    it 'falls back from Postgres when user lacks INSERT privilege on memory_traces' do
+      allow(described_class).to receive(:local_store_available?).and_return(false)
+      allow(described_class).to receive(:configured_trace_store).and_return(nil)
+      allow(described_class).to receive(:resolve_agent_id).and_return('agent-1')
+
+      conn = double('connection', adapter_scheme: :postgres)
+      allow(conn).to receive(:table_exists?).and_return(true)
+      allow(Legion::Data).to receive(:respond_to?).and_call_original
+      allow(Legion::Data).to receive(:respond_to?).with(:connection).and_return(true)
+      allow(Legion::Data).to receive(:connection).and_return(conn)
+      allow(Legion::Data).to receive(:can_write?).with(:memory_traces).and_return(false)
+      allow(Legion::Data).to receive(:can_write?).with(:memory_associations).and_return(true)
+
+      allow(Legion::Cache).to receive(:respond_to?).with(:connected?).and_return(false)
+      allow(described_class::Helpers::Store).to receive(:new).with(partition_id: 'agent-1').and_return(:fallback_store)
+
+      expect(described_class.shared_store).to eq(:fallback_store)
+    end
+
+    it 'falls back from Postgres when user lacks INSERT privilege on memory_associations' do
+      allow(described_class).to receive(:local_store_available?).and_return(false)
+      allow(described_class).to receive(:configured_trace_store).and_return(nil)
+      allow(described_class).to receive(:resolve_agent_id).and_return('agent-1')
+
+      conn = double('connection', adapter_scheme: :postgres)
+      allow(conn).to receive(:table_exists?).and_return(true)
+      allow(Legion::Data).to receive(:respond_to?).and_call_original
+      allow(Legion::Data).to receive(:respond_to?).with(:connection).and_return(true)
+      allow(Legion::Data).to receive(:connection).and_return(conn)
+      allow(Legion::Data).to receive(:can_write?).with(:memory_traces).and_return(true)
+      allow(Legion::Data).to receive(:can_write?).with(:memory_associations).and_return(false)
+
+      allow(Legion::Cache).to receive(:respond_to?).with(:connected?).and_return(false)
+      allow(described_class::Helpers::Store).to receive(:new).with(partition_id: 'agent-1').and_return(:fallback_store)
+
+      expect(described_class.shared_store).to eq(:fallback_store)
+    end
   end
 end

metadata

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