From 39adaaff119dc1f983ffc260ec3093611c66b8a7 Mon Sep 17 00:00:00 2001 From: yiliang114 <1204183885@qq.com> Date: Sun, 23 Nov 2025 15:17:40 +0800 Subject: [PATCH] refactor(vscode-ide-companion): minor adjustments to SaveSessionDialog and SessionManager components --- .../src/webview/components/SaveSessionDialog.tsx | 4 +++- .../src/webview/components/SessionManager.tsx | 10 +++++----- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/packages/vscode-ide-companion/src/webview/components/SaveSessionDialog.tsx b/packages/vscode-ide-companion/src/webview/components/SaveSessionDialog.tsx index df66386c..9198edb2 100644 --- a/packages/vscode-ide-companion/src/webview/components/SaveSessionDialog.tsx +++ b/packages/vscode-ide-companion/src/webview/components/SaveSessionDialog.tsx @@ -92,7 +92,9 @@ export const SaveSessionDialog: React.FC = ({ value={tag} onChange={(e) => { setTag(e.target.value); - if (error) setError(''); + if (error) { + setError(''); + } }} placeholder="e.g., project-planning, bug-fix, research" className={error ? 'error' : ''} diff --git a/packages/vscode-ide-companion/src/webview/components/SessionManager.tsx b/packages/vscode-ide-companion/src/webview/components/SessionManager.tsx index dcb790fd..57fe0bc2 100644 --- a/packages/vscode-ide-companion/src/webview/components/SessionManager.tsx +++ b/packages/vscode-ide-companion/src/webview/components/SessionManager.tsx @@ -32,11 +32,6 @@ export const SessionManager: React.FC = ({ const [isLoading, setIsLoading] = useState(false); const [searchQuery, setSearchQuery] = useState(''); - // Load sessions when component mounts - useEffect(() => { - loadSessions(); - }, [loadSessions]); - const loadSessions = React.useCallback(() => { setIsLoading(true); vscode.postMessage({ @@ -45,6 +40,11 @@ export const SessionManager: React.FC = ({ }); }, [vscode]); + // Load sessions when component mounts + useEffect(() => { + loadSessions(); + }, [loadSessions]); + // Listen for session list updates useEffect(() => { const handleMessage = (event: MessageEvent) => {