chore(clients): add default hitsPerPage for browse objects helper

Author: Fluf22

clients/algoliasearch-client-swift/Sources/Search/Extra/SearchClientExtension.swift

@@ -243,9 +243,13 @@ public extension SearchClient {
         aggregator: @escaping (BrowseResponse<T>) -> Void,
         requestOptions: RequestOptions? = nil
     ) async throws -> BrowseResponse<T> {
-        try await createIterable(
+        var updatedBrowseParams = browseParams
+        if updatedBrowseParams.hitsPerPage == nil {
+          updatedBrowseParams.hitsPerPage = 1000
+        }
+
+        return try await createIterable(
             execute: { previousResponse in
-                var updatedBrowseParams = browseParams
                 if let previousResponse {
                     updatedBrowseParams.cursor = previousResponse.cursor
                 }

templates/go/search_helpers.mustache

@@ -305,6 +305,10 @@ func (c *APIClient) BrowseObjects(
 	browseParams BrowseParamsObject,
 	opts ...IterableOption,
 ) error {
+	if browseParams.HitsPerPage == nil {
+    browseParams.HitsPerPage = utils.ToPtr(int32(1000))
+  }
+
 	_, err := CreateIterable( //nolint:wrapcheck
 		func(previousResponse *BrowseResponse, previousErr error) (*BrowseResponse, error) {
 			if previousResponse != nil {

templates/java/api_helpers.mustache

@@ -298,6 +298,10 @@ public GetApiKeyResponse waitForApiKey(String key, ApiKeyOperation operation) {
 public <T> Iterable<T> browseObjects(String indexName, BrowseParamsObject params, Class<T> innerType, RequestOptions requestOptions) {
   final Holder<String> currentCursor = new Holder<>();
 
+  if (params.getHitsPerPage() == null) {
+    params.setHitsPerPage(1000);
+  }
+
   return AlgoliaIterableHelper.createIterable(
     () -> {
       BrowseResponse<T> response = this.browse(indexName, params, innerType, requestOptions);

templates/javascript/clients/client/api/helpers.mustache

@@ -169,6 +169,10 @@ browseObjects<T>(
   }: BrowseOptions<BrowseResponse<T>> & BrowseProps,
   requestOptions?: RequestOptions
 ): Promise<BrowseResponse<T>> {
+  if (browseParams.hitsPerPage == null) {
+    browseParams.hitsPerPage = 1000;
+  }
+
   return createIterablePromise<BrowseResponse<T>>({
     func: (previousResponse) => {
       return this.browse(

templates/python/search_helpers.mustache

@@ -128,6 +128,8 @@
         """
         Helper: Iterate on the `browse` method of the client to allow aggregating objects of an index.
         """
+        browse_params.hits_per_page = browse_params.hits_per_page or 1000
+
         {{^isSyncClient}}async {{/isSyncClient}}def _func(_prev: Optional[BrowseResponse]) -> BrowseResponse:
             if _prev is not None and _prev.cursor is not None:
                 browse_params.cursor = _prev.cursor

templates/ruby/search_helpers.mustache

@@ -95,6 +95,8 @@ end
 # @param request_options [Hash] the requestOptions to send along with the query, they will be forwarded to the `browse` method.
 # @param block [Proc] the block to execute on each object of the index.
 def browse_objects(index_name, browse_params = Search::BrowseParamsObject.new, request_options = {}, &block)
+  browse_params[:hits_per_page] = browse_params[:hits_per_page] || 1000
+
   hits = []
   loop do
     res = browse(index_name, browse_params, request_options)