pi-crew 0.9.36 → 0.9.38

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-crew",
-  "version": "0.9.36",
+  "version": "0.9.38",
   "description": "Pi extension for coordinated AI teams, workflows, worktrees, and async task orchestration",
   "author": "baphuongna",
   "license": "MIT",

package/src/config/config.ts

@@ -694,7 +694,7 @@ function parseIsolationPolicy(value: unknown): CrewRuntimeConfig["isolationPolic
 
 function parseRuntimeConfig(value: unknown): CrewRuntimeConfig | undefined {
 	const obj = asRecord(value);
-	if (!obj) return undefined;
+	if (!obj) return { inheritContext: true } as CrewRuntimeConfig;
 	const runtime: CrewRuntimeConfig = {
 		mode: parseWithSchema(
 			Type.Union([Type.Literal("auto"), Type.Literal("scaffold"), Type.Literal("child-process"), Type.Literal("live-session")]),
@@ -704,7 +704,7 @@ function parseRuntimeConfig(value: unknown): CrewRuntimeConfig | undefined {
 		allowChildProcessFallback: parseWithSchema(Type.Boolean(), obj.allowChildProcessFallback),
 		maxTurns: parsePositiveInteger(obj.maxTurns, LIMIT_CEILINGS.runtimeMaxTurns),
 		graceTurns: parsePositiveInteger(obj.graceTurns, LIMIT_CEILINGS.runtimeGraceTurns),
-		inheritContext: parseWithSchema(Type.Boolean(), obj.inheritContext),
+		inheritContext: parseWithSchema(Type.Boolean(), obj.inheritContext) ?? true,
 		promptMode: parseWithSchema(Type.Union([Type.Literal("replace"), Type.Literal("append")]), obj.promptMode),
 		groupJoin: parseWithSchema(Type.Union([Type.Literal("off"), Type.Literal("group"), Type.Literal("smart")]), obj.groupJoin),
 		groupJoinAckTimeoutMs: parsePositiveInteger(obj.groupJoinAckTimeoutMs, 86_400_000),

package/src/extension/crew-vibes/figures.ts

@@ -1,5 +1,5 @@
 import type { SpeedConfig } from "./config.ts";
-import { isWebTerminal } from "./font-detect.ts";
+import { hasCrewFontFile, isWebTerminal } from "./font-detect.ts";
 
 /**
  * Crew figures replacing the original cat glyphs from pi-speeed / pi-chonk.
@@ -57,12 +57,19 @@ const PUA_CREW_FRAMES: readonly string[] = [
 /**
  * Return the indicator frames for the given style.
  *
- * - `"braille"` (default): standard Unicode braille spinner, works everywhere
- * - `"pua"`: runner-pose glyphs from crew-vibes.ttf, requires font install
+ * Default is `"braille"` — standard Unicode braille spinner (⠋⠙⠹⠸⠼⠴⠦⠧⠇⠏)
+ * that renders correctly on every terminal, every platform, with zero
+ * font dependency. PUA runner-glyphs require crew-vibes.ttf installed AND
+ * a terminal that does font fallback for Private Use Area codepoints —
+ * Windows Terminal does NOT do this reliably, showing garbage chars even
+ * with the font installed.
+ *
+ * To opt into the PUA runner look, pass `style: "pua"` explicitly.
  */
-export function crewFrames(style: "braille" | "pua" = "pua"): readonly string[] {
-	// Web terminals (gotty, wetty) cannot render PUA glyphs — fall back to braille
-	if (style === "pua" && isWebTerminal()) return BRAILLE_FRAMES;
+export function crewFrames(style: "braille" | "pua" = "braille"): readonly string[] {
+	// Even when explicitly requested, fall back to braille if the font is
+	// missing or we're on a web terminal (which can't render PUA at all).
+	if (style === "pua" && (isWebTerminal() || !hasCrewFontFile())) return BRAILLE_FRAMES;
 	return style === "pua" ? PUA_CREW_FRAMES : BRAILLE_FRAMES;
 }
 

package/src/extension/crew-vibes/index.ts

@@ -1,5 +1,24 @@
 import type { ExtensionAPI, ExtensionCommandContext, ExtensionContext } from "@earendil-works/pi-coding-agent";
 import { requestRender, setFooter } from "../../ui/pi-ui-compat.ts";
+import { logInternalError } from "../../utils/internal-error.ts";
+
+/**
+ * Wrap a pi UI call in try/catch. If the call throws (e.g. terminal doesn't
+ * support PUA glyphs, theme helper returns undefined, or pi's API signature
+ * changed), log it and silently fall back to pi's default behavior — which
+ * is simply "do nothing", letting pi show its own built-in indicator.
+ *
+ * Crew-vibes must NEVER break the user's session. A broken spinner is not
+ * worth a crashed pi.
+ */
+function safeUiCall(scope: string, fn: () => void): void {
+	try {
+		fn();
+	} catch (error) {
+		logInternalError(`crew-vibes.${scope}`, error, undefined, "warn");
+	}
+}
+
 import { type CrewVibesConfig, loadConfig, PROVIDER_STATUS_ID, saveConfig } from "./config.ts";
 import { intervalForSpeed } from "./figures.ts";
 import { type CrewVibesFooterSource, createCrewVibesFooter } from "./footer.ts";
@@ -72,44 +91,48 @@ export function registerCrewVibes(pi: ExtensionAPI): void {
 	function installFooter(ctx: ExtensionContext): void {
 		if (!ctx?.hasUI) return;
 		if (!metersActive()) {
-			setFooter(ctx, undefined);
+			safeUiCall("clear-footer", () => setFooter(ctx, undefined));
 			return;
 		}
-		setFooter(ctx, (tui, theme, footerData) => createCrewVibesFooter({ tui, theme, footerData, ctx, source: footerSource }));
-		requestRender(ctx);
+		safeUiCall("install-footer", () => {
+			setFooter(ctx, (tui, theme, footerData) => createCrewVibesFooter({ tui, theme, footerData, ctx, source: footerSource }));
+			requestRender(ctx);
+		});
 	}
 
 	/** Trigger a footer repaint; the footer recomputes capacity/quota on render. */
 	function refreshFooter(ctx: ExtensionContext): void {
-		if (ctx?.hasUI) requestRender(ctx);
+		if (ctx?.hasUI) safeUiCall("refresh-footer", () => requestRender(ctx));
 	}
 
 	function publishSpeedFooter(ctx: ExtensionContext, speed = footerAnimator.value()): void {
 		if (!config.enabled || !config.speed.enabled || !config.speed.footer) {
-			setSpeedStatus(ctx, config, undefined);
+			safeUiCall("clear-speed-status", () => setSpeedStatus(ctx, config, undefined));
 			return;
 		}
-		setSpeedStatus(ctx, config, renderSpeedFooter(themeOf(ctx), config.speed, speed));
+		safeUiCall("publish-speed-status", () => setSpeedStatus(ctx, config, renderSpeedFooter(themeOf(ctx), config.speed, speed)));
 	}
 
 	function applyIndicator(ctx: ExtensionContext, speed: number | null, force = false): void {
 		if (!ctx.hasUI || !ctx.ui.setWorkingIndicator) return;
 		if (!config.enabled || !config.speed.enabled || !config.speed.indicator) {
-			ctx.ui.setWorkingIndicator();
+			safeUiCall("reset-indicator", () => ctx.ui.setWorkingIndicator?.());
 			return;
 		}
 		const next = intervalForSpeed(config.speed, speed);
 		if (!force && Math.abs(next - currentIntervalMs) < 10) return;
-		ctx.ui.setWorkingIndicator({
-			frames: crewIndicatorFrames(themeOf(ctx)),
-			intervalMs: next,
-		});
+		safeUiCall("set-indicator", () =>
+			ctx.ui.setWorkingIndicator?.({
+				frames: crewIndicatorFrames(themeOf(ctx)),
+				intervalMs: next,
+			}),
+		);
 		currentIntervalMs = next;
 	}
 
 	function renderWorking(ctx: ExtensionContext, speed: number | null): void {
 		if (!config.enabled || !config.speed.enabled || !ctx.hasUI) return;
-		ctx.ui.setWorkingMessage(renderWorkingMessage(themeOf(ctx), config.speed, speed));
+		safeUiCall("set-working-message", () => ctx.ui.setWorkingMessage?.(renderWorkingMessage(themeOf(ctx), config.speed, speed)));
 	}
 
 	function stopLiveTimer(): void {
@@ -315,7 +338,7 @@ export function registerCrewVibes(pi: ExtensionAPI): void {
 		stopLiveTimer();
 		if (ctx && config.enabled && ctx.hasUI) {
 			applyIndicator(ctx, speedTracker.lastTokS);
-			ctx.ui.setWorkingMessage();
+			safeUiCall("clear-working-message", () => ctx.ui.setWorkingMessage?.());
 		}
 	});
 

package/src/extension/team-tool/context.ts

@@ -1,4 +1,5 @@
 import type { ExtensionContext } from "@earendil-works/pi-coding-agent";
+
 import type { PiTeamsConfig } from "../../config/config.ts";
 import type { MetricRegistry } from "../../observability/metric-registry.ts";
 import type { RunSnapshotCache } from "../../ui/run-snapshot-cache.ts";
@@ -47,6 +48,40 @@ function extractTextContent(content: unknown): string {
 		.join("\n");
 }
 
+/**
+ * Maximum character budget for inherited parent context (~3K tokens).
+ * When exceeded, oldest messages are dropped first (most-recent-first retention).
+ */
+export const MAX_PARENT_CONTEXT_CHARS = 12_000;
+
+/** Maximum chars for a single assistant message before truncation. */
+const MAX_ASSISTANT_MSG_CHARS = 500;
+
+/** Truncated length for oversized assistant messages. */
+const TRUNCATED_ASSISTANT_CHARS = 200;
+
+/** Messages starting with these patterns and exceeding this size are likely
+ *  file dumps / bash output — skip them to keep context relevant and compact. */
+const NOISY_THRESHOLD = 1_000;
+const NOISY_PREFIXES = ["```", "total ", "drwx", "-rw", "import ", "export "];
+
+/**
+ * Check if a message looks like noisy file/tool output (code dumps, ls output,
+ * import lists) that bloats context without helping the subagent.
+ */
+function isNoisyContent(text: string): boolean {
+	if (text.length < NOISY_THRESHOLD) return false;
+	return NOISY_PREFIXES.some((prefix) => text.startsWith(prefix));
+}
+
+/**
+ * Build a compact parent conversation context for subagent inheritance.
+ *
+ * Extracts recent user messages, assistant reasoning, and compaction summaries
+ * from the parent session. Applies a character budget (most-recent-first
+ * retention) and filters noisy content (file dumps, long bash output) to keep
+ * the inherited context relevant without bloating the subagent's token budget.
+ */
 export function buildParentContext(ctx: TeamContext): string | undefined {
 	const branch = ctx.sessionManager?.getBranch?.();
 	if (!Array.isArray(branch) || branch.length === 0) return undefined;
@@ -58,21 +93,43 @@ export function buildParentContext(ctx: TeamContext): string | undefined {
 			message?: unknown;
 			summary?: unknown;
 		};
-		if (record.type === "compaction" && typeof record.summary === "string") parts.push(`[Summary]: ${record.summary}`);
+		// Compaction summaries are always valuable — keep in full.
+		if (record.type === "compaction" && typeof record.summary === "string") {
+			parts.push(`[Summary]: ${record.summary}`);
+			continue;
+		}
 		const message =
 			record.message && typeof record.message === "object" && !Array.isArray(record.message)
 				? (record.message as { role?: unknown; content?: unknown })
 				: undefined;
 		if (!message || (message.role !== "user" && message.role !== "assistant")) continue;
-		const text = extractTextContent(message.content).trim();
-		if (text) parts.push(`[${message.role === "user" ? "User" : "Assistant"}]: ${text}`);
+		let text = extractTextContent(message.content).trim();
+		if (!text) continue;
+		// Filter: skip noisy content (file dumps, long bash output).
+		if (isNoisyContent(text)) continue;
+		// Truncate: long assistant messages get shortened to key points.
+		if (message.role === "assistant" && text.length > MAX_ASSISTANT_MSG_CHARS) {
+			text = `${text.slice(0, TRUNCATED_ASSISTANT_CHARS)}…`;
+		}
+		parts.push(`[${message.role === "user" ? "User" : "Assistant"}]: ${text}`);
 	}
 	if (!parts.length) return undefined;
+
+	// Budget: keep most-recent messages first, drop oldest when over budget.
+	let totalChars = 0;
+	const budgeted: string[] = [];
+	for (const part of [...parts].reverse()) {
+		if (totalChars + part.length > MAX_PARENT_CONTEXT_CHARS) break;
+		budgeted.unshift(part);
+		totalChars += part.length;
+	}
+	if (!budgeted.length) return undefined;
+
 	return [
 		`# Parent Conversation Context`,
 		"The following context was inherited from the parent Pi session. Treat it as reference-only.",
 		"",
-		parts.join("\n\n"),
+		budgeted.join("\n\n"),
 	].join("\n");
 }
 

package/src/extension/team-tool/handle-settings.ts

@@ -14,7 +14,7 @@ const EFFECTIVE_DEFAULTS: Record<string, unknown> = {
 	"runtime.mode": "auto",
 	"runtime.maxTurns": 10000,
 	"runtime.graceTurns": 5,
-	"runtime.inheritContext": false,
+	"runtime.inheritContext": true,
 	"runtime.promptMode": "replace",
 	"runtime.completionMutationGuard": "warn",
 	"runtime.isolationPolicy": undefined,

package/src/extension/team-tool/run.ts

@@ -832,6 +832,7 @@ export async function handleRun(params: TeamToolParamsValue, ctx: TeamContext):
 					runtime,
 					runtimeConfig: executedConfig.runtime,
 					parentContext: buildParentContext(ctx),
+
 					parentModel: ctx.model,
 					modelRegistry: ctx.modelRegistry,
 					modelOverride: params.model,

package/src/ui/settings-overlay.ts

@@ -327,7 +327,7 @@ const EFFECTIVE_DEFAULTS: Record<string, unknown> = {
 	"runtime.mode": "auto",
 	"runtime.maxTurns": 10000,
 	"runtime.graceTurns": 5,
-	"runtime.inheritContext": false,
+	"runtime.inheritContext": true,
 	"runtime.promptMode": "replace",
 	"runtime.completionMutationGuard": "warn",
 	"runtime.isolationPolicy": undefined,

package/src/ui/tool-renderers/index.ts

@@ -9,6 +9,7 @@
 import { type Container, Text, visibleWidth } from "@earendil-works/pi-tui";
 import type { CrewAgentRecord } from "../../runtime/crew-agent-runtime.ts";
 import { truncateToWidth } from "../../utils/visual.ts";
+import { spinnerFrame } from "../spinner.ts";
 import type { CrewTheme } from "../theme-adapter.ts";
 import { formatDuration, formatTokens, truncLine } from "../tool-render.ts";
 import { briefToolResult, isBrief } from "./brief-mode.ts";
@@ -373,9 +374,25 @@ function renderAgentResult(result: Record<string, unknown>, options: unknown, th
 	// isPartial = agent still running
 	const isPartial = (options as Record<string, unknown>)?.isPartial === true;
 	if (isPartial && !ctx.expanded) {
-		const spinner = theme.fg("warning", "◉");
-		const label = theme.fg("muted", "agent working...");
-		contentLines.push(padVisual(` ${spinner} ${label}`, innerW));
+		const spinner = theme.fg("accent", spinnerFrame(d.agentId as string));
+		const parts: string[] = [theme.fg("muted", "working")];
+
+		// Extract progress from onUpdate content (formatCompactToolProgress output)
+		const progressText = extractContentText(result?.content);
+		if (progressText) {
+			const elapsedMatch = progressText.match(/elapsed=(\d+)s/);
+			const tokenMatch = progressText.match(/tokens=(\d+)/);
+			const toolMatch = progressText.match(/tool:\s+(\S+)/);
+			const elapsedSec = elapsedMatch ? parseInt(elapsedMatch[1]!, 10) : 0;
+			const tokens = tokenMatch ? parseInt(tokenMatch[1]!, 10) : 0;
+			const tps = elapsedSec > 0 ? Math.round(tokens / elapsedSec) : 0;
+
+			if (elapsedSec > 0) parts.push(theme.fg("dim", formatDuration(elapsedSec * 1000)));
+			if (tps > 0) parts.push(theme.fg("dim", `${formatTokens(tps)} tok/s`));
+			if (toolMatch) parts.push(theme.fg("dim", `· ${toolMatch[1]}`));
+		}
+
+		contentLines.push(padVisual(` ${spinner} ${parts.join("  ")}`, innerW));
 		return buildFrameBottom(contentLines, w, theme, "borderAccent", "borderAccent");
 	}
 

package/src/ui/widget/widget-formatters.ts

@@ -18,6 +18,7 @@ import { computeLiveDurationMs } from "../live-duration.ts";
 // verbatim (no truncation/crash) — these are rare one-off states, not jitter.
 const TOOLS_METRIC_WIDTH = 8; // "127 tools"
 const TOKENS_METRIC_WIDTH = 10; // "1.2k tok", "12.3M tok"
+const TPS_METRIC_WIDTH = 9; // "411 tok/s"
 const CTX_METRIC_WIDTH = 7; // "100% ctx"
 const DURATION_METRIC_WIDTH = 6; // "120.0s"
 
@@ -139,10 +140,19 @@ export function agentStats(agent: CrewAgentRecord, liveHandle?: LiveAgentHandle)
 			/* ignore */
 		}
 		const ms = computeLiveDurationMs(act);
+		if (total > 0 && ms > 1000) {
+			const tps = Math.round(total / (ms / 1000));
+			if (tps > 0) parts.push(alignMetric(`${formatTokensCompact(tps)}/s`, TPS_METRIC_WIDTH));
+		}
 		parts.push(alignMetric(`${(ms / 1000).toFixed(1)}s`, DURATION_METRIC_WIDTH));
 	} else {
 		if (agent.toolUses) parts.push(alignMetric(`${agent.toolUses} tools`, TOOLS_METRIC_WIDTH));
 		if (agent.progress?.tokens) parts.push(alignMetric(formatTokensCompact(agent.progress.tokens), TOKENS_METRIC_WIDTH));
+		const ageMs = agent.startedAt ? Math.max(0, Date.now() - new Date(agent.startedAt).getTime()) : 0;
+		if (agent.progress?.tokens && ageMs > 1000) {
+			const tps = Math.round(agent.progress.tokens / (ageMs / 1000));
+			if (tps > 0) parts.push(alignMetric(`${formatTokensCompact(tps)}/s`, TPS_METRIC_WIDTH));
+		}
 		const age = elapsed(agent.completedAt ?? agent.startedAt);
 		if (age) parts.push(alignMetric(age, DURATION_METRIC_WIDTH));
 	}