fix: resolve linting and TypeScript compilation errors

- Remove unused imports (React, useState, useEffect, waitFor)
- Remove unused clearOnSubmit parameter from useAutosaveDraft
- Prefix hasInitialDraft with _ to indicate intentionally unused
- Fix useCallback dependency arrays in ChatView
- Fix TypeScript error in ChatView.autosave.test.tsx

Author: jsboige

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

@@ -177,11 +177,10 @@ const ChatViewComponent: React.ForwardRefRenderFunction<ChatViewRef, ChatViewPro
 		draftContent: inputValue,
 		updateDraft: setInputValue,
 		clearDraft,
-		hasInitialDraft,
+		hasInitialDraft: _hasInitialDraft,
 	} = useAutosaveDraft({
 		key: currentTaskItem?.id || "default",
 		debounceMs: 100,
-		clearOnSubmit: true,
 	})
 	const inputValueRef = useRef(inputValue)
 	const textAreaRef = useRef<HTMLTextAreaElement>(null)
@@ -593,7 +592,7 @@ const ChatViewComponent: React.ForwardRefRenderFunction<ChatViewRef, ChatViewPro
 		// setPrimaryButtonText(undefined)
 		// setSecondaryButtonText(undefined)
 		disableAutoScrollRef.current = false
-	}, [])
+	}, [clearDraft])
 
 	/**
 	 * Handles sending messages to the extension
@@ -665,7 +664,7 @@ const ChatViewComponent: React.ForwardRefRenderFunction<ChatViewRef, ChatViewPro
 				handleChatReset()
 			}
 		},
-		[handleChatReset, markFollowUpAsAnswered, sendingDisabled, clearDraft], // messagesRef and clineAskRef are stable
+		[handleChatReset, markFollowUpAsAnswered, sendingDisabled], // messagesRef and clineAskRef are stable
 	)
 
 	const handleSetChatBoxMessage = useCallback(
@@ -732,7 +731,7 @@ const ChatViewComponent: React.ForwardRefRenderFunction<ChatViewRef, ChatViewPro
 			setClineAsk(undefined)
 			setEnableButtons(false)
 		},
-		[clineAsk, startNewTask],
+		[clineAsk, startNewTask, clearDraft],
 	)
 
 	const handleSecondaryButtonClick = useCallback(
@@ -782,7 +781,7 @@ const ChatViewComponent: React.ForwardRefRenderFunction<ChatViewRef, ChatViewPro
 			setClineAsk(undefined)
 			setEnableButtons(false)
 		},
-		[clineAsk, startNewTask, isStreaming],
+		[clineAsk, startNewTask, isStreaming, clearDraft],
 	)
 
 	const { info: model } = useSelectedModel(apiConfiguration)
@@ -1504,7 +1503,15 @@ const ChatViewComponent: React.ForwardRefRenderFunction<ChatViewRef, ChatViewPro
 				setInputValue(preservedInput)
 			}
 		},
-		[handleSendMessage, setInputValue, switchToMode, alwaysAllowModeSwitch, clineAsk, markFollowUpAsAnswered],
+		[
+			handleSendMessage,
+			setInputValue,
+			switchToMode,
+			alwaysAllowModeSwitch,
+			clineAsk,
+			markFollowUpAsAnswered,
+			inputValue,
+		],
 	)
 
 	const handleBatchFileResponse = useCallback((response: { [key: string]: boolean }) => {

webview-ui/src/components/chat/__tests__/ChatView.autosave.test.tsx

@@ -95,7 +95,7 @@ describe("ChatView Autosave Integration", () => {
 		hideAnnouncement: vi.fn(),
 	}
 
-	const createMockState = (overrides: any = {}) => ({
+	const _createMockState = (overrides: any = {}) => ({
 		version: "1.0.0",
 		clineMessages: [],
 		taskHistory: [],
@@ -109,12 +109,10 @@ describe("ChatView Autosave Integration", () => {
 		...overrides,
 	})
 
-	const renderChatView = (stateOverrides: any = {}) => {
-		const mockState = createMockState(stateOverrides)
-
+	const renderChatView = (_stateOverrides: any = {}) => {
 		return render(
 			<QueryClientProvider client={queryClient}>
-				<ExtensionStateContextProvider initialState={mockState}>
+				<ExtensionStateContextProvider>
 					<ChatView {...defaultProps} />
 				</ExtensionStateContextProvider>
 			</QueryClientProvider>,

webview-ui/src/components/chat/__tests__/ChatView.raceCondition.spec.tsx

@@ -17,8 +17,8 @@
  * - Let the autosave provide recovery if something goes wrong
  */
 
-import React, { useState, useCallback, useEffect } from "react"
-import { render, screen, fireEvent, waitFor, act } from "@testing-library/react"
+import React, { useCallback } from "react"
+import { render, screen, fireEvent, act } from "@testing-library/react"
 import { vi, describe, it, expect, beforeEach, afterEach } from "vitest"
 import { useAutosaveDraft } from "../../../hooks/useAutosaveDraft"
 
@@ -47,7 +47,6 @@ const MessageEditorWithRaceCondition: React.FC<MessageEditorProps> = ({
 	const { draftContent, updateDraft, clearDraft } = useAutosaveDraft({
 		key: taskId,
 		debounceMs: 100,
-		clearOnSubmit: true,
 	})
 
 	const handleSend = useCallback(() => {

webview-ui/src/hooks/__tests__/useAutosaveDraft.test.tsx

@@ -1,4 +1,3 @@
-import React from "react"
 import { renderHook, act } from "@testing-library/react"
 import { vi, describe, it, expect, beforeEach, afterEach } from "vitest"
 import { useAutosaveDraft } from "../useAutosaveDraft"

webview-ui/src/hooks/useAutosaveDraft.ts

@@ -1,4 +1,4 @@
-import React, { useState, useEffect, useCallback, useRef } from "react"
+import { useState, useEffect, useCallback, useRef } from "react"
 
 /**
  * Configuration options for the useAutosaveDraft hook
@@ -8,8 +8,6 @@ export interface UseAutosaveDraftOptions {
 	key: string
 	/** Debounce delay in milliseconds before saving to localStorage (default: 100ms for fast responsiveness) */
 	debounceMs?: number
-	/** Whether to automatically clear the draft after submission */
-	clearOnSubmit?: boolean
 	/** Prefix for localStorage keys to avoid collisions */
 	storagePrefix?: string
 }
@@ -62,7 +60,7 @@ export interface UseAutosaveDraftReturn {
  * ```
  */
 export const useAutosaveDraft = (options: UseAutosaveDraftOptions): UseAutosaveDraftReturn => {
-	const { key, debounceMs = 100, clearOnSubmit = true, storagePrefix = "roo-draft" } = options
+	const { key, debounceMs = 100, storagePrefix = "roo-draft" } = options
 
 	// Local state for the hook
 	const [draftContent, setDraftContent] = useState<string>("")