supercov 4.0.0 → 4.0.1

package/docs/workspace-isolation.md

@@ -39,10 +39,16 @@ and the instrumented copy is not what you wrote:
 - Biome, oxlint, dprint, cspell and knip run on your source as you wrote it:
   the copy's `node_modules/.bin` starts each of them in a view of the copy
   that holds every file in its original text. A file the tool writes there
-  (a report, a cache) is the command's output like any other. A change it
-  makes to a source file (`--write`, `--fix`) is not applied during a measured
-  run, and the run says so. To have another tool that reads source as text run
-  the same way, name it: `SUPERCOV_SOURCE_TOOLS=typos,stylelint`.
+  (a report, a cache) is the command's output like any other. To have another
+  tool that reads source as text run the same way, name it:
+  `SUPERCOV_SOURCE_TOOLS=typos,stylelint`.
+- A change one of these tools makes to a source file (`prettier --write`,
+  `eslint --fix`, `biome check --write`, `dprint fmt`) is made in your
+  project when the command ends, and a tool later in the same command reads
+  the changed text. The tests ran the copy instrumented before the change, so
+  the run measured the file as it was, says so, and reads as stale; the next
+  run measures the changed file. A file you edited yourself while the command
+  ran keeps your edit, and the run names it.
 - Coverage tools the command runs itself (tap, c8, nyc, Jest's and Vitest's
   `--coverage`) still collect and report coverage. They measure the
   instrumented copy, and report close to what they report without Supercov,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "supercov",
-  "version": "4.0.0",
+  "version": "4.0.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.0.0",
-    "@supercov/cli-darwin-x64": "4.0.0",
-    "@supercov/cli-linux-arm64-gnu": "4.0.0",
-    "@supercov/cli-linux-arm64-musl": "4.0.0",
-    "@supercov/cli-linux-x64-gnu": "4.0.0",
-    "@supercov/cli-linux-x64-musl": "4.0.0",
-    "@supercov/cli-win32-arm64": "4.0.0",
-    "@supercov/cli-win32-x64": "4.0.0"
+    "@supercov/cli-darwin-arm64": "4.0.1",
+    "@supercov/cli-darwin-x64": "4.0.1",
+    "@supercov/cli-linux-arm64-gnu": "4.0.1",
+    "@supercov/cli-linux-arm64-musl": "4.0.1",
+    "@supercov/cli-linux-x64-gnu": "4.0.1",
+    "@supercov/cli-linux-x64-musl": "4.0.1",
+    "@supercov/cli-win32-arm64": "4.0.1",
+    "@supercov/cli-win32-x64": "4.0.1"
   },
   "peerDependencies": {
     "@playwright/test": ">=1.55.0",

package/runtime/javascript/register.mjs

@@ -8,7 +8,7 @@ var __rewriteRelativeImportExtension = (this && this.__rewriteRelativeImportExte
 };
 import Module, { register, syncBuiltinESMExports } from "node:module";
 import fs, { closeSync, openSync, readFileSync, realpathSync, unlinkSync } from "node:fs";
-import { isAbsolute, relative, resolve, sep } from "node:path";
+import { dirname, isAbsolute, relative, resolve, sep } from "node:path";
 import { fileURLToPath } from "node:url";
 import { installLaunchSupervisor, wrapImportedCapability } from "./launchSupervisor.mjs";
 import { __supercovBindCapabilityWrapper } from "./capability.mjs";
@@ -205,6 +205,8 @@ function installAuthoredSourceView() {
         authored = new Set();
     }
     const authoredRoot = fileURLToPath(new URL("./.authored/", import.meta.url));
+    // What a tool of the command wrote to a rewritten file: `--fix`, `--write`.
+    const changedRoot = fileURLToPath(new URL("./.changed/", import.meta.url));
     const workspace = fileURLToPath(new URL("../../", import.meta.url));
     const roots = [...new Set([workspace, (() => {
                 try {
@@ -231,9 +233,28 @@ function installAuthoredSourceView() {
         }
         return undefined;
     };
+    const { existsSync: exists, mkdirSync: makeDirectory } = fs;
+    // A rewritten file reads as the command has it now: what one of its
+    // tools made of it, or what its author wrote.
     const authoredPath = (path) => {
         const local = inside(path);
-        return local !== undefined && authored.has(local) ? resolve(authoredRoot, local) : path;
+        if (local === undefined || !authored.has(local))
+            return path;
+        const changed = resolve(changedRoot, local);
+        return exists(changed) ? changed : resolve(authoredRoot, local);
+    };
+    // And it is written beside the instrumented copy, never over it. Prettier
+    // with `--write` and ESLint with `--fix` replaced the copy with the
+    // source they had read: the file then ran unmeasured, and one whose tests
+    // passed read 0% covered. Supercov gives the project the change when the
+    // command ends.
+    const changedPath = (path) => {
+        const local = inside(path);
+        if (local === undefined || !authored.has(local))
+            return path;
+        const changed = resolve(changedRoot, local);
+        makeDirectory(dirname(changed), { recursive: true });
+        return changed;
     };
     // A recursive listing names nested entries by relative path, or as
     // entries whose parent is inside .supercov.
@@ -250,7 +271,17 @@ function installAuthoredSourceView() {
         ? entries
         : entries.filter((entry) => !hidden(entry));
     const { readFileSync: readSync, readFile: readCallback, readdirSync: listSync, readdir: listCallback } = fs;
-    const { readFile: readPromise, readdir: listPromise } = fs.promises;
+    const { readFile: readPromise, readdir: listPromise, writeFile: writePromise } = fs.promises;
+    const { writeFileSync: writeSync, writeFile: writeCallback } = fs;
+    fs.writeFileSync = function writeFileSync(path, ...rest) {
+        return Reflect.apply(writeSync, this, [changedPath(path), ...rest]);
+    };
+    fs.writeFile = function writeFile(path, ...rest) {
+        return Reflect.apply(writeCallback, this, [changedPath(path), ...rest]);
+    };
+    fs.promises.writeFile = function writeFile(path, ...rest) {
+        return Reflect.apply(writePromise, this, [changedPath(path), ...rest]);
+    };
     fs.readFileSync = function readFileSync(path, ...rest) {
         return Reflect.apply(readSync, this, [authoredPath(path), ...rest]);
     };