@tested/cli 0.1.0 → 0.1.1

package/README.md

@@ -12,7 +12,7 @@ Node >= 20.19.
 ```bash
 pnpm add -D @tested/cli
 # or
-npx @tested/cli
+npx @tested/cli          # runs `tested` (`td` is the same CLI after install)
 ```
 
 CI uses the composite Action (`tested-hq/cli/action@main`). The Action clones this repo and builds the CLI.
@@ -110,6 +110,18 @@ $ echo $?
 1
 ```
 
+If the patch has no executable lines in scope (tests-only, comments-only, docs-only, or ignored files), the patch gate is **skipped** — not reported as 0% coverage. Project still applies.
+
+```
+$ tested check
+tested.dev — coverage gate  [PASS]
+
+  Patch     -  no executable lines in the patch  [SKIP]
+  Project   64.1%  (threshold 60)  [PASS]
+
+No executable lines in the patch — patch gate skipped. Project threshold met.
+```
+
 If `thresholds` is missing from `.tested.yaml`, `tested check` prints a notice on stderr and exits 0 — configs that haven't opted in stay green.
 
 ### GitHub Actions

package/dist/td.js

@@ -44,6 +44,8 @@ function badge(kind) {
       return pc.yellow("[WARN]");
     case "info":
       return pc.cyan("[INFO]");
+    case "skip":
+      return pc.cyan("[SKIP]");
   }
 }
 function metricBar(pct3, width = 10) {
@@ -179,6 +181,8 @@ function buildInitYaml(args) {
   if (args.testRunner) {
     lines.push(`testRunner: ${args.testRunner}`);
   }
+  lines.push("# Patch gate is skipped when the diff has no executable lines");
+  lines.push("# (tests-only, comments-only, docs-only, or ignored files).");
   lines.push("thresholds:");
   lines.push("  patch: 80");
   lines.push("  project: 60");
@@ -545,7 +549,9 @@ var FileCoverageSchema = z2.object({
 var CoverageTotalsSchema = z2.object({
   executable: z2.number().int().nonnegative(),
   covered: z2.number().int().nonnegative(),
-  pct: z2.number().min(0).max(100)
+  pct: z2.number().min(0).max(100),
+  /** Present when executable === 0 — not a 0% coverage result. */
+  empty: z2.literal(true).optional()
 });
 var ProjectTotalsSchema = CoverageTotalsSchema.extend({
   delta: z2.number().nullable()
@@ -614,7 +620,14 @@ async function headSha(ctx) {
   return (await ctx.git.revparse(["HEAD"])).trim();
 }
 async function unifiedDiff(ctx, base) {
-  return ctx.git.diff([`${base}...HEAD`]);
+  try {
+    const mergeBase = (await ctx.git.raw(["merge-base", base, "HEAD"])).trim();
+    if (mergeBase) {
+      return ctx.git.diff([`${base}...HEAD`]);
+    }
+  } catch {
+  }
+  return ctx.git.diff([base, "HEAD"]);
 }
 async function remoteUrl(ctx, remote = "origin") {
   return (await ctx.git.raw(["remote", "get-url", remote])).trim();
@@ -763,6 +776,10 @@ function assertWithinRoot(root, resolvedPath) {
 }
 
 // src/core/patch.ts
+var EMPTY_PATCH_REASON = "no executable lines in the patch";
+function isEmptyPatch(totals) {
+  return totals.executable === 0;
+}
 function pct(covered, executable) {
   if (executable === 0) return 0;
   return Math.round(covered / executable * 1e3) / 10;
@@ -862,10 +879,10 @@ function uncoveredRanges(file) {
 function buildDiffOutput(args) {
   const patch = computePatchCoverage(args.files, args.addedByFile);
   const project = computeProjectCoverage(args.files);
-  const fileNames = /* @__PURE__ */ new Set([
-    ...patch.byFile.keys(),
-    ...project.byFile.keys()
-  ]);
+  const emptyPatch = isEmptyPatch(patch.totals);
+  const fileNames = new Set(
+    emptyPatch ? [] : [...patch.byFile.keys(), ...project.byFile.keys()]
+  );
   const files = [];
   for (const name of fileNames) {
     const file = args.files.find((f) => f.path === name);
@@ -882,7 +899,7 @@ function buildDiffOutput(args) {
     schemaVersion: 1,
     base: args.base,
     head: args.head,
-    patch: patch.totals,
+    patch: emptyPatch ? { ...patch.totals, empty: true } : patch.totals,
     project: { ...project.totals, delta: args.projectDelta ?? null },
     files,
     ignored: [...args.ignored]
@@ -1865,8 +1882,6 @@ function buildCiSnippet() {
     "    runs-on: ubuntu-latest",
     "    steps:",
     "      - uses: actions/checkout@v4",
-    "        with:",
-    "          fetch-depth: 0",
     "      - uses: tested-hq/cli/action@main",
     "        with:",
     "          # pin ref for reproducible installs (do not use floating tags in prod)",
@@ -1931,6 +1946,12 @@ function formatSetupHuman(opts) {
     ])
   );
   lines.push("");
+  lines.push(
+    dim(
+      "Patch gate is skipped when the diff has no executable lines (tests-only, docs, comments)."
+    )
+  );
+  lines.push("");
   lines.push(tip("re-check anytime: tested doctor"));
   lines.push("");
   return lines.join("\n");
@@ -2196,8 +2217,8 @@ function coloredPctCell(pct3, width = 6) {
 function formatMetricRow(label, pct3, covered, executable, emptyNote) {
   const labelPad = label.padEnd(8);
   if (executable === 0) {
-    const note = emptyNote ?? "no executable lines";
-    return `  ${labelPad}  ${dim("-".padEnd(6))}  ${dim(note)}`;
+    const note = emptyNote ?? EMPTY_PATCH_REASON;
+    return `  ${labelPad}  ${dim("-".padEnd(6))}  ${note}`;
   }
   const pctStr = coloredPctCell(pct3);
   const bar = metricBar(pct3);
@@ -2216,7 +2237,7 @@ function formatHuman(out, opts = {}) {
       out.patch.pct,
       out.patch.covered,
       out.patch.executable,
-      "no executable lines in patch"
+      EMPTY_PATCH_REASON
     )
   );
   const projectRow = formatMetricRow(
@@ -2229,7 +2250,7 @@ function formatHuman(out, opts = {}) {
   if (opts.thresholds) {
     const patchPass = out.patch.pct >= opts.thresholds.patch;
     const projectPass = out.project.pct >= opts.thresholds.project;
-    const patchOk = out.patch.executable === 0 ? true : patchPass;
+    const patchOk = isEmptyPatch(out.patch) ? true : patchPass;
     const overall = patchOk && projectPass;
     const details = [];
     if (!patchOk) {
@@ -2240,7 +2261,7 @@ function formatHuman(out, opts = {}) {
         `project ${out.project.pct.toFixed(1)}% < ${opts.thresholds.project}%`
       );
     }
-    const detail = details.length > 0 ? dim(`  ${details.join("; ")}`) : "";
+    const detail = details.length > 0 ? dim(`  ${details.join("; ")}`) : isEmptyPatch(out.patch) ? dim(`  ${EMPTY_PATCH_REASON}`) : "";
     lines.push(
       `  ${"Gate".padEnd(8)}  ${overall ? badge("pass") : badge("fail")}${detail}`
     );
@@ -2253,12 +2274,15 @@ function formatHuman(out, opts = {}) {
       );
     }
   }
-  if (out.files.length > 0) {
+  if (isEmptyPatch(out.patch)) {
+    lines.push("");
+    lines.push(dim("No executable lines in the patch \u2014 patch gate does not apply."));
+  } else if (out.files.length > 0) {
     lines.push("");
     lines.push(heading("Files in diff:"));
     const anyPatch = out.files.some((f) => f.patchCoverage !== null);
     if (!anyPatch) {
-      lines.push(dim("  (project coverage \u2014 no executable lines in patch)"));
+      lines.push(dim(`  (project coverage \u2014 ${EMPTY_PATCH_REASON})`));
     }
     for (const f of out.files) {
       const hasPatch = f.patchCoverage !== null;
@@ -2344,7 +2368,7 @@ ${tip("add thresholds.patch / thresholds.project to enforce")}
   const projectPct = diff.project.pct;
   const patchThreshold = config.thresholds.patch;
   const projectThreshold = config.thresholds.project;
-  const patchSkipped = diff.patch.executable === 0;
+  const patchSkipped = isEmptyPatch(diff.patch);
   const patchPass = patchSkipped ? true : patchPct >= patchThreshold;
   const projectPass = projectPct >= projectThreshold;
   const overall = patchPass && projectPass ? "pass" : "fail";
@@ -2355,10 +2379,11 @@ ${tip("add thresholds.patch / thresholds.project to enforce")}
         pct: patchPct,
         threshold: patchThreshold,
         pass: patchPass,
-        ...patchSkipped ? { skipped: true } : {}
+        ...patchSkipped ? { skipped: true, reason: EMPTY_PATCH_REASON } : {}
       },
       project: { pct: projectPct, threshold: projectThreshold, pass: projectPass },
-      overall
+      overall,
+      ...patchSkipped ? { note: EMPTY_PATCH_REASON } : {}
     };
     return {
       skipped: false,
@@ -2377,7 +2402,7 @@ ${tip("add thresholds.patch / thresholds.project to enforce")}
   lines.push("");
   if (patchSkipped) {
     lines.push(
-      `  ${"Patch".padEnd(8)}  ${dim("-")}  ${dim("(no executable lines \u2014 skipped)")}  ${badge("info")}`
+      `  ${"Patch".padEnd(8)}  ${dim("-")}  ${EMPTY_PATCH_REASON}  ${badge("skip")}`
     );
   } else {
     lines.push(formatMetricLine("Patch", patchPct, patchThreshold, patchPass));
@@ -2385,10 +2410,17 @@ ${tip("add thresholds.patch / thresholds.project to enforce")}
   lines.push(formatMetricLine("Project", projectPct, projectThreshold, projectPass));
   if (overall === "fail") {
     lines.push("");
+    if (patchSkipped) {
+      lines.push(dim("No executable lines in the patch \u2014 patch gate skipped."));
+    }
     lines.push(tip("add tests for uncovered ranges: tested diff"));
   } else {
     lines.push("");
-    lines.push(dim(patchSkipped ? "project thresholds met (patch skipped)" : "thresholds met"));
+    lines.push(
+      dim(
+        patchSkipped ? "No executable lines in the patch \u2014 patch gate skipped. Project threshold met." : "thresholds met"
+      )
+    );
   }
   lines.push("");
   return {
@@ -2561,7 +2593,7 @@ function createProgram() {
       "Agent loop:",
       "  tested setup \u2192 tested run \u2192 tested diff \u2192 tested check \u2192 tested push --pr <n>"
     ].join("\n")
-  ).version("0.1.0");
+  ).version("0.1.1");
   registerSetupCommand(program2);
   registerDoctorCommand(program2);
   registerInitCommand(program2);

package/dist/tested.js

@@ -44,6 +44,8 @@ function badge(kind) {
       return pc.yellow("[WARN]");
     case "info":
       return pc.cyan("[INFO]");
+    case "skip":
+      return pc.cyan("[SKIP]");
   }
 }
 function metricBar(pct3, width = 10) {
@@ -179,6 +181,8 @@ function buildInitYaml(args) {
   if (args.testRunner) {
     lines.push(`testRunner: ${args.testRunner}`);
   }
+  lines.push("# Patch gate is skipped when the diff has no executable lines");
+  lines.push("# (tests-only, comments-only, docs-only, or ignored files).");
   lines.push("thresholds:");
   lines.push("  patch: 80");
   lines.push("  project: 60");
@@ -545,7 +549,9 @@ var FileCoverageSchema = z2.object({
 var CoverageTotalsSchema = z2.object({
   executable: z2.number().int().nonnegative(),
   covered: z2.number().int().nonnegative(),
-  pct: z2.number().min(0).max(100)
+  pct: z2.number().min(0).max(100),
+  /** Present when executable === 0 — not a 0% coverage result. */
+  empty: z2.literal(true).optional()
 });
 var ProjectTotalsSchema = CoverageTotalsSchema.extend({
   delta: z2.number().nullable()
@@ -614,7 +620,14 @@ async function headSha(ctx) {
   return (await ctx.git.revparse(["HEAD"])).trim();
 }
 async function unifiedDiff(ctx, base) {
-  return ctx.git.diff([`${base}...HEAD`]);
+  try {
+    const mergeBase = (await ctx.git.raw(["merge-base", base, "HEAD"])).trim();
+    if (mergeBase) {
+      return ctx.git.diff([`${base}...HEAD`]);
+    }
+  } catch {
+  }
+  return ctx.git.diff([base, "HEAD"]);
 }
 async function remoteUrl(ctx, remote = "origin") {
   return (await ctx.git.raw(["remote", "get-url", remote])).trim();
@@ -763,6 +776,10 @@ function assertWithinRoot(root, resolvedPath) {
 }
 
 // src/core/patch.ts
+var EMPTY_PATCH_REASON = "no executable lines in the patch";
+function isEmptyPatch(totals) {
+  return totals.executable === 0;
+}
 function pct(covered, executable) {
   if (executable === 0) return 0;
   return Math.round(covered / executable * 1e3) / 10;
@@ -862,10 +879,10 @@ function uncoveredRanges(file) {
 function buildDiffOutput(args) {
   const patch = computePatchCoverage(args.files, args.addedByFile);
   const project = computeProjectCoverage(args.files);
-  const fileNames = /* @__PURE__ */ new Set([
-    ...patch.byFile.keys(),
-    ...project.byFile.keys()
-  ]);
+  const emptyPatch = isEmptyPatch(patch.totals);
+  const fileNames = new Set(
+    emptyPatch ? [] : [...patch.byFile.keys(), ...project.byFile.keys()]
+  );
   const files = [];
   for (const name of fileNames) {
     const file = args.files.find((f) => f.path === name);
@@ -882,7 +899,7 @@ function buildDiffOutput(args) {
     schemaVersion: 1,
     base: args.base,
     head: args.head,
-    patch: patch.totals,
+    patch: emptyPatch ? { ...patch.totals, empty: true } : patch.totals,
     project: { ...project.totals, delta: args.projectDelta ?? null },
     files,
     ignored: [...args.ignored]
@@ -1865,8 +1882,6 @@ function buildCiSnippet() {
     "    runs-on: ubuntu-latest",
     "    steps:",
     "      - uses: actions/checkout@v4",
-    "        with:",
-    "          fetch-depth: 0",
     "      - uses: tested-hq/cli/action@main",
     "        with:",
     "          # pin ref for reproducible installs (do not use floating tags in prod)",
@@ -1931,6 +1946,12 @@ function formatSetupHuman(opts) {
     ])
   );
   lines.push("");
+  lines.push(
+    dim(
+      "Patch gate is skipped when the diff has no executable lines (tests-only, docs, comments)."
+    )
+  );
+  lines.push("");
   lines.push(tip("re-check anytime: tested doctor"));
   lines.push("");
   return lines.join("\n");
@@ -2196,8 +2217,8 @@ function coloredPctCell(pct3, width = 6) {
 function formatMetricRow(label, pct3, covered, executable, emptyNote) {
   const labelPad = label.padEnd(8);
   if (executable === 0) {
-    const note = emptyNote ?? "no executable lines";
-    return `  ${labelPad}  ${dim("-".padEnd(6))}  ${dim(note)}`;
+    const note = emptyNote ?? EMPTY_PATCH_REASON;
+    return `  ${labelPad}  ${dim("-".padEnd(6))}  ${note}`;
   }
   const pctStr = coloredPctCell(pct3);
   const bar = metricBar(pct3);
@@ -2216,7 +2237,7 @@ function formatHuman(out, opts = {}) {
       out.patch.pct,
       out.patch.covered,
       out.patch.executable,
-      "no executable lines in patch"
+      EMPTY_PATCH_REASON
     )
   );
   const projectRow = formatMetricRow(
@@ -2229,7 +2250,7 @@ function formatHuman(out, opts = {}) {
   if (opts.thresholds) {
     const patchPass = out.patch.pct >= opts.thresholds.patch;
     const projectPass = out.project.pct >= opts.thresholds.project;
-    const patchOk = out.patch.executable === 0 ? true : patchPass;
+    const patchOk = isEmptyPatch(out.patch) ? true : patchPass;
     const overall = patchOk && projectPass;
     const details = [];
     if (!patchOk) {
@@ -2240,7 +2261,7 @@ function formatHuman(out, opts = {}) {
         `project ${out.project.pct.toFixed(1)}% < ${opts.thresholds.project}%`
       );
     }
-    const detail = details.length > 0 ? dim(`  ${details.join("; ")}`) : "";
+    const detail = details.length > 0 ? dim(`  ${details.join("; ")}`) : isEmptyPatch(out.patch) ? dim(`  ${EMPTY_PATCH_REASON}`) : "";
     lines.push(
       `  ${"Gate".padEnd(8)}  ${overall ? badge("pass") : badge("fail")}${detail}`
     );
@@ -2253,12 +2274,15 @@ function formatHuman(out, opts = {}) {
       );
     }
   }
-  if (out.files.length > 0) {
+  if (isEmptyPatch(out.patch)) {
+    lines.push("");
+    lines.push(dim("No executable lines in the patch \u2014 patch gate does not apply."));
+  } else if (out.files.length > 0) {
     lines.push("");
     lines.push(heading("Files in diff:"));
     const anyPatch = out.files.some((f) => f.patchCoverage !== null);
     if (!anyPatch) {
-      lines.push(dim("  (project coverage \u2014 no executable lines in patch)"));
+      lines.push(dim(`  (project coverage \u2014 ${EMPTY_PATCH_REASON})`));
     }
     for (const f of out.files) {
       const hasPatch = f.patchCoverage !== null;
@@ -2344,7 +2368,7 @@ ${tip("add thresholds.patch / thresholds.project to enforce")}
   const projectPct = diff.project.pct;
   const patchThreshold = config.thresholds.patch;
   const projectThreshold = config.thresholds.project;
-  const patchSkipped = diff.patch.executable === 0;
+  const patchSkipped = isEmptyPatch(diff.patch);
   const patchPass = patchSkipped ? true : patchPct >= patchThreshold;
   const projectPass = projectPct >= projectThreshold;
   const overall = patchPass && projectPass ? "pass" : "fail";
@@ -2355,10 +2379,11 @@ ${tip("add thresholds.patch / thresholds.project to enforce")}
         pct: patchPct,
         threshold: patchThreshold,
         pass: patchPass,
-        ...patchSkipped ? { skipped: true } : {}
+        ...patchSkipped ? { skipped: true, reason: EMPTY_PATCH_REASON } : {}
       },
       project: { pct: projectPct, threshold: projectThreshold, pass: projectPass },
-      overall
+      overall,
+      ...patchSkipped ? { note: EMPTY_PATCH_REASON } : {}
     };
     return {
       skipped: false,
@@ -2377,7 +2402,7 @@ ${tip("add thresholds.patch / thresholds.project to enforce")}
   lines.push("");
   if (patchSkipped) {
     lines.push(
-      `  ${"Patch".padEnd(8)}  ${dim("-")}  ${dim("(no executable lines \u2014 skipped)")}  ${badge("info")}`
+      `  ${"Patch".padEnd(8)}  ${dim("-")}  ${EMPTY_PATCH_REASON}  ${badge("skip")}`
     );
   } else {
     lines.push(formatMetricLine("Patch", patchPct, patchThreshold, patchPass));
@@ -2385,10 +2410,17 @@ ${tip("add thresholds.patch / thresholds.project to enforce")}
   lines.push(formatMetricLine("Project", projectPct, projectThreshold, projectPass));
   if (overall === "fail") {
     lines.push("");
+    if (patchSkipped) {
+      lines.push(dim("No executable lines in the patch \u2014 patch gate skipped."));
+    }
     lines.push(tip("add tests for uncovered ranges: tested diff"));
   } else {
     lines.push("");
-    lines.push(dim(patchSkipped ? "project thresholds met (patch skipped)" : "thresholds met"));
+    lines.push(
+      dim(
+        patchSkipped ? "No executable lines in the patch \u2014 patch gate skipped. Project threshold met." : "thresholds met"
+      )
+    );
   }
   lines.push("");
   return {
@@ -2561,7 +2593,7 @@ function createProgram() {
       "Agent loop:",
       "  tested setup \u2192 tested run \u2192 tested diff \u2192 tested check \u2192 tested push --pr <n>"
     ].join("\n")
-  ).version("0.1.0");
+  ).version("0.1.1");
   registerSetupCommand(program2);
   registerDoctorCommand(program2);
   registerInitCommand(program2);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tested/cli",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Coverage your agent can use. CLI for patch + project coverage with agent-readable JSON output.",
   "license": "MIT",
   "homepage": "https://tested.dev",
@@ -18,7 +18,7 @@
   "packageManager": "pnpm@11.3.0",
   "bin": {
     "tested": "./dist/tested.js",
-    "td": "./dist/td.js"
+    "td": "./dist/tested.js"
   },
   "files": [
     "dist",