@vitest-agent/ui 2.0.15 → 2.1.0

package/dispatcher/cells/single-project-pass.js

@@ -4,9 +4,10 @@ import { renderAgentStringAsInk } from "../ink-helpers.js";
 
 //#region src/dispatcher/cells/single-project-pass.ts
 const renderAgent = (inputs) => {
-	const modules = inputs.state.moduleOrder.length;
-	const noun = modules === 1 ? "module" : "modules";
-	return `${[formatTotals(inputs.state), `${modules} ${noun} all-passed.`].join("\n\n")}\n${buildFooter(inputs)}`;
+	const { passCount, failCount, skipCount, timeoutCount } = inputs.state.totals;
+	if (passCount + failCount + skipCount + timeoutCount === 0) return `${[formatTotals(inputs.state), "0 tests collected. A zero-test run usually means a wrong working directory, a filter that matched nothing, or a load-time error — verify before trusting it."].join("\n\n")}\n${buildFooter(inputs)}`;
+	const collected = inputs.state.collectedModules ?? inputs.state.moduleOrder.length;
+	return `${(collected === 0 ? [formatTotals(inputs.state)] : [formatTotals(inputs.state), `${collected} ${collected === 1 ? "module" : "modules"} all-passed.`]).join("\n\n")}\n${buildFooter(inputs)}`;
 };
 const renderSingleProjectPass = {
 	agent: renderAgent,

package/dispatcher/helpers.js

@@ -18,7 +18,8 @@ const formatTotals = (state) => {
 	const parts = [`${passCount}/${passCount + failCount + skipCount} passed`];
 	if (failCount > 0) parts.push(`${failCount} failed`);
 	if (skipCount > 0) parts.push(`${skipCount} skipped`);
-	return `Tests: ${parts.join(", ")} (${formatDisplayDuration(durationMs)})`;
+	const suffix = state.collectedModules !== void 0 && state.collectedModules > 0 ? ` across ${state.collectedModules} files` : "";
+	return `Tests: ${parts.join(", ")} (${formatDisplayDuration(durationMs)})${suffix}`;
 };
 /**
 * Locate the sole {@link TestRecord} in a `single-test` shape state.

package/index.d.ts

@@ -297,6 +297,16 @@ declare const RenderState: Schema.Struct<{
     readonly detail: Schema.String;
     readonly targetTool: Schema.optional<Schema.String>;
   }>>;
+  /**
+   * Count of every collected module, passing modules included. Folded
+   * from `RunFinished.collectedModules`. Renderers prefer this over
+   * `moduleOrder.length` for "N modules all-passed" copy — a report
+   * replay only queues failing modules, so `moduleOrder` alone
+   * undercounts a green run to zero (issue #204). Optional — a run
+   * that never carried the field (older replay data, hand-built
+   * fixtures) falls back to `moduleOrder.length`.
+   */
+  readonly collectedModules: Schema.optional<Schema.Number>;
 }>;
 /** @public */
 type RenderState = typeof RenderState.Type;
@@ -443,6 +453,14 @@ declare const AgentReport: Schema.Struct<{
     readonly failed: Schema.Number;
     readonly skipped: Schema.Number;
     readonly duration: Schema.Number;
+    /**
+     * Count of every collected module, passing modules included.
+     * `AgentReport.failed` only carries failing modules, so a green run
+     * has no other way to know how many files actually ran — without
+     * this a fully-passing replay reads as "0 modules all-passed"
+     * (issue #204). Optional to keep existing fixtures/reports valid.
+     */
+    readonly modules: Schema.optional<Schema.Number>;
   }>;
   readonly failed: Schema.$Array<Schema.Struct<{
     readonly file: Schema.String;
@@ -761,6 +779,14 @@ declare const RunEvent: Schema.Union<readonly [Schema.TaggedStruct<"RunStarted",
   readonly skipCount: Schema.Number;
   readonly durationMs: Schema.Number;
   readonly timeoutCount: Schema.optional<Schema.Number>;
+  /**
+   * Count of every collected module, passing modules included.
+   * Carries `AgentReport.summary.modules` (or the live module
+   * count) into the event stream so the reducer can fold the
+   * true count into `RenderState` even when `moduleOrder` only
+   * tracks failing modules (report replay). See issue #204.
+   */
+  readonly collectedModules: Schema.optional<Schema.Number>;
 }>]>;
 /** @public */
 type RunEvent = typeof RunEvent.Type;
@@ -898,6 +924,12 @@ interface VitestTestSuite {
     line: number;
     column: number;
   };
+  /**
+   * Errors attached to the suite entity itself — a `beforeAll`/`afterAll`
+   * hook throw lands here, not on any individual `TestCase`. Optional
+   * because older duck-typed callers may not supply it.
+   */
+  errors?(): Array<VitestModuleError>;
 }
 /**
  * Duck-typed Vitest TestModule from the Reporter v2 API.
@@ -1236,6 +1268,7 @@ declare const RunEventChannel_base: Context.ServiceClass<RunEventChannel, "vites
   readonly skipCount: number;
   readonly durationMs: number;
   readonly timeoutCount?: number | undefined;
+  readonly collectedModules?: number | undefined;
 }>>;
 /**
  * Service tag for the run-event channel. Consumers depend on this tag;

package/index.js

@@ -33,7 +33,7 @@ import { ActionSeverity, CoverageFile, CoverageGap, CoverageMetric, CoverageRend
 *
 * @public
 */
