Merge #1456

1456: Changes related to the next Meilisearch release (v1.1.0) r=bidoubiwa a=meili-bot

Related to this issue: meilisearch/integration-guides#251

This PR:
- gathers the changes related to the next Meilisearch release (v1.1.0) so that this package is ready when the official release is out.
- should pass the tests against the [latest pre-release of Meilisearch](https://github.com/meilisearch/meilisearch/releases).
- might eventually contain test failures until the Meilisearch v1.1.0 is out.

⚠️ This PR should NOT be merged until the next release of Meilisearch (v1.1.0) is out.

_This PR is auto-generated for the [pre-release week](https://github.com/meilisearch/integration-guides/blob/main/resources/pre-release-week.md) purpose._


Co-authored-by: meili-bot <[email protected]>
Co-authored-by: cvermand <[email protected]>

Author: 3 people

README.md

@@ -402,6 +402,16 @@ client.index<T>('xxx').search(query: string, options: SearchParams = {}, config?
 client.index<T>('xxx').searchGet(query: string, options: SearchParams = {}, config?: Partial<Request>): Promise<SearchResponse<T>>
 ```
 
+### Multi Search
+
+#### [Make multiple search requests](https://docs.meilisearch.com/reference/api/multi-search.html)
+
+```ts
+client.multiSearch(queries?: MultiSearchParams, config?: Partial<Request>): Promise<Promise<MultiSearchResponse<T>>>
+```
+
+`multiSearch` uses the `POST` method when performing its request to Meilisearch.
+
 ### Documents <!-- omit in toc -->
 
 #### [Add or replace multiple documents](https://docs.meilisearch.com/reference/api/documents.html#add-or-replace-documents)

src/clients/client.ts

@@ -32,6 +32,8 @@ import {
   SwapIndexesParams,
   CancelTasksQuery,
   DeleteTasksQuery,
+  MultiSearchParams,
+  MultiSearchResponse,
 } from '../types'
 import { HttpRequests } from '../http-requests'
 import { TaskClient, Task } from '../task'
@@ -189,6 +191,40 @@ class Client {
     return await this.httpRequest.post(url, params)
   }
 
+  ///
+  /// Multi Search
+  ///
+
+  /**
+   * Perform multiple search queries.
+   *
+   * It is possible to make multiple search queries on the same index or on
+   * different ones
+   *
+   * @example
+   *
+   * ```ts
+   * client.multiSearch({
+   *   queries: [
+   *     { indexUid: 'movies', q: 'wonder' },
+   *     { indexUid: 'books', q: 'flower' },
+   *   ],
+   * })
+   * ```
+   *
+   * @param queries - Search queries
+   * @param config - Additional request configuration options
+   * @returns Promise containing the search responses
+   */
+  async multiSearch<T extends Record<string, any> = Record<string, any>>(
+    queries?: MultiSearchParams,
+    config?: Partial<Request>
+  ): Promise<MultiSearchResponse<T>> {
+    const url = `/multi-search`
+
+    return await this.httpRequest.post(url, queries, undefined, config)
+  }
+
   ///
   /// TASKS
   ///

src/types/types.ts

@@ -107,6 +107,12 @@ export type SearchRequestGET = Pagination &
     showMatchesPosition?: boolean
   }
 
+export type MultiSearchQuery = SearchParams & { indexUid: string }
+
+export type MultiSearchParams = {
+  queries: MultiSearchQuery[]
+}
+
 export type CategoriesDistribution = {
   [category: string]: number
 }
@@ -124,6 +130,9 @@ export type Hit<T = Record<string, any>> = T & {
 
 export type Hits<T = Record<string, any>> = Array<Hit<T>>
 
+export type FacetStat = { min: number; max: number }
+export type FacetStats = Record<string, FacetStat>
+
 export type SearchResponse<
   T = Record<string, any>,
   S extends SearchParams | undefined = undefined
@@ -132,6 +141,7 @@ export type SearchResponse<
   processingTimeMs: number
   facetDistribution?: FacetDistribution
   query: string
+  facetStats?: FacetStats
 } & (undefined extends S
   ? Partial<FinitePagination & InfinitePagination>
   : true extends IsFinitePagination<NonNullable<S>>
@@ -169,6 +179,12 @@ type HasPage<S extends SearchParams> = undefined extends S['page']
   ? false
   : true
 
+export type MultiSearchResult<T> = SearchResponse<T> & { indexUid: string }
+
+export type MultiSearchResponse<T = Record<string, any>> = {
+  results: Array<MultiSearchResult<T>>
+}
+
 export type FieldDistribution = {
   [field: string]: number
 }

tests/search.test.ts

@@ -17,6 +17,13 @@ const emptyIndex = {
   uid: 'empty_test',
 }
 
+type Books = {
+  id: number
+  title: string
+  comment: string
+  genre: string
+}
+
 const dataset = [
   {
     id: 123,
@@ -85,6 +92,52 @@ describe.each([
     await client.waitForTask(task2)
   })
 
+  test(`${permission} key: Multi index search no queries`, async () => {
+    const client = await getClient(permission)
+    const response = await client.multiSearch({
+      queries: [],
+    })
+
+    expect(response.results.length).toEqual(0)
+  })
+
+  test(`${permission} key: Multi index search with one query`, async () => {
+    const client = await getClient(permission)
+    const response = await client.multiSearch({
+      queries: [{ indexUid: index.uid, q: 'prince' }],
+    })
+
+    expect(response.results[0].hits.length).toEqual(2)
+  })
+
+  test(`${permission} key: Multi index search with multiple queries`, async () => {
+    const client = await getClient(permission)
+    const response = await client.multiSearch({
+      queries: [
+        { indexUid: index.uid, q: 'something' },
+        { indexUid: emptyIndex.uid, q: 'something' },
+      ],
+    })
+
+    expect(response.results.length).toEqual(2)
+  })
+
+  test(`${permission} key: Multi index search with one query`, async () => {
+    const client = await getClient(permission)
+
+    type MyIndex = {
+      id: 1
+    }
+
+    const response = await client.multiSearch<MyIndex & Books>({
+      queries: [{ indexUid: index.uid, q: 'prince' }],
+    })
+
+    expect(response.results[0].hits.length).toEqual(2)
+    expect(response.results[0].hits[0].id).toEqual(456)
+    expect(response.results[0].hits[0].title).toEqual('Le Petit Prince')
+  })
+
   test(`${permission} key: Basic search`, async () => {
     const client = await getClient(permission)
     const response = await client.index(index.uid).search('prince', {})
@@ -94,6 +147,7 @@ describe.each([
     expect(response).toHaveProperty('offset', 0)
     expect(response).toHaveProperty('processingTimeMs', expect.any(Number))
     expect(response).toHaveProperty('query', 'prince')
+    expect(response.facetStats).toBeUndefined()
     expect(response.hits.length).toEqual(2)
     // @ts-expect-error Not present in the SearchResponse type because neither `page` or `hitsPerPage` is provided in the search params.
     expect(response.hitsPerPage).toBeUndefined()
@@ -453,12 +507,16 @@ describe.each([
     const client = await getClient(permission)
     const response = await client.index(index.uid).search('a', {
       filter: ['genre = romance'],
-      facets: ['genre'],
+      facets: ['genre', 'id'],
     })
 
     expect(response).toHaveProperty('facetDistribution', {
       genre: { romance: 2 },
+      id: { '123': 1, '2': 1 },
     })
+
+    expect(response.facetStats).toEqual({ id: { min: 2, max: 123 } })
+    expect(response.facetStats?.['id']?.max).toBe(123)
     expect(response).toHaveProperty('hits', expect.any(Array))
     expect(response.hits.length).toEqual(2)
   })
@@ -739,6 +797,14 @@ describe.each([{ permission: 'No' }])(
         ErrorStatusCode.MISSING_AUTHORIZATION_HEADER
       )
     })
+
+    test(`${permission} key: Try multi search and be denied`, async () => {
+      const client = await getClient(permission)
+      await expect(client.multiSearch({ queries: [] })).rejects.toHaveProperty(
+        'code',
+        ErrorStatusCode.MISSING_AUTHORIZATION_HEADER
+      )
+    })
   }
 )