documentrix 0.3.1 → 0.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c3ab97426ab9fbd4ec832a422a18d574f4c03b11f22391cbc6eded16b9ba0609
-  data.tar.gz: 991215fb26b4165a4b3562075d9742e3756218e2a24f3f32a550e232f59fb92f
+  metadata.gz: 332c67275b90bcc797cdd8e8df75a751bcfa7a52c759c9bcbe4a55970e905401
+  data.tar.gz: c546f89c61613b11d18c2cd724cadae8c303a988f925ad1624c5121f511e88fd
 SHA512:
-  metadata.gz: d67e453e27428bcd8364349e988556af844caab5faa8a8edb3dcfba9650554b8dbfc66ad322a6f9f59d5ac64350d7006bf899a16b6c83f92909793301b6954d5
-  data.tar.gz: 49706d4984b27ee7b3d0f8ef8a9efec4a8e74d8bef78d78f15ed5c76f5bacc0faf20c36cef739aa78c1f68e5733696d16999bbd14228ae7be8e6cf9c9baf0a75
+  metadata.gz: c80a4c253b4fe367fae1dbb7e6aea50e274a495a0026a7b676e1e962b80c33806fe89976338c12931f11a5059e384fdfddcfe4665f68bde406b118edd8c04d15
+  data.tar.gz: '09292fd5d6aed6c939c0244467c743d515e68e5e253d7f443ef96a9c295a61689f1be5b9aec53e402287b6b6c7d305757eb26f79f5ade0161f7e82d534a7b3bf'

data/CHANGES.md

@@ -1,5 +1,28 @@
 # Changes
 
+## 2026-05-20 v0.3.2
+
+### Performance Improvements
+
+- Optimized collections lookup by memoizing
+  `Documentrix::Documents#collections` using `@collections_cache`.
+- Added `Documentrix::Documents#invalidate_collections_cache!` to reset the
+  memoized list.
+- Integrated `invalidate_collections_cache!` into `add`, `delete`, `clear`,
+  `source_remove`, and `rename_collection` to maintain cache consistency.
+
+### Database & Cache
+
+- Standardized SQL keyword casing to uppercase for data types (e.g., `FLOAT`,
+  `TEXT`, `INTEGER`, `JSON`) within `SQLiteCache`.
+
+### Documentation & Testing
+
+- Refined documentation and return type descriptions for
+  `Documentrix::Documents#find` and `Documentrix::Documents#prefix`.
+- Updated `spec/documents_spec.rb` to ensure `invalidate_collections_cache!` is
+  correctly triggered during mutations.
+
 ## 2026-05-18 v0.3.1
 
 - Fixed scoping bugs in `clear_by_source` and `source_exist?` by implementing a

data/documentrix.gemspec

@@ -1,9 +1,9 @@
 # -*- encoding: utf-8 -*-
-# stub: documentrix 0.3.1 ruby lib
+# stub: documentrix 0.3.2 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "documentrix".freeze
-  s.version = "0.3.1".freeze
+  s.version = "0.3.2".freeze
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib".freeze]

data/lib/documentrix/documents/cache/sqlite_cache.rb

