MediaMetz/gemini-cli
1
0
Fork 0
mirror of https://github.com/google-gemini/gemini-cli.git synced 2026-03-13 23:51:16 -07:00
Code Issues Packages Projects Releases Wiki Activity

refactor(core): centralize core tool definitions and support model-specific schemas

Browse Source
This commit is contained in:
Aishanee Shah
2026-02-09 03:52:43 +00:00
parent 991551b3f5
commit 3f051bc58e
8 changed files with 218 additions and 123 deletions
Download Patch File Download Diff File Expand all files Collapse all files

175
packages/core/src/tools/definitions/coreTools.ts
View File

@@ -4,11 +4,17 @@
* SPDX-License-Identifier: Apache-2.0
*/
import { Type } from '@google/genai';
import type { ToolDefinition } from './types.js';
import { READ_FILE_TOOL_NAME, SHELL_TOOL_NAME } from '../tool-names.js';
import * as os from 'node:os';
// Centralized tool names to avoid circular dependencies
export const GLOB_TOOL_NAME = 'glob';
export const GREP_TOOL_NAME = 'grep_search';
export const LS_TOOL_NAME = 'list_directory';
export const READ_FILE_TOOL_NAME = 'read_file';
export const SHELL_TOOL_NAME = 'run_shell_command';
export const WRITE_FILE_TOOL_NAME = 'write_file';
// ============================================================================
// READ_FILE TOOL
// ============================================================================
@@ -18,21 +24,21 @@ export const READ_FILE_DEFINITION: ToolDefinition = {
name: READ_FILE_TOOL_NAME,
description: `Reads and returns the content of a specified file. If the file is large, the content will be truncated. The tool's response will clearly indicate if truncation has occurred and will provide details on how to read more of the file using the 'offset' and 'limit' parameters. Handles text, images (PNG, JPG, GIF, WEBP, SVG, BMP), audio files (MP3, WAV, AIFF, AAC, OGG, FLAC), and PDF files. For text files, it can read specific line ranges.`,
parametersJsonSchema: {
type: Type.OBJECT,
type: 'object',
properties: {
file_path: {
description: 'The path to the file to read.',
type: Type.STRING,
type: 'string',
},
offset: {
description:
"Optional: For text files, the 0-based line number to start reading from. Requires 'limit' to be set. Use for paginating through large files.",
type: Type.NUMBER,
type: 'number',
},
limit: {
description:
"Optional: For text files, maximum number of lines to read. Use with 'offset' to paginate through large files. If omitted, reads the entire file (if feasible, up to a default limit).",
type: Type.NUMBER,
type: 'number',
},
},
required: ['file_path'],
@@ -40,6 +46,153 @@ export const READ_FILE_DEFINITION: ToolDefinition = {
},
};
// ============================================================================
// WRITE_FILE TOOL
// ============================================================================
export const WRITE_FILE_DEFINITION: ToolDefinition = {
base: {
name: WRITE_FILE_TOOL_NAME,
description: `Writes content to a specified file in the local filesystem.
The user has the ability to modify \`content\`. If modified, this will be stated in the response.`,
parametersJsonSchema: {
type: 'object',
properties: {
file_path: {
description: 'The path to the file to write to.',
type: 'string',
},
content: {
description: 'The content to write to the file.',
type: 'string',
},
},
required: ['file_path', 'content'],
},
},
};
// ============================================================================
// GREP TOOL
// ============================================================================
export const GREP_DEFINITION: ToolDefinition = {
base: {
name: GREP_TOOL_NAME,
description:
'Searches for a regular expression pattern within file contents. Max 100 matches.',
parametersJsonSchema: {
type: 'object',
properties: {
pattern: {
description: `The regular expression (regex) pattern to search for within file contents (e.g., 'function\\s+myFunction', 'import\\s+\\{.*\\}\\s+from\\s+.*').`,
type: 'string',
},
dir_path: {
description:
'Optional: The absolute path to the directory to search within. If omitted, searches the current working directory.',
type: 'string',
},
include: {
description: `Optional: A glob pattern to filter which files are searched (e.g., '*.js', '*.{ts,tsx}', 'src/**'). If omitted, searches all files (respecting potential global ignores).`,
type: 'string',
},
},
required: ['pattern'],
},
},
};
// ============================================================================
// GLOB TOOL
// ============================================================================
export const GLOB_DEFINITION: ToolDefinition = {
base: {
name: GLOB_TOOL_NAME,
description:
'Efficiently finds files matching specific glob patterns (e.g., `src/**/*.ts`, `**/*.md`), returning absolute paths sorted by modification time (newest first). Ideal for quickly locating files based on their name or path structure, especially in large codebases.',
parametersJsonSchema: {
type: 'object',
properties: {
pattern: {
description:
"The glob pattern to match against (e.g., '**/*.py', 'docs/*.md').",
type: 'string',
},
dir_path: {
description:
'Optional: The absolute path to the directory to search within. If omitted, searches the root directory.',
type: 'string',
},
case_sensitive: {
description:
'Optional: Whether the search should be case-sensitive. Defaults to false.',
type: 'boolean',
},
respect_git_ignore: {
description:
'Optional: Whether to respect .gitignore patterns when finding files. Only available in git repositories. Defaults to true.',
type: 'boolean',
},
respect_gemini_ignore: {
description:
'Optional: Whether to respect .geminiignore patterns when finding files. Defaults to true.',
type: 'boolean',
},
},
required: ['pattern'],
},
},
};
// ============================================================================
// LS TOOL
// ============================================================================
export const LS_DEFINITION: ToolDefinition = {
base: {
name: LS_TOOL_NAME,
description:
'Lists the names of files and subdirectories directly within a specified directory path. Can optionally ignore entries matching provided glob patterns.',
parametersJsonSchema: {
type: 'object',
properties: {
dir_path: {
description: 'The path to the directory to list',
type: 'string',
},
ignore: {
description: 'List of glob patterns to ignore',
items: {
type: 'string',
},
type: 'array',
},
file_filtering_options: {
description:
'Optional: Whether to respect ignore patterns from .gitignore or .geminiignore',
type: 'object',
properties: {
respect_git_ignore: {
description:
'Optional: Whether to respect .gitignore patterns when listing files. Only available in git repositories. Defaults to true.',
type: 'boolean',
},
respect_gemini_ignore: {
description:
'Optional: Whether to respect .geminiignore patterns when listing files. Defaults to true.',
type: 'boolean',
},
},
},
},
required: ['dir_path'],
},
},
};
// ============================================================================
// SHELL TOOL
// ============================================================================
@@ -95,24 +248,24 @@ export function getShellDefinition(
name: SHELL_TOOL_NAME,
description: getShellToolDescription(enableInteractiveShell),
parametersJsonSchema: {
type: Type.OBJECT,
type: 'object',
properties: {
command: {
type: Type.STRING,
type: 'string',
description: getCommandDescription(),
},
description: {
type: Type.STRING,
type: 'string',
description:
'Brief description of the command for the user. Be specific and concise. Ideally a single sentence. Can be up to 3 sentences for clarity. No line breaks.',
},
dir_path: {
type: Type.STRING,
type: 'string',
description:
'(OPTIONAL) The path of the directory to run the command in. If not provided, the project root directory is used. Must be a directory within the workspace and must already exist.',
},
is_background: {
type: Type.BOOLEAN,
type: 'boolean',
description:
'Set to true if this command should be run in the background (e.g. for long-running servers or watchers). The command will be started, allowed to run for a brief moment to check for immediate errors, and then moved to the background.',
},

40
packages/core/src/tools/glob.ts
View File

@@ -17,6 +17,8 @@ import { ToolErrorType } from './tool-error.js';
import { GLOB_TOOL_NAME } from './tool-names.js';
import { getErrorMessage } from '../utils/errors.js';
import { debugLogger } from '../utils/debugLogger.js';
import { GLOB_DEFINITION } from './definitions/coreTools.js';
import { resolveToolDeclaration } from './definitions/resolver.js';
// Subset of 'Path' interface provided by 'glob' that we can implement for testing
export interface GlobPath {
@@ -268,39 +270,9 @@ export class GlobTool extends BaseDeclarativeTool<GlobToolParams, ToolResult> {
super(
GlobTool.Name,
'FindFiles',
'Efficiently finds files matching specific glob patterns (e.g., `src/**/*.ts`, `**/*.md`), returning absolute paths sorted by modification time (newest first). Ideal for quickly locating files based on their name or path structure, especially in large codebases.',
GLOB_DEFINITION.base.description!,
Kind.Search,
{
properties: {
pattern: {
description:
"The glob pattern to match against (e.g., '**/*.py', 'docs/*.md').",
type: 'string',
},
dir_path: {
description:
'Optional: The absolute path to the directory to search within. If omitted, searches the root directory.',
type: 'string',
},
case_sensitive: {
description:
'Optional: Whether the search should be case-sensitive. Defaults to false.',
type: 'boolean',
},
respect_git_ignore: {
description:
'Optional: Whether to respect .gitignore patterns when finding files. Only available in git repositories. Defaults to true.',
type: 'boolean',
},
respect_gemini_ignore: {
description:
'Optional: Whether to respect .geminiignore patterns when finding files. Defaults to true.',
type: 'boolean',
},
},
required: ['pattern'],
type: 'object',
},
GLOB_DEFINITION.base.parametersJsonSchema,
messageBus,
true,
false,
@@ -360,4 +332,8 @@ export class GlobTool extends BaseDeclarativeTool<GlobToolParams, ToolResult> {
_toolDisplayName,
);
}
override getSchema(modelId?: string) {
return resolveToolDeclaration(GLOB_DEFINITION, modelId);
}
}

28
packages/core/src/tools/grep.ts
View File

@@ -25,6 +25,8 @@ import type { FileExclusions } from '../utils/ignorePatterns.js';
import { ToolErrorType } from './tool-error.js';
import { GREP_TOOL_NAME } from './tool-names.js';
import { debugLogger } from '../utils/debugLogger.js';
import { GREP_DEFINITION } from './definitions/coreTools.js';
import { resolveToolDeclaration } from './definitions/resolver.js';
// --- Interfaces ---
@@ -576,27 +578,9 @@ export class GrepTool extends BaseDeclarativeTool<GrepToolParams, ToolResult> {
super(
GrepTool.Name,
'SearchText',
'Searches for a regular expression pattern within file contents. Max 100 matches.',
GREP_DEFINITION.base.description!,
Kind.Search,
{
properties: {
pattern: {
description: `The regular expression (regex) pattern to search for within file contents (e.g., 'function\\s+myFunction', 'import\\s+\\{.*\\}\\s+from\\s+.*').`,
type: 'string',
},
dir_path: {
description:
'Optional: The absolute path to the directory to search within. If omitted, searches the current working directory.',
type: 'string',
},
include: {
description: `Optional: A glob pattern to filter which files are searched (e.g., '*.js', '*.{ts,tsx}', 'src/**'). If omitted, searches all files (respecting potential global ignores).`,
type: 'string',
},
},
required: ['pattern'],
type: 'object',
},
GREP_DEFINITION.base.parametersJsonSchema,
messageBus,
true,
false,
@@ -659,4 +643,8 @@ export class GrepTool extends BaseDeclarativeTool<GrepToolParams, ToolResult> {
_toolDisplayName,
);
}
override getSchema(modelId?: string) {
return resolveToolDeclaration(GREP_DEFINITION, modelId);
}
}

43
packages/core/src/tools/ls.ts
View File

@@ -15,6 +15,8 @@ import { DEFAULT_FILE_FILTERING_OPTIONS } from '../config/constants.js';
import { ToolErrorType } from './tool-error.js';
import { LS_TOOL_NAME } from './tool-names.js';
import { debugLogger } from '../utils/debugLogger.js';
import { LS_DEFINITION } from './definitions/coreTools.js';
import { resolveToolDeclaration } from './definitions/resolver.js';
/**
* Parameters for the LS tool
@@ -277,42 +279,9 @@ export class LSTool extends BaseDeclarativeTool<LSToolParams, ToolResult> {
super(
LSTool.Name,
'ReadFolder',
'Lists the names of files and subdirectories directly within a specified directory path. Can optionally ignore entries matching provided glob patterns.',
LS_DEFINITION.base.description!,
Kind.Search,
{
properties: {
dir_path: {
description: 'The path to the directory to list',
type: 'string',
},
ignore: {
description: 'List of glob patterns to ignore',
items: {
type: 'string',
},
type: 'array',
},
file_filtering_options: {
description:
'Optional: Whether to respect ignore patterns from .gitignore or .geminiignore',
type: 'object',
properties: {
respect_git_ignore: {
description:
'Optional: Whether to respect .gitignore patterns when listing files. Only available in git repositories. Defaults to true.',
type: 'boolean',
},
respect_gemini_ignore: {
description:
'Optional: Whether to respect .geminiignore patterns when listing files. Defaults to true.',
type: 'boolean',
},
},
},
},
required: ['dir_path'],
type: 'object',
},
LS_DEFINITION.base.parametersJsonSchema,
messageBus,
true,
false,
@@ -348,4 +317,8 @@ export class LSTool extends BaseDeclarativeTool<LSToolParams, ToolResult> {
_toolDisplayName,
);
}
override getSchema(modelId?: string) {
return resolveToolDeclaration(LS_DEFINITION, modelId);
}
}

2
packages/core/src/tools/read-file.ts
View File

@@ -173,7 +173,7 @@ export class ReadFileTool extends BaseDeclarativeTool<
'ReadFile',
READ_FILE_DEFINITION.base.description!,
Kind.Read,
READ_FILE_DEFINITION.base.parameters!,
READ_FILE_DEFINITION.base.parametersJsonSchema,
messageBus,
true,
false,

2
packages/core/src/tools/shell.ts
View File

@@ -472,7 +472,7 @@ export class ShellTool extends BaseDeclarativeTool<
'Shell',
definition.base.description!,
Kind.Execute,
definition.base.parameters!,
definition.base.parametersJsonSchema,
messageBus,
false, // output is not markdown
true, // output can be updated

26
packages/core/src/tools/tool-names.ts
View File

@@ -4,21 +4,35 @@
* SPDX-License-Identifier: Apache-2.0
*/
import {
GLOB_TOOL_NAME,
GREP_TOOL_NAME,
LS_TOOL_NAME,
READ_FILE_TOOL_NAME,
SHELL_TOOL_NAME,
WRITE_FILE_TOOL_NAME,
} from './definitions/coreTools.js';
// Centralized constants for tool names.
// This prevents circular dependencies that can occur when other modules (like agents)
// need to reference a tool's name without importing the tool's implementation.
export const GLOB_TOOL_NAME = 'glob';
export {
GLOB_TOOL_NAME,
GREP_TOOL_NAME,
LS_TOOL_NAME,
READ_FILE_TOOL_NAME,
SHELL_TOOL_NAME,
WRITE_FILE_TOOL_NAME,
};
export const WRITE_TODOS_TOOL_NAME = 'write_todos';
export const WRITE_FILE_TOOL_NAME = 'write_file';
export const WEB_SEARCH_TOOL_NAME = 'google_web_search';
export const WEB_FETCH_TOOL_NAME = 'web_fetch';
export const EDIT_TOOL_NAME = 'replace';
export const SHELL_TOOL_NAME = 'run_shell_command';
export const GREP_TOOL_NAME = 'grep_search';
export const READ_MANY_FILES_TOOL_NAME = 'read_many_files';
export const READ_FILE_TOOL_NAME = 'read_file';
export const LS_TOOL_NAME = 'list_directory';
export const LS_TOOL_NAME_LEGACY = 'list_directory'; // Just to be safe if anything used the old exported name directly
export const MEMORY_TOOL_NAME = 'save_memory';
export const GET_INTERNAL_DOCS_TOOL_NAME = 'get_internal_docs';
export const ACTIVATE_SKILL_TOOL_NAME = 'activate_skill';

25
packages/core/src/tools/write-file.ts
View File

@@ -48,6 +48,8 @@ import { getSpecificMimeType } from '../utils/fileUtils.js';
import { getLanguageFromFilePath } from '../utils/language-detection.js';
import type { MessageBus } from '../confirmation-bus/message-bus.js';
import { debugLogger } from '../utils/debugLogger.js';
import { WRITE_FILE_DEFINITION } from './definitions/coreTools.js';
import { resolveToolDeclaration } from './definitions/resolver.js';
/**
* Parameters for the WriteFile tool
@@ -445,24 +447,9 @@ export class WriteFileTool
super(
WriteFileTool.Name,
'WriteFile',
`Writes content to a specified file in the local filesystem.
The user has the ability to modify \`content\`. If modified, this will be stated in the response.`,
WRITE_FILE_DEFINITION.base.description!,
Kind.Edit,
{
properties: {
file_path: {
description: 'The path to the file to write to.',
type: 'string',
},
content: {
description: 'The content to write to the file.',
type: 'string',
},
},
required: ['file_path', 'content'],
type: 'object',
},
WRITE_FILE_DEFINITION.base.parametersJsonSchema,
messageBus,
true,
false,
@@ -514,6 +501,10 @@ export class WriteFileTool
);
}
override getSchema(modelId?: string) {
return resolveToolDeclaration(WRITE_FILE_DEFINITION, modelId);
}
getModifyContext(
abortSignal: AbortSignal,
): ModifyContext<WriteFileToolParams> {