From baf39042c8631d53dfac4e57148404749cde14b3 Mon Sep 17 00:00:00 2001 From: Jaana Dogan Date: Mon, 21 Apr 2025 12:59:31 -0700 Subject: Remove duplicate CLI tools module, remove the global tool registry (#89) --- packages/cli/src/ui/components/messages/ToolMessage.tsx | 2 +- packages/cli/src/ui/hooks/useGeminiStream.ts | 2 +- packages/cli/src/ui/types.ts | 6 ++++-- 3 files changed, 6 insertions(+), 4 deletions(-) (limited to 'packages/cli/src/ui') diff --git a/packages/cli/src/ui/components/messages/ToolMessage.tsx b/packages/cli/src/ui/components/messages/ToolMessage.tsx index f21e1d28..53f31db2 100644 --- a/packages/cli/src/ui/components/messages/ToolMessage.tsx +++ b/packages/cli/src/ui/components/messages/ToolMessage.tsx @@ -9,7 +9,7 @@ import { Box, Text } from 'ink'; import Spinner from 'ink-spinner'; import { IndividualToolCallDisplay, ToolCallStatus } from '../../types.js'; import { DiffRenderer } from './DiffRenderer.js'; -import { FileDiff, ToolResultDisplay } from '../../../tools/tools.js'; +import { FileDiff, ToolResultDisplay } from '@gemini-code/server'; import { Colors } from '../../colors.js'; export const ToolMessage: React.FC = ({ diff --git a/packages/cli/src/ui/hooks/useGeminiStream.ts b/packages/cli/src/ui/hooks/useGeminiStream.ts index 2728c394..ffdd9967 100644 --- a/packages/cli/src/ui/hooks/useGeminiStream.ts +++ b/packages/cli/src/ui/hooks/useGeminiStream.ts @@ -35,7 +35,6 @@ import { IndividualToolCallDisplay, ToolCallStatus, } from '../types.js'; -import { toolRegistry } from '../../tools/tool-registry.js'; const addHistoryItem = ( setHistory: React.Dispatch>, @@ -53,6 +52,7 @@ export const useGeminiStream = ( setHistory: React.Dispatch>, config: Config, ) => { + const toolRegistry = config.getToolRegistry(); const [streamingState, setStreamingState] = useState( StreamingState.Idle, ); diff --git a/packages/cli/src/ui/types.ts b/packages/cli/src/ui/types.ts index 41b2a944..fe135909 100644 --- a/packages/cli/src/ui/types.ts +++ b/packages/cli/src/ui/types.ts @@ -4,8 +4,10 @@ * SPDX-License-Identifier: Apache-2.0 */ -import { ToolCallConfirmationDetails } from '@gemini-code/server'; -import { ToolResultDisplay } from '../tools/tools.js'; +import { + ToolCallConfirmationDetails, + ToolResultDisplay, +} from '@gemini-code/server'; // Only defining the state enum needed by the UI export enum StreamingState { -- cgit v1.2.3