refactor: clean up unused code and fix linting issues in prompt history

- Remove unused CursorPositionState interface from ChatTextArea
- Remove unused destructured variables from usePromptHistory hook
- Fix missing dependency in useEffect dependency array
- Rename unused parameter with underscore prefix

Related to #4139

Author: hannesrudolph

webview-ui/src/components/chat/ChatTextArea.tsx

@@ -46,11 +46,6 @@ interface ChatTextAreaProps {
 	modeShortcutText: string
 }
 
-interface CursorPositionState {
-	value: string
-	afterRender?: "SET_CURSOR_FIRST_LINE" | "SET_CURSOR_LAST_LINE" | "SET_CURSOR_START"
-}
-
 const ChatTextArea = forwardRef<HTMLTextAreaElement, ChatTextAreaProps>(
 	(
 		{
@@ -162,11 +157,6 @@ const ChatTextArea = forwardRef<HTMLTextAreaElement, ChatTextAreaProps>(
 
 		// Use custom hook for prompt history navigation
 		const {
-			historyIndex,
-			setHistoryIndex,
-			tempInput,
-			setTempInput,
-			promptHistory,
 			inputValueWithCursor,
 			setInputValueWithCursor,
 			handleHistoryNavigation,
@@ -491,7 +481,7 @@ const ChatTextArea = forwardRef<HTMLTextAreaElement, ChatTextAreaProps>(
 			}
 
 			setInputValueWithCursor({ value: inputValueWithCursor.value })
-		}, [inputValueWithCursor])
+		}, [inputValueWithCursor, setInputValueWithCursor])
 
 		// Ref to store the search timeout.
 		const searchTimeoutRef = useRef<NodeJS.Timeout | null>(null)

webview-ui/src/components/chat/hooks/usePromptHistory.ts

@@ -79,7 +79,7 @@ export const usePromptHistory = ({
 
 	// Reset history navigation when user types (but not when we're setting it programmatically)
 	const resetOnInputChange = useCallback(
-		(newValue: string) => {
+		(_newValue: string) => {
 			if (historyIndex !== -1) {
 				setHistoryIndex(-1)
 				setTempInput("")