Merge #1458 #1459

1458: Add multi search method of Meilisearch v1.1 r=bidoubiwa a=bidoubiwa

Introduces the `client.multiSearch()` method as per the [specifications](meilisearch/specifications#225)

SDK requirements: meilisearch/integration-guides#251

1459: Add facetStats type in searchResponse for MS v1.1.0 r=bidoubiwa a=bidoubiwa

As per [the specification](meilisearch/specifications#224)

SDK requirements: meilisearch/integration-guides#251

A new response field, `facetStats` is returned when `facets` is used in the search parameters. It contains the min max value of facets that contain numeric values.

I added some test `@brunoocasali` to ensure my typing is correct





Co-authored-by: cvermand <[email protected]>
Co-authored-by: Charlotte Vermandel <[email protected]>

Author: meili-bors[bot]

README.md

@@ -362,6 +362,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

@@ -106,6 +106,10 @@ export type SearchRequestGET = Pagination &
     showMatchesPosition?: boolean
   }
 
+export type MultiSearchParams = {
+  queries: Array<SearchParams & { indexUid: string }>
+}
+
 export type CategoriesDistribution = {
   [category: string]: number
 }
@@ -131,6 +135,7 @@ export type SearchResponse<
   processingTimeMs: number
   facetDistribution?: FacetDistribution
   query: string
+  facetStats?: Record<string, { min?: number; max?: number }>
 } & (undefined extends S
   ? Partial<FinitePagination & InfinitePagination>
   : true extends IsFinitePagination<NonNullable<S>>
@@ -168,6 +173,10 @@ type HasPage<S extends SearchParams> = undefined extends S['page']
   ? false
   : true
 
+export type MultiSearchResponse<T = Record<string, any>> = {
+  results: Array<SearchResponse<T> & { indexUid: string }>
+}
+
 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,17 @@ 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).toHaveProperty('facetStats', { id: { min: 2, max: 123 } })
+    expect(response.facetStats?.['id']?.min).toBe(2)
+    expect(response.facetStats?.['id']?.max).toBe(123)
     expect(response).toHaveProperty('hits', expect.any(Array))
     expect(response.hits.length).toEqual(2)
   })
@@ -739,6 +798,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
+      )
+    })
   }
 )