From 482aeaff102f1f97f67bed04442dbd23a6424f2d Mon Sep 17 00:00:00 2001 From: Evan Senter Date: Fri, 18 Apr 2025 21:55:02 +0100 Subject: Warn if `npm run start` is out of date. (#20) * Adding some wiring to allow the Ink app to warn if there are local development changes that haven't been captured in the recent build of the Gemini CLI. * Adding a new useAppEffects.ts file that wores some useEffect handlers in. * Updating package-lock.json to resolve `npm ci` issues. * Updating package-lock.json and package.json to resolve `npm ci` issues. --- packages/cli/src/ui/App.tsx | 47 ++++++++++++++++++++++++++++----------------- 1 file changed, 29 insertions(+), 18 deletions(-) (limited to 'packages/cli/src/ui/App.tsx') diff --git a/packages/cli/src/ui/App.tsx b/packages/cli/src/ui/App.tsx index 1bc0f6c6..7eae3727 100644 --- a/packages/cli/src/ui/App.tsx +++ b/packages/cli/src/ui/App.tsx @@ -1,5 +1,8 @@ import React, { useState, useEffect } from 'react'; import { Box, Text } from 'ink'; +import fs from 'fs'; +import path from 'path'; +import os from 'os'; import type { HistoryItem } from './types.js'; import { useGeminiStream } from './hooks/useGeminiStream.js'; import { useLoadingIndicator } from './hooks/useLoadingIndicator.js'; @@ -11,6 +14,12 @@ import InputPrompt from './components/InputPrompt.js'; import Footer from './components/Footer.js'; import { StreamingState } from '../core/gemini-stream.js'; import { PartListUnion } from '@google/genai'; +import { + useStartupWarnings, + useInitializationErrorEffect, +} from './hooks/useAppEffects.js'; + +const warningsFilePath = path.join(os.tmpdir(), 'gemini-code-cli-warnings.txt'); interface AppProps { directory: string; @@ -19,11 +28,15 @@ interface AppProps { const App = ({ directory }: AppProps) => { const [query, setQuery] = useState(''); const [history, setHistory] = useState([]); + const [startupWarnings, setStartupWarnings] = useState([]); const { streamingState, submitQuery, initError } = useGeminiStream(setHistory); const { elapsedTime, currentLoadingPhrase } = useLoadingIndicator(streamingState); + useStartupWarnings(setStartupWarnings); + useInitializationErrorEffect(initError, history, setHistory); + const handleInputSubmit = (value: PartListUnion) => { submitQuery(value) .then(() => { @@ -34,24 +47,6 @@ const App = ({ directory }: AppProps) => { }); }; - useEffect(() => { - if ( - initError && - !history.some( - (item) => item.type === 'error' && item.text?.includes(initError), - ) - ) { - setHistory((prev) => [ - ...prev, - { - id: Date.now(), - type: 'error', - text: `Initialization Error: ${initError}. Please check API key and configuration.`, - } as HistoryItem, - ]); - } - }, [initError, history]); - const isWaitingForToolConfirmation = history.some( (item) => item.type === 'tool_group' && @@ -63,6 +58,22 @@ const App = ({ directory }: AppProps) => {
+ {startupWarnings.length > 0 && ( + + {startupWarnings.map((warning, index) => ( + + {warning} + + ))} + + )} + {initError && -- cgit v1.2.3