mongodb_meilisearch 1.1.0 → 1.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f57c18174244b2885536a990a2d602ce833e949dcf205112c78f30567a6eee6a
-  data.tar.gz: bb5bdd6c8cb90a6e1334c4606063a56b71515510f0f45c3e1ae2d817b4497163
+  metadata.gz: c7873b1a21e75522aa4fdaa063231997319faae90ddb9a4774cd814bf18a4afc
+  data.tar.gz: b4f241003a1ff709c1247f6411cec39831459795d8e17c429a9b7b187ff790cd
 SHA512:
-  metadata.gz: cb3e4f69dad6365682470020e3acc864492c45d79de54fb65fbab6fb9eaacb0811d8af02414f4df324a9d0cf64278646e0eac3062e7aff1a9873aa2f264d0d64
-  data.tar.gz: 54aa4eea1cfe0b378e9ad93c719eb31a5c35c96c08ef842d975efb8391cf1b55b0ca53e9ca3d8daf98626a02c30ea817e9623c78de0b6ce14ade726473767304
+  metadata.gz: 8ad38b1b7f7c14baaee4c12491affeb62c9b9a700ec41d3bddcdf3c095288c084c9bf8f6c9feb28300b441e5fd94380179c9f334c7d2df078d48410221fbc05f
+  data.tar.gz: 5792131d06698d2af16194078e1d5a8bfbf37b9084502f25183d5f3c347d712d2eeac7e604edf0298800a03795e573336adbdc9a5b1d47929691f9288e974027

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    mongodb_meilisearch (1.1.0)
+    mongodb_meilisearch (1.1.1)
       meilisearch
       mongoid (~> 7.0)
       rails

data/README.md

@@ -323,6 +323,16 @@ Be careful to not add documents that are already in the index.
   WARNING: if you think you should use this, you're probably 
   mistaken. 
 
+#### Indexes 
+By default every model gets its own search index. This means that 
+`Foo.search("some text")` will only search `Foo` objects. To have a
+search cross objects you'll need to use a "Shared Index" (see below).
+
+
+The name of the index isn't important when not using shared indexes.
+By default a model's index is the snake cased form of the class name. 
+For example, data for `MyWidget` models will be stored in the `my_widget` index.
+
 #### Shared indexes
 Imagine you have a `Note` and a `Comment` model, sharing an index so that
 you can perform a single search and have search results for both models
@@ -442,10 +452,14 @@ To invoke any of Meilisearch's custom search options (see [their documentation](
 
 `MyModel.search("search term", options: <my custom options>)`
 
-The Meilisearch-ruby gem should be able to convert keys from snake case to 
+Currently the Meilisearch-ruby gem can convert keys from snake case to 
 camel case. For example `hits_per_page` will become `hitsPerPage`. 
-Meilisearch ultimately wants camel case. Follow their documentation 
-to see what's available and what type of options to pass it. Note that your 
+Meilisearch ultimately wants camel case (`camelCase`) parameter keys, 
+_but_ `meilisearch-ruby` wants snake case (`snake_case`). 
+
+Follow Meilisearch's documentation 
+to see what's available and what type of options to pass it, but convert 
+them to snake case first. Note that your 
 options keys and values must all be simple JSON values.
 
 If for some reason that still isn't enough, you can work with the 

data/lib/mongodb_meilisearch/version.rb

@@ -5,5 +5,5 @@ module MongodbMeilisearch
   # @note This library will adhere to strict semantic versioning.
   # See https://semver.org/
   #
-  VERSION = "1.1.0"
+  VERSION = "1.1.1"
 end

data/lib/search/class_methods.rb

@@ -109,6 +109,7 @@ module Search
     # @param filtered_by_class [Boolean] - defaults to filtering results by the class
     #        your searching from. Ex. Foo.search("something") it will
     #        have Meilisearch filter on records where `object_class` == `"Foo"`
+    #        This simplifies working with shared indexes.
     # @return [Hash[String, [Array[String | Document]] a hash with keys corresponding
     # to the classes of objects returned, and a value of an array of ids or
     # Mongoid::Document objects sorted by match strength. It will ALSO have a key named
@@ -135,13 +136,13 @@ module Search
 
       # TODO: break this if/else out into a separate function
       if ids_only
