@ferris1225/pi-subagents 0.14.0 → 0.15.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ferris1225/pi-subagents",
-  "version": "0.14.0",
+  "version": "0.15.0",
   "description": "Focused sub-agent delegation for pi: explore / worker / reviewer agents in isolated context, with proactive dispatch injection and per-agent model selection.",
   "type": "module",
   "license": "MIT",

package/src/index.ts

@@ -134,7 +134,7 @@ function formatCompletionBlock(result: SingleResult, maxResultLines: number): st
 	const fallbackNote = result.modelFallbackFrom
 		? ` (model fell back from ${result.modelFallbackFrom} to ${result.model ?? "main-window model"})`
 		: "";
-	const lines = [`### [${result.agent}] ${status}${usage ? ` (${usage})` : ""}${fallbackNote}`, "", `Task: ${formatTaskSummary(result.task)}`, "", text];
+	const lines = [`### [${result.agent}] ${status}${usage ? ` (${usage})` : ""}${fallbackNote}`, "", `Task: ${formatTaskSummary(result.task, 80, false)}`, "", text];
 	if (truncated) {
 		// The full text lives on disk so the main agent can read it on demand.
 		lines.push("", `(output truncated to ${maxResultLines} lines; full result: ${writeResultArtifact(output, result.agent)})`);
@@ -587,14 +587,14 @@ export default function (pi: ExtensionAPI): void {
 			if (args.tasks && args.tasks.length > 0) {
 				let text = `${theme.fg("toolTitle", theme.bold("subagent "))}${theme.fg("accent", `parallel (${args.tasks.length})`)}`;
 				for (const t of args.tasks.slice(0, 4)) {
-					const preview = t.task.length > 48 ? `${t.task.slice(0, 48)}…` : t.task;
+					const preview = formatTaskSummary(t.task, 48);
 					text += `\n  ${theme.fg("accent", t.agent)} ${theme.fg("dim", preview)}`;
 				}
 				if (args.tasks.length > 4) text += `\n  ${theme.fg("dim", `… +${args.tasks.length - 4} more`)}`;
 				return new Text(text, 0, 0);
 			}
 			const task: string = args.task ?? "";
-			const preview = task.length > 60 ? `${task.slice(0, 60)}…` : task;
+			const preview = formatTaskSummary(task, 60);
 			return new Text(
 				`${theme.fg("toolTitle", theme.bold("subagent "))}${theme.fg("accent", args.agent ?? "?")} ${theme.fg("dim", preview)}`,
 				0,
@@ -663,9 +663,9 @@ export default function (pi: ExtensionAPI): void {
 							// parent reviewer; summarize() already carries the relationLabel.
 							const head = r.groupId ? theme.fg("dim", "  ↳ ") : " ";
 							const note = r.annotation ? theme.fg("dim", ` · ${r.annotation}`) : "";
-							lines.push(truncateToWidth(`${head}${icon} ${monitor.summarize(r)} · ${label}${note}`, width, ""));
+							lines.push(truncateToWidth(`${head}${icon} #${r.id} ${monitor.summarize(r)} · ${label}${note}`, width, ""));
 							if (r.status === "queued" || r.status === "running") {
-								lines.push(truncateToWidth(theme.fg("dim", `     task: ${formatTaskSummary(r.task)}`), width, ""));
+								lines.push(truncateToWidth(theme.fg("dim", `     task: ${formatTaskSummary(r.task, Math.max(20, width - 11))}`), width, ""));
 							}
 							// Activity sits one indent level below the agent name.
 							if (r.activity) lines.push(truncateToWidth(theme.fg("dim", `     ${r.activity}`), width, ""));

package/src/monitor.ts

@@ -56,23 +56,145 @@ export interface RunChainMeta {
 
 const TASK_SUMMARY_MAX = 80;
 const TASK_SUMMARY_ELLIPSIS = "…";
+/** Columns reserved at the END of a truncated summary so the distinguishing
+ * keywords (paths, symbols, ...) survive; the head gets the rest. */
+const TASK_SUMMARY_TAIL_MAX = 28;
+/** Tail share of a non-default maxWidth (narrow widgets keep a usable tail). */
+const TASK_SUMMARY_TAIL_SHARE = 0.35;
+const TASK_SUMMARY_TAIL_MIN = 8;
+const TASK_SUMMARY_KEY_SEP = " · ";
+/** kebab/snake words that are task boilerplate, never distinguishing signal. */
+const KEY_FRAGMENT_STOPWORDS = new Set([
+	"self-contained",
+	"read-only",
+	"write-only",
+	"auto-fix",
+	"re-review",
+	"one-line",
+	"pre-commit",
+]);
 const graphemeSegmenter = new Intl.Segmenter(undefined, { granularity: "grapheme" });
 
-/** One-line task preview, capped by terminal display columns (including the ellipsis). */
-export function formatTaskSummary(task: string): string {
-	const oneLine = stripVTControlCharacters(task).replace(/\s+/g, " ").trim();
-	if (visibleWidth(oneLine) <= TASK_SUMMARY_MAX) return oneLine;
+interface KeyFragment {
+	text: string;
+	index: number;
+}
 
-	const prefixMax = TASK_SUMMARY_MAX - visibleWidth(TASK_SUMMARY_ELLIPSIS);
-	let prefix = "";
-	let prefixWidth = 0;
-	for (const { segment } of graphemeSegmenter.segment(oneLine)) {
+/**
+ * Pull the most distinguishing fragments out of a task: file paths, quoted
+ * phrases, camelCase/PascalCase symbols and kebab/snake compounds. Sorted by
+ * first occurrence and deduped (a path covers its own sub-fragments). These
+ * are what make parallel tasks of the same agent look different.
+ */
+export function extractKeyFragments(text: string): string[] {
+	const fragments: KeyFragment[] = [];
+	const add = (re: RegExp, group = 0): void => {
+		for (const m of text.matchAll(re)) {
+			const g = m[group];
+			if (g === undefined) continue;
+			fragments.push({ text: g, index: m.index ?? 0 });
+		}
+	};
+	// Quoted phrases first (highest signal).
+	add(/["'`]([^"'`]{4,60})["'`]/g, 1);
+	// Paths with a known extension (src/index.ts, build/out.js.map).
+	add(/(?<![A-Za-z0-9_.-])[A-Za-z0-9_.-]+\.[A-Za-z0-9]{1,5}(?![A-Za-z0-9_.-])/g);
+	// Paths with a slash but no extension (src/components, .github/workflows).
+	add(/(?<![A-Za-z0-9_.-])[A-Za-z0-9_.-]+(?:[\\/][A-Za-z0-9_.-]+)+(?![\\/])/g);
+	// camelCase / PascalCase identifiers (function or type names).
+	add(/\b[a-z][a-zA-Z0-9]*[A-Z][a-zA-Z0-9]*\b/g);
+	// snake_case / kebab-case compound words.
+	add(/\b[a-z][a-z0-9]+[-_][a-z0-9][a-z0-9_-]*\b/g);
+
+	fragments.sort((a, b) => a.index - b.index);
+	const seen = new Set<string>();
+	const out: string[] = [];
+	for (const f of fragments) {
+		const t = f.text.trim();
+		if (t.length < 4 || KEY_FRAGMENT_STOPWORDS.has(t)) continue;
+		if (seen.has(t)) continue;
+		// A longer fragment (the full path) covers its own sub-fragments.
+		if (out.some((o) => o.includes(t) || t.includes(o))) continue;
+		seen.add(t);
+		out.push(t);
+	}
+	return out;
+}
+
+function takeGraphemes(segments: string[], maxWidth: number): string {
+	let width = 0;
+	const out: string[] = [];
+	for (const segment of segments) {
 		const segmentWidth = visibleWidth(segment);
-		if (prefixWidth + segmentWidth > prefixMax) break;
-		prefix += segment;
-		prefixWidth += segmentWidth;
+		if (width + segmentWidth > maxWidth) break;
+		out.push(segment);
+		width += segmentWidth;
+	}
+	return out.join("");
+}
+
+function tailGraphemes(segments: string[], maxWidth: number): string {
+	let width = 0;
+	const tail: string[] = [];
+	for (let i = segments.length - 1; i >= 0; i--) {
+		const segmentWidth = visibleWidth(segments[i]);
+		if (width + segmentWidth > maxWidth) break;
+		tail.unshift(segments[i]);
+		width += segmentWidth;
+	}
+	return tail.join("");
+}
+
+/**
+ * One-line task preview, capped by `maxWidth` display columns (default 80).
+ * `keysOnly` (default): extracted key fragments (paths, quoted phrases,
+ * symbols) are shown bare — the agent name is already displayed next to the
+ * task line, so templated prose ("explore: trace how ...") adds nothing.
+ * `keysOnly: false` keeps the prose as `head…tail` (used for completion
+ * messages, where the Task line is the reader's only context).
+ * Grapheme-safe — CJK, ZWJ emoji and combining sequences are never split.
+ */
+export function formatTaskSummary(task: string, maxWidth: number = TASK_SUMMARY_MAX, keysOnly = true): string {
+	const oneLine = stripVTControlCharacters(task).replace(/\s+/g, " ").trim();
+	if (maxWidth <= 0 || visibleWidth(oneLine) <= maxWidth) return oneLine;
+
+	const segments = [...graphemeSegmenter.segment(oneLine)].map((s) => s.segment);
+	const ellipsisWidth = visibleWidth(TASK_SUMMARY_ELLIPSIS);
+
+	if (keysOnly) {
+		const fragments = extractKeyFragments(oneLine);
+		if (fragments.length > 0) {
+			const keyMax = maxWidth - 1;
+			let keys = "";
+			for (const fragment of fragments) {
+				const piece = keys ? `${TASK_SUMMARY_KEY_SEP}${fragment}` : fragment;
+				const total = keys + piece;
+				if (visibleWidth(total) > keyMax) {
+					// Budget exhausted: keep what fits, unless nothing fits yet.
+					if (!keys) {
+						// A single over-long fragment keeps its tail
+						// (extension/symbol) and is prefixed with the ellipsis.
+						const fragmentSegments = [...graphemeSegmenter.segment(piece)].map((s) => s.segment);
+						keys = `${TASK_SUMMARY_ELLIPSIS}${tailGraphemes(fragmentSegments, keyMax - ellipsisWidth)}`;
+					}
+					break;
+				}
+				keys = total;
+			}
+			return keys;
+		}
+	}
+
+	// No distinctive fragments (or prose mode): fall back to head…tail.
+	const tailMax = Math.max(
+		TASK_SUMMARY_TAIL_MIN,
+		Math.min(TASK_SUMMARY_TAIL_MAX, Math.round(maxWidth * TASK_SUMMARY_TAIL_SHARE)),
+	);
+	const headMax = maxWidth - ellipsisWidth - tailMax;
+	if (headMax <= 0) {
+		return `${TASK_SUMMARY_ELLIPSIS}${tailGraphemes(segments, maxWidth - ellipsisWidth)}`;
 	}
-	return `${prefix}${TASK_SUMMARY_ELLIPSIS}`;
+	return `${takeGraphemes(segments, headMax)}${TASK_SUMMARY_ELLIPSIS}${tailGraphemes(segments, tailMax)}`;
 }
 
 function formatTokens(count: number): string {