supercov 4.8.0 → 4.8.1

package/docs/coverage-model.md

@@ -96,16 +96,7 @@ Both are useful:
 
 Code that runs while no test is running counts in the run's total and in no
 test's: a server starting before the first test, modules loading, work between
-tests.
-
-A request to your app server that names no test -- a webhook from a mock
-Stripe server Supercov does not measure -- is credited to the test running when
-the server handled it, if that test was the only one running then. A mock that
-runs under Supercov passes the test's identity on in its own requests, and its
-webhook is credited exactly. The run's
-`Attribution` line counts those tests as `with server work credited by time`,
-and each one's `runs <id> test` page says so. Work done while tests ran at
-once, or while none ran, stays no test's. When a run has any, the summary's `By test kind` table ends with a
+tests. When a run has any, the summary's `By test kind` table ends with a
 `no test` row for what only that covered, which is why a run with one kind of
 test can show a total above that kind's own:
 
@@ -119,9 +110,22 @@ By test kind
 directory has any, so a directory's `All` never stands above its kinds
 unexplained.
 
+A request to your app server that names no test -- a webhook from a mock
+Stripe server Supercov does not measure -- is credited to the test running when
+the server handled it, if that test was the only one running then. Where a
+server serves one worker's tests, as a server started per worker does, only
+that worker's tests count, so the request is credited while other workers run
+theirs. A mock that runs under Supercov passes the test's identity on in its
+own requests, and its webhook is credited exactly. The run's `Attribution`
+line counts tests credited by time as `with server work credited by time`, and
+each one's `runs <id> test` page says so. Work done while tests ran at once on
+the same server, or while none ran, stays no test's.
+
 A test that made assertions and has no covered line to its name is counted in
 a warning on the summary. `npx supercov runs latest tests without-evidence`
-lists every such test, with its file, kind and outcome.
+lists every such test, with its file, kind and outcome. Where its file imports
+no measured source, it says so: such a test can only have checked test code,
+mocks or fixtures.
 
 See [Supported suites](supported-suites.md) for the attribution available from
 each runner.

package/package.json

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

package/runtime/javascript/runtime.mjs

@@ -467,6 +467,25 @@ function serverRecordKey(record) {
   const request = record.request ? `request-${record.request}:` : "";
   return `${request}${(_a8 = record.phaseId) != null ? _a8 : "unscoped"}:${record.type}:${suffix}`;
 }
+// Notes, once per test attempt, that this process served a request naming
+// it. A request here that names no test is credited by time only among the
+// attempts this process served: with a server per worker, the worker's test,
+// whatever the other workers run meanwhile.
+function noteServedAttempt(scope) {
+  if (!state.servedAttempts)
+    state.servedAttempts = /* @__PURE__ */ new Set();
+  const key = attemptKey(scope);
+  if (state.servedAttempts.has(key))
+    return;
+  state.servedAttempts.add(key);
+  const fs = getFs();
+  if (!fs)
+    return;
+  try {
+    appendDurableBackgroundRecord(fs, scope.runId, { type: "served", id: scope.attemptId, timestampMs: Date.now() });
+  } catch (e) {
+  }
+}
 // Numbers the requests a server handles that name no test.
 function nextUnscopedRequest() {
   state.unscopedRequests = (state.unscopedRequests || 0) + 1;
@@ -1400,6 +1419,8 @@ function installNodeRequestPropagation() {
       // the server process were created by a shared before hook.
       const incoming = requestCoverageContext(args[0]);
       if (incoming) context = incoming.scope ? incoming : context != null ? context : currentRequestContext();
+      if (incoming && incoming.scope)
+        noteServedAttempt(incoming.scope);
       // A request that names no test, to a server no test owns, is numbered:
       // the report credits its work to the test running when it came.
       if (incoming && serverPhaseStorage && !(context && (context.scope || context.phaseId)))