sapient/oh-my-openagent
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: use rest params in LSP sendNotification to avoid undefined serialization (#2185)

Browse Source 
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
This commit is contained in:
YeonGyu-Kim
2026-03-12 01:24:42 +09:00
parent 1c09b9869c
commit 08b411fc3b
1 changed files with 8 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/tools/lsp/lsp-client-transport.ts
View File

@@ -114,7 +114,9 @@ export class LSPClientTransport {
read()
}
protected async sendRequest<T>(method: string, params?: unknown): Promise<T> {
protected sendRequest<T>(method: string): Promise<T>
protected sendRequest<T>(method: string, params: unknown): Promise<T>
protected async sendRequest<T>(method: string, ...args: [] | [unknown]): Promise<T> {
if (!this.connection) throw new Error("LSP client not started")
if (this.processExited || (this.proc && this.proc.exitCode !== null)) {
@@ -130,7 +132,7 @@ export class LSPClientTransport {
}, this.REQUEST_TIMEOUT)
})
const requestPromise = this.connection.sendRequest(method, params) as Promise<T>
const requestPromise = this.connection.sendRequest(method, ...args) as Promise<T>
try {
const result = await Promise.race([requestPromise, timeoutPromise])
@@ -142,10 +144,12 @@ export class LSPClientTransport {
}
}
protected sendNotification(method: string, params?: unknown): void {
protected sendNotification(method: string): void
protected sendNotification(method: string, params: unknown): void
protected sendNotification(method: string, ...args: [] | [unknown]): void {
if (!this.connection) return
if (this.processExited || (this.proc && this.proc.exitCode !== null)) return
this.connection.sendNotification(method, params)
this.connection.sendNotification(method, ...args)
}
isAlive(): boolean {