@quenty/cli-output-helpers 1.17.2 → 1.18.0

package/dist/reporting/summary-table-reporter.js.map

@@ -1 +1 @@
-{"version":3,"file":"summary-table-reporter.js","sourceRoot":"","sources":["../../src/reporting/summary-table-reporter.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAClD,OAAO,EAAE,gBAAgB,EAAE,MAAM,iBAAiB,CAAC;AACnD,OAAO,EAAE,YAAY,EAAsB,MAAM,eAAe,CAAC;AACjE,OAAO,EAAE,WAAW,EAAoB,MAAM,mBAAmB,CAAC;AAElE,OAAO,EAAE,oBAAoB,EAAE,cAAc,EAAE,MAAM,sBAAsB,CAAC;AAW5E;;;GAGG;AACH,MAAM,OAAO,oBAAqB,SAAQ,YAAY;IAC5C,MAAM,CAAgB;IACtB,aAAa,CAAS;IACtB,aAAa,CAAS;IACtB,YAAY,CAAS;IAE7B,YAAY,KAAoB,EAAE,OAAqC;QACrE,KAAK,EAAE,CAAC;QACR,IAAI,CAAC,MAAM,GAAG,KAAK,CAAC;QACpB,IAAI,CAAC,aAAa,GAAG,OAAO,EAAE,YAAY,IAAI,QAAQ,CAAC;QACvD,IAAI,CAAC,aAAa,GAAG,OAAO,EAAE,YAAY,IAAI,QAAQ,CAAC;QACvD,IAAI,CAAC,YAAY,GAAG,OAAO,EAAE,WAAW,IAAI,QAAQ,CAAC;IACvD,CAAC;IAEQ,KAAK,CAAC,SAAS;QACtB,MAAM,OAAO,GAAG,IAAI,CAAC,MAAM;aACxB,cAAc,EAAE;aAChB,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,CAAC;aACpB,MAAM,CAAC,CAAC,CAAC,EAAsB,EAAE,CAAC,CAAC,KAAK,SAAS,CAAC,CAAC;QACtD,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC;YAAE,OAAO;QAEjC,MAAM,QAAQ,GAAG,IAAI,CAAC,MAAM,CAAC,WAAW,EAAE,CAAC;QAC3C,MAAM,MAAM,GAAG,OAAO,CAAC,MAAM,GAAG,QAAQ,CAAC,MAAM,CAAC;QAChD,MAAM,UAAU,GAAG,IAAI,CAAC,GAAG,EAAE,GAAG,IAAI,CAAC,MAAM,CAAC,WAAW,CAAC;QAExD,IAAI,aAAa,GAAG,CAAC,CAAC;QAEtB,MAAM,OAAO,GAAiC;YAC5C;gBACE,MAAM,EAAE,SAAS;gBACjB,KAAK,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,WAAW;gBAC3B,QAAQ,EAAE,EAAE;aACb;YACD;gBACE,MAAM,EAAE,QAAQ;gBAChB,KAAK,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC,CAAC;gBAClC,MAAM,EAAE,CAAC,KAAK,EAAE,CAAC,EAAE,EAAE,CACnB,IAAI,CAAC,YAAY,CAAC,KAAK,EAAE,CAAC,EAAE,GAAG,EAAE,CAAC,aAAa,EAAE,CAAC;gBACpD,QAAQ,EAAE,EAAE;aACb;YACD;gBACE,MAAM,EAAE,UAAU;gBAClB,KAAK,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,gBAAgB,CAAC,CAAC,CAAC,UAAU,CAAC;gBAC5C,MAAM,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,YAAY,CAAC,SAAS,CAAC,CAAC,CAAC;aACzC;SACF,CAAC;QAEF,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;QAChB,OAAO,CAAC,GAAG,CAAC,WAAW,CAAC,OAAO,EAAE,OAAO,CAAC,CAAC,CAAC;QAE3C,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;QAChB,MAAM,UAAU,GAAG,YAAY,CAAC,aAAa,CAAC,GAAG,MAAM,SAAS,CAAC,CAAC;QAClE,MAAM,UAAU,GACd,QAAQ,CAAC,MAAM,GAAG,CAAC;YACjB,CAAC,CAAC,YAAY,CAAC,WAAW,CAAC,GAAG,QAAQ,CAAC,MAAM,SAAS,CAAC;YACvD,CAAC,CAAC,GAAG,QAAQ,CAAC,MAAM,SAAS,CAAC;QAClC,MAAM,SAAS,GAAG,YAAY,CAAC,SAAS,CACtC,MAAM,gBAAgB,CAAC,UAAU,CAAC,EAAE,CACrC,CAAC;QACF,MAAM,IAAI,GAAG,OAAO,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,UAAU,CAAC;QAC3D,OAAO,CAAC,GAAG,CACT,GAAG,OAAO,CAAC,MAAM,IAAI,IAAI,IAAI,IAAI,CAAC,YAAY,KAAK,UAAU,KAAK,UAAU,IAAI,SAAS,EAAE,CAC5F,CAAC;QAEF,IAAI,aAAa,GAAG,CAAC,EAAE,CAAC;YACtB,OAAO,CAAC,GAAG,CACT,YAAY,CAAC,aAAa,CACxB,KAAK,aAAa,gDAAgD,CACnE,CACF,CAAC;QACJ,CAAC;IACH,CAAC;IAEO,YAAY,CAAC,MAAqB;QACxC,IAAI,MAAM,CAAC,OAAO,EAAE,CAAC;YACnB,MAAM,YAAY,GAAG,oBAAoB,CAAC,MAAM,CAAC,eAAe,CAAC,CAAC;YAClE,OAAO,YAAY;gBACjB,CAAC,CAAC,GAAG,IAAI,CAAC,aAAa,IAAI,YAAY,EAAE;gBACzC,CAAC,CAAC,IAAI,CAAC,aAAa,CAAC;QACzB,CAAC;QACD,MAAM,WAAW,GAAG,MAAM,CAAC,WAAW,CAAC;QACvC,MAAM,YAAY,GAAG,MAAM,CAAC,YAAY,IAAI,IAAI,CAAC,aAAa,CAAC;QAC/D,OAAO,WAAW,CAAC,CAAC,CAAC,GAAG,YAAY,OAAO,WAAW,EAAE,CAAC,CAAC,CAAC,YAAY,CAAC;IAC1E,CAAC;IAEO,YAAY,CAClB,KAAa,EACb,MAAqB,EACrB,UAAsB;QAEtB,IAAI,MAAM,CAAC,OAAO,EAAE,CAAC;YACnB,MAAM,KAAK,GAAG,cAAc,CAAC,MAAM,CAAC,eAAe,CAAC,CAAC;YACrD,IAAI,KAAK;gBAAE,UAAU,EAAE,CAAC;YACxB,OAAO,KAAK;gBACV,CAAC,CAAC,YAAY,CAAC,aAAa,CAAC,KAAK,CAAC;gBACnC,CAAC,CAAC,YAAY,CAAC,aAAa,CAAC,KAAK,CAAC,CAAC;QACxC,CAAC;QACD,OAAO,YAAY,CAAC,WAAW,CAAC,KAAK,CAAC,CAAC;IACzC,CAAC;CACF"}
+{"version":3,"file":"summary-table-reporter.js","sourceRoot":"","sources":["../../src/reporting/summary-table-reporter.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAClD,OAAO,EAAE,gBAAgB,EAAE,MAAM,iBAAiB,CAAC;AACnD,OAAO,EAAE,YAAY,EAAsB,MAAM,eAAe,CAAC;AACjE,OAAO,EAAE,WAAW,EAAoB,MAAM,mBAAmB,CAAC;AAElE,OAAO,EACL,WAAW,EACX,gBAAgB,EAChB,mBAAmB,EACnB,YAAY,GAEb,MAAM,oBAAoB,CAAC;AAgB5B;;;GAGG;AACH,MAAM,OAAO,oBAAqB,SAAQ,YAAY;IAC5C,MAAM,CAAgB;IACtB,aAAa,CAAS;IACtB,aAAa,CAAS;IACtB,YAAY,CAAS;IACrB,kBAAkB,CAAU;IAEpC,YAAY,KAAoB,EAAE,OAAqC;QACrE,KAAK,EAAE,CAAC;QACR,IAAI,CAAC,MAAM,GAAG,KAAK,CAAC;QACpB,IAAI,CAAC,aAAa,GAAG,OAAO,EAAE,YAAY,IAAI,QAAQ,CAAC;QACvD,IAAI,CAAC,aAAa,GAAG,OAAO,EAAE,YAAY,IAAI,QAAQ,CAAC;QACvD,IAAI,CAAC,YAAY,GAAG,OAAO,EAAE,WAAW,IAAI,QAAQ,CAAC;QACrD,IAAI,CAAC,kBAAkB,GAAG,OAAO,EAAE,iBAAiB,IAAI,KAAK,CAAC;IAChE,CAAC;IAEQ,KAAK,CAAC,SAAS;QACtB,MAAM,OAAO,GAAG,IAAI,CAAC,MAAM;aACxB,cAAc,EAAE;aAChB,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,CAAC;aACpB,MAAM,CAAC,CAAC,CAAC,EAAsB,EAAE,CAAC,CAAC,KAAK,SAAS,CAAC,CAAC;QACtD,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC;YAAE,OAAO;QAEjC,MAAM,QAAQ,GAAG,IAAI,CAAC,MAAM,CAAC,WAAW,EAAE,CAAC;QAC3C,MAAM,MAAM,GAAG,OAAO,CAAC,MAAM,GAAG,QAAQ,CAAC,MAAM,CAAC;QAChD,MAAM,UAAU,GAAG,IAAI,CAAC,GAAG,EAAE,GAAG,IAAI,CAAC,MAAM,CAAC,WAAW,CAAC;QAExD,yEAAyE;QACzE,6DAA6D;QAC7D,MAAM,QAAQ,GAAG,IAAI,GAAG,CACtB,OAAO,CAAC,GAAG,CAAC,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC,MAAM,EAAE,IAAI,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,CACzD,CAAC;QAEF,MAAM,OAAO,GAAiC;YAC5C;gBACE,MAAM,EAAE,SAAS;gBACjB,KAAK,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,WAAW;gBAC3B,QAAQ,EAAE,EAAE;aACb;YACD;gBACE,MAAM,EAAE,QAAQ;gBAChB,KAAK,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,gBAAgB,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAE,CAAC;gBAChD,MAAM,EAAE,CAAC,KAAK,EAAE,CAAC,EAAE,EAAE,CAAC,WAAW,CAAC,KAAK,EAAE,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAE,CAAC,QAAQ,CAAC;gBACnE,QAAQ,EAAE,EAAE;aACb;YACD;gBACE,MAAM,EAAE,UAAU;gBAClB,KAAK,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,gBAAgB,CAAC,CAAC,CAAC,UAAU,CAAC;gBAC5C,MAAM,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,YAAY,CAAC,SAAS,CAAC,CAAC,CAAC;aACzC;SACF,CAAC;QAEF,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;QAChB,OAAO,CAAC,GAAG,CAAC,WAAW,CAAC,OAAO,EAAE,OAAO,CAAC,CAAC,CAAC;QAE3C,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;QAChB,MAAM,UAAU,GAAG,YAAY,CAAC,aAAa,CAAC,GAAG,MAAM,SAAS,CAAC,CAAC;QAClE,MAAM,UAAU,GACd,QAAQ,CAAC,MAAM,GAAG,CAAC;YACjB,CAAC,CAAC,YAAY,CAAC,WAAW,CAAC,GAAG,QAAQ,CAAC,MAAM,SAAS,CAAC;YACvD,CAAC,CAAC,GAAG,QAAQ,CAAC,MAAM,SAAS,CAAC;QAClC,MAAM,SAAS,GAAG,YAAY,CAAC,SAAS,CACtC,MAAM,gBAAgB,CAAC,UAAU,CAAC,EAAE,CACrC,CAAC;QACF,MAAM,IAAI,GAAG,OAAO,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,UAAU,CAAC;QAC3D,OAAO,CAAC,GAAG,CACT,GAAG,OAAO,CAAC,MAAM,IAAI,IAAI,IAAI,IAAI,CAAC,YAAY,KAAK,UAAU,KAAK,UAAU,IAAI,SAAS,EAAE,CAC5F,CAAC;QAEF,KAAK,MAAM,EAAE,OAAO,EAAE,IAAI,YAAY,CAAC,CAAC,GAAG,QAAQ,CAAC,MAAM,EAAE,CAAC,CAAC,EAAE,CAAC;YAC/D,OAAO,CAAC,GAAG,CAAC,YAAY,CAAC,aAAa,CAAC,KAAK,OAAO,EAAE,CAAC,CAAC,CAAC;QAC1D,CAAC;IACH,CAAC;IAEO,QAAQ,CAAC,MAAqB;QACpC,OAAO,mBAAmB,CAAC,MAAM,EAAE;YACjC,YAAY,EAAE,IAAI,CAAC,aAAa;YAChC,YAAY,EAAE,IAAI,CAAC,aAAa;YAChC,iBAAiB,EAAE,IAAI,CAAC,kBAAkB;SAC3C,CAAC,CAAC;IACL,CAAC;CACF"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quenty/cli-output-helpers",
-  "version": "1.17.2",
+  "version": "1.18.0",
   "description": "Helpers to generate Nevermore package and game templates",
   "type": "module",
   "keywords": [
@@ -59,5 +59,5 @@
   "engines": {
     "node": ">=16"
   },
-  "gitHead": "2808f7b0f98a18163c6819aeb5fd9064496bc63a"
+  "gitHead": "f75f93d5b2ac51cf7da3b7599788dcce31fd8a29"
 }

package/src/outputHelper.ts

@@ -1,5 +1,6 @@
 import chalk from 'chalk';
 import { AsyncLocalStorage } from 'async_hooks';
+import { isCI } from './cli-utils.js';
 
 export type BoxOptions = {
   centered?: boolean;
@@ -230,6 +231,27 @@ export class OutputHelper {
     console.log(this._hasAnsi(message) ? message : this.formatWarning(message));
   }
 
+  /**
+   * Open a collapsible section, so a run of related output can be folded away.
+   *
+   * In CI this is a workflow command; elsewhere it is a plain header, since a
+   * `::group::` line is only noise in a terminal that cannot fold it. Always
+   * pair with {@link endGroup} — an unclosed group swallows everything printed
+   * after it.
+   */
+  public static startGroup(title: string, useGroups = isCI()): void {
+    console.log(
+      useGroups ? `::group::${title}` : this.formatDim(`── ${title} ──`)
+    );
+  }
+
+  /** Close the section opened by {@link startGroup}. */
+  public static endGroup(useGroups = isCI()): void {
+    if (useGroups) {
+      console.log('::endgroup::');
+    }
+  }
+
   /**
    * Logs hint to the console
    * @param message Message to write

package/src/reporting/aggregated-batch-reporter.ts

@@ -0,0 +1,45 @@
+import { OutputHelper } from '../outputHelper.js';
+import { BaseReporter } from './reporter.js';
+
+/**
+ * Batch progress for a run that is one execution rather than many.
+ *
+ * An aggregated batch has no per-package moment to report: every package starts
+ * in the same tick, before the task that runs them exists, and finishes in the
+ * same tick when it returns. A reporter built around per-package callbacks can
+ * only invent structure that is not there — which is what produced one empty
+ * group per package, and filed the shared upload under whichever package's
+ * async context happened to reach the promise first.
+ *
+ * So this one says nothing per package. The run is printed once, by whoever
+ * holds it, and the summary table recaps at the end.
+ */
+export class AggregatedBatchReporter extends BaseReporter {
+  private _actionVerb: string;
+  private _total: number;
+
+  constructor(total: number, options?: { actionVerb?: string }) {
+    super();
+    this._total = total;
+    this._actionVerb = options?.actionVerb ?? 'Processing';
+  }
+
+  override async startAsync(): Promise<void> {
+    OutputHelper.info(
+      `${this._actionVerb} ${this._total} packages in one batch`
+    );
+  }
+
+  /**
+   * Print a run that has already happened: its output, then what reading that
+   * output turned up.
+   *
+   * Taken as rendered lines rather than as a log to render, so the decision of
+   * what a line should look like stays in one place and this stays a printer.
+   */
+  printRun(renderedLines: string[]): void {
+    for (const line of renderedLines) {
+      console.log(line);
+    }
+  }
+}

package/src/reporting/github/formatting.ts

@@ -15,12 +15,14 @@ import {
   type IStateTracker,
   type PackageState,
 } from '../state/state-tracker.js';
+import { formatProgressInline } from '../progress-format.js';
 import {
-  formatProgressInline,
-  formatProgressResult,
-  isEmptyTestRun,
-  isUnreportedTestRun,
-} from '../progress-format.js';
+  formatStatusText,
+  resolveResultStatus,
+  statusIcon,
+  tallyCaveats,
+  type ResultStatus,
+} from '../result-status.js';
 
 // ── Public types ────────────────────────────────────────────────────────────
 
@@ -145,26 +147,37 @@ export function formatResultStatus(
   failureLabel: string,
   expectsTestCounts = false
 ): string {
-  const duration = formatDurationMs(pkg.durationMs);
-  const progressText = formatProgressResult(pkg.progressSummary);
-  const empty = isEmptyTestRun(pkg.progressSummary);
+  const status = resolveResultStatus(pkg, {
+    successLabel,
+    failureLabel,
+    expectsTestCounts,
+  });
 
-  if (pkg.success) {
-    if (expectsTestCounts && isUnreportedTestRun(pkg.progressSummary)) {
-      return `⚠️ **Unverified** — no test counts reported (${duration})`;
-    }
-    const label = progressText
-      ? `${successLabel} ${progressText}`
-      : successLabel;
-    return empty ? `⚠️ ${label} (${duration})` : `✅ ${label} (${duration})`;
-  }
+  // Bolded verdict, plain caveats: a reader scans a column of these for the
+  // verdict, and bolding the qualifier too would flatten the difference between
+  // them. Everything else — which states exist, what they are called, what
+  // counts as a warning — is the resolver's.
+  const verdict =
+    status.severity === 'failure'
+      ? status.failedPhase
+        ? `**${status.label}** at ${status.failedPhase}`
+        : `**${status.label}**`
+      : status.label;
+  const caveats = status.caveats.length > 0 ? ` - ${caveatText(status)}` : '';
+  const head = [verdict, status.progress].filter(Boolean).join(' ');
+
+  return `${statusIcon(
+    status.severity,
+    'emoji'
+  )} ${head}${caveats} (${formatDurationMs(pkg.durationMs)})`;
+}
 
-  const failedPhase = pkg.failedPhase;
-  const effectiveLabel = pkg.failureLabel ?? failureLabel;
-  const label = failedPhase
-    ? `**${effectiveLabel}** at ${failedPhase}`
-    : `**${effectiveLabel}**`;
-  return `❌ ${label} (${duration})`;
+/** The caveat half of a status line, without the verdict in front of it. */
+function caveatText(status: ResultStatus): string {
+  return formatStatusText({ ...status, label: '', progress: '' }).replace(
+    /^ *- */,
+    ''
+  );
 }
 
 export function getActionsRunUrl(): string | undefined {
@@ -292,7 +305,7 @@ export function formatGithubTableBody(
         statusText = formatResultStatus(
           pkg.result!,
           config.successLabel ?? 'Passed',
-          config.failureLabel ?? 'Failed',
+          config.failureLabel ?? 'FAILED',
           config.expectsTestCounts ?? false
         );
         break;
@@ -314,18 +327,19 @@ export function formatGithubTableBody(
   if (allDone) {
     const passed = packages.filter((p) => p.status === 'passed').length;
     const failed = packages.filter((p) => p.status === 'failed').length;
-    // Read the final result, not the live progress — progress is cleared once a
-    // package resolves, so an end-of-run check against it never fires.
-    const emptyRuns = packages.filter((p) =>
-      isEmptyTestRun(p.result?.progressSummary ?? p.progress)
-    ).length;
-    const unreported = config.expectsTestCounts
-      ? packages.filter(
-          (p) =>
-            p.status === 'passed' &&
-            isUnreportedTestRun(p.result?.progressSummary)
-        ).length
-      : 0;
+    // Tallied off the same statuses the rows were drawn from, so a line here
+    // cannot contradict the column above it — which it could when each was
+    // counted separately. Read from the final result, not the live progress:
+    // progress is cleared once a package resolves.
+    const statuses = packages
+      .filter((p) => p.result !== undefined)
+      .map((p) =>
+        resolveResultStatus(p.result!, {
+          successLabel: config.successLabel ?? 'Passed',
+          failureLabel: config.failureLabel ?? 'FAILED',
+          expectsTestCounts: config.expectsTestCounts ?? false,
+        })
+      );
     const verb = config.summaryVerb ?? 'tested';
     const unit = packages.length === 1 ? 'package' : 'packages';
     footer = `**${
@@ -333,11 +347,8 @@ export function formatGithubTableBody(
     } ${unit} ${verb}, ${passed} passed, ${failed} failed** in ${formatDurationMs(
       elapsedMs
     )}`;
-    if (emptyRuns > 0) {
-      footer += `\n⚠️ ${emptyRuns} package(s) ran 0 tests — check test discovery`;
-    }
-    if (unreported > 0) {
-      footer += `\n⚠️ ${unreported} package(s) reported no test counts — a pass here proves nothing about whether tests ran`;
+    for (const { message } of tallyCaveats(statuses)) {
+      footer += `\n⚠️ ${message}`;
     }
   } else {
     const done = packages.filter(

package/src/reporting/grouped-reporter.ts

@@ -3,10 +3,11 @@ import { formatDurationMs, isCI } from '../cli-utils.js';
 import { type PackageResult, BaseReporter } from './reporter.js';
 import { type IStateTracker } from './state/state-tracker.js';
 import {
-  formatProgressResult,
-  isEmptyTestRun,
-  isUnreportedTestRun,
-} from './progress-format.js';
+  colorStatus,
+  formatStatusText,
+  resolveResultStatus,
+  statusIcon,
+} from './result-status.js';
 
 export interface GroupedReporterOptions {
   showLogs: boolean;
@@ -79,45 +80,22 @@ export class GroupedReporter extends BaseReporter {
 
     const showLogs = this._options.showLogs || !result.success;
     const duration = formatDurationMs(result.durationMs);
-    const successLabel = this._options.successLabel ?? 'Passed';
-    const failureLabel =
-      result.failureLabel ?? this._options.failureLabel ?? 'FAILED';
 
-    const progressText = formatProgressResult(result.progressSummary);
-    const empty = isEmptyTestRun(result.progressSummary);
-
-    const unverified =
-      (this._options.expectsTestCounts ?? false) &&
-      isUnreportedTestRun(result.progressSummary);
-
-    if (result.success) {
-      const label = unverified
-        ? 'Unverified — no test counts reported'
-        : progressText
-        ? `${successLabel} ${progressText}`
-        : successLabel;
-      const formatted =
-        empty || unverified
-          ? OutputHelper.formatWarning(`${label} ⚠`)
-          : OutputHelper.formatSuccess(label);
-      const icon =
-        empty || unverified
-          ? OutputHelper.formatWarning('⚠')
-          : OutputHelper.formatSuccess('✓');
-      console.log(
-        `  ${icon} ${formatted} ${OutputHelper.formatDim(`(${duration})`)}`
-      );
-    } else {
-      const failedPhase = result.failedPhase;
-      const label = failedPhase
-        ? `${failureLabel} at ${failedPhase}`
-        : failureLabel;
-      console.log(
-        `  ${OutputHelper.formatError('✗')} ${OutputHelper.formatError(
-          label
-        )} ${OutputHelper.formatDim(`(${duration})`)}`
-      );
-    }
+    const status = resolveResultStatus(result, {
+      successLabel: this._options.successLabel,
+      failureLabel: this._options.failureLabel,
+      expectsTestCounts: this._options.expectsTestCounts,
+    });
+
+    console.log(
+      `  ${colorStatus(
+        statusIcon(status.severity, 'ascii'),
+        status.severity
+      )} ${colorStatus(
+        formatStatusText(status),
+        status.severity
+      )} ${OutputHelper.formatDim(`(${duration})`)}`
+    );
 
     if (showLogs) {
       if (result.logs) {

package/src/reporting/index.ts

@@ -26,6 +26,7 @@ export {
   WatchResultReporter,
   type WatchResultReporterOptions,
 } from './watch-result-reporter.js';
+export { AggregatedBatchReporter } from './aggregated-batch-reporter.js';
 export { CompositeResultReporter } from './composite-result-reporter.js';
 export {
   buildResultReporter,
@@ -52,6 +53,18 @@ export {
   isEmptyTestRun,
   summarizeFailure,
 } from './progress-format.js';
+export {
+  colorStatus,
+  formatStatusText,
+  resolveResultStatus,
+  statusIcon,
+  tallyCaveats,
+  type ResolveResultStatusOptions,
+  type ResultCaveat,
+  type ResultSeverity,
+  type ResultStatus,
+  type ResultStatusInput,
+} from './result-status.js';
 
 // State tracking
 export {

package/src/reporting/reporter.ts

@@ -5,6 +5,16 @@
  * and end in a terminal state (passed / failed).
  */
 
+/**
+ * Something that makes a result less than it appears.
+ *
+ * All three are compatible with a pass, which is the point: a run that passed
+ * without proving anything must not render as one that did. Declared here
+ * rather than beside the resolver that reads them, so a result type can carry
+ * one without depending on how it is drawn.
+ */
+export type ResultCaveat = 'no-counts' | 'logs-lost' | 'empty-run';
+
 /** Execution phases a package can move through. */
 export type JobPhase =
   | 'waiting'
@@ -71,6 +81,14 @@ export interface PackageResult {
   failedPhase?: JobPhase;
   /** Per-result override of the reporter's default failure label. */
   failureLabel?: string;
+  /**
+   * What qualifies this result, for facts the reporters cannot derive.
+   *
+   * `empty-run` and `no-counts` come from the counts themselves and are worked
+   * out by `resolveResultStatus`; anything here is something only the producer
+   * knows, such as a run whose log output the engine dropped.
+   */
+  caveats?: ResultCaveat[];
 }
 
 /** Summary of a complete batch run. */

package/src/reporting/result-status.test.ts

@@ -0,0 +1,149 @@
+import { describe, it, expect } from 'vitest';
+import {
+  formatStatusText,
+  resolveResultStatus,
+  statusIcon,
+  tallyCaveats,
+  type ResultStatusInput,
+} from './result-status.js';
+
+function counts(passed: number, total: number): ResultStatusInput {
+  return {
+    success: true,
+    progressSummary: {
+      kind: 'test-counts',
+      passed,
+      failed: total - passed,
+      total,
+    },
+  };
+}
+
+describe('resolveResultStatus', () => {
+  it('calls a clean pass a success', () => {
+    const status = resolveResultStatus(counts(68, 68));
+
+    expect(status.severity).toBe('success');
+    expect(formatStatusText(status)).toBe('Passed (68/68)');
+  });
+
+  it('keeps a failure a failure however it is qualified', () => {
+    // A caveat narrows what a result proves; it never softens a failure into a
+    // warning, or a red row would go amber for having lost its logs.
+    const status = resolveResultStatus({
+      ...counts(3, 4),
+      success: false,
+      caveats: ['logs-lost'],
+    });
+
+    expect(status.severity).toBe('failure');
+    expect(formatStatusText(status)).toBe('FAILED (3/4) - logs lost');
+  });
+
+  it('demotes a pass that carries a caveat to a warning', () => {
+    const status = resolveResultStatus({
+      ...counts(35, 35),
+      caveats: ['logs-lost'],
+    });
+
+    expect(status.severity).toBe('warning');
+    expect(formatStatusText(status)).toBe('Passed (35/35) - logs lost');
+  });
+
+  it('flags a pass that ran no tests', () => {
+    const status = resolveResultStatus(counts(0, 0));
+
+    expect(status.severity).toBe('warning');
+    expect(status.caveats).toEqual(['empty-run']);
+  });
+
+  it('calls a pass with no counts unverified, when counts were expected', () => {
+    // The claim being avoided: a green row with nothing behind it is
+    // indistinguishable from one that tested something.
+    const status = resolveResultStatus(
+      { success: true },
+      { expectsTestCounts: true }
+    );
+
+    expect(status.severity).toBe('warning');
+    expect(formatStatusText(status)).toBe(
+      'Unverified - no test counts reported'
+    );
+  });
+
+  it('says nothing about missing counts where none were expected', () => {
+    // A deploy has no test counts and is not suspect for lacking them.
+    const status = resolveResultStatus({ success: true });
+
+    expect(status.severity).toBe('success');
+    expect(status.caveats).toEqual([]);
+  });
+
+  it('reports every caveat that applies, not just the first', () => {
+    const status = resolveResultStatus(
+      { ...counts(0, 0), caveats: ['logs-lost'] },
+      { expectsTestCounts: true }
+    );
+
+    expect(status.caveats).toEqual(['logs-lost', 'empty-run']);
+    expect(formatStatusText(status)).toBe(
+      'Passed (0/0) - logs lost, ran 0 tests'
+    );
+  });
+
+  it('names the phase a failure happened in', () => {
+    const status = resolveResultStatus({
+      success: false,
+      failedPhase: 'uploading',
+    });
+
+    expect(formatStatusText(status)).toBe('FAILED at uploading');
+  });
+
+  it('takes the labels the caller uses for its own kind of run', () => {
+    const status = resolveResultStatus(
+      { success: true },
+      { successLabel: 'Deployed' }
+    );
+
+    expect(formatStatusText(status)).toBe('Deployed');
+  });
+});
+
+describe('tallyCaveats', () => {
+  it('counts each caveat across a run', () => {
+    const statuses = [
+      resolveResultStatus({ ...counts(5, 5), caveats: ['logs-lost'] }),
+      resolveResultStatus({ ...counts(9, 9), caveats: ['logs-lost'] }),
+      resolveResultStatus(counts(0, 0)),
+    ];
+
+    expect(tallyCaveats(statuses)).toEqual([
+      {
+        caveat: 'logs-lost',
+        count: 2,
+        message: expect.stringContaining('2 package(s) lost their log output'),
+      },
+      {
+        caveat: 'empty-run',
+        count: 1,
+        message: '1 package(s) ran 0 tests — check test discovery',
+      },
+    ]);
+  });
+
+  it('says nothing when every result is clean', () => {
+    expect(tallyCaveats([resolveResultStatus(counts(3, 3))])).toEqual([]);
+  });
+});
+
+describe('statusIcon', () => {
+  it('uses emoji for markdown and single-width glyphs for a terminal', () => {
+    // A variation-selector emoji is two columns wide, which breaks the aligned
+    // status column in the summary table.
+    expect(statusIcon('warning', 'emoji')).toBe('⚠️');
+    expect(statusIcon('warning', 'ascii')).toBe('⚠');
+    expect(statusIcon('success', 'ascii')).toBe('✓');
+    expect(statusIcon('failure', 'ascii')).toBe('✗');
+  });
+});