pi-bro 0.24.0 → 0.25.1

package/src/modal-kit.ts

@@ -71,6 +71,9 @@ export type FrameParts = {
 	notice?: string;
 	/** Omit when the body prints its own hints (config's SettingsList). */
 	hints?: string;
+	/** Lines rendered under the hints, e.g. a focusable action row. Wrapped by the caller; a line
+	 * wider than the frame is truncated, so keep a single label narrower than the modal. */
+	footerAfterHints?: string;
 	/** false: size to the content instead of padding the body to the full height. */
 	fill?: boolean;
 	/** Body rows the notice must leave free; it shrinks to one line before taking them. */
@@ -94,10 +97,12 @@ export function renderFrame(theme: Theme, width: number, height: number, parts:
 	const footer = parts.footer ?? [];
 	// Wrapped, not truncated: Esc comes last in every hint line and must not be cut off.
 	const hints = parts.hints ? wrapTextWithAnsi(theme.fg("dim", parts.hints), inner) : [];
+	// The action row is already wrapped by the caller, so keep its own line breaks.
+	const belowHints = parts.footerAfterHints ? parts.footerAfterHints.split("\n").map((line) => truncateToWidth(line, inner, "")) : [];
 	const notice = parts.notice ? wrapTextWithAnsi(parts.notice, inner).slice(0, 3) : [];
-	while (notice.length > 1 && height - 5 - footer.length - hints.length - notice.length < (parts.minBody ?? 0)) notice.pop();
+	while (notice.length > 1 && height - 5 - footer.length - hints.length - belowHints.length - notice.length < (parts.minBody ?? 0)) notice.pop();
 	// The notice sits above the footer so it is dropped before a composer when space runs out.
-	const below = [...notice, ...footer, ...hints];
+	const below = [...notice, ...footer, ...hints, ...belowHints];
 	const bodyRows = Math.max(0, height - 5 - below.length);
 	const body = bodyRows ? parts.body(bodyRows, inner).slice(0, bodyRows) : [];
 	if (parts.fill !== false) while (body.length < bodyRows) body.push("");
@@ -174,6 +179,34 @@ export class Scroller {
 	}
 }
 
+/**
+ * Render a focusable action row: the focused button carries a leading marker and accent colour, so a
+ * keypress always has exactly one visible target. Callers own focus, keys and activation; this only
+ * draws, which is why Guided Review can keep its own navigation and layout.
+ */
+export function buttonLabels(theme: Theme, labels: readonly string[], focused?: number): string[] {
+	return labels.map((label, index) => {
+		const text = `${index === focused ? "›" : ""}[${label}]`;
+		return index === focused ? theme.bold(theme.fg("accent", text)) : text;
+	});
+}
+
+/**
+ * Pack whole buttons onto lines with `visibleWidth`, never splitting one: a label broken mid-word
+ * makes the focused target ambiguous, which is the misclick this row exists to prevent.
+ */
+export function wrapButtons(buttons: readonly string[], width: number): string[] {
+	const lines: string[] = [];
+	let line = "";
+	for (const button of buttons) {
+		if (!line) { line = button; continue; }
+		if (visibleWidth(`${line} ${button}`) <= width) line += ` ${button}`;
+		else { lines.push(line); line = button; }
+	}
+	if (line) lines.push(line);
+	return lines;
+}
+
 /**
  * Forward a pointer event to a pi-tui child drawn at `area` inside the overlay, in the child's own
  * coordinates (without `area`, the event already is). Pi 0.84 components have no handleMouse, hence

package/src/prompt.ts

@@ -159,7 +159,7 @@ export function buildAdvisorPrompt(steering: AdvisorSteeringInput, snapshot: str
 	const questionSection = question?.trim()
 		? `## Executor's question\n\n${question.trim()}`
 		: "## Executor's question\n\nNone was given. Use your own judgment about what advice would help most, given the snapshot below.";
-	return `You are the Bro advisor: a separate process an executor coding agent voluntarily consulted mid-task, in the pi-bro Pi extension. You have real tool access in this workspace and are running with permissions auto-approved -- you can freely read files, search, and run read-oriented commands to verify claims. Investigate before advising: do not just restate what the snapshot below reports as true.
+	return `You are the Bro advisor: a separate process an executor coding agent voluntarily consulted mid-task, in the pi-bro Pi extension. You have real tool access in this workspace and are running with permissions auto-approved -- you can freely read files, search, and run read-oriented commands when needed. Use the snapshot below first; only read files or run commands when something you need is missing from the snapshot, may be stale after later edits, or a key claim is in doubt.
 
 Your job is strictly advisory. Return findings and recommendations in your reply; do not edit files or otherwise implement the change yourself -- the executor remains responsible for implementation.
 
@@ -167,11 +167,11 @@ ${steeringSection}
 
 ## Context snapshot from the executor's session
 
-This is background/evidence captured from the executor's own conversation. It is the executor's own account of what happened, not independently verified by you -- treat it as a starting point to check, not as ground truth:
+This is background/evidence captured from the executor's own conversation, including its tool calls and results. It is the executor's own account of what happened, not independently verified by you -- rely on the tool outputs already shown here before running new checks:
 
 ${snapshot}
 
 ${questionSection}
 
-Begin with a one-line answer or verdict. Then give concise findings, evidence, and recommended next actions grounded in what you verified yourself in the workspace. Omit investigation narration, waiting updates, and progress reports.`;
+Begin with a one-line answer or verdict. Then give concise findings, evidence, and recommended next actions grounded in the snapshot and any workspace checks you needed. Omit investigation narration, waiting updates, and progress reports.`;
 }