supercov 4.12.0 → 4.12.2

package/docs/cli.md

@@ -699,7 +699,7 @@ rest of the run and leaves their old records out:
 
 ```sh supercov-example
 npx supercov runs <suite-run> tests affected --names   # after changing a mock
-npx supercov -- npx playwright test <those tests>
+npx supercov -- <those tests alone>
 npx supercov merge
 ```
 
@@ -723,7 +723,7 @@ own:
 
 ```yaml
 # Each of two jobs:
-- run: npx supercov -- npx playwright test --shard=${{ matrix.shard }}/2
+- run: npx supercov -- <test command for shard ${{ matrix.shard }}>
 - uses: actions/upload-artifact@v4
   with: { name: "supercov-${{ matrix.shard }}", path: .supercov/runs, include-hidden-files: true }
 # Then one job:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "supercov",
-  "version": "4.12.0",
+  "version": "4.12.2",
   "description": "Coverage, security and code quality for coding agents",
   "keywords": [
     "coverage",
@@ -83,7 +83,7 @@
     "eval:skill": "claude plugin eval plugins/supercov --scaffold --trust-plugin --no-publish --model claude-sonnet-5 --judge-model claude-haiku-4-5 --threshold 0.66 --tag behavior trigger --allow-tools Bash Edit Write --max-cost-usd 15",
     "eval:skill-triggers": "claude plugin eval plugins/supercov --scaffold --trust-plugin --no-publish --model claude-sonnet-5 --judge-model claude-haiku-4-5 --threshold 0.66 --tag trigger --ablation none --max-cost-usd 5",
     "test:engine": "cargo fmt --all -- --check && cargo clippy --workspace --all-targets -- -D warnings && cargo test --workspace && npm run test:runtime && npm run test:rust-assets && npm run test:engine-integrations",
-    "test:engine-integrations": "cargo build -p supercov && node scripts/html-report-integration.mjs && node scripts/rust-process-supervision.mjs && node scripts/rust-direct-node-integration.mjs && node scripts/rust-public-run-integration.mjs && node scripts/rust-embedded-runtime-integration.mjs && node scripts/rust-direct-vitest-integration.mjs && node scripts/rust-direct-jest-integration.mjs && node scripts/mocha-failure-output.mjs && node scripts/failure-output.mjs node ava tap && node scripts/side-by-side.mjs node && node scripts/merge-across.mjs && node scripts/mock-reach.mjs && node scripts/rust-direct-playwright-integration.mjs && node scripts/rust-custom-browser-playwright-integration.mjs && node scripts/cross-origin-requests.mjs && node scripts/leaving-page.mjs && node scripts/background-by-time.mjs && node scripts/python-server-playwright.mjs && node scripts/ruby-server-playwright.mjs && node scripts/go-server-playwright.mjs && node scripts/jvm-server-playwright.mjs && node scripts/docker-server-playwright.mjs && node scripts/rust-classic-browser-integration.mjs && node scripts/rust-top-packages-integration.mjs && node scripts/rust-generic-esbuild-integration.mjs && node scripts/rust-generic-tsc-integration.mjs && node scripts/rust-generic-build-matrix.mjs && node scripts/rust-vite-playwright-integration.mjs",
+    "test:engine-integrations": "cargo build -p supercov && node scripts/html-report-integration.mjs && node scripts/rust-process-supervision.mjs && node scripts/rust-direct-node-integration.mjs && node scripts/rust-public-run-integration.mjs && node scripts/rust-embedded-runtime-integration.mjs && node scripts/rust-direct-vitest-integration.mjs && node scripts/rust-direct-jest-integration.mjs && node scripts/mocha-failure-output.mjs && node scripts/failure-output.mjs node ava tap && node scripts/side-by-side.mjs node && node scripts/merge-across.mjs && node scripts/mock-reach.mjs && node scripts/rust-direct-playwright-integration.mjs && node scripts/rust-custom-browser-playwright-integration.mjs && node scripts/cross-origin-requests.mjs && node scripts/leaving-page.mjs && node scripts/shared-page.mjs && node scripts/background-by-time.mjs && node scripts/python-server-playwright.mjs && node scripts/ruby-server-playwright.mjs && node scripts/go-server-playwright.mjs && node scripts/jvm-server-playwright.mjs && node scripts/docker-server-playwright.mjs && node scripts/rust-classic-browser-integration.mjs && node scripts/rust-top-packages-integration.mjs && node scripts/rust-generic-esbuild-integration.mjs && node scripts/rust-generic-tsc-integration.mjs && node scripts/rust-generic-build-matrix.mjs && node scripts/rust-vite-playwright-integration.mjs",
     "test:platform": "cargo test --workspace && cargo build -p supercov && node scripts/rust-process-supervision.mjs && node scripts/workspace-crash-integration.mjs",
     "test:native-package": "cargo build --release -p supercov && node scripts/native-package-integration.mjs && node scripts/native-release-set-integration.mjs",
     "test:pypi-wheel": "node scripts/pypi-wheel-integration.mjs",
@@ -120,14 +120,14 @@
     "test:windows-tls": "node scripts/windows-tls-test.mjs"
   },
   "optionalDependencies": {
-    "@supercov/cli-darwin-arm64": "4.12.0",
-    "@supercov/cli-darwin-x64": "4.12.0",
-    "@supercov/cli-linux-arm64-gnu": "4.12.0",
-    "@supercov/cli-linux-arm64-musl": "4.12.0",
-    "@supercov/cli-linux-x64-gnu": "4.12.0",
-    "@supercov/cli-linux-x64-musl": "4.12.0",
-    "@supercov/cli-win32-arm64": "4.12.0",
-    "@supercov/cli-win32-x64": "4.12.0"
+    "@supercov/cli-darwin-arm64": "4.12.2",
+    "@supercov/cli-darwin-x64": "4.12.2",
+    "@supercov/cli-linux-arm64-gnu": "4.12.2",
+    "@supercov/cli-linux-arm64-musl": "4.12.2",
+    "@supercov/cli-linux-x64-gnu": "4.12.2",
+    "@supercov/cli-linux-x64-musl": "4.12.2",
+    "@supercov/cli-win32-arm64": "4.12.2",
+    "@supercov/cli-win32-x64": "4.12.2"
   },
   "peerDependencies": {
     "@playwright/test": ">=1.55.0",

package/runtime/javascript/playwright.mjs

@@ -391,6 +391,42 @@ class CoveragePhaseController {
             }
         }
     }
+    /**
+     * Start the pages an earlier test left open (a page a serial describe
+     * opens in `beforeAll`, a worker-scoped one) with no evidence, and return
+     * what they held. Their documents kept every hit since they loaded, and
+     * each later test was credited with what the tests before it ran. What
+     * they held is the earlier tests' (already recorded) and work between
+     * tests, such as a `beforeAll` signing in: no test's. A document that
+     * already has one of this test's phases is left alone: a reset that
+     * arrives late must not drop what this test ran.
+     */
+    async takeCarriedEvidence() {
+        await Promise.all([...this.pendingRegistrations]);
+        const prefix = `${this.scope.attemptId}:`;
+        const take = (target) => bounded(target
+            .evaluate((prefix) => {
+            const coverageGlobal = globalThis;
+            if (String(coverageGlobal.__SUPERCOV_PHASE_ID__ ?? "").startsWith(prefix))
+                return undefined;
+            const getSnapshot = coverageGlobal.__SUPERCOV_COVERAGE_SNAPSHOT__;
+            const reset = coverageGlobal.__SUPERCOV_RESET__;
+            if (!getSnapshot || !reset)
+                return undefined;
+            const snapshot = JSON.stringify(getSnapshot());
+            reset();
+            return snapshot;
+        }, prefix)
+            .then((text) => (typeof text === "string" ? JSON.parse(text) : undefined))
+            .catch(() => undefined), WORKER_ACTIVATION_WAIT_MS);
+        const taken = await Promise.all(this.allPages().flatMap((page) => [
+            ...page.frames().map(take),
+            ...page.workers().map(take),
+        ]));
+        const snapshots = taken.filter((snapshot) => snapshot && snapshot !== WORKER_TIMED_OUT &&
+            (snapshot.hits?.length || snapshot.events?.length || snapshot.decisions?.length));
+        return snapshots.length > 0 ? mergeSnapshots(snapshots) : undefined;
+    }
     /**
      * A page is registered once, and whoever asks while that is under way
      * waits for it. Playwright announces a page to the context's listener
@@ -1458,6 +1494,25 @@ function mergeCollectedPhases(controllerPhases, fallbackPhases) {
     });
     return [...controllerPhases, ...remaining];
 }
+/**
+ * Pool runners may cycle-restore a VM immediately after Playwright exits,
+ * which can discard or overwrite the normal artifact copy. Each record is
+ * written to one uniquely named, one-shot evidence file in the runner's shared
+ * directory as well. No test streams into this path.
+ */
+function appendEvidenceLine(serialized) {
+    const evidenceDirectory = process.env["SUPERCOV_EVIDENCE_DIR"] ??
+        (GENERATED_EVIDENCE_DIRECTORY.startsWith("__")
+            ? undefined
+            : GENERATED_EVIDENCE_DIRECTORY);
+    if (!evidenceDirectory)
+        return;
+    const resolvedDirectory = resolve(process.cwd(), evidenceDirectory);
+    const append = process.env.SUPERCOV_DURABLE_EVIDENCE_EACH_TEST === "1"
+        ? appendJsonLineDurableSync
+        : appendJsonLineSync;
+    append(resolve(resolvedDirectory, `playwright-worker-${evidenceWriterIdentity()}-${process.pid}.mcdc.jsonl`), serialized);
+}
 const instrumentedFixtures = {
     page: async ({ page }, use, testInfo) => {
         const scope = executionScope(testInfo);
@@ -1507,6 +1562,17 @@ const instrumentedFixtures = {
             activeController = controller;
             directRuntime()?.activateCoverageScope(scope);
             await controller.adoptTrackedContexts().catch(() => undefined);
+            const carried = await controller.takeCarriedEvidence().catch(() => undefined);
+            if (carried) {
+                appendEvidenceLine(`${JSON.stringify({
+                    testId: `__supercov_background__:playwright:${scope.attemptId}`,
+                    test: "Playwright work between tests",
+                    status: "unknown",
+                    provenance: { runner: "background", kind: "background", source: "explicit" },
+                    role: "background",
+                    browser: [carried],
+                })}\n`);
+            }
             const serverOutput = serverEvidencePath(scope);
             mkdirSync(serverEvidenceDirectory(scope), { recursive: true });
             rmSync(serverOutput, { force: true });
@@ -1543,22 +1609,7 @@ const instrumentedFixtures = {
                         browser,
                         server,
                     };
-                    const serialized = `${JSON.stringify(payload)}\n`;
-                    // Pool runners may cycle-restore a VM immediately after Playwright
-                    // exits, which can discard or overwrite the normal artifact copy.
-                    // Write one uniquely named, one-shot evidence file to the runner's
-                    // shared directory as well. No test streams into this path.
-                    const evidenceDirectory = process.env["SUPERCOV_EVIDENCE_DIR"] ??
-                        (GENERATED_EVIDENCE_DIRECTORY.startsWith("__")
-                            ? undefined
-                            : GENERATED_EVIDENCE_DIRECTORY);
-                    if (evidenceDirectory) {
-                        const resolvedDirectory = resolve(process.cwd(), evidenceDirectory);
-                        const append = process.env.SUPERCOV_DURABLE_EVIDENCE_EACH_TEST === "1"
-                            ? appendJsonLineDurableSync
-                            : appendJsonLineSync;
-                        append(resolve(resolvedDirectory, `playwright-worker-${evidenceWriterIdentity()}-${process.pid}.mcdc.jsonl`), serialized);
-                    }
+                    appendEvidenceLine(`${JSON.stringify(payload)}\n`);
                 }
                 finally {
                     await timed("controller.dispose", () => controller.dispose());

package/runtime/javascript/runtime.mjs

@@ -451,6 +451,12 @@ function persistBrowser() {
   persistBrowserListenersInstalled = true;
   try {
     addEventListener("beforeunload", handOverBrowser);
+    // An iframe the page removes (a client-side navigation away from the
+    // page that held it, an embed closed) gets no `beforeunload`: what it ran
+    // went with it, though the test clicked in it. Its `pagehide` reaches the
+    // collector, unlike a top-level document's.
+    if (typeof window !== "undefined" && window !== window.top)
+      addEventListener("pagehide", handOverBrowser);
     addEventListener("pagehide", persistBrowserNow);
     addEventListener("visibilitychange", () => {
       if (typeof document !== "undefined" && document.visibilityState === "hidden")