feat(templates): rename userAgent to algoliaAgent (#508)

Co-authored-by: Clément Vannicatte <[email protected]>

Author: Haroenv

.github/workflows/check.yml

@@ -272,6 +272,13 @@ jobs:
         with:
           type: minimal
 
+      - name: Download JavaScript clients
+        if: ${{ steps.cache.outputs.cache-hit != 'true' }}
+        uses: ./.github/actions/restore-artifacts
+        with:
+          javascript: true
+          type: all
+
       - name: Build 'algoliasearch' client
         if: ${{ steps.cache.outputs.cache-hit != 'true' }}
         run: yarn cli build clients javascript algoliasearch

clients/algoliasearch-client-java-2/algoliasearch-core/src/main/java/com/algolia/utils/UserAgent.java renamed to clients/algoliasearch-client-java-2/algoliasearch-core/src/main/java/com/algolia/utils/AlgoliaAgent.java

@@ -3,13 +3,13 @@
 import java.util.LinkedHashSet;
 import java.util.Set;
 
-public class UserAgent {
+public class AlgoliaAgent {
 
   private final Set<String> segments;
 
   private String finalValue;
 
-  public UserAgent(String clientVersion) {
+  public AlgoliaAgent(String clientVersion) {
     this.finalValue = String.format("Algolia for Java (%s)", clientVersion);
     this.segments = new LinkedHashSet<String>();
     this.addSegment(new Segment("JVM", System.getProperty("java.version")));

clients/algoliasearch-client-javascript/packages/algoliasearch/builds/browser.ts

@@ -45,7 +45,7 @@ export function algoliasearch(
       write: 30,
     },
     requester: options?.requester ?? createXhrRequester(),
-    userAgents: [{ segment: 'Browser' }],
+    algoliaAgents: [{ segment: 'Browser' }],
     authMode: 'WithinQueryParameters',
     responsesCache: createMemoryCache(),
     requestsCache: createMemoryCache({ serializable: false }),

clients/algoliasearch-client-javascript/packages/algoliasearch/builds/node.ts

@@ -41,7 +41,7 @@ export function algoliasearch(
       write: 30,
     },
     requester: options?.requester ?? createHttpRequester(),
-    userAgents: [{ segment: 'Node.js', version: process.versions.node }],
+    algoliaAgents: [{ segment: 'Node.js', version: process.versions.node }],
     responsesCache: createNullCache(),
     requestsCache: createNullCache(),
     hostsCache: createMemoryCache(),

clients/algoliasearch-client-javascript/packages/client-common/index.ts

@@ -2,6 +2,6 @@ export * from './src/createAuth';
 export * from './src/createEchoRequester';
 export * from './src/cache';
 export * from './src/transporter';
-export * from './src/createUserAgent';
-export * from './src/getUserAgent';
+export * from './src/createAlgoliaAgent';
+export * from './src/getAlgoliaAgent';
 export * from './src/types';

clients/algoliasearch-client-javascript/packages/client-common/src/createAlgoliaAgent.ts

@@ -0,0 +1,20 @@
+import type { AlgoliaAgentOptions, AlgoliaAgent } from './types';
+
+export function createAlgoliaAgent(version: string): AlgoliaAgent {
+  const algoliaAgent = {
+    value: `Algolia for JavaScript (${version})`,
+    add(options: AlgoliaAgentOptions): AlgoliaAgent {
+      const addedAlgoliaAgent = `; ${options.segment}${
+        options.version !== undefined ? ` (${options.version})` : ''
+      }`;
+
+      if (algoliaAgent.value.indexOf(addedAlgoliaAgent) === -1) {
+        algoliaAgent.value = `${algoliaAgent.value}${addedAlgoliaAgent}`;
+      }
+
+      return algoliaAgent;
+    },
+  };
+
+  return algoliaAgent;
+}

clients/algoliasearch-client-javascript/packages/client-common/src/createEchoRequester.ts

@@ -2,7 +2,7 @@ import type { EchoResponse, EndRequest, Request, Response } from './types';
 
 export type UrlParams = {
   host: string;
-  userAgent: string;
+  algoliaAgent: string;
   searchParams: EchoResponse['searchParams'];
 };
 
@@ -20,7 +20,7 @@ export function createEchoRequester({
     { headers, url, connectTimeout, responseTimeout }: EndRequest,
     { data, ...originalRequest }: Request
   ): Promise<Response> {
-    const { host, searchParams, userAgent } = getUrlParams(url);
+    const { host, searchParams, algoliaAgent } = getUrlParams(url);
     const originalData =
       data && Object.entries(data).length > 0 ? data : undefined;
 
@@ -31,7 +31,7 @@ export function createEchoRequester({
         headers,
         connectTimeout,
         responseTimeout,
-        userAgent: userAgent ? encodeURI(userAgent) : undefined,
+        algoliaAgent: algoliaAgent ? encodeURI(algoliaAgent) : undefined,
         searchParams,
         data: originalData,
       }),

clients/algoliasearch-client-javascript/packages/client-common/src/createUserAgent.ts

@@ -0,0 +1,25 @@
+import { createAlgoliaAgent } from './createAlgoliaAgent';
+import type { AlgoliaAgentOptions, AlgoliaAgent } from './types';
+
+export type GetAlgoliaAgent = {
+  algoliaAgents: AlgoliaAgentOptions[];
+  client: string;
+  version: string;
+};
+
+export function getAlgoliaAgent({
+  algoliaAgents,
+  client,
+  version,
+}: GetAlgoliaAgent): AlgoliaAgent {
+  const defaultAlgoliaAgent = createAlgoliaAgent(version).add({
+    segment: client,
+    version,
+  });
+
+  algoliaAgents.forEach((algoliaAgent) =>
+    defaultAlgoliaAgent.add(algoliaAgent)
+  );
+
+  return defaultAlgoliaAgent;
+}

clients/algoliasearch-client-javascript/packages/client-common/src/getAlgoliaAgent.ts

@@ -36,7 +36,7 @@ export function createTransporter({
   hostsCache,
   baseHeaders,
   baseQueryParameters,
-  userAgent,
+  algoliaAgent,
   timeouts,
   requester,
   requestsCache,
@@ -107,7 +107,7 @@ export function createTransporter({
       : {};
 
     const queryParameters: QueryParameters = {
-      'x-algolia-agent': userAgent.value,
+      'x-algolia-agent': algoliaAgent.value,
       ...baseQueryParameters,
       ...dataQueryParameters,
     };
@@ -355,7 +355,7 @@ export function createTransporter({
     hostsCache,
     requester,
     timeouts,
-    userAgent,
+    algoliaAgent,
     baseHeaders,
     baseQueryParameters,
     hosts,

clients/algoliasearch-client-javascript/packages/client-common/src/getUserAgent.ts

@@ -3,7 +3,7 @@ import type { Host } from './Host';
 import type { Requester } from './Requester';
 import type {
   Timeouts,
-  UserAgentOptions,
+  AlgoliaAgentOptions,
   TransporterOptions,
 } from './Transporter';
 
@@ -17,7 +17,7 @@ export type CreateClientOptions = Pick<
   apiKey: string;
   requester: Requester;
   timeouts: Timeouts;
-  userAgents: UserAgentOptions[];
+  algoliaAgents: AlgoliaAgentOptions[];
   hosts?: Host[];
   authMode?: AuthMode;
 };

clients/algoliasearch-client-javascript/packages/client-common/src/transporter/createTransporter.ts

@@ -37,5 +37,5 @@ export type EchoResponse = Request & {
   headers: Headers;
   responseTimeout: number;
   searchParams?: Record<string, string>;
-  userAgent?: string;
+  algoliaAgent?: string;
 };

clients/algoliasearch-client-javascript/packages/client-common/src/types/CreateClient.ts

@@ -45,7 +45,7 @@ export type StackFrame = {
   triesLeft: number;
 };
 
-export type UserAgentOptions = {
+export type AlgoliaAgentOptions = {
   /**
    * The segment. Usually the integration name.
    */
@@ -57,7 +57,7 @@ export type UserAgentOptions = {
   version?: string;
 };
 
-export type UserAgent = {
+export type AlgoliaAgent = {
   /**
    * The raw value of the user agent.
    */
@@ -66,7 +66,7 @@ export type UserAgent = {
   /**
    * Mutates the current user agent ading the given user agent options.
    */
-  add: (options: UserAgentOptions) => UserAgent;
+  add: (options: AlgoliaAgentOptions) => AlgoliaAgent;
 };
 
 export type Timeouts = {
@@ -133,7 +133,7 @@ export type TransporterOptions = {
   /**
    * The user agent used. Sent on query parameters.
    */
-  userAgent: UserAgent;
+  algoliaAgent: AlgoliaAgent;
 };
 
 export type Transporter = {
@@ -175,7 +175,7 @@ export type Transporter = {
   /**
    * The user agent used. Sent on query parameters.
    */
-  userAgent: UserAgent;
+  algoliaAgent: AlgoliaAgent;
 
   /**
    * The headers used on each request.

clients/algoliasearch-client-javascript/packages/client-common/src/types/Requester.ts

@@ -3,7 +3,7 @@ import type { UrlParams } from '@experimental-api-clients-automation/client-comm
 
 function getUrlParams(url: string): UrlParams {
   const { host, searchParams: urlSearchParams } = new URL(url);
-  const userAgent = urlSearchParams.get('x-algolia-agent') || '';
+  const algoliaAgent = urlSearchParams.get('x-algolia-agent') || '';
   const searchParams = {};
 
   for (const [k, v] of urlSearchParams) {
@@ -16,7 +16,7 @@ function getUrlParams(url: string): UrlParams {
 
   return {
     host,
-    userAgent,
+    algoliaAgent,
     searchParams:
       Object.entries(searchParams).length === 0 ? undefined : searchParams,
   };

clients/algoliasearch-client-javascript/packages/client-common/src/types/Transporter.ts

@@ -5,7 +5,7 @@ import type { UrlParams } from '@experimental-api-clients-automation/client-comm
 
 function getUrlParams(url: string): UrlParams {
   const { host, searchParams: urlSearchParams } = new URL(url);
-  const userAgent = urlSearchParams.get('x-algolia-agent') || '';
+  const algoliaAgent = urlSearchParams.get('x-algolia-agent') || '';
   const searchParams = {};
 
   for (const [k, v] of urlSearchParams) {
@@ -18,7 +18,7 @@ function getUrlParams(url: string): UrlParams {
 
   return {
     host,
-    userAgent,
+    algoliaAgent,
     searchParams:
       Object.entries(searchParams).length === 0 ? undefined : searchParams,
   };

clients/algoliasearch-client-javascript/packages/requester-browser-xhr/src/echoRequester.ts

@@ -3,7 +3,7 @@
 namespace Algolia\AlgoliaSearch\RequestOptions;
 
 use Algolia\AlgoliaSearch\Configuration\Configuration;
-use Algolia\AlgoliaSearch\Support\UserAgent;
+use Algolia\AlgoliaSearch\Support\AlgoliaAgent;
 
 final class RequestOptionsFactory
 {
@@ -50,9 +50,9 @@ private function normalize($options)
             'headers' => [
                 'X-Algolia-Application-Id' => $this->config->getAppId(),
                 'X-Algolia-API-Key' => $this->config->getAlgoliaApiKey(),
-                'User-Agent' => $this->config->getUserAgent() !== null
-                        ? $this->config->getUserAgent()
-                        : UserAgent::get(),
+                'User-Agent' => $this->config->getAlgoliaAgent() !== null
+                        ? $this->config->getAlgoliaAgent()
+                        : AlgoliaAgent::get(),
                 'Content-Type' => 'application/json',
             ],
             'queryParameters' => [],

clients/algoliasearch-client-javascript/packages/requester-node-http/src/echoRequester.ts

@@ -4,7 +4,7 @@
 
 use Algolia\AlgoliaSearch\Algolia;
 
-final class UserAgent
+final class AlgoliaAgent
 {
     private static $value;
 
@@ -19,7 +19,7 @@ public static function get()
         return self::$value;
     }
 
-    public static function addCustomUserAgent($segment, $version)
+    public static function addCustomAlgoliaAgent($segment, $version)
     {
         self::$value = null;
         self::$customSegments[trim($segment, ' ')] = trim($version, ' ');

clients/algoliasearch-client-php/lib/RequestOptions/RequestOptionsFactory.php

@@ -59,7 +59,7 @@ private void setDefaultGeneratorOptions() {
 
     additionalProperties.put("apiName", apiName);
     additionalProperties.put("capitalizedApiName", Utils.capitalize(apiName));
-    additionalProperties.put("userAgent", Utils.capitalize(CLIENT));
+    additionalProperties.put("algoliaAgent", Utils.capitalize(CLIENT));
     additionalProperties.put("gitRepoId", "algoliasearch-client-javascript");
   }
 

clients/algoliasearch-client-php/lib/Support/UserAgent.php renamed to clients/algoliasearch-client-php/lib/Support/AlgoliaAgent.php

@@ -5,7 +5,7 @@
 import com.algolia.exceptions.AlgoliaRuntimeException;
 import com.algolia.model.insights.*;
 import com.algolia.api.InsightsClient;
-import com.algolia.utils.UserAgent;
+import com.algolia.utils.AlgoliaAgent;
 import io.github.cdimascio.dotenv.Dotenv;
 
 public class Insights {
@@ -15,10 +15,10 @@ public static void main(String[] args) {
     InsightsClient client = new InsightsClient(
       dotenv.get("ALGOLIA_APPLICATION_ID"),
       dotenv.get("ALGOLIA_SEARCH_KEY"),
-      new UserAgent.Segment[] {
-        new UserAgent.Segment("test", "8.0.0"),
-        new UserAgent.Segment("JVM", "11.0.14"),
-        new UserAgent.Segment("no version"),
+      new AlgoliaAgent.Segment[] {
+        new AlgoliaAgent.Segment("test", "8.0.0"),
+        new AlgoliaAgent.Segment("JVM", "11.0.14"),
+        new AlgoliaAgent.Segment("no version"),
       }
     );