-const CURRENT_UI_VERSION = "2.0.15";
+const CURRENT_UI_VERSION = "2.1.0";
 
 //#endregion
 export { ActionSeverity, CURRENT_UI_VERSION, CountColumns, CoverageBlock, CoverageFile, CoverageGap, CoverageMetric, CoverageRenderState, DURATION_CELL_WIDTH, FailureRecord, FailureSection, FailuresSection, ModuleHeader, ModuleRecord, ProjectRow, RenderState, RenderTotals, RunEvent, RunEventChannel, RunEventChannelLive, SPINNER_FRAMES, SPINNER_FRAME_MS, SUITE_LOAD_FAILURE_LABEL, StatusIcon, StreamApp, SuggestedActionRecord, SuggestedActions, TagColumns, TestRecord, TestRow, TrendLine, accumulateUntilFinished, buildFooter, classifyOutcome, classifyRunShape, dispatch, dispatchInk, dispatcherTable, dominantClassification, forEachRenderState, formatDisplayDuration, initialRenderState, publish, publishAll, reduceRenderState, reduceRenderStateAll, renderAgent, renderStateStream, spinnerFrame, spinnerFrameForTime, subscribeRaw, synthesizeFromAgentReport, synthesizeRunEvents, tagUnion };

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vitest-agent/ui",
-	"version": "2.0.15",
+	"version": "2.1.0",
 	"private": false,
 	"description": "Shared event-sourced renderer for vitest-agent. React Ink view for humans, plain-string view for agents, both fed by the same reducer over a RunEvent stream.",
 	"keywords": [
@@ -39,7 +39,7 @@
 		"./package.json": "./package.json"
 	},
 	"dependencies": {
-		"@vitest-agent/sdk": "2.0.15",
+		"@vitest-agent/sdk": "2.1.0",
 		"effect": "4.0.0-beta.107"
 	},
 	"peerDependencies": {

package/reducer.js

@@ -218,7 +218,8 @@ const reduceRenderState = (state, event) => {
 				skipCount: e.skipCount,
 				timeoutCount: e.timeoutCount ?? state.totals.timeoutCount,
 				durationMs: e.durationMs
-			}
+			},
+			...e.collectedModules !== void 0 && { collectedModules: e.collectedModules }
 		}),
 		RunTimedOut: () => ({
 			...state,

package/render-agent.js

@@ -17,11 +17,14 @@ const formatHeader = (state) => {
 };
 const formatModulesSection = (state) => {
 	const modules = state.moduleOrder.map((path) => state.modules[path]).filter((m) => m !== void 0);
-	if (modules.length === 0) return null;
-	if (modules.filter((m) => m.failCount > 0 || m.skipCount > 0).length === 0) {
-		const noun = modules.length === 1 ? "module" : "modules";
-		return `${modules.length} ${noun} all-passed.`;
+	const { passCount, failCount, skipCount, timeoutCount } = state.totals;
+	if (passCount + failCount + skipCount + timeoutCount === 0) return "0 tests collected. A zero-test run usually means a wrong working directory, a filter that matched nothing, or a load-time error — verify before trusting it.";
+	const collected = state.collectedModules ?? modules.length;
+	if (modules.length === 0) {
+		if (collected === 0) return null;
+		return `${collected} ${collected === 1 ? "module" : "modules"} all-passed.`;
 	}
+	if (modules.filter((m) => m.failCount > 0 || m.skipCount > 0).length === 0) return `${collected} ${collected === 1 ? "module" : "modules"} all-passed.`;
 	const lines = ["Modules:"];
 	for (const m of modules) {
 		const parts = [];

package/synthesize.js

@@ -180,7 +180,8 @@ const synthesizeRunEvents = (modules, options = {}) => {
 		failCount: totalFail,
 		skipCount: totalSkip,
 		durationMs: totalDuration,
-		...totalTimeout > 0 && { timeoutCount: totalTimeout }
+		...totalTimeout > 0 && { timeoutCount: totalTimeout },
+		collectedModules: modules.length
 	});
 	return events;
 };
@@ -360,7 +361,8 @@ const synthesizeFromAgentReport = (report, options = {}) => {
 		failCount: report.summary.failed + suiteFailureCount,
 		skipCount: report.summary.skipped,
 		durationMs: report.summary.duration,
-		...totalTimeoutCount > 0 && { timeoutCount: totalTimeoutCount }
+		...totalTimeoutCount > 0 && { timeoutCount: totalTimeoutCount },
+		...report.summary.modules !== void 0 && { collectedModules: report.summary.modules }
 	});
 	return events;
 };