Revert splitting of multiSearch, and instead use overloads

Author: flevi29

.code-samples.meilisearch.yaml

@@ -807,7 +807,7 @@ distinct_attribute_guide_filterable_1: |-
 distinct_attribute_guide_distinct_parameter_1: |-
   client.index('products').search({ q: 'white shirt', distinct: 'sku' })
 multi_search_federated_1: |-
-  client.federatedMultiSearch({
+  client.multiSearch({
     federation: {},
     queries: [
       {

src/meilisearch.ts

@@ -217,7 +217,16 @@ export class MeiliSearch {
   /// Multi Search
   ///
 
-  async #multiSearch(
+  /** {@link https://www.meilisearch.com/docs/reference/api/multi_search} */
+  multiSearch(
+    multiSearch: MultiSearch,
+    init?: ExtraRequestInit,
+  ): Promise<SearchResults>;
+  multiSearch(
+    federatedSearch: FederatedSearch,
+    init?: ExtraRequestInit,
+  ): Promise<FederatedSearchResult>;
+  async multiSearch(
     body: MultiSearchOrFederatedSearch,
     init?: ExtraRequestInit,
   ): Promise<SearchResultsOrFederatedSearchResult> {
@@ -228,18 +237,6 @@ export class MeiliSearch {
     });
   }
 
-  /** {@link https://www.meilisearch.com/docs/reference/api/multi_search} */
-  readonly multiSearch = this.#multiSearch.bind(this) as (
-    multiSearch: MultiSearch,
-    init?: ExtraRequestInit,
-  ) => Promise<SearchResults>;
-
-  /** {@link https://www.meilisearch.com/docs/reference/api/multi_search} */
-  readonly federatedMultiSearch = this.#multiSearch.bind(this) as (
-    federatedSearch: FederatedSearch,
-    init?: ExtraRequestInit,
-  ) => Promise<FederatedSearchResult>;
-
   ///
   ///  Network
   ///

src/types/functions.ts

@@ -6,4 +6,3 @@ export * from "./shared.js";
 export * from "./task-and-batch.js";
 export * from "./token.js";
 export * from "./types.js";
-export * from "./functions.js";

src/types/index.ts

@@ -401,11 +401,10 @@ test.concurrent(`${federatedMultiSearch.name} method`, async () => {
     facetsByIndex: { [INDEX_UID]: ["genres", "popularity"] },
   };
 
-  const { hits, facetsByIndex, remoteErrors } =
-    await client.federatedMultiSearch({
-      queries,
-      federation,
-    });
+  const { hits, facetsByIndex, remoteErrors } = await client.multiSearch({
+    queries,
+    federation,
+  });
 
   for (const {
     _federation: { weightedRankingScore, ..._federation },
@@ -425,7 +424,7 @@ test.concurrent(`${federatedMultiSearch.name} method`, async () => {
   // TODO: Maybe could get an error response for this, to validate it against
   assert.deepEqual(remoteErrors, {});
 
-  const { facetDistribution, facetStats } = await client.federatedMultiSearch({
+  const { facetDistribution, facetStats } = await client.multiSearch({
     queries,
     federation: { ...federation, mergeFacets: { maxValuesPerFacet: 100 } },
   });

tests/search.test.ts

@@ -53,15 +53,14 @@ export const [
     },
   },
   {
-    name: "federatedMultiSearch" satisfies keyof Pick<
-      MeiliSearch,
-      "federatedMultiSearch"
-    >,
+    name: "federatedMultiSearch" satisfies `federated${Capitalize<
+      keyof Pick<MeiliSearch, "multiSearch">
+    >}`,
     searchMethod: async (searchQuery) => {
       const { offset, limit, facets, ...restOfSearchQuery } =
         searchQuery as SearchQueryWithOffsetLimit;
 
-      const { hits, ...result } = await client.federatedMultiSearch({
+      const { hits, ...result } = await client.multiSearch({
         queries: [{ indexUid: INDEX_UID, ...restOfSearchQuery }],
         federation: {
           offset: offset ?? undefined,