summaryrefslogtreecommitdiff
path: root/packages/cli/src/tools/grep.tool.ts
diff options
context:
space:
mode:
authorTaylor Mullen <[email protected]>2025-04-21 10:53:11 -0400
committerN. Taylor Mullen <[email protected]>2025-04-21 11:07:09 -0400
commit81f0f618f7ecc439e67447bf98065d87e22483c0 (patch)
tree82851f180c8ba5263427e3586bc738590fc49153 /packages/cli/src/tools/grep.tool.ts
parente351baf10f06d2a1d1872bf2a6d7e9e709effed9 (diff)
Fix Gemini Code's (GC) smarts.
- The tl;dr; is that GC couldn't see what the user was saying when tool call events happened in response. The rason why this was happening was because we were instantly invoking tools that the model told us to invoke and then instantly re-requesting. This resulted in the bug because the genai APIs can't update the chat history before a full response has been completed (doesn't know how to update if it's incomplete). - To address the above issue I had to do quite the large refactor. The gist is that now turns truly drive everything on the server (vs. a server client split). This ensured that when we got tool invocations we could control when/how re-requesting would happen and then also ensure that history was updated. This change also meant that the server would act as an event publisher to enable the client to react to events rather than try and weave in complex logic between the events. - A BIG change that this changeset incudes is the removal of all of the CLI tools in favor of the server tools. - Removed some dead code as part of this - **NOTE: Confirmations are still broken (they were broken prior to this); however, I've set them up to be able to work in the future, I'll dot hat in a follow up to be less breaking to others.** Fixes https://b.corp.google.com/issues/412320087
Diffstat (limited to 'packages/cli/src/tools/grep.tool.ts')
-rw-r--r--packages/cli/src/tools/grep.tool.ts79
1 files changed, 0 insertions, 79 deletions
diff --git a/packages/cli/src/tools/grep.tool.ts b/packages/cli/src/tools/grep.tool.ts
deleted file mode 100644
index 50cff362..00000000
--- a/packages/cli/src/tools/grep.tool.ts
+++ /dev/null
@@ -1,79 +0,0 @@
-/**
- * @license
- * Copyright 2025 Google LLC
- * SPDX-License-Identifier: Apache-2.0
- */
-
-// Import core logic and types from the server package
-import { GrepLogic, GrepToolParams, ToolResult } from '@gemini-code/server';
-
-// Import CLI-specific base class and types
-import { BaseTool } from './tools.js';
-import { ToolCallConfirmationDetails } from '../ui/types.js';
-
-// --- Interfaces (Params defined in server package) ---
-
-// --- GrepTool CLI Wrapper Class ---
-
-/**
- * CLI wrapper for the Grep tool
- */
-export class GrepTool extends BaseTool<GrepToolParams, ToolResult> {
- static readonly Name: string = GrepLogic.Name; // Use name from logic
-
- // Core logic instance from the server package
- private coreLogic: GrepLogic;
-
- /**
- * Creates a new instance of the GrepTool CLI wrapper
- * @param rootDirectory Root directory to ground this tool in.
- */
- constructor(rootDirectory: string) {
- // Instantiate the core logic from the server package
- const coreLogicInstance = new GrepLogic(rootDirectory);
-
- // Initialize the CLI BaseTool
- super(
- GrepTool.Name,
- 'SearchText', // Define display name here
- 'Searches for a regular expression pattern within the content of files in a specified directory (or current working directory). Can filter files by a glob pattern. Returns the lines containing matches, along with their file paths and line numbers.', // Define description here
- (coreLogicInstance.schema.parameters as Record<string, unknown>) ?? {},
- );
-
- this.coreLogic = coreLogicInstance;
- }
-
- /**
- * Delegates validation to the core logic
- */
- validateToolParams(params: GrepToolParams): string | null {
- return this.coreLogic.validateToolParams(params);
- }
-
- /**
- * Delegates getting description to the core logic
- */
- getDescription(params: GrepToolParams): string {
- return this.coreLogic.getDescription(params);
- }
-
- /**
- * Define confirmation behavior (Grep likely doesn't need confirmation)
- */
- shouldConfirmExecute(
- // eslint-disable-next-line @typescript-eslint/no-unused-vars
- params: GrepToolParams,
- ): Promise<ToolCallConfirmationDetails | false> {
- return Promise.resolve(false);
- }
-
- /**
- * Delegates execution to the core logic
- */
- async execute(params: GrepToolParams): Promise<ToolResult> {
- return this.coreLogic.execute(params);
- }
-
- // Removed private methods (resolveAndValidatePath, performGrepSearch, etc.)
- // as they are now part of GrepLogic in the server package.
-}