@cevek/screentest 0.3.2 → 0.3.3

package/README.md

@@ -159,12 +159,15 @@ Or pass `--worker-url` / `--token` to `screentest` directly.
 ## CI
 
 ```yaml
-- run: npx screentest serve # builds image + starts daemon
+- run: npx screentest serve # build image + start daemon
 - run: CI=1 npx vitest run --config vitest.screentest.config.ts
 ```
 
-In `CI=1` the orchestrator exits with vitest's code and never tries to
-open the UI.
+In `CI=1` the globalSetup exits with vitest's code and never tries to
+open the UI. Full recipe (GitHub Actions example, docker layer caching
+to skip the ~2 min cold-start build, downloading actuals for offline
+review, pnpm/yarn variants) is in the [CI guide on
+GitHub](https://github.com/x-cevek/screentest/blob/main/docs/ci.md).
 
 ## Input format reference
 

package/dist/global-setup.js

@@ -83,7 +83,8 @@ function countLeaves(doc) {
   for (const g of doc.groups) walk(g.items);
   return n;
 }
-async function generateDoc(projectRoot, snapshotFile, actualDir, docFile, cacheDir) {
+async function generateDoc(projectRoot, snapshotFile, actualDir, docFile, cacheDir, opts = {}) {
+  const detectDeletions = opts.detectDeletions ?? true;
   const snap = await readSnapshot(snapshotFile);
   const expectedByPath = indexSnapshot(snap);
   const actuals = await walkActual(actualDir);
@@ -121,21 +122,24 @@ async function generateDoc(projectRoot, snapshotFile, actualDir, docFile, cacheD
     });
     summary.change++;
   }
-  for (const [key, hash] of expectedByPath) {
-    if (seen.has(key) || !hash) continue;
-    insertLeaf(doc, key.split("/"), {
-      type: "deleted",
-      expectedHash: hash,
-      patchSnapshotJsonFile: snapshotFile
-    });
-    summary.deleted++;
+  if (detectDeletions) {
+    for (const [key, hash] of expectedByPath) {
+      if (seen.has(key) || !hash) continue;
+      insertLeaf(doc, key.split("/"), {
+        type: "deleted",
+        expectedHash: hash,
+        patchSnapshotJsonFile: snapshotFile
+      });
+      summary.deleted++;
+    }
   }
   await fs.mkdir(cacheDir, { recursive: true });
   await fs.writeFile(docFile, JSON.stringify(doc, null, 2));
   const total = countLeaves(doc);
+  const deletedSummary = detectDeletions ? `deleted: ${summary.deleted}, ` : "";
   process.stdout.write(
     `Wrote ${docFile}
-  ${total} diff${total === 1 ? "" : "s"} to review (new: ${summary.new}, change: ${summary.change}, deleted: ${summary.deleted}, unchanged: ${summary.unchanged})
+  ${total} diff${total === 1 ? "" : "s"} to review (new: ${summary.new}, change: ${summary.change}, ${deletedSummary}unchanged: ${summary.unchanged}${detectDeletions ? "" : "; deletion detection disabled \u2014 subset run"})
 `
   );
   return total;
@@ -208,6 +212,40 @@ async function resolveWsEndpoint() {
   );
   return ws;
 }
+function detectVitestFilter() {
+  if (process.env.SCREENTEST_FORCE_DELETION_DETECTION) {
+    return { filtered: false, reason: null };
+  }
+  const argv = process.argv;
+  for (let i = 0; i < argv.length; i++) {
+    const a = argv[i];
+    if (a === "-t" || a === "--testNamePattern") return { filtered: true, reason: a };
+    if (a?.startsWith("--testNamePattern=")) return { filtered: true, reason: a };
+  }
+  const runIdx = argv.indexOf("run");
+  if (runIdx === -1) return { filtered: false, reason: null };
+  const flagsTakingValue = /* @__PURE__ */ new Set([
+    "--config",
+    "-c",
+    "--reporter",
+    "--root",
+    "-r",
+    "--mode",
+    "--inspect-brk",
+    "--inspect",
+    "--coverage.reporter",
+    "--shard"
+  ]);
+  for (let i = runIdx + 1; i < argv.length; i++) {
+    const a = argv[i];
+    if (a.startsWith("-")) {
+      if (flagsTakingValue.has(a)) i += 1;
+      continue;
+    }
+    return { filtered: true, reason: a };
+  }
+  return { filtered: false, reason: null };
+}
 function spawnUI(docFile) {
   const here = dirname(fileURLToPath(import.meta.url));
   const bin = join3(here, "index.js");
@@ -229,6 +267,15 @@ async function setup(project) {
   return async () => {
     if (server) await server.close().catch(() => {
     });
+    const filter = detectVitestFilter();
+    if (filter.filtered) {
+      process.stderr.write(
+        `[screentest] subset run detected (${filter.reason}) \u2014 skipping deletion
+[screentest] detection so untouched tests in snapshot.json are preserved.
+[screentest] Run the full suite without filters to see deletions.
+`
+      );
+    }
     let total = 0;
     try {
       total = await generateDoc(
@@ -236,7 +283,8 @@ async function setup(project) {
         paths.snapshotFile,
         paths.actualDir,
         paths.docFile,
-        paths.cacheDir
+        paths.cacheDir,
+        { detectDeletions: !filter.filtered }
       );
     } catch (err) {
       const msg = err instanceof Error ? err.message : String(err);

package/dist/index.js

@@ -15243,7 +15243,8 @@ function countLeaves(doc) {
   for (const g of doc.groups) walk(g.items);
   return n;
 }
-async function generateDoc(projectRoot, snapshotFile, actualDir, docFile, cacheDir) {
+async function generateDoc(projectRoot, snapshotFile, actualDir, docFile, cacheDir, opts = {}) {
+  const detectDeletions = opts.detectDeletions ?? true;
   const snap = await readSnapshot(snapshotFile);
   const expectedByPath = indexSnapshot(snap);
   const actuals = await walkActual(actualDir);
@@ -15281,21 +15282,24 @@ async function generateDoc(projectRoot, snapshotFile, actualDir, docFile, cacheD
     });
     summary.change++;
   }
-  for (const [key, hash2] of expectedByPath) {
-    if (seen.has(key) || !hash2) continue;
-    insertLeaf(doc, key.split("/"), {
-      type: "deleted",
-      expectedHash: hash2,
-      patchSnapshotJsonFile: snapshotFile
-    });
-    summary.deleted++;
+  if (detectDeletions) {
+    for (const [key, hash2] of expectedByPath) {
+      if (seen.has(key) || !hash2) continue;
+      insertLeaf(doc, key.split("/"), {
+        type: "deleted",
+        expectedHash: hash2,
+        patchSnapshotJsonFile: snapshotFile
+      });
+      summary.deleted++;
+    }
   }
   await fs4.mkdir(cacheDir, { recursive: true });
   await fs4.writeFile(docFile, JSON.stringify(doc, null, 2));
   const total = countLeaves(doc);
+  const deletedSummary = detectDeletions ? `deleted: ${summary.deleted}, ` : "";
   process.stdout.write(
     `Wrote ${docFile}
-  ${total} diff${total === 1 ? "" : "s"} to review (new: ${summary.new}, change: ${summary.change}, deleted: ${summary.deleted}, unchanged: ${summary.unchanged})
+  ${total} diff${total === 1 ? "" : "s"} to review (new: ${summary.new}, change: ${summary.change}, ${deletedSummary}unchanged: ${summary.unchanged}${detectDeletions ? "" : "; deletion detection disabled \u2014 subset run"})
 `
   );
   return total;

package/package.json

@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.3.2",
+  "version": "0.3.3",
   "description": "Local desktop tool for visual screenshot-test review — CLI, runner helpers, and review UI shipped as a single package.",
   "license": "MIT",
   "type": "module",