@vitest-agent/ui 5.1.1 → 5.2.0

package/dispatcher/helpers.js

@@ -152,22 +152,23 @@ const formatTrendLine = (trend) => {
 	return `Trend: ${trend.direction} (${Fmt.plural(trend.runCount, "run")})`;
 };
 /**
-* Format a compact projects-table line for one project. Pads the name
-* column to `nameWidth` so a column of rows aligns when joined with
-* newlines. Status glyph is `✓` for clean projects, `✗` for any
-* project carrying failures or violations.
+* The two cells of one project's row: the status glyph with the name, then
+* the counts, duration and (for a multi-tag project) the tag counts. The
+* glyph is `✓` for a clean project and `✗` for one carrying failures or
+* timeouts.
 */
-const formatProjectRow = (project, nameWidth) => {
+const projectCells = (project) => {
 	const timeoutCount = project.timeoutCount ?? 0;
 	const total = project.passCount + project.failCount + project.skipCount + timeoutCount;
-	const glyph = statusGlyph(project.failCount > 0 || timeoutCount > 0 ? "failure" : "success");
+	const failing = project.failCount > 0 || timeoutCount > 0;
+	const glyph = statusGlyph(failing ? "failure" : "success");
 	const countParts = [`${project.passCount}/${total} passed`];
 	if (project.failCount > 0) countParts.push(`${project.failCount} failed`);
 	if (timeoutCount > 0) countParts.push(`${timeoutCount} timed out`);
-	const counts = project.failCount > 0 || timeoutCount > 0 ? countParts.join(", ") : `${project.passCount} passed`;
+	const counts = failing ? countParts.join(", ") : `${project.passCount} passed`;
 	const tagSuffix = formatTagCountSuffix(project.tagCounts);
-	const base = `  ${glyph} ${project.name.padEnd(nameWidth)} ${counts} (${formatDisplayDuration(project.durationMs)})`;
-	return tagSuffix.length === 0 ? base : `${base}  ${tagSuffix}`;
+	const base = `${counts} (${formatDisplayDuration(project.durationMs)})`;
+	return [`${glyph} ${project.name}`, tagSuffix.length === 0 ? base : `${base}  ${tagSuffix}`];
 };
 const formatTagCountSuffix = (tagCounts) => {
 	if (tagCounts === void 0) return "";
@@ -176,14 +177,29 @@ const formatTagCountSuffix = (tagCounts) => {
 	return [...entries].sort(([a], [b]) => a.localeCompare(b)).map(([tag, count]) => `${tag}:${count}`).join("  ");
 };
 /**
+* A render width no table reaches, so the kit never truncates a cell: a
+* project name or a coverage path is printed whole however long it is
+* (issue #237 follow-up).
+*/
+const UNBOUNDED_WIDTH = Number.MAX_SAFE_INTEGER;
+/**
 * Format the `Projects (N):` block as an array of lines including the
-* leading header and each project row. The longest project name sets
-* the padded column width so the counts align vertically.
+* leading header and each project row, indented two spaces.
+*
+* @remarks
+* The rows are a header-less, space-aligned kit `Doc.table` rendered plain,
+* so the name column is padded by display width (a wide or combining
+* character in a project name keeps the counts aligned) rather than by
+* UTF-16 length.
 */
 const formatProjectsTable = (projects) => {
 	if (projects.length === 0) return [];
-	const nameWidth = projects.reduce((max, p) => Math.max(max, p.name.length), 0);
-	return [`Projects (${projects.length}):`, ...projects.map((p) => formatProjectRow(p, nameWidth))];
+	const table = Doc.table([{ header: "" }, { header: "" }], projects.map(projectCells));
+	const rows = Render.plain([table], Render.contextOf({
+		audience: "agent",
+		width: UNBOUNDED_WIDTH
+	})).split("\n").map((row) => `  ${row}`);
+	return [`Projects (${projects.length}):`, ...rows];
 };
 /**
 * Format the `Total:` footer for a workspace run.
@@ -197,55 +213,55 @@ const formatWorkspaceTotal = (projects) => formatTotalsLine({
 	durationMs: sumOf(projects, (p) => p.durationMs)
 });
 const sumOf = (projects, pick) => projects.reduce((sum, p) => sum + pick(p), 0);
-const TABLE_COL_FILE_MIN = 60;
 /**
-* Format the `Files below aspirational target:` block — a pipe-delimited
-* table truncated to the first `limit` files with a "+N more" suffix.
-* Returns an empty array when `belowTarget` is empty.
+* Format the `Files below aspirational target:` block — an istanbul-shaped
+* pipe table (`Doc.table` with `style: "pipe"`) of the first `limit` files,
+* followed by a `… N more` line when files were omitted. Returns an empty
+* array when `belowTarget` is empty.
 *
-* The file column is sized to the longest path among the printed rows
-* (never narrower than {@link TABLE_COL_FILE_MIN}), so a printed path is
-* never truncated (issue #237 follow-up) — only the omitted rows (past
-* `limit`) are unaffected, since their width never entered the
-* calculation. Each path is written through `link` after the column is
-* padded, so a hyperlink wraps the path alone and never the padding.
+* @remarks
+* The table renders at {@link UNBOUNDED_WIDTH}, so the kit never truncates a
+* cell and a printed path is always whole (issue #237 follow-up). Columns are
+* padded by display width and the percentage cells are centred by the kit.
+* Each path is a `Doc.link` written through `link`, so a hyperlink wraps the
+* path alone and never the padding. It renders through `Render.ansi` at
+* colour `none` (as {@link formatFailure} does), since `plain` never calls
+* `link`.
 */
 const formatBelowTargetTable = (belowTarget, limit, link = (_target, label) => label) => {
 	if (belowTarget.length === 0) return [];
-	const top = belowTarget.slice(0, limit);
-	const omitted = belowTarget.length - top.length;
-	const fileColWidth = Math.max(TABLE_COL_FILE_MIN, ...top.map((f) => f.file.length + 2));
-	const header = [
-		"Files below aspirational target:",
-		buildTableSeparator(fileColWidth),
-		buildTableHeader(fileColWidth),
-		buildTableSeparator(fileColWidth)
-	];
-	const rows = top.map((file) => buildTableRow(file, fileColWidth, link));
-	const footer = [];
-	if (omitted > 0) footer.push(`… ${omitted} more (use the test_coverage MCP tool for the full list)`);
-	return [
-		...header,
-		...rows,
-		...footer
-	];
-};
-const buildTableSeparator = (fileColWidth) => {
-	return `${"-".repeat(fileColWidth)}|---------|---------|---------|---------|-------------------`;
-};
-const buildTableHeader = (fileColWidth) => {
-	return ` ${"File".padEnd(fileColWidth - 1)}| % Stmts | % Branch| % Funcs | % Lines | Uncovered Line #s`;
-};
-const buildTableRow = (file, fileColWidth, link) => {
-	return `${` ${link({ file: file.file }, file.file)}${" ".repeat(Math.max(0, fileColWidth - 1 - file.file.length))}`}|${pctCell(file.summary.statements)}|${pctCell(file.summary.branches)}|${pctCell(file.summary.functions)}|${pctCell(file.summary.lines)}|${` ${file.uncoveredLines}`}`;
-};
-const pctCell = (n) => {
-	const text = `${Math.round(n)}`;
-	const pad = Math.max(0, 9 - text.length);
-	const left = Math.floor(pad / 2);
-	const right = pad - left;
-	return `${" ".repeat(left + 1)}${text}${" ".repeat(right)}`;
+	const pct = (header) => ({
+		header,
+		align: "center"
+	});
+	const table = Doc.table([
+		{ header: "File" },
+		pct("% Stmts"),
+		pct("% Branch"),
+		pct("% Funcs"),
+		pct("% Lines"),
+		{ header: "Uncovered Line #s" }
+	], belowTarget.map((file) => [
+		Doc.link({ file: file.file }, file.file, { suffix: false }),
+		pctCell(file.summary.statements),
+		pctCell(file.summary.branches),
+		pctCell(file.summary.functions),
+		pctCell(file.summary.lines),
+		file.uncoveredLines
+	]), {
+		style: "pipe",
+		cap: limit,
+		overflow: (hidden) => `… ${hidden} more (use the test_coverage MCP tool for the full list)`
+	});
+	return ["Files below aspirational target:", ...Render.ansi([table], {
+		...Render.contextOf({
+			audience: "human",
+			width: UNBOUNDED_WIDTH
+		}),
+		link: (target, label) => "file" in target ? link(target, label) : label
+	}).split("\n")];
 };
+const pctCell = (n) => `${Math.round(n)}`;
 const SEVERITY_LABEL = {
 	info: "info",
 	warn: "warn",
@@ -256,6 +272,8 @@ const SEVERITY_LABEL = {
 * `- <severity>: <title> (tool: <targetTool>)` with the detail indented under
 * it. Empty when there is nothing to suggest. Shared by `renderAgent` and the
 * dispatcher, so the one-shot and cell paths print the same list (issue #544).
+*
+* @public
 */
 const formatSuggestedActionLines = (actions) => {
 	if (actions.length === 0) return [];
@@ -267,4 +285,4 @@ const formatSuggestedActionLines = (actions) => {
 	return lines;
 };
 
-export { formatBelowTargetTable, formatCoverageJudgmentLine, formatCoverageSummaryLines, formatDisplayDuration, formatFailure, formatGlobShortfallLines, formatPercent, formatProjectRow, formatProjectsTable, formatSuggestedActionLines, formatTestName, formatTotals, formatTrendLine, formatWorkspaceTotal, soleModulePath, soleTest };
+export { formatBelowTargetTable, formatCoverageJudgmentLine, formatCoverageSummaryLines, formatDisplayDuration, formatFailure, formatGlobShortfallLines, formatPercent, formatProjectsTable, formatSuggestedActionLines, formatTestName, formatTotals, formatTrendLine, formatWorkspaceTotal, soleModulePath, soleTest };

package/index.d.ts

@@ -1334,6 +1334,15 @@ export declare const buildFooter: (inputs: DispatchInputs) => string;
  * @public
  */
 export declare const formatDisplayDuration: (ms: number) => string;
+/**
+ * The suggested-actions block as a compact list: `Actions:` then, per action,
+ * `- <severity>: <title> (tool: <targetTool>)` with the detail indented under
+ * it. Empty when there is nothing to suggest. Shared by `renderAgent` and the
+ * dispatcher, so the one-shot and cell paths print the same list (issue #544).
+ *
+ * @public
+ */
+export declare const formatSuggestedActionLines: (actions: ReadonlyArray<SuggestedActionRecord>) => ReadonlyArray<string>;
 /**
  * Apply a single event to the previous state and return the next.
  *

package/index.js

@@ -2,6 +2,7 @@ import { classifyOutcome, classifyRunShape } from "./dispatcher/classify.js";
 import { buildFooter, dominantClassification } from "./dispatcher/footer.js";
 import { VitestAgentStatus, VitestAgentTokens, inkStyle, statusGlyph, statusInkStyle } from "./theme.js";
 import { formatDisplayDuration } from "./format-duration.js";
+import { formatSuggestedActionLines } from "./dispatcher/helpers.js";
 import { dispatch, dispatcherTable } from "./dispatcher/dispatch.js";
 import { reduceRenderState, reduceRenderStateAll } from "./reducer.js";
 import { renderAgent } from "./render-agent.js";
@@ -17,6 +18,6 @@ import { ActionSeverity, CoverageFile, CoverageGap, CoverageMetric, CoverageRend
 *
 * @public
 */
-const CURRENT_UI_VERSION = "5.1.1";
+const CURRENT_UI_VERSION = "5.2.0";
 
-export { ActionSeverity, CURRENT_UI_VERSION, CoverageFile, CoverageGap, CoverageMetric, CoverageRenderState, FailureRecord, ModuleRecord, RenderState, RenderTotals, RunEvent, SPINNER_FRAMES, SPINNER_FRAME_MS, SUITE_FAILURE_LABEL, SUITE_LOAD_FAILURE_LABEL, SuggestedActionRecord, TestRecord, VitestAgentStatus, VitestAgentTokens, buildFooter, classifyOutcome, classifyRunShape, dispatch, dispatcherTable, dominantClassification, formatDisplayDuration, initialRenderState, inkStyle, reduceRenderState, reduceRenderStateAll, renderAgent, spinnerFrame, statusGlyph, statusInkStyle, synthesizeFromAgentReport, synthesizeRunEvents };
+export { ActionSeverity, CURRENT_UI_VERSION, CoverageFile, CoverageGap, CoverageMetric, CoverageRenderState, FailureRecord, ModuleRecord, RenderState, RenderTotals, RunEvent, SPINNER_FRAMES, SPINNER_FRAME_MS, SUITE_FAILURE_LABEL, SUITE_LOAD_FAILURE_LABEL, SuggestedActionRecord, TestRecord, VitestAgentStatus, VitestAgentTokens, buildFooter, classifyOutcome, classifyRunShape, dispatch, dispatcherTable, dominantClassification, formatDisplayDuration, formatSuggestedActionLines, initialRenderState, inkStyle, reduceRenderState, reduceRenderStateAll, renderAgent, spinnerFrame, statusGlyph, statusInkStyle, synthesizeFromAgentReport, synthesizeRunEvents };

package/ink/CountColumns.js

@@ -1,4 +1,5 @@
 import { VitestAgentTokens, inkStyle, statusGlyph, statusInkStyle } from "../theme.js";
+import { Fmt } from "@effected/cli";
 import { useGlyphs } from "@effected/cli/ui";
 import { Text } from "ink";
 import { jsxs } from "react/jsx-runtime";
@@ -14,7 +15,7 @@ import { jsxs } from "react/jsx-runtime";
 const DURATION_CELL_WIDTH = 7;
 const cell = (count, status, glyphs) => /* @__PURE__ */ jsxs(Text, {
 	...count > 0 ? statusInkStyle(status) : inkStyle(VitestAgentTokens.zero),
-	children: [String(count).padStart(4), statusGlyph(status, glyphs)]
+	children: [Fmt.pad(String(count), 4, { align: "right" }), statusGlyph(status, glyphs)]
 });
 /**
 * Renders four colored count columns (pass ✓, fail ✗, skip ↷, timeout ⧖)

package/ink/ProjectRow.js

@@ -3,11 +3,19 @@ import { formatDisplayDuration } from "../format-duration.js";
 import { CountColumns, DURATION_CELL_WIDTH } from "./CountColumns.js";
 import { StatusIcon } from "./StatusIcon.js";
 import { TagColumns } from "./TagColumns.js";
+import { Fmt } from "@effected/cli";
 import { Box, Text } from "ink";
 import { Fragment, jsx, jsxs } from "react/jsx-runtime";
 
 // src/ink/ProjectRow.tsx
 /**
+* One row of the `workspace`-shape `stream` view: a per-Vitest-project
+* rollup. Carries the project name and the four-outcome count columns.
+*
+* The glyph animates while the project has running modules and resolves
+* to the outcome on finish: ✗ failure, ⧖ timeout, ↷ skip-only, ✓ pass.
+*/
+/**
 * Pick the glyph element for the project row using the spec's severity
 * precedence (`✗ > ⧖ > ⚠ > ↷ > ✓`). Skip-only projects (no passes, no
 * failures, no timeouts) resolve to `↷`, not a false-positive `✓`.
@@ -34,7 +42,7 @@ const ProjectRow = ({ project, counts, running, timedOut = false, elapsedMs, fra
 	projectGlyph(counts, running, timedOut, frame),
 	/* @__PURE__ */ jsxs(Text, { children: [
 		" ",
-		project.name.padEnd(nameWidth),
+		Fmt.pad(project.name, nameWidth),
 		" "
 	] }),
 	/* @__PURE__ */ jsx(CountColumns, {
@@ -45,7 +53,7 @@ const ProjectRow = ({ project, counts, running, timedOut = false, elapsedMs, fra
 	}),
 	/* @__PURE__ */ jsxs(Text, {
 		dimColor: true,
-		children: [" ", formatDisplayDuration(elapsedMs).padStart(7)]
+		children: [" ", Fmt.pad(formatDisplayDuration(elapsedMs), 7, { align: "right" })]
 	}),
 	tagUnion.length > 0 ? /* @__PURE__ */ jsxs(Fragment, { children: [/* @__PURE__ */ jsx(Text, { children: "  " }), /* @__PURE__ */ jsx(TagColumns, {
 		tags: tagUnion,

package/ink/StreamApp.js

@@ -133,8 +133,7 @@ const ModuleStreamRow = ({ module, nowMs, frame, nameWidth, timedOut, tagUnion:
 		glyph,
 		/* @__PURE__ */ jsxs(Text, { children: [
 			" ",
-			link({ file: module.modulePath }, module.modulePath),
-			" ".repeat(Math.max(0, nameWidth - module.modulePath.length)),
+			Fmt.pad(link({ file: module.modulePath }, module.modulePath), nameWidth),
 			" "
 		] }),
 		/* @__PURE__ */ jsx(CountColumns, {
@@ -145,7 +144,7 @@ const ModuleStreamRow = ({ module, nowMs, frame, nameWidth, timedOut, tagUnion:
 		}),
 		/* @__PURE__ */ jsxs(Text, {
 			dimColor: true,
-			children: [" ", formatDisplayDuration(moduleElapsedMs(module, nowMs)).padStart(7)]
+			children: [" ", Fmt.pad(formatDisplayDuration(moduleElapsedMs(module, nowMs)), 7, { align: "right" })]
 		}),
 		rowTagUnion.length > 0 ? /* @__PURE__ */ jsxs(Fragment, { children: [/* @__PURE__ */ jsx(Text, { children: "  " }), /* @__PURE__ */ jsx(TagColumns, {
 			tags: rowTagUnion,
@@ -246,7 +245,7 @@ const FailureItem = ({ failure }) => {
 const TotalsLine = ({ totals, labelWidth }) => /* @__PURE__ */ jsxs(Text, { children: [
 	/* @__PURE__ */ jsx(Text, {
 		bold: true,
-		children: labelWidth !== void 0 ? "Total:".padEnd(labelWidth) : "Total:"
+		children: labelWidth !== void 0 ? Fmt.pad("Total:", labelWidth) : "Total:"
 	}),
 	" ",
 	/* @__PURE__ */ jsx(CountColumns, {
@@ -257,7 +256,7 @@ const TotalsLine = ({ totals, labelWidth }) => /* @__PURE__ */ jsxs(Text, { chil
 	}),
 	/* @__PURE__ */ jsxs(Text, {
 		dimColor: true,
-		children: [" ", formatDisplayDuration(totals.durationMs).padStart(7)]
+		children: [" ", Fmt.pad(formatDisplayDuration(totals.durationMs), 7, { align: "right" })]
 	})
 ] });
 /**
@@ -349,7 +348,7 @@ const liveRegion = (state, shape, nowMs, frame, glyphs, link) => {
 		})]
 	});
 	if (shape === "workspace") {
-		const nameWidth = Math.max(0, ...groups.map((g) => g.name.length));
+		const nameWidth = Math.max(0, ...groups.map((g) => Fmt.width(g.name)));
 		const groupTagCounts = new Map(groups.map((g) => [g.name, mergeTagCounts(g.modules)]));
 		const workspaceTags = tagUnion(groups.map((g) => groupTagCounts.get(g.name)));
 		let runningVisible = 0;
@@ -415,7 +414,7 @@ const liveRegion = (state, shape, nowMs, frame, glyphs, link) => {
 		] });
 	}
 	if (shape === "single-project") {
-		const nameWidth = Math.max(0, ...ordered.map((m) => m.modulePath.length));
+		const nameWidth = Math.max(0, ...ordered.map((m) => Fmt.width(m.modulePath)));
 		const moduleTags = tagUnion(ordered.map((m) => m.tagCounts));
 		let runningVisible = 0;
 		let runningOverflow = 0;

package/ink/TagColumns.js

@@ -1,4 +1,5 @@
 import { VitestAgentTokens, inkStyle } from "../theme.js";
+import { Fmt } from "@effected/cli";
 import { Text } from "ink";
 import { jsx, jsxs } from "react/jsx-runtime";
 
@@ -40,7 +41,7 @@ const TagColumns = ({ tags, counts }) => {
 				i > 0 ? "  " : "",
 				tag,
 				":",
-				String(count).padStart(4)
+				Fmt.pad(String(count), 4, { align: "right" })
 			]
 		}, tag);
 	}) });

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vitest-agent/ui",
-	"version": "5.1.1",
+	"version": "5.2.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": [
@@ -44,14 +44,14 @@
 		"./package.json": "./package.json"
 	},
 	"dependencies": {
-		"@vitest-agent/sdk": "8.0.0",
+		"@vitest-agent/sdk": "9.0.0",
 		"effect": "^4.0.6"
 	},
 	"peerDependencies": {
-		"@effected/cli": "^0.22.1",
-		"@effected/env": "^0.5.0",
-		"@effected/glob": "^0.14.0",
-		"@effected/walker": "^0.20.0",
+		"@effected/cli": "^0.23.0",
+		"@effected/env": "^0.6.0",
+		"@effected/glob": "^0.15.0",
+		"@effected/walker": "^0.21.0",
 		"ink": "^8.0.0",
 		"react": "^19.3.0"
 	},