@oh-my-pi/pi-coding-agent 13.6.1 → 13.6.2

package/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 ## [Unreleased]
 
+## [13.6.2] - 2026-03-03
+### Fixed
+
+- Fixed LM Studio API key retrieval to use configured provider name instead of hardcoded 'lm-studio'
+- Fixed resource content handling to properly check for empty text values (null/undefined)
+- Fixed resource refresh tracking to prevent stale promise reuse when server connection changes
+- Fixed update target resolution to properly handle cases where binary path cannot be resolved
+
 ## [13.6.1] - 2026-03-03
 
 ### Fixed

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "13.6.1",
+	"version": "13.6.2",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/can1357/oh-my-pi",
 	"author": "Can Boluk",
@@ -41,12 +41,12 @@
 	},
 	"dependencies": {
 		"@mozilla/readability": "^0.6",
-		"@oh-my-pi/omp-stats": "13.6.1",
-		"@oh-my-pi/pi-agent-core": "13.6.1",
-		"@oh-my-pi/pi-ai": "13.6.1",
-		"@oh-my-pi/pi-natives": "13.6.1",
-		"@oh-my-pi/pi-tui": "13.6.1",
-		"@oh-my-pi/pi-utils": "13.6.1",
+		"@oh-my-pi/omp-stats": "13.6.2",
+		"@oh-my-pi/pi-agent-core": "13.6.2",
+		"@oh-my-pi/pi-ai": "13.6.2",
+		"@oh-my-pi/pi-natives": "13.6.2",
+		"@oh-my-pi/pi-tui": "13.6.2",
+		"@oh-my-pi/pi-utils": "13.6.2",
 		"@sinclair/typebox": "^0.34",
 		"@xterm/headless": "^6.0",
 		"ajv": "^8.18",

package/src/cli/update-cli.ts

@@ -47,14 +47,6 @@ async function getBunGlobalBinDir(): Promise<string | undefined> {
 	}
 }
 
