supercov 0.0.52 → 0.0.53

package/README.md

@@ -136,17 +136,23 @@ The denominator comes from source structure before the run, so adding or removin
 
 ## Install for your language
 
-The same binary ships through each language's own package manager, at the same version, from one release:
+Measuring a project should not start by installing another language. The same binary, at the same version, from one release:
 
 ```bash
-npx supercov -- npm test                          # npm
-uvx --from supercov-cli supercov -- pytest        # PyPI
-gem install supercov && supercov -- bundle exec rspec   # RubyGems
-cargo binstall supercov && supercov -- cargo test  # crates.io
+npx supercov -- npm test                                 # npm
+uvx --from supercov-cli supercov -- pytest               # PyPI
+gem install supercov && supercov -- bundle exec rspec    # RubyGems
+cargo binstall supercov && supercov -- cargo test        # crates.io
+go run github.com/supercorp-ai/supercov/cmd/supercov@latest -- go test ./...   # Go
+brew install supercorp-ai/tap/supercov && supercov -- ./gradlew test           # Homebrew
 ```
 
 `pip install supercov-cli` and `gem install supercov` install a wheel or gem that carries the binary for your platform; nothing is compiled. `cargo binstall` downloads that same binary from the GitHub release, while plain `cargo install supercov` builds it from source and needs Rust 1.95.
 
+A Go project needs Go and nothing else. Like any `go run` with a version suffix it resolves by module path and ignores the `go.mod` in your current directory, so it neither needs nor touches your module.
+
+Java and Kotlin have no registry of their own here, so a JVM project takes the binary directly -- Homebrew above, `npx` if Node is already present, or the platform archive from the [latest release](https://github.com/supercorp-ai/supercov/releases/latest). Maven and Gradle are driven as your test command, not as a plugin. Supercov adds the JUnit Platform launcher its measurement needs to the build file inside its own isolated workspace copy -- your `pom.xml` or `build.gradle` is never edited.
+
 ## Supported languages
 
 | Language | Status | Start with |
@@ -156,7 +162,7 @@ cargo binstall supercov && supercov -- cargo test  # crates.io
 | Rust | Available | `npx supercov -- cargo test` |
 | Python | Available | `npx supercov -- pytest` |
 | Ruby | Available | `npx supercov -- rspec` |
-| Go | Available | `npx supercov -- go test ./...` |
+| Go | Available | `go run github.com/supercorp-ai/supercov/cmd/supercov@latest -- go test ./...` |
 | Java | Available | `npx supercov -- mvn test` |
 | Kotlin | Available | `npx supercov -- ./gradlew test` |
 | Zig | Coming soon | — |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "supercov",
-  "version": "0.0.52",
+  "version": "0.0.53",
   "description": "Coverage for coding agents and software factories \ud83c\udf19",
   "license": "MIT",
   "repository": {
@@ -97,14 +97,14 @@
     "test:launcher": "go vet ./cmd/... && go test ./cmd/..."
   },
   "optionalDependencies": {
-    "@supercov/cli-darwin-arm64": "0.0.52",
-    "@supercov/cli-darwin-x64": "0.0.52",
-    "@supercov/cli-linux-arm64-gnu": "0.0.52",
-    "@supercov/cli-linux-arm64-musl": "0.0.52",
-    "@supercov/cli-linux-x64-gnu": "0.0.52",
-    "@supercov/cli-linux-x64-musl": "0.0.52",
-    "@supercov/cli-win32-arm64": "0.0.52",
-    "@supercov/cli-win32-x64": "0.0.52"
+    "@supercov/cli-darwin-arm64": "0.0.53",
+    "@supercov/cli-darwin-x64": "0.0.53",
+    "@supercov/cli-linux-arm64-gnu": "0.0.53",
+    "@supercov/cli-linux-arm64-musl": "0.0.53",
+    "@supercov/cli-linux-x64-gnu": "0.0.53",
+    "@supercov/cli-linux-x64-musl": "0.0.53",
+    "@supercov/cli-win32-arm64": "0.0.53",
+    "@supercov/cli-win32-x64": "0.0.53"
   },
   "peerDependencies": {
     "@playwright/test": ">=1.55.0",

package/runtime/javascript/runtime.mjs

@@ -751,8 +751,19 @@ function withNodeAssertionPhase(operation, source, callback) {
   const scope = context.scope;
   if (!scope)
     return callback();
-  const existing = context.phaseId && assertionPhaseState(scope).phaseIds.has(context.phaseId);
-  if (existing)
+  // A lexically instrumented occurrence is an authored assertion site in its
+  // own right, even when it runs inside another assertion's callback: the
+  // validator passed to assert.throws is the ordinary case, and its inner
+  // assertions are what claim *what* the error says. Skipping every nested
+  // phase left those with no passing occurrence, so they could never earn
+  // credit however carefully they were mapped.
+  //
+  // What must still be skipped is the same call seen twice -- the module proxy
+  // firing inside the lexical wrapper. That path has no lexical source and
+  // arrives as a lazy stack fallback, so the shape of `source` tells them
+  // apart without comparing coordinates the transform has moved.
+  const nested = context.phaseId && assertionPhaseState(scope).phaseIds.has(context.phaseId);
+  if (nested && typeof source !== "string")
     return callback();
   // A lexical occurrence already identifies its source. Stack fallback is lazy
   // and qualified so transformed coordinates cannot impersonate original ones.
@@ -1244,20 +1255,35 @@ function tryBegin(successId, catchId) {
   return { successId, catchId, caught: false };
 }
 function tryCatch(frame, value) {
-  frame.caught = true;
+  // Record the outcome the moment it is known, not when the construct is left.
+  // The commit used to sit in the generated `finally`, and `process.exit()` in
+  // a catch body skips `finally` -- so a catch that ran and whose statements
+  // were credited was still reported as never entered. Statement probes fire
+  // in place and survived, which is what made the report contradict itself.
+  if (!frame.caught) {
+    frame.caught = true;
+    coverageHit(frame.catchId);
+  }
   return value;
 }
 function tryEnd(frame) {
-  coverageHit(frame.caught ? frame.catchId : frame.successId);
+  // Completing without catching is only knowable here.
+  if (!frame.caught) coverageHit(frame.successId);
 }
 function loopBegin(zeroId, enteredId) {
   return { zeroId, enteredId, entered: false };
 }
 function loopEntered(frame) {
-  frame.entered = true;
+  // Same reason as tryCatch: a loop body that exits the process would
+  // otherwise report zero iterations. Guarded so a loop still counts once,
+  // however many times it iterates.
+  if (!frame.entered) {
+    frame.entered = true;
+    coverageHit(frame.enteredId);
+  }
 }
 function loopEnd(frame) {
-  coverageHit(frame.entered ? frame.enteredId : frame.zeroId);
+  if (!frame.entered) coverageHit(frame.zeroId);
 }
 function mcdcBegin(id, meta) {
   if (!state.decisions.has(id)) {