@ia-ccun/code-agent-cli 0.0.53 → 0.0.54-beta.3

package/config/agent/extensions/review.ts

@@ -27,9 +27,18 @@
  * Note: PR review requires a clean working tree (no uncommitted changes to tracked files).
  */
 
-import type { ExtensionAPI, ExtensionContext, ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
+import type {
+	ExtensionAPI,
+	ExtensionContext,
+	ExtensionCommandContext,
+} from "@earendil-works/pi-coding-agent";
 import { DynamicBorder, BorderedLoader } from "@earendil-works/pi-coding-agent";
-import { Container, type SelectItem, SelectList, Text } from "@earendil-works/pi-tui";
+import {
+	Container,
+	type SelectItem,
+	SelectList,
+	Text,
+} from "@earendil-works/pi-tui";
 import path from "node:path";
 import { promises as fs } from "node:fs";
 
@@ -54,7 +63,11 @@ function setReviewWidget(ctx: ExtensionContext, active: boolean) {
 	}
 
 	ctx.ui.setWidget("review", (_tui, theme) => {
-		const text = new Text(theme.fg("warning", "Review session active, return with /end-review"), 0, 0);
+		const text = new Text(
+			theme.fg("warning", "Review session active, return with /end-review"),
+			0,
+			0,
+		);
 		return {
 			render(width: number) {
 				return text.render(width);
@@ -107,18 +120,19 @@ const BASE_BRANCH_PROMPT_WITH_MERGE_BASE =
 	"Review the code changes against the base branch '{baseBranch}'. The merge base commit for this comparison is {mergeBaseSha}. Run `git diff {mergeBaseSha}` to inspect the changes relative to {baseBranch}. Provide prioritized, actionable findings.";
 
 const BASE_BRANCH_PROMPT_FALLBACK =
-	"Review the code changes against the base branch '{branch}'. Start by finding the merge diff between the current branch and {branch}'s upstream e.g. (`git merge-base HEAD \"$(git rev-parse --abbrev-ref \"{branch}@{upstream}\")\"`), then run `git diff` against that SHA to see what changes we would merge into the {branch} branch. Provide prioritized, actionable findings.";
+	'Review the code changes against the base branch \'{branch}\'. Start by finding the merge diff between the current branch and {branch}\'s upstream e.g. (`git merge-base HEAD "$(git rev-parse --abbrev-ref "{branch}@{upstream}")"`), then run `git diff` against that SHA to see what changes we would merge into the {branch} branch. Provide prioritized, actionable findings.';
 
 const COMMIT_PROMPT_WITH_TITLE =
 	'Review the code changes introduced by commit {sha} ("{title}"). Provide prioritized, actionable findings.';
 
-const COMMIT_PROMPT = "Review the code changes introduced by commit {sha}. Provide prioritized, actionable findings.";
+const COMMIT_PROMPT =
+	"Review the code changes introduced by commit {sha}. Provide prioritized, actionable findings.";
 
 const PULL_REQUEST_PROMPT =
-	'Review pull request #{prNumber} ("{title}") against the base branch \'{baseBranch}\'. The merge base commit for this comparison is {mergeBaseSha}. Run `git diff {mergeBaseSha}` to inspect the changes that would be merged. Provide prioritized, actionable findings.';
+	"Review pull request #{prNumber} (\"{title}\") against the base branch '{baseBranch}'. The merge base commit for this comparison is {mergeBaseSha}. Run `git diff {mergeBaseSha}` to inspect the changes that would be merged. Provide prioritized, actionable findings.";
 
 const PULL_REQUEST_PROMPT_FALLBACK =
-	'Review pull request #{prNumber} ("{title}") against the base branch \'{baseBranch}\'. Start by finding the merge base between the current branch and {baseBranch} (e.g., `git merge-base HEAD {baseBranch}`), then run `git diff` against that SHA to see the changes that would be merged. Provide prioritized, actionable findings.';
+	"Review pull request #{prNumber} (\"{title}\") against the base branch '{baseBranch}'. Start by finding the merge base between the current branch and {baseBranch} (e.g., `git merge-base HEAD {baseBranch}`), then run `git diff` against that SHA to see the changes that would be merged. Provide prioritized, actionable findings.";
 
 const FOLDER_REVIEW_PROMPT =
 	"Review the code in the following paths: {paths}. This is a snapshot review (not a diff). Read the files directly in these paths and provide prioritized, actionable findings.";
@@ -192,7 +206,9 @@ Provide your findings in a clear, structured format:
 
 Output all findings the author would fix if they knew about them. If there are no qualifying findings, explicitly state the code looks good. Don't stop at the first finding - list every qualifying issue.`;
 
-async function loadProjectReviewGuidelines(cwd: string): Promise<string | null> {
+async function loadProjectReviewGuidelines(
+	cwd: string,
+): Promise<string | null> {
 	let currentDir = path.resolve(cwd);
 
 	while (true) {
@@ -238,14 +254,22 @@ async function getMergeBase(
 		]);
 
 		if (upstreamCode === 0 && upstream.trim()) {
-			const { stdout: mergeBase, code } = await pi.exec("git", ["merge-base", "HEAD", upstream.trim()]);
+			const { stdout: mergeBase, code } = await pi.exec("git", [
+				"merge-base",
+				"HEAD",
+				upstream.trim(),
+			]);
 			if (code === 0 && mergeBase.trim()) {
 				return mergeBase.trim();
 			}
 		}
 
 		// Fall back to using the branch directly
-		const { stdout: mergeBase, code } = await pi.exec("git", ["merge-base", "HEAD", branch]);
+		const { stdout: mergeBase, code } = await pi.exec("git", [
+			"merge-base",
+			"HEAD",
+			branch,
+		]);
 		if (code === 0 && mergeBase.trim()) {
 			return mergeBase.trim();
 		}
@@ -260,7 +284,10 @@ async function getMergeBase(
  * Get list of local branches
  */
 async function getLocalBranches(pi: ExtensionAPI): Promise<string[]> {
-	const { stdout, code } = await pi.exec("git", ["branch", "--format=%(refname:short)"]);
+	const { stdout, code } = await pi.exec("git", [
+		"branch",
+		"--format=%(refname:short)",
+	]);
 	if (code !== 0) return [];
 	return stdout
 		.trim()
@@ -271,8 +298,16 @@ async function getLocalBranches(pi: ExtensionAPI): Promise<string[]> {
 /**
  * Get list of recent commits
  */
-async function getRecentCommits(pi: ExtensionAPI, limit: number = 10): Promise<Array<{ sha: string; title: string }>> {
-	const { stdout, code } = await pi.exec("git", ["log", `--oneline`, `-n`, `${limit}`]);
+async function getRecentCommits(
+	pi: ExtensionAPI,
+	limit: number = 10,
+): Promise<Array<{ sha: string; title: string }>> {
+	const { stdout, code } = await pi.exec("git", [
+		"log",
+		`--oneline`,
+		`-n`,
+		`${limit}`,
+	]);
 	if (code !== 0) return [];
 
 	return stdout
@@ -303,7 +338,10 @@ async function hasPendingChanges(pi: ExtensionAPI): Promise<boolean> {
 	if (code !== 0) return false;
 
 	// Filter out untracked files (lines starting with ??)
-	const lines = stdout.trim().split("\n").filter((line) => line.trim());
+	const lines = stdout
+		.trim()
+		.split("\n")
+		.filter((line) => line.trim());
 	const trackedChanges = lines.filter((line) => !line.startsWith("??"));
 	return trackedChanges.length > 0;
 }
@@ -334,10 +372,16 @@ function parsePrReference(ref: string): number | null {
 /**
  * Get PR information from GitHub CLI
  */
-async function getPrInfo(pi: ExtensionAPI, prNumber: number): Promise<{ baseBranch: string; title: string; headBranch: string } | null> {
+async function getPrInfo(
+	pi: ExtensionAPI,
+	prNumber: number,
+): Promise<{ baseBranch: string; title: string; headBranch: string } | null> {
 	const { stdout, code } = await pi.exec("gh", [
-		"pr", "view", String(prNumber),
-		"--json", "baseRefName,title,headRefName",
+		"pr",
+		"view",
+		String(prNumber),
+		"--json",
+		"baseRefName,title,headRefName",
 	]);
 
 	if (code !== 0) return null;
@@ -357,8 +401,15 @@ async function getPrInfo(pi: ExtensionAPI, prNumber: number): Promise<{ baseBran
 /**
  * Checkout a PR using GitHub CLI
  */
-async function checkoutPr(pi: ExtensionAPI, prNumber: number): Promise<{ success: boolean; error?: string }> {
-	const { stdout, stderr, code } = await pi.exec("gh", ["pr", "checkout", String(prNumber)]);
+async function checkoutPr(
+	pi: ExtensionAPI,
+	prNumber: number,
+): Promise<{ success: boolean; error?: string }> {
+	const { stdout, stderr, code } = await pi.exec("gh", [
+		"pr",
+		"checkout",
+		String(prNumber),
+	]);
 
 	if (code !== 0) {
 		return { success: false, error: stderr || stdout || "Failed to checkout PR" };
@@ -383,7 +434,11 @@ async function getCurrentBranch(pi: ExtensionAPI): Promise<string | null> {
  */
 async function getDefaultBranch(pi: ExtensionAPI): Promise<string> {
 	// Try to get from remote HEAD
-	const { stdout, code } = await pi.exec("git", ["symbolic-ref", "refs/remotes/origin/HEAD", "--short"]);
+	const { stdout, code } = await pi.exec("git", [
+		"symbolic-ref",
+		"refs/remotes/origin/HEAD",
+		"--short",
+	]);
 	if (code === 0 && stdout.trim()) {
 		return stdout.trim().replace("origin/", "");
 	}
@@ -399,7 +454,10 @@ async function getDefaultBranch(pi: ExtensionAPI): Promise<string> {
 /**
  * Build the review prompt based on target
  */
-async function buildReviewPrompt(pi: ExtensionAPI, target: ReviewTarget): Promise<string> {
+async function buildReviewPrompt(
+	pi: ExtensionAPI,
+	target: ReviewTarget,
+): Promise<string> {
 	switch (target.type) {
 		case "uncommitted":
 			return UNCOMMITTED_PROMPT;
@@ -407,17 +465,20 @@ async function buildReviewPrompt(pi: ExtensionAPI, target: ReviewTarget): Promis
 		case "baseBranch": {
 			const mergeBase = await getMergeBase(pi, target.branch);
 			if (mergeBase) {
-				return BASE_BRANCH_PROMPT_WITH_MERGE_BASE.replace(/{baseBranch}/g, target.branch).replace(
-					/{mergeBaseSha}/g,
-					mergeBase,
-				);
+				return BASE_BRANCH_PROMPT_WITH_MERGE_BASE.replace(
+					/{baseBranch}/g,
+					target.branch,
+				).replace(/{mergeBaseSha}/g, mergeBase);
 			}
 			return BASE_BRANCH_PROMPT_FALLBACK.replace(/{branch}/g, target.branch);
 		}
 
 		case "commit":
 			if (target.title) {
-				return COMMIT_PROMPT_WITH_TITLE.replace("{sha}", target.sha).replace("{title}", target.title);
+				return COMMIT_PROMPT_WITH_TITLE.replace("{sha}", target.sha).replace(
+					"{title}",
+					target.title,
+				);
 			}
 			return COMMIT_PROMPT.replace("{sha}", target.sha);
 
@@ -427,14 +488,15 @@ async function buildReviewPrompt(pi: ExtensionAPI, target: ReviewTarget): Promis
 		case "pullRequest": {
 			const mergeBase = await getMergeBase(pi, target.baseBranch);
 			if (mergeBase) {
-				return PULL_REQUEST_PROMPT
-					.replace(/{prNumber}/g, String(target.prNumber))
+				return PULL_REQUEST_PROMPT.replace(/{prNumber}/g, String(target.prNumber))
 					.replace(/{title}/g, target.title)
 					.replace(/{baseBranch}/g, target.baseBranch)
 					.replace(/{mergeBaseSha}/g, mergeBase);
 			}
-			return PULL_REQUEST_PROMPT_FALLBACK
-				.replace(/{prNumber}/g, String(target.prNumber))
+			return PULL_REQUEST_PROMPT_FALLBACK.replace(
+				/{prNumber}/g,
+				String(target.prNumber),
+			)
 				.replace(/{title}/g, target.title)
 				.replace(/{baseBranch}/g, target.baseBranch);
 		}
@@ -455,19 +517,26 @@ function getUserFacingHint(target: ReviewTarget): string {
 			return `changes against '${target.branch}'`;
 		case "commit": {
 			const shortSha = target.sha.slice(0, 7);
-			return target.title ? `commit ${shortSha}: ${target.title}` : `commit ${shortSha}`;
+			return target.title
+				? `commit ${shortSha}: ${target.title}`
+				: `commit ${shortSha}`;
 		}
 		case "custom":
-			return target.instructions.length > 40 ? target.instructions.slice(0, 37) + "..." : target.instructions;
+			return target.instructions.length > 40
+				? target.instructions.slice(0, 37) + "..."
+				: target.instructions;
 
 		case "pullRequest": {
-			const shortTitle = target.title.length > 30 ? target.title.slice(0, 27) + "..." : target.title;
+			const shortTitle =
+				target.title.length > 30 ? target.title.slice(0, 27) + "..." : target.title;
 			return `PR #${target.prNumber}: ${shortTitle}`;
 		}
 
 		case "folder": {
 			const joined = target.paths.join(", ");
-			return joined.length > 40 ? `folders: ${joined.slice(0, 37)}...` : `folders: ${joined}`;
+			return joined.length > 40
+				? `folders: ${joined.slice(0, 37)}...`
+				: `folders: ${joined}`;
 		}
 	}
 }
@@ -475,10 +544,22 @@ function getUserFacingHint(target: ReviewTarget): string {
 // Review preset options for the selector (keep this order stable)
 const REVIEW_PRESETS = [
 	{ value: "uncommitted", label: "Review uncommitted changes", description: "" },
-	{ value: "baseBranch", label: "Review against a base branch", description: "(local)" },
+	{
+		value: "baseBranch",
+		label: "Review against a base branch",
+		description: "(local)",
+	},
 	{ value: "commit", label: "Review a commit", description: "" },
-	{ value: "pullRequest", label: "Review a pull request", description: "(GitHub PR)" },
-	{ value: "folder", label: "Review a folder (or more)", description: "(snapshot, not diff)" },
+	{
+		value: "pullRequest",
+		label: "Review a pull request",
+		description: "(GitHub PR)",
+	},
+	{
+		value: "folder",
+		label: "Review a folder (or more)",
+		description: "(snapshot, not diff)",
+	},
 	{ value: "custom", label: "Custom review instructions", description: "" },
 ] as const;
 
@@ -487,10 +568,10 @@ export default function reviewExtension(pi: ExtensionAPI) {
 		applyReviewState(ctx);
 	});
 
-	pi.on("session_switch", (_event, ctx) => {
-		applyReviewState(ctx);
+	// pi 1.1.0: session_switch 改名为 session_before_switch
+	pi.on("session_before_switch", (_event, _ctx) => {
+		applyReviewState(_ctx);
 	});
-
 	pi.on("session_tree", (_event, ctx) => {
 		applyReviewState(ctx);
 	});
@@ -498,7 +579,9 @@ export default function reviewExtension(pi: ExtensionAPI) {
 	/**
 	 * Determine the smart default review type based on git state
 	 */
-	async function getSmartDefault(): Promise<"uncommitted" | "baseBranch" | "commit"> {
+	async function getSmartDefault(): Promise<
+		"uncommitted" | "baseBranch" | "commit"
+	> {
 		// Priority 1: If there are uncommitted changes, default to reviewing them
 		if (await hasUncommittedChanges(pi)) {
 			return "uncommitted";
@@ -518,7 +601,9 @@ export default function reviewExtension(pi: ExtensionAPI) {
 	/**
 	 * Show the review preset selector
 	 */
-	async function showReviewSelector(ctx: ExtensionContext): Promise<ReviewTarget | null> {
+	async function showReviewSelector(
+		ctx: ExtensionContext,
+	): Promise<ReviewTarget | null> {
 		// Determine smart default (but keep the list order stable)
 		const smartDefault = await getSmartDefault();
 		const items: SelectItem[] = REVIEW_PRESETS.map((preset) => ({
@@ -526,47 +611,55 @@ export default function reviewExtension(pi: ExtensionAPI) {
 			label: preset.label,
 			description: preset.description,
 		}));
-		const smartDefaultIndex = items.findIndex((item) => item.value === smartDefault);
+		const smartDefaultIndex = items.findIndex(
+			(item) => item.value === smartDefault,
+		);
 
 		while (true) {
-			const result = await ctx.ui.custom<string | null>((tui, theme, _kb, done) => {
-				const container = new Container();
-				container.addChild(new DynamicBorder((str) => theme.fg("accent", str)));
-				container.addChild(new Text(theme.fg("accent", theme.bold("Select a review preset"))));
-
-				const selectList = new SelectList(items, Math.min(items.length, 10), {
-					selectedPrefix: (text) => theme.fg("accent", text),
-					selectedText: (text) => theme.fg("accent", text),
-					description: (text) => theme.fg("muted", text),
-					scrollInfo: (text) => theme.fg("dim", text),
-					noMatch: (text) => theme.fg("warning", text),
-				});
-
-				// Preselect the smart default without reordering the list
-				if (smartDefaultIndex >= 0) {
-					selectList.setSelectedIndex(smartDefaultIndex);
-				}
-
-				selectList.onSelect = (item) => done(item.value);
-				selectList.onCancel = () => done(null);
-
-				container.addChild(selectList);
-				container.addChild(new Text(theme.fg("dim", "Press enter to confirm or esc to go back")));
-				container.addChild(new DynamicBorder((str) => theme.fg("accent", str)));
+			const result = await ctx.ui.custom<string | null>(
+				(tui, theme, _kb, done) => {
+					const container = new Container();
+					container.addChild(new DynamicBorder((str) => theme.fg("accent", str)));
+					container.addChild(
+						new Text(theme.fg("accent", theme.bold("Select a review preset"))),
+					);
+
+					const selectList = new SelectList(items, Math.min(items.length, 10), {
+						selectedPrefix: (text) => theme.fg("accent", text),
+						selectedText: (text) => theme.fg("accent", text),
+						description: (text) => theme.fg("muted", text),
+						scrollInfo: (text) => theme.fg("dim", text),
+						noMatch: (text) => theme.fg("warning", text),
+					});
+
+					// Preselect the smart default without reordering the list
+					if (smartDefaultIndex >= 0) {
+						selectList.setSelectedIndex(smartDefaultIndex);
+					}
 
-				return {
-					render(width: number) {
-						return container.render(width);
-					},
-					invalidate() {
-						container.invalidate();
-					},
-					handleInput(data: string) {
-						selectList.handleInput(data);
-						tui.requestRender();
-					},
-				};
-			});
+					selectList.onSelect = (item) => done(item.value);
+					selectList.onCancel = () => done(null);
+
+					container.addChild(selectList);
+					container.addChild(
+						new Text(theme.fg("dim", "Press enter to confirm or esc to go back")),
+					);
+					container.addChild(new DynamicBorder((str) => theme.fg("accent", str)));
+
+					return {
+						render(width: number) {
+							return container.render(width);
+						},
+						invalidate() {
+							container.invalidate();
+						},
+						handleInput(data: string) {
+							selectList.handleInput(data);
+							tui.requestRender();
+						},
+					};
+				},
+			);
 
 			if (!result) return null;
 
@@ -614,17 +707,23 @@ export default function reviewExtension(pi: ExtensionAPI) {
 	/**
 	 * Show branch selector for base branch review
 	 */
-	async function showBranchSelector(ctx: ExtensionContext): Promise<ReviewTarget | null> {
+	async function showBranchSelector(
+		ctx: ExtensionContext,
+	): Promise<ReviewTarget | null> {
 		const branches = await getLocalBranches(pi);
 		const currentBranch = await getCurrentBranch(pi);
 		const defaultBranch = await getDefaultBranch(pi);
 
 		// Never offer the current branch as a base branch (reviewing against itself is meaningless).
-		const candidateBranches = currentBranch ? branches.filter((b) => b !== currentBranch) : branches;
+		const candidateBranches = currentBranch
+			? branches.filter((b) => b !== currentBranch)
+			: branches;
 
 		if (candidateBranches.length === 0) {
 			ctx.ui.notify(
-				currentBranch ? `No other branches found (current branch: ${currentBranch})` : "No branches found",
+				currentBranch
+					? `No other branches found (current branch: ${currentBranch})`
+					: "No branches found",
 				"error",
 			);
 			return null;
@@ -646,7 +745,9 @@ export default function reviewExtension(pi: ExtensionAPI) {
 		const result = await ctx.ui.custom<string | null>((tui, theme, _kb, done) => {
 			const container = new Container();
 			container.addChild(new DynamicBorder((str) => theme.fg("accent", str)));
-			container.addChild(new Text(theme.fg("accent", theme.bold("Select base branch"))));
+			container.addChild(
+				new Text(theme.fg("accent", theme.bold("Select base branch"))),
+			);
 
 			const selectList = new SelectList(items, Math.min(items.length, 10), {
 				selectedPrefix: (text) => theme.fg("accent", text),
@@ -656,14 +757,15 @@ export default function reviewExtension(pi: ExtensionAPI) {
 				noMatch: (text) => theme.fg("warning", text),
 			});
 
-			// Enable search
-			selectList.searchable = true;
-
 			selectList.onSelect = (item) => done(item.value);
 			selectList.onCancel = () => done(null);
 
 			container.addChild(selectList);
-			container.addChild(new Text(theme.fg("dim", "Type to filter • enter to select • esc to cancel")));
+			container.addChild(
+				new Text(
+					theme.fg("dim", "Type to filter • enter to select • esc to cancel"),
+				),
+			);
 			container.addChild(new DynamicBorder((str) => theme.fg("accent", str)));
 
 			return {
@@ -687,7 +789,9 @@ export default function reviewExtension(pi: ExtensionAPI) {
 	/**
 	 * Show commit selector
 	 */
-	async function showCommitSelector(ctx: ExtensionContext): Promise<ReviewTarget | null> {
+	async function showCommitSelector(
+		ctx: ExtensionContext,
+	): Promise<ReviewTarget | null> {
 		const commits = await getRecentCommits(pi, 20);
 
 		if (commits.length === 0) {
@@ -701,49 +805,54 @@ export default function reviewExtension(pi: ExtensionAPI) {
 			description: "",
 		}));
 
-		const result = await ctx.ui.custom<{ sha: string; title: string } | null>((tui, theme, _kb, done) => {
-			const container = new Container();
-			container.addChild(new DynamicBorder((str) => theme.fg("accent", str)));
-			container.addChild(new Text(theme.fg("accent", theme.bold("Select commit to review"))));
-
-			const selectList = new SelectList(items, Math.min(items.length, 10), {
-				selectedPrefix: (text) => theme.fg("accent", text),
-				selectedText: (text) => theme.fg("accent", text),
-				description: (text) => theme.fg("muted", text),
-				scrollInfo: (text) => theme.fg("dim", text),
-				noMatch: (text) => theme.fg("warning", text),
-			});
+		const result = await ctx.ui.custom<{ sha: string; title: string } | null>(
+			(tui, theme, _kb, done) => {
+				const container = new Container();
+				container.addChild(new DynamicBorder((str) => theme.fg("accent", str)));
+				container.addChild(
+					new Text(theme.fg("accent", theme.bold("Select commit to review"))),
+				);
 
-			// Enable search
-			selectList.searchable = true;
+				const selectList = new SelectList(items, Math.min(items.length, 10), {
+					selectedPrefix: (text) => theme.fg("accent", text),
+					selectedText: (text) => theme.fg("accent", text),
+					description: (text) => theme.fg("muted", text),
+					scrollInfo: (text) => theme.fg("dim", text),
+					noMatch: (text) => theme.fg("warning", text),
+				});
 
-			selectList.onSelect = (item) => {
-				const commit = commits.find((c) => c.sha === item.value);
-				if (commit) {
-					done(commit);
-				} else {
-					done(null);
-				}
-			};
-			selectList.onCancel = () => done(null);
+				selectList.onSelect = (item) => {
+					const commit = commits.find((c) => c.sha === item.value);
+					if (commit) {
+						done(commit);
+					} else {
+						done(null);
+					}
+				};
+				selectList.onCancel = () => done(null);
 
-			container.addChild(selectList);
-			container.addChild(new Text(theme.fg("dim", "Type to filter • enter to select • esc to cancel")));
-			container.addChild(new DynamicBorder((str) => theme.fg("accent", str)));
+				container.addChild(selectList);
+				container.addChild(
+					new Text(
+						theme.fg("dim", "Type to filter • enter to select • esc to cancel"),
+					),
+				);
+				container.addChild(new DynamicBorder((str) => theme.fg("accent", str)));
 
-			return {
-				render(width: number) {
-					return container.render(width);
-				},
-				invalidate() {
-					container.invalidate();
-				},
-				handleInput(data: string) {
-					selectList.handleInput(data);
-					tui.requestRender();
-				},
-			};
-		});
+				return {
+					render(width: number) {
+						return container.render(width);
+					},
+					invalidate() {
+						container.invalidate();
+					},
+					handleInput(data: string) {
+						selectList.handleInput(data);
+						tui.requestRender();
+					},
+				};
+			},
+		);
 
 		if (!result) return null;
 		return { type: "commit", sha: result.sha, title: result.title };
@@ -752,7 +861,9 @@ export default function reviewExtension(pi: ExtensionAPI) {
 	/**
 	 * Show custom instructions input
 	 */
-	async function showCustomInput(ctx: ExtensionContext): Promise<ReviewTarget | null> {
+	async function showCustomInput(
+		ctx: ExtensionContext,
+	): Promise<ReviewTarget | null> {
 		const result = await ctx.ui.editor(
 			"Enter review instructions:",
 			"Review the code for security vulnerabilities and potential bugs...",
@@ -772,7 +883,9 @@ export default function reviewExtension(pi: ExtensionAPI) {
 	/**
 	 * Show folder input
 	 */
-	async function showFolderInput(ctx: ExtensionContext): Promise<ReviewTarget | null> {
+	async function showFolderInput(
+		ctx: ExtensionContext,
+	): Promise<ReviewTarget | null> {
 		const result = await ctx.ui.editor(
 			"Enter folders/files to review (space-separated or one per line):",
 			".",
@@ -788,10 +901,15 @@ export default function reviewExtension(pi: ExtensionAPI) {
 	/**
 	 * Show PR input and handle checkout
 	 */
-	async function showPrInput(ctx: ExtensionContext): Promise<ReviewTarget | null> {
+	async function showPrInput(
+		ctx: ExtensionContext,
+	): Promise<ReviewTarget | null> {
 		// First check for pending changes that would prevent branch switching
 		if (await hasPendingChanges(pi)) {
-			ctx.ui.notify("Cannot checkout PR: you have uncommitted changes. Please commit or stash them first.", "error");
+			ctx.ui.notify(
+				"Cannot checkout PR: you have uncommitted changes. Please commit or stash them first.",
+				"error",
+			);
 			return null;
 		}
 
@@ -805,7 +923,10 @@ export default function reviewExtension(pi: ExtensionAPI) {
 
 		const prNumber = parsePrReference(prRef);
 		if (!prNumber) {
-			ctx.ui.notify("Invalid PR reference. Enter a number or GitHub PR URL.", "error");
+			ctx.ui.notify(
+				"Invalid PR reference. Enter a number or GitHub PR URL.",
+				"error",
+			);
 			return null;
 		}
 
@@ -814,13 +935,19 @@ export default function reviewExtension(pi: ExtensionAPI) {
 		const prInfo = await getPrInfo(pi, prNumber);
 
 		if (!prInfo) {
-			ctx.ui.notify(`Could not find PR #${prNumber}. Make sure gh is authenticated and the PR exists.`, "error");
+			ctx.ui.notify(
+				`Could not find PR #${prNumber}. Make sure gh is authenticated and the PR exists.`,
+				"error",
+			);
 			return null;
 		}
 
 		// Check again for pending changes (in case something changed)
 		if (await hasPendingChanges(pi)) {
-			ctx.ui.notify("Cannot checkout PR: you have uncommitted changes. Please commit or stash them first.", "error");
+			ctx.ui.notify(
+				"Cannot checkout PR: you have uncommitted changes. Please commit or stash them first.",
+				"error",
+			);
 			return null;
 		}
 
@@ -846,10 +973,17 @@ export default function reviewExtension(pi: ExtensionAPI) {
 	/**
 	 * Execute the review
 	 */
-	async function executeReview(ctx: ExtensionCommandContext, target: ReviewTarget, useFreshSession: boolean): Promise<void> {
+	async function executeReview(
+		ctx: ExtensionCommandContext,
+		target: ReviewTarget,
+		useFreshSession: boolean,
+	): Promise<void> {
 		// Check if we're already in a review
 		if (reviewOriginId) {
-			ctx.ui.notify("Already in a review. Use /end-review to finish first.", "warning");
+			ctx.ui.notify(
+				"Already in a review. Use /end-review to finish first.",
+				"warning",
+			);
 			return;
 		}
 
@@ -858,7 +992,10 @@ export default function reviewExtension(pi: ExtensionAPI) {
 			// Store current position (where we'll return to)
 			const originId = ctx.sessionManager.getLeafId() ?? undefined;
 			if (!originId) {
-				ctx.ui.notify("Failed to determine review origin. Try again from a session with messages.", "error");
+				ctx.ui.notify(
+					"Failed to determine review origin. Try again from a session with messages.",
+					"error",
+				);
 				return;
 			}
 			reviewOriginId = originId;
@@ -881,7 +1018,10 @@ export default function reviewExtension(pi: ExtensionAPI) {
 			// Navigate to first user message to create a new branch from that point
 			// Label it as "code-review" so it's visible in the tree
 			try {
-				const result = await ctx.navigateTree(firstUserMessage.id, { summarize: false, label: "code-review" });
+				const result = await ctx.navigateTree(firstUserMessage.id, {
+					summarize: false,
+					label: "code-review",
+				});
 				if (result.cancelled) {
 					reviewOriginId = undefined;
 					return;
@@ -889,7 +1029,10 @@ export default function reviewExtension(pi: ExtensionAPI) {
 			} catch (error) {
 				// Clean up state if navigation fails
 				reviewOriginId = undefined;
-				ctx.ui.notify(`Failed to start review: ${error instanceof Error ? error.message : String(error)}`, "error");
+				ctx.ui.notify(
+					`Failed to start review: ${error instanceof Error ? error.message : String(error)}`,
+					"error",
+				);
 				return;
 			}
 
@@ -903,7 +1046,10 @@ export default function reviewExtension(pi: ExtensionAPI) {
 			setReviewWidget(ctx, true);
 
 			// Persist review state so tree navigation can restore/reset it
-			pi.appendEntry(REVIEW_STATE_TYPE, { active: true, originId: lockedOriginId });
+			pi.appendEntry(REVIEW_STATE_TYPE, {
+				active: true,
+				originId: lockedOriginId,
+			});
 		}
 
 		const prompt = await buildReviewPrompt(pi, target);
@@ -931,7 +1077,9 @@ export default function reviewExtension(pi: ExtensionAPI) {
 	 * Parse command arguments for direct invocation
 	 * Returns the target or a special marker for PR that needs async handling
 	 */
-	function parseArgs(args: string | undefined): ReviewTarget | { type: "pr"; ref: string } | null {
+	function parseArgs(
+		args: string | undefined,
+	): ReviewTarget | { type: "pr"; ref: string } | null {
 		if (!args?.trim()) return null;
 
 		const parts = args.trim().split(/\s+/);
@@ -980,16 +1128,25 @@ export default function reviewExtension(pi: ExtensionAPI) {
 	/**
 	 * Handle PR checkout and return a ReviewTarget (or null on failure)
 	 */
-	async function handlePrCheckout(ctx: ExtensionContext, ref: string): Promise<ReviewTarget | null> {
+	async function handlePrCheckout(
+		ctx: ExtensionContext,
+		ref: string,
+	): Promise<ReviewTarget | null> {
 		// First check for pending changes
 		if (await hasPendingChanges(pi)) {
-			ctx.ui.notify("Cannot checkout PR: you have uncommitted changes. Please commit or stash them first.", "error");
+			ctx.ui.notify(
+				"Cannot checkout PR: you have uncommitted changes. Please commit or stash them first.",
+				"error",
+			);
 			return null;
 		}
 
 		const prNumber = parsePrReference(ref);
 		if (!prNumber) {
-			ctx.ui.notify("Invalid PR reference. Enter a number or GitHub PR URL.", "error");
+			ctx.ui.notify(
+				"Invalid PR reference. Enter a number or GitHub PR URL.",
+				"error",
+			);
 			return null;
 		}
 
@@ -998,7 +1155,10 @@ export default function reviewExtension(pi: ExtensionAPI) {
 		const prInfo = await getPrInfo(pi, prNumber);
 
 		if (!prInfo) {
-			ctx.ui.notify(`Could not find PR #${prNumber}. Make sure gh is authenticated and the PR exists.`, "error");
+			ctx.ui.notify(
+				`Could not find PR #${prNumber}. Make sure gh is authenticated and the PR exists.`,
+				"error",
+			);
 			return null;
 		}
 
@@ -1023,7 +1183,8 @@ export default function reviewExtension(pi: ExtensionAPI) {
 
 	// Register the /review command
 	pi.registerCommand("review", {
-		description: "Review code changes (PR, uncommitted, branch, commit, folder, or custom)",
+		description:
+			"Review code changes (PR, uncommitted, branch, commit, folder, or custom)",
 		handler: async (args, ctx) => {
 			if (!ctx.hasUI) {
 				ctx.ui.notify("Review requires interactive mode", "error");
@@ -1032,7 +1193,10 @@ export default function reviewExtension(pi: ExtensionAPI) {
 
 			// Check if we're already in a review
 			if (reviewOriginId) {
-				ctx.ui.notify("Already in a review. Use /end-review to finish first.", "warning");
+				ctx.ui.notify(
+					"Already in a review. Use /end-review to finish first.",
+					"warning",
+				);
 				return;
 			}
 
@@ -1084,7 +1248,10 @@ export default function reviewExtension(pi: ExtensionAPI) {
 
 				if (messageCount > 0) {
 					// Existing session - ask user which mode they want
-					const choice = await ctx.ui.select("Start review in:", ["Empty branch", "Current session"]);
+					const choice = await ctx.ui.select("Start review in:", [
+						"Empty branch",
+						"Current session",
+					]);
 
 					if (choice === undefined) {
 						if (fromSelector) {
@@ -1161,7 +1328,10 @@ Instructions:
 		if (state?.active) {
 			setReviewWidget(ctx, false);
 			pi.appendEntry(REVIEW_STATE_TYPE, { active: false });
-			ctx.ui.notify("Review state was missing origin info; cleared review status.", "warning");
+			ctx.ui.notify(
+				"Review state was missing origin info; cleared review status.",
+				"warning",
+			);
 		}
 
 		return undefined;
@@ -1187,7 +1357,10 @@ Instructions:
 		const originId = getActiveReviewOrigin(ctx);
 		if (!originId) {
 			if (!getReviewState(ctx)?.active) {
-				ctx.ui.notify("Not in a review branch (use /review first, or review was started in current session mode)", "info");
+				ctx.ui.notify(
+					"Not in a review branch (use /review first, or review was started in current session mode)",
+					"info",
+				);
 			}
 			return;
 		}
@@ -1215,11 +1388,17 @@ Instructions:
 				try {
 					const result = await ctx.navigateTree(originId, { summarize: false });
 					if (result.cancelled) {
-						ctx.ui.notify("Navigation cancelled. Use /end-review to try again.", "info");
+						ctx.ui.notify(
+							"Navigation cancelled. Use /end-review to try again.",
+							"info",
+						);
 						return;
 					}
 				} catch (error) {
-					ctx.ui.notify(`Failed to return: ${error instanceof Error ? error.message : String(error)}`, "error");
+					ctx.ui.notify(
+						`Failed to return: ${error instanceof Error ? error.message : String(error)}`,
+						"error",
+					);
 					return;
 				}
 
@@ -1228,23 +1407,39 @@ Instructions:
 				return;
 			}
 
-			const summaryResult = await ctx.ui.custom<{ cancelled: boolean; error?: string } | null>((tui, theme, _kb, done) => {
-				const loader = new BorderedLoader(tui, theme, "Returning and summarizing review branch...");
+			const summaryResult = await ctx.ui.custom<{
+				cancelled: boolean;
+				error?: string;
+			} | null>((tui, theme, _kb, done) => {
+				const loader = new BorderedLoader(
+					tui,
+					theme,
+					"Returning and summarizing review branch...",
+				);
 				loader.onAbort = () => done(null);
 
-				ctx.navigateTree(originId, {
-					summarize: true,
-					customInstructions: REVIEW_SUMMARY_PROMPT,
-					replaceInstructions: true,
-				})
+				ctx
+					.navigateTree(originId, {
+						summarize: true,
+						customInstructions: REVIEW_SUMMARY_PROMPT,
+						replaceInstructions: true,
+					})
 					.then(done)
-					.catch((err) => done({ cancelled: false, error: err instanceof Error ? err.message : String(err) }));
+					.catch((err) =>
+						done({
+							cancelled: false,
+							error: err instanceof Error ? err.message : String(err),
+						}),
+					);
 
 				return loader;
 			});
 
 			if (summaryResult === null) {
-				ctx.ui.notify("Summarization cancelled. Use /end-review to try again.", "info");
+				ctx.ui.notify(
+					"Summarization cancelled. Use /end-review to try again.",
+					"info",
+				);
 				return;
 			}
 
@@ -1254,7 +1449,10 @@ Instructions:
 			}
 
 			if (summaryResult.cancelled) {
-				ctx.ui.notify("Navigation cancelled. Use /end-review to try again.", "info");
+				ctx.ui.notify(
+					"Navigation cancelled. Use /end-review to try again.",
+					"info",
+				);
 				return;
 			}
 
@@ -1269,7 +1467,10 @@ Instructions:
 			}
 
 			pi.sendUserMessage(REVIEW_FIX_FINDINGS_PROMPT, { deliverAs: "followUp" });
-			ctx.ui.notify("Review complete! Returned and queued a follow-up to fix findings.", "info");
+			ctx.ui.notify(
+				"Review complete! Returned and queued a follow-up to fix findings.",
+				"info",
+			);
 		} finally {
 			endReviewInProgress = false;
 		}