-function getRealPathOrOriginal(filePath: string): string {
-	try {
-		return fs.realpathSync(filePath);
-	} catch {
-		return filePath;
-	}
-}
-
 function normalizePathForComparison(filePath: string): string {
 	const normalized = path.normalize(filePath);
 	if (process.platform === "win32") return normalized.toLowerCase();
@@ -62,16 +54,13 @@ function normalizePathForComparison(filePath: string): string {
 }
 
 function isPathInDirectory(filePath: string, directoryPath: string): boolean {
-	const normalizedPath = normalizePathForComparison(getRealPathOrOriginal(filePath));
-	const normalizedDirectory = normalizePathForComparison(getRealPathOrOriginal(directoryPath));
+	const normalizedPath = normalizePathForComparison(path.resolve(filePath));
+	const normalizedDirectory = normalizePathForComparison(path.resolve(directoryPath));
 	const relativePath = path.relative(normalizedDirectory, normalizedPath);
 	return relativePath === "" || (!relativePath.startsWith("..") && !path.isAbsolute(relativePath));
 }
 
-interface UpdateTarget {
-	method: "bun" | "binary";
-	path: string;
-}
+type UpdateTarget = { method: "bun" } | { method: "binary"; path: string };
 
 function resolveUpdateMethod(ompPath: string, bunBinDir: string | undefined): "bun" | "binary" {
 	if (!bunBinDir) return "binary";
@@ -82,10 +71,18 @@ export function _resolveUpdateMethodForTest(ompPath: string, bunBinDir: string |
 	return resolveUpdateMethod(ompPath, bunBinDir);
 }
 async function resolveUpdateTarget(): Promise<UpdateTarget> {
-	const ompPath = resolveOmpPath() ?? process.execPath;
 	const bunBinDir = await getBunGlobalBinDir();
-	const method = resolveUpdateMethod(ompPath, bunBinDir);
-	return { method, path: ompPath };
+	const ompPath = resolveOmpPath();
+
+	if (ompPath) {
+		const method = resolveUpdateMethod(ompPath, bunBinDir);
+		if (method === "bun") return { method };
+		return { method, path: ompPath };
+	}
+
+	if (bunBinDir) return { method: "bun" };
+
+	throw new Error(`Could not resolve ${APP_NAME} binary path in PATH`);
 }
 
 /**

package/src/config/model-registry.ts

@@ -883,7 +883,7 @@ export class ModelRegistry {
 		const modelsUrl = `${baseUrl}/models`;
 
 		const headers: Record<string, string> = { ...(providerConfig.headers ?? {}) };
-		const apiKey = await this.authStorage.getApiKey("lm-studio");
+		const apiKey = await this.authStorage.getApiKey(providerConfig.provider);
 		if (apiKey && apiKey !== DEFAULT_LOCAL_TOKEN && apiKey !== kNoAuth) {
 			headers.Authorization = `Bearer ${apiKey}`;
 		}

package/src/internal-urls/mcp-protocol.ts

@@ -137,14 +137,14 @@ export class McpProtocolHandler implements ProtocolHandler {
 
 		const textParts: string[] = [];
 		for (const item of result.contents) {
-			if (item.text) {
+			if (item.text !== undefined && item.text !== null) {
 				textParts.push(item.text);
 			} else if (item.blob) {
 				textParts.push(`[Binary content: ${item.mimeType ?? "unknown"}, base64 length ${item.blob.length}]`);
 			}
 		}
 
-		const content = textParts.join("\n---\n") || "(empty resource)";
+		const content = textParts.length > 0 ? textParts.join("\n---\n") : "(empty resource)";
 		return {
 			url: url.href,
 			content,

package/src/mcp/manager.ts

@@ -126,7 +126,7 @@ export class MCPManager {
 	#notificationsEnabled = false;
 	#notificationsEpoch = 0;
 	#subscribedResources = new Map<string, Set<string>>();
-	#pendingResourceRefresh = new Map<string, Promise<void>>();
+	#pendingResourceRefresh = new Map<string, { connection: MCPServerConnection; promise: Promise<void> }>();
 
 	constructor(
 		private cwd: string,
@@ -590,6 +590,7 @@ export class MCPManager {
 		this.#pendingConnections.delete(name);
 		this.#pendingToolLoads.delete(name);
 		this.#sources.delete(name);
+		this.#pendingResourceRefresh.delete(name);
 
 		const connection = this.#connections.get(name);
 
@@ -622,6 +623,7 @@ export class MCPManager {
 
 		this.#pendingConnections.clear();
 		this.#pendingToolLoads.clear();
+		this.#pendingResourceRefresh.clear();
 		this.#sources.clear();
 		this.#connections.clear();
 		this.#tools = [];
@@ -660,13 +662,13 @@ export class MCPManager {
 	 * Refresh resources from a specific server.
 	 */
 	async refreshServerResources(name: string): Promise<void> {
+		const connection = this.#connections.get(name);
+		if (!connection || !serverSupportsResources(connection.capabilities)) return;
+
 		const existing = this.#pendingResourceRefresh.get(name);
-		if (existing) return existing;
+		if (existing && existing.connection === connection) return existing.promise;
 
 		const doRefresh = async (): Promise<void> => {
-			const connection = this.#connections.get(name);
-			if (!connection || !serverSupportsResources(connection.capabilities)) return;
-
 			// Clear cached resources
 			connection.resources = undefined;
 			connection.resourceTemplates = undefined;
@@ -716,11 +718,12 @@ export class MCPManager {
 		};
 
 		const promise = doRefresh().finally(() => {
-			if (this.#pendingResourceRefresh.get(name) === promise) {
+			const pending = this.#pendingResourceRefresh.get(name);
+			if (pending?.promise === promise) {
 				this.#pendingResourceRefresh.delete(name);
 			}
 		});
-		this.#pendingResourceRefresh.set(name, promise);
+		this.#pendingResourceRefresh.set(name, { connection, promise });
 		return promise;
 	}
 

package/src/mcp/smithery-registry.ts

@@ -1,3 +1,4 @@
+import { logger } from "@oh-my-pi/pi-utils";
 import type { MCPServerConfig } from "./types";
 
 const SMITHERY_REGISTRY_BASE_URL = "https://registry.smithery.ai";
@@ -324,8 +325,12 @@ async function fetchServerDetailsFromEntry(
 ): Promise<SmitheryServerDetails | null> {
 	const candidates = resolveDetailPathCandidates(entry);
 	for (const candidate of candidates) {
-		const details = await fetchServerDetails(candidate, options);
-		if (details) return details;
+		try {
+			const details = await fetchServerDetails(candidate, options);
+			if (details) return details;
+		} catch (error) {
+			logger.debug("Smithery detail fetch candidate failed", { candidate, error: String(error) });
+		}
 	}
 	return null;
 }
@@ -439,14 +444,31 @@ export async function searchSmitheryRegistry(
 		);
 	});
 
+	const detailFailures: Array<{ identity: string; error: string }> = [];
 	const results = await Promise.all(
 		uniqueEntries.map(async entry => {
-			const details = await fetchServerDetailsFromEntry(entry, { apiKey: options?.apiKey });
-			if (!details) return null;
-			return toSearchResult(entry, details);
+			try {
+				const details = await fetchServerDetailsFromEntry(entry, { apiKey: options?.apiKey });
+				if (!details) return null;
+				return toSearchResult(entry, details);
+			} catch (error) {
+				detailFailures.push({
+					identity: getEntryIdentityKey(entry) ?? entry.id ?? "unknown",
+					error: String(error),
+				});
+				return null;
+			}
 		}),
 	);
 
+	if (detailFailures.length > 0) {
+		logger.warn("Smithery detail fetch failed for some entries", {
+			query,
+			failedEntries: detailFailures.length,
+			totalEntries: uniqueEntries.length,
+			sample: detailFailures.slice(0, 3),
+		});
+	}
 	return results.filter((result): result is SmitherySearchResult => result !== null).slice(0, limit);
 }