snapdrift 0.5.0 → 0.5.1

package/CHANGELOG.md

@@ -2,6 +2,17 @@
 
 ## Unreleased
 
+## 0.5.1 - 2026-07-14
+
+### Fixes
+
+- **Stale-capture false-negative guard (#93)** — capture logs now print the fully resolved target URL per route (`... -> https://.../path`), so a `baseUrl` pointing at production instead of the PR preview is visible at a glance. Additionally, `SnapProvider.diff` now emits a warning when a diff run compared more than one route against a real baseline yet every comparison came back pixel-identical (0% drift) — the tell-tale signature of capturing a stale or wrong page rather than a genuine "no change". The guard only warns (never fails), since a PR with no visual change legitimately produces the same shape.
+
+### Dependencies
+
+- Updated `js-yaml` from 4.1.0 to 5.2.1, including ESM import compatibility changes.
+- Updated the pinned `actions/checkout` action from v6.0.3 to v7.0.0.
+
 ## 0.5.0 - 2026-06-08
 
 ### Features

package/lib/init-from-action.mjs

@@ -2,7 +2,7 @@
 
 import fs from 'node:fs/promises';
 import path from 'node:path';
-import yaml from 'js-yaml';
+import * as yaml from 'js-yaml';
 
 /** @typedef {import('../types/visual-diff-types').InitWarning} InitWarning */
 

package/lib/snap-provider.mjs

@@ -870,6 +870,11 @@ export class SnapProvider {
     let changedScreenshots = 0;
     let missingInBaseline = 0;
     let missingInCurrent = 0;
+    // Track real comparisons (a baseline was attached) and how many came back
+    // pixel-identical (0% drift) so we can flag the "captured a stale/wrong
+    // page" pattern below. See issue #93.
+    let comparedWithBaseline = 0;
+    let exactZeroMatches = 0;
 
     /** @type {import('../types/visual-diff-types').VisualDiffChangedItem[]} */
     const changed = [];
@@ -919,6 +924,7 @@ export class SnapProvider {
             location: 'baseline'
           });
         } else if (capture.diffPct != null && capture.diffPct > config.diff.threshold) {
+          comparedWithBaseline++;
           changedScreenshots++;
           changed.push({
             id: capture.routeId || capture.id,
@@ -934,11 +940,35 @@ export class SnapProvider {
             status: 'changed'
           });
         } else {
+          comparedWithBaseline++;
+          if (capture.diffPct === 0) {
+            exactZeroMatches++;
+          }
           matchedScreenshots++;
         }
       }
     }
 
+    // False-negative guard: when a diff run compared every route against a real
+    // baseline yet every single comparison came back pixel-identical (0% drift),
+    // the most likely explanation is that the client captured a stale or wrong
+    // page (baseUrl pointing at production instead of the PR preview, the
+    // preview not being deployed yet, or a caching layer) rather than a genuine
+    // "nothing changed". Warn so a real regression isn't silently missed. We
+    // only warn (never fail): a PR with no visual change legitimately produces
+    // this same shape. We require more than one compared route — a single
+    // pixel-identical route is indistinguishable from an ordinary clean diff,
+    // whereas several independent routes all landing at exactly 0% is the
+    // tell-tale signature of a single stale source page. See issue #93.
+    if (comparedWithBaseline > 1 && exactZeroMatches === comparedWithBaseline) {
+      process.stderr.write(
+        `[SnapDrift] Warning: all ${comparedWithBaseline} compared route(s) were pixel-identical ` +
+        `to the baseline (0% drift). If you expected a visual change, the captured page may be ` +
+        `stale or wrong — verify the resolved baseUrl points at the PR preview and that the ` +
+        `preview was deployed before capture ran. See issue #93.\n`
+      );
+    }
+
     /** @type {import('../types/visual-diff-types').VisualDiffSummary} */
     const summary = {
       startedAt: run.startedAt || new Date().toISOString(),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "snapdrift",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "private": false,
   "type": "module",
   "description": "SnapDrift captures, compares, and reports UI drift for Playwright-based app checks.",
@@ -41,7 +41,7 @@
     "@eslint/js": "^10.0.1",
     "@types/jest": "^30.0.0",
     "@types/js-yaml": "^4.0.9",
-    "@types/node": "^25.3.5",
+    "@types/node": "^26.1.0",
     "@types/pngjs": "^6.0.5",
     "eslint": "^10.2.0",
     "globals": "^17.4.0",
@@ -95,11 +95,11 @@
     "registry": "https://registry.npmjs.org"
   },
   "dependencies": {
-    "@snapdrift/adapter-fs": "^1.0.0",
+    "@snapdrift/adapter-fs": "^1.0.1",
     "@snapdrift/adapter-report-md": "^1.1.0",
     "@snapdrift/compare-core": "^1.0.0",
     "@snapdrift/manifest": "^1.2.0",
-    "js-yaml": "^4.1.0",
+    "js-yaml": "^5.2.1",
     "playwright": "^1.59.1",
     "pngjs": "^7.0.0"
   }