diff options
Diffstat (limited to 'packages/core/src/ide')
| -rw-r--r-- | packages/core/src/ide/ide-client.ts | 106 | ||||
| -rw-r--r-- | packages/core/src/ide/ideContext.ts | 59 |
2 files changed, 164 insertions, 1 deletions
diff --git a/packages/core/src/ide/ide-client.ts b/packages/core/src/ide/ide-client.ts index 8f967147..42b79c44 100644 --- a/packages/core/src/ide/ide-client.ts +++ b/packages/core/src/ide/ide-client.ts @@ -9,7 +9,14 @@ import { DetectedIde, getIdeDisplayName, } from '../ide/detect-ide.js'; -import { ideContext, IdeContextNotificationSchema } from '../ide/ideContext.js'; +import { + ideContext, + IdeContextNotificationSchema, + IdeDiffAcceptedNotificationSchema, + IdeDiffClosedNotificationSchema, + CloseDiffResponseSchema, + DiffUpdateResult, +} from '../ide/ideContext.js'; import { Client } from '@modelcontextprotocol/sdk/client/index.js'; import { StreamableHTTPClientTransport } from '@modelcontextprotocol/sdk/client/streamableHttp.js'; @@ -42,6 +49,7 @@ export class IdeClient { }; private readonly currentIde: DetectedIde | undefined; private readonly currentIdeDisplayName: string | undefined; + private diffResponses = new Map<string, (result: DiffUpdateResult) => void>(); private constructor() { this.currentIde = detectIde(); @@ -77,6 +85,75 @@ export class IdeClient { await this.establishConnection(port); } + /** + * A diff is accepted with any modifications if the user performs one of the + * following actions: + * - Clicks the checkbox icon in the IDE to accept + * - Runs `command+shift+p` > "Gemini CLI: Accept Diff in IDE" to accept + * - Selects "accept" in the CLI UI + * - Saves the file via `ctrl/command+s` + * + * A diff is rejected if the user performs one of the following actions: + * - Clicks the "x" icon in the IDE + * - Runs "Gemini CLI: Close Diff in IDE" + * - Selects "no" in the CLI UI + * - Closes the file + */ + async openDiff( + filePath: string, + newContent?: string, + ): Promise<DiffUpdateResult> { + return new Promise<DiffUpdateResult>((resolve, reject) => { + this.diffResponses.set(filePath, resolve); + this.client + ?.callTool({ + name: `openDiff`, + arguments: { + filePath, + newContent, + }, + }) + .catch((err) => { + logger.debug(`callTool for ${filePath} failed:`, err); + reject(err); + }); + }); + } + + async closeDiff(filePath: string): Promise<string | undefined> { + try { + const result = await this.client?.callTool({ + name: `closeDiff`, + arguments: { + filePath, + }, + }); + + if (result) { + const parsed = CloseDiffResponseSchema.parse(result); + return parsed.content; + } + } catch (err) { + logger.debug(`callTool for ${filePath} failed:`, err); + } + return; + } + + // Closes the diff. Instead of waiting for a notification, + // manually resolves the diff resolver as the desired outcome. + async resolveDiffFromCli(filePath: string, outcome: 'accepted' | 'rejected') { + const content = await this.closeDiff(filePath); + const resolver = this.diffResponses.get(filePath); + if (resolver) { + if (outcome === 'accepted') { + resolver({ status: 'accepted', content }); + } else { + resolver({ status: 'rejected', content: undefined }); + } + this.diffResponses.delete(filePath); + } + } + disconnect() { this.setState( IDEConnectionStatus.Disconnected, @@ -175,6 +252,33 @@ export class IdeClient { `IDE connection error. The connection was lost unexpectedly. Please try reconnecting by running /ide enable`, ); }; + this.client.setNotificationHandler( + IdeDiffAcceptedNotificationSchema, + (notification) => { + const { filePath, content } = notification.params; + const resolver = this.diffResponses.get(filePath); + if (resolver) { + resolver({ status: 'accepted', content }); + this.diffResponses.delete(filePath); + } else { + logger.debug(`No resolver found for ${filePath}`); + } + }, + ); + + this.client.setNotificationHandler( + IdeDiffClosedNotificationSchema, + (notification) => { + const { filePath } = notification.params; + const resolver = this.diffResponses.get(filePath); + if (resolver) { + resolver({ status: 'rejected', content: undefined }); + this.diffResponses.delete(filePath); + } else { + logger.debug(`No resolver found for ${filePath}`); + } + }, + ); } private async establishConnection(port: string) { diff --git a/packages/core/src/ide/ideContext.ts b/packages/core/src/ide/ideContext.ts index 588e25ee..3052c029 100644 --- a/packages/core/src/ide/ideContext.ts +++ b/packages/core/src/ide/ideContext.ts @@ -36,10 +36,69 @@ export type IdeContext = z.infer<typeof IdeContextSchema>; * Zod schema for validating the 'ide/contextUpdate' notification from the IDE. */ export const IdeContextNotificationSchema = z.object({ + jsonrpc: z.literal('2.0'), method: z.literal('ide/contextUpdate'), params: IdeContextSchema, }); +export const IdeDiffAcceptedNotificationSchema = z.object({ + jsonrpc: z.literal('2.0'), + method: z.literal('ide/diffAccepted'), + params: z.object({ + filePath: z.string(), + content: z.string(), + }), +}); + +export const IdeDiffClosedNotificationSchema = z.object({ + jsonrpc: z.literal('2.0'), + method: z.literal('ide/diffClosed'), + params: z.object({ + filePath: z.string(), + content: z.string().optional(), + }), +}); + +export const CloseDiffResponseSchema = z + .object({ + content: z + .array( + z.object({ + text: z.string(), + type: z.literal('text'), + }), + ) + .min(1), + }) + .transform((val, ctx) => { + try { + const parsed = JSON.parse(val.content[0].text); + const innerSchema = z.object({ content: z.string().optional() }); + const validationResult = innerSchema.safeParse(parsed); + if (!validationResult.success) { + validationResult.error.issues.forEach((issue) => ctx.addIssue(issue)); + return z.NEVER; + } + return validationResult.data; + } catch (_) { + ctx.addIssue({ + code: z.ZodIssueCode.custom, + message: 'Invalid JSON in text content', + }); + return z.NEVER; + } + }); + +export type DiffUpdateResult = + | { + status: 'accepted'; + content?: string; + } + | { + status: 'rejected'; + content: undefined; + }; + type IdeContextSubscriber = (ideContext: IdeContext | undefined) => void; /** |
