supercov 4.2.0 → 4.2.1

package/docs/cli.md

@@ -354,7 +354,17 @@ npx supercov diff <older-run> <newer-run>
 
 `diff` reports gains and losses. Use it after adding a test to prove that the
 expected behavior became covered without an unexplained regression elsewhere.
-Neither input run is changed.
+Neither input run is changed. Under the totals it lists what was lost, marked
+`-`, and then what was gained, marked `+`:
+
+```
+gained: 1 lines, 1 branches, 0 MC/DC conditions
+lost: 1 lines, 1 branches, 0 MC/DC conditions
+- line src/fee.js:5
+- branch src/fee.js:2 false
++ line src/fee.js:3
++ branch src/fee.js:2 true
+```
 
 Two runs of the same code can differ a little: what an end-to-end suite
 reaches depends on timing, on the order of tests, and on state outside the

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "supercov",
-  "version": "4.2.0",
+  "version": "4.2.1",
   "description": "Coverage, security and code quality for coding agents",
   "keywords": [
     "coverage",
@@ -119,14 +119,14 @@
     "test:windows-tls": "node scripts/windows-tls-test.mjs"
   },
   "optionalDependencies": {
-    "@supercov/cli-darwin-arm64": "4.2.0",
-    "@supercov/cli-darwin-x64": "4.2.0",
-    "@supercov/cli-linux-arm64-gnu": "4.2.0",
-    "@supercov/cli-linux-arm64-musl": "4.2.0",
-    "@supercov/cli-linux-x64-gnu": "4.2.0",
-    "@supercov/cli-linux-x64-musl": "4.2.0",
-    "@supercov/cli-win32-arm64": "4.2.0",
-    "@supercov/cli-win32-x64": "4.2.0"
+    "@supercov/cli-darwin-arm64": "4.2.1",
+    "@supercov/cli-darwin-x64": "4.2.1",
+    "@supercov/cli-linux-arm64-gnu": "4.2.1",
+    "@supercov/cli-linux-arm64-musl": "4.2.1",
+    "@supercov/cli-linux-x64-gnu": "4.2.1",
+    "@supercov/cli-linux-x64-musl": "4.2.1",
+    "@supercov/cli-win32-arm64": "4.2.1",
+    "@supercov/cli-win32-x64": "4.2.1"
   },
   "peerDependencies": {
     "@playwright/test": ">=1.55.0",

package/runtime/javascript/playwright.mjs

@@ -198,6 +198,7 @@ class CoveragePhaseController {
     cdpSessions = new Map();
     newDocumentScriptIds = new Map();
     pendingRegistrations = new Set();
+    pageRegistrations = new WeakMap();
     scriptUpdate = Promise.resolve();
     proxyCache = new WeakMap();
     runtimeSnapshots;
@@ -381,8 +382,27 @@ class CoveragePhaseController {
             }
         }
     }
-    async registerPage(page) {
-        if (this.pages.has(page) || this.disposed)
+    /**
+     * 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
+     * before the fixture that hands it to the test has it, so the listener
+     * started the registration and the fixture, finding the page known,
+     * returned at once: the test's first `page.goto` began before the page had
+     * the session a phase is installed through. The document it loaded did not
+     * know its phase, and its hits were left to be placed by their time.
+     */
+    registerPage(page) {
+        if (this.disposed)
+            return Promise.resolve();
+        let registration = this.pageRegistrations.get(page);
+        if (!registration) {
+            registration = this.registerNewPage(page);
+            this.pageRegistrations.set(page, registration);
+        }
+        return registration;
+    }
+    async registerNewPage(page) {
+        if (this.pages.has(page))
             return;
         timingCount("registerPage");
         this.pages.add(page);
@@ -393,8 +413,15 @@ class CoveragePhaseController {
             adopted: !this.contexts.has(context) && tracked && !headersKnown,
         });
         const cdp = await page.context().newCDPSession(page).catch(() => undefined);
-        if (cdp)
+        if (cdp) {
+            // Chromium runs a session's new-document scripts only once the
+            // session has enabled the Page domain. Without it the script that
+            // tells a document its phase was accepted and never ran: a page
+            // learned its phase only from the next action, after it had
+            // loaded, and everything it ran while loading was placed by time.
+            await cdp.send("Page.enable").catch(() => undefined);
             this.cdpSessions.set(page, cdp);
+        }
         const phaseId = this.requestPhaseId();
         if (phaseId)
             await this.activatePage(page, phaseId);