@applitools/eyes-cypress 3.60.8 → 3.60.10

package/CHANGELOG.md

@@ -1,5 +1,93 @@
 # Changelog
 
+## [3.60.10](https://github.com/Applitools-Dev/sdk/compare/js/eyes-cypress@3.60.9...js/eyes-cypress@3.60.10) (2026-07-21)
+
+
+### Bug Fixes
+
+* revert per-test diff-attribution afterEach getResults | FLD-4628 ([#4001](https://github.com/Applitools-Dev/sdk/issues/4001)) ([9bb8811](https://github.com/Applitools-Dev/sdk/commit/9bb88118532ed60ea985e77f7d93e147a72caa12))
+
+
+### Dependencies
+
+* @applitools/dom-shared bumped to 1.2.2
+  #### Bug Fixes
+
+  * CSP-safe sandbox opt-out for the MV3 extension content script | AD-14808 ([#3969](https://github.com/Applitools-Dev/sdk/issues/3969)) ([772aefa](https://github.com/Applitools-Dev/sdk/commit/772aefaa3e2bc1eb345fa8d3f6c91ad0913f82e7))
+* @applitools/core-base bumped to 1.36.2
+  #### Bug Fixes
+
+  * guard logEventBatch against a logger-less event ([#4007](https://github.com/Applitools-Dev/sdk/issues/4007)) ([bb8c9b2](https://github.com/Applitools-Dev/sdk/commit/bb8c9b242801e08dd573a77e261d5fb4540cd68e))
+* @applitools/dom-snapshot bumped to 4.17.6
+
+* @applitools/snippets bumped to 2.9.4
+  #### Bug Fixes
+
+  * bound lazyLoad scroll loop to prevent poll-timeout abort | FLD-4680 ([#3972](https://github.com/Applitools-Dev/sdk/issues/3972)) ([4deea6b](https://github.com/Applitools-Dev/sdk/commit/4deea6b1f6f8ec53bd989cd6bbd021cac2ee3c1e))
+
+
+
+* @applitools/dom-capture bumped to 11.8.4
+
+* @applitools/driver bumped to 1.26.6
+
+* @applitools/spec-driver-webdriver bumped to 1.6.6
+
+* @applitools/spec-driver-selenium bumped to 1.8.6
+
+* @applitools/spec-driver-playwright bumped to 1.9.6
+
+* @applitools/spec-driver-puppeteer bumped to 1.8.6
+
+* @applitools/screenshoter bumped to 3.12.25
+
+* @applitools/nml-client bumped to 1.11.36
+
+* @applitools/ec-client bumped to 1.12.38
+
+* @applitools/core bumped to 4.66.2
+  #### Bug Fixes
+
+  * bound lazyLoad scroll loop to prevent poll-timeout abort | FLD-4680 ([#3972](https://github.com/Applitools-Dev/sdk/issues/3972)) ([4deea6b](https://github.com/Applitools-Dev/sdk/commit/4deea6b1f6f8ec53bd989cd6bbd021cac2ee3c1e))
+  * send the old-infra retry in its check's slot | AD-14675 ([#3968](https://github.com/Applitools-Dev/sdk/issues/3968)) ([946b926](https://github.com/Applitools-Dev/sdk/commit/946b92632a511f1ed23f2f65d0382ef8d7177ea7))
+
+
+
+* @applitools/core-universal bumped to 1.0.24
+
+* @applitools/eyes bumped to 1.43.7
+
+
+## [3.60.9](https://github.com/Applitools-Dev/sdk/compare/js/eyes-cypress@3.60.8...js/eyes-cypress@3.60.9) (2026-07-08)
+
+
+### Dependencies
+
+* @applitools/core-base bumped to 1.36.1
+  #### Bug Fixes
+
+  * forward Exact match-level tuning to the server across JS + Java | FLD-4685 ([#3974](https://github.com/Applitools-Dev/sdk/issues/3974)) ([fea351c](https://github.com/Applitools-Dev/sdk/commit/fea351c76c80f63bf658d9573b3d34e0537e0cce))
+* @applitools/eyes bumped to 1.43.6
+  #### Bug Fixes
+
+  * forward Exact match-level tuning to the server across JS + Java | FLD-4685 ([#3974](https://github.com/Applitools-Dev/sdk/issues/3974)) ([fea351c](https://github.com/Applitools-Dev/sdk/commit/fea351c76c80f63bf658d9573b3d34e0537e0cce))
+
+
+
+* @applitools/nml-client bumped to 1.11.35
+
+* @applitools/ec-client bumped to 1.12.37
+
+* @applitools/core bumped to 4.66.1
+  #### Bug Fixes
+
+  * stop UFG old-infra retry after a retry fails | AD-14675 ([#3970](https://github.com/Applitools-Dev/sdk/issues/3970)) ([1531893](https://github.com/Applitools-Dev/sdk/commit/1531893cc9c098f3ade329ceae7164600c184fa5))
+
+
+
+* @applitools/core-universal bumped to 1.0.23
+
+
 ## [3.60.8](https://github.com/Applitools-Dev/sdk/compare/js/eyes-cypress@3.60.7...js/eyes-cypress@3.60.8) (2026-06-28)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@applitools/eyes-cypress",
-  "version": "3.60.8",
+  "version": "3.60.10",
   "homepage": "https://applitools.com/tutorials/sdks/cypress",
   "license": "SEE LICENSE IN LICENSE",
   "main": "./index.js",
@@ -52,9 +52,9 @@
     "setup": "run --top-level xvfb:setup"
   },
   "dependencies": {
-    "@applitools/core": "4.66.0",
-    "@applitools/core-universal": "1.0.22",
-    "@applitools/eyes": "1.43.5",
+    "@applitools/core": "4.66.2",
+    "@applitools/core-universal": "1.0.24",
+    "@applitools/eyes": "1.43.7",
     "@applitools/functional-commons": "1.6.0",
     "@applitools/logger": "2.3.0",
     "@applitools/utils": "1.15.1",

package/src/browser/commands.js

@@ -1,4 +1,4 @@
-/* global Cypress,cy,after,afterEach */
+/* global Cypress,cy,after */
 'use strict'
 const spec = require('../../dist/browser/spec-driver')
 const Refer = require('../../dist/browser/refer')
@@ -26,39 +26,10 @@ let manager,
   _summary,
   connectedToUniversal,
   openAndGlobalConfig,
-  isOpen = false,
-  // Per-test diff-attribution state: did the current test open Eyes, and has it already been checked.
-  eyesOpenedInTest = false,
-  currentTestHandled = false
+  isOpen = false
 
 const deleteTest = options => socket.request('Core.deleteTest', options)
 
-const DIFFS_OR_ERRORS_MESSAGE = 'Eyes-Cypress detected diffs or errors'
-
-// When per-test attribution is on, throws on a diff/error from within the current test's own execution
-// context so Cypress blames the test that produced it. Called from eyesClose() (after it closes Eyes)
-// and from the afterEach() hook; currentTestHandled makes it run once per test. It does NOT close Eyes —
-// closing would change abort / duplicate-removal semantics for tests that intentionally leave Eyes open,
-// and the results are available from the checks alone.
-async function throwOnDiffForCurrentTest() {
-  if (currentTestHandled || !isOpen) return
-  currentTestHandled = true
-
-  if (!perTestDiffAttribution) return
-
-  // Awaiting the results blocks the test until its UFG render completes — required for correct attribution.
-  const results = await socket.request('Eyes.getResults', {eyes, settings: {throwErr: false}})
-  if (results && results.length > 0) {
-    // TAP files are written once in the after() hook, not here.
-    const resultConfig = makeResultConfig({shouldCreateTapFile: false})
-    const message = await socket.request('Test.printTestResults', {testResults: results, resultConfig})
-    printToLog(`Eyes: per-test diff check message: ${message}`)
-    if (message && message.includes(DIFFS_OR_ERRORS_MESSAGE)) {
-      throw new Error(message)
-    }
-  }
-}
-
 async function getSummary() {
   if (_summary) return _summary
   await Promise.all(closePromiseArr)
@@ -96,10 +67,6 @@ const shouldCallAfterHook =
 
 const notFailCypressAfterAllSpecs = !Cypress.config('appliConfFile').failCypressAfterAllSpecs
 
-// True when diffs are attributed per-test (thrown from the test's own context) instead of batched into
-// the spec-level after() hook.
-const perTestDiffAttribution = Cypress.config('eyesFailCypressOnDiff') && notFailCypressAfterAllSpecs
-
 Cypress.Commands.add('eyesGetAllTestResults', () => {
   Cypress.log({name: 'Eyes: getAllTestResults'})
   printToLog(
@@ -145,26 +112,12 @@ if (shouldCallAfterHook) {
         isCurrentTestDisabled = false
         return
       }
-      // Always finalize results and run post-spec tasks (e.g. writing TAP files) here. With per-test
-      // attribution on, the afterEach() hook below already throws diffs against the correct test, so
-      // suppress the throw here — throwing in this spec-level after() hook would re-blame the last test.
       const summary = await getSummary()
-      await throwErrorIfExistsInSummary(summary, Cypress.config('eyesFailCypressOnDiff'), false, perTestDiffAttribution)
+      await throwErrorIfExistsInSummary(summary, Cypress.config('eyesFailCypressOnDiff'), false)
     })
   })
 }
 
-// With per-test attribution on, check each test's Eyes from an afterEach() hook — which runs in that
-// test's own context — so a diff is blamed on the test that produced it. Covers the common case where
-// the test relies on the SDK's automatic post-spec close. (Does not close Eyes — see throwOnDiffForCurrentTest.)
-if (perTestDiffAttribution) {
-  afterEach(() => {
-    if (!eyesOpenedInTest) return
-    eyesOpenedInTest = false
-    return cy.then({timeout: 86400000}, () => throwOnDiffForCurrentTest())
-  })
-}
-
 let isCurrentTestDisabled
 
 Cypress.Commands.add('eyesOpen', function (args = {}) {
@@ -216,8 +169,6 @@ Cypress.Commands.add('eyesOpen', function (args = {}) {
 
     eyes = await socket.request('EyesManager.openEyes', {manager, target, config: openAndGlobalConfig})
     isOpen = true
-    eyesOpenedInTest = true
-    currentTestHandled = false
   })
 })
 
@@ -240,25 +191,22 @@ Cypress.Commands.add('eyesCheckWindow', (args = {}) => {
 })
 
 Cypress.Commands.add('eyesClose', () => {
-  return cy.then({timeout: 86400000}, async () => {
+  return cy.then({timeout: 86400000}, () => {
+    if (isCurrentTestDisabled) return
+    if (!isOpen) throw new EyesNotOpenError('eyesClose')
+
+    Cypress.log({name: 'Eyes: close'})
     if (isCurrentTestDisabled) {
       isCurrentTestDisabled = false
       return
     }
-    if (!isOpen) throw new EyesNotOpenError('eyesClose')
 
-    Cypress.log({name: 'Eyes: close'})
-
-    // Eyes.close in core no longer waits on results, so await it explicitly.
+    // Eyes.close in core is not waiting on results anymore. So we should return it in order to await it
     const p = socket.request('Eyes.close', {eyes, config: openAndGlobalConfig}).catch(err => {
       console.log('Error in cy.eyesClose', err)
     })
     closePromiseArr.push(p)
-    await p
-
-    // Run the per-test diff check inside the test's own context (the afterEach() hook does the same for
-    // tests that don't call eyesClose(); currentTestHandled prevents doing it twice).
-    await throwOnDiffForCurrentTest()
+    return p
   })
 })
 
@@ -295,24 +243,15 @@ function isNonVisualTestError(testResult) {
   return result
 }
 
-function makeResultConfig({shouldThrowError = true, shouldCreateTapFile}) {
-  return {
+async function throwErrorIfExistsInSummary(summary, shouldThrowError, shouldDisplayCriticalErrors) {
+  const resultConfig = {
     showLogs: Cypress.config('appliConfFile').showLogs,
-    shouldThrowError,
+    shouldThrowError: shouldThrowError,
     isTextTerminal: Cypress.config('isTextTerminal'),
     tapDirPath: Cypress.config('appliConfFile').tapDirPath,
     tapFileName: Cypress.config('appliConfFile').tapFileName,
-    shouldCreateTapFile,
+    shouldCreateTapFile: shouldDoPostSpecTasks,
   }
-}
-
-async function throwErrorIfExistsInSummary(
-  summary,
-  shouldThrowError,
-  shouldDisplayCriticalErrors,
-  suppressDiffThrow = false,
-) {
-  const resultConfig = makeResultConfig({shouldThrowError, shouldCreateTapFile: shouldDoPostSpecTasks})
   const testResults = summary.results?.map(({result}) => result) ?? []
   const filteredTestResults = shouldDisplayCriticalErrors
     ? testResults.filter(testResult => testResult.error && isNonVisualTestError(testResult))
@@ -324,10 +263,9 @@ async function throwErrorIfExistsInSummary(
     }, results message: ${message}`,
   )
   if (
-    (!suppressDiffThrow &&
-      !!getGlobalConfigProperty('eyesFailCypressOnDiff') &&
+    (!!getGlobalConfigProperty('eyesFailCypressOnDiff') &&
       message &&
-      message.includes(DIFFS_OR_ERRORS_MESSAGE)) ||
+      message.includes('Eyes-Cypress detected diffs or errors')) ||
     (shouldDisplayCriticalErrors && filteredTestResults.length > 0 && message)
   ) {
     printToLog('Eyes: throwErrorIfExistsInSummary throwing error')

package/types/expose.d.ts

@@ -123,6 +123,7 @@ export type CypressCheckSettings = {
     name?: string;
     region?: LegacyRegion;
     matchLevel?: "None" | "Layout1" | "Layout" | "Layout2" | "Content" | "IgnoreColors" | "Strict" | "Dynamic" | "Exact";
+    exact?: { minDiffIntensity?: number; minDiffWidth?: number; minDiffHeight?: number; matchThreshold?: number; };
     useDom?: boolean;
     sendDom?: boolean;
     enablePatterns?: boolean;
@@ -296,7 +297,7 @@ export type CypressEyesConfig = {
     cut?: { top: number; right: number; bottom: number; left: number; } | { x: number; y: number; width: number; height: number; };
     debugScreenshots?: { save: boolean; path?: string; prefix?: string; };
     defaultMatchSettings?: {
-        exact?: { minDiffIntensity: number; minDiffWidth: number; minDiffHeight: number; matchThreshold: number; };
+        exact?: { minDiffIntensity?: number; minDiffWidth?: number; minDiffHeight?: number; matchThreshold?: number; };
         matchLevel?: "None" | "Layout1" | "Layout" | "Layout2" | "Content" | "IgnoreColors" | "Strict" | "Dynamic" | "Exact";
         ignoreCaret?: boolean;
         useDom?: boolean;
@@ -447,7 +448,7 @@ export type appliConfFile = {
     cut?: { top: number; right: number; bottom: number; left: number; } | { x: number; y: number; width: number; height: number; };
     debugScreenshots?: { save: boolean; path?: string; prefix?: string; };
     defaultMatchSettings?: {
-        exact?: { minDiffIntensity: number; minDiffWidth: number; minDiffHeight: number; matchThreshold: number; };
+        exact?: { minDiffIntensity?: number; minDiffWidth?: number; minDiffHeight?: number; matchThreshold?: number; };
         matchLevel?: "None" | "Layout1" | "Layout" | "Layout2" | "Content" | "IgnoreColors" | "Strict" | "Dynamic" | "Exact";
         ignoreCaret?: boolean;
         useDom?: boolean;