askui 0.30.0 → 0.32.0

package/dist/esm/utils/http/http-client-got.js

@@ -7,11 +7,12 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
         step((generator = generator.apply(thisArg, _arguments || [])).next());
     });
 };
-import got from 'got';
+import got, { HTTPError, } from 'got';
 import { CookieJar } from 'tough-cookie';
 import { logger } from '../../lib';
 import { Credentials } from './credentials';
 import { httpClientErrorHandler } from './custom-errors';
+import { UnkownHttpClientError } from './custom-errors/unkown-http-client-error';
 function buildRetryLog(requestUrl, errorCode, statusCode, errorMessage, delayInMs, delayReason, attemptCount) {
     const failureReasons = [];
     if (statusCode !== undefined && statusCode >= 400) {
@@ -40,6 +41,16 @@ export class HttpClientGot {
         const gotExtendOptions = this.buildGotExtendOptions(proxyAgents);
         this.askuiGot = got.extend(gotExtendOptions);
     }
+    /**
+     * Configures got with retry behavior for transient server errors.
+     *
+     * Got retries requests that fail with retryable status codes
+     * (500, 502, 503, 504, etc.) up to `limit` times using exponential backoff.
+     *
+     * POST requests are only retried if their URL is registered in `urlsToRetry`
+     * (see `shouldRetryOnError`), to avoid retrying non-idempotent calls
+     * to unknown endpoints.
+     */
     buildGotExtendOptions(proxyAgents) {
         const gotExtendOptions = {
             retry: {
@@ -87,6 +98,11 @@ export class HttpClientGot {
         }
         return gotExtendOptions;
     }
+    /**
+     * Only retry POST requests if the URL is explicitly registered in `urlsToRetry`
+     * (populated by InferenceClient with inference endpoint URLs).
+     * Non-POST requests (GET, PUT, etc.) are always retried.
+     */
     shouldRetryOnError(error) {
         var _a;
         return (((_a = error.request) === null || _a === void 0 ? void 0 : _a.options.method) !== 'POST'
@@ -112,19 +128,31 @@ export class HttpClientGot {
     }
     post(url, data) {
         return __awaiter(this, void 0, void 0, function* () {
+            // Note: We intentionally do NOT set `throwHttpErrors: false` here.
+            // Got must throw on non-2xx responses so that its built-in retry mechanism
+            // (configured in buildGotExtendOptions) can kick in for transient server errors
+            // (500, 502, 503, 504). After all retries are exhausted, got throws an HTTPError
+            // which we catch below and convert into our custom error hierarchy.
             const options = this.injectHeadersAndCookies(url, {
                 json: data,
                 responseType: 'json',
-                throwHttpErrors: false,
             });
-            const { body, statusCode, headers } = yield this.askuiGot.post(url, options);
-            if (headers['deprecation'] !== undefined) {
-                logger.warn(headers['deprecation']);
+            try {
+                const { body, headers } = yield this.askuiGot.post(url, options);
+                if (headers['deprecation'] !== undefined) {
+                    logger.warn(headers['deprecation']);
+                }
+                return { body, headers };
             }
-            if (statusCode !== 200) {
-                throw httpClientErrorHandler(statusCode, JSON.stringify(body));
+            catch (error) {
+                // After got exhausts all retries, it throws HTTPError.
+                // Convert it to our custom error types (ServerHttpClientError,
+                // AuthenticationHttpClientError, etc.) for consistent error handling.
+                if (error instanceof HTTPError) {
+                    throw httpClientErrorHandler(error.response.statusCode, error.response.body);
+                }
+                throw new UnkownHttpClientError(error instanceof Error ? error.message : String(error));
             }
-            return { body, headers };
         });
     }
     get(url_1) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "askui",
-    "version": "0.30.0",
+    "version": "0.32.0",
     "license": "MIT",
     "author": "askui GmbH <info@askui.com> (http://www.askui.com/)",
     "description": "Reliable, automated end-to-end-testing that depends on what is shown on your screen instead of the technology you are running on",
@@ -42,7 +42,8 @@
         "test": "cross-env NODE_EXTRA_CA_CERTS='test/proxy/certs/unit_test.pem' jest '/src'",
         "postinstall": "node -e \"require('./bin/askui-postinstall')\"",
         "release:prerelease": "cross-env HUSKY=0 release-it --preRelease=next",
-        "release": "cross-env HUSKY=0 release-it"
+        "release": "cross-env HUSKY=0 release-it",
+        "generate:SBOM": "cyclonedx-npm package.json --omit dev --gather-license-texts --output-file bom.json --output-format JSON"
     },
     "files": [
         "dist/cjs/",
@@ -73,6 +74,7 @@
         "yup": "1.4.0"
     },
     "devDependencies": {
+        "@cyclonedx/cyclonedx-npm": "^4.1.2",
         "@release-it/bumper": "6.0.1",
         "@release-it/conventional-changelog": "8.0.2",
         "@types/fs-extra": "11.0.4",