pi-bro 0.19.1 → 0.19.3

package/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 All notable changes to pi-bro are documented here.
 
+## [0.19.3] - 2026-09-30
+
+### Fixed
+
+- Send large Agy explain, Show, and BTW prompts over stdin instead of exceeding argument-size limits. Older Agy versions report an upgrade hint rather than silently falling back (#73).
+- Retain newest turns and message tails when bounding Show and BTW context, preserving JSON message framing (#73).
+
+## [0.19.2] - 2026-09-30
+
+### Fixed
+
+- Preserve Show's diagram controls, captured source, and steering when reopening and rerunning a result. Recreate missing temporary diagrams without a model call and retain the previous diagram after a failed rerun (#71).
+
 ## [0.19.1] - 2026-09-30
 
 ### Fixed

package/backend.ts

@@ -211,8 +211,8 @@ export function parseBtwAgyLine(line: string): { delta?: string; result?: string
 	return { conversationId };
 }
 
-// explain/show and btw all print the prompt as a single --print argv value and read a stream-json
-// result off stdout; only sandbox flag, cwd, timeout, and (btw only) --conversation differ.
+// Explain/show and BTW use argv for small prompts and stdin NDJSON for large
+// prompts; sandbox, cwd, timeout, and continuation semantics stay unchanged.
 async function executeArgvPrint(
 	request: BackendRequest,
 	selection: AgySelection,
@@ -221,6 +221,9 @@ async function executeArgvPrint(
 	killEscalationMs: number,
 	deadlineMsOverride: number | undefined,
 ): Promise<BackendOutcome> {
+	// Keep legacy CLI compatibility for small prompts; large inputs use the advisor's
+	// stdin protocol rather than risking Linux's per-argument byte limit.
+	const stdinPrompt = Buffer.byteLength(request.prompt, "utf8") >= 120_000;
 	const isBtw = request.feature === "btw";
 	const full = isBtw && request.access === "workspace-full";
 	const deadlineMs = deadlineMsOverride ?? (isBtw ? (full ? 610_000 : 130_000) : 125_000);
@@ -251,12 +254,11 @@ async function executeArgvPrint(
 				"--print-timeout",
 				printTimeout,
 				...(request.continuation ? ["--conversation", request.continuation.id] : []),
-				"--print",
-				request.prompt,
+				...(stdinPrompt ? ["--input-format", "stream-json"] : ["--print", request.prompt]),
 			],
 			{
 				cwd: full ? request.cwd : runDirectory,
-				stdio: ["ignore", "pipe", "pipe"],
+				stdio: [stdinPrompt ? "pipe" : "ignore", "pipe", "pipe"],
 				windowsHide: true,
 				detached: process.platform !== "win32",
 			},
@@ -278,6 +280,11 @@ async function executeArgvPrint(
 			processError = error;
 		});
 
+		if (stdinPrompt) {
+			child.stdin?.on("error", () => { /* Report early exits through the close/error path. */ });
+			child.stdin?.end(`${JSON.stringify({ event: "user", message: { content: request.prompt } })}\n`);
+		}
+
 		const lines = createInterface({ input: child.stdout!, crlfDelay: Infinity });
 		try {
 			for await (const line of lines) {
@@ -327,7 +334,9 @@ async function executeArgvPrint(
 		if (exitSignal || code === null) {
 			return { status: "failure", message: unexpectedSignalMessage(exitSignal), partialText: partial || undefined };
 		}
-		if (code !== 0) return { status: "failure", message: agyFailureMessage(action, { code, killed: false, stderr }) };
+		if (code !== 0) return { status: "failure", message: stdinPrompt && advisorFlagErrorHint(stderr)
+			? withDoctor("Large prompts require Agy 1.1.15+ with stdin support. Run `agy update`. " + stderr.trim())
+			: agyFailureMessage(action, { code, killed: false, stderr }) };
 
 		const text = final.trim();
 		if (!text) return { status: "failure", message: withDoctor(stderr.trim() || emptyTextMessage) };

package/bro.ts

@@ -1475,7 +1475,8 @@ function serializeShowTurns(turns: readonly ShowTurn[]): string {
 	return turns.flatMap((turn) => turn.entries).join("\n\n");
 }
 
-export function captureShowTranscript(ctx: ExtensionCommandContext, turnsRequested: number): BroSource | undefined {
+export function captureShowTranscript(ctx: ExtensionCommandContext, turnsRequested: number, maxLength = MAX_TEXT_LENGTH): BroSource | undefined {
+	if (!Number.isSafeInteger(maxLength) || maxLength < 128) throw new Error("Transcript limit must be an integer of at least 128 characters.");
 	const turns: ShowTurn[] = [];
 	for (const entry of ctx.sessionManager.getBranch()) {
 		if (entry.type !== "message") continue;
@@ -1503,7 +1504,7 @@ export function captureShowTranscript(ctx: ExtensionCommandContext, turnsRequest
 	if (seen === 0) return undefined;
 
 	let text = serializeShowTurns(turns.slice(start));
-	while (text.length > MAX_TEXT_LENGTH && start < turns.length - 1) {
+	while (text.length > maxLength && start < turns.length - 1) {
 		let next = turns.length;
 		for (let index = start + 1; index < turns.length; index += 1) {
 			if (turns[index]!.startsTurn) {
@@ -1515,7 +1516,22 @@ export function captureShowTranscript(ctx: ExtensionCommandContext, turnsRequest
 		start = next;
 		text = serializeShowTurns(turns.slice(start));
 	}
-	if (text.length > MAX_TEXT_LENGTH) text = `${text.slice(0, MAX_TEXT_LENGTH)}\n[… transcript truncated …]`;
+	if (text.length > maxLength) {
+		// Prefer the newest complete messages even within an oversized single turn.
+		const entries = turns.slice(start).flatMap(turn => turn.entries);
+		const notice = "[… earlier messages truncated …]\n";
+		while (entries.length > 1 && entries.join("\n\n").length + notice.length > maxLength) entries.shift();
+		text = entries.join("\n\n");
+		if (text.length + notice.length > maxLength) {
+			const newline = text.indexOf("\n");
+			const header = text.slice(0, newline);
+			const content = JSON.parse(text.slice(newline + 1)) as string;
+			const marker = "[… earlier text truncated …] ";
+			let tail = content.slice(-(maxLength - header.length - marker.length - 8));
+			while (`${header}\n${JSON.stringify(marker + tail)}`.length > maxLength) tail = tail.slice(Math.max(1, Math.floor(tail.length / 10)));
+			text = `${header}\n${JSON.stringify(marker + tail)}`;
+		} else text = notice + text;
+	}
 	const kept = turns.slice(start).filter((turn) => turn.startsTurn).length;
 	return { text: text.trim(), label: `last ${Math.max(1, kept)} turn${kept > 1 ? "s" : ""} · conversation only` };
 }
@@ -2942,6 +2958,12 @@ async function showBroModal(ctx: ExtensionCommandContext, options: BroModalOptio
 				options.retryLabel ?? "simplify again",
 			);
 
+			const present = (result: ModalResult, notice = "") => {
+				const display = result.htmlPath ? stripShowHtmlFence(result.text) : result.text;
+				modal.setResult(display, options.retryable ?? Boolean(options.run), notice, result.source?.label, result.text, result.model);
+				modal.setHtmlPath(result.htmlPath ?? "");
+			};
+
 			execute = (source?: BroSource) => {
 				if (!options.run || controller || closed) return;
 				const previous = current;
@@ -2958,16 +2980,14 @@ async function showBroModal(ctx: ExtensionCommandContext, options: BroModalOptio
 						if (closed || nextController.signal.aborted) return;
 						current = result;
 						options.onResult?.(result);
-						const display = result.htmlPath ? stripShowHtmlFence(result.text) : result.text;
-						modal.setResult(display, options.retryable ?? true, "", result.source?.label, result.text, result.model);
-						if (result.htmlPath) modal.setHtmlPath(result.htmlPath);
+						present(result);
 					})
 					.catch((error) => {
 						if (closed || nextController.signal.aborted) return;
 						const message = error instanceof Error ? error.message : String(error);
 						if (previous) {
 							current = previous;
-							modal.setResult(previous.text, options.retryable ?? true, `Retry failed: ${message}`, previous.source?.label, previous.text, previous.model);
+							present(previous, `Retry failed: ${message}`);
 						} else {
 							modal.setError(message);
 						}
@@ -2980,7 +3000,7 @@ async function showBroModal(ctx: ExtensionCommandContext, options: BroModalOptio
 			if (options.text !== undefined) {
 				modal.setStatic(options.kind ?? "help", options.text, options.copyable ?? false);
 			} else if (current) {
-				modal.setResult(current.text, options.retryable ?? Boolean(options.run), "", current.source?.label, current.text, current.model);
+				present(current);
 			} else {
 				execute();
 			}
@@ -3318,10 +3338,7 @@ async function openBtwModal(
 				}
 				if (thread.turns.length === 0) {
 					thread.conversationId = undefined;
-					let context = captureShowTranscript(ctx, BTW_CONTEXT_TURNS)?.text;
-					if (context && context.length > BTW_CONTEXT_MAX) {
-						context = `${context.slice(0, BTW_CONTEXT_MAX)}\n[… context truncated …]`;
-					}
+					const context = captureShowTranscript(ctx, BTW_CONTEXT_TURNS, BTW_CONTEXT_MAX)?.text;
 					thread.context = context;
 				}
 				// Resume natively when possible; otherwise seed the fresh native session with the main-session
@@ -3479,10 +3496,14 @@ async function openBtwModal(
 }
 
 export default async function bro(pi: ExtensionAPI) {
-	let lastResult: BroResult | undefined;
+	let lastResult: ModalResult | undefined;
+	let lastShowSteering: string | undefined;
 	let btwThread: BtwThread | undefined;
-	const remember = (result: ModalResult) => {
-		if (result.source) lastResult = { source: result.source, text: result.text, model: result.model };
+	const remember = (result: ModalResult, showSteering?: string) => {
+		if (result.source) {
+			lastResult = result;
+			lastShowSteering = showSteering;
+		}
 	};
 
 	pi.on("session_start", async (_event, _ctx) => {
@@ -3659,7 +3680,7 @@ export default async function bro(pi: ExtensionAPI) {
 						loadingText: "Drawing what happened…",
 						retryLabel: "show again",
 						run: runShow,
-						onResult: remember,
+						onResult: (result) => remember(result, steering),
 					});
 				} catch (error) {
 					ctx.ui.notify(withDoctor(error), "error");
@@ -3860,10 +3881,29 @@ export default async function bro(pi: ExtensionAPI) {
 					return;
 				}
 
+				const steering = lastShowSteering;
+				if (steering !== undefined) {
+					const html = extractShowHtml(lastResult.text);
+					// Temp files may have been cleaned by the OS or another Bro session.
+					if (html && ctx.mode === "tui") {
+						try { lastResult.htmlPath = await writeShowHtml(html); }
+						catch (error) {
+							lastResult.htmlPath = undefined;
+							ctx.ui.notify(`Diagram file unavailable: ${errorMessage(error)}`, "warning");
+						}
+					}
+				}
 				await showBroModal(ctx, {
 					result: lastResult,
-					run,
-					onResult: remember,
+					loadingText: steering !== undefined ? "Drawing what happened…" : undefined,
+					retryLabel: steering !== undefined ? "show again" : undefined,
+					run: steering === undefined ? run : async (signal, source, onProgress) => {
+						if (!source) throw new Error("No captured Show source.");
+						const result = await runShowExplanation(source.text, steering, signal, await readSettings(), onProgress);
+						const html = extractShowHtml(result.text);
+						return { source, ...result, ...(html ? { htmlPath: await writeShowHtml(html) } : {}) };
+					},
+					onResult: (result) => remember(result, steering),
 				});
 				return;
 			}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-bro",
-  "version": "0.19.1",
+  "version": "0.19.3",
   "description": "An Earendil Pi extension that explains pasted text, assistant responses, local documents, public webpages, and recent session turns (as shapes) in a context-isolated window, opens a sandboxed side conversation with /bro btw, and provides a second-opinion advisor tool for executor agents.",
   "type": "module",
   "license": "MIT",