@oh-my-pi/pi-coding-agent 16.2.9 → 16.2.12

package/src/mcp/transports/stdio.test.ts

@@ -1,6 +1,6 @@
 import { describe, expect, it } from "bun:test";
 
-import { resolveStdioSpawnCommand } from "./stdio";
+import { resolveStdioSpawnCommand, StdioTransport } from "./stdio";
 
 describe("resolveStdioSpawnCommand", () => {
 	it("hides Windows executable MCP servers when the host has no console", async () => {
@@ -43,3 +43,53 @@ describe("resolveStdioSpawnCommand", () => {
 		});
 	});
 });
+
+// Regression for #3945: request() awaited stdin.write/flush, so a child that
+// stops draining stdin would park the async fn past the timeout timer and past
+// `return promise`, orphaning the deferred rejection and hanging the caller
+// forever. `sleep` is POSIX-only, so the check is scoped to non-Windows hosts.
+describe.skipIf(process.platform === "win32")("StdioTransport request write stall", () => {
+	it("rejects with the timeout error when the child never drains stdin", async () => {
+		const timeoutMs = 400;
+		const orphaned: Error[] = [];
+		const captureOrphan = (reason: unknown) => {
+			if (reason instanceof Error) orphaned.push(reason);
+		};
+		process.on("unhandledRejection", captureOrphan);
+
+		// `sleep 60` accepts a stdin pipe but never reads it; a 1 MB payload
+		// overruns the OS pipe buffer plus any FileSink JS-side buffering, so
+		// Bun's write() returns a Promise that only settles if the child reads.
+		const transport = new StdioTransport({
+			command: "sleep",
+			args: ["60"],
+			timeout: timeoutMs,
+		});
+
+		try {
+			await transport.connect();
+
+			const bigParam = "x".repeat(1024 * 1024);
+			const started = performance.now();
+			const outcome = await transport.request("tools/call", { name: "noop", arguments: { blob: bigParam } }).then(
+				() => ({ kind: "resolved" as const }),
+				(error: unknown) => ({ kind: "rejected" as const, error }),
+			);
+			const elapsedMs = performance.now() - started;
+
+			expect(outcome.kind).toBe("rejected");
+			if (outcome.kind !== "rejected") return;
+			if (!(outcome.error instanceof Error)) {
+				throw new Error(`expected Error rejection, got ${String(outcome.error)}`);
+			}
+			expect(outcome.error.message).toContain(`Request timeout after ${timeoutMs}ms`);
+			// Generous ceiling: bare rejection latency plus room for slow CI. The
+			// pre-fix behavior was an unbounded hang, not a slightly-late reject.
+			expect(elapsedMs).toBeLessThan(timeoutMs + 1500);
+			expect(orphaned).toEqual([]);
+		} finally {
+			process.off("unhandledRejection", captureOrphan);
+			await transport.close();
+		}
+	}, 8000);
+});

package/src/mcp/transports/stdio.ts

