@applitools/eyes-playwright 1.47.4 → 1.47.5

package/CHANGELOG.md

@@ -1,5 +1,45 @@
 # Changelog
 
+## [1.47.5](https://github.com/Applitools-Dev/sdk/compare/js/eyes-playwright@1.47.4...js/eyes-playwright@1.47.5) (2026-06-04)
+
+
+### Bug Fixes
+
+* centralize APPLITOOLS_* env reads for fixture configuration | FLD-4601 ([#3888](https://github.com/Applitools-Dev/sdk/issues/3888)) ([9b3180f](https://github.com/Applitools-Dev/sdk/commit/9b3180f23e0d42a2326a0bef038382b4da07bb1a))
+* use _apiName for Page/Locator type check on Playwright 1.60+ | FLD-4598 ([#3872](https://github.com/Applitools-Dev/sdk/issues/3872)) ([7149bef](https://github.com/Applitools-Dev/sdk/commit/7149bef4ea3e352d37383d21e8267989a0090591))
+
+
+### Dependencies
+
+* @applitools/nml-client bumped to 1.11.29
+
+* @applitools/core-base bumped to 1.35.0
+  #### Features
+
+  * auto-accept UFG diffs from infra updates | AD-9583 ([#3797](https://github.com/Applitools-Dev/sdk/issues/3797)) ([fde553c](https://github.com/Applitools-Dev/sdk/commit/fde553c947aa3c040660ef5d07855fa4a1412399))
+* @applitools/core bumped to 4.64.0
+  #### Features
+
+  * auto-accept UFG diffs from infra updates | AD-9583 ([#3797](https://github.com/Applitools-Dev/sdk/issues/3797)) ([fde553c](https://github.com/Applitools-Dev/sdk/commit/fde553c947aa3c040660ef5d07855fa4a1412399))
+  * support default match level from renderinfo | AD-13499 ([#3799](https://github.com/Applitools-Dev/sdk/issues/3799)) ([c7e8aeb](https://github.com/Applitools-Dev/sdk/commit/c7e8aeb3f231d9b4e6e368d7ae5f87115c5cad09))
+
+
+  #### Bug Fixes
+
+  * source hideCaret from settings.hideCaret in NML classic capture | AD-14053 ([#3878](https://github.com/Applitools-Dev/sdk/issues/3878)) ([f07b4f5](https://github.com/Applitools-Dev/sdk/commit/f07b4f5c90bb9b7de0300cef93f23dc833268a7c))
+
+
+
+* @applitools/ec-client bumped to 1.12.31
+
+* @applitools/eyes bumped to 1.43.0
+  #### Features
+
+  * auto-accept UFG diffs from infra updates | AD-9583 ([#3797](https://github.com/Applitools-Dev/sdk/issues/3797)) ([fde553c](https://github.com/Applitools-Dev/sdk/commit/fde553c947aa3c040660ef5d07855fa4a1412399))
+
+
+
+
 ## [1.47.4](https://github.com/Applitools-Dev/sdk/compare/js/eyes-playwright@1.47.3...js/eyes-playwright@1.47.4) (2026-05-26)
 
 

package/dist/fixture/defaults/env.js

@@ -0,0 +1,85 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || function (mod) {
+    if (mod && mod.__esModule) return mod;
+    var result = {};
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    __setModuleDefault(result, mod);
+    return result;
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.applyEnvDefaults = void 0;
+const utils = __importStar(require("@applitools/utils"));
+// Empty-string env values are treated as unset so the fixture's fallback wins —
+// CI systems commonly yield "" for declared-but-unfilled vars (e.g. a missing
+// GH Actions secret), and a blank batch name / branch name is never intended.
+// NOTE: intentionally differs from core's env block, which accepts ''.
+function readString(name) {
+    const value = utils.general.getEnvValue(name);
+    return value ? value : undefined;
+}
+function readBoolean(name) {
+    const raw = readString(name);
+    if (raw === undefined)
+        return undefined;
+    return ['true', '1'].includes(raw);
+}
+// Assigns `value` to `obj[key]` only when there is a concrete value to write
+// AND the slot is still nullish. `??=` alone is unsafe here because
+// `obj[key] ??= undefined` still defines `key` as an enumerable property —
+// and the legacy `eyes` `Configuration` class's setters (e.g. `set branchName`)
+// throw `IllegalArgument` when an undefined key is assigned through them.
+function assignIfDefined(obj, key, value) {
+    if (value === undefined)
+        return;
+    if (obj[key] === undefined || obj[key] === null)
+        obj[key] = value;
+}
+/**
+ * Mirrors the APPLITOOLS_* env block in @applitools/core's open-eyes so the
+ * fixture honors the same env contract before applying any Playwright-aware
+ * default — otherwise a fallback set via `??=` in the fixture would short-
+ * circuit core's later `??=` env read (see FLD-4601). Only writes keys that
+ * have a concrete value, so the merged config never contains undefined
+ * properties that the legacy `eyes` Configuration setters would reject.
+ * Mutates `configuration` in place.
+ */
+function applyEnvDefaults(configuration) {
+    var _a, _b, _c, _d;
+    const batchId = readString('BATCH_ID');
+    const batchBuildId = readString('BATCH_BUILD_ID');
+    const batchName = readString('BATCH_NAME');
+    const batchSequence = readString('BATCH_SEQUENCE');
+    const batchNotify = readBoolean('BATCH_NOTIFY');
+    if (batchId !== undefined ||
+        batchBuildId !== undefined ||
+        batchName !== undefined ||
+        batchSequence !== undefined ||
+        batchNotify !== undefined) {
+        (_a = configuration.batch) !== null && _a !== void 0 ? _a : (configuration.batch = {});
+        assignIfDefined(configuration.batch, 'id', batchId);
+        assignIfDefined(configuration.batch, 'buildId', batchBuildId);
+        assignIfDefined(configuration.batch, 'name', batchName);
+        assignIfDefined(configuration.batch, 'sequenceName', batchSequence);
+        assignIfDefined(configuration.batch, 'notifyOnCompletion', batchNotify);
+    }
+    assignIfDefined(configuration, 'branchName', (_b = readString('BRANCH')) !== null && _b !== void 0 ? _b : readString('BRANCH_NAME'));
+    assignIfDefined(configuration, 'parentBranchName', (_c = readString('PARENT_BRANCH')) !== null && _c !== void 0 ? _c : readString('PARENT_BRANCH_NAME'));
+    assignIfDefined(configuration, 'baselineBranchName', (_d = readString('BASELINE_BRANCH')) !== null && _d !== void 0 ? _d : readString('BASELINE_BRANCH_NAME'));
+}
+exports.applyEnvDefaults = applyEnvDefaults;

package/dist/fixture/defaults/eyesConfiguration.js

@@ -3,8 +3,9 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getFinalEyesConfiguration = void 0;
+exports.getFinalEyesConfiguration = exports.addBatchDefaults = void 0;
 const name_1 = require("./batch/name");
+const env_1 = require("./env");
 const path_1 = __importDefault(require("path"));
 function addConfigurationProperties({ properties = [], testInfo, }) {
     const result = [
@@ -33,7 +34,8 @@ function addConfigurationProperties({ properties = [], testInfo, }) {
 }
 function addBatchDefaults({ configuration }) {
     var _a, _b, _c, _d;
-    const batch = (_a = configuration.batch) !== null && _a !== void 0 ? _a : {};
+    (0, env_1.applyEnvDefaults)(configuration);
+    const batch = ((_a = configuration.batch) !== null && _a !== void 0 ? _a : (configuration.batch = {}));
     (_b = batch.id) !== null && _b !== void 0 ? _b : (batch.id = `generated-${process.ppid}`);
     (_c = batch.name) !== null && _c !== void 0 ? _c : (batch.name = (0, name_1.getBatchName)({ appName: configuration.appName }));
     (_d = batch.properties) !== null && _d !== void 0 ? _d : (batch.properties = []);
@@ -46,6 +48,7 @@ function addBatchDefaults({ configuration }) {
     }
     return batch;
 }
+exports.addBatchDefaults = addBatchDefaults;
 function getFinalEyesConfiguration({ playwrightConfig = {}, projectConfig = {}, eyesConfig = {}, testInfo, }) {
     var _a, _b, _c;
     const configuration = {

package/dist/fixture/toHaveScreenshot.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.expectTypes = exports.expect = void 0;
+exports.expectTypes = exports.getApiName = exports.expect = void 0;
 const test_1 = require("@playwright/test");
 const index_1 = require("./index");
 const getEyes_1 = require("./getEyes");
@@ -24,7 +24,7 @@ exports.expect = test_1.expect.extend({
             // }
         }
         expectTypes(pageOrLocator, ['Page', 'Locator'], 'toHaveScreenshot');
-        const [targetPage, locator] = pageOrLocator.constructor.name === 'Page'
+        const [targetPage, locator] = getApiName(pageOrLocator) === 'Page'
             ? [pageOrLocator, undefined]
             : [pageOrLocator.page(), pageOrLocator];
         // For multi-page support: Always use the main page that has Eyes opened
@@ -140,8 +140,20 @@ exports.expect = test_1.expect.extend({
         };
     },
 });
+/**
+ * Returns the Playwright API name for a given object.
+ * Playwright 1.60+ uses a private `_apiName` property (e.g. `"Page"`, `"Locator"`) instead of
+ * relying on `constructor.name`. This is because the classes are defined as `class _Page` /
+ * `class _Locator`, so `constructor.name` returns `"_Page"` / `"_Locator"` in those versions.
+ * We fall back to `constructor.name` for older Playwright versions that do not expose `_apiName`.
+ */
+function getApiName(receiver) {
+    var _a, _b, _c;
+    return (_c = (_a = receiver === null || receiver === void 0 ? void 0 : receiver._apiName) !== null && _a !== void 0 ? _a : (_b = receiver === null || receiver === void 0 ? void 0 : receiver.constructor) === null || _b === void 0 ? void 0 : _b.name) !== null && _c !== void 0 ? _c : '';
+}
+exports.getApiName = getApiName;
 function expectTypes(receiver, types, matcherName) {
-    if (typeof receiver !== 'object' || !types.includes(receiver.constructor.name)) {
+    if (typeof receiver !== 'object' || !types.includes(getApiName(receiver))) {
         const commaSeparated = types.slice();
         const lastType = commaSeparated.pop();
         const typesString = commaSeparated.length ? commaSeparated.join(', ') + ' or ' + lastType : lastType;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@applitools/eyes-playwright",
-  "version": "1.47.4",
+  "version": "1.47.5",
   "description": "Applitools Eyes SDK for Playwright",
   "keywords": [
     "eyes-playwright",
@@ -63,7 +63,7 @@
     "up:framework": "echo \"$(jq '.devDependencies.playwright = $ENV.APPLITOOLS_FRAMEWORK_VERSION' ./package.json)\" > ./package.json"
   },
   "dependencies": {
-    "@applitools/eyes": "1.42.1",
+    "@applitools/eyes": "1.43.0",
     "@applitools/logger": "2.2.12",
     "@applitools/req": "1.10.2",
     "@applitools/spec-driver-playwright": "1.9.2",

package/types/index.d.ts

@@ -1935,6 +1935,7 @@ export type TestResultsPlain = {
     readonly url?: undefined | string;
     readonly server?: undefined | { eyesServerUrl: string; apiKey: string; proxy?: undefined | ProxySettingsPlain; };
     readonly keepIfDuplicate?: undefined | boolean;
+    readonly isAutoAcceptedInfraUpdate?: undefined | boolean;
 };
 export class TestResults implements Required<TestResultsPlain> {
     get id(): string;
@@ -2036,6 +2037,8 @@ export class TestResults implements Required<TestResultsPlain> {
         useDnsCache?: undefined | boolean;
     };
     get keepIfDuplicate(): boolean;
+    get isAutoAcceptedInfraUpdate(): boolean;
+    getIsAutoAcceptedInfraUpdate(): boolean;
     isPassed(): boolean;
     delete(): Promise<void>;
     deleteSession(): Promise<void>;