pi-bro 0.19.2 → 0.19.3

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 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

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` };
 }
@@ -3322,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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-bro",
-  "version": "0.19.2",
+  "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",