@oh-my-pi/pi-coding-agent 17.2.7 → 17.2.9

package/dist/types/async/job-manager.d.ts

@@ -51,6 +51,11 @@ export interface AsyncJobDeliveryState {
     nextRetryAt?: number;
     pendingJobIds: string[];
 }
+export interface AsyncJobReapResult {
+    settled: boolean;
+    pendingJobIds: string[];
+    completion: Promise<void>;
+}
 export interface AsyncJobRegisterOptions {
     id?: string;
     /** Registry id of the agent that owns this job; used to scope cancelAll. */
@@ -167,6 +172,13 @@ export declare class AsyncJobManager {
         timeoutMs?: number;
         excludeSuppressed?: boolean;
     }): Promise<boolean>;
+    /**
+     * Cancel every job owned by `ownerId`, then wait only until `deadlineAt`.
+     * The returned completion keeps waiting for actual process settlement when
+     * the deadline expires, so callers can move that cleanup out of the
+     * user-visible Task wait without losing ownership of the live work.
+     */
+    cancelAndReapOwnerJobs(ownerId: string, deadlineAt: number): Promise<AsyncJobReapResult>;
     drainDeliveries(options?: {
         timeoutMs?: number;
         filter?: AsyncJobFilter;

package/dist/types/discovery/agents.d.ts

@@ -1,4 +1,14 @@
 import type { LoadContext } from "../capability/types.js";
+interface UserPathCandidateOptions {
+    platform?: NodeJS.Platform;
+    env?: NodeJS.ProcessEnv;
+    windowsUserProfile?: () => string | undefined;
+    wslPath?: (windowsPath: string) => string | undefined;
+}
+/** Resolve the Windows host profile home exposed to WSL, if available. */
+export declare function getWslWindowsHomeCandidate(options?: UserPathCandidateOptions): string | undefined;
+/** User-level paths: ~/.agent[s]/<segments>, plus the Windows host profile under WSL. */
+export declare function getUserPathCandidates(ctx: LoadContext, ...segments: string[]): string[];
 /**
  * Project-level paths: walk up from cwd to repoRoot, returning `.agent/<segments>`
  * and `.agents/<segments>` at each ancestor.
@@ -10,3 +20,4 @@ import type { LoadContext } from "../capability/types.js";
  * directory — see https://github.com/can1357/oh-my-pi/issues/1116.
  */
 export declare function getProjectPathCandidates(ctx: LoadContext, ...segments: string[]): string[];
+export {};

package/dist/types/extensibility/legacy-pi-coding-agent-shim.d.ts

@@ -92,6 +92,16 @@ export interface WriteOperations {
 export interface WriteToolOptions {
     operations?: WriteOperations;
 }
+/**
+ * Convert an image attachment to PNG using the legacy package-root contract.
+ *
+ * Invalid or unsupported image data returns `null`, matching Pi's historical
+ * helper instead of surfacing Bun's decoder error to extensions.
+ */
+export declare function convertToPng(base64Data: string, mimeType: string): Promise<{
+    data: string;
+    mimeType: string;
+} | null>;
 /** Format the active shortcut for legacy extensions that render keybinding hints. */
 export declare function keyText(action: Keybinding): string;
 /** Parse frontmatter using the historical Pi package-root helper. */

package/dist/types/hindsight/state.d.ts

@@ -93,7 +93,6 @@ export declare class HindsightSessionState {
     retainSession(messages: HindsightMessage[]): Promise<void>;
     maybeRetainOnAgentEnd(): Promise<void>;
     forceRetainCurrentSession(): Promise<void>;
-    maybeRecallOnAgentStart(): Promise<void>;
     beforeAgentStartPrompt(promptText: string): Promise<string | undefined>;
     recallForCompaction(messages: HindsightMessage[]): Promise<string | undefined>;
     runMentalModelLoad(scope: BankScope): Promise<void>;

package/dist/types/lsp/utils.d.ts

@@ -14,6 +14,14 @@ export declare function fileToUri(filePath: string): string;
  * Handles Windows drive letters correctly.
  */
 export declare function uriToFile(uri: string): string;
+/** Map that treats equivalent file URI spellings as the same key. */
+export declare class EquivalentUriMap<Value> extends Map<string, Value> {
+    #private;
+    delete(uri: string): boolean;
+    get(uri: string): Value | undefined;
+    has(uri: string): boolean;
+    set(uri: string, value: Value): this;
+}
 /**
  * Convert diagnostic severity number to string name.
  */

package/dist/types/modes/session-observer-registry.d.ts

@@ -28,6 +28,8 @@ export declare class SessionObserverRegistry {
     /** Add a change listener. Returns unsubscribe function. */
     onChange(cb: (kind: SessionObserverChangeKind) => void): () => void;
     setMainSession(sessionFile?: string): void;
+    /** Return one tracked session without copying or sorting the registry. */
+    getSession(id: string): ObservableSession | undefined;
     getSessions(): ObservableSession[];
     getActiveSubagentCount(): number;
     /** Clear all tracked sessions (e.g. on session switch). Keeps EventBus subscriptions and listeners. */

package/dist/types/registry/agent-lifecycle.d.ts

@@ -108,5 +108,5 @@ export declare class AgentLifecycleManager {
         tombstone?: boolean;
     }): Promise<boolean>;
     /** Teardown everything (process exit / main session dispose). */
-    dispose(): Promise<void>;
+    dispose(deadlineAt?: number): Promise<void>;
 }

package/dist/types/session/agent-session-types.d.ts

@@ -65,6 +65,13 @@ export interface UsageFallbackConfirmation {
     to: string;
     remainingPercent: number | undefined;
 }
+/**
+ * Confirms whether a reserve-triggered model fallback may proceed.
+ *
+ * Interactive callers use the confirmation details to present the pending
+ * route change; aborting `signal` cancels that pending confirmation.
+ */
+export type UsageFallbackConfirmer = (confirmation: UsageFallbackConfirmation, signal: AbortSignal) => Promise<boolean>;
 /** Identifies a retry fallback chain already entered during startup model resolution. */
 export interface InitialRetryFallbackState {
     /** Role whose configured primary was unavailable. */

package/dist/types/session/agent-session.d.ts

@@ -20,7 +20,7 @@ import { type Effort } from "@oh-my-pi/pi-ai";
 import { type AdvisorConfig, type AdvisorRuntimeStatus } from "../advisor/index.js";
 import { AsyncJobManager } from "../async/index.js";
 import type { ModelRegistry } from "../config/model-registry.js";
-import { type ResolvedModelRoleValue } from "../config/model-resolver.js";
+import type { ResolvedModelRoleValue } from "../config/model-resolver.js";
 import { type PromptTemplate } from "../config/prompt-templates.js";
 import type { Settings, SkillsSettings } from "../config/settings.js";
 import { RawSseDebugBuffer } from "../debug/raw-sse-buffer.js";
@@ -50,7 +50,7 @@ import type { TodoPhase } from "../tools/todo.js";
 import type { InspectImageMode } from "../utils/inspect-image-mode.js";
 import type { VibeModeState } from "../vibe/state.js";
 import type { AgentSessionEventListener } from "./agent-session-events.js";
-import type { AgentSessionConfig, AgentSessionDisposeOptions, AsyncJobSnapshot, CommandMetadataChangedListener, ContextUsageBreakdown, FollowUpOptions, FreshSessionResult, HandoffResult, ModelCycleResult, Prewalk, PromptOptions, ResetSessionContextResult, ResolvedRoleModel, RestoredQueuedMessage, RoleModelCycle, RoleModelCycleResult, SessionHandoffOptions, SessionOAuthAccountList, SessionStats, UsageFallbackConfirmation } from "./agent-session-types.js";
+import type { AgentSessionConfig, AgentSessionDisposeOptions, AsyncJobSnapshot, CommandMetadataChangedListener, ContextUsageBreakdown, FollowUpOptions, FreshSessionResult, HandoffResult, ModelCycleResult, Prewalk, PromptOptions, ResetSessionContextResult, ResolvedRoleModel, RestoredQueuedMessage, RoleModelCycle, RoleModelCycleResult, SessionHandoffOptions, SessionOAuthAccountList, SessionStats, UsageFallbackConfirmer } from "./agent-session-types.js";
 import type { ClientBridge } from "./client-bridge.js";
 import { type CustomMessage, type CustomMessagePayload } from "./messages.js";
 import { type AdvisorStats } from "./session-advisors.js";
@@ -228,7 +228,7 @@ export declare class AgentSession {
     /** Resolved selector while retry routing is using a fallback model. */
     get retryFallbackModel(): string | undefined;
     /** Install the interactive decision surface for reserve-triggered model changes. */
-    setUsageFallbackConfirmer(confirmer: ((confirmation: UsageFallbackConfirmation) => Promise<boolean>) | undefined): void;
+    setUsageFallbackConfirmer(confirmer: UsageFallbackConfirmer | undefined): void;
     /** Effective thinking level applied to the agent (the resolved level when `auto`). */
     get thinkingLevel(): ThinkingLevel | undefined;
     /** The selector the user configured: `auto` when auto mode is active, else the effective level. */
@@ -267,6 +267,8 @@ export declare class AgentSession {
     getLastAssistantMessage(): AssistantMessage | undefined;
     /** Current effective system prompt blocks (includes any per-turn extension modifications) */
     get systemPrompt(): string[];
+    /** Marks streamed text as committed or buffered for turn-recovery replay decisions. */
+    setTextOutputCommitted(committed: boolean): void;
     /** Current retry attempt (0 if not retrying) */
     get retryAttempt(): number;
     /** Names of tools currently exposed at the top level. */

package/dist/types/session/turn-recovery.d.ts

@@ -4,7 +4,7 @@ import type { ModelRegistry } from "../config/model-registry.js";
 import type { Settings } from "../config/settings.js";
 import { type ConfiguredThinkingLevel } from "../thinking.js";
 import type { AgentSessionEvent } from "./agent-session-events.js";
-import type { InitialRetryFallbackState } from "./agent-session-types.js";
+import type { InitialRetryFallbackState, UsageFallbackConfirmer } from "./agent-session-types.js";
 import { type RetryFallbackSelector } from "./retry-fallback-chains.js";
 import type { SessionManager } from "./session-manager.js";
 /** Result shape shared with automatic maintenance recovery. */
@@ -22,6 +22,8 @@ export interface TurnRecoveryHost {
     modelRegistry: ModelRegistry;
     configWarnings: string[];
     model(): Model | undefined;
+    /** Whether streamed text has already been committed to the active output sink. */
+    textOutputCommitted(): boolean;
     thinkingLevel(): ThinkingLevel | undefined;
     configuredThinkingLevel(): ConfiguredThinkingLevel | undefined;
     setThinkingLevel(level: ConfiguredThinkingLevel | undefined): void;
@@ -100,6 +102,8 @@ export declare class TurnRecovery {
     }): Promise<RecoveryCompactionResult>;
     /** Restores the configured primary after fallback cooldown expiry. */
     maybeRestoreRetryFallbackPrimary(): Promise<void>;
+    /** Applies model fallback policy from live usage health before a turn starts. */
+    maybeApplyUsageAwareFallback(signal: AbortSignal, confirmer?: UsageFallbackConfirmer): Promise<boolean>;
     /** Applies automatic retry, credential rotation, and model fallback policy. */
     handleRetryableError(message: AssistantMessage, options?: {
         allowModelFallback?: boolean;
@@ -181,7 +185,7 @@ export declare class TurnRecovery {
         pinFallback?: boolean;
         apiKey?: string;
         signal?: AbortSignal;
-    }): Promise<void>;
+    }): Promise<boolean>;
     /**
      * True when the current turn failed on a Fireworks Fast (`-fast`) model in a
      * way that should degrade to the reliable base (Standard) model. Fast is a

package/dist/types/task/executor.d.ts

@@ -209,6 +209,8 @@ export interface ExecutorOptions {
      * set this false so disposal unregisters them instead of leaving idle peers.
      */
     keepAlive?: boolean;
+    /** Internal ownership handoff for cleanup that outlives the visible Task result. */
+    onCleanupDeferred?: (completion: Promise<void>) => void;
 }
 interface FinalizeSubprocessOutputArgs {
     rawOutput: string;
@@ -294,6 +296,8 @@ export declare function finalizeSubagentLifecycle(args: {
     isolated: boolean;
     agentIdleTtlMs: number;
     reviveSession: AgentReviver | null;
+    cleanupDeadlineAt?: number;
+    onCleanupDeferred?: (completion: Promise<void>) => void;
 }): Promise<void>;
 /** Options for {@link runSubagentFollowUpTurn}. */
 export interface FollowUpTurnOptions {

package/dist/types/tools/browser/launch.d.ts

@@ -115,6 +115,8 @@ export declare function applyStealthPatches(browser: Browser, page: Page, state:
     browserSession: CDPSession | null;
     override: UserAgentOverride | null;
 }): Promise<void>;
+/** Exposes executable candidates for detection tests. */
+export declare function systemChromiumCandidatesForTest(platform?: NodeJS.Platform, home?: string, which?: (name: string) => string | undefined): string[];
 export declare function stealthIgnoreDefaultArgsForTest(executablePath: string | undefined): string[];
 export declare function targetSupportsUserAgentOverrideForTest(target: Target): boolean;
 export declare function configureUserAgentTargetsForTest(browser: Browser, state: {

package/dist/types/tools/path-utils.d.ts

@@ -44,6 +44,13 @@ export declare function splitPathAndSel(rawPath: string): {
  * plus selector `1-2` (issue #4618). `lstat` inspects the entry itself, so a
  * dangling symlink is still detected as present; ambiguous errors resolve to
  * `"unknown"` so callers keep the raw path instead of guessing.
+ *
+ * `ENAMETOOLONG` resolves to `"missing"` rather than `"unknown"`: a path whose
+ * component or whole length exceeds the OS limit can never name a real single
+ * entry, so it is strictly stronger evidence of non-existence than `ENOENT`.
+ * Without this, a semicolon-joined `path` list long enough to trip the limit
+ * (bare filenames past `NAME_MAX`, or a total past `PATH_MAX`) was read as one
+ * literal path and the delimited split was suppressed (issue #7597).
  */
 export declare function probeLiteralPathExists(filePath: string, cwd: string): Promise<"exists" | "missing" | "unknown">;
 /**

package/dist/types/tools/read.d.ts

@@ -28,6 +28,8 @@ export interface ReadToolDetails {
     meta?: OutputMeta;
     /** Full on-disk byte size recorded before applying a file range. */
     fileSize?: number;
+    /** Full source line count when the read reached EOF and the count is exact. */
+    totalLines?: number;
     /** Raw text + start line for user-visible TUI rendering, set when content is text-like.
      * Mirrors the same lines the model receives but without hashline/line-number prefixes,
      * so the TUI can render the file content with its own gutter without re-parsing the formatted text. */

package/dist/types/tools/shell-tokenize.d.ts

@@ -13,13 +13,31 @@
  */
 export declare function tokenizeShellSegments(command: string): string[][];
 /**
- * Returns the original text of flat shell command segments. Unlike
+ * A flat shell command segment with the context needed to decide interception.
+ *
+ * @see extractFlatShellCommandSegments
+ */
+export interface FlatShellCommandSegment {
+    /** Original segment text with quoting and escaping preserved. */
+    text: string;
+    /**
+     * True when this segment consumes the previous stage's stdout via an
+     * unquoted `|` or `|&`. Blank and comment-only continuation lines preserve
+     * the pending pipe state. Such a stage reads piped stdin, so path-based
+     * dedicated tools (read/grep/glob) cannot replace it. `||`, `;`, `&`, and
+     * `&&` start an independent command and leave this false.
+     */
+    pipedStdin: boolean;
+}
+/**
+ * Returns the flat shell command segments with the original text of each. Unlike
  * `tokenizeShellSegments`, this preserves quoting and escaping so the results
- * are safe to match against user-configured regular expressions.
+ * are safe to match against user-configured regular expressions, and flags
+ * segments that receive piped stdin.
  *
  * The extractor deliberately declines to split syntax whose execution context
  * cannot be determined with this small scanner (heredocs, command substitution,
  * backticks, grouping, and malformed quoting). Callers must still check the
  * complete input in that case.
  */
-export declare function extractFlatShellCommandSegments(command: string): string[];
+export declare function extractFlatShellCommandSegments(command: string): FlatShellCommandSegment[];

package/dist/types/utils/changelog.d.ts

@@ -39,9 +39,10 @@ export declare function formatStartupChangelogSummary(selection: StartupChangelo
 export declare function resolveBundledChangelogPath(assetPath: string, moduleUrl: string | URL): string | URL;
 export declare function parseChangelog(changelogPath: string | undefined): Promise<ChangelogEntry[]>;
 /**
- * Compare versions. Returns: -1 if v1 < v2, 0 if v1 === v2, 1 if v1 > v2
+ * Compare changelog entries by their parsed version parts.
+ * Returns: -1 if v1 < v2, 0 if v1 === v2, 1 if v1 > v2
  */
-export declare function compareVersions(v1: ChangelogEntry, v2: ChangelogEntry): number;
+export declare function compareChangelogEntries(v1: ChangelogEntry, v2: ChangelogEntry): number;
 /**
  * Parse an omp changelog marker version into comparable parts.
  */

package/dist/types/utils/late-cleanup.d.ts

@@ -0,0 +1,2 @@
+/** Keep timed-out cleanup reachable until its resources really settle. */
+export declare function trackLateCleanup(work: Promise<void>, context: Record<string, unknown>): void;

package/dist/types/vibe/runtime.d.ts

@@ -113,6 +113,8 @@ export declare class VibeSessionRegistry {
         ownerId: string;
         state?: VibeSessionState;
     }): void;
+    /** Override the teardown grace period for deterministic lifecycle tests. */
+    setTeardownGraceForTesting(timeoutMs: number): void;
     ownerScope(session: VibeParentSession): VibeOwnerScope;
     /** Re-open spawn admission after an explicit Vibe-mode entry. */
     activateScope(scope: VibeOwnerScope): void;

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "17.2.7",
+	"version": "17.2.9",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://omp.sh",
 	"author": "Can Boluk",
@@ -52,18 +52,18 @@
 		"@agentclientprotocol/sdk": "1.2.1",
 		"@babel/parser": "^7.29.7",
 		"@mozilla/readability": "^0.6.0",
-		"@oh-my-pi/hashline": "17.2.7",
-		"@oh-my-pi/omp-stats": "17.2.7",
-		"@oh-my-pi/omptype": "17.2.7",
-		"@oh-my-pi/pi-agent-core": "17.2.7",
-		"@oh-my-pi/pi-ai": "17.2.7",
-		"@oh-my-pi/pi-catalog": "17.2.7",
-		"@oh-my-pi/pi-mnemopi": "17.2.7",
-		"@oh-my-pi/pi-natives": "17.2.7",
-		"@oh-my-pi/pi-tui": "17.2.7",
-		"@oh-my-pi/pi-utils": "17.2.7",
-		"@oh-my-pi/pi-wire": "17.2.7",
-		"@oh-my-pi/snapcompact": "17.2.7",
+		"@oh-my-pi/hashline": "17.2.9",
+		"@oh-my-pi/omp-stats": "17.2.9",
+		"@oh-my-pi/omptype": "17.2.9",
+		"@oh-my-pi/pi-agent-core": "17.2.9",
+		"@oh-my-pi/pi-ai": "17.2.9",
+		"@oh-my-pi/pi-catalog": "17.2.9",
+		"@oh-my-pi/pi-mnemopi": "17.2.9",
+		"@oh-my-pi/pi-natives": "17.2.9",
+		"@oh-my-pi/pi-tui": "17.2.9",
+		"@oh-my-pi/pi-utils": "17.2.9",
+		"@oh-my-pi/pi-wire": "17.2.9",
+		"@oh-my-pi/snapcompact": "17.2.9",
 		"@opentelemetry/api": "^1.9.1",
 		"@opentelemetry/api-logs": "^0.220.0",
 		"@opentelemetry/context-async-hooks": "^2.9.0",

package/scripts/omp

@@ -36,7 +36,16 @@ mkdir -p "$launch_dir"
 OMP_LAUNCH_CWD=$PWD
 export OMP_LAUNCH_CWD
 cd "$launch_dir"
+# Herdr 0.7.5 validates the foreground process name. The macOS shell can
+# preserve OMP's identity while still running the development CLI through Bun.
+run_bun() {
+	if [ "${HERDR_ENV:-}" = 1 ] && [ "$(uname -s)" = Darwin ]; then
+		exec -a omp bun "$@"
+	fi
+	exec bun "$@"
+}
+
 if [ -n "${PI_TIMING:-}" ]; then
-	exec bun --preload "$preload" --preload "$timing_preload" "$cli" "$@"
+	run_bun --preload "$preload" --preload "$timing_preload" "$cli" "$@"
 fi
-exec bun --preload "$preload" "$cli" "$@"
+run_bun --preload "$preload" "$cli" "$@"

package/src/async/job-manager.ts

@@ -94,6 +94,12 @@ export interface AsyncJobDeliveryState {
 	pendingJobIds: string[];
 }
 
+export interface AsyncJobReapResult {
+	settled: boolean;
+	pendingJobIds: string[];
+	completion: Promise<void>;
+}
+
 export interface AsyncJobRegisterOptions {
 	id?: string;
 	/** Registry id of the agent that owns this job; used to scope cancelAll. */
@@ -490,6 +496,26 @@ export class AsyncJobManager {
 		}
 	}
 
+	/**
+	 * Cancel every job owned by `ownerId`, then wait only until `deadlineAt`.
+	 * The returned completion keeps waiting for actual process settlement when
+	 * the deadline expires, so callers can move that cleanup out of the
+	 * user-visible Task wait without losing ownership of the live work.
+	 */
+	async cancelAndReapOwnerJobs(ownerId: string, deadlineAt: number): Promise<AsyncJobReapResult> {
+		this.cancelAll({ ownerId });
+		const timeoutMs = Math.max(0, deadlineAt - Date.now());
+		const settled = await this.waitForOwnerJobs(ownerId, { timeoutMs });
+		if (settled) {
+			return { settled: true, pendingJobIds: [], completion: Promise.resolve() };
+		}
+		const pendingJobIds = this.getAllJobs({ ownerId })
+			.filter(job => job.status === "running" || job.status === "cancelled")
+			.map(job => job.id);
+		const completion = this.waitForOwnerJobs(ownerId).then(() => {});
+		return { settled: false, pendingJobIds, completion };
+	}
+
 	async #waitForAllUntil(deadline: number): Promise<boolean> {
 		const promises = Array.from(this.#jobs.values()).map(job => job.promise);
 		if (promises.length === 0) return true;

package/src/cli/stats-cli.ts

@@ -136,11 +136,11 @@ export async function runStatsCommand(cmd: StatsCommandArgs): Promise<void> {
 	}
 
 	// Start the dashboard server
-	const { port } = await startServer(cmd.port);
-	console.log(chalk.green(`Dashboard available at: http://localhost:${port}`));
+	const { hostname, port } = await startServer(cmd.port);
+	const url = `http://${hostname}:${port}`;
+	console.log(chalk.green(`Dashboard available at: ${url}`));
 
 	// Open browser
-	const url = `http://localhost:${port}`;
 	openPath(url);
 
 	console.log("Press Ctrl+C to stop\n");

package/src/cli/update-cli.ts

@@ -10,7 +10,7 @@ import * as os from "node:os";
 import * as path from "node:path";
 import { Transform } from "node:stream";
 import { pipeline } from "node:stream/promises";
-import { $env, $which, APP_NAME, isEnoent, VERSION } from "@oh-my-pi/pi-utils";
+import { $env, $which, APP_NAME, compareVersions, isEnoent, VERSION } from "@oh-my-pi/pi-utils";
 import { $ } from "bun";
 import chalk from "chalk";
 import { theme } from "../modes/theme/theme";
@@ -498,24 +498,6 @@ async function getLatestRelease(): Promise<ReleaseInfo> {
 	};
 }
 
-/**
- * Compare semver versions. Returns:
- * - negative if a < b
- * - 0 if a == b
- * - positive if a > b
- */
-function compareVersions(a: string, b: string): number {
-	const pa = a.split(".").map(Number);
-	const pb = b.split(".").map(Number);
-
-	for (let i = 0; i < Math.max(pa.length, pb.length); i++) {
-		const na = pa[i] || 0;
-		const nb = pb[i] || 0;
-		if (na !== nb) return na - nb;
-	}
-	return 0;
-}
-
 interface BunInstallCachePruneResult {
 	scannedPackages: number;
 	removedEntries: number;
@@ -532,42 +514,6 @@ function stripBunCacheVersionSuffix(name: string): string {
 	return metadataIndex === -1 ? name : name.slice(0, metadataIndex);
 }
 
-function compareSemverIdentifier(a: string, b: string): number {
-	const aNumber = /^\d+$/.test(a);
-	const bNumber = /^\d+$/.test(b);
-	if (aNumber && bNumber) return Number(a) - Number(b);
-	if (aNumber) return -1;
-	if (bNumber) return 1;
-	return a.localeCompare(b);
-}
-
-function compareSemverLikeVersions(a: string, b: string): number {
-	const [aCoreWithPrerelease] = a.split("+", 1);
-	const [bCoreWithPrerelease] = b.split("+", 1);
-	const [aCore, aPrerelease] = aCoreWithPrerelease.split("-", 2);
-	const [bCore, bPrerelease] = bCoreWithPrerelease.split("-", 2);
-	const aParts = aCore.split(".");
-	const bParts = bCore.split(".");
-	for (let i = 0; i < Math.max(aParts.length, bParts.length); i++) {
-		const diff = Number(aParts[i] ?? 0) - Number(bParts[i] ?? 0);
-		if (diff !== 0 && Number.isFinite(diff)) return diff;
-	}
-	if (!aPrerelease && !bPrerelease) return 0;
-	if (!aPrerelease) return 1;
-	if (!bPrerelease) return -1;
-	const aPrereleaseParts = aPrerelease.split(".");
-	const bPrereleaseParts = bPrerelease.split(".");
-	for (let i = 0; i < Math.max(aPrereleaseParts.length, bPrereleaseParts.length); i++) {
-		const aPart = aPrereleaseParts[i];
-		const bPart = bPrereleaseParts[i];
-		if (aPart === undefined) return -1;
-		if (bPart === undefined) return 1;
-		const diff = compareSemverIdentifier(aPart, bPart);
-		if (diff !== 0) return diff;
-	}
-	return 0;
-}
-
 async function readdirIfExists(dir: string): Promise<fs.Dirent[]> {
 	try {
 		return await fs.promises.readdir(dir, { withFileTypes: true });
@@ -689,7 +635,7 @@ export async function pruneBunInstallCache(
 		scannedPackages++;
 		let latestVersion: string | undefined;
 		for (const version of group.actualDirs.keys()) {
-			if (!latestVersion || compareSemverLikeVersions(version, latestVersion) > 0) latestVersion = version;
+			if (!latestVersion || compareVersions(version, latestVersion) > 0) latestVersion = version;
 		}
 		if (!latestVersion) continue;
 		for (const [version, paths] of group.actualDirs) {

package/src/config/config-file.ts

@@ -109,11 +109,11 @@ export class ConfigError extends Error {
 		this.#message = message;
 	}
 
-	get message(): string {
+	override get message(): string {
 		return this.#message;
 	}
 
-	toString(): string {
+	override toString(): string {
 		return this.message;
 	}
 }

package/src/config/keybindings.ts

@@ -607,12 +607,12 @@ export class KeybindingsManager extends TuiKeybindingsManager {
 		this.setUserBindings(mergeKeybindingsConfig(inheritedConfig, profileConfig));
 	}
 
-	setUserBindings(userBindings: KeybindingsConfig): void {
+	override setUserBindings(userBindings: KeybindingsConfig): void {
 		this.#userBindings = userBindings;
 		super.setUserBindings(userBindings);
 	}
 
-	getKeys(keybinding: Keybinding): KeyId[] {
+	override getKeys(keybinding: Keybinding): KeyId[] {
 		const keys = super.getKeys(keybinding);
 		const fallbackKey = getFallbackKey(keybinding);
 		if (fallbackKey === undefined || this.#userBindings[keybinding] !== undefined) return keys;
@@ -620,7 +620,7 @@ export class KeybindingsManager extends TuiKeybindingsManager {
 		return removeKey(keys, fallbackKey);
 	}
 
-	getResolvedBindings(): KeybindingsConfig {
+	override getResolvedBindings(): KeybindingsConfig {
 		const resolved = super.getResolvedBindings();
 		resolved[FOLLOW_UP_KEYBINDING] = keyConfigValue(this.getKeys(FOLLOW_UP_KEYBINDING));
 		return resolved;

package/src/config/model-discovery.ts

@@ -724,6 +724,31 @@ export async function discoverLlamaCppModelRuntimeMetadata(
 	}
 }
 
+/**
+ * Read image-input support from an OpenAI-compatible `/v1/models` row. Handles
+ * direct `input` arrays, Synthetic-style top-level `input_modalities`, and
+ * OpenRouter-style `architecture.input_modalities`; returns undefined when none
+ * is present so the bundled reference (or the `["text"]` default) can take over.
+ */
+function extractOpenAIModelsListInputCapabilities(item: {
+	input?: unknown;
+	input_modalities?: unknown;
+	architecture?: unknown;
+}): ("text" | "image")[] | undefined {
+	const modalities = new Set<string>();
+	const collect = (value: unknown): void => {
+		if (!Array.isArray(value)) return;
+		for (const entry of value) {
+			if (typeof entry === "string") modalities.add(entry.toLowerCase());
+		}
+	};
+	collect(item.input);
+	collect(item.input_modalities);
+	if (isRecord(item.architecture)) collect(item.architecture.input_modalities);
+	if (modalities.size === 0) return undefined;
+	return modalities.has("image") ? ["text", "image"] : ["text"];
+}
+
 export async function discoverOpenAIModelsList(
 	providerConfig: DiscoveryProviderConfig,
 	ctx: DiscoveryContext,
@@ -752,7 +777,14 @@ export async function discoverOpenAIModelsList(
 				}
 				headers = h;
 				return (await res.json()) as {
-					data?: Array<{ id?: string; max_model_len?: unknown; context_length?: unknown }>;
+					data?: Array<{
+						id?: string;
+						max_model_len?: unknown;
+						context_length?: unknown;
+						input?: unknown;
+						input_modalities?: unknown;
+						architecture?: unknown;
+					}>;
 				};
 			}),
 			nativeMetadataPromise,
@@ -796,7 +828,9 @@ export async function discoverOpenAIModelsList(
 				baseUrl,
 				reasoning: reference?.reasoning ?? false,
 				thinking: inheritReferenceThinking(undefined, reference, providerConfig.provider),
-				input: nativeMetadataForModel?.input ?? reference?.input ?? ["text"],
+				input: nativeMetadataForModel?.input ??
+					extractOpenAIModelsListInputCapabilities(item) ??
+					reference?.input ?? ["text"],
 				...(providerConfig.discovery.type === "lm-studio" ? { imageInputDecoder: "stb" as const } : {}),
 				// Proxy/gateway pricing is provider-specific and rarely matches
 				// upstream bundled catalogs, so keep costs local-unknown even

package/src/config/model-registry.ts

@@ -1715,7 +1715,10 @@ export class ModelRegistry {
 			return resolveOllamaModelCacheProviderId(providerConfig.provider, providerConfig.baseUrl);
 		}
 		if (providerConfig.discovery.type === "openai-models-list") {
-			return `${providerConfig.provider}:openai-models-list-context-v2`;
+			// context-v3 invalidates rows cached before server-advertised input
+			// modalities were parsed from `/v1/models`; warm v2 rows pinned
+			// vision-capable ids at `input: ["text"]` until a forced refresh.
+			return `${providerConfig.provider}:openai-models-list-context-v3`;
 		}
 		if (providerConfig.discovery.type === "litellm") {
 			// rich-v2 invalidates rows cached before reseller usage-suffix stripping