oh-my-opencode 4.18.0 → 4.18.2

package/packages/lsp-core/src/tools/navigation.ts

@@ -15,12 +15,12 @@ export async function executeLspGotoDefinition(
 	const character = requireNumber(params, "character");
 
 	try {
-		const result = await withLspClient(
-			filePath,
-			async (client) => client.definition(filePath, line, character),
-			"definition",
-			clientOptions(signal),
-		);
+			const result = await withLspClient(
+				filePath,
+				async (client) => client.definition(filePath, line, character, signal),
+				"definition",
+				clientOptions(signal),
+			);
 		const locations = !result ? [] : Array.isArray(result) ? result : [result];
 		const details: LspGotoDefinitionDetails = { filePath, line, character, locations };
 		if (locations.length === 0) return text("No definition found", details);
@@ -47,12 +47,12 @@ export async function executeLspFindReferences(
 	const includeDeclaration = optionalBoolean(params, "includeDeclaration") ?? true;
 
 	try {
-		const result = await withLspClient(
-			filePath,
-			async (client) => client.references(filePath, line, character, includeDeclaration),
-			"references",
-			clientOptions(signal),
-		);
+			const result = await withLspClient(
+				filePath,
+				async (client) => client.references(filePath, line, character, includeDeclaration, signal),
+				"references",
+				clientOptions(signal),
+			);
 		const references = Array.isArray(result) ? result : [];
 		const total = references.length;
 		const truncated = total > DEFAULT_MAX_REFERENCES;

package/packages/lsp-core/src/tools/rename.ts

@@ -1,6 +1,5 @@
 import { formatApplyResult, formatPrepareRenameResult } from "../lsp/formatters.js";
 import { withLspClient } from "../lsp/client-wrapper.js";
-import { applyWorkspaceEdit } from "../lsp/workspace-edit.js";
 import { missingDependencyResult } from "../missing-dependency-result.js";
 import { clientOptions, requireNumber, requireString } from "./parameters.js";
 import { text } from "./result.js";
@@ -15,12 +14,12 @@ export async function executeLspPrepareRename(
 	const character = requireNumber(params, "character");
 
 	try {
-		const result = await withLspClient(
-			filePath,
-			async (client) => client.prepareRename(filePath, line, character),
-			"prepareRename",
-			clientOptions(signal),
-		);
+			const result = await withLspClient(
+				filePath,
+				async (client) => client.prepareRename(filePath, line, character, signal),
+				"prepareRename",
+				clientOptions(signal),
+			);
 		const details: LspPrepareRenameDetails = { filePath, line, character, result };
 		return text(formatPrepareRenameResult(result), details);
 	} catch (error) {
@@ -45,18 +44,14 @@ export async function executeLspRename(
 	const newName = requireString(params, "newName");
 
 	try {
-		const edit = await withLspClient(
+		const result = await withLspClient(
 			filePath,
-			async (client, workspaceRoot) => ({
-				edit: await client.rename(filePath, line, character, newName),
-				workspaceRoot,
-			}),
+			async (client) => client.rename(filePath, line, character, newName, signal),
 			"rename",
 			clientOptions(signal),
 		);
-		const apply = applyWorkspaceEdit(edit.edit, { workspaceRoot: edit.workspaceRoot });
-		const details: LspRenameDetails = { filePath, line, character, newName, apply, edit: edit.edit };
-		return text(formatApplyResult(apply), details, !apply.success);
+		const details: LspRenameDetails = { filePath, line, character, newName, apply: result.apply, edit: result.edit };
+		return text(formatApplyResult(result.apply), details, !result.apply.success);
 	} catch (error) {
 		const missingDependency = missingDependencyResult(error, {
 			filePath,

package/packages/lsp-core/src/tools/symbols.ts

@@ -36,21 +36,21 @@ export async function executeLspSymbols(
 				});
 			}
 
+				const symbols = await withLspClient(
+					filePath,
+					async (client) => client.workspaceSymbols(query, signal),
+					"workspaceSymbols",
+					clientOptions(signal),
+				);
+			return formatSymbolsResult(filePath, scope, symbols, limit, query);
+		}
+
 			const symbols = await withLspClient(
 				filePath,
-				async (client) => client.workspaceSymbols(query),
-				"workspaceSymbols",
+				async (client) => client.documentSymbols(filePath, signal),
+				"documentSymbols",
 				clientOptions(signal),
 			);
-			return formatSymbolsResult(filePath, scope, symbols, limit, query);
-		}
-
-		const symbols = await withLspClient(
-			filePath,
-			async (client) => client.documentSymbols(filePath),
-			"documentSymbols",
-			clientOptions(signal),
-		);
 		return formatSymbolsResult(filePath, scope, symbols, limit);
 	} catch (error) {
 		const query = optionalString(params, "query");

package/packages/lsp-core/src/tools/types.ts

@@ -49,7 +49,8 @@ export interface LspDiagnosticsDetails {
 	totalDiagnostics: number;
 	truncated: boolean;
 	error?: string;
-	errorKind?: "missing_dependency" | "no_files" | "invalid_path";
+	errorKind?: "freshness_timeout" | "missing_dependency" | "no_files" | "invalid_path";
+	fileFailures?: Array<{ file: string; error: string }>;
 }
 
 export interface LspGotoDefinitionDetails {