coverage-check 0.2.2 → 0.2.3

package/dist/src/commands/check.mjs

@@ -35,6 +35,7 @@ export async function runCheck(args) {
     const stripPrefixes = buildStripPrefixes(args.stripPrefixes);
     const reports = [];
     const suiteSources = [];
+    const parsedSources = [];
     if (args.store !== null) {
         const suites = await args.store.list();
         for (const suite of suites) {
@@ -45,6 +46,7 @@ export async function runCheck(args) {
                 const lcov = parseLcov(buf.toString("utf8"), stripPrefixes);
                 reports.push(lcov);
                 suiteSources.push({ suite, source: "store", lcov });
+                parsedSources.push({ name: `suite '${suite}'`, lcov });
             }
         }
     }
@@ -54,6 +56,7 @@ export async function runCheck(args) {
         const lcov = parseLcov(readFileSync(f, "utf8"), stripPrefixes);
         reports.push(lcov);
         freshLcovs.push(lcov);
+        parsedSources.push({ name: `file '${f}'`, lcov });
     }
     if (freshLcovs.length > 0) {
         suiteSources.push({
@@ -75,6 +78,27 @@ export async function runCheck(args) {
         stderr(`coverage-check: git diff failed: ${err}`);
         return 2;
     }
+    if (diff.size > 0) {
+        for (const { name, lcov: sourceLcov } of parsedSources) {
+            let contributes = false;
+            for (const [file, changedLines] of diff) {
+                const fileLines = sourceLcov.get(file);
+                if (fileLines) {
+                    for (const lineNo of changedLines) {
+                        if (fileLines.has(lineNo)) {
+                            contributes = true;
+                            break;
+                        }
+                    }
+                }
+                if (contributes)
+                    break;
+            }
+            if (!contributes) {
+                stderr(`coverage-check: warning: coverage from ${name} contributed 0 coverable lines to the patch result. This may indicate a path prefix mismatch.`);
+            }
+        }
+    }
     const { buckets, informational } = computePatchCoverage(diff, lcov, rules);
     const passed = buckets.every((b) => b.passed);
     const result = { buckets, informational, passed };

package/dist/src/lcov-parser.mjs

@@ -11,13 +11,21 @@ export function parseLcov(text, stripPrefixes = []) {
         const line = raw.trimEnd();
         if (line.startsWith("SF:")) {
             let path = line.slice(3);
+            let stripped = false;
             for (const prefix of stripPrefixes) {
                 if (path.startsWith(prefix)) {
                     path = path.slice(prefix.length);
+                    stripped = true;
                     break;
                 }
             }
             path = normalizePath(path);
+            if (!stripped && (path.startsWith("/") || /^[A-Z]:\//i.test(path))) {
+                const match = path.match(/^.*?\/_?work\/([^/]+)\/\1\//);
+                if (match) {
+                    path = path.slice(match[0].length);
+                }
+            }
             currentLines = result.get(path) ?? new Map();
             result.set(path, currentLines);
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "coverage-check",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "description": "Patch-coverage gate: checks that newly added lines meet per-path coverage thresholds. Supports per-suite LCOV accumulation for conditional CI.",
   "license": "MIT",
   "author": "Jonathan Ong",