@tested/cli 0.1.8 → 0.1.9

package/README.md

@@ -25,7 +25,7 @@ CI uses the composite Action (`tested-hq/cli/action@main`). It installs `@tested
 ```yaml
 - uses: tested-hq/cli/action@main
   with:
-    version: 0.1.8
+    version: 0.1.9
     token: ${{ secrets.TESTED_TOKEN }}
 ```
 
@@ -132,7 +132,7 @@ If `thresholds` is missing from `.tested.yaml`, `tested check` prints a notice o
 
 ### Flags (per package)
 
-Independent floors so a monorepo total cannot hide one package. Each flag is graded from **this run's** coverage files — no carryforward.
+Independent floors so a monorepo total cannot hide one package. Each flag is graded from **this run's** coverage files.
 
 ```yaml
 flags:
@@ -144,11 +144,11 @@ flags:
     paths: ["apps/api/**"]
 ```
 
-`tested check` applies global `thresholds` plus each flag whose paths appear in the merged coverage. Per-flag patch is new executable lines in those paths. A flag with no files this run is **missing** (fail / pending) — never last week's numbers.
+`tested check` applies global `thresholds` plus each flag whose paths appear in the merged coverage. Per-flag patch is new executable lines in those paths. A flag with no files this run is **skipped** (not 0%) — an affected-graph or scoped job that did not collect that package is not a fail.
 
-A job already scoped to one package: `tested check --flag frontend` (Action `flag:`). That coverage file **is** the flag.
+A job already scoped to one package: `tested check --flag frontend` (Action `flag:`). That coverage file **is** the flag. Other packages are omitted from this upload.
 
-`--json` lists per-flag results (`flags.frontend.patchCheck` → `tested.dev / patch / frontend`). `tested push` sends that same `flags` map on ingest so the app can post those GitHub checks. `tested diff --json` includes it too.
+`--json` lists per-flag results (`flags.frontend.patchCheck` → `tested.dev / patch / frontend`). Skipped flags have `status: missing` and `skipped: true` with no `executable` / `pct`. `tested push` omits skipped flags from ingest so the last successful upload for that package still stands. `tested diff --json` includes the same map as check.
 
 ## Coverage formats
 
@@ -195,7 +195,7 @@ Or pass `--file` repeatedly / Action `files`. A CI matrix must not conclude the
 ```yaml
 - uses: tested-hq/cli/action@main
   with:
-    version: 0.1.8
+    version: 0.1.9
     push: true
     pr-number: ${{ github.event.pull_request.number }}
     token: ${{ secrets.TESTED_TOKEN }}

package/dist/td.js

@@ -1129,7 +1129,7 @@ var TestedConfigSchema = z2.object({
   }).optional(),
   /**
    * Per-package gates. Each flag is graded from this run's coverage files
-   * only — a missing path is missing (pending/fail), never last week's totals.
+   * only — a missing path is skipped (not 0%), never another flag's totals.
    */
   flags: z2.record(FlagNameSchema, FlagConfigSchema).optional()
 });
