@takaro/apiclient 0.0.0-dev.fc39ea2 → 0.0.0-dev.fd2dfe7

package/src/generated/base.ts

@@ -1,7 +1,7 @@
 /* tslint:disable */
 /* eslint-disable */
 /**
- * Takaro app-api
+ * Takaro API
  * No description provided (generated by Openapi Generator https://github.com/openapitools/openapi-generator)
  *
  * The version of the OpenAPI document: 0.0.0

package/src/generated/common.ts

@@ -1,7 +1,7 @@
 /* tslint:disable */
 /* eslint-disable */
 /**
- * Takaro app-api
+ * Takaro API
  * No description provided (generated by Openapi Generator https://github.com/openapitools/openapi-generator)
  *
  * The version of the OpenAPI document: 0.0.0

package/src/generated/configuration.ts

@@ -1,7 +1,7 @@
 /* tslint:disable */
 /* eslint-disable */
 /**
- * Takaro app-api
+ * Takaro API
  * No description provided (generated by Openapi Generator https://github.com/openapitools/openapi-generator)
  *
  * The version of the OpenAPI document: 0.0.0
@@ -117,7 +117,7 @@ export class Configuration {
    * @return True if the given MIME is JSON, false otherwise.
    */
   public isJsonMime(mime: string): boolean {
-    const jsonMime: RegExp = new RegExp('^(application/json|[^;/ \t]+/[^;/ \t]+[+]json)[ \t]*(;.*)?$', 'i');
+    const jsonMime: RegExp = new RegExp('^(application\/json|[^;/ \t]+\/[^;/ \t]+[+]json)[ \t]*(;.*)?$', 'i');
     return mime !== null && (jsonMime.test(mime) || mime.toLowerCase() === 'application/json-patch+json');
   }
 }

package/src/generated/index.ts

@@ -1,7 +1,7 @@
 /* tslint:disable */
 /* eslint-disable */
 /**
- * Takaro app-api
+ * Takaro API
  * No description provided (generated by Openapi Generator https://github.com/openapitools/openapi-generator)
  *
  * The version of the OpenAPI document: 0.0.0

package/src/lib/__tests__/baseClient.unit.test.ts

@@ -0,0 +1,17 @@
+import { describe, it } from 'node:test';
+import { expect } from '@takaro/test';
+import { BaseApiClient, DEFAULT_REQUEST_TIMEOUT_MS } from '../baseClient.js';
+
+describe('BaseApiClient timeouts', () => {
+  it('sets a default request timeout so a stalled request cannot hang forever', () => {
+    const client = new BaseApiClient({ url: 'http://localhost:3000', log: false });
+
+    expect(client.axiosInstance.defaults.timeout).to.equal(DEFAULT_REQUEST_TIMEOUT_MS);
+  });
+
+  it('lets a caller override the timeout', () => {
+    const client = new BaseApiClient({ url: 'http://localhost:3000', log: false, timeoutMs: 5000 });
+
+    expect(client.axiosInstance.defaults.timeout).to.equal(5000);
+  });
+});

package/src/lib/baseClient.ts

@@ -1,9 +1,24 @@
 import { MetaApi } from '../generated/api.js';
 import axios, { AxiosError, AxiosInstance, AxiosRequestConfig } from 'axios';
 
+/**
+ * Axios defaults to NO timeout, so a request whose TCP connection stalls blocks its
+ * await forever. During incident run-29688761260 a single POST hung for ~80 minutes
+ * (Tempo trace 15f5708701c38676355a0e42cb9f7ffe: a client span with no server span),
+ * freezing 3 of 4 CI shards until the 53-minute job timeout.
+ *
+ * 120s is deliberately generous — the goal is "not infinite", not "tight". The slowest
+ * legitimate single call is a domain create under full 4-shard CI load at roughly 30s.
+ * Individual long-running calls should pass a per-request `{ timeout }` override rather
+ * than raising this.
+ */
+export const DEFAULT_REQUEST_TIMEOUT_MS = 120_000;
+
 export interface IBaseApiClientConfig {
   url: string;
   log?: Logger | false;
+  /** Per-request timeout in ms. Defaults to DEFAULT_REQUEST_TIMEOUT_MS. */
+  timeoutMs?: number;
 }
 
 interface Logger {
@@ -23,13 +38,20 @@ export class BaseApiClient<T extends IBaseApiClientConfig> {
   };
 
   constructor(protected readonly config: T) {
+    // Browsers refuse to let JS set the User-Agent header (it's a "forbidden header").
+    // Only include it when running outside a browser context (Node.js / server-side).
+    // Use globalThis so this compiles under both DOM and non-DOM TypeScript lib targets.
+    const isBrowser =
+      typeof (globalThis as Record<string, unknown>)['window'] !== 'undefined' &&
+      typeof (globalThis as Record<string, unknown>)['document'] !== 'undefined';
     const axiosConfig: AxiosRequestConfig = {
       baseURL: this.config.url,
       headers: {
         'Content-Type': 'application/json',
-        'User-Agent': 'Takaro-API-Client',
+        ...(!isBrowser && { 'User-Agent': 'Takaro-API-Client' }),
       },
       withCredentials: true,
+      timeout: this.config.timeoutMs ?? DEFAULT_REQUEST_TIMEOUT_MS,
     };
     this.axios = this.addLoggers(axios.create(axiosConfig));
 

package/src/lib/client.ts

@@ -14,13 +14,14 @@ import {
   EventApi,
   PlayerOnGameServerApi,
   ItemApi,
-  StatsApi,
   ShopOrderApi,
   ShopListingApi,
   ShopCategoryApi,
+  ShopActionApi,
   TrackingApi,
   EntityApi,
   AnalyticsApi,
+  InviteLinkApi,
 } from '../generated/api.js';
 import { BaseApiClient, IBaseApiClientConfig } from './baseClient.js';
 
@@ -236,8 +237,8 @@ export class Client extends BaseApiClient<IApiClientConfig> {
     );
   }
 
-  get stats() {
-    return new StatsApi(
+  get shopListing() {
+    return new ShopListingApi(
       {
         isJsonMime: this.isJsonMime,
       },
@@ -246,8 +247,8 @@ export class Client extends BaseApiClient<IApiClientConfig> {
     );
   }
 
-  get shopListing() {
-    return new ShopListingApi(
+  get shopCategory() {
+    return new ShopCategoryApi(
       {
         isJsonMime: this.isJsonMime,
       },
@@ -256,8 +257,8 @@ export class Client extends BaseApiClient<IApiClientConfig> {
     );
   }
 
-  get shopCategory() {
-    return new ShopCategoryApi(
+  get shopAction() {
+    return new ShopActionApi(
       {
         isJsonMime: this.isJsonMime,
       },
@@ -305,4 +306,14 @@ export class Client extends BaseApiClient<IApiClientConfig> {
       this.axios,
     );
   }
+
+  get inviteLink() {
+    return new InviteLinkApi(
+      {
+        isJsonMime: this.isJsonMime,
+      },
+      '',
+      this.axios,
+    );
+  }
 }