@henryqw/pi-pr 2.0.4 → 2.1.0

package/README.md

@@ -23,7 +23,9 @@ Run `/pr` without arguments in a GitHub checkout. It reads the current branch pu
 | Surface | Type | Purpose |
 | --- | --- | --- |
 | Footer | ui | Show a linked `PR #number` and one plain-language status. |
-| Widget hint | ui | Show at most one hint for the next `/pr` step. |
+| Widget | ui | Show one actionable icon-prefixed `Run /pr to …` hint. |
+
+The footer already shows the pull request and status. Actionable widgets omit duplicate identity and status. Each uses one semantic status icon, a space, and a plain `Run /pr to …` route. `✗` marks errors, `!` warnings, `✓` success, and `●` accent or neutral routes. In TUI, only the icon uses a theme color. RPC and non-TUI output use the same plain text without ANSI.
 
 ## Flow
 
@@ -65,7 +67,7 @@ Ordinary conversation comments do not trigger a route or block a merge. Changes
 
 The footer and widget load at session start. They refresh after local commits, PR creation, pushes, and each dispatched workflow settles. They also poll every 30 seconds. Polling updates presentation only and may be stale.
 
-The create hint stays hidden until the local branch has a commit beyond its creation point. Any displayed hint clears as soon as `/pr` starts. A dispatched workflow keeps it hidden until the agent settles. A direct merge, no-action route, or failed command refreshes the hint when the handler finishes.
+The create widget stays hidden until the local branch has a commit beyond its creation point. Any displayed widget clears as soon as `/pr` starts. A dispatched workflow keeps it hidden until the agent settles. A direct merge, no-action route, or failed command refreshes the widget when the handler finishes.
 
 Presentation uses route priority, so draft appears before running CI. `/pr` reads fresh state before routing or merging. The command is authoritative for actions.
 

package/extensions/pr-ui.ts

@@ -1,4 +1,4 @@
-import { getCapabilities, hyperlink } from "@earendil-works/pi-tui";
+import { getCapabilities, hyperlink, truncateToWidth } from "@earendil-works/pi-tui";
 import {
 	deriveNextStep,
 	type NextStep,
@@ -93,6 +93,10 @@ export function formatPrFooter(display: PrDisplay, theme: PrTheme): string | und
 	return `${link} · ${theme.fg(display.footer.color, display.footer.text)}`;
 }
 
-export function formatPrWidget(display: PrDisplay): string | undefined {
-	return display.widget;
+export function formatPrWidget(display: PrDisplay, theme?: PrTheme, width?: number): string[] | undefined {
+	if (display.widget === undefined) return undefined;
+	const color = display.footer?.color ?? "accent";
+	const icon = color === "error" ? "✗" : color === "warning" ? "!" : color === "success" ? "✓" : "●";
+	const line = `${theme ? theme.fg(color, icon) : icon} ${display.widget}`;
+	return [width === undefined ? line : truncateToWidth(line, Math.max(1, width))];
 }

package/extensions/pr.ts

@@ -12,6 +12,7 @@ import {
 	formatPrFooter,
 	formatPrWidget,
 	projectPrDisplay,
+	type PrDisplay,
 } from "./pr-ui.ts";
 
 const POLL_INTERVAL_MS = 30_000;
@@ -39,10 +40,25 @@ export default function pullRequestExtension(
 	let active: AbortController | undefined;
 	let queued = false;
 	let refreshFailureReported = false;
-	let displayedWidget: string | undefined;
+	let displayedWidget: PrDisplay | undefined;
 	let commandGeneration = 0;
 	const activeInvocations = new Map<number, "routing" | "workflow">();
 
+	const setWidget = (ctx: ExtensionContext, display: PrDisplay | undefined): void => {
+		if (display?.widget === undefined) {
+			ctx.ui.setWidget(UI_KEY, undefined);
+			return;
+		}
+		if (ctx.mode === "tui") {
+			ctx.ui.setWidget(UI_KEY, (_tui, theme) => ({
+				invalidate() {},
+				render: (width) => formatPrWidget(display, theme, width)!,
+			}));
+			return;
+		}
+		ctx.ui.setWidget(UI_KEY, formatPrWidget(display));
+	};
+
 	const render = (
 		ctx: ExtensionContext,
 		pullRequest: Awaited<ReturnType<typeof loadCurrentPullRequest>>,
@@ -53,10 +69,10 @@ export default function pullRequestExtension(
 		if (pullRequest !== null && footer === undefined) {
 			throw new Error("Current pull request display is missing a footer");
 		}
-		displayedWidget = formatPrWidget(display);
+		displayedWidget = display.widget === undefined ? undefined : display;
 		const widget = activeInvocations.size > 0 ? undefined : displayedWidget;
 		ctx.ui.setStatus(UI_KEY, footer);
-		ctx.ui.setWidget(UI_KEY, widget === undefined ? undefined : [widget]);
+		setWidget(ctx, widget);
 	};
 
 	const stop = (): void => {
@@ -172,7 +188,7 @@ export default function pullRequestExtension(
 			const generation = sessionGeneration;
 			const invocation = ++commandGeneration;
 			activeInvocations.set(invocation, "routing");
-			if (displayedWidget !== undefined) ctx.ui.setWidget(UI_KEY, undefined);
+			if (displayedWidget !== undefined) setWidget(ctx, undefined);
 			let nextStep: Awaited<ReturnType<typeof commandHandler>>;
 			try {
 				nextStep = await commandHandler(args, ctx);
@@ -181,7 +197,7 @@ export default function pullRequestExtension(
 					cancelRefresh();
 					activeInvocations.delete(invocation);
 					if (!activeInvocations.size) {
-						ctx.ui.setWidget(UI_KEY, displayedWidget === undefined ? undefined : [displayedWidget]);
+						setWidget(ctx, displayedWidget);
 					}
 					refreshInBackground();
 				}
@@ -192,7 +208,7 @@ export default function pullRequestExtension(
 			if (nextStep !== "none" && nextStep !== "merge") {
 				activeInvocations.set(invocation, "workflow");
 				if (nextStep === "create") ctx.ui.setStatus(UI_KEY, undefined);
-				ctx.ui.setWidget(UI_KEY, undefined);
+				setWidget(ctx, undefined);
 			} else {
 				activeInvocations.delete(invocation);
 				refreshInBackground();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@henryqw/pi-pr",
-  "version": "2.0.4",
+  "version": "2.1.0",
   "description": "Run /pr to automatically choose the current pull request's next safe lifecycle step: create, update, address feedback, fix CI, or merge when ready.",
   "keywords": [
     "pi-package",