-        options.merge!({attributesToRetrieve: [pk.to_s]})
+        options.merge!({attributes_to_retrieve: [pk.to_s]})
       else
         # Don't care what you add, but we need the primary key and object_class
-        options[:attributesToRetrieve] = [] unless options.has_key?(:attributesToRetrieve)
+        options[:attributes_to_retrieve] = [] unless options.has_key?(:attributes_to_retrieve)
         [pk.to_s, "object_class"].each do |attr|
-          unless options[:attributesToRetrieve].include?(attr)
-            options[:attributesToRetrieve] << attr
+          unless options[:attributes_to_retrieve].include?(attr)
+            options[:attributes_to_retrieve] << attr
           end
         end
       end
@@ -260,6 +261,7 @@ module Search
     #    - each document hash is presumed to have been created by way of
     #      search_indexable_hash
     def add_documents(new_documents, async: true)
+      configure_attributes_and_index_if_needed!
       if async
         search_index.add_documents(new_documents, primary_search_key)
       else
@@ -393,17 +395,38 @@ module Search
     # @return [Array] - an array of attributes configured as sortable
     # in the index.
     def meilisearch_sortable_attributes
-      # Search::Client.instance.http_get(
-      search_index.http_get(
-        "/indexes/#{search_index}/settings/sortable-attributes"
-      )
+      @_meilisearch_sortable_attributes ||= search_index.get_sortable_attributes
     end
 
     def meilisearch_filterable_attributes
-      # Search::Client.instance.http_get(
-      search_index.http_get(
-        "/indexes/#{search_index}/settings/filterable-attributes"
-      )
+      @_meilisearch_filterable_attributes ||= search_index.get_filterable_attributes
+    end
+
+    def reset_cached_data!
+      @_meilisearch_filterable_attributes = nil
+      @_filterable_attributes             = nil
+      @_meilisearch_sortable_attributes   = nil
+      @_sortable_attributes               = nil
+    end
+
+    # Guarantees that the filterable attributes have been configured
+    # This should be called before
+    def configure_attributes_and_index_if_needed!
+      return if unfilterable?
+      indexes_filterable_attributes = []
+
+      begin
+        indexes_filterable_attributes = meilisearch_filterable_attributes
+      rescue ::MeiliSearch::ApiError => e
+        # this is expected to happen the first time an instance
+        # of a new model is saved.
+        raise unless e.message.match?(/Index `\S+` not found\./)
+        Search::Client.instance.create_index(search_index_name)
+      end
+
+      return if indexes_filterable_attributes.include?("object_class")
+      set_filterable_attributes
+      set_sortable_attributes
     end
 
     # Updates the filterable attributes in the search index.
@@ -579,7 +602,7 @@ module Search
     def reindex_core(async: true)
       # no point in continuing if this fails...
       delete_all_documents!
-
+      reset_cached_data!
       # this conveniently lines up with the batch size of 100
       # that Mongoid gives us
       documents = []

data/lib/search/client.rb

@@ -30,7 +30,7 @@ module Search
       if @client.respond_to? m.to_sym
         @client.send(m, *args, &block)
       else
-        raise ArgumentError.new("Method `#{m}` doesn't exist.")
+        raise ArgumentError.new("Method `#{m}` doesn't exist in #{@client.inspect}.")
       end
     end
 

data/lib/search/instance_methods.rb

@@ -2,6 +2,7 @@ module Search
   module InstanceMethods
     # Adds this record to the search index asynchronously
     def add_to_search
+      self.class.configure_attributes_and_index_if_needed!
       search_index.add_documents(
         [search_indexable_hash],
         primary_search_key.to_s
@@ -10,6 +11,7 @@ module Search
 
     # Adds this record to the search index synchronously
     def add_to_search!
+      self.class.configure_attributes_and_index_if_needed!
       index     = search_index
       documents = [search_indexable_hash]
       pk        = primary_search_key.to_s

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mongodb_meilisearch
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.1.1
 platform: ruby
 authors:
 - masukomi
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-09-16 00:00:00.000000000 Z
+date: 2023-11-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -187,7 +187,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.17
+rubygems_version: 3.4.20
 signing_key:
 specification_version: 4
 summary: MeiliSearch integration for MongoDB