@@ -443,18 +443,18 @@ class Documentrix::Documents::Cache::SQLiteCache
     @database.enable_load_extension(false)
     execute %{
       CREATE VIRTUAL TABLE IF NOT EXISTS embeddings USING vec0(
-        embedding float[#@embedding_length]
+        embedding FLOAT[#@embedding_length]
       )
     }
     execute %{
       CREATE TABLE IF NOT EXISTS records (
-        key          text NOT NULL PRIMARY KEY ON CONFLICT REPLACE,
-        text         text NOT NULL DEFAULT '',
-        embedding_id integer,
-        norm         float NOT NULL DEFAULT 0.0,
-        source       text,
-        digest       text,
-        tags         json NOT NULL DEFAULT [],
+        key          TEXT NOT NULL PRIMARY KEY ON CONFLICT REPLACE,
+        text         TEXT NOT NULL DEFAULT '',
+        embedding_id INTEGER,
+        norm         FLOAT NOT NULL DEFAULT 0.0,
+        source       TEXT,
+        digest       TEXT,
+        tags         JSON NOT NULL DEFAULT [],
         FOREIGN KEY(embedding_id) REFERENCES embeddings(id) ON DELETE CASCADE
       )
     }

data/lib/documentrix/documents.rb

@@ -162,7 +162,7 @@ class Documentrix::Documents
       infobar.progress by: batch.size
     end
     infobar.newline
-    self
+    invalidate_collections_cache!
   end
   alias << add
 
@@ -201,7 +201,9 @@ class Documentrix::Documents
   # @return [ FalseClass, TrueClass ] true if the text was removed, false
   #         otherwise.
   def delete(text)
-    @cache.delete(key(text))
+    res = @cache.delete(key(text))
+    invalidate_collections_cache! if res
+    res
   end
 
   # The size method returns the number of texts stored in the cache of this
@@ -220,7 +222,7 @@ class Documentrix::Documents
   # @return [ Documentrix::Documents ] self
   def clear(tags: nil)
     @cache.clear(tags:)
-    self
+    invalidate_collections_cache!
   end
 
   # Normalizes the source identifier to a canonical form.
@@ -318,7 +320,7 @@ class Documentrix::Documents
   def source_remove(source, digest: nil)
     source = normalize_source(source)
     @cache.clear_by_source(source, digest:, operator: '!=')
-    self
+    invalidate_collections_cache!
   end
 
   # The find method searches for strings within the cache by computing their
@@ -359,7 +361,7 @@ class Documentrix::Documents
   # @param text_count [Integer] the maximum number of records to return
   # @param opts [Hash] additional options passed to #find, such as:
   #   * :tags [Array<String>] filter results by tags
-  #   * :prompt [String] a prompt to use for the search
+  #   * :prompt [String] use for the search
   #   * :min_similarity [Numeric] minimum similarity score
   #
   # @example
@@ -379,7 +381,9 @@ class Documentrix::Documents
   #
   # @return [Array] An array of unique collection names
   def collections
-    ([ default_collection ] + @cache.collections('%s-' % class_prefix)).uniq
+    @collections_cache ||= (
+      [ default_collection ] + @cache.collections('%s-' % class_prefix)
+    ).uniq
   end
 
   # Rename the current collection, moving all keys from the old prefix to a new
@@ -395,6 +399,7 @@ class Documentrix::Documents
     new_prefix = '%s-%s-' % [ class_prefix, new_collection ]
     @cache.move_prefix(prefix, new_prefix)
     self.collection = new_collection
+    invalidate_collections_cache!
   end
 
   # The tags method returns an array of unique tags from the cache.
@@ -406,6 +411,18 @@ class Documentrix::Documents
 
   private
 
+  # Resets the memoized list of collections.
+  #
+  # This is called whenever a mutation occurs that could change the set of
+  # existing collections, ensuring that the #collections method returns a
+  # fresh, accurate list on the next call.
+  #
+  # @return [ Documentrix::Documents ] self
+  def invalidate_collections_cache!
+    @collections_cache = nil
+    self
+  end
+
   # The connect_cache method initializes and returns an instance of the
   # specified cache class.
   #
@@ -482,7 +499,7 @@ class Documentrix::Documents
   # The prefix method returns a string that is used as the prefix for keys in
   # the cache of the currently configured collection.
   #
-  # @return [ String ] The prefix string
+  # @return [ String ] the prefix string
   def prefix
     '%s-%s-' % [ class_prefix, @collection ]
   end

data/lib/documentrix/version.rb

@@ -1,6 +1,6 @@
 module Documentrix
   # Documentrix version
-  VERSION         = '0.3.1'
+  VERSION         = '0.3.2'
   VERSION_ARRAY   = VERSION.split('.').map(&:to_i) # :nodoc:
   VERSION_MAJOR   = VERSION_ARRAY[0] # :nodoc:
   VERSION_MINOR   = VERSION_ARRAY[1] # :nodoc:

data/spec/documents_spec.rb

@@ -23,6 +23,8 @@ describe Documentrix::Documents do
     expect(ollama).to receive(:embed).
       with(model:, input: %w[ foo bar ], options: nil).
       and_return(double(embeddings: [ [ 0.1 ], [ 0.2 ] ]))
+    expect(documents).to receive(:invalidate_collections_cache!).
+      and_call_original
     expect(documents.add(%w[ foo bar ])).to eq documents
     expect(documents.exist?('foo')).to eq true
     expect(documents.exist?('bar')).to eq true
@@ -33,6 +35,8 @@ describe Documentrix::Documents do
     expect(ollama).to receive(:embed).
       with(model:, input: %w[ foo ], options: nil).
       and_return(double(embeddings: [ [ 0.1 ] ]))
+    expect(documents).to receive(:invalidate_collections_cache!).
+      and_call_original
     expect(documents << 'foo').to eq documents
     expect(documents.exist?('foo')).to eq true
     expect(documents.exist?('bar')).to eq false
@@ -123,6 +127,8 @@ describe Documentrix::Documents do
 
     it 'can delete texts' do
       expect(documents << 'foo').to eq documents
+      expect(documents).to receive(:invalidate_collections_cache!).
+        and_call_original
       expect {
         documents.delete('foo')
       }.to change { documents.exist?('foo') }.from(true).to(false)
@@ -136,6 +142,8 @@ describe Documentrix::Documents do
 
     it 'can clear texts' do
       expect(documents << 'foo').to eq documents
+      expect(documents).to receive(:invalidate_collections_cache!).
+        and_call_original
       expect {
         documents.clear
       }.to change { documents.size }.from(1).to(0)
@@ -148,9 +156,13 @@ describe Documentrix::Documents do
       expect(documents.add('foo', tags: %w[ test ])).to eq documents
       expect(documents.add('bar', tags: %w[ test2 ])).to eq documents
       expect(documents.tags.to_a).to eq %w[ test test2 ]
+      expect(documents).to receive(:invalidate_collections_cache!).
+        and_call_original
       expect {
         documents.clear tags: 'test'
       }.to change { documents.size }.from(2).to(1)
+      expect(documents).to receive(:invalidate_collections_cache!).
+        and_call_original
       expect {
         documents.clear tags: :test2
       }.to change { documents.size }.from(1).to(0)
@@ -166,6 +178,8 @@ describe Documentrix::Documents do
 
       expect(documents.size).to eq 3
 
+      expect(documents).to receive(:invalidate_collections_cache!).
+        and_call_original
       documents.source_remove('source1')
 
       expect(documents.size).to eq 1
@@ -182,6 +196,8 @@ describe Documentrix::Documents do
       documents.collection = :foo
       documents << 'foo'
       expect(documents.collections).to eq %i[ default foo ]
+      expect(documents).to receive(:invalidate_collections_cache!).
+        and_call_original
       documents.rename_collection(:bar)
       expect(documents.collection).to eq :bar
       expect(documents.collections).to eq %i[ default bar ]
@@ -195,6 +211,8 @@ describe Documentrix::Documents do
       documents.collection = :bar
       documents << 'foo'
       expect(documents.collections).to eq %i[ default foo bar ]
+      expect(documents).not_to receive(:invalidate_collections_cache!).
+        and_call_original
       expect {
         documents.rename_collection(:foo)
       }.to raise_error(ArgumentError, 'new collection foo already exists!')
@@ -243,6 +261,8 @@ describe Documentrix::Documents do
       documents.add('foo', source: 's1')
 
       expect(ollama).not_to receive(:embed)
+      expect(documents).to receive(:invalidate_collections_cache!).
+        and_call_original
       documents.source_update(['foo'], source: 's1')
       expect(documents.exist?('foo')).to be true
     end
@@ -255,6 +275,8 @@ describe Documentrix::Documents do
       allow(documents.cache).to receive(:compute_file_digest).with('s1').and_return('d2')
 
       expect(ollama).to receive(:embed).once
+      expect(documents).to receive(:invalidate_collections_cache!).
+        at_least(1).and_call_original
       documents.source_update(['bar'], source: 's1')
 
       expect(documents.exist?('bar')).to be true
@@ -263,6 +285,8 @@ describe Documentrix::Documents do
 
     it 'updates the source if it is an URL' do
       expect(ollama).to receive(:embed).once
+      expect(documents).to receive(:invalidate_collections_cache!).
+        and_call_original
       documents.source_update('foo', source: 'https://www.example.com/s1')
       expect(documents.exist?('foo')).to be true
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: documentrix
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.3.2
 platform: ruby
 authors:
 - Florian Frank