@applitools/core 4.50.0 → 4.50.1

package/CHANGELOG.md

@@ -1,5 +1,28 @@
 # Changelog
 
+## [4.50.1](https://github.com/Applitools-Dev/sdk/compare/js/core@4.50.0...js/core@4.50.1) (2025-10-09)
+
+
+### Bug Fixes
+
+* various config fixes for storybook-addon ([#3257](https://github.com/Applitools-Dev/sdk/issues/3257)) ([86fd4d1](https://github.com/Applitools-Dev/sdk/commit/86fd4d114122bbaf675b5fa361b0e967595ca296))
+
+
+### Dependencies
+
+* @applitools/dom-snapshot bumped to 4.13.9
+  #### Bug Fixes
+
+  * verbose Unknown CSS object model logging | AD-11542 | FLD-3687 ([#3261](https://github.com/Applitools-Dev/sdk/issues/3261)) ([ba85d32](https://github.com/Applitools-Dev/sdk/commit/ba85d3287a81af109db1a7b407e5ead20f395d9f))
+* @applitools/nml-client bumped to 1.11.8
+
+* @applitools/core-base bumped to 1.28.2
+  #### Bug Fixes
+
+  * various config fixes for storybook-addon ([#3257](https://github.com/Applitools-Dev/sdk/issues/3257)) ([86fd4d1](https://github.com/Applitools-Dev/sdk/commit/86fd4d114122bbaf675b5fa361b0e967595ca296))
+* @applitools/ec-client bumped to 1.12.10
+
+
 ## [4.50.0](https://github.com/Applitools-Dev/sdk/compare/js/core@4.49.0...js/core@4.50.0) (2025-10-08)
 
 

package/dist/open-eyes.js

@@ -23,7 +23,7 @@ var __importStar = (this && this.__importStar) || function (mod) {
     return result;
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getLatestCommitInfoFromEnvVars = exports.makeOpenEyes = void 0;
+exports.handleConcurrency = exports.getLatestCommitInfoFromEnvVars = exports.makeOpenEyes = void 0;
 const driver_1 = require("@applitools/driver");
 const core_1 = require("./classic/core");
 const core_2 = require("./ufg/core");
@@ -105,27 +105,14 @@ function makeOpenEyes({ type: defaultType = 'classic', clients, batch, removeDup
         }
         const account = await core.getAccountInfo({ settings: eyesServerSettings, logger });
         const componentConcurrency = account.serverConcurrency.componentConcurrency;
-        // TODO remove this hacky override `useServerConcurrency` once we want to override for all SDKs
-        // remember when implementing: other SDKs use session concurrency, not component concurrency like storybook
-        if (useServerConcurrency) {
-            if (concurrency && concurrency < componentConcurrency) {
-                if (shouldPrintConcurrencyWarning) {
-                    // TODO make this warning more generic
-                    logger.console.warn(`Eyes Storybook "testConcurrency" explicitly set to ${concurrency}, which is lower than your available limit of ${componentConcurrency}. Using the lower value as requested.`);
-                }
-                settings.sessionConcurrency = concurrency;
-            }
-            else {
-                settings.sessionConcurrency = componentConcurrency;
-                if (concurrency && shouldPrintConcurrencyWarning) {
-                    logger.console.warn(`Overriding user-defined "testConcurrency". Using organization concurrency limit from Eyes server: ${settings.sessionConcurrency}`);
-                }
-            }
-            shouldPrintConcurrencyWarning = false;
-        }
-        else {
-            settings.sessionConcurrency = concurrency !== null && concurrency !== void 0 ? concurrency : types_1.DEFAULT_CONCURRENCY;
-        }
+        settings.sessionConcurrency = handleConcurrency({
+            userConcurrency: concurrency,
+            componentConcurrency,
+            useServerConcurrency,
+            shouldPrintConcurrencyWarning,
+            logger,
+        });
+        shouldPrintConcurrencyWarning = false;
         void core.logEvent({
             settings: {
                 ...settings,
@@ -282,3 +269,35 @@ exports.getLatestCommitInfoFromEnvVars = getLatestCommitInfoFromEnvVars;
 function isRealBatchId(batchId) {
     return !!batchId && !batchId.startsWith('generated');
 }
+function handleConcurrency({ userConcurrency, componentConcurrency, useServerConcurrency, shouldPrintConcurrencyWarning, logger, }) {
+    let sessionConcurrency;
+    // if user concurrency is 0 or negative, we ignore it and use server concurrency or default
+    if (userConcurrency !== undefined && userConcurrency <= 0) {
+        if (shouldPrintConcurrencyWarning) {
+            logger.console.warn(`"testConcurrency" is invalid. Using server concurrency or default concurrency instead.`);
+        }
+        userConcurrency = undefined;
+    }
+    // TODO only eyes-storybook uses this hacky override `useServerConcurrency`, not all SDKs
+    // remember when implementing for all SDKs: other SDKs use session concurrency, not component concurrency like storybook
+    if (useServerConcurrency) {
+        if (userConcurrency && userConcurrency < componentConcurrency) {
+            if (shouldPrintConcurrencyWarning) {
+                // TODO make this warning more generic
+                logger.console.warn(`Eyes Storybook "testConcurrency" explicitly set to ${userConcurrency}, which is lower than your available limit of ${componentConcurrency}. Using the lower value as requested.`);
+            }
+            sessionConcurrency = userConcurrency;
+        }
+        else {
+            sessionConcurrency = componentConcurrency;
+            if (userConcurrency && shouldPrintConcurrencyWarning) {
+                logger.console.warn(`Overriding user-defined "testConcurrency". Using organization concurrency limit from Eyes server: ${sessionConcurrency}`);
+            }
+        }
+    }
+    else {
+        sessionConcurrency = userConcurrency !== null && userConcurrency !== void 0 ? userConcurrency : types_1.DEFAULT_CONCURRENCY;
+    }
+    return sessionConcurrency;
+}
+exports.handleConcurrency = handleConcurrency;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@applitools/core",
-  "version": "4.50.0",
+  "version": "4.50.1",
   "homepage": "https://applitools.com",
   "bugs": {
     "url": "https://github.com/applitools/eyes.sdk.javascript1/issues"
@@ -79,13 +79,13 @@
     }
   },
   "dependencies": {
-    "@applitools/core-base": "1.28.1",
+    "@applitools/core-base": "1.28.2",
     "@applitools/dom-capture": "11.6.5",
-    "@applitools/dom-snapshot": "4.13.8",
+    "@applitools/dom-snapshot": "4.13.9",
     "@applitools/driver": "1.23.5",
-    "@applitools/ec-client": "1.12.9",
+    "@applitools/ec-client": "1.12.10",
     "@applitools/logger": "2.2.4",
-    "@applitools/nml-client": "1.11.7",
+    "@applitools/nml-client": "1.11.8",
     "@applitools/req": "1.8.4",
     "@applitools/screenshoter": "3.12.6",
     "@applitools/snippets": "2.7.0",

package/types/open-eyes.d.ts

@@ -34,4 +34,11 @@ export declare function getLatestCommitInfoFromEnvVars(logger: Logger): {
     timestamp: string;
     sha: string;
 } | undefined;
+export declare function handleConcurrency({ userConcurrency, componentConcurrency, useServerConcurrency, shouldPrintConcurrencyWarning, logger, }: {
+    userConcurrency: number | undefined;
+    componentConcurrency: number;
+    useServerConcurrency: boolean;
+    shouldPrintConcurrencyWarning: boolean;
+    logger: Logger;
+}): number;
 export {};