@henryqw/pi-pr 3.1.9 → 3.1.10

package/README.md

@@ -45,10 +45,12 @@ Add optional instructions to guide a creation, branch-update, CI-fix, or feedbac
 | --- | --- | --- |
 | `/pr [instructions]` | command | Run the current pull request's next safe route. |
 | Footer | ui | Show a linked `PR #number` and one plain-language status. |
-| Widget | ui | Show one actionable icon-prefixed `Run /pr to …` hint. |
+| Widget | ui | Show one action hint or transient routing status. |
 
 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.
 
+The widget switches to `⠋ Checking pull request…` as soon as `/pr` starts discovery. The braille spinner animates in TUI mode. RPC receives one plain static line. The footer stays unchanged. The routing widget clears after route selection and before any prompt, notification, mutation, or workflow dispatch.
+
 ## Flow
 
 Each footer entry is one linked `PR #number` plus one plain-language status: `N unresolved`, `draft`, `open`, `approved`, `CI running`, `CI failed`, `changes requested`, `base update required`, `merge conflict`, `merge-ready`, `merged`, or `closed`. Colors support the text; they do not carry meaning alone.
@@ -115,7 +117,7 @@ The footer and widget load at session start. A directory outside a Git worktree
 
 They refresh after local commits, PR creation, pushes, and each dispatched workflow settles. During creation, intermediate refreshes wait until the workflow settles. They also refresh after any successful delegated task settles. Active Git worktrees poll every 30 seconds. Polling updates presentation only and may be stale.
 
-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.
+The create widget stays hidden until the local branch has a commit beyond its creation point. `/pr` replaces any hint with routing feedback while it selects a route. The feedback clears before route interaction. A dispatched workflow keeps the widget hidden until the agent settles. Direct and no-action routes refresh it after completion. A failed command restores the prior hint and schedules a refresh.
 
 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-command.ts

@@ -27,7 +27,11 @@ const WORKFLOWS: Record<WorkflowNextStep, string> = {
 };
 
 type PrCommandPi = Pick<ExtensionAPI, "exec" | "getCommands" | "sendUserMessage">;
