@oh-my-pi/pi-coding-agent 3.25.0 → 3.31.0

package/src/modes/interactive/components/extensions/extension-list.ts

@@ -31,7 +31,7 @@ export interface ExtensionListCallbacks {
 	masterSwitchProvider?: string | null;
 }
 
-const MAX_VISIBLE = 25;
+const DEFAULT_MAX_VISIBLE = 15;
 
 /** Flattened list item for rendering */
 type ListItem =
@@ -48,14 +48,21 @@ export class ExtensionList implements Component {
 	private focused = false;
 	private callbacks: ExtensionListCallbacks;
 	private masterSwitchProvider: string | null = null;
+	private maxVisible: number;
 
-	constructor(extensions: Extension[], callbacks: ExtensionListCallbacks = {}) {
+	constructor(extensions: Extension[], callbacks: ExtensionListCallbacks = {}, maxVisible?: number) {
 		this.extensions = extensions;
 		this.callbacks = callbacks;
 		this.masterSwitchProvider = callbacks.masterSwitchProvider ?? null;
+		this.maxVisible = maxVisible ?? DEFAULT_MAX_VISIBLE;
 		this.rebuildList();
 	}
 
+	setMaxVisible(maxVisible: number): void {
+		this.maxVisible = maxVisible;
+		this.clampSelection();
+	}
+
 	setExtensions(extensions: Extension[]): void {
 		this.extensions = extensions;
 		this.rebuildList();
@@ -126,7 +133,7 @@ export class ExtensionList implements Component {
 
 		// Calculate visible range
 		const startIdx = this.scrollOffset;
-		const endIdx = Math.min(startIdx + MAX_VISIBLE, this.listItems.length);
+		const endIdx = Math.min(startIdx + this.maxVisible, this.listItems.length);
 
 		// Render visible items
 		for (let i = startIdx; i < endIdx; i++) {
@@ -143,7 +150,7 @@ export class ExtensionList implements Component {
 		}
 
 		// Scroll indicator
-		if (this.listItems.length > MAX_VISIBLE) {
+		if (this.listItems.length > this.maxVisible) {
 			const indicator = theme.fg("muted", `  (${this.selectedIndex + 1}/${this.listItems.length})`);
 			lines.push(indicator);
 		}
@@ -388,8 +395,8 @@ export class ExtensionList implements Component {
 		// Adjust scroll offset
 		if (this.selectedIndex < this.scrollOffset) {
 			this.scrollOffset = this.selectedIndex;
-		} else if (this.selectedIndex >= this.scrollOffset + MAX_VISIBLE) {
-			this.scrollOffset = this.selectedIndex - MAX_VISIBLE + 1;
+		} else if (this.selectedIndex >= this.scrollOffset + this.maxVisible) {
+			this.scrollOffset = this.selectedIndex - this.maxVisible + 1;
 		}
 	}
 
@@ -470,8 +477,8 @@ export class ExtensionList implements Component {
 	private moveSelectionDown(): void {
 		if (this.selectedIndex < this.listItems.length - 1) {
 			this.selectedIndex++;
-			if (this.selectedIndex >= this.scrollOffset + MAX_VISIBLE) {
-				this.scrollOffset = this.selectedIndex - MAX_VISIBLE + 1;
+			if (this.selectedIndex >= this.scrollOffset + this.maxVisible) {
+				this.scrollOffset = this.selectedIndex - this.maxVisible + 1;
 			}
 			this.notifySelectionChange();
 		}

package/src/modes/interactive/components/hook-editor.ts

@@ -7,6 +7,7 @@ import * as fs from "node:fs";
 import * as os from "node:os";
 import * as path from "node:path";
 import { Container, Editor, isCtrlG, isEscape, Spacer, Text, type TUI } from "@oh-my-pi/pi-tui";
+import { nanoid } from "nanoid";
 import { getEditorTheme, theme } from "../theme/theme";
 import { DynamicBorder } from "./dynamic-border";
 
@@ -90,7 +91,7 @@ export class HookEditorComponent extends Container {
 		}
 
 		const currentText = this.editor.getText();
-		const tmpFile = path.join(os.tmpdir(), `omp-hook-editor-${Date.now()}.md`);
+		const tmpFile = path.join(os.tmpdir(), `omp-hook-editor-${nanoid()}.md`);
 
 		try {
 			fs.writeFileSync(tmpFile, currentText, "utf-8");

package/src/modes/interactive/components/model-selector.ts

@@ -34,7 +34,7 @@ interface ScopedModelItem {
 	thinkingLevel: string;
 }
 
-type ModelRole = "default" | "smol" | "slow";
+type ModelRole = "default" | "smol" | "slow" | "temporary";
 
 interface MenuAction {
 	label: string;
@@ -75,6 +75,7 @@ export class ModelSelectorComponent extends Container {
 	private errorMessage?: string;
 	private tui: TUI;
 	private scopedModels: ReadonlyArray<ScopedModelItem>;
+	private temporaryOnly: boolean;
 
 	// Tab state
 	private providers: string[] = [ALL_TAB];
@@ -92,6 +93,7 @@ export class ModelSelectorComponent extends Container {
 		scopedModels: ReadonlyArray<ScopedModelItem>,
 		onSelect: (model: Model<any>, role: string) => void,
 		onCancel: () => void,
+		options?: { temporaryOnly?: boolean },
 	) {
 		super();
 
@@ -102,6 +104,7 @@ export class ModelSelectorComponent extends Container {
 		this.scopedModels = scopedModels;
 		this.onSelectCallback = onSelect;
 		this.onCancelCallback = onCancel;
+		this.temporaryOnly = options?.temporaryOnly ?? false;
 
 		// Load current role assignments from settings
 		this._loadRoleModels();
@@ -483,10 +486,16 @@ export class ModelSelectorComponent extends Container {
 			return;
 		}
 
-		// Enter - open context menu
+		// Enter - open context menu or select directly in temporary mode
 		if (isEnter(keyData)) {
-			if (this.filteredModels[this.selectedIndex]) {
-				this.openMenu();
+			const selectedModel = this.filteredModels[this.selectedIndex];
+			if (selectedModel) {
+				if (this.temporaryOnly) {
+					// In temporary mode, skip menu and select directly
+					this.handleSelect(selectedModel.model, "temporary");
+				} else {
+					this.openMenu();
+				}
 			}
 			return;
 		}
@@ -536,6 +545,12 @@ export class ModelSelectorComponent extends Container {
 	}
 
 	private handleSelect(model: Model<any>, role: ModelRole): void {
+		// For temporary role, don't save to settings - just notify caller
+		if (role === "temporary") {
+			this.onSelectCallback(model, role);
+			return;
+		}
+
 		// Save to settings
 		this.settingsManager.setModelRole(role, `${model.provider}/${model.id}`);
 

package/src/modes/interactive/interactive-mode.ts

@@ -23,6 +23,7 @@ import {
 	TUI,
 	visibleWidth,
 } from "@oh-my-pi/pi-tui";
+import { nanoid } from "nanoid";
 import { getAuthPath, getDebugLogPath } from "../../config";
 import type { AgentSession, AgentSessionEvent } from "../../core/agent-session";
 import type { ExtensionUIContext } from "../../core/extensions/index";
@@ -941,9 +942,9 @@ export class InteractiveMode {
 		this.editor.onCtrlD = () => this.handleCtrlD();
 		this.editor.onCtrlZ = () => this.handleCtrlZ();
 		this.editor.onShiftTab = () => this.cycleThinkingLevel();
-		this.editor.onCtrlP = () => this.cycleModel("forward");
-		this.editor.onShiftCtrlP = () => this.cycleModel("backward");
-		this.editor.onCtrlY = () => this.cycleRoleModel();
+		this.editor.onCtrlP = () => this.cycleRoleModel();
+		this.editor.onShiftCtrlP = () => this.cycleRoleModel({ temporary: true });
+		this.editor.onCtrlY = () => this.showModelSelector({ temporaryOnly: true });
 
 		// Global debug handler on TUI (works regardless of focus)
 		this.ui.onDebug = () => this.handleDebugCommand();
@@ -951,9 +952,6 @@ export class InteractiveMode {
 		this.editor.onCtrlO = () => this.toggleToolOutputExpansion();
 		this.editor.onCtrlT = () => this.toggleThinkingBlockVisibility();
 		this.editor.onCtrlG = () => this.openExternalEditor();
-		this.editor.onCtrlY = () => {
-			void this.toggleVoiceListening();
-		};
 		this.editor.onQuestionMark = () => this.handleHotkeysCommand();
 		this.editor.onCtrlV = () => this.handleImagePaste();
 
@@ -991,6 +989,14 @@ export class InteractiveMode {
 	private setupEditorSubmitHandler(): void {
 		this.editor.onSubmit = async (text: string) => {
 			text = text.trim();
+
+			// Empty submit while streaming with queued messages: flush queues immediately
+			if (!text && this.session.isStreaming && this.session.queuedMessageCount > 0) {
+				// Abort current stream and let queued messages be processed
+				await this.session.abort();
+				return;
+			}
+
 			if (!text) return;
 
 			// Handle slash commands
@@ -1994,27 +2000,9 @@ export class InteractiveMode {
 		}
 	}
 
-	private async cycleModel(direction: "forward" | "backward"): Promise<void> {
-		try {
-			const result = await this.session.cycleModel(direction);
-			if (result === undefined) {
-				const msg = this.session.scopedModels.length > 0 ? "Only one model in scope" : "Only one model available";
-				this.showStatus(msg);
-			} else {
-				this.statusLine.invalidate();
-				this.updateEditorBorderColor();
-				const thinkingStr =
-					result.model.reasoning && result.thinkingLevel !== "off" ? ` (thinking: ${result.thinkingLevel})` : "";
-				this.showStatus(`Switched to ${result.model.name || result.model.id}${thinkingStr}`);
-			}
-		} catch (error) {
-			this.showError(error instanceof Error ? error.message : String(error));
-		}
-	}
-
-	private async cycleRoleModel(): Promise<void> {
+	private async cycleRoleModel(options?: { temporary?: boolean }): Promise<void> {
 		try {
-			const result = await this.session.cycleRoleModels(["slow", "default", "smol"]);
+			const result = await this.session.cycleRoleModels(["slow", "default", "smol"], options);
 			if (!result) {
 				this.showStatus("Only one role model available");
 				return;
@@ -2025,7 +2013,8 @@ export class InteractiveMode {
 			const roleLabel = result.role === "default" ? "default" : result.role;
 			const thinkingStr =
 				result.model.reasoning && result.thinkingLevel !== "off" ? ` (thinking: ${result.thinkingLevel})` : "";
-			this.showStatus(`Switched to ${roleLabel}: ${result.model.name || result.model.id}${thinkingStr}`);
+			const tempLabel = options?.temporary ? " (temporary)" : "";
+			this.showStatus(`Switched to ${roleLabel}: ${result.model.name || result.model.id}${thinkingStr}${tempLabel}`);
 		} catch (error) {
 			this.showError(error instanceof Error ? error.message : String(error));
 		}
@@ -2068,7 +2057,7 @@ export class InteractiveMode {
 		}
 
 		const currentText = this.editor.getText();
-		const tmpFile = path.join(os.tmpdir(), `omp-editor-${Date.now()}.omp.md`);
+		const tmpFile = path.join(os.tmpdir(), `omp-editor-${nanoid()}.omp.md`);
 
 		try {
 			// Write current content to temp file
@@ -2255,7 +2244,7 @@ export class InteractiveMode {
 	 */
 	private showExtensionsDashboard(): void {
 		this.showSelector((done) => {
-			const dashboard = new ExtensionDashboard(process.cwd(), this.settingsManager);
+			const dashboard = new ExtensionDashboard(process.cwd(), this.settingsManager, this.ui.terminal.rows);
 			dashboard.onClose = () => {
 				done();
 				this.ui.requestRender();
@@ -2379,7 +2368,7 @@ export class InteractiveMode {
 		}
 	}
 
-	private showModelSelector(): void {
+	private showModelSelector(options?: { temporaryOnly?: boolean }): void {
 		this.showSelector((done) => {
 			const selector = new ModelSelectorComponent(
 				this.ui,
@@ -2389,24 +2378,36 @@ export class InteractiveMode {
 				this.session.scopedModels,
 				async (model, role) => {
 					try {
-						// Only update agent state for default role
-						if (role === "default") {
+						if (role === "temporary") {
+							// Temporary: update agent state but don't persist to settings
+							await this.session.setModelTemporary(model);
+							this.statusLine.invalidate();
+							this.updateEditorBorderColor();
+							this.showStatus(`Temporary model: ${model.id}`);
+							done();
+							this.ui.requestRender();
+						} else if (role === "default") {
+							// Default: update agent state and persist
 							await this.session.setModel(model, role);
 							this.statusLine.invalidate();
 							this.updateEditorBorderColor();
+							this.showStatus(`Default model: ${model.id}`);
+							// Don't call done() - selector stays open for role assignment
+						} else {
+							// Other roles (smol, slow): just update settings, not current model
+							const roleLabel = role === "smol" ? "Smol" : role;
+							this.showStatus(`${roleLabel} model: ${model.id}`);
+							// Don't call done() - selector stays open
 						}
-						// For other roles (small), just show status - settings already updated by selector
-						const roleLabel = role === "default" ? "Default" : role === "smol" ? "Smol" : role;
-						this.showStatus(`${roleLabel} model: ${model.id}`);
 					} catch (error) {
 						this.showError(error instanceof Error ? error.message : String(error));
 					}
-					// Don't call done() - selector stays open
 				},
 				() => {
 					done();
 					this.ui.requestRender();
 				},
+				options,
 			);
 			return { component: selector, focus: selector };
 		});
@@ -2994,8 +2995,10 @@ export class InteractiveMode {
 | \`Ctrl+D\` | Exit (when editor is empty) |
 | \`Ctrl+Z\` | Suspend to background |
 | \`Shift+Tab\` | Cycle thinking level |
-| \`Ctrl+P\` | Cycle models |
-| \`Ctrl+Y\` | Cycle role models (slow/default/smol) |
+| \`Ctrl+P\` | Cycle role models (slow/default/smol) |
+| \`Shift+Ctrl+P\` | Cycle role models (temporary) |
+| \`Ctrl+Y\` | Select model (temporary) |
+| \`Ctrl+L\` | Select model (set roles) |
 | \`Ctrl+O\` | Toggle tool output expansion |
 | \`Ctrl+T\` | Toggle thinking block visibility |
 | \`Ctrl+G\` | Edit message in external editor |

package/src/modes/interactive/theme/theme.ts

@@ -315,18 +315,18 @@ const UNICODE_SYMBOLS: SymbolMap = {
 	"icon.rewind": "↩",
 	// pick: ⚡ | alt: ✨ ✦
 	"icon.auto": "⚡",
-	// pick: SK | alt: 🧠 🎓
-	"icon.extensionSkill": "SK",
-	// pick: TL | alt: 🛠 ⚙
-	"icon.extensionTool": "TL",
+	// pick: ✧ | alt: ⚙ SK 🧠
+	"icon.extensionSkill": "✧",
+	// pick: ⚒ | alt: ⛭ TL 🛠
+	"icon.extensionTool": "⚒",
 	// pick: / | alt: ⌘ ⌥
 	"icon.extensionSlashCommand": "/",
-	// pick: MCP | alt: 🔌 🧩
-	"icon.extensionMcp": "MCP",
-	// pick: RL | alt: ⚖ 📏
-	"icon.extensionRule": "RL",
-	// pick: HK | alt: 🪝 ⚓
-	"icon.extensionHook": "HK",
+	// pick: ◈ | alt: ⧫ MCP 🔌
+	"icon.extensionMcp": "◈",
+	// pick: § | alt: ⚖ RL 📏
+	"icon.extensionRule": "§",
+	// pick: ↪ | alt: ⚓ HK 🪝
+	"icon.extensionHook": "↪",
 	// pick: PR | alt: 💬 ✎
 	"icon.extensionPrompt": "PR",
 	// pick: CF | alt: 📄 📎
@@ -356,10 +356,10 @@ const UNICODE_SYMBOLS: SymbolMap = {
 	"format.bullet": "•",
 	// pick: – | alt: — ― -
 	"format.dash": "–",
-	// pick: [ | alt: ⟦ ⟨
-	"format.bracketLeft": "[",
-	// pick: ] | alt: ⟧ ⟩
-	"format.bracketRight": "]",
+	// pick: ⟨ | alt: [ ⟦
+	"format.bracketLeft": "⟨",
+	// pick: ⟩ | alt: ] ⟧
+	"format.bracketRight": "⟩",
 	// Markdown-specific
 	// pick: │ | alt: ┃ ║
 	"md.quoteBorder": "│",
@@ -574,15 +574,15 @@ const NERD_SYMBOLS: SymbolMap = {
 	"icon.extensionInstruction": "\uf02d",
 	// Thinking Levels - emoji labels
 	// pick: 🤨 min | alt:  min  min
-	"thinking.minimal": "🤨 min",
+	"thinking.minimal": "\u{F0E7} min",
 	// pick: 🤔 low | alt:  low  low
-	"thinking.low": "🤔 low",
+	"thinking.low": "\u{F10C} low",
 	// pick: 🤓 med | alt:  med  med
-	"thinking.medium": "🤓 med",
+	"thinking.medium": "\u{F192} med",
 	// pick: 🤯 high | alt:  high  high
-	"thinking.high": "🤯 high",
+	"thinking.high": "\u{F111} high",
 	// pick: 🧠 xhi | alt:  xhi  xhi
-	"thinking.xhigh": "🧠 xhi",
+	"thinking.xhigh": "\u{F06D} xhi",
 	// Checkboxes
 	// pick:  | alt:  
 	"checkbox.checked": "\uf14a",
@@ -595,10 +595,10 @@ const NERD_SYMBOLS: SymbolMap = {
 	"format.bullet": "\uf111",
 	// pick: – | alt: — ― -
 	"format.dash": "\u2013",
-	// pick: [ | alt: ⟦ ⟨
-	"format.bracketLeft": "[",
-	// pick: ] | alt: ⟧ ⟩
-	"format.bracketRight": "]",
+	// pick: ⟨ | alt: [ ⟦
+	"format.bracketLeft": "⟨",
+	// pick: ⟩ | alt: ] ⟧
+	"format.bracketRight": "⟩",
 	// Markdown-specific
 	// pick: │ | alt: ┃ ║
 	"md.quoteBorder": "\u2502",
@@ -608,41 +608,41 @@ const NERD_SYMBOLS: SymbolMap = {
 	"md.bullet": "\uf111",
 	// Language icons (nerd font devicons)
 	"lang.default": "",
-	"lang.typescript": "",
-	"lang.javascript": "",
-	"lang.python": "",
-	"lang.rust": "",
-	"lang.go": "",
-	"lang.java": "",
-	"lang.c": "",
-	"lang.cpp": "",
-	"lang.csharp": "",
-	"lang.ruby": "",
-	"lang.php": "",
-	"lang.swift": "",
-	"lang.kotlin": "",
-	"lang.shell": "",
-	"lang.html": "",
-	"lang.css": "",
-	"lang.json": "",
-	"lang.yaml": "",
-	"lang.markdown": "",
-	"lang.sql": "",
-	"lang.docker": "",
-	"lang.lua": "",
-	"lang.text": "",
-	"lang.env": "",
-	"lang.toml": "",
-	"lang.xml": "󰗀",
-	"lang.ini": "",
-	"lang.conf": "",
-	"lang.log": "󰌱",
-	"lang.csv": "󰈛",
-	"lang.tsv": "󰈛",
-	"lang.image": "󰈟",
-	"lang.pdf": "󰈦",
-	"lang.archive": "",
-	"lang.binary": "󰆚",
+	"lang.typescript": "\u{E628}",
+	"lang.javascript": "\u{E60C}",
+	"lang.python": "\u{E606}",
+	"lang.rust": "\u{E7A8}",
+	"lang.go": "\u{E627}",
+	"lang.java": "\u{E738}",
+	"lang.c": "\u{E61E}",
+	"lang.cpp": "\u{E61D}",
+	"lang.csharp": "\u{E7BC}",
+	"lang.ruby": "\u{E791}",
+	"lang.php": "\u{E608}",
+	"lang.swift": "\u{E755}",
+	"lang.kotlin": "\u{E634}",
+	"lang.shell": "\u{E795}",
+	"lang.html": "\u{E736}",
+	"lang.css": "\u{E749}",
+	"lang.json": "\u{E60B}",
+	"lang.yaml": "\u{E615}",
+	"lang.markdown": "\u{E609}",
+	"lang.sql": "\u{E706}",
+	"lang.docker": "\u{E7B0}",
+	"lang.lua": "\u{E620}",
+	"lang.text": "\u{E612}",
+	"lang.env": "\u{E615}",
+	"lang.toml": "\u{E615}",
+	"lang.xml": "\u{F05C0}",
+	"lang.ini": "\u{E615}",
+	"lang.conf": "\u{E615}",
+	"lang.log": "\u{F0331}",
+	"lang.csv": "\u{F021B}",
+	"lang.tsv": "\u{F021B}",
+	"lang.image": "\u{F021F}",
+	"lang.pdf": "\u{F0226}",
+	"lang.archive": "\u{F187}",
+	"lang.binary": "\u{F019A}",
 };
 
 const ASCII_SYMBOLS: SymbolMap = {

package/src/modes/rpc/rpc-mode.ts

@@ -11,6 +11,7 @@
  * - Extension UI: Extension UI requests are emitted, client responds with extension_ui_response
  */
 
+import { nanoid } from "nanoid";
 import type { AgentSession } from "../../core/agent-session";
 import type { ExtensionUIContext } from "../../core/extensions/index";
 import { theme } from "../interactive/theme/theme";
@@ -66,7 +67,7 @@ export async function runRpcMode(session: AgentSession): Promise<never> {
 	 */
 	const createExtensionUIContext = (): ExtensionUIContext => ({
 		async select(title: string, options: string[]): Promise<string | undefined> {
-			const id = globalThis.crypto.randomUUID();
+			const id = nanoid();
 			return new Promise((resolve, reject) => {
 				pendingExtensionRequests.set(id, {
 					resolve: (response: RpcExtensionUIResponse) => {
@@ -85,7 +86,7 @@ export async function runRpcMode(session: AgentSession): Promise<never> {
 		},
 
 		async confirm(title: string, message: string): Promise<boolean> {
-			const id = globalThis.crypto.randomUUID();
+			const id = nanoid();
 			return new Promise((resolve, reject) => {
 				pendingExtensionRequests.set(id, {
 					resolve: (response: RpcExtensionUIResponse) => {
@@ -104,7 +105,7 @@ export async function runRpcMode(session: AgentSession): Promise<never> {
 		},
 
 		async input(title: string, placeholder?: string): Promise<string | undefined> {
-			const id = globalThis.crypto.randomUUID();
+			const id = nanoid();
 			return new Promise((resolve, reject) => {
 				pendingExtensionRequests.set(id, {
 					resolve: (response: RpcExtensionUIResponse) => {
@@ -126,7 +127,7 @@ export async function runRpcMode(session: AgentSession): Promise<never> {
 			// Fire and forget - no response needed
 			output({
 				type: "extension_ui_request",
-				id: globalThis.crypto.randomUUID(),
+				id: nanoid(),
 				method: "notify",
 				message,
 				notifyType: type,
@@ -137,7 +138,7 @@ export async function runRpcMode(session: AgentSession): Promise<never> {
 			// Fire and forget - no response needed
 			output({
 				type: "extension_ui_request",
-				id: globalThis.crypto.randomUUID(),
+				id: nanoid(),
 				method: "setStatus",
 				statusKey: key,
 				statusText: text,
@@ -149,7 +150,7 @@ export async function runRpcMode(session: AgentSession): Promise<never> {
 			if (content === undefined || Array.isArray(content)) {
 				output({
 					type: "extension_ui_request",
-					id: globalThis.crypto.randomUUID(),
+					id: nanoid(),
 					method: "setWidget",
 					widgetKey: key,
 					widgetLines: content as string[] | undefined,
@@ -162,7 +163,7 @@ export async function runRpcMode(session: AgentSession): Promise<never> {
 			// Fire and forget - host can implement terminal title control
 			output({
 				type: "extension_ui_request",
-				id: globalThis.crypto.randomUUID(),
+				id: nanoid(),
 				method: "setTitle",
 				title,
 			} as RpcExtensionUIRequest);
@@ -177,7 +178,7 @@ export async function runRpcMode(session: AgentSession): Promise<never> {
 			// Fire and forget - host can implement editor control
 			output({
 				type: "extension_ui_request",
-				id: globalThis.crypto.randomUUID(),
+				id: nanoid(),
 				method: "set_editor_text",
 				text,
 			} as RpcExtensionUIRequest);
@@ -190,7 +191,7 @@ export async function runRpcMode(session: AgentSession): Promise<never> {
 		},
 
 		async editor(title: string, prefill?: string): Promise<string | undefined> {
-			const id = globalThis.crypto.randomUUID();
+			const id = nanoid();
 			return new Promise((resolve, reject) => {
 				pendingExtensionRequests.set(id, {
 					resolve: (response: RpcExtensionUIResponse) => {

package/src/prompts/review-request.md

@@ -0,0 +1,27 @@
+## Code Review Request
+
+### Mode
+{MODE}
+
+### Changed Files ({FILE_COUNT} files, +{LINES_ADDED}/-{LINES_REMOVED} lines)
+
+{FILE_TABLE}
+
+{EXCLUDED_SECTION}
+
+### Distribution Guidelines
+
+{DISTRIBUTION_GUIDANCE}
+
+{GROUPING_GUIDANCE}
+
+### Reviewer Instructions
+
+Each reviewer agent should:
+1. Focus ONLY on its assigned files
+2. {DIFF_INSTRUCTION}
+3. Read full file context as needed via the `read` tool
+4. Call `report_finding` for each issue found
+5. Call `complete` with verdict when done
+
+{DIFF_SECTION}