@vitest-agent/ui 5.0.9 → 5.1.1

package/dispatcher/cells/single-file-fail.js

@@ -4,7 +4,7 @@ import { formatFailure, soleModulePath } from "../helpers.js";
 
 // src/dispatcher/cells/single-file-fail.ts
 const DEFAULT_WIDTH = 80;
-const renderAgent = (inputs) => {
+const renderAgent = (inputs, opts) => {
 	const modulePath = soleModulePath(inputs.state);
 	if (modulePath === void 0) return "";
 	const { passCount, failCount, skipCount, timeoutCount, durationMs } = inputs.state.totals;
@@ -15,7 +15,7 @@ const renderAgent = (inputs) => {
 	const sections = [`${modulePath}: ${parts.join(", ")} (${formatDisplayDuration(durationMs)})`];
 	if (inputs.state.failures.length > 0) {
 		sections.push("Failures:");
-		for (const failure of inputs.state.failures) sections.push(...formatFailure(failure, DEFAULT_WIDTH));
+		for (const failure of inputs.state.failures) sections.push(...formatFailure(failure, DEFAULT_WIDTH, opts.link));
 	}
 	return `${sections.join("\n")}\n${buildFooter(inputs)}`;
 };

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

@@ -3,11 +3,11 @@ import { formatFailure, formatTotals } from "../helpers.js";
 
 // src/dispatcher/cells/single-project-fail.ts
 const DEFAULT_WIDTH = 80;
-const renderAgent = (inputs) => {
+const renderAgent = (inputs, opts) => {
 	const sections = [[formatTotals(inputs.state)]];
 	if (inputs.state.failures.length > 0) {
 		const failureLines = ["Failures:"];
-		for (const failure of inputs.state.failures) failureLines.push(...formatFailure(failure, DEFAULT_WIDTH));
+		for (const failure of inputs.state.failures) failureLines.push(...formatFailure(failure, DEFAULT_WIDTH, opts.link));
 		sections.push(failureLines);
 	}
 	const moduleLines = ["Modules:"];

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

@@ -3,7 +3,7 @@ import { formatBelowTargetTable, formatCoverageSummaryLines, formatTotals, forma
 
 // src/dispatcher/cells/single-project-threshold.ts
 const BELOW_TARGET_LIMIT = 5;
-const renderAgent = (inputs) => {
+const renderAgent = (inputs, opts) => {
 	const sections = [[formatTotals(inputs.state)]];
 	const coverage = formatCoverageSummaryLines(inputs.state);
 	const trend = formatTrendLine(inputs.trend);
@@ -11,7 +11,7 @@ const renderAgent = (inputs) => {
 	summaryLines.push(...coverage);
 	if (trend !== null) summaryLines.push(trend);
 	if (summaryLines.length > 0) sections.push(summaryLines);
-	const below = formatBelowTargetTable(inputs.belowTarget, BELOW_TARGET_LIMIT);
+	const below = formatBelowTargetTable(inputs.belowTarget, BELOW_TARGET_LIMIT, opts.link);
 	if (below.length > 0) sections.push([...below]);
 	return `${sections.map((section) => section.join("\n")).join("\n\n")}\n${buildFooter(inputs)}`;
 };

package/dispatcher/cells/single-test-fail.js

@@ -5,13 +5,13 @@ import { formatFailure, formatTestName, soleTest } from "../helpers.js";
 
 // src/dispatcher/cells/single-test-fail.ts
 const DEFAULT_WIDTH = 80;
-const renderAgent = (inputs) => {
+const renderAgent = (inputs, opts) => {
 	const test = soleTest(inputs.state);
 	if (test === void 0) return "";
 	const duration = test.durationMs ?? 0;
 	const header = `${statusGlyph("failure")} ${formatTestName(test)} (${formatDisplayDuration(duration)})`;
 	const failure = inputs.state.failures[0];
-	return `${failure === void 0 ? `${header}\n` : `${header}\n${formatFailure(failure, DEFAULT_WIDTH).join("\n")}\n`}${buildFooter(inputs)}`;
+	return `${failure === void 0 ? `${header}\n` : `${header}\n${formatFailure(failure, DEFAULT_WIDTH, opts.link).join("\n")}\n`}${buildFooter(inputs)}`;
 };
 const renderSingleTestFail = { agent: renderAgent };
 

package/dispatcher/cells/workspace-fail.js

@@ -3,13 +3,13 @@ import { formatFailure, formatProjectsTable, formatTrendLine, formatWorkspaceTot
 
 // src/dispatcher/cells/workspace-fail.ts
 const DEFAULT_WIDTH = 80;
-const renderAgent = (inputs) => {
+const renderAgent = (inputs, opts) => {
 	const sections = [];
 	const projects = formatProjectsTable(inputs.projects);
 	if (projects.length > 0) sections.push([...projects]);
 	if (inputs.state.failures.length > 0) {
 		const failureLines = ["Failures:"];
-		for (const failure of inputs.state.failures) failureLines.push(...formatFailure(failure, DEFAULT_WIDTH));
+		for (const failure of inputs.state.failures) failureLines.push(...formatFailure(failure, DEFAULT_WIDTH, opts.link));
 		sections.push(failureLines);
 	}
 	const trend = formatTrendLine(inputs.trend);

package/dispatcher/cells/workspace-threshold.js

@@ -3,7 +3,7 @@ import { formatBelowTargetTable, formatCoverageSummaryLines, formatProjectsTable
 
 // src/dispatcher/cells/workspace-threshold.ts
 const BELOW_TARGET_LIMIT = 10;
-const renderAgent = (inputs) => {
+const renderAgent = (inputs, opts) => {
 	const sections = [];
 	const projects = formatProjectsTable(inputs.projects);
 	if (projects.length > 0) sections.push([...projects]);
@@ -13,7 +13,7 @@ const renderAgent = (inputs) => {
 	const trend = formatTrendLine(inputs.trend);
 	if (trend !== null) summaryLines.push(trend);
 	if (summaryLines.length > 0) sections.push(summaryLines);
-	const below = formatBelowTargetTable(inputs.belowTarget, BELOW_TARGET_LIMIT);
+	const below = formatBelowTargetTable(inputs.belowTarget, BELOW_TARGET_LIMIT, opts.link);
 	if (below.length > 0) sections.push([...below]);
 	if (inputs.projects.length > 0) sections.push([formatWorkspaceTotal(inputs.projects)]);
 	return `${sections.map((section) => section.join("\n")).join("\n\n")}\n${buildFooter(inputs)}`;

package/dispatcher/dispatch.js

@@ -1,3 +1,4 @@
+import { formatSuggestedActionLines } from "./helpers.js";
 import { renderSingleFileFail } from "./cells/single-file-fail.js";
 import { renderSingleFilePass } from "./cells/single-file-pass.js";
 import { renderSingleFileThreshold } from "./cells/single-file-threshold.js";
@@ -10,7 +11,7 @@ import { renderSingleTestThreshold } from "./cells/single-test-threshold.js";
 import { renderWorkspaceFail } from "./cells/workspace-fail.js";
 import { renderWorkspacePass } from "./cells/workspace-pass.js";
 import { renderWorkspaceThreshold } from "./cells/workspace-threshold.js";
-import { formatScopedCoverageNote, formatStrayOutputNote } from "@vitest-agent/sdk";
+import { formatConsoleLeaksNote, formatScopedCoverageNote, formatStrayOutputNote } from "@vitest-agent/sdk";
 
 // src/dispatcher/dispatch.ts
 /**
@@ -55,8 +56,40 @@ const dispatch = (inputs, opts) => {
 	const body = dispatcherTable[inputs.shape][inputs.outcome].agent(inputs, opts);
 	const note = scopedCoverageNoteFor(inputs);
 	const withNote = note !== null ? `${body}\n${note}` : body;
-	const stray = strayOutputNoteFor(inputs);
-	return stray !== null ? `${withNote}\n${stray}\n` : withNote;
+	const trailers = [
+		suggestedActionsNoteFor(inputs),
+		strayOutputNoteFor(inputs),
+		consoleLeaksNoteFor(inputs)
+	].filter((trailer) => trailer !== null);
+	return trailers.length === 0 ? withNote : `${withNote}\n${trailers.join("\n\n")}\n`;
+};
+/**
+* The suggested-actions list for the run, or `null` when there is nothing to
+* suggest. `dispatch` and `dispatchInk` append it after the cell's output;
+* cells never emit it themselves.
+*
+* @param inputs - the classified dispatch inputs
+* @returns the list's lines joined, or `null`
+* @internal
+*/
+const suggestedActionsNoteFor = (inputs) => {
+	const lines = formatSuggestedActionLines(inputs.state.suggestedActions);
+	return lines.length === 0 ? null : lines.join("\n");
+};
+/**
+* The console-leaks note for a run whose passing tests wrote `console.*`
+* output, or `null` when none did. Appended after the cell's output like the
+* stray-output note; cells never emit it themselves.
+*
+* @param inputs - the classified dispatch inputs
+* @returns the note's lines joined, or `null`
+* @internal
+*/
+const consoleLeaksNoteFor = (inputs) => {
+	const leaks = inputs.state.consoleLeaks;
+	if (leaks === void 0) return null;
+	const lines = formatConsoleLeaksNote(leaks);
+	return lines.length === 0 ? null : lines.join("\n");
 };
 /**
 * The stray-output note for a run that wrote past Vitest's console capture,
@@ -88,4 +121,4 @@ const scopedCoverageNoteFor = (inputs) => {
 	return formatScopedCoverageNote(cov.scopedFiles ?? 0, cov.totalFiles);
 };
 
-export { dispatch, dispatcherTable, scopedCoverageNoteFor, strayOutputNoteFor };
+export { consoleLeaksNoteFor, dispatch, dispatcherTable, scopedCoverageNoteFor, strayOutputNoteFor, suggestedActionsNoteFor };

package/dispatcher/helpers.js

@@ -57,23 +57,30 @@ const formatTestName = (test) => {
 * by default to keep the agent string compact.
 *
 * @remarks
-* A kit document rendered plain for the agent audience: one compact list
-* item whose title is verbatim (never wrapped), whose first message line is
-* a truncated `Doc.line`, and whose diff is `Doc.diffText` with `truncate`.
+* A kit document rendered at colour `none` (so no paint, whatever the
+* audience): one compact list item whose title line is never wrapped and
+* opens with the test file as a `Doc.link` (written through `link`, so a
+* human at a terminal with hyperlink support gets an OSC 8 link to the
+* file and everyone else the path alone), whose first message line is a
+* truncated `Doc.line`, and whose diff is `Doc.diffText` with `truncate`.
 * Each content line is cut to `width - 2` (floor 20) after the item indent.
 * The kit sanitizes the text, so a tab becomes a space and an escape
-* sequence is removed.
+* sequence is removed — which is why the link is a `Doc.link` the renderer
+* writes, never an escape pasted into the text.
 */
-const formatFailure = (f, width) => {
+const formatFailure = (f, width, link = (_target, label) => label) => {
 	const suite = f.suitePath.length > 0 ? `${f.suitePath.join(" > ")} > ` : "";
 	const classification = f.classification !== null ? ` [${f.classification}]` : "";
-	const parts = [Doc.verbatim(`${f.modulePath} > ${suite}${f.testName}${classification}`)];
+	const parts = [Doc.line([Doc.link({ file: f.modulePath }, f.modulePath, { suffix: false }), Doc.text(` > ${suite}${f.testName}${classification}`)], { wrap: false })];
 	if (f.error?.message !== void 0) parts.push(Doc.line(f.error.message.split("\n", 1)[0] ?? "", { truncate: true }));
 	if (f.error?.diff !== void 0) parts.push(Doc.diffText(f.error.diff, { truncate: true }));
-	return Render.plain([Doc.list([Doc.section(void 0, parts)], { compact: true })], Render.contextOf({
-		audience: "agent",
-		width: Math.max(22, width)
-	})).split("\n");
+	return Render.ansi([Doc.list([Doc.section(void 0, parts)], { compact: true })], {
+		...Render.contextOf({
+			audience: "human",
+			width: Math.max(22, width)
+		}),
+		link: (target, label) => "file" in target ? link(target, label) : label
+	}).split("\n");
 };
 /**
 * The `Glob aggregates below threshold:` block — one
@@ -200,9 +207,10 @@ const TABLE_COL_FILE_MIN = 60;
 * (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.
+* calculation. Each path is written through `link` after the column is
+* padded, so a hyperlink wraps the path alone and never the padding.
 */
-const formatBelowTargetTable = (belowTarget, limit) => {
+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;
@@ -213,7 +221,7 @@ const formatBelowTargetTable = (belowTarget, limit) => {
 		buildTableHeader(fileColWidth),
 		buildTableSeparator(fileColWidth)
 	];
-	const rows = top.map((file) => buildTableRow(file, 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 [
@@ -228,8 +236,8 @@ const buildTableSeparator = (fileColWidth) => {
 const buildTableHeader = (fileColWidth) => {
 	return ` ${"File".padEnd(fileColWidth - 1)}| % Stmts | % Branch| % Funcs | % Lines | Uncovered Line #s`;
 };
-const buildTableRow = (file, fileColWidth) => {
-	return `${` ${file.file.padEnd(fileColWidth - 1)}`}|${pctCell(file.summary.statements)}|${pctCell(file.summary.branches)}|${pctCell(file.summary.functions)}|${pctCell(file.summary.lines)}|${` ${file.uncoveredLines}`}`;
+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)}`;
@@ -238,5 +246,25 @@ const pctCell = (n) => {
 	const right = pad - left;
 	return `${" ".repeat(left + 1)}${text}${" ".repeat(right)}`;
 };
+const SEVERITY_LABEL = {
+	info: "info",
+	warn: "warn",
+	blocker: "blocker"
+};
+/**
+* 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).
+*/
+const formatSuggestedActionLines = (actions) => {
+	if (actions.length === 0) return [];
+	const lines = ["Actions:"];
+	for (const action of actions) {
+		const tool = action.targetTool !== void 0 ? ` (tool: ${action.targetTool})` : "";
+		lines.push(`- ${SEVERITY_LABEL[action.severity]}: ${action.title}${tool}`, `  ${action.detail}`);
+	}
+	return lines;
+};
 
-export { formatBelowTargetTable, formatCoverageJudgmentLine, formatCoverageSummaryLines, formatDisplayDuration, formatFailure, formatGlobShortfallLines, formatPercent, formatProjectRow, formatProjectsTable, formatTestName, formatTotals, formatTrendLine, formatWorkspaceTotal, soleModulePath, soleTest };
+export { formatBelowTargetTable, formatCoverageJudgmentLine, formatCoverageSummaryLines, formatDisplayDuration, formatFailure, formatGlobShortfallLines, formatPercent, formatProjectRow, formatProjectsTable, formatSuggestedActionLines, formatTestName, formatTotals, formatTrendLine, formatWorkspaceTotal, soleModulePath, soleTest };