@oh-my-pi/pi-coding-agent 11.14.4 → 12.0.0

package/CHANGELOG.md

@@ -2,6 +2,23 @@
 
 ## [Unreleased]
 
+## [12.0.0] - 2026-02-12
+
+### Added
+
+- Added `getAllServerNames()` method to MCPManager for enumerating all known servers
+
+### Changed
+
+- Changed default edit mode from `patch` to `hashline` for more precise code modifications
+- Changed `readHashLines` setting default from false to true to enable hash line reading by default
+
+### Fixed
+
+- Fixed `omp setup` crashing with uncaught exception when no component argument provided; now shows help ([#35](https://github.com/can1357/oh-my-pi/issues/35))
+- Fixed `/mcp list` showing "No MCP servers configured" when servers are loaded from discovery sources like `.claude.json`, `.cursor/mcp.json`, `.vscode/mcp.json` ([#34](https://github.com/can1357/oh-my-pi/issues/34))
+- Fixed model selector sorting to show newest models first within each provider instead of alphabetical; `-latest` aliases now appear before dated versions ([#37](https://github.com/can1357/oh-my-pi/issues/37))
+
 ## [11.14.4] - 2026-02-12
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "11.14.4",
+	"version": "12.0.0",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"ompConfig": {
@@ -88,12 +88,12 @@
 	},
 	"dependencies": {
 		"@mozilla/readability": "0.6.0",
-		"@oh-my-pi/omp-stats": "11.14.4",
-		"@oh-my-pi/pi-agent-core": "11.14.4",
-		"@oh-my-pi/pi-ai": "11.14.4",
-		"@oh-my-pi/pi-natives": "11.14.4",
-		"@oh-my-pi/pi-tui": "11.14.4",
-		"@oh-my-pi/pi-utils": "11.14.4",
+		"@oh-my-pi/omp-stats": "12.0.0",
+		"@oh-my-pi/pi-agent-core": "12.0.0",
+		"@oh-my-pi/pi-ai": "12.0.0",
+		"@oh-my-pi/pi-natives": "12.0.0",
+		"@oh-my-pi/pi-tui": "12.0.0",
+		"@oh-my-pi/pi-utils": "12.0.0",
 		"@sinclair/typebox": "^0.34.48",
 		"@xterm/headless": "^6.0.0",
 		"ajv": "^8.17.1",

package/src/cli.ts

@@ -6,6 +6,12 @@
 import { type CommandEntry, run } from "@oh-my-pi/pi-utils/cli";
 import { APP_NAME, VERSION } from "./config";
 
+// Detect known Bun errata that cause TUI crashes (e.g. Bun.stringWidth mishandling OSC sequences).
+if (Bun.stringWidth("\x1b[0m\x1b]8;;\x07") !== 0) {
+	process.stderr.write(`error: Bun runtime errata detected (v${Bun.version}). Please update Bun: bun upgrade\n`);
+	process.exit(1);
+}
+
 process.title = APP_NAME;
 
 const commands: CommandEntry[] = [

package/src/commands/setup.ts

@@ -1,7 +1,7 @@
 /**
  * Install dependencies for optional features.
  */
-import { Args, Command, Flags } from "@oh-my-pi/pi-utils/cli";
+import { Args, Command, Flags, renderCommandHelp } from "@oh-my-pi/pi-utils/cli";
 import { runSetupCommand, type SetupCommandArgs, type SetupComponent } from "../cli/setup-cli";
 import { initTheme } from "../modes/theme/theme";
 
@@ -13,7 +13,7 @@ export default class Setup extends Command {
 	static args = {
 		component: Args.string({
 			description: "Component to install",
-			required: true,
+			required: false,
 			options: COMPONENTS,
 		}),
 	};
@@ -25,6 +25,10 @@ export default class Setup extends Command {
 
 	async run(): Promise<void> {
 		const { args, flags } = await this.parse(Setup);
+		if (!args.component) {
+			renderCommandHelp("omp", "setup", Setup);
+			return;
+		}
 		const cmd: SetupCommandArgs = {
 			component: args.component as SetupComponent,
 			flags: {
@@ -32,7 +36,6 @@ export default class Setup extends Command {
 				check: flags.check,
 			},
 		};
-
 		await initTheme();
 		await runSetupCommand(cmd);
 	}

package/src/config/settings-schema.ts

@@ -249,7 +249,7 @@ export const SETTINGS_SCHEMA = {
 	},
 	readHashLines: {
 		type: "boolean",
-		default: false,
+		default: true,
 		ui: {
 			tab: "config",
 			label: "Read hash lines",
@@ -743,7 +743,7 @@ export const SETTINGS_SCHEMA = {
 	"edit.mode": {
 		type: "enum",
 		values: ["replace", "patch", "hashline"] as const,
-		default: "patch",
+		default: "hashline",
 		ui: {
 			tab: "config",
 			label: "Edit mode",

package/src/mcp/manager.ts

@@ -343,6 +343,15 @@ export class MCPManager {
 		return Array.from(this.#connections.keys());
 	}
 
+	/**
+	 * Get all known server names (connected, connecting, or discovered).
+	 */
+	getAllServerNames(): string[] {
+		return Array.from(
+			new Set([...this.#sources.keys(), ...this.#connections.keys(), ...this.#pendingConnections.keys()]),
+		);
+	}
+
 	/**
 	 * Disconnect from a specific server.
 	 */

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

@@ -203,6 +203,9 @@ export class ModelSelectorComponent extends Container {
 			return i;
 		};
 
+		const dateRe = /-(\d{8})$/;
+		const latestRe = /-latest$/;
+
 		models.sort((a, b) => {
 			const aKey = `${a.provider}/${a.id}`;
 			const bKey = `${b.provider}/${b.id}`;
@@ -216,10 +219,33 @@ export class ModelSelectorComponent extends Container {
 			const bMru = mruIndex.get(bKey) ?? Number.MAX_SAFE_INTEGER;
 			if (aMru !== bMru) return aMru - bMru;
 
-			// Finally alphabetical by provider, then id
+			// By provider, then recency within provider
 			const providerCmp = a.provider.localeCompare(b.provider);
 			if (providerCmp !== 0) return providerCmp;
-			return a.id.localeCompare(b.id);
+
+			const aIsLatest = latestRe.test(a.id);
+			const bIsLatest = latestRe.test(b.id);
+			const aDate = a.id.match(dateRe)?.[1] ?? "";
+			const bDate = b.id.match(dateRe)?.[1] ?? "";
+
+			// Both have dates or latest tags — sort by recency
+			const aHasRecency = aIsLatest || aDate !== "";
+			const bHasRecency = bIsLatest || bDate !== "";
+
+			// Models with recency info come before those without
+			if (aHasRecency !== bHasRecency) return aHasRecency ? -1 : 1;
+
+			// If neither has recency info, fall back to alphabetical
+			if (!aHasRecency) return a.id.localeCompare(b.id);
+
+			// -latest always sorts first within recency group
+			if (aIsLatest !== bIsLatest) return aIsLatest ? -1 : 1;
+
+			// Both have dates — descending (newest first)
+			if (aDate && bDate) return bDate.localeCompare(aDate);
+
+			// One has date, other is latest — latest first
+			return aIsLatest ? -1 : bIsLatest ? 1 : a.id.localeCompare(b.id);
 		});
 	}
 

package/src/modes/controllers/mcp-command-controller.ts

@@ -4,6 +4,7 @@
  * Handles /mcp subcommands for managing MCP servers.
  */
 import { Spacer, Text } from "@oh-my-pi/pi-tui";
+import type { SourceMeta } from "../../capability/types";
 import { analyzeAuthError, discoverOAuthEndpoints, MCPManager } from "../../mcp";
 import { connectToServer, disconnectServer, listTools } from "../../mcp/client";
 import {
@@ -765,7 +766,20 @@ export class MCPCommandController {
 			const userServers = Object.keys(userConfig.mcpServers ?? {});
 			const projectServers = Object.keys(projectConfig.mcpServers ?? {});
 
-			if (userServers.length === 0 && projectServers.length === 0) {
+			// Collect runtime-discovered servers not in config files
+			const configServerNames = new Set([...userServers, ...projectServers]);
+			const discoveredServers: { name: string; source: SourceMeta }[] = [];
+			if (this.ctx.mcpManager) {
+				for (const name of this.ctx.mcpManager.getAllServerNames()) {
+					if (configServerNames.has(name)) continue;
+					const source = this.ctx.mcpManager.getSource(name);
+					if (source) {
+						discoveredServers.push({ name, source });
+					}
+				}
+			}
+
+			if (userServers.length === 0 && projectServers.length === 0 && discoveredServers.length === 0) {
 				this.#showMessage(
 					[
 						"",
@@ -826,6 +840,39 @@ export class MCPCommandController {
 				lines.push("");
 			}
 
+			// Show discovered servers (from .claude.json, .cursor/mcp.json, .vscode/mcp.json, etc.)
+			if (discoveredServers.length > 0) {
+				// Group by source display name + path
+				const bySource = new Map<string, typeof discoveredServers>();
+				for (const entry of discoveredServers) {
+					const key = `${entry.source.providerName}|${entry.source.path}`;
+					let group = bySource.get(key);
+					if (!group) {
+						group = [];
+						bySource.set(key, group);
+					}
+					group.push(entry);
+				}
+
+				for (const [key, entries] of bySource) {
+					const sepIdx = key.indexOf("|");
+					const providerName = key.slice(0, sepIdx);
+					const sourcePath = key.slice(sepIdx + 1);
+					const shortPath = sourcePath.replace(process.env.HOME ?? "", "~");
+					lines.push(theme.fg("accent", providerName) + theme.fg("muted", ` (${shortPath}):`));
+					for (const { name } of entries) {
+						const state = this.ctx.mcpManager!.getConnectionStatus(name);
+						const status =
+							state === "connected"
+								? theme.fg("success", " ● connected")
+								: state === "connecting"
+									? theme.fg("muted", " ◌ connecting")
+									: theme.fg("muted", " ○ not connected");
+						lines.push(`  ${theme.fg("accent", name)}${status}`);
+					}
+					lines.push("");
+				}
+			}
 			this.#showMessage(lines.join("\n"));
 		} catch (error) {
 			this.ctx.showError(`Failed to list servers: ${error instanceof Error ? error.message : String(error)}`);