From dd533c2af7018ae22cd2f1f3850a2461517534ad Mon Sep 17 00:00:00 2001 From: hannesrudolph Date: Thu, 17 Jul 2025 13:19:26 -0600 Subject: [PATCH] refactor: convert showSuggestions from state to constant - Addresses code review feedback about unused state variable - showSuggestions is always true and never changes, so it doesn't need to be state - All tests pass and no linting issues --- webview-ui/src/components/chat/CommandExecution.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/webview-ui/src/components/chat/CommandExecution.tsx b/webview-ui/src/components/chat/CommandExecution.tsx index 560424546d..d8451c2ae6 100644 --- a/webview-ui/src/components/chat/CommandExecution.tsx +++ b/webview-ui/src/components/chat/CommandExecution.tsx @@ -50,7 +50,7 @@ export const CommandExecution = ({ executionId, text, icon, title }: CommandExec const [isExpanded, setIsExpanded] = useState(terminalShellIntegrationDisabled) const [streamingOutput, setStreamingOutput] = useState("") const [status, setStatus] = useState(null) - const [showSuggestions] = useState(true) + const showSuggestions = true // The command's output can either come from the text associated with the // task message (this is the case for completed commands) or from the