@@ -1164,17 +1164,21 @@ var DiffOutputSchema = z2.object({
   ignored: z2.array(z2.string())
 });
 var FlagMetricJsonSchema = z2.object({
-  pct: z2.number().min(0).max(100),
+  /** Omitted when this flag had no files this run (`skipped: true`). */
+  pct: z2.number().min(0).max(100).optional(),
   threshold: z2.number().min(0).max(100),
-  pass: z2.boolean(),
-  executable: z2.number().int().nonnegative(),
-  covered: z2.number().int().nonnegative(),
+  /** Omitted when skipped — a missing flag is not a 0% fail. */
+  pass: z2.boolean().optional(),
+  executable: z2.number().int().nonnegative().optional(),
+  covered: z2.number().int().nonnegative().optional(),
   skipped: z2.literal(true).optional(),
   reason: z2.string().optional()
 });
 var FlagResultJsonSchema = z2.object({
   status: z2.enum(["pass", "fail", "missing"]),
   present: z2.boolean(),
+  /** True when this flag had no coverage files this run (not a 0% result). */
+  skipped: z2.literal(true).optional(),
   reason: z2.string().optional(),
   patchCheck: z2.string(),
   projectCheck: z2.string(),
@@ -1742,13 +1746,11 @@ function totalsToMetric(totals, threshold, kind) {
     covered: totals.covered
   };
 }
-function missingMetric(threshold) {
+function missingMetric(threshold, reason) {
   return {
-    pct: 0,
     threshold,
-    pass: false,
-    executable: 0,
-    covered: 0
+    skipped: true,
+    reason
   };
 }
 function checkSlug(kind, name) {
@@ -1778,8 +1780,8 @@ function missingFlag(name, thresholds, reason) {
     reason,
     patchCheck: checkSlug("patch", name),
     projectCheck: checkSlug("project", name),
-    patch: missingMetric(thresholds.patch),
-    project: missingMetric(thresholds.project)
+    patch: missingMetric(thresholds.patch, reason),
+    project: missingMetric(thresholds.project, reason)
   };
 }
 function evaluateFlags(input) {
@@ -1810,7 +1812,7 @@ function evaluateFlags(input) {
   return results;
 }
 function flagsPass(results) {
-  return results.every((f) => f.status === "pass");
+  return results.every((f) => f.status !== "fail");
 }
 function flagsToJson(results) {
   const out = {};
@@ -1818,19 +1820,44 @@ function flagsToJson(results) {
     out[flag.name] = {
       status: flag.status,
       present: flag.present,
+      ...flag.status === "missing" ? { skipped: true } : {},
       ...flag.reason ? { reason: flag.reason } : {},
       patchCheck: flag.patchCheck,
       projectCheck: flag.projectCheck,
-      patch: flag.patch,
-      project: flag.project
+      patch: metricToJson(flag.patch),
+      project: metricToJson(flag.project)
     };
   }
   return out;
 }
+function metricToJson(metric) {
+  if (metric.skipped) {
+    return {
+      threshold: metric.threshold,
+      skipped: true,
+      ...metric.reason ? { reason: metric.reason } : {}
+    };
+  }
+  return {
+    pct: metric.pct ?? 0,
+    threshold: metric.threshold,
+    pass: metric.pass ?? false,
+    executable: metric.executable ?? 0,
+    covered: metric.covered ?? 0,
+    ...metric.reason ? { reason: metric.reason } : {}
+  };
+}
+function flagsToIngestJson(results) {
+  const present = results.filter((f) => f.present);
+  return present.length > 0 ? flagsToJson(present) : void 0;
+}
 function resolveFlagsJson(input) {
   const results = evaluateFlags(input);
   return results.length > 0 ? flagsToJson(results) : void 0;
 }
+function resolveIngestFlagsJson(input) {
+  return flagsToIngestJson(evaluateFlags(input));
+}
 
 // src/core/coverage-merge.ts
 function emptyToUndef(value) {
@@ -2553,7 +2580,7 @@ async function executePush(cli, deps) {
   let flags;
   if (!handshakeOnly && !computeDiffFn) {
     try {
-      flags = resolveFlagsJson({
+      flags = resolveIngestFlagsJson({
         config,
         files,
         addedByFile,
@@ -3072,7 +3099,7 @@ import pc3 from "picocolors";
 // package.json
 var package_default = {
   name: "@tested/cli",
-  version: "0.1.8",
+  version: "0.1.9",
   description: "Coverage your agent can use. CLI for patch + project coverage with agent-readable JSON output.",
   license: "MIT",
   homepage: "https://tested.dev",
@@ -3718,6 +3745,12 @@ function formatMetricLine(label, pct3, threshold, pass, indent = "  ") {
   const status = pass ? badge("pass") : badge("fail");
   return `${indent}${label.padEnd(8)}  ${pctStr}%  (threshold ${threshold})  ${status}`;
 }
+function presentPct(metric) {
+  if (metric.pct === void 0 || metric.pass === void 0) {
+    throw new Error("present flag metric missing pct/pass");
+  }
+  return { pct: metric.pct, pass: metric.pass };
+}
 function formatFlagLines(results) {
   if (results.length === 0) return [];
   const lines = [""];
@@ -3734,16 +3767,18 @@ function formatFlagLines(results) {
         `    ${"Patch".padEnd(8)}  ${dim("-")}  ${EMPTY_PATCH_REASON}  ${badge("skip")}`
       );
     } else {
+      const patch = presentPct(flag.patch);
       lines.push(
-        formatMetricLine("Patch", flag.patch.pct, flag.patch.threshold, flag.patch.pass, "    ")
+        formatMetricLine("Patch", patch.pct, flag.patch.threshold, patch.pass, "    ")
       );
     }
+    const project = presentPct(flag.project);
     lines.push(
       formatMetricLine(
         "Project",
-        flag.project.pct,
+        project.pct,
         flag.project.threshold,
-        flag.project.pass,
+        project.pass,
         "    "
       )
     );
@@ -3831,7 +3866,7 @@ ${tip("add thresholds.patch / thresholds.project to enforce")}
     if (missing.length > 0) {
       lines.push(
         dim(
-          "Missing flags fail this run (no carryforward). Collect that package's coverage or scope the job with --flag."
+          "Flags with no files this run are skipped (not 0%). Scope the job with --flag when this coverage file is one package."
         )
       );
     }

package/dist/tested.js

@@ -1129,7 +1129,7 @@ var TestedConfigSchema = z2.object({
   }).optional(),
   /**
    * Per-package gates. Each flag is graded from this run's coverage files
-   * only — a missing path is missing (pending/fail), never last week's totals.
+   * only — a missing path is skipped (not 0%), never another flag's totals.
    */
   flags: z2.record(FlagNameSchema, FlagConfigSchema).optional()
 });
@@ -1164,17 +1164,21 @@ var DiffOutputSchema = z2.object({
   ignored: z2.array(z2.string())
 });
 var FlagMetricJsonSchema = z2.object({
-  pct: z2.number().min(0).max(100),
+  /** Omitted when this flag had no files this run (`skipped: true`). */
+  pct: z2.number().min(0).max(100).optional(),
   threshold: z2.number().min(0).max(100),
-  pass: z2.boolean(),
-  executable: z2.number().int().nonnegative(),
-  covered: z2.number().int().nonnegative(),
+  /** Omitted when skipped — a missing flag is not a 0% fail. */
+  pass: z2.boolean().optional(),
+  executable: z2.number().int().nonnegative().optional(),
+  covered: z2.number().int().nonnegative().optional(),
   skipped: z2.literal(true).optional(),
   reason: z2.string().optional()
 });
 var FlagResultJsonSchema = z2.object({
   status: z2.enum(["pass", "fail", "missing"]),
   present: z2.boolean(),
+  /** True when this flag had no coverage files this run (not a 0% result). */
+  skipped: z2.literal(true).optional(),
   reason: z2.string().optional(),
   patchCheck: z2.string(),
   projectCheck: z2.string(),
@@ -1742,13 +1746,11 @@ function totalsToMetric(totals, threshold, kind) {
     covered: totals.covered
   };
 }
-function missingMetric(threshold) {
+function missingMetric(threshold, reason) {
   return {
-    pct: 0,
     threshold,
-    pass: false,
-    executable: 0,
-    covered: 0
+    skipped: true,
+    reason
   };
 }
 function checkSlug(kind, name) {
@@ -1778,8 +1780,8 @@ function missingFlag(name, thresholds, reason) {
     reason,
     patchCheck: checkSlug("patch", name),
     projectCheck: checkSlug("project", name),
-    patch: missingMetric(thresholds.patch),
-    project: missingMetric(thresholds.project)
+    patch: missingMetric(thresholds.patch, reason),
+    project: missingMetric(thresholds.project, reason)
   };
 }
 function evaluateFlags(input) {
@@ -1810,7 +1812,7 @@ function evaluateFlags(input) {
   return results;
 }
 function flagsPass(results) {
-  return results.every((f) => f.status === "pass");
+  return results.every((f) => f.status !== "fail");
 }
 function flagsToJson(results) {
   const out = {};
@@ -1818,19 +1820,44 @@ function flagsToJson(results) {
     out[flag.name] = {
       status: flag.status,
       present: flag.present,
+      ...flag.status === "missing" ? { skipped: true } : {},
       ...flag.reason ? { reason: flag.reason } : {},
       patchCheck: flag.patchCheck,
       projectCheck: flag.projectCheck,
-      patch: flag.patch,
-      project: flag.project
+      patch: metricToJson(flag.patch),
+      project: metricToJson(flag.project)
     };
   }
   return out;
 }
+function metricToJson(metric) {
+  if (metric.skipped) {
+    return {
+      threshold: metric.threshold,
+      skipped: true,
+      ...metric.reason ? { reason: metric.reason } : {}
+    };
+  }
+  return {
+    pct: metric.pct ?? 0,
+    threshold: metric.threshold,
+    pass: metric.pass ?? false,
+    executable: metric.executable ?? 0,
+    covered: metric.covered ?? 0,
+    ...metric.reason ? { reason: metric.reason } : {}
+  };
+}
+function flagsToIngestJson(results) {
+  const present = results.filter((f) => f.present);
+  return present.length > 0 ? flagsToJson(present) : void 0;
+}
 function resolveFlagsJson(input) {
   const results = evaluateFlags(input);
   return results.length > 0 ? flagsToJson(results) : void 0;
 }
+function resolveIngestFlagsJson(input) {
+  return flagsToIngestJson(evaluateFlags(input));
+}
 
 // src/core/coverage-merge.ts
 function emptyToUndef(value) {
@@ -2553,7 +2580,7 @@ async function executePush(cli, deps) {
   let flags;
   if (!handshakeOnly && !computeDiffFn) {
     try {
-      flags = resolveFlagsJson({
+      flags = resolveIngestFlagsJson({
         config,
         files,
         addedByFile,
@@ -3072,7 +3099,7 @@ import pc3 from "picocolors";
 // package.json
 var package_default = {
   name: "@tested/cli",
-  version: "0.1.8",
+  version: "0.1.9",
   description: "Coverage your agent can use. CLI for patch + project coverage with agent-readable JSON output.",
   license: "MIT",
   homepage: "https://tested.dev",
@@ -3718,6 +3745,12 @@ function formatMetricLine(label, pct3, threshold, pass, indent = "  ") {
   const status = pass ? badge("pass") : badge("fail");
   return `${indent}${label.padEnd(8)}  ${pctStr}%  (threshold ${threshold})  ${status}`;
 }
+function presentPct(metric) {
+  if (metric.pct === void 0 || metric.pass === void 0) {
+    throw new Error("present flag metric missing pct/pass");
+  }
+  return { pct: metric.pct, pass: metric.pass };
+}
 function formatFlagLines(results) {
   if (results.length === 0) return [];
   const lines = [""];
@@ -3734,16 +3767,18 @@ function formatFlagLines(results) {
         `    ${"Patch".padEnd(8)}  ${dim("-")}  ${EMPTY_PATCH_REASON}  ${badge("skip")}`
       );
     } else {
+      const patch = presentPct(flag.patch);
       lines.push(
-        formatMetricLine("Patch", flag.patch.pct, flag.patch.threshold, flag.patch.pass, "    ")
+        formatMetricLine("Patch", patch.pct, flag.patch.threshold, patch.pass, "    ")
       );
     }
+    const project = presentPct(flag.project);
     lines.push(
       formatMetricLine(
         "Project",
-        flag.project.pct,
+        project.pct,
         flag.project.threshold,
-        flag.project.pass,
+        project.pass,
         "    "
       )
     );
@@ -3831,7 +3866,7 @@ ${tip("add thresholds.patch / thresholds.project to enforce")}
     if (missing.length > 0) {
       lines.push(
         dim(
-          "Missing flags fail this run (no carryforward). Collect that package's coverage or scope the job with --flag."
+          "Flags with no files this run are skipped (not 0%). Scope the job with --flag when this coverage file is one package."
         )
       );
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tested/cli",
-  "version": "0.1.8",
+  "version": "0.1.9",
   "description": "Coverage your agent can use. CLI for patch + project coverage with agent-readable JSON output.",
   "license": "MIT",
   "homepage": "https://tested.dev",