@@ -575,17 +575,27 @@ export class StdioTransport implements MCPTransport {
 
 		const stdin = this.#process.stdin;
 		const message = `${JSON.stringify(request)}\n`;
-		try {
-			// Await both: Bun's FileSink can surface a broken pipe either as a
-			// synchronous throw or as a rejected Promise (the EPIPE arrives on a
-			// processTicksAndRejections tick). Awaiting funnels both into this catch
-			// so the request rejects cleanly instead of leaving a floating rejected
-			// promise that crashes the process via the unhandledRejection handler.
-			await stdin.write(message);
-			await stdin.flush();
-		} catch (error: unknown) {
+		const failFromSend = (error: unknown) => {
+			if (settled) return;
 			cleanup();
 			reject(error instanceof Error ? error : new Error(String(error)));
+		};
+		try {
+			// Never `await` write/flush. Bun's FileSink returns a pending Promise
+			// once the OS pipe buffer fills (default ~64 KB on POSIX), and a
+			// subprocess that stops draining stdin will park those awaits forever.
+			// Awaiting here would keep the async fn stuck above `return promise`,
+			// past the timeout timer and the abort handler, orphaning the deferred
+			// rejection and hanging the caller (#3945). Route sync throws (Windows
+			// EPIPE) and async rejections (POSIX EPIPE on processTicksAndRejections)
+			// into `reject()` while leaving the returned promise free to settle
+			// from the response, timer, abort signal, or read-loop transport-close.
+			const wrote = stdin.write(message);
+			if (isThenable(wrote)) wrote.then(undefined, failFromSend);
+			const flushed = stdin.flush();
+			if (isThenable(flushed)) flushed.then(undefined, failFromSend);
+		} catch (error) {
+			failFromSend(error);
 		}
 
 		return promise;

package/src/memories/index.ts

@@ -1219,7 +1219,6 @@ async function resolveMemoryModel(options: {
 		const resolved = resolveModelRoleValue(requestedModel, modelRegistry.getAll(), {
 			settings: session.settings,
 			matchPreferences: getModelMatchPreferences(session.settings),
-			modelRegistry,
 		});
 		if (resolved.model) return resolved.model;
 	}

package/src/mnemopi/backend.ts

@@ -511,7 +511,7 @@ async function resolveMnemopiProviderOptions(
 	}
 
 	try {
-		const resolved = resolveRoleSelection(["tiny", "smol"], settings, modelRegistry.getAvailable(), modelRegistry);
+		const resolved = resolveRoleSelection(["tiny", "smol"], settings, modelRegistry.getAvailable());
 		const model = resolved?.model;
 		if (!model) {
 			logger.warn("Mnemopi: llmMode=smol but no tiny/smol model resolved; continuing without LLM.");

package/src/modes/acp/acp-agent.ts

@@ -53,7 +53,7 @@ import {
 } from "../../extensibility/extensions";
 import { runExtensionCompact } from "../../extensibility/extensions/compact-handler";
 import { getSessionSlashCommands } from "../../extensibility/extensions/get-commands-handler";
-import { buildSkillPromptMessage } from "../../extensibility/skills";
+import { buildSkillPromptMessage, parseSkillInvocation } from "../../extensibility/skills";
 import { loadSlashCommands } from "../../extensibility/slash-commands";
 import { resolveLocalUrlToPath } from "../../internal-urls";
 import { MCPManager } from "../../mcp/manager";
@@ -831,21 +831,18 @@ export class AcpAgent implements Agent {
 	}
 
 	async #tryRunSkillCommand(record: ManagedSessionRecord, text: string): Promise<boolean> {
-		if (!text.startsWith("/skill:")) {
+		if (!record.session.skillsSettings?.enableSkillCommands) {
 			return false;
 		}
-		if (!record.session.skillsSettings?.enableSkillCommands) {
+		const parsed = parseSkillInvocation(text);
+		if (!parsed) {
 			return false;
 		}
-		const spaceIndex = text.indexOf(" ");
-		const commandName = spaceIndex === -1 ? text.slice(1) : text.slice(1, spaceIndex);
-		const args = spaceIndex === -1 ? "" : text.slice(spaceIndex + 1).trim();
-		const skillName = commandName.slice("skill:".length);
-		const skill = record.session.skills.find(candidate => candidate.name === skillName);
+		const skill = record.session.skills.find(candidate => candidate.name === parsed.name);
 		if (!skill) {
 			return false;
 		}
-		const built = await buildSkillPromptMessage(skill, args);
+		const built = await buildSkillPromptMessage(skill, parsed.args);
 		await record.session.promptCustomMessage({
 			customType: SKILL_PROMPT_MESSAGE_TYPE,
 			content: built.message,

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

@@ -58,21 +58,6 @@ function makeRoleBadgeToken(label: string, color: ThemeColor, assigned: RoleAssi
 	return `${badge} ${theme.fg("dim", `(${thinkingLabel})`)}`;
 }
 
-function normalizeSearchText(value: string): string {
-	return value
-		.toLowerCase()
-		.replace(/[^a-z0-9]+/g, " ")
-		.trim();
-}
-
-function compactSearchText(value: string): string {
-	return value.toLowerCase().replace(/[^a-z0-9]+/g, "");
-}
-
-function getAlphaSearchTokens(query: string): string[] {
-	return [...normalizeSearchText(query).matchAll(/[a-z]+/g)].map(match => match[0]).filter(token => token.length > 0);
-}
-
 function computeModelRank(model: Model, roles: Record<string, RoleAssignment | undefined>): number {
 	let i = 0;
 	while (i < MODEL_ROLE_IDS.length) {
@@ -94,17 +79,6 @@ interface ModelItem {
 	selector: string;
 }
 
-interface CanonicalModelItem {
-	kind: "canonical";
-	id: string;
-	model: Model;
-	selector: string;
-	variantCount: number;
-	searchText: string;
-	normalizedSearchText: string;
-	compactSearchText: string;
-}
-
 interface ScopedModelItem {
 	model: Model;
 	thinkingLevel?: string;
@@ -134,12 +108,8 @@ interface ProviderTabState {
 	providerId?: string;
 }
 const ALL_TAB = "ALL";
-const CANONICAL_TAB = "CANONICAL";
 
-const STATIC_PROVIDER_TABS: ProviderTabState[] = [
-	{ id: ALL_TAB, label: ALL_TAB },
-	{ id: CANONICAL_TAB, label: CANONICAL_TAB },
-];
+const STATIC_PROVIDER_TABS: ProviderTabState[] = [{ id: ALL_TAB, label: ALL_TAB }];
 
 const MODEL_TAB_REFRESH_DEBOUNCE_MS = 120;
 
@@ -168,8 +138,6 @@ export class ModelSelectorComponent extends Container {
 	#menuContainer: Container;
 	#allModels: ModelItem[] = [];
 	#filteredModels: ModelItem[] = [];
-	#canonicalModels: CanonicalModelItem[] = [];
-	#filteredCanonicalModels: CanonicalModelItem[] = [];
 	#selectedIndex: number = 0;
 	#roles = {} as Record<string, RoleAssignment | undefined>;
 	#settings = null as unknown as Settings;
@@ -297,8 +265,7 @@ export class ModelSelectorComponent extends Container {
 		// Hydrate synchronously from the current registry snapshot so the first
 		// Enter after opening the selector acts on cached models instead of being
 		// dropped while the offline refresh promise is still pending. This stays
-		// on the open path, so it must remain cheap — heavy lifting lives in the
-		// registry's one-pass getCanonicalModelSelections.
+		// on the open path, so it must remain cheap.
 		this.#syncFromRegistryState();
 
 		// Reconcile with cached discovery state in the background. A --models
@@ -342,7 +309,6 @@ export class ModelSelectorComponent extends Container {
 			const resolved = resolveModelRoleValue(roleValue, allModels, {
 				settings: this.#settings,
 				matchPreferences,
-				modelRegistry: this.#modelRegistry,
 			});
 			if (resolved.model) {
 				nextRoles[role] = {
@@ -360,7 +326,6 @@ export class ModelSelectorComponent extends Container {
 				const resolved = resolveModelRoleValue(`pi/${role}`, candidates, {
 					settings: this.#settings,
 					matchPreferences,
-					modelRegistry: this.#modelRegistry,
 				});
 				if (!resolved.model) continue;
 				nextRoles[role] = {
@@ -444,30 +409,6 @@ export class ModelSelectorComponent extends Container {
 		});
 	}
 
-	#sortCanonicalModels(models: CanonicalModelItem[], { skipRoleRank = false }: { skipRoleRank?: boolean } = {}): void {
-		const mruOrder = this.#settings.getStorage()?.getModelUsageOrder() ?? [];
-		const mruIndex = new Map(mruOrder.map((key, i) => [key, i]));
-
-		const modelRank = (item: CanonicalModelItem) => computeModelRank(item.model, this.#roles);
-
-		models.sort((a, b) => {
-			if (!skipRoleRank) {
-				const aRank = modelRank(a);
-				const bRank = modelRank(b);
-				if (aRank !== bRank) return aRank - bRank;
-			}
-
-			const aMru = mruIndex.get(`${a.model.provider}/${a.model.id}`) ?? Number.MAX_SAFE_INTEGER;
-			const bMru = mruIndex.get(`${b.model.provider}/${b.model.id}`) ?? Number.MAX_SAFE_INTEGER;
-			if (aMru !== bMru) return aMru - bMru;
-
-			const providerCmp = a.model.provider.localeCompare(b.model.provider);
-			if (providerCmp !== 0) return providerCmp;
-
-			return a.id.localeCompare(b.id);
-		});
-	}
-
 	#loadModelsFromCurrentRegistryState(): void {
 		let models: ModelItem[];
 		if (this.#scopedModels.length > 0) {
@@ -498,8 +439,6 @@ export class ModelSelectorComponent extends Container {
 			} catch (error) {
 				this.#allModels = [];
 				this.#filteredModels = [];
-				this.#canonicalModels = [];
-				this.#filteredCanonicalModels = [];
 				this.#errorMessage = error instanceof Error ? error.message : String(error);
 				return;
 			}
@@ -507,42 +446,14 @@ export class ModelSelectorComponent extends Container {
 
 		const candidates = models.map(item => item.model);
 		this.#loadRoleModels(candidates);
-		const canonicalSelections = this.#modelRegistry.getCanonicalModelSelections({
-			availableOnly: this.#scopedModels.length === 0,
-			candidates,
-		});
-		const canonicalModels = canonicalSelections.map(({ record, model: selectedModel }): CanonicalModelItem => {
-			const searchText = [
-				record.id,
-				record.name,
-				selectedModel.provider,
-				selectedModel.id,
-				selectedModel.name,
-				...record.variants.flatMap(variant => [variant.selector, variant.model.name]),
-			].join(" ");
-			return {
-				kind: "canonical",
-				id: record.id,
-				model: selectedModel,
-				selector: record.id,
-				variantCount: record.variants.length,
-				searchText,
-				normalizedSearchText: normalizeSearchText(searchText),
-				compactSearchText: compactSearchText(searchText),
-			};
-		});
 
 		this.#sortModels(models);
-		this.#sortCanonicalModels(canonicalModels);
 
 		this.#allModels = models;
 		this.#filteredModels = models;
-		this.#canonicalModels = canonicalModels;
-		this.#filteredCanonicalModels = canonicalModels;
-		const visibleModels = this.#isCanonicalTab() ? canonicalModels : models;
 		this.#selectedIndex = this.#coerceSelectedIndex(
-			Math.min(this.#selectedIndex, Math.max(0, visibleModels.length - 1)),
-			visibleModels,
+			Math.min(this.#selectedIndex, Math.max(0, models.length - 1)),
+			models,
 		);
 	}
 
@@ -717,10 +628,6 @@ export class ModelSelectorComponent extends Container {
 		return this.#getActiveTab().providerId;
 	}
 
-	#isCanonicalTab(): boolean {
-		return this.#getActiveTabId() === CANONICAL_TAB;
-	}
-
 	#isModelOverCurrentContext(model: Model): boolean {
 		const contextWindow = model.contextWindow ?? 0;
 		return this.#currentContextTokens > 0 && contextWindow > 0 && this.#currentContextTokens > contextWindow;
@@ -734,7 +641,7 @@ export class ModelSelectorComponent extends Container {
 		return ` ${theme.status.disabled} context>${formatNumber(model.contextWindow ?? 0).toLowerCase()}`;
 	}
 
-	#isItemDisabled(item: ModelItem | CanonicalModelItem): boolean {
+	#isItemDisabled(item: ModelItem): boolean {
 		return this.#isModelOverContextLimit(item.model);
 	}
 
@@ -745,14 +652,11 @@ export class ModelSelectorComponent extends Container {
 		return this.#formatCurrentContextLimitSuffix(model);
 	}
 
-	#getVisibleItems(): ReadonlyArray<ModelItem | CanonicalModelItem> {
-		return this.#isCanonicalTab() ? this.#filteredCanonicalModels : this.#filteredModels;
+	#getVisibleItems(): ReadonlyArray<ModelItem> {
+		return this.#filteredModels;
 	}
 
-	#coerceSelectedIndex(
-		index: number,
-		visibleItems: ReadonlyArray<ModelItem | CanonicalModelItem> = this.#getVisibleItems(),
-	): number {
+	#coerceSelectedIndex(index: number, visibleItems: ReadonlyArray<ModelItem> = this.#getVisibleItems()): number {
 		const maxIndex = visibleItems.length - 1;
 		if (maxIndex < 0) {
 			return 0;
@@ -798,21 +702,16 @@ export class ModelSelectorComponent extends Container {
 	}
 
 	#filterModels(query: string): void {
-		const activeTabId = this.#getActiveTabId();
 		const activeProviderId = this.#getActiveProviderId();
-		const isCanonicalTab = activeTabId === CANONICAL_TAB;
 
-		// Start with all models or filter by provider/canonical view
 		let baseModels = this.#allModels;
-		const baseCanonicalModels = this.#canonicalModels;
 		if (activeProviderId) {
 			baseModels = this.#allModels.filter(m => m.provider === activeProviderId);
 		}
 
-		// Apply fuzzy filter if query is present
 		if (query.trim()) {
 			// If user is searching from a provider tab, auto-switch to ALL to show global provider results.
-			if (activeProviderId && !isCanonicalTab) {
+			if (activeProviderId) {
 				this.#activeTabIndex = 0;
 				if (this.#tabBar && this.#tabBar.getActiveIndex() !== 0) {
 					this.#tabBar.setActiveIndex(0);
@@ -822,55 +721,22 @@ export class ModelSelectorComponent extends Container {
 				baseModels = this.#allModels;
 			}
 
-			if (isCanonicalTab) {
-				const alphaTokens = getAlphaSearchTokens(query);
-				const alphaFiltered =
-					alphaTokens.length === 0
-						? baseCanonicalModels
-						: baseCanonicalModels.filter(item =>
-								alphaTokens.every(token => item.normalizedSearchText.includes(token)),
-							);
-				const compactQuery = compactSearchText(query);
-				const substringFiltered =
-					compactQuery.length === 0
-						? alphaFiltered
-						: alphaFiltered.filter(item => item.compactSearchText.includes(compactQuery));
-				const fuzzySource =
-					substringFiltered.length > 0
-						? substringFiltered
-						: alphaFiltered.length > 0
-							? alphaFiltered
-							: baseCanonicalModels;
-				// Fuzzy provides the candidate set, but `${provider}/${id}` scoring
-				// is biased by provider-prefix length (e.g. `openai/X` beats
-				// `openai-codex/X` purely because the prefix is shorter). Re-sort by
-				// affinity — MRU then version — so the user's actually-used model
-				// wins. Role rank is skipped: when narrowing by query, a weakly
-				// matching default model should not be promoted above a stronger
-				// non-default match.
-				const fuzzyMatches = fuzzyFilter(fuzzySource, query, ({ searchText }) => searchText);
-				this.#sortCanonicalModels(fuzzyMatches, { skipRoleRank: true });
-				this.#filteredCanonicalModels = fuzzyMatches;
-			} else {
-				// Match against the displayed "provider/id" string so the user can
-				// type what they see: bare names (`mimo`, `kimi`), provider prefixes
-				// (`openrouter`), or scoped queries (`openrouter/mimo`) all flow
-				// through the same fuzzy matcher. The score is biased by provider-
-				// prefix length, so re-sort by MRU/version afterwards; skip role
-				// rank so a weakly matching default doesn't trump a stronger match.
-				const fuzzyMatches = fuzzyFilter(baseModels, query, ({ id, provider }) => `${provider}/${id}`);
-				this.#sortModels(fuzzyMatches, { skipRoleRank: true });
-				this.#filteredModels = fuzzyMatches;
-			}
+			// Match against the displayed "provider/id" string so the user can
+			// type what they see: bare names (`mimo`, `kimi`), provider prefixes
+			// (`openrouter`), or scoped queries (`openrouter/mimo`) all flow
+			// through the same fuzzy matcher. The score is biased by provider-
+			// prefix length, so re-sort by MRU/version afterwards; skip role
+			// rank so a weakly matching default doesn't trump a stronger match.
+			const fuzzyMatches = fuzzyFilter(baseModels, query, ({ id, provider }) => `${provider}/${id}`);
+			this.#sortModels(fuzzyMatches, { skipRoleRank: true });
+			this.#filteredModels = fuzzyMatches;
 		} else {
 			this.#filteredModels = baseModels;
-			this.#filteredCanonicalModels = baseCanonicalModels;
 		}
 
-		const visibleItems = isCanonicalTab ? this.#filteredCanonicalModels : this.#filteredModels;
 		this.#selectedIndex = this.#coerceSelectedIndex(
-			Math.min(this.#selectedIndex, Math.max(0, visibleItems.length - 1)),
-			visibleItems,
+			Math.min(this.#selectedIndex, Math.max(0, this.#filteredModels.length - 1)),
+			this.#filteredModels,
 		);
 		this.#updateList();
 	}
@@ -940,8 +806,7 @@ export class ModelSelectorComponent extends Container {
 
 	#updateList(): void {
 		this.#listContainer.clear();
-		const isCanonicalTab = this.#isCanonicalTab();
-		const visibleItems = isCanonicalTab ? this.#filteredCanonicalModels : this.#filteredModels;
+		const visibleItems = this.#filteredModels;
 
 		const maxVisible = 10;
 		const startIndex = Math.max(
@@ -959,8 +824,6 @@ export class ModelSelectorComponent extends Container {
 		for (let i = startIndex; i < endIndex; i++) {
 			const item = visibleItems[i];
 			if (!item) continue;
-			const canonicalItem = isCanonicalTab ? (item as CanonicalModelItem) : undefined;
-			const providerItem = isCanonicalTab ? undefined : (item as ModelItem);
 
 			const isSelected = i === this.#selectedIndex;
 			const isDisabled = this.#isItemDisabled(item);
@@ -988,25 +851,17 @@ export class ModelSelectorComponent extends Container {
 			let line = "";
 			if (isSelected) {
 				const prefix = theme.fg("accent", `${theme.nav.cursor} `);
-				if (isCanonicalTab) {
-					const variants = theme.fg("dim", ` [${canonicalItem?.variantCount ?? 0}]`);
-					const backing = theme.fg("dim", ` -> ${item.model.provider}/${item.model.id}`);
-					line = `${prefix}${theme.fg("accent", item.id)}${variants}${backing}${badgeText}${disabledSuffix}`;
-				} else if (showProvider) {
-					const providerPrefix = theme.fg("dim", `${providerItem?.provider ?? ""}/`);
-					line = `${prefix}${providerPrefix}${theme.fg("accent", providerItem?.id ?? item.id)}${badgeText}${disabledSuffix}`;
+				if (showProvider) {
+					const providerPrefix = theme.fg("dim", `${item.provider}/`);
+					line = `${prefix}${providerPrefix}${theme.fg("accent", item.id)}${badgeText}${disabledSuffix}`;
 				} else {
 					line = `${prefix}${theme.fg("accent", item.id)}${badgeText}${disabledSuffix}`;
 				}
 			} else {
 				const prefix = "  ";
-				if (isCanonicalTab) {
-					const variants = theme.fg("dim", ` [${canonicalItem?.variantCount ?? 0}]`);
-					const backing = theme.fg("dim", ` -> ${item.model.provider}/${item.model.id}`);
-					line = `${prefix}${item.id}${variants}${backing}${badgeText}${disabledSuffix}`;
-				} else if (showProvider) {
-					const providerPrefix = theme.fg("dim", `${providerItem?.provider ?? ""}/`);
-					line = `${prefix}${providerPrefix}${providerItem?.id ?? item.id}${badgeText}${disabledSuffix}`;
+				if (showProvider) {
+					const providerPrefix = theme.fg("dim", `${item.provider}/`);
+					line = `${prefix}${providerPrefix}${item.id}${badgeText}${disabledSuffix}`;
 				} else {
 					line = `${prefix}${item.id}${badgeText}${disabledSuffix}`;
 				}
@@ -1044,9 +899,6 @@ export class ModelSelectorComponent extends Container {
 				return;
 			}
 			this.#listContainer.addChild(new Spacer(1));
-			const suffix = isCanonicalTab
-				? ` (${selected.model.provider}/${selected.model.id}, ${(selected as CanonicalModelItem).variantCount} variants)`
-				: "";
 			const limitWarning = this.#isItemDisabled(selected)
 				? theme.fg(
 						"dim",
@@ -1054,7 +906,7 @@ export class ModelSelectorComponent extends Container {
 					)
 				: "";
 			this.#listContainer.addChild(
-				new Text(theme.fg("muted", `  Model Name: ${selected.model.name}${suffix}`) + limitWarning, 0, 0),
+				new Text(theme.fg("muted", `  Model Name: ${selected.model.name}`) + limitWarning, 0, 0),
 			);
 		}
 	}
@@ -1086,10 +938,8 @@ export class ModelSelectorComponent extends Container {
 		return foundIndex >= 0 ? foundIndex : 0;
 	}
 
-	#getSelectedItem(): ModelItem | CanonicalModelItem | undefined {
-		return this.#isCanonicalTab()
-			? this.#filteredCanonicalModels[this.#selectedIndex]
-			: this.#filteredModels[this.#selectedIndex];
+	#getSelectedItem(): ModelItem | undefined {
+		return this.#filteredModels[this.#selectedIndex];
 	}
 
 	#coerceMenuSelectedIndex(index: number): number {
@@ -1100,7 +950,7 @@ export class ModelSelectorComponent extends Container {
 		return Math.max(0, Math.min(index, maxIndex));
 	}
 
-	#moveMenuSelection(delta: number, _selectedItem: ModelItem | CanonicalModelItem, optionCount: number): void {
+	#moveMenuSelection(delta: number, _selectedItem: ModelItem, optionCount: number): void {
 		this.#menuSelectedIndex = (this.#menuSelectedIndex + delta + optionCount) % optionCount;
 		this.#updateMenu();
 	}
@@ -1375,11 +1225,7 @@ export class ModelSelectorComponent extends Container {
 		}
 	}
 
-	#handleSelect(
-		item: ModelItem | CanonicalModelItem,
-		role: string | null,
-		thinkingLevel?: ConfiguredThinkingLevel,
-	): void {
+	#handleSelect(item: ModelItem, role: string | null, thinkingLevel?: ConfiguredThinkingLevel): void {
 		if (this.#isItemDisabled(item)) {
 			return;
 		}

package/src/modes/components/todo-reminder.ts

@@ -3,7 +3,9 @@ import { theme } from "../../modes/theme/theme";
 import type { TodoItem } from "../../tools/todo";
 
 /**
- * Component that renders a todo completion reminder notification.
+ * Component that renders a todo completion reminder notification, committed into
+ * the transcript like a TTSR notification so it stays anchored in history rather
+ * than floating above the editor.
  * Shows when the agent stops with incomplete todos.
  */
 export class TodoReminderComponent extends Container {
@@ -16,6 +18,8 @@ export class TodoReminderComponent extends Container {
 	) {
 		super();
 
+		this.addChild(new Spacer(1));
+
 		this.#box = new Box(1, 1, t => theme.inverse(theme.fg("warning", t)));
 		this.#box.setIgnoreTight(true);
 		this.addChild(this.#box);

package/src/modes/controllers/event-controller.ts

@@ -609,15 +609,6 @@ export class EventController {
 			}
 			for (const content of this.ctx.streamingMessage.content) {
 				if (content.type !== "toolCall") continue;
-				// Anthropic/OpenAI open a streamed tool block with an empty id (and
-				// `{}` args) before the id/arguments arrive; Gemini assembles the
-				// whole call first, so it never hits this. Keying `pendingTools` by
-				// "" would create a placeholder card, and the later real-id frame —
-				// `pendingTools.has(realId)` false — would create a SECOND card,
-				// orphaning the blank one (no `tool_execution_*` event ever carries
-				// "", so it is never matched, updated, or removed). Defer until the
-				// provider assigns the real id.
-				if (!content.id) continue;
 				if (content.name === "read") {
 					if (!readArgsHaveTarget(content.arguments)) {
 						// Args still streaming — defer until path is parseable so we can route to the
@@ -978,13 +969,9 @@ export class EventController {
 		}
 		// Update todo display when todo tool completes
 		if (event.toolName === "todo" && !event.isError) {
-			const hadTodoReminder = (this.ctx.todoReminderContainer?.children.length ?? 0) > 0;
-			this.ctx.todoReminderContainer?.clear();
 			const details = event.result.details as { phases?: TodoPhase[] } | undefined;
 			if (details?.phases) {
 				this.ctx.setTodos(details.phases);
-			} else if (hadTodoReminder) {
-				this.ctx.ui.requestRender();
 			}
 		} else if (event.toolName === "todo" && event.isError) {
 			const textContent = event.result.content.find(
@@ -1281,9 +1268,7 @@ export class EventController {
 
 	async #handleTodoReminder(event: Extract<AgentSessionEvent, { type: "todo_reminder" }>): Promise<void> {
 		const component = new TodoReminderComponent(event.todos, event.attempt, event.maxAttempts);
-		this.ctx.todoReminderContainer.clear();
-		this.ctx.todoReminderContainer.addChild(component);
-		this.ctx.ui.requestRender();
+		this.ctx.present(component);
 	}
 
 	async #handleTodoAutoClear(_event: Extract<AgentSessionEvent, { type: "todo_auto_clear" }>): Promise<void> {