-export type PrCommandHandler = (args: string, ctx: ExtensionCommandContext) => Promise<NextStep>;
+export type PrCommandHandler = (
+	args: string,
+	ctx: ExtensionCommandContext,
+	onRouteResolved?: (nextStep: NextStep) => void,
+) => Promise<NextStep>;
 
 type PrCommandDependencies = {
 	loadCurrentPullRequest?: typeof loadCurrentPullRequest;
@@ -175,10 +179,11 @@ export function createPrCommandHandler(
 ): PrCommandHandler {
 	const load = dependencies.loadCurrentPullRequest ?? loadCurrentPullRequest;
 	const link = dependencies.linkInferredPullRequest ?? linkInferredPullRequest;
-	return async (args, ctx) => {
+	return async (args, ctx, onRouteResolved) => {
 		const instructions = args.trim();
 		const discovery = await load(pi, ctx);
 		const nextStep = deriveNextStep(discovery);
+		onRouteResolved?.(nextStep);
 		if (instructions && !(nextStep in WORKFLOWS)) {
 			throw new Error("The current /pr route does not accept instructions");
 		}

package/extensions/pr.ts

@@ -3,6 +3,7 @@ import {
 	type ExtensionAPI,
 	type ExtensionContext,
 } from "@earendil-works/pi-coding-agent";
+import { truncateToWidth, type Component, type TUI } from "@earendil-works/pi-tui";
 import { createHerdrClient } from "@henryqw/pi-herdr";
 import { createPrCommandHandler } from "./pr-command.ts";
 import {
@@ -24,6 +25,9 @@ import {
 } from "./pr-ui.ts";
 
 const POLL_INTERVAL_MS = 30_000;
+const ROUTING_SPINNER_INTERVAL_MS = 80;
+const ROUTING_SPINNER_FRAMES = ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"];
+const ROUTING_WIDGET_TEXT = "Checking pull request…";
 const HERDR_TIMEOUT_MS = 10_000;
 const UI_KEY = "pi-pr";
 const OBSERVATION_ENTRY = "pi-pr-observation";
@@ -130,9 +134,23 @@ export default function pullRequestExtension(
 	let mergeCompleted = false;
 	let displayedWidget: PrDisplay | undefined;
 	let commandGeneration = 0;
-	const activeInvocations = new Map<number, "routing" | "create-workflow" | "workflow">();
+	const activeInvocations = new Map<number, "routing" | "resolved" | "create-workflow" | "workflow">();
+	let widgetKind: "presentation" | "routing" = "presentation";
+	let routingSpinnerFrame = 0;
+	let routingSpinnerTimer: ReturnType<typeof setInterval> | undefined;
+	let routingSpinnerTui: TUI | undefined;
+	let routingSpinnerComponent: (Component & { dispose(): void }) | undefined;
+
+	const stopRoutingSpinner = (): void => {
+		if (routingSpinnerTimer !== undefined) clearInterval(routingSpinnerTimer);
+		routingSpinnerTimer = undefined;
+		routingSpinnerTui = undefined;
+		routingSpinnerComponent = undefined;
+	};
 
 	const setWidget = (ctx: ExtensionContext, display: PrDisplay | undefined): void => {
+		stopRoutingSpinner();
+		widgetKind = "presentation";
 		if (display?.widget === undefined) {
 			ctx.ui.setWidget(UI_KEY, undefined);
 			return;
@@ -147,6 +165,57 @@ export default function pullRequestExtension(
 		ctx.ui.setWidget(UI_KEY, formatPrWidget(display));
 	};
 
+	const setRoutingWidget = (ctx: ExtensionContext): void => {
+		if (widgetKind === "routing") return;
+		stopRoutingSpinner();
+		widgetKind = "routing";
+		routingSpinnerFrame = 0;
+		if (ctx.mode !== "tui") {
+			ctx.ui.setWidget(UI_KEY, [`${ROUTING_SPINNER_FRAMES[0]} ${ROUTING_WIDGET_TEXT}`]);
+			return;
+		}
+
+		ctx.ui.setWidget(UI_KEY, (tui, theme) => {
+			let cachedWidth: number | undefined;
+			let cachedLines: string[] | undefined;
+			const component: Component & { dispose(): void } = {
+				invalidate() {
+					cachedWidth = undefined;
+					cachedLines = undefined;
+				},
+				render(width) {
+					if (width <= 0) return [];
+					if (cachedLines !== undefined && cachedWidth === width) return cachedLines;
+					const line = `${theme.fg("accent", ROUTING_SPINNER_FRAMES[routingSpinnerFrame]!)} ${ROUTING_WIDGET_TEXT}`;
+					cachedWidth = width;
+					cachedLines = [truncateToWidth(line, Math.max(1, width))];
+					return cachedLines;
+				},
+				dispose() {
+					if (routingSpinnerComponent !== component) return;
+					stopRoutingSpinner();
+					widgetKind = "presentation";
+				},
+			};
+			routingSpinnerTui = tui;
+			routingSpinnerComponent = component;
+			return component;
+		});
+		routingSpinnerTimer = setInterval(() => {
+			routingSpinnerFrame = (routingSpinnerFrame + 1) % ROUTING_SPINNER_FRAMES.length;
+			routingSpinnerComponent?.invalidate();
+			routingSpinnerTui?.requestRender();
+		}, ROUTING_SPINNER_INTERVAL_MS);
+	};
+
+	const reconcileWidget = (ctx: ExtensionContext): void => {
+		if ([...activeInvocations.values()].includes("routing")) {
+			setRoutingWidget(ctx);
+			return;
+		}
+		setWidget(ctx, activeInvocations.size > 0 ? undefined : displayedWidget);
+	};
+
 	const render = (
 		ctx: ExtensionContext,
 		discovery: Awaited<ReturnType<typeof loadCurrentPullRequest>>,
@@ -159,7 +228,7 @@ export default function pullRequestExtension(
 			lastDiscovery = "inactive";
 			displayedWidget = undefined;
 			ctx.ui.setStatus(UI_KEY, undefined);
-			setWidget(ctx, undefined);
+			reconcileWidget(ctx);
 			return;
 		}
 		const display = projectPrDisplay(discovery, localCommit);
@@ -168,9 +237,8 @@ export default function pullRequestExtension(
 			throw new Error("Pull request display is missing a footer");
 		}
 		displayedWidget = display.widget === undefined ? undefined : display;
-		const widget = activeInvocations.size > 0 ? undefined : displayedWidget;
 		ctx.ui.setStatus(UI_KEY, footer);
-		setWidget(ctx, widget);
+		reconcileWidget(ctx);
 		if (discovery.kind === "blocked") {
 			const key = discoveryIssueKey(discovery.issue);
 			if (lastBlockedIssueKey !== key) {
@@ -203,6 +271,8 @@ export default function pullRequestExtension(
 		displayedWidget = undefined;
 		commandGeneration = 0;
 		activeInvocations.clear();
+		stopRoutingSpinner();
+		widgetKind = "presentation";
 		if (timer !== undefined) clearInterval(timer);
 		timer = undefined;
 		active?.abort();
@@ -255,8 +325,9 @@ export default function pullRequestExtension(
 				if (!controller.signal.aborted && sessionGeneration === generation) {
 					if (!displayEstablished) {
 						const unavailable = unavailablePrDisplay();
+						displayedWidget = undefined;
 						ctx.ui.setStatus(UI_KEY, formatPrFooter(unavailable, ctx.ui.theme));
-						setWidget(ctx, undefined);
+						reconcileWidget(ctx);
 						displayEstablished = true;
 					}
 					reportRefreshFailure();
@@ -352,17 +423,21 @@ export default function pullRequestExtension(
 			const generation = sessionGeneration;
 			const invocation = ++commandGeneration;
 			activeInvocations.set(invocation, "routing");
-			if (displayedWidget !== undefined) setWidget(ctx, undefined);
+			reconcileWidget(ctx);
+			const routeResolved = (): void => {
+				if (sessionGeneration !== generation || activeInvocations.get(invocation) !== "routing") return;
+				activeInvocations.set(invocation, "resolved");
+				reconcileWidget(ctx);
+			};
 			let nextStep: Awaited<ReturnType<typeof commandHandler>>;
 			try {
-				nextStep = await commandHandler(args, ctx);
+				nextStep = await commandHandler(args, ctx, routeResolved);
+				routeResolved();
 			} catch (error) {
 				if (sessionGeneration === generation) {
 					cancelRefresh();
 					activeInvocations.delete(invocation);
-					if (!activeInvocations.size) {
-						setWidget(ctx, displayedWidget);
-					}
+					reconcileWidget(ctx);
 					refreshInBackground();
 				}
 				throw error;
@@ -372,7 +447,7 @@ export default function pullRequestExtension(
 			if (WORKFLOW_ROUTES.has(nextStep)) {
 				activeInvocations.set(invocation, nextStep === "create" ? "create-workflow" : "workflow");
 				if (nextStep === "create") ctx.ui.setStatus(UI_KEY, undefined);
-				setWidget(ctx, undefined);
+				reconcileWidget(ctx);
 			} else {
 				if (nextStep === "merge") mergeCompleted = true;
 				activeInvocations.delete(invocation);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@henryqw/pi-pr",
-  "version": "3.1.9",
+  "version": "3.1.10",
   "description": "Run /pr to safely discover or link the current pull request, then create, update, address feedback, fix CI, or merge when ready.",
   "keywords": [
     "pi-package",