@vitest-agent/reporter 6.0.1 → 6.1.0

package/defaultReporter.js

@@ -4,7 +4,7 @@ import { renderGithubLog, strayOutputOf, toDisplayPath } from "./githubLog.js";
 import { LiveViewEnv, startLiveView } from "./liveView.js";
 import { Doc, Fmt, Render, Status } from "@effected/cli";
 import { RUN_REPORT_FILE_SCHEMA_URL, formatStrayOutputNote } from "@vitest-agent/sdk";
-import { classifyOutcome, classifyRunShape, dispatch, reduceRenderStateAll, synthesizeFromAgentReport } from "@vitest-agent/ui";
+import { classifyOutcome, classifyRunShape, dispatch, formatSuggestedActionLines, reduceRenderStateAll, synthesizeFromAgentReport } from "@vitest-agent/ui";
 import { Effect } from "effect";
 
 // src/defaultReporter.ts
@@ -328,13 +328,22 @@ const renderGithubSummary = (input) => {
 */
 const DefaultVitestAgentReporter = (kit) => {
 	const live = kit.config.consoleMode === "stream" && kit.runEvents !== void 0 ? startLiveView(kit.runEvents) : void 0;
+	let actionLines = [];
+	let actionsPrinted = false;
+	const closeLive = live !== void 0 ? async () => {
+		await live.close();
+		if (actionsPrinted || actionLines.length === 0) return;
+		actionsPrinted = true;
+		process.stdout.write(`\n${actionLines.join("\n")}\n`);
+	} : void 0;
 	return {
-		...live !== void 0 ? {
-			close: live.close,
-			printStrayLine: live.printStrayLine
+		...live !== void 0 && closeLive !== void 0 ? {
+			close: closeLive,
+			printStrayLine: live.printAbove
 		} : {},
 		render(input, renderKit) {
 			const out = [];
+			if (live !== void 0) actionLines = formatSuggestedActionLines(input.suggestedActions ?? []);
 			if (shouldRenderForMode(renderKit.config.consoleMode)) {
 				const events = input.reports.flatMap((r) => synthesizeFromAgentReport(r, { ...input.suggestedActions !== void 0 ? { suggestedActions: input.suggestedActions } : {} }));
 				const state = reduceRenderStateAll(events);
@@ -358,7 +367,8 @@ const DefaultVitestAgentReporter = (kit) => {
 					$schema: RUN_REPORT_FILE_SCHEMA_URL,
 					schemaVersion: 1,
 					generatedAt: (/* @__PURE__ */ new Date()).toISOString(),
-					reports: input.reports
+					reports: input.reports,
+					...input.runDiagnostics !== void 0 ? { runDiagnostics: input.runDiagnostics } : {}
 				}, null, 2)}\n`
 			});
 			out.push({

package/index.d.ts

@@ -616,6 +616,85 @@ type Executor = typeof Executor.Type;
 declare const DetailLevel: Schema.Literals<readonly ["minimal", "neutral", "standard", "verbose"]>;
 /** @public */
 type DetailLevel = typeof DetailLevel.Type;
+/**
+ * Which Vitest performance diagnostic a hint came from.
+ * @public
+ */
+declare const PerformanceHintKind: Schema.Literals<readonly ["environment", "import", "transform", "isolate"]>;
+/** @public */
+type PerformanceHintKind = typeof PerformanceHintKind.Type;
+/**
+ * One performance hint derived from Vitest's run diagnostics. `project` is
+ * omitted for a hint that applies to the whole run (for example `isolate`).
+ * @public
+ */
+declare const PerformanceHint: Schema.Struct<{
+  readonly kind: Schema.Literals<readonly ["environment", "import", "transform", "isolate"]>;
+  readonly project: Schema.optionalKey<Schema.String>;
+  readonly message: Schema.String;
+  readonly remedy: Schema.String;
+  readonly detail: Schema.String;
+  readonly learnMoreUrl: Schema.String;
+  readonly estimatedSavingMs: Schema.optionalKey<Schema.Finite>;
+  readonly share: Schema.optionalKey<Schema.Finite>;
+}>;
+/** @public */
+type PerformanceHint = typeof PerformanceHint.Type;
+/**
+ * Wall-clock timing of the plugin's post-run work, in milliseconds.
+ * @public
+ */
+declare const PostRunTiming: Schema.Struct<{
+  readonly persistMs: Schema.Finite;
+  readonly coverageScanMs: Schema.Finite;
+  readonly totalMs: Schema.Finite;
+  readonly renderMs: Schema.optionalKey<Schema.Finite>;
+  readonly vitestDurationMs: Schema.optionalKey<Schema.Finite>;
+}>;
+/** @public */
+type PostRunTiming = typeof PostRunTiming.Type;
+/**
+ * Whether the run's results were persisted to the database.
+ * @public
+ */
+declare const PersistenceStatus: Schema.Struct<{
+  readonly ok: Schema.Boolean;
+  readonly reason: Schema.optionalKey<Schema.String>;
+  readonly migrationFailed: Schema.optionalKey<Schema.Boolean>;
+}>;
+/** @public */
+type PersistenceStatus = typeof PersistenceStatus.Type;
+/**
+ * Run-level diagnostics: performance hints, post-run timing and persistence
+ * status (issue #544).
+ * @public
+ */
+declare const RunDiagnostics: Schema.Struct<{
+  readonly performanceHints: Schema.$Array<Schema.Struct<{
+    readonly kind: Schema.Literals<readonly ["environment", "import", "transform", "isolate"]>;
+    readonly project: Schema.optionalKey<Schema.String>;
+    readonly message: Schema.String;
+    readonly remedy: Schema.String;
+    readonly detail: Schema.String;
+    readonly learnMoreUrl: Schema.String;
+    readonly estimatedSavingMs: Schema.optionalKey<Schema.Finite>;
+    readonly share: Schema.optionalKey<Schema.Finite>;
+  }>>;
+  readonly postRun: Schema.optionalKey<Schema.Struct<{
+    readonly persistMs: Schema.Finite;
+    readonly coverageScanMs: Schema.Finite;
+    readonly totalMs: Schema.Finite;
+    readonly renderMs: Schema.optionalKey<Schema.Finite>;
+    readonly vitestDurationMs: Schema.optionalKey<Schema.Finite>;
+  }>>;
+  readonly persistence: Schema.optionalKey<Schema.Struct<{
+    readonly ok: Schema.Boolean;
+    readonly reason: Schema.optionalKey<Schema.String>;
+    readonly migrationFailed: Schema.optionalKey<Schema.Boolean>;
+  }>>;
+}>;
+/** @public */
+type RunDiagnostics = typeof RunDiagnostics.Type;
 /**
  * The reducer projects this union into `RenderState`. Renderers
  * never read the raw event stream — they read the projected state.
@@ -1058,6 +1137,8 @@ interface ReporterRenderInput {
    * when the plugin has nothing to suggest (issue #544).
    */
   readonly suggestedActions?: ReadonlyArray<SuggestedActionRecord>;
+  /** Run-level diagnostics (performance hints, post-run timing, persistence status); absent when none were collected (issue #544). */
+  readonly runDiagnostics?: RunDiagnostics;
   readonly trendSummary?: {
     readonly direction: "improving" | "regressing" | "stable";
     readonly runCount: number;
@@ -1233,5 +1314,5 @@ export declare const DefaultVitestAgentReporter: VitestAgentReporterFactory;
  * @public
  */
 export declare const CURRENT_REPORTER_VERSION: string;
-export type { AgentReport, CellOptions, ConsoleMode, DetailLevel, DispatchInputs, Environment, Executor, FileCoverageReport, FileLinkTarget, HumanReportRenderOptions, ProjectSummary, RenderState, RenderedOutput, ReportRenderOptions, ReporterKit, ReporterRenderInput, ResolvedReporterConfig, ResolvedThresholds, RunEvent, RunOutcome, RunShape, SuggestedActionRecord, TestClassification, Transport, TrendSummary, VitestAgentReporter, VitestAgentReporterFactory };
+export type { AgentReport, CellOptions, ConsoleMode, DetailLevel, DispatchInputs, Environment, Executor, FileCoverageReport, FileLinkTarget, HumanReportRenderOptions, PerformanceHint, PerformanceHintKind, PersistenceStatus, PostRunTiming, ProjectSummary, RenderState, RenderedOutput, ReportRenderOptions, ReporterKit, ReporterRenderInput, ResolvedReporterConfig, ResolvedThresholds, RunDiagnostics, RunEvent, RunOutcome, RunShape, SuggestedActionRecord, TestClassification, Transport, TrendSummary, VitestAgentReporter, VitestAgentReporterFactory };
 //# sourceMappingURL=index.d.ts.map

package/index.js

@@ -8,6 +8,6 @@ import { DefaultVitestAgentReporter, buildDispatchInputs, renderAgentStringForRe
 *
 * @public
 */
-const CURRENT_REPORTER_VERSION = "6.0.1";
+const CURRENT_REPORTER_VERSION = "6.1.0";
 
 export { CURRENT_REPORTER_VERSION, DefaultVitestAgentReporter, buildDispatchInputs, renderAgentStringForReport, renderHumanStringForReport };

package/liveView.js

@@ -2,7 +2,7 @@ import { plainFileLink, resolveFileLink } from "./fileLinks.js";
 import { CliEnv, Doc } from "@effected/cli";
 import { initialRenderState } from "@vitest-agent/sdk";
 import { SPINNER_FRAME_MS, reduceRenderState, renderAgent } from "@vitest-agent/ui";
-import { Cause, Deferred, Effect, Exit, Layer, PubSub, Scope } from "effect";
+import { Cause, Effect, Exit, Layer } from "effect";
 import * as NodeServices from "@effect/platform-node/NodeServices";
 import { CliUi } from "@effected/cli/ui";
 
@@ -29,29 +29,35 @@ import { CliUi } from "@effected/cli/ui";
 * modules, coverage, suggested actions), each line kept whole; it does not
 * try to reproduce the Ink frame, and drawing it never renders through Ink.
 *
-* Lifetime: one scope per live view, held for the reporter's whole life. The
-* subscription is made in it synchronously, before the factory returns, so
-* the first `RunStarted` is never lost, and the view takes from it directly
-* (`events: subscription`). At Vitest's close the plugin calls
-* {@link LiveRunView.close} before it shuts the channel down: close runs the
-* kit's `LiveHandle.close` (it folds every message still queued in the
-* subscription and commits the last run, so a tail published just before the
-* close is kept), then closes the scope. The view never relies on
+* Lifetime: the kit's `CliUi.liveDetached` runs the view outside Effect for
+* the reporter's whole life. It subscribes to the channel synchronously,
+* before the factory returns, so the first `RunStarted` is never lost. At
+* Vitest's close the plugin calls the view's `close` before it shuts the
+* channel down: `close` folds every message still queued in the
+* subscription and commits the last run (so a tail published just before
+* the close is kept), then releases the view. The view never relies on
 * `PubSub.shutdown` to end it: shutdown drops whatever a subscriber has not
 * taken yet. It is never closed at `onTestRunEnd`, which fires on every
 * watch rerun.
 */
 /**
+* The Ink frame's draw function for a view whose paths are drawn through
+* `link`. The one sanctioned dynamic import outside mcp's `main.ts` (with
+* `humanReport.ts`'s): React and Ink load only when a run draws. The view
+* module exports a factory, so the writer reaches the tree without this
+* module importing React.
+*/
+const streamRender = (link) => CliUi.lazyView(() => import("./streamView.js").then((view) => view.makeStreamView(link)));
+/**
 * The live view's options, everything but `events`, for a view whose paths
-* are drawn through `link`: what the reporter hands `CliUi.live` once it has
-* resolved the writer from its environment (`resolveFileLink`).
+* are drawn through `link`.
 *
 * @internal
 */
 const liveViewOptionsWith = (link) => ({
 	initial: initialRenderState,
 	reduce: reduceRenderState,
-	render: CliUi.lazyView(() => import("./streamView.js").then((view) => view.makeStreamView(link))),
+	render: streamRender(link),
 	isStart: (event) => event._tag === "RunStarted",
 	isTerminal: (event) => event._tag === "RunFinished" || event._tag === "RunTimedOut",
 	begins: (event, before, after) => event._tag === "RunStarted" || before.phase === "idle" && after.phase !== "idle",
@@ -81,21 +87,17 @@ const liveViewOptions = liveViewOptionsWith(plainFileLink);
 */
 const LiveViewEnv = CliEnv.layer().pipe(Layer.provide(NodeServices.layer));
 /**
-* Print one line of stray output (bytes a test process wrote straight to the
-* terminal) through the view's `logConsole`: above the frame while a run is
-* drawn, straight to the stream otherwise, so it never lands under the frame
-* where the next redraw would strand it.
+* Start a live view over `channel` with the kit's `CliUi.liveDetached`. The
+* subscription is taken before this returns, so an event published right
+* after is seen.
 *
-* @internal
-*/
-const printAbove = (handle) => (stream, line) => {
-	if (stream === "stderr") handle.logConsole.error(line);
-	else handle.logConsole.log(line);
-	return true;
-};
-/**
-* Start a live view over `channel`. The subscription is taken before this
-* returns, so an event published right after is seen.
+* The returned view's `printAbove` is the reporter's `printStrayLine`: it
+* prints the line above the frame and answers `true` only while a run's
+* frame is mounted, declining (`false`, nothing written) before the view is
+* up, between runs, when the view is not interactive (it never mounts), and
+* once `close` has been called, so the plugin writes the line itself.
+* `close` is idempotent and never rejects; a view that died of anything but
+* an interrupt is logged as a warning.
 *
 * @param channel - the plugin's run-event channel
 * @param env - the environment the view reads; {@link LiveViewEnv} by default (a test
@@ -104,29 +106,15 @@ const printAbove = (handle) => (stream, line) => {
 * @internal
 */
 const startLiveView = (channel, env = LiveViewEnv) => {
-	const scope = Effect.runSync(Scope.make());
-	const subscription = Effect.runSync(PubSub.subscribe(channel).pipe(Scope.provide(scope)));
-	const mounted = Effect.runSync(Deferred.make());
-	const program = resolveFileLink.pipe(Effect.flatMap((link) => CliUi.live({
-		...liveViewOptionsWith(link),
-		events: subscription
-	})), Effect.onExit((exit) => Deferred.done(mounted, exit)), Effect.flatMap((handle) => handle.done), Scope.provide(scope), Effect.provide(env));
-	Effect.runFork(program);
-	let closing;
-	let print;
-	const handle = Effect.runPromise(Deferred.await(mounted));
-	handle.then((mountedHandle) => {
-		if (closing === void 0) print = printAbove(mountedHandle);
-	}, () => void 0);
-	return {
-		handle,
-		printStrayLine: (stream, line) => print?.(stream, line) ?? false,
-		close: () => {
-			print = void 0;
-			closing ??= Effect.runPromise(Deferred.await(mounted).pipe(Effect.flatMap((handle) => handle.close), Effect.catchCause((cause) => Cause.hasInterruptsOnly(cause) ? Effect.void : Effect.logWarning("vitest-agent: the live view ended with an error", cause)), Effect.ensuring(Scope.close(scope, Exit.void))));
-			return closing;
-		}
-	};
+	const view = CliUi.liveDetached({
+		...liveViewOptions,
+		render: Effect.map(resolveFileLink, streamRender),
+		events: channel
+	}, env);
+	view.exit.then((exit) => {
+		if (Exit.isFailure(exit) && !Cause.hasInterruptsOnly(exit.cause)) Effect.runFork(Effect.logWarning("vitest-agent: the live view ended with an error", exit.cause));
+	});
+	return view;
 };
 
-export { LiveViewEnv, liveViewOptionsWith, printAbove, startLiveView };
+export { LiveViewEnv, liveViewOptions, liveViewOptionsWith, startLiveView };

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vitest-agent/reporter",
-	"version": "6.0.1",
+	"version": "6.1.0",
 	"private": false,
 	"description": "Named VitestAgentReporterFactory implementations for the vitest-agent ecosystem.",
 	"keywords": [
@@ -40,31 +40,17 @@
 	},
 	"dependencies": {
 		"@effect/platform-node": "^4.0.6",
-		"@vitest-agent/sdk": "8.0.0",
-		"@vitest-agent/ui": "5.1.1",
+		"@vitest-agent/sdk": "9.0.0",
+		"@vitest-agent/ui": "5.2.0",
 		"effect": "^4.0.6",
 		"ink": "^8.0.0",
 		"react": "^19.3.1"
 	},
 	"peerDependencies": {
-		"@effected/cli": "^0.22.1",
-		"@effected/env": "^0.5.0",
-		"@effected/glob": "^0.14.0",
-		"@effected/walker": "^0.20.0",
-		"@vitest/coverage-istanbul": "^5.0.0",
-		"@vitest/coverage-v8": "^5.0.0",
-		"vitest": "^5.0.0"
-	},
-	"peerDependenciesMeta": {
-		"@vitest/coverage-istanbul": {
-			"optional": true
-		},
-		"@vitest/coverage-v8": {
-			"optional": true
-		},
-		"vitest": {
-			"optional": false
-		}
+		"@effected/cli": "^0.23.0",
+		"@effected/env": "^0.6.0",
+		"@effected/glob": "^0.15.0",
+		"@effected/walker": "^0.21.0"
 	},
 	"engines": {
 		"node": ">